[Piglit] [PATCH] glsl-1.30: Allow empty declarations with precision qualifier

Kenneth Graunke kenneth at whitecape.org
Thu Aug 15 08:47:33 PDT 2013


On Tuesday, August 13, 2013 01:17:15 PM you wrote:
> From: Ian Romanick <ian.d.romanick at intel.com>
> 
> This compiles on both NVIDIA (304.64 on GTX 260) and AMD (Catalyst
> 12.06 on Radeon HD 3650).  It also compiles on all versions of Mesa
> that support GLSL 1.30.
> 
> Signed-off-by: Ian Romanick <ian.d.romanick at intel.com>
> Cc: Kenneth Graunke <kenneth at whitecape.org>
> ---
>  tests/glslparsertest/glsl2/precision-05.vert | 13 +++++++------
>  1 file changed, 7 insertions(+), 6 deletions(-)
> 
> diff --git a/tests/glslparsertest/glsl2/precision-05.vert
> b/tests/glslparsertest/glsl2/precision-05.vert index a7fd4d9..413d231
> 100644
> --- a/tests/glslparsertest/glsl2/precision-05.vert
> +++ b/tests/glslparsertest/glsl2/precision-05.vert
> @@ -1,13 +1,14 @@
>  // [config]
> -// expect_result: fail
> +// expect_result: pass
>  // glsl_version: 1.30
> -//
> -// # NOTE: Config section was auto-generated from file
> -// # NOTE: 'glslparser.tests' at git revision
> -// # NOTE: 6cc17ae70b70d150aa1751f8e28db7b2a9bd50f0
>  // [end config]
> 
>  #version 130
> -/* FAIL - keyword 'precision' required. */
> +
> +/* Keyword 'precision' is required required for this to set the default
> + * precision.  However, the language allows empty declarations with or
> without + * a precision qualifier.  Other shipping implementations allow
> this syntax, + * and there is nothing in the spec that forbids it.
> + */
>  lowp float;
>  mediump int;

Looks good to me.

Reviewed-by: Kenneth Graunke <kenneth at whitecape.org>


More information about the Piglit mailing list