On Thursday, September 15, 2016 3:12:34 PM PDT Ian Romanick wrote: > From: Ian Romanick <ian.d.roman...@intel.com> > > 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 <ian.d.roman...@intel.com> > Cc: mesa-sta...@lists.freedesktop.org > --- > 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 <kenn...@whitecape.org> 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 mesa-dev@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/mesa-dev