[clang] 0f1361b - [Driver] Fix a warning
Kazu Hirata via cfe-commits
cfe-commits at lists.llvm.org
Wed Aug 7 13:10:39 PDT 2024
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;
}
More information about the cfe-commits
mailing list