From 3025c3ededf955fa7f1da7bd7124346c1693f9f2 Mon Sep 17 00:00:00 2001 From: Juergen Ributzka Date: Wed, 12 Jan 2022 14:01:59 -0800 Subject: Replace PlatformKind with PlatformType. The PlatformKind/PlatformType enums contain the same information, which requires them to be kept in-sync. This commit changes over to PlatformType as the sole source of truth, which allows the removal of the redundant PlatformKind. The majority of the changes were in LLD and TextAPI. Reviewed By: cishida Differential Revision: https://reviews.llvm.org/D117163 --- llvm/lib/TextAPI/TextStubCommon.cpp | 53 ++++++++++++++++++------------------- 1 file changed, 26 insertions(+), 27 deletions(-) (limited to 'llvm/lib/TextAPI/TextStubCommon.cpp') diff --git a/llvm/lib/TextAPI/TextStubCommon.cpp b/llvm/lib/TextAPI/TextStubCommon.cpp index c2713b9..29b74f9 100644 --- a/llvm/lib/TextAPI/TextStubCommon.cpp +++ b/llvm/lib/TextAPI/TextStubCommon.cpp @@ -49,8 +49,8 @@ void ScalarTraits::output(const PlatformSet &Values, void *IO, assert((!Ctx || Ctx->FileKind != FileType::Invalid) && "File type is not set in context"); - if (Ctx && Ctx->FileKind == TBD_V3 && Values.count(PlatformKind::macOS) && - Values.count(PlatformKind::macCatalyst)) { + if (Ctx && Ctx->FileKind == TBD_V3 && Values.count(PLATFORM_MACOS) && + Values.count(PLATFORM_MACCATALYST)) { OS << "zippered"; return; } @@ -60,31 +60,31 @@ void ScalarTraits::output(const PlatformSet &Values, void *IO, default: llvm_unreachable("unexpected platform"); break; - case PlatformKind::macOS: + case PLATFORM_MACOS: OS << "macosx"; break; - case PlatformKind::iOSSimulator: + case PLATFORM_IOSSIMULATOR: LLVM_FALLTHROUGH; - case PlatformKind::iOS: + case PLATFORM_IOS: OS << "ios"; break; - case PlatformKind::watchOSSimulator: + case PLATFORM_WATCHOSSIMULATOR: LLVM_FALLTHROUGH; - case PlatformKind::watchOS: + case PLATFORM_WATCHOS: OS << "watchos"; break; - case PlatformKind::tvOSSimulator: + case PLATFORM_TVOSSIMULATOR: LLVM_FALLTHROUGH; - case PlatformKind::tvOS: + case PLATFORM_TVOS: OS << "tvos"; break; - case PlatformKind::bridgeOS: + case PLATFORM_BRIDGEOS: OS << "bridgeos"; break; - case PlatformKind::macCatalyst: + case PLATFORM_MACCATALYST: OS << "iosmac"; break; - case PlatformKind::driverKit: + case PLATFORM_DRIVERKIT: OS << "driverkit"; break; } @@ -98,28 +98,27 @@ StringRef ScalarTraits::input(StringRef Scalar, void *IO, if (Scalar == "zippered") { if (Ctx && Ctx->FileKind == FileType::TBD_V3) { - Values.insert(PlatformKind::macOS); - Values.insert(PlatformKind::macCatalyst); + Values.insert(PLATFORM_MACOS); + Values.insert(PLATFORM_MACCATALYST); return {}; } return "invalid platform"; } - auto Platform = StringSwitch(Scalar) - .Case("unknown", PlatformKind::unknown) - .Case("macosx", PlatformKind::macOS) - .Case("ios", PlatformKind::iOS) - .Case("watchos", PlatformKind::watchOS) - .Case("tvos", PlatformKind::tvOS) - .Case("bridgeos", PlatformKind::bridgeOS) - .Case("iosmac", PlatformKind::macCatalyst) - .Default(PlatformKind::unknown); - - if (Platform == PlatformKind::macCatalyst) + auto Platform = StringSwitch(Scalar) + .Case("macosx", PLATFORM_MACOS) + .Case("ios", PLATFORM_IOS) + .Case("watchos", PLATFORM_WATCHOS) + .Case("tvos", PLATFORM_TVOS) + .Case("bridgeos", PLATFORM_BRIDGEOS) + .Case("iosmac", PLATFORM_MACCATALYST) + .Default(PLATFORM_UNKNOWN); + + if (Platform == PLATFORM_MACCATALYST) if (Ctx && Ctx->FileKind != FileType::TBD_V3) return "invalid platform"; - if (Platform == PlatformKind::unknown) + if (Platform == PLATFORM_UNKNOWN) return "unknown platform"; Values.insert(Platform); @@ -226,7 +225,7 @@ StringRef ScalarTraits::input(StringRef Scalar, void *, UUID &Value) { if (UUID.empty()) return "invalid uuid string pair"; Value.second = std::string(UUID); - Value.first = Target{getArchitectureFromName(Arch), PlatformKind::unknown}; + Value.first = Target{getArchitectureFromName(Arch), PLATFORM_UNKNOWN}; return {}; } -- cgit v1.1