<div dir="ltr">On 19 April 2013 12:05, 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">Signed-off-by: Jordan Justen <<a href="mailto:jordan.l.justen@intel.com">jordan.l.justen@intel.com</a>><br>
---<br>
 .../compiler/interface-blocks-qualifiers-used.vert |   21 ++++++++++++++++++++<br>
 1 file changed, 21 insertions(+)<br>
 create mode 100644 tests/spec/glsl-1.50/compiler/interface-blocks-qualifiers-used.vert<br>
<br>
diff --git a/tests/spec/glsl-1.50/compiler/interface-blocks-qualifiers-used.vert b/tests/spec/glsl-1.50/compiler/interface-blocks-qualifiers-used.vert<br>
new file mode 100644<br>
index 0000000..b106d4a<br>
--- /dev/null<br>
+++ b/tests/spec/glsl-1.50/compiler/interface-blocks-qualifiers-used.vert<br>
@@ -0,0 +1,21 @@<br>
+// [config]<br>
+// expect_result: pass<br>
+// glsl_version: 1.50<br>
+// check_link: true<br>
+// [end config]<br>
+//<br>
+// Tests that supported qualifiers can be compiled.<br>
+<br>
+#version 150<br>
+<br>
+out block {<br>
+    smooth float a;<br>
+    flat float b;<br>
+    noperspective float c;<br>
+    centroid out float d;<br>
+} inst;<br>
+<br>
+void main()<br>
+{<br>
+}<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 class="gmail_extra">I'm concerned that this test isn't adequate for testing the functionality.  It's easy to imagine a bug in which the qualifiers "smooth", "flat", "noperspective", and "centroid" were allowed by the parser inside interface blocks, but had no effect.<br>
<br></div><div class="gmail_extra">I'd recommend making a shader_runner test that compares smooth, flat, noperspective, and centroid varyings inside an interface block with smooth, flat, noperspective, and centroid varyings declared outside of interface blocks.  If the varyings compare equal in the fragment shader, then that should be an adequate test, since we have other piglit tests to verify that these qualifiers work when outside of interface blocks.<br>
<br></div><div class="gmail_extra">Note: to test centroid adequately, we'd need to expand shader_runner to support multisampling.  That's going to be a bit of a pain--either we'll have to expand the piglit/waffle interface to allow shader_runner to request a multisampled visual (that's my preference), or we'll have to modify shader_runner to do its rendering into an fbo and then blit to the window (kinda hacky but should get the job done).  I'd be ok if we want to punt on this for now given the pain involved.<br>
</div></div>