Skip to content

Commit 22233fa

Browse files
svenvhvmaksimo
authored andcommitted
Remove unnecessary switch case for OpSelect
With SPIRVSelect inheriting from SPIRVInstTemplateBase, this can now be handled through the switch default. Original commit: KhronosGroup/SPIRV-LLVM-Translator@b5441a4
1 parent 994373d commit 22233fa

File tree

1 file changed

+0
-4
lines changed

1 file changed

+0
-4
lines changed

llvm-spirv/lib/SPIRV/SPIRVWriter.cpp

Lines changed: 0 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -3885,10 +3885,6 @@ LLVMToSPIRVBase::transBuiltinToInstWithoutDecoration(Op OC, CallInst *CI,
38853885
return BM->addAsyncGroupCopy(BArgs[0], BArgs[1], BArgs[2], BArgs[3],
38863886
BArgs[4], BArgs[5], BB);
38873887
} break;
3888-
case OpSelect: {
3889-
auto BArgs = transValue(getArguments(CI), BB);
3890-
return BM->addSelectInst(BArgs[0], BArgs[1], BArgs[2], BB);
3891-
}
38923888
case OpSampledImage: {
38933889
// Clang can generate SPIRV-friendly call for OpSampledImage instruction,
38943890
// i.e. __spirv_SampledImage... But it can't generate correct return type

0 commit comments

Comments
 (0)