<div dir="ltr">On 30 July 2013 09:26, Paul Berry <span dir="ltr"><<a href="mailto:stereotype441@gmail.com" target="_blank">stereotype441@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 dir="ltr"><div>On 27 July 2013 12:29, Jordan Justen <span dir="ltr"><<a href="mailto:jljusten@gmail.com" target="_blank">jljusten@gmail.com</a>></span> wrote:<br>

</div><div class="gmail_extra"><div class="gmail_quote"><div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
Series Reviewed-by: Jordan Justen <<a href="mailto:jordan.l.justen@intel.com" target="_blank">jordan.l.justen@intel.com</a>><br>
<br>
I was wondering if it would be worth continuing to support the gs4<br>
tests? (PATCH 1) Probably not.<br>
<br>
One other note not directly related to this series. I tried running<br>
this series on an nVidia binary driver with GL 3.3 support. I ran:<br>
./piglit-run.py -t glsl-1.50 tests/all.tests results<br>
<br>
Interestingly, the result was only about 50% of the tests passed. I<br>
saw a similar result on master.<br></blockquote><div><br></div></div><div>Hmm, I don't seem to remember the pass rate being that bad on nVidia.  I'll do a comparison on my system before pushing this series.<br></div>

</div></div></div></blockquote><div><br></div><div>Ok, I checked into this and everything looks like an nVidia bug except for the failures in textureSize and texelFetch.  Those tests are failing because in patch 6, when I converted those tests over to use GLSL 1.50, I forgot to remove references to gl_PositionIn (which only exists in ARB_geometry_shader4).<br>
<br>I'm going to go ahead and push patches 1-5, and I'll rework patch 6 for a future series.<br></div></div></div></div>