[Piglit] [PATCH 3/3] glsl-1.20: Add a bunch of outerProduct execution tests

[Chad Versace]

On 04/28/2011 02:55 PM, Ian Romanick wrote:
> From: Ian Romanick <ian.d.romanick at intel.com>
> 
> ---
>  .../execution/fs-outerProduct-mat2.shader_test     |   45 +++++++
>  .../execution/fs-outerProduct-mat2x2.shader_test   |   45 +++++++
>  .../execution/fs-outerProduct-mat2x3.shader_test   |   45 +++++++
>  .../execution/fs-outerProduct-mat2x4.shader_test   |   45 +++++++
>  .../execution/fs-outerProduct-mat3.shader_test     |   45 +++++++
>  .../execution/fs-outerProduct-mat3x2.shader_test   |   45 +++++++
>  .../execution/fs-outerProduct-mat3x3.shader_test   |   45 +++++++
>  .../execution/fs-outerProduct-mat3x4.shader_test   |   45 +++++++
>  .../execution/fs-outerProduct-mat4.shader_test     |   45 +++++++
>  .../execution/fs-outerProduct-mat4x2.shader_test   |   45 +++++++
>  .../execution/fs-outerProduct-mat4x3.shader_test   |   45 +++++++
>  .../execution/fs-outerProduct-mat4x4.shader_test   |   45 +++++++
>  tests/spec/glsl-1.20/execution/outerProduct.sh     |  126 ++++++++++++++++++++

Linear algebra done in shell... that's a gem.

>  .../execution/vs-outerProduct-mat2.shader_test     |   48 ++++++++
>  .../execution/vs-outerProduct-mat2x2.shader_test   |   48 ++++++++
>  .../execution/vs-outerProduct-mat2x3.shader_test   |   48 ++++++++
>  .../execution/vs-outerProduct-mat2x4.shader_test   |   48 ++++++++
>  .../execution/vs-outerProduct-mat3.shader_test     |   48 ++++++++
>  .../execution/vs-outerProduct-mat3x2.shader_test   |   48 ++++++++
>  .../execution/vs-outerProduct-mat3x3.shader_test   |   48 ++++++++
>  .../execution/vs-outerProduct-mat3x4.shader_test   |   48 ++++++++
>  .../execution/vs-outerProduct-mat4.shader_test     |   48 ++++++++
>  .../execution/vs-outerProduct-mat4x2.shader_test   |   48 ++++++++
>  .../execution/vs-outerProduct-mat4x3.shader_test   |   48 ++++++++
>  .../execution/vs-outerProduct-mat4x4.shader_test   |   48 ++++++++



You need this hunk to actually add the tests.

---
> diff --git a/tests/all.tests b/tests/all.tests
> index 8ece555..ba15370 100644
> --- a/tests/all.tests
> +++ b/tests/all.tests
> @@ -666,6 +666,10 @@ spec['glsl-1.20'] = Group()
>  import_glsl_parser_tests(spec['glsl-1.20'],
>                          os.path.join(os.path.dirname(__file__), 'spec', 'glsl-1.20'),
>                          ['preprocessor', 'compiler'])
> +spec['glsl-1.20']['execution'] = Group()
> +add_shader_test_dir(spec['glsl-1.20']['execution'],
> +             os.path.join(os.path.dirname(__file__), 'spec', 'glsl-1.20', 'execution'),
> +             recursive=True)
>  
>  # Group spec/glsl-1.30
>  spec['glsl-1.30'] = Group()
---

With that hunk added,
Reviewed-by: Chad Versace <chad.versace at intel.com>

-- 
Chad Versace
chad at chad-versace.us