On Sat, Mar 28, 2015 at 12:23 AM, Kenneth Graunke <[email protected]> wrote: > These are nir_cf_nodes, not ALU instructions. > Also, use unreachable() to preempt said review feedback. > > Signed-off-by: Kenneth Graunke <[email protected]> > --- > src/glsl/nir/nir_validate.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/src/glsl/nir/nir_validate.c b/src/glsl/nir/nir_validate.c > index f247ae0..79c23db 100644 > --- a/src/glsl/nir/nir_validate.c > +++ b/src/glsl/nir/nir_validate.c > @@ -680,7 +680,7 @@ validate_cf_node(nir_cf_node *node, validate_state *state) > break; > > default: > - assert(!"Invalid ALU instruction type"); > + unreachable(!"Invalid CF node type");
unreachable just takes a str, not a condition passed through to an assert. > break; should probably remove the break while you're at it. > } > } > -- > 2.3.4 > > _______________________________________________ > mesa-dev mailing list > [email protected] > http://lists.freedesktop.org/mailman/listinfo/mesa-dev _______________________________________________ mesa-dev mailing list [email protected] http://lists.freedesktop.org/mailman/listinfo/mesa-dev
