[Piglit] [PATCH 1/2] Add tests for ARB_transform_feedback3

Brian Paul brian.e.paul at gmail.com
Thu Jul 5 06:31:58 PDT 2012


This is a pretty big patch and I didn't study every line, but it looks
OK overall.  Just minor comments below.

For both: Reviewed-by: Brian Paul <brianp at vmware.com>


On Thu, Jul 5, 2012 at 4:40 AM, Marek Olšák <maraeo at gmail.com> wrote:
> ---
>  tests/all.tests                                 |   11 +
>  tests/spec/ext_transform_feedback/output-type.c |  624 ++++++++++++++++-------
>  2 files changed, 445 insertions(+), 190 deletions(-)
>
> diff --git a/tests/all.tests b/tests/all.tests
> index 07eba93..22cb896 100644
> --- a/tests/all.tests
> +++ b/tests/all.tests
> @@ -1778,6 +1778,17 @@ arb_transform_feedback2 = Group()
>  spec['ARB_transform_feedback2'] = arb_transform_feedback2
>  arb_transform_feedback2['draw-auto'] = PlainExecTest(['arb_transform_feedback2-draw-auto', '-auto'])
>
> +arb_transform_feedback3 = Group()
> +spec['ARB_transform_feedback3'] = arb_transform_feedback3
> +
> +for param in ['gl_NextBuffer-1', 'gl_NextBuffer-2', 'gl_SkipComponents1-1',
> +              'gl_SkipComponents1-2', 'gl_SkipComponents1-3', 'gl_SkipComponents2',
> +              'gl_SkipComponents3', 'gl_SkipComponents4',
> +              'gl_NextBuffer-gl_SkipComponents1-gl_NextBuffer',
> +              'gl_NextBuffer-gl_NextBuffer', 'gl_SkipComponents1234']:
> +        arb_transform_feedback3[param] = concurrent_test(
> +                'ext_transform_feedback-output-type {0}'.format(param))
> +
>  arb_uniform_buffer_object = Group()
>  spec['ARB_uniform_buffer_object'] = arb_uniform_buffer_object
>  import_glsl_parser_tests(spec['ARB_uniform_buffer_object'],
> diff --git a/tests/spec/ext_transform_feedback/output-type.c b/tests/spec/ext_transform_feedback/output-type.c
> index 8b82a1f..016eb01 100644
> --- a/tests/spec/ext_transform_feedback/output-type.c
> +++ b/tests/spec/ext_transform_feedback/output-type.c
> @@ -35,15 +35,18 @@ PIGLIT_GL_TEST_MAIN(
>      32 /*window_height*/,
>      GLUT_DOUBLE | GLUT_RGB | GLUT_ALPHA)
>
> +#define DEFAULT_VALUE 0.123456
> +
>  struct test_desc {
>         const char *name;
>         const char *vs;
>         unsigned num_varyings;
>         const char *varyings[16];
>         bool is_floating_point;
> -       unsigned num_elements;
> -       float expected_float[256];
> -       GLint expected_int[256];
> +       unsigned num_elements[4];
> +       float expected_float[4][256];
> +       GLint expected_int[4][256];

Some comments would be helpful, and/or replace 4 with a MAX_STREAMS #define.
The 256 should probably be a #define also to give some clue to what it means.


More information about the Piglit mailing list