<div dir="ltr">On 17 February 2013 10:31, Jordan Justen <span dir="ltr"><<a href="mailto:jordan.l.justen@intel.com" target="_blank">jordan.l.justen@intel.com</a>></span> wrote:<br><div class="gmail_extra"><div class="gmail_quote">
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">This tests to see if an interface block with a type mismatch<br>
between the VS and FS causes a link error.<br>
<br>
Signed-off-by: Jordan Justen <<a href="mailto:jordan.l.justen@intel.com">jordan.l.justen@intel.com</a>><br>
---<br>
 ...terface-blocks-member-type-mismatch.shader_test |   28 ++++++++++++++++++++<br>
 1 file changed, 28 insertions(+)<br>
 create mode 100644 tests/spec/glsl-1.50/linker/interface-blocks-member-type-mismatch.shader_test<br>
<br>
diff --git a/tests/spec/glsl-1.50/linker/interface-blocks-member-type-mismatch.shader_test b/tests/spec/glsl-1.50/linker/interface-blocks-member-type-mismatch.shader_test<br>
new file mode 100644<br>
index 0000000..a996a4d<br>
--- /dev/null<br>
+++ b/tests/spec/glsl-1.50/linker/interface-blocks-member-type-mismatch.shader_test<br>
@@ -0,0 +1,28 @@<br>
+[require]<br>
+GLSL >= 1.50<br>
+<br>
+[vertex shader]<br>
+#version 150<br>
+<br>
+uniform block {<br>
+       vec4 a; // a is vec3 in FS<br>
+} inst_a;<br>
+<br>
+void main()<br>
+{<br></blockquote><div><br></div><div>Although GLSL 1.50 no longer requires the shader to assign to gl_Position, I'm aware of at least one implementation (Apple with AMD chipsets) which runs into problems if gl_Position is not assigned.  To be on the safe side, I would recommend inserting "gl_Position = vec4(0.0);" here, just so that an implementation won't erroneously pass this test due to bugs with gl_Position.<br>
<br>I'd recommend making similar changes to patches 4-6.<br><br></div><div>With that change, this patch is:<br><br></div><div>Reviewed-by: Paul Berry <<a href="mailto:stereotype441@gmail.com">stereotype441@gmail.com</a>><br>
</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
+}<br>
+<br>
+[fragment shader]<br>
+#version 150<br>
+<br>
+uniform block {<br>
+       vec3 a; // a is vec4 in VS<br>
+} inst_b;<br>
+<br>
+void main()<br>
+{<br>
+}<br>
+<br>
+[test]<br>
+link error<br>
+<br>
<span class="HOEnZb"><font color="#888888">--<br>
1.7.10.4<br>
<br>
_______________________________________________<br>
Piglit mailing list<br>
<a href="mailto:Piglit@lists.freedesktop.org">Piglit@lists.freedesktop.org</a><br>
<a href="http://lists.freedesktop.org/mailman/listinfo/piglit" target="_blank">http://lists.freedesktop.org/mailman/listinfo/piglit</a><br>
</font></span></blockquote></div><br></div></div>