[Piglit] [PATCH 8/8] arb_gpu_shader5: Add fma constant eval execution test.

Matt Turner mattst88 at gmail.com
Fri Aug 23 16:38:14 PDT 2013


On Fri, Aug 23, 2013 at 7:29 AM, Paul Berry <stereotype441 at gmail.com> wrote:
> This test doesn't verify that fma() is constant-folded.  I'd recommend doing
> the array size trick (see my comments on patch 5/8) here as well.
>
> With that changed, the patch is:
>
> Reviewed-by: Paul Berry <stereotype441 at gmail.com>
>
> Patches 1-4 and 6-7 are also:
>
> Reviewed-by: Paul Berry <stereotype441 at gmail.com>

I've fixed both patches 5 and 8 in the same way as I've done for the
preexisting const gpu shader5 tests (see "[PATCH] arb_gpu_shader5: Use
array sizing to catch bad constant folding.").

Thanks for the reviews and suggestions!
Matt


More information about the Piglit mailing list