On 08/14/2018 12:41 AM, Tapani Pälli wrote:
> Signed-off-by: Tapani Pälli <tapani.pa...@intel.com>
> Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=107550
> ---
>  .../spec/glsl-1.10/compiler/invalid-array-as-function-param.vert | 9 
> +++++++++
>  1 file changed, 9 insertions(+)
>  create mode 100644 
> tests/spec/glsl-1.10/compiler/invalid-array-as-function-param.vert
> 
> diff --git 
> a/tests/spec/glsl-1.10/compiler/invalid-array-as-function-param.vert 
> b/tests/spec/glsl-1.10/compiler/invalid-array-as-function-param.vert
> new file mode 100644
> index 000000000..d285680de
> --- /dev/null
> +++ b/tests/spec/glsl-1.10/compiler/invalid-array-as-function-param.vert

I was thinking maybe dereference-scalar-as-array-for-...  The we can
also have dereference-scalar-as-array-for-assignment,
dereference-scalar-as-array-for-operation, etc.  I can write those
later, if you like.

> @@ -0,0 +1,9 @@
> +// [config]
> +// expect_result: fail
> +// glsl_version: 1.10
> +// [end config]
> +

If you add

void func(int x)
{
    return;
}

so that there's only one compilation error,

Reviewed-by: Ian Romanick <ian.d.roman...@intel.com>

> +void main()
> +{
> +     func(0[2]);
> +}
> 

_______________________________________________
Piglit mailing list
Piglit@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/piglit

Reply via email to