[Piglit] [PATCH V3] arb_shading_language_420pack: test multiple layout qualifiers on the same line

[Timothy Arceri]

On Fri, 2016-01-15 at 12:40 +1100, Timothy Arceri wrote:
> Test results:
> Nvidia GeForce 840M - NVIDIA 352.41
> 
> The results from the Nvidia binary are a little all over the place
> with these
> tests, most of the tests with arb_shading_language_420pack enabled
> fail but
> if the GLSL version is changed to 4.20 they pass.
> The only tests that pass with arb_shading_language_420pack enabled
> are the
> global tests but that seems to be because they do not detect the
> duplicates
> and fail the negative tests.
> 
> Mesa 11.2 (dev):
> The negative tests correctly pass, the possitive tests fail.
> 
> V3: drop tests that should target ARB_enhanced_layouts
> V2: fix version mismatch in location tests, and prefix variable with
> out
> in global tests
> 
> Cc: Matt Turner <mattst88 at gmail.com>
> ---
>  ...e-qualifier-in-single-declaration-420-pack.vert | 22
> ++++++++++++++++++
>  ...ualifier-in-single-declaration-no-420-pack.vert | 21
> +++++++++++++++++
>  ...e-qualifier-in-single-declaration-420-pack.geom | 25
> ++++++++++++++++++++
>  ...ualifier-in-single-declaration-no-420-pack.geom | 24
> +++++++++++++++++++
>  ...fier-in-single-global-declaration-420-pack.geom | 27
> ++++++++++++++++++++++
>  ...r-in-single-global-declaration-no-420-pack.geom | 26
> +++++++++++++++++++++
>  6 files changed, 145 insertions(+)
>  create mode 100644
> tests/spec/arb_shading_language_420pack/compiler/layout
> -qualifiers/multiple-location-multiple-qualifier-in-single
> -declaration-420-pack.vert
>  create mode 100644
> tests/spec/arb_shading_language_420pack/compiler/layout
> -qualifiers/multiple-location-multiple-qualifier-in-single
> -declaration-no-420-pack.vert
>  create mode 100644
> tests/spec/arb_shading_language_420pack/compiler/layout
> -qualifiers/multiple-stream-multiple-qualifier-in-single-declaration
> -420-pack.geom
>  create mode 100644
> tests/spec/arb_shading_language_420pack/compiler/layout
> -qualifiers/multiple-stream-multiple-qualifier-in-single-declaration
> -no-420-pack.geom
>  create mode 100644
> tests/spec/arb_shading_language_420pack/compiler/layout
> -qualifiers/multiple-stream-multiple-qualifier-in-single-global
> -declaration-420-pack.geom
>  create mode 100644
> tests/spec/arb_shading_language_420pack/compiler/layout
> -qualifiers/multiple-stream-multiple-qualifier-in-single-global
> -declaration-no-420-pack.geom
> 
> diff --git a/tests/spec/arb_shading_language_420pack/compiler/layout
> -qualifiers/multiple-location-multiple-qualifier-in-single
> -declaration-420-pack.vert
> b/tests/spec/arb_shading_language_420pack/compiler/layout
> -qualifiers/multiple-location-multiple-qualifier-in-single
> -declaration-420-pack.vert
> new file mode 100644
> index 0000000..9551476


I'm going to re-name of all these before pushing to remove the second
"multiple" as its not needed.