[Piglit] [PATCH] glsl-es-3.00: test for 'invariant(all)' usage in a fragment shader

Ian Romanick idr at freedesktop.org
Thu Nov 6 14:40:28 PST 2014


The test looks fine enough, but could you add a spec quotation to defend it?

On 11/04/2014 04:19 AM, Tapani Pälli wrote:
> Signed-off-by: Tapani Pälli <tapani.palli at intel.com>
> ---
>  tests/spec/glsl-es-3.00/compiler/invariant_all.frag | 12 ++++++++++++
>  1 file changed, 12 insertions(+)
>  create mode 100644 tests/spec/glsl-es-3.00/compiler/invariant_all.frag
> 
> diff --git a/tests/spec/glsl-es-3.00/compiler/invariant_all.frag b/tests/spec/glsl-es-3.00/compiler/invariant_all.frag
> new file mode 100644
> index 0000000..37031cf
> --- /dev/null
> +++ b/tests/spec/glsl-es-3.00/compiler/invariant_all.frag
> @@ -0,0 +1,12 @@
> +#version 300 es

The #version shouldn't be necessary with the current infrastructure, right?

> +// [config]
> +// expect_result: fail
> +// glsl_version: 3.00
> +// [end config]
> +//
> +// Check that 'invariant(all)' cannot be used in fragment shader
> +
> +#pragma STDGL invariant(all)
> +void main()
> +{
> +}
> 



More information about the Piglit mailing list