How did this even compile?

Should we cc stable?

Reviewed-by: Lionel Landwerlin <lionel.g.landwer...@intel.com>

On 29/06/17 18:33, Jason Ekstrand wrote:
It's closing a "{" at the begining of a switch case.
---
  src/compiler/spirv/vtn_variables.c | 2 +-
  1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/src/compiler/spirv/vtn_variables.c 
b/src/compiler/spirv/vtn_variables.c
index 974adb5..31d1d76 100644
--- a/src/compiler/spirv/vtn_variables.c
+++ b/src/compiler/spirv/vtn_variables.c
@@ -1490,10 +1490,10 @@ vtn_handle_variables(struct vtn_builder *b, SpvOp 
opcode,
            */
           vtn_foreach_decoration(b, interface_type->val, var_decoration_cb, 
var);
           break;
+      }
case vtn_variable_mode_param:
           unreachable("Not created through OpVariable");
-      }
case vtn_variable_mode_ubo:
        case vtn_variable_mode_ssbo:


_______________________________________________
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/mesa-dev

Reply via email to