<div dir="ltr">On 22 February 2013 15:23, Jordan Justen <span dir="ltr"><<a href="mailto:jljusten@gmail.com" target="_blank">jljusten@gmail.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"><div class="HOEnZb"><div class="h5">On Fri, Feb 22, 2013 at 10:07 AM, Paul Berry <<a href="mailto:stereotype441@gmail.com">stereotype441@gmail.com</a>> wrote:<br>

> On 17 February 2013 10:31, Jordan Justen <<a href="mailto:jordan.l.justen@intel.com">jordan.l.justen@intel.com</a>> wrote:<br>
>><br>
>> 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>
>> ++++++++++++++++++++<br>
>>  1 file changed, 28 insertions(+)<br>
>>  create mode 100644<br>
>> tests/spec/glsl-1.50/linker/interface-blocks-member-type-mismatch.shader_test<br>
>><br>
>> diff --git<br>
>> a/tests/spec/glsl-1.50/linker/interface-blocks-member-type-mismatch.shader_test<br>
>> 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>
>> +++<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>
><br>
><br>
> Although GLSL 1.50 no longer requires the shader to assign to gl_Position,<br>
> I'm aware of at least one implementation (Apple with AMD chipsets) which<br>
> runs into problems if gl_Position is not assigned.  To be on the safe side,<br>
> I would recommend inserting "gl_Position = vec4(0.0);" here, just so that an<br>
> implementation won't erroneously pass this test due to bugs with<br>
> gl_Position.<br>
><br>
> I'd recommend making similar changes to patches 4-6.<br>
<br>
</div></div>I'll add it for 3 & 6 since they are negative tests. We wouldn't want<br>
to add this for tests that are expected to pass, right?<br>
<span class="HOEnZb"><font color="#888888"><br>
-Jordan<br>
</font></span></blockquote></div><br></div><div class="gmail_extra">Sure, I could go either way on what we do with the positive tests.<br></div></div>