nir/spirv: Make a decoration switch complete
authorJason Ekstrand <jason.ekstrand@intel.com>
Thu, 2 Jun 2016 21:36:58 +0000 (14:36 -0700)
committerJason Ekstrand <jason.ekstrand@intel.com>
Sat, 4 Jun 2016 02:29:28 +0000 (19:29 -0700)
Getting rid of the default case makes the compiler warn if we are missing
cases.  While we're here, we also add the one missing case.

Signed-off-by: Jason Ekstrand <jason@jlekstrand.net>
Cc: "12.0" <mesa-stable@lists.freedesktop.org>
src/compiler/spirv/spirv_to_nir.c

index f1fa6eedfae5a2de6059fd3d00f3a6f4bfcb4705..f1bbfd511ff11888484f2047ab1f21b56e358214 100644 (file)
@@ -533,6 +533,7 @@ struct_member_decoration_cb(struct vtn_builder *b,
    case SpvDecorationIndex:
    case SpvDecorationBinding:
    case SpvDecorationDescriptorSet:
+   case SpvDecorationLinkageAttributes:
    case SpvDecorationNoContraction:
    case SpvDecorationInputAttachmentIndex:
       vtn_warn("Decoration not allowed on struct members: %s",
@@ -553,9 +554,6 @@ struct_member_decoration_cb(struct vtn_builder *b,
       vtn_warn("Decoraiton only allowed for CL-style kernels: %s",
                spirv_decoration_to_string(dec->decoration));
       break;
-
-   default:
-      unreachable("Unhandled member decoration");
    }
 }