[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