On Thursday, September 15, 2016 3:12:34 PM PDT Ian Romanick wrote: > From: Ian Romanick <[email protected]> > > At this point in the code, s must be visit_continue. If the child > returned visit_stop, visit_stop is the only correct thing to return. > > Found by inspection. > > Signed-off-by: Ian Romanick <[email protected]> > Cc: [email protected] > --- > src/compiler/glsl/ir_hv_accept.cpp | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/src/compiler/glsl/ir_hv_accept.cpp > b/src/compiler/glsl/ir_hv_accept.cpp > index 213992a..5cc6a34 100644 > --- a/src/compiler/glsl/ir_hv_accept.cpp > +++ b/src/compiler/glsl/ir_hv_accept.cpp > @@ -147,7 +147,7 @@ ir_expression::accept(ir_hierarchical_visitor *v) > goto done; > > case visit_stop: > - return s; > + return visit_stop; > } > } > >
Patches 1-2, 7-8, and 13 are: Reviewed-by: Kenneth Graunke <[email protected]> You may as well land those ahead of the rest of the series.
signature.asc
Description: This is a digitally signed message part.
_______________________________________________ mesa-dev mailing list [email protected] https://lists.freedesktop.org/mailman/listinfo/mesa-dev
