<div dir="ltr">Thanks for taking a look.  This patch changes the generating scripts and what they produce.  They're both checked in.  The situation is different for some other generated test scripts.<div><br></div><div>
Stuart</div></div><div class="gmail_extra"><br><br><div class="gmail_quote">On Mon, Mar 11, 2013 at 7:29 PM, Tom Gall <span dir="ltr"><<a href="mailto:tom.gall@linaro.org" target="_blank">tom.gall@linaro.org</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">On Mon, Mar 11, 2013 at 12:41 PM, Stuart Abercrombie<br>
<<a href="mailto:sabercrombie@chromium.org">sabercrombie@chromium.org</a>> wrote:<br>
> It will be added based on the GLSL requirement instead.<br>
><br>
> Also make it clearer that these shader_test files are automatically generated.<br>
<br>
</div>Why patch the tests instead of the script that generated them?<br>
<div class="im"><br>
> Same number of passes on Sandy Bridge running all.tests without glean.<br>
><br>
> Signed-off-by: Stuart Abercrombie <<a href="mailto:sabercrombie@chromium.org">sabercrombie@chromium.org</a>><br>
</div><snip><br>
<br>
Reviewed-by: Tom Gall <<a href="mailto:tom.gall@linaro.org">tom.gall@linaro.org</a>><br>
<span class="HOEnZb"><font color="#888888"><br>
--<br>
Regards,<br>
Tom<br>
<br>
"Where's the kaboom!? There was supposed to be an earth-shattering<br>
kaboom!" Marvin Martian<br>
Tech Lead, Graphics Working Group | Linaro.org │ Open source software<br>
for ARM SoCs<br>
w) tom.gall att <a href="http://linaro.org" target="_blank">linaro.org</a><br>
h) tom_gall att <a href="http://mac.com" target="_blank">mac.com</a><br>
</font></span></blockquote></div><br></div>