[Mesa-dev] [Mesa-stable] [PATCH] glsl: fix stream qualifier for blocks with an instance name
Emil Velikov
emil.l.velikov at gmail.com
Thu Nov 5 05:24:58 PST 2015
On 17 October 2015 at 10:51, Timothy Arceri <t_arceri at yahoo.com.au> wrote:
> This also removes the validation from the parser as it is not required
> and once arb_enhanced_layouts comes along we wont be able to do validation
> on the stream qualifier in the parser anyway as it adds constant expression
> support to the stream qualifier.
>
> Cc: Samuel Iglesias Gonsalvez <siglesias at igalia.com>
> Cc: 11.0 <mesa-stable at lists.freedesktop.org>
> ---
>
> Fixes two new piglit tests I've sent to the piglit list:
> http://patchwork.freedesktop.org/patch/62097/
> http://patchwork.freedesktop.org/patch/62098/
>
> src/glsl/ast_to_hir.cpp | 26 +++++++++++++++-----------
> src/glsl/glsl_parser.yy | 11 -----------
> src/glsl/lower_named_interface_blocks.cpp | 1 +
> 3 files changed, 16 insertions(+), 22 deletions(-)
>
Hmm I thought I mentioned it already, but here it goes:
Tim, can you please provide a fix against 11.0 if we still want this
in the stable branch.
Thanks
Emil
More information about the mesa-dev
mailing list