Skip to content

Commit 83f831d

Browse files
committed
Fix-forward -Wcovered-switch-default error in #134429
Remove the default case to avoid buildbot error (https://lab.llvm.org/buildbot/#/builders/66/builds/12382/steps/8/logs/stdio): ``` /home/b/sanitizer-x86_64-linux/build/llvm-project/llvm/lib/Target/SPIRV/SPIRVGlobalRegistry.cpp:85:3: error: default label in switch which covers all enumeration values [-Werror,-Wcovered-switch-default] 85 | default: | ^ ``` (also https://llvm.org/docs/CodingStandards.html#don-t-use-default-labels-in-fully-covered-switches-over-enumerations)
1 parent 4ea57b3 commit 83f831d

File tree

1 file changed

+0
-3
lines changed

1 file changed

+0
-3
lines changed

llvm/lib/Target/SPIRV/SPIRVGlobalRegistry.cpp

Lines changed: 0 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -82,9 +82,6 @@ storageClassRequiresExplictLayout(SPIRV::StorageClass::StorageClass SC) {
8282
case SPIRV::StorageClass::DeviceOnlyINTEL:
8383
case SPIRV::StorageClass::HostOnlyINTEL:
8484
return false;
85-
default:
86-
llvm_unreachable("Unknown storage class");
87-
return false;
8885
}
8986
}
9087

0 commit comments

Comments
 (0)