[Piglit] [PATCH] glsl-1.50 compiler: Test that gl_FragCoord redeclaration recognized by compiler

[Ian Romanick]

On 08/08/2013 10:16 AM, Steve Miller wrote:
> V2: adjusted format and comments for consistency
> ---
>   .../layout-qualifier-basic-check.frag              | 30 ++++++++++++++++++++++
>   1 file changed, 30 insertions(+)
>   create mode 100644 tests/spec/glsl-1.50/compiler/fragment_coord_conventions/layout-qualifier-basic-check.frag
>
> diff --git a/tests/spec/glsl-1.50/compiler/fragment_coord_conventions/layout-qualifier-basic-check.frag b/tests/spec/glsl-1.50/compiler/fragment_coord_conventions/layout-qualifier-basic-check.frag
> new file mode 100644
> index 0000000..4dbc60e
> --- /dev/null
> +++ b/tests/spec/glsl-1.50/compiler/fragment_coord_conventions/layout-qualifier-basic-check.frag
> @@ -0,1 +1,30 @@
> +/*  [config]
> + *  expect_result: pass
> + *  glsl_version: 1.50
> + *  check_link: false
> + */ [end config]

I don't think you ran this. :)

> +
> +/* Section 4.3.8.1 (Input Layout Qualifiers) of the GLSL 1.50 spec says:
> + *
> + *     "Fragment shaders can have an input layout only for redeclaring the
> + *     built-in variable gl_FragCoord (see section 7.2 Fragment Shader
> + *     Special Variables). The layout qualifier identifiers for
> + *     gl_FragCoord are
> + *
> + *     layout-qualifier-id:
> + *         origin_upper_left
> + *         pixel_center_integer"
> + *
> + * This test is just a check to make sure the arb_fragment_coord_conventions
> + * extension is switched on in core compiler.
> + *
> + */
> +
> +#version 150
> +
> +layout(origin_upper_left, pixel_center_integer) in vec4 gl_FragCoord;
> +
> +void main()
> +{
> +     gl_FragColor = gl_FragCoord.xyzz;
> +}
>