From 6df10711063c91be4702de4d05447de1be7224e7 Mon Sep 17 00:00:00 2001 From: Jonas Devlieghere Date: Mon, 27 Jan 2020 10:17:46 -0800 Subject: [PATCH] [llvm][TextAPI/MachO] Support writing single macCatalyst platform TAPI currently lacks a way to emit the macCatalyst platform. For TBD_V3 is does support zippered frameworks given that both macOS and macCatalyst are part of the PlatformSet. Differential revision: https://reviews.llvm.org/D73325 --- lib/TextAPI/MachO/TextStubCommon.cpp | 3 +++ unittests/TextAPI/TextStubV3Tests.cpp | 4 ---- 2 files changed, 3 insertions(+), 4 deletions(-) diff --git a/lib/TextAPI/MachO/TextStubCommon.cpp b/lib/TextAPI/MachO/TextStubCommon.cpp index 183c5d5a93b..a7fd58fba5f 100644 --- a/lib/TextAPI/MachO/TextStubCommon.cpp +++ b/lib/TextAPI/MachO/TextStubCommon.cpp @@ -74,6 +74,9 @@ void ScalarTraits::output(const PlatformSet &Values, void *IO, case PlatformKind::bridgeOS: OS << "bridgeos"; break; + case PlatformKind::macCatalyst: + OS << "iosmac"; + break; } } diff --git a/unittests/TextAPI/TextStubV3Tests.cpp b/unittests/TextAPI/TextStubV3Tests.cpp index 6d27700758e..4a05920805e 100644 --- a/unittests/TextAPI/TextStubV3Tests.cpp +++ b/unittests/TextAPI/TextStubV3Tests.cpp @@ -329,15 +329,11 @@ TEST(TBDv3, Platform_macCatalyst) { EXPECT_EQ(FileType::TBD_V3, File->getFileType()); EXPECT_EQ(Platform, *File->getPlatforms().begin()); - // It's not currently possible to emit the iomac platform. Enable this once - // that's fixed. -#if 0 SmallString<4096> Buffer; raw_svector_ostream OS(Buffer); auto WriteResult = TextAPIWriter::writeToStream(OS, *File); EXPECT_TRUE(!WriteResult); EXPECT_EQ(stripWhitespace(tbd_v3_platform_iosmac), stripWhitespace(Buffer.c_str())); -#endif } TEST(TBDv3, Platform_zippered) {