Fix MSVC warning C4065: 'default' but no 'case' labels
authorCorentin Wallez <cwallez@google.com>
Mon, 29 Oct 2018 15:24:00 +0000 (16:24 +0100)
committerCorentin Wallez <cwallez@google.com>
Mon, 29 Oct 2018 15:24:00 +0000 (16:24 +0100)
This would happen in SpvPostProcess when we don't enable AMD extensions.

SPIRV/SpvPostProcess.cpp

index 27f145d..653edf4 100755 (executable)
@@ -121,8 +121,8 @@ void Builder::postProcessType(const Instruction& inst, Id typeId)
     case OpUConvert:
         break;
     case OpExtInst:
-        switch (inst.getImmediateOperand(1)) {
 #if AMD_EXTENSIONS
+        switch (inst.getImmediateOperand(1)) {
         case GLSLstd450Frexp:
         case GLSLstd450FrexpStruct:
             if (getSpvVersion() < glslang::EShTargetSpv_1_3 && containsType(typeId, OpTypeInt, 16))
@@ -134,10 +134,10 @@ void Builder::postProcessType(const Instruction& inst, Id typeId)
             if (getSpvVersion() < glslang::EShTargetSpv_1_3 && containsType(typeId, OpTypeFloat, 16))
                 addExtension(spv::E_SPV_AMD_gpu_shader_half_float);
             break;
-#endif
         default:
             break;
         }
+#endif
         break;
     default:
         if (basicTypeOp == OpTypeFloat && width == 16)