<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Thu, Jan 25, 2018 at 10:08 AM, Matt Turner <span dir="ltr"><<a href="mailto:mattst88@gmail.com" target="_blank">mattst88@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="">On Fri, Dec 15, 2017 at 5:12 PM, Jason Ekstrand <<a href="mailto:jason@jlekstrand.net">jason@jlekstrand.net</a>> wrote:<br>
> 18fde36ced4279f2577097a1a7d31b<wbr>55f2f5f141 changed the way temporary<br>
> registers were allocated in lower_integer_multiplication so that we<br>
> allocate regs_written(inst) space and keep the stride of the original<br>
> destination register.  This was to ensure that any MUL which originally<br>
> followed the CHV/BXT integer multiply regioning restrictions would<br>
> continue to follow those restrictions even after lowering.  This works<br>
> fine except that I forgot to reset the register file to VGRF so, even<br>
> though they were assigned a number from alloc.allocate(), they had the<br>
> wrong register file.  This caused some GLES 3.0 CTS tests to start<br>
> failing on Sandy Bridge due to attempted reads from the MRF:<br>
><br>
>     ES3-CTS.functional.shaders.<wbr>precision.int.highp_mul_<wbr>fragment.snbm64<br>
>     ES3-CTS.functional.shaders.<wbr>precision.int.mediump_mul_<wbr>fragment.snbm64<br>
>     ES3-CTS.functional.shaders.<wbr>precision.int.lowp_mul_<wbr>fragment.snbm64<br>
>     ES3-CTS.functional.shaders.<wbr>precision.uint.highp_mul_<wbr>fragment.snbm64<br>
>     ES3-CTS.functional.shaders.<wbr>precision.uint.mediump_mul_<wbr>fragment.snbm64<br>
>     ES3-CTS.functional.shaders.<wbr>precision.uint.lowp_mul_<wbr>fragment.snbm64<br>
><br>
> This commit remedies this problem by, instead of copying inst->dst and<br>
> overwriting nr, just make a new register and set the region to match<br>
> inst->dst.<br>
><br>
> Cc: Matt Turner <<a href="mailto:mattst88@gmail.com">mattst88@gmail.com</a>><br>
> Bugzilla: <a href="https://bugs.freedesktop.org/show_bug.cgi?id=103626" rel="noreferrer" target="_blank">https://bugs.freedesktop.org/<wbr>show_bug.cgi?id=103626</a><br>
> Fixes: 18fde36ced4279f2577097a1a7d31b<wbr>55f2f5f141<br>
> Cc: "17.3" <<a href="mailto:mesa-stable@lists.freedesktop.org">mesa-stable@lists.<wbr>freedesktop.org</a>><br>
<br>
</span>Thanks. Sorry this got lost. Looks like it was sent the day I started vacation.<br>
<br>
Reviewed-by: Matt Turner <<a href="mailto:mattst88@gmail.com">mattst88@gmail.com</a>><br>
</blockquote></div></div><div class="gmail_extra"><br></div><div class="gmail_extra">Thanks!  I'll give it one more run through Jenkins and land it.<br></div></div>