OSDN Git Service

nir/spirv: Make a decoration switch complete
authorJason Ekstrand <jason.ekstrand@intel.com>
Thu, 2 Jun 2016 21:36:58 +0000 (14:36 -0700)
committerEmil Velikov <emil.l.velikov@gmail.com>
Tue, 14 Jun 2016 14:48:36 +0000 (15:48 +0100)
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>
(cherry picked from commit 526a8de22d85bcdc3a46b2e16e0073527616d95d)

src/compiler/spirv/spirv_to_nir.c

index f1fa6ee..f1bbfd5 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");
    }
 }