On Thu, 2015-08-20 at 10:46 -0700, Ian Romanick wrote: > I might have been tempted to put this and the previous patch before > patch 6, but meh.
Yeah I did it this way as 6 is really a fix independent of AoA and I wanted to be able to push it without waiting on the AoA patches to be fully reviewed (I sent it separately a couple of time without getting a review). For the record I've pushed it so it will be in Mesa 11.0. I'll push these ones when I land the rest of the series. > > This patch is > > Reviewed-by: Ian Romanick <ian.d.roman...@intel.com> > > On 08/19/2015 09:37 PM, Timothy Arceri wrote: > > Reviewed-by: Tapani Pälli <tapani.pa...@intel.com> > > --- > > src/glsl/ast_to_hir.cpp | 2 +- > > 1 file changed, 1 insertion(+), 1 deletion(-) > > > > diff --git a/src/glsl/ast_to_hir.cpp b/src/glsl/ast_to_hir.cpp > > index ee5ff9d..54569a8 100644 > > --- a/src/glsl/ast_to_hir.cpp > > +++ b/src/glsl/ast_to_hir.cpp > > @@ -2114,7 +2114,7 @@ validate_binding_qualifier(struct > > _mesa_glsl_parse_state *state, > > } > > > > const struct gl_context *const ctx = state->ctx; > > - unsigned elements = type->is_array() ? type->length : 1; > > + unsigned elements = type->is_array() ? type->arrays_of_arrays_size() : > > 1; > > unsigned max_index = qual->binding + elements - 1; > > const glsl_type *base_type = type->without_array(); > > > > > _______________________________________________ mesa-dev mailing list mesa-dev@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/mesa-dev
