On 6 August 2013 10:37, Nicholas Mack <[email protected]> wrote: > --- > .../glsl-1.50/linker/unused-outputs.shader_test | 48 > ++++++++++++++++++++++ > 1 file changed, 48 insertions(+) > create mode 100644 tests/spec/glsl-1.50/linker/unused-outputs.shader_test >
Since this test actually performs drawing, it should go in the "execution" directory. With that changed, this patch is: Reviewed-by: Paul Berry <[email protected]> > > diff --git a/tests/spec/glsl-1.50/linker/unused-outputs.shader_test > b/tests/spec/glsl-1.50/linker/unused-outputs.shader_test > new file mode 100644 > index 0000000..ed44d60 > --- /dev/null > +++ b/tests/spec/glsl-1.50/linker/unused-outputs.shader_test > @@ -0,0 +1,48 @@ > +# Test that superfluous declarations are allowed > +# > +# GLSLangSpec.1.50, 4.3.6 (): > +# "Only output variables that are read by the subsequent pipeline stage > need > +# to be written; it is allowed to have superfluous declarations of output > +# variables." > + > +[require] > +GLSL >= 1.50 > + > +[vertex shader] > +#version 150 > + > +in vec4 vertex; > + > +out float a; > +out float b; > +out vec2 c; > + > +void main() > +{ > + a = 3.0; > + b = 1.0; > + gl_Position = vertex; > +} > + > +[fragment shader] > +#version 150 > + > +in float b; > +in vec2 c; > + > +void main() > +{ > + gl_FragColor = vec4(0.0, b, 0.0, 1.0); > +} > + > +[vertex data] > +vertex/float/2 > +-1.0 -1.0 > + 1.0 -1.0 > + 1.0 1.0 > +-1.0 1.0 > + > +[test] > +draw arrays GL_TRIANGLE_FAN 0 4 > +probe all rgba 0.0 1.0 0.0 1.0 > + > -- > 1.8.3.1 > > _______________________________________________ > Piglit mailing list > [email protected] > http://lists.freedesktop.org/mailman/listinfo/piglit >
_______________________________________________ Piglit mailing list [email protected] http://lists.freedesktop.org/mailman/listinfo/piglit
