[Piglit] [PATCH 00/13] Simplify GLSL minimum-maximum testing

Eric Anholt eric at anholt.net
Mon Aug 26 15:38:26 PDT 2013


Ian Romanick <idr at freedesktop.org> writes:

> This series simplifies GLSL minimum-maximum testing.  Future GLSL
> versions can be tested by simply cut-and-pasting from the spec instead
> of copying and modifying a bunch of .shader_test files.  See patch 8
> (glsl-3.30: Verify the limits in section 7.4) and patch 9
> (glsl-es-3.30: Verify the limits in section 7.3) for examples.  This
> also makes it much easier to review future such patches.
>
> There are some additional advantages (copied from the commit message of
> patch 2):
>
>     1. All of the built-in constants and their expected values are kept
>     in one place per GLSL version.
>     
>     2. The tests use the array-size idiom of many piglit tests which
>     guarantees that the values being tested are actually contants.
>     
>     3. Since the tests are not execution tests, they run much faster.
>     The single top-level test can also run concurrently with other
>     tests.  The speed difference is trivial most of the time, but in
>     simlation environments, each pixel drawn takes a very long time.
>     You're welcome, Chad.
>
> This is the first series in my current piglit yak-shaving branch on
> freedesktop.org.  I have some additional serieseseses that will land
> there later today.

Assuming the minor change in patch 10, all but patches 2 and 12 get my
Reviewed-by.  (And I don't feel too strongly about any of the feedback
in patch 2/10 anyway).
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 835 bytes
Desc: not available
URL: <http://lists.freedesktop.org/archives/piglit/attachments/20130826/b92e3aaf/attachment.pgp>


More information about the Piglit mailing list