On Fri, 2016-03-04 at 21:11 -0800, Kenneth Graunke wrote:
> The lexer/parser use a symbol table to classify identifiers as
> variables, functions, or structure types.
> 
> For some reason, we neglected to add variables in simple declarations
> such as
> 
>     int x = 5;
> 
> but did add subsequent variables in multi-declarations:
> 
>     int x = 5, y = 6; // y gets added, but not x, for some reason
> 
> Fixes Piglit's spec/glsl-1.20/compiler/scoping-struct-vs-
> variable.vert.
> 
> Fixes four dEQP-GLES2.functional.shaders.scoping.valid subcases:
> - local_int_variable_hides_struct_type_vertex
> - local_int_variable_hides_struct_type_fragment
> - local_struct_variable_hides_struct_type_vertex
> - local_struct_variable_hides_struct_type_fragment
> 
> Signed-off-by: Kenneth Graunke <kenn...@whitecape.org>

Reviewed-by: Timothy Arceri <timothy.arc...@collabora.com>


_______________________________________________
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/mesa-dev

Reply via email to