[Piglit] [PATCH 1/2] glslparsertest: Allow ! in extension list to require non-support.
kenneth at whitecape.org
Mon Mar 12 12:59:30 PDT 2012
On 03/12/2012 12:39 PM, Eric Anholt wrote:
> On Mon, 12 Mar 2012 00:10:16 -0700, Kenneth Graunke<kenneth at whitecape.org> wrote:
>> For example:
>> // [config]
>> // required_extensions: !GL_ARB_draw_instanced
>> // [end config]
>> will cause the test to run only if the GL_ARB_draw_instanced is _not_
> Seems like a reasonable feature.
> Reviewed-by: Eric Anholt<eric at anholt.net>
> For the feature macro testing, I'm thinking we should probably have a C
> test that encodes what we understand about how the feature macros work
> and how to test them using command-line args, instead of writing tests
> per-extension (or, in the current case, just for the one extension we
> bothered with).
> I haven't reviewed the second patch, because while it looks sane to me,
> I seem to recall that idr's interpretation was that the macros didn't
> exist unless you #extensioned. But I may be totally wrong.
idr was mistaken. The macros always exist...and in fact, that's
precisely what makes them useful. That lets you do things like:
#define instanceID gl_InstanceIDARB
uniform int instanceID;
and then have one set of shaders which work regardless of whether the
extension is present.
More information about the Piglit