[TextAPI] Update reader to be supported by lib/Object
Summary:
To be able to use the TextAPI/Reader for tbd file consumption (by libObject)
it gets passed a MemoryBufferRef which isn't castable to MemoryBuffer.
Updated the tests to expect that input as well.
Reviewers: ributzka, steven_wu
Reviewed By: steven_wu
Subscribers: hiraditya, dexonsmith, llvm-commits
Tags: #llvm
Differential Revision: https://reviews.llvm.org/D66147
llvm-svn: 369119
diff --git a/llvm/unittests/TextAPI/TextStubV1Tests.cpp b/llvm/unittests/TextAPI/TextStubV1Tests.cpp
index 54c0ff2..383bc41 100644
--- a/llvm/unittests/TextAPI/TextStubV1Tests.cpp
+++ b/llvm/unittests/TextAPI/TextStubV1Tests.cpp
@@ -82,8 +82,7 @@
" thread-local-symbols: [ _tlv3 ]\n"
"...\n";
- auto Buffer = MemoryBuffer::getMemBuffer(tbd_v1_file1, "Test.tbd");
- auto Result = TextAPIReader::get(std::move(Buffer));
+ auto Result = TextAPIReader::get(MemoryBufferRef(tbd_v1_file1, "Test.tbd"));
EXPECT_TRUE(!!Result);
auto File = std::move(Result.get());
EXPECT_EQ(FileType::TBD_V1, File->getFileType());
@@ -127,8 +126,7 @@
"install-name: Test.dylib\n"
"...\n";
- auto Buffer = MemoryBuffer::getMemBuffer(tbd_v1_file2, "Test.tbd");
- auto Result = TextAPIReader::get(std::move(Buffer));
+ auto Result = TextAPIReader::get(MemoryBufferRef(tbd_v1_file2, "Test.tbd"));
EXPECT_TRUE(!!Result);
auto File = std::move(Result.get());
EXPECT_EQ(FileType::TBD_V1, File->getFileType());
@@ -205,8 +203,8 @@
"install-name: Test.dylib\n"
"...\n";
- auto Buffer = MemoryBuffer::getMemBuffer(tbd_v1_platform_macos, "Test.tbd");
- auto Result = TextAPIReader::get(std::move(Buffer));
+ auto Result =
+ TextAPIReader::get(MemoryBufferRef(tbd_v1_platform_macos, "Test.tbd"));
EXPECT_TRUE(!!Result);
auto File = std::move(Result.get());
EXPECT_EQ(FileType::TBD_V1, File->getFileType());
@@ -220,8 +218,8 @@
"install-name: Test.dylib\n"
"...\n";
- auto Buffer = MemoryBuffer::getMemBuffer(tbd_v1_platform_ios, "Test.tbd");
- auto Result = TextAPIReader::get(std::move(Buffer));
+ auto Result =
+ TextAPIReader::get(MemoryBufferRef(tbd_v1_platform_ios, "Test.tbd"));
EXPECT_TRUE(!!Result);
auto File = std::move(Result.get());
EXPECT_EQ(FileType::TBD_V1, File->getFileType());
@@ -235,8 +233,8 @@
"install-name: Test.dylib\n"
"...\n";
- auto Buffer = MemoryBuffer::getMemBuffer(tbd_v1_platform_watchos, "Test.tbd");
- auto Result = TextAPIReader::get(std::move(Buffer));
+ auto Result =
+ TextAPIReader::get(MemoryBufferRef(tbd_v1_platform_watchos, "Test.tbd"));
EXPECT_TRUE(!!Result);
auto File = std::move(Result.get());
EXPECT_EQ(FileType::TBD_V1, File->getFileType());
@@ -250,8 +248,8 @@
"install-name: Test.dylib\n"
"...\n";
- auto Buffer = MemoryBuffer::getMemBuffer(tbd_v1_platform_tvos, "Test.tbd");
- auto Result = TextAPIReader::get(std::move(Buffer));
+ auto Result =
+ TextAPIReader::get(MemoryBufferRef(tbd_v1_platform_tvos, "Test.tbd"));
EXPECT_TRUE(!!Result);
auto File = std::move(Result.get());
EXPECT_EQ(FileType::TBD_V1, File->getFileType());
@@ -265,9 +263,8 @@
"install-name: Test.dylib\n"
"...\n";
- auto Buffer =
- MemoryBuffer::getMemBuffer(tbd_v1_platform_bridgeos, "Test.tbd");
- auto Result = TextAPIReader::get(std::move(Buffer));
+ auto Result =
+ TextAPIReader::get(MemoryBufferRef(tbd_v1_platform_bridgeos, "Test.tbd"));
EXPECT_TRUE(!!Result);
auto File = std::move(Result.get());
EXPECT_EQ(FileType::TBD_V1, File->getFileType());
@@ -282,8 +279,8 @@
"swift-version: 1.0\n"
"...\n";
- auto Buffer = MemoryBuffer::getMemBuffer(tbd_v1_swift_1_0, "Test.tbd");
- auto Result = TextAPIReader::get(std::move(Buffer));
+ auto Result =
+ TextAPIReader::get(MemoryBufferRef(tbd_v1_swift_1_0, "Test.tbd"));
EXPECT_TRUE(!!Result);
auto File = std::move(Result.get());
EXPECT_EQ(FileType::TBD_V1, File->getFileType());
@@ -298,8 +295,8 @@
"swift-version: 1.1\n"
"...\n";
- auto Buffer = MemoryBuffer::getMemBuffer(tbd_v1_swift_1_1, "Test.tbd");
- auto Result = TextAPIReader::get(std::move(Buffer));
+ auto Result =
+ TextAPIReader::get(MemoryBufferRef(tbd_v1_swift_1_1, "Test.tbd"));
EXPECT_TRUE(!!Result);
auto File = std::move(Result.get());
EXPECT_EQ(FileType::TBD_V1, File->getFileType());
@@ -314,8 +311,8 @@
"swift-version: 2.0\n"
"...\n";
- auto Buffer = MemoryBuffer::getMemBuffer(tbd_v1_swift_2_0, "Test.tbd");
- auto Result = TextAPIReader::get(std::move(Buffer));
+ auto Result =
+ TextAPIReader::get(MemoryBufferRef(tbd_v1_swift_2_0, "Test.tbd"));
EXPECT_TRUE(!!Result);
auto File = std::move(Result.get());
EXPECT_EQ(FileType::TBD_V1, File->getFileType());
@@ -330,8 +327,8 @@
"swift-version: 3.0\n"
"...\n";
- auto Buffer = MemoryBuffer::getMemBuffer(tbd_v1_swift_3_0, "Test.tbd");
- auto Result = TextAPIReader::get(std::move(Buffer));
+ auto Result =
+ TextAPIReader::get(MemoryBufferRef(tbd_v1_swift_3_0, "Test.tbd"));
EXPECT_TRUE(!!Result);
auto File = std::move(Result.get());
EXPECT_EQ(FileType::TBD_V1, File->getFileType());
@@ -346,8 +343,8 @@
"swift-version: 4.0\n"
"...\n";
- auto Buffer = MemoryBuffer::getMemBuffer(tbd_v1_swift_4_0, "Test.tbd");
- auto Result = TextAPIReader::get(std::move(Buffer));
+ auto Result =
+ TextAPIReader::get(MemoryBufferRef(tbd_v1_swift_4_0, "Test.tbd"));
EXPECT_FALSE(!!Result);
auto errorMessage = toString(Result.takeError());
EXPECT_EQ("malformed file\nTest.tbd:5:16: error: invalid Swift ABI "
@@ -363,8 +360,7 @@
"swift-version: 5\n"
"...\n";
- auto Buffer = MemoryBuffer::getMemBuffer(tbd_v1_swift_5, "Test.tbd");
- auto Result = TextAPIReader::get(std::move(Buffer));
+ auto Result = TextAPIReader::get(MemoryBufferRef(tbd_v1_swift_5, "Test.tbd"));
EXPECT_TRUE(!!Result);
auto File = std::move(Result.get());
EXPECT_EQ(FileType::TBD_V1, File->getFileType());
@@ -379,8 +375,8 @@
"swift-version: 99\n"
"...\n";
- auto Buffer = MemoryBuffer::getMemBuffer(tbd_v1_swift_99, "Test.tbd");
- auto Result = TextAPIReader::get(std::move(Buffer));
+ auto Result =
+ TextAPIReader::get(MemoryBufferRef(tbd_v1_swift_99, "Test.tbd"));
EXPECT_TRUE(!!Result);
auto File = std::move(Result.get());
EXPECT_EQ(FileType::TBD_V1, File->getFileType());
@@ -395,9 +391,8 @@
"install-name: Test.dylib\n"
"...\n";
- auto Buffer =
- MemoryBuffer::getMemBuffer(tbd_v1_file_unknown_architecture, "Test.tbd");
- auto Result = TextAPIReader::get(std::move(Buffer));
+ auto Result = TextAPIReader::get(
+ MemoryBufferRef(tbd_v1_file_unknown_architecture, "Test.tbd"));
EXPECT_TRUE(!!Result);
}
@@ -407,9 +402,8 @@
"platform: newOS\n"
"...\n";
- auto Buffer =
- MemoryBuffer::getMemBuffer(tbd_v1_file_unknown_platform, "Test.tbd");
- auto Result = TextAPIReader::get(std::move(Buffer));
+ auto Result = TextAPIReader::get(
+ MemoryBufferRef(tbd_v1_file_unknown_platform, "Test.tbd"));
EXPECT_FALSE(!!Result);
auto errorMessage = toString(Result.takeError());
EXPECT_EQ("malformed file\nTest.tbd:3:11: error: unknown platform\nplatform: "
@@ -423,8 +417,8 @@
"foobar: \"Unsupported key\"\n"
"...\n";
- auto Buffer = MemoryBuffer::getMemBuffer(malformed_file1, "Test.tbd");
- auto Result = TextAPIReader::get(std::move(Buffer));
+ auto Result =
+ TextAPIReader::get(MemoryBufferRef(malformed_file1, "Test.tbd"));
EXPECT_FALSE(!!Result);
auto errorMessage = toString(Result.takeError());
ASSERT_EQ("malformed file\nTest.tbd:2:1: error: missing required key "
@@ -440,8 +434,8 @@
"foobar: \"Unsupported key\"\n"
"...\n";
- auto Buffer = MemoryBuffer::getMemBuffer(malformed_file2, "Test.tbd");
- auto Result = TextAPIReader::get(std::move(Buffer));
+ auto Result =
+ TextAPIReader::get(MemoryBufferRef(malformed_file2, "Test.tbd"));
EXPECT_FALSE(!!Result);
auto errorMessage = toString(Result.takeError());
ASSERT_EQ(