Hi Timothy, Thanks for the review! Can you please push this patch ?
вт, 28 авг. 2018 г. в 3:45, Timothy Arceri <tarc...@itsqueeze.com>: > Reviewed-by: Timothy Arceri <tarc...@itsqueeze.com> > > On 27/08/18 22:19, Vadym Shovkoplias wrote: > > This test exposes a Mesa GLSL linker bug. The test fails with the > > following error message: > > > > error: fragment shader input `foo' has no matching output in the > previous > > stage > > > > Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=105731 > > Signed-off-by: Vadym Shovkoplias <vadym.shovkopl...@globallogic.com> > > --- > > ...rstage-multiple-vertex-objects.shader_test | 33 +++++++++++++++++++ > > 1 file changed, 33 insertions(+) > > create mode 100644 > tests/spec/glsl-1.30/linker/interstage-multiple-vertex-objects.shader_test > > > > diff --git > a/tests/spec/glsl-1.30/linker/interstage-multiple-vertex-objects.shader_test > b/tests/spec/glsl-1.30/linker/interstage-multiple-vertex-objects.shader_test > > new file mode 100644 > > index 000000000..dd168d434 > > --- /dev/null > > +++ > b/tests/spec/glsl-1.30/linker/interstage-multiple-vertex-objects.shader_test > > @@ -0,0 +1,33 @@ > > +# Exercises a Mesa GLSL linker bug. > > +# > > +# Output "foo" variable is not used in the "main" vertex shader > > +# but used in fragment shader > > + > > +[require] > > +GLSL >= 1.30 > > + > > +[vertex shader] > > +out vec4 foo; > > +void unused() > > +{ > > + foo=vec4(1); > > +} > > + > > +[vertex shader] > > +in vec4 pos; > > +void main() > > +{ > > + gl_Position = pos; > > +} > > + > > +[fragment shader] > > +in vec4 foo; > > +out vec4 color; > > + > > +void main() > > +{ > > + gl_FragColor=foo; > > +} > > + > > +[test] > > +link success > > >
_______________________________________________ Piglit mailing list Piglit@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/piglit