diff options
| author | Jason Ekstrand <[email protected]> | 2016-06-02 14:36:58 -0700 |
|---|---|---|
| committer | Jason Ekstrand <[email protected]> | 2016-06-03 19:29:28 -0700 |
| commit | 526a8de22d85bcdc3a46b2e16e0073527616d95d (patch) | |
| tree | 038e581d90d1da19fba2866d805ff5481b1967bf | |
| parent | 62c6e94bd69a1287ebb4908f4ee67b8f730466cc (diff) | |
nir/spirv: Make a decoration switch complete
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 <[email protected]>
Cc: "12.0" <[email protected]>
| -rw-r--r-- | src/compiler/spirv/spirv_to_nir.c | 4 |
1 files changed, 1 insertions, 3 deletions
diff --git a/src/compiler/spirv/spirv_to_nir.c b/src/compiler/spirv/spirv_to_nir.c index f1fa6eedfae..f1bbfd511ff 100644 --- a/src/compiler/spirv/spirv_to_nir.c +++ b/src/compiler/spirv/spirv_to_nir.c @@ -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"); } } |