[Piglit] [PATCH 2/2] variable-index-write.sh: extend to support glsl es 1.00

2013-04-18 Thread Tom Gall
Please apply. Initially posted ~ a month ago. No negative feedback. From tests/spec/glsl-1.10 extend variable-index-write.sh so that it can also correctly generates variable index write tests that support glsl-es-1.00. 1.00 added as a valid parameter to generated tests for glsl-es-1.00. For

Re: [Piglit] [PATCH 2/2] variable-index-write.sh: extend to support glsl es 1.00

2013-03-29 Thread Stuart Abercrombie
I guess if there's no chance this will be extended to GLSL ES 1.10 or later then it's OK to use 100. Reviewed-by: Stuart Abercrombie sabercrom...@chromium.org On Thu, Mar 28, 2013 at 12:03 PM, Tom Gall tom.g...@linaro.org wrote: On Thu, Mar 28, 2013 at 1:50 PM, Stuart Abercrombie

Re: [Piglit] [PATCH 2/2] variable-index-write.sh: extend to support glsl es 1.00

2013-03-28 Thread Stuart Abercrombie
The content looks good. I'm just not sure if supplying 100 as a parameter should be the trigger for GLES GLSL 1.00. It doesn't seem particularly obvious, although I know there is no regular GL GLSL version 1.00. A separate preexisting issue is the location of the file, which is again

Re: [Piglit] [PATCH 2/2] variable-index-write.sh: extend to support glsl es 1.00

2013-03-28 Thread Tom Gall
On Thu, Mar 28, 2013 at 1:50 PM, Stuart Abercrombie sabercrom...@google.com wrote: The content looks good. I'm just not sure if supplying 100 as a parameter should be the trigger for GLES GLSL 1.00. It doesn't seem particularly obvious, although I know there is no regular GL GLSL version

[Piglit] [PATCH 2/2] variable-index-write.sh: extend to support glsl es 1.00

2013-03-20 Thread Tom Gall
From tests/spec/glsl-1.10 extend variable-index-write.sh so that it can also correctly generates variable index write tests that support glsl-es-1.00. 1.00 added as a valid parameter to generated tests for glsl-es-1.00. For 1.00 tests in the vertex shader an orthographic projection is applied