[Piglit] [PATCH] Remove #version from variable indexing shader tests.

Chad Versace chad.versace at linux.intel.com
Fri Dec 14 10:26:29 PST 2012


On 12/13/2012 10:00 AM, Stuart Abercrombie wrote:
> The idea is to ultimately remove explicit #version directives from all
> .shader_test files, so we can just rely on GLSL >=, and then the ES
> analog, allowing these tests to run on GL or GLES -- what Paul Berry
> outlined a while back, basically.

I see. If we want to run a shader_test under both GL and GLES, one
of the many steps to arrive there is to generate the #version directive
dynamically. That makes perfect sense.
 
> This patch is confined to the variable indexing tests.  It removes
> #version outputting from
> 
> tests/spec/glsl-1.10/variable-index-read.sh
> tests/spec/glsl-1.10/variable-index-write.sh
> 
> and the files generated for 1.10/1.20 from these scripts.  I've
> included the script modification below.
> 
> The follow-up was intended to be removal of #version from other
> .shader_test files, and other regularization like what I sent in
> before.
> 
> Since other work is being done for GLES I don't know if this is still
> a reasonable way forward.  Thoughts?

I think this patch is the right way to go forward, it does not conflict
with other GLES changes occurring in Piglit.

Reviewed-by: Chad Versace <chad.versace at linux.intel.com>


More information about the Piglit mailing list