<div dir="ltr"><div style>I've basically been following the sequence Paul Berry suggested here: <a href="http://lists.freedesktop.org/archives/piglit/2012-August/003162.html">http://lists.freedesktop.org/archives/piglit/2012-August/003162.html</a>.</div>
<div style>It seems to be a reasonable way of getting to a point where some subset of .shader_test scripts can be run with both GL and GLES.</div><div><br></div>As far as I know, removing the explicit #version directive does not in itself make any tests run somewhere new.  It's just a prerequisite.  While there are tests that will never transfer to GLES, continued support of explicit #version directives means new tests will inevitably added with it present.  If I remove it from all tests I can remove support for it, and ensure that future tests have better odds of supporting both APIs.<div>
<br></div><div>Aside from this, I see them as removing redundancy, given that we already have GLSL >= everywhere.</div><div><br></div><div style>Stuart</div></div><div class="gmail_extra"><br><br><div class="gmail_quote">
On Thu, Apr 4, 2013 at 9:11 AM, Eric Anholt <span dir="ltr"><<a href="mailto:eric@anholt.net" target="_blank">eric@anholt.net</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div class="im">Stuart Abercrombie <<a href="mailto:sabercrombie@chromium.org">sabercrombie@chromium.org</a>> writes:<br>
<br>
> It will be added based on the GLSL requirement instead.<br>
><br>
> Same number of passes on Sandy Bridge running all.tests without glean.<br>
<br>
</div>I don't like this anti-#version crusade.  I'd like to see only patches<br>
to remove them if it actually makes the tests run somewhere new.  These<br>
tests in particular are for specific behavior of GLSL 1.10, and GLES<br>
should have equivalent tests that test for their particular limits.<br>
</blockquote></div><br></div>