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 <[email protected]>
> ---
>  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()
> +{
> +}
> 

_______________________________________________
Piglit mailing list
[email protected]
http://lists.freedesktop.org/mailman/listinfo/piglit

Reply via email to