[Piglit] [PATCH] glsl-1.50: do not link interface-blocks-containing-unsized-arrays.geom
Iago Toral Quiroga
itoral at igalia.com
Fri Nov 4 12:33:14 UTC 2016
Linking means that we will validate in/out blocks between consumer and
producer stages. In this case it should fail because for geometry shaders
piglit will include a dummy vertex shader that does not have a matching
output block.
This has not been a problem until now because the linker was not detecting
this situation properly and did not produce the expected link error. I am
sending a patch to Mesa to fix this and once that lands this will start
to fail without this patch.
---
.../compiler/interface-blocks-containing-unsized-arrays.geom | 7 ++++++-
1 file changed, 6 insertions(+), 1 deletion(-)
diff --git a/tests/spec/glsl-1.50/compiler/interface-blocks-containing-unsized-arrays.geom b/tests/spec/glsl-1.50/compiler/interface-blocks-containing-unsized-arrays.geom
index 169122d..a11f291 100644
--- a/tests/spec/glsl-1.50/compiler/interface-blocks-containing-unsized-arrays.geom
+++ b/tests/spec/glsl-1.50/compiler/interface-blocks-containing-unsized-arrays.geom
@@ -1,12 +1,17 @@
// [config]
// expect_result: pass
// glsl_version: 1.50
-// check_link: true
+// check_link: false
// [end config]
//
// Test that an interface block may contain members which are unsized
// arrays. Both GLSL 1.10 and GLSL 1.20 style array declarations are
// tested.
+//
+// We cannot check linking because with geometry shaders, piglit adds
+// a dummy vertex shader and in this case, for linking to work, we need
+// that dummy vertex shader to include a matching output block, which
+// it doesn't.
#version 150
layout(triangles) in;
--
2.7.4
More information about the Piglit
mailing list