[Piglit] [PATCH] glsl-1.20: test declaring gl_Position with the invariant qualifier.

Marek Olšák maraeo at gmail.com
Thu Oct 20 10:27:16 UTC 2016


If it's just a linker test, the draw call is unnecessary. I think this
can be used instead:
[test]
link success

Whether or not you use that fix:

Reviewed-by: Marek Olšák <marek.olsak at amd.com>

Marek

On Wed, Oct 19, 2016 at 1:28 AM, Brian Paul <brianp at vmware.com> wrote:
> To exercise a Mesa GLSL regression.
> ---
>  .../linker/invariant-position.shader_test          | 26 ++++++++++++++++++++++
>  1 file changed, 26 insertions(+)
>  create mode 100644 tests/spec/glsl-1.20/linker/invariant-position.shader_test
>
> diff --git a/tests/spec/glsl-1.20/linker/invariant-position.shader_test b/tests/spec/glsl-1.20/linker/invariant-position.shader_test
> new file mode 100644
> index 0000000..4bbed5b
> --- /dev/null
> +++ b/tests/spec/glsl-1.20/linker/invariant-position.shader_test
> @@ -0,0 +1,26 @@
> +[require]
> +GLSL >= 1.20
> +
> +# This test exercises a Mesa GLSL regression where declaring a pre-defined
> +# VS input as 'invariant' cause a linker failure.
> +
> +[vertex shader]
> +#version 120
> +invariant gl_Position;
> +void main()
> +{
> +  gl_Position = ftransform();
> +}
> +
> +
> +[fragment shader]
> +#version 120
> +void main()
> +{
> +   gl_FragColor = vec4(1.0);
> +}
> +
> +
> +[test]
> +draw rect -1 -1 2 2
> +probe rgba 0 0  1 1 1 1
> --
> 1.9.1
>
> _______________________________________________
> Piglit mailing list
> Piglit at lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/piglit


More information about the Piglit mailing list