<div dir="ltr"><div><div><div>Hi Brian,<br><br></div>I have been working on the "gl_Position not written check"  (tests/spec/glsl-1.10/linker/) test from glean to shader_test files.<a href="http://lpaste.net/143421"> Here</a> is what I have so far.  I'm sure there may be some errors since this test does not have the fragment shader aspects. I am not very familiar with shader_test files. I will appreciate if you could correct this sample test so I could submit the patch and move on to the others.<br><br></div>Thanks,<br></div>Juliet<br></div><div class="gmail_extra"><br><div class="gmail_quote">On Wed, Oct 14, 2015 at 9:18 PM, Juliet Fru <span dir="ltr"><<a href="mailto:julietfru@gmail.com" target="_blank">julietfru@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr">Thanks, I will look at them As soon as possible.<br></div><div class="HOEnZb"><div class="h5"><div class="gmail_extra"><br><div class="gmail_quote">On Wed, Oct 14, 2015 at 6:09 PM, Brian Paul <span dir="ltr"><<a href="mailto:brianp@vmware.com" target="_blank">brianp@vmware.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span>On 10/14/2015 02:33 AM, Juliet Fru wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
Hello Brian,<br>
<br>
    Please post a plan for what you're doing before you do all the work<br>
    to make sure you don't do anything unnecessary.<br>
<br>
    Like I said before, most of the glsl1 tests should already be<br>
    covered by existing shader runner tests.  There's probably not too<br>
    many that need to be ported.<br>
<br>
<br>
             I don't like this at all. These tests should become<br>
        shader_test files<br>
             -- not just a straight port from glean to the piglit framework.<br>
<br>
<br>
I have been looking at the shader_test files and the shader_lang tests I<br>
see they're covered in the shader directory. I don't know if porting<br>
this test is necessary if not I'll like to know which tests should be<br>
moved as the shader_test files.<br>
</blockquote>
<br></span>
I think just a hand-full of tests could be ported to shader_runner tests.<br>
<br>
"current name" and (place to put new test):<br>
<br>
"gl_Position not written check"  (tests/spec/glsl-1.10/linker/)<br>
<br>
"varying var mismatch"    (tests/spec/glsl-1.10/linker/)<br>
<br>
"varying read but not written"  (tests/spec/glsl-1.10/linker/)<br>
<br>
"illegal assignment"   (tests/spec/glsl-1.10/compiler/)<br>
<br>
"continue with no loop"   (tests/spec/glsl-1.10/compiler/)<br>
<br>
"break with no loop"      (tests/spec/glsl-1.10/compiler/)<br>
<br>
"if (boolean-scalar) check"  (tests/spec/glsl-1.10/compiler/)<br>
<br>
<br>
There _may_ be shader runner tests already for some of these, but I didn't find them with a quick search.<span><font color="#888888"><br>
<br>
-Brian<br>
<br>
</font></span></blockquote></div><br></div>
</div></div></blockquote></div><br></div>