[Piglit] [PATCH V2] arb_shader_image_load_store: fix syntax error in execution tests

2016-01-19 Thread Timothy Arceri
In GLSL 1.50 layout qualifiers must come before the storage qualifier. A recent fix in Mesa exposed this issue. V2: rename IMAGE_T -> IMAGE_UNIFORM_T, fix DST/SRC_IMAGE_T miss in V1 Cc: Francisco Jerez --- tests/spec/arb_shader_image_load_store/atomicity.c | 2 +- tests/spec/arb_shader

Re: [Piglit] [PATCH V2] arb_shader_image_load_store: fix syntax error in execution tests

2016-01-19 Thread Francisco Jerez
Timothy Arceri writes: > In GLSL 1.50 layout qualifiers must come before the storage > qualifier. A recent fix in Mesa exposed this issue. > > V2: rename IMAGE_T -> IMAGE_UNIFORM_T, fix DST/SRC_IMAGE_T miss in V1 > > Cc: Francisco Jerez Reviewed-by: Francisco Jerez > --- > tests/spec/arb_sha

Re: [Piglit] [PATCH V2] arb_shader_image_load_store: fix syntax error in execution tests

2016-01-20 Thread Ilia Mirkin
On Tue, Jan 19, 2016 at 9:21 PM, Timothy Arceri wrote: > @@ -98,7 +98,7 @@ header_hunk(const struct grid_info grid) > "#define H %d\n" > "#define N %d\n" > "#define GRID_T %s\n" > - "#define RET_IMAGE_T layout(%s) %s2D\n", > +

Re: [Piglit] [PATCH V2] arb_shader_image_load_store: fix syntax error in execution tests

2016-01-20 Thread Timothy Arceri
On Wed, 2016-01-20 at 03:42 -0500, Ilia Mirkin wrote: > On Tue, Jan 19, 2016 at 9:21 PM, Timothy Arceri > wrote: > > @@ -98,7 +98,7 @@ header_hunk(const struct grid_info grid) > > "#define H %d\n" > > "#define N %d\n" > > "#define GRID_T %s\n"