On Thursday, November 06, 2014 02:40:28 PM Ian Romanick wrote:
> The test looks fine enough, but could you add a spec quotation to defend it?
> 
> On 11/04/2014 04:19 AM, Tapani Pälli wrote:
> > Signed-off-by: Tapani Pälli <tapani.pa...@intel.com>
> > ---
> >  tests/spec/glsl-es-3.00/compiler/invariant_all.frag | 12 ++++++++++++
> >  1 file changed, 12 insertions(+)
> >  create mode 100644 tests/spec/glsl-es-3.00/compiler/invariant_all.frag
> > 
> > diff --git a/tests/spec/glsl-es-3.00/compiler/invariant_all.frag 
> > b/tests/spec/glsl-es-3.00/compiler/invariant_all.frag
> > new file mode 100644
> > index 0000000..37031cf
> > --- /dev/null
> > +++ b/tests/spec/glsl-es-3.00/compiler/invariant_all.frag
> > @@ -0,0 +1,12 @@
> > +#version 300 es
> 
> The #version shouldn't be necessary with the current infrastructure, right?

Actually, we do. I think this is a bug.

> 
> > +// [config]
> > +// expect_result: fail
> > +// glsl_version: 3.00
> > +// [end config]
> > +//
> > +// Check that 'invariant(all)' cannot be used in fragment shader
> > +
> > +#pragma STDGL invariant(all)
> > +void main()
> > +{
> > +}
> > 
> 
> _______________________________________________
> Piglit mailing list
> Piglit@lists.freedesktop.org
> http://lists.freedesktop.org/mailman/listinfo/piglit
> 

Attachment: signature.asc
Description: This is a digitally signed message part.

_______________________________________________
Piglit mailing list
Piglit@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/piglit

Reply via email to