On Tuesday, February 13, 2018 2:57:07 PM PST Jason Ekstrand wrote: > This fixes the build in clang > --- > src/mesa/drivers/dri/i965/brw_nir_uniforms.cpp | 3 ++- > 1 file changed, 2 insertions(+), 1 deletion(-) > > diff --git a/src/mesa/drivers/dri/i965/brw_nir_uniforms.cpp > b/src/mesa/drivers/dri/i965/brw_nir_uniforms.cpp > index 10a4ff4..69da83a 100644 > --- a/src/mesa/drivers/dri/i965/brw_nir_uniforms.cpp > +++ b/src/mesa/drivers/dri/i965/brw_nir_uniforms.cpp > @@ -254,7 +254,8 @@ brw_nir_lower_patch_vertices_in_to_uniform(nir_shader > *nir) > gl_state_index16 tokens[STATE_LENGTH] = { > STATE_INTERNAL, > nir->info.stage == MESA_SHADER_TESS_CTRL ? > - STATE_TCS_PATCH_VERTICES_IN : STATE_TES_PATCH_VERTICES_IN, > + (gl_state_index16)STATE_TCS_PATCH_VERTICES_IN : > + (gl_state_index16)STATE_TES_PATCH_VERTICES_IN, > }; > var->num_state_slots = 1; > var->state_slots = >
This is fine, but I prefer your plan from IRC: 1. Add STATE_MAX_VALUE = 0xffff to the enum. 2. Mark the enum PACKED. 3. Drop gl_state_index16 again, since gl_state_index should now be the desired size, and also an actual enum so it follows the actual C++ enum rules. I suppose the downside is that it could cause "case not handled in switch" warnings... --Ken
signature.asc
Description: This is a digitally signed message part.
_______________________________________________ mesa-dev mailing list mesa-dev@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/mesa-dev