Author: Kazu Hirata Date: 2024-08-07T13:10:31-07:00 New Revision: 0f1361baf650641a59aaa1710d7a0b7b02f2e56d
URL: https://github.com/llvm/llvm-project/commit/0f1361baf650641a59aaa1710d7a0b7b02f2e56d DIFF: https://github.com/llvm/llvm-project/commit/0f1361baf650641a59aaa1710d7a0b7b02f2e56d.diff LOG: [Driver] Fix a warning This patch fixes: clang/lib/Driver/ToolChains/Darwin.cpp:2937:3: error: default label in switch which covers all enumeration values [-Werror,-Wcovered-switch-default] Added: Modified: clang/lib/Driver/ToolChains/Darwin.cpp Removed: ################################################################################ diff --git a/clang/lib/Driver/ToolChains/Darwin.cpp b/clang/lib/Driver/ToolChains/Darwin.cpp index fbaa0ff22659f..ee6890d5b98f0 100644 --- a/clang/lib/Driver/ToolChains/Darwin.cpp +++ b/clang/lib/Driver/ToolChains/Darwin.cpp @@ -2927,14 +2927,12 @@ static bool sdkSupportsBuiltinModules( const Darwin::DarwinPlatformKind &TargetPlatform, const Darwin::DarwinEnvironmentKind &TargetEnvironment, const std::optional<DarwinSDKInfo> &SDKInfo) { - switch (TargetEnvironment) { - case Darwin::NativeEnvironment: - case Darwin::Simulator: - case Darwin::MacCatalyst: + if (TargetEnvironment == Darwin::NativeEnvironment || + TargetEnvironment == Darwin::Simulator || + TargetEnvironment == Darwin::MacCatalyst) { // Standard xnu/Mach/Darwin based environments // depend on the SDK version. - break; - default: + } else { // All other environments support builtin modules from the start. return true; } _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits