On Mon, Nov 2, 2015 at 1:30 AM, Vinson Lee <v...@freedesktop.org> wrote: > nir/nir_control_flow.c: In function ‘split_block_cursor.isra.11’: > nir/nir_control_flow.c:460:15: warning: ‘after’ may be used uninitialized in > this function [-Wmaybe-uninitialized] > *_after = after; > ^ > nir/nir_control_flow.c:458:16: warning: ‘before’ may be used uninitialized in > this function [-Wmaybe-uninitialized] > *_before = before; > ^ > > Signed-off-by: Vinson Lee <v...@freedesktop.org> > --- > src/glsl/nir/nir_control_flow.c | 3 +++ > 1 file changed, 3 insertions(+) > > diff --git a/src/glsl/nir/nir_control_flow.c b/src/glsl/nir/nir_control_flow.c > index 7f51c4f..96395a4 100644 > --- a/src/glsl/nir/nir_control_flow.c > +++ b/src/glsl/nir/nir_control_flow.c > @@ -452,6 +452,9 @@ split_block_cursor(nir_cursor cursor, > before = split_block_before_instr(nir_instr_next(cursor.instr)); > } > break; > + > + default: > + unreachable("not reached"); > }
We've avoided adding this, because it seems like an absurd warning. The enum that's the subject of the switch has four members, all of which are handled in the switch. Researching this, it appears that gcc doesn't consider this warning indicate a bug because of some combination of (1) enums in C are just ints and can contain values not associated with an enum member, and (2) it would apparently require extra (generated) code to ensure that the value is that of an enum member. I don't imagine we'll want to add more members to the enum, so maybe it's okay... _______________________________________________ mesa-dev mailing list mesa-dev@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/mesa-dev