<div dir="ltr"><div>I had a few minor comments but everything looks pretty good.  Do with the comments what you will.<br><br></div>Reviewed-by: Jason Ekstrand <<a href="mailto:jason@jlekstrand.net">jason@jlekstrand.net</a>><br></div><div class="gmail_extra"><br><div class="gmail_quote">On Mon, May 16, 2016 at 9:22 PM, Francisco Jerez <span dir="ltr"><<a href="mailto:currojerez@riseup.net" target="_blank">currojerez@riseup.net</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">This series fixes the FS register spilling code to apply correct<br>
channel masks to the spilled data regardless of the datatype and<br>
execution controls of the generating instruction, which is especially<br>
important for instructions manipulating data of type size other than<br>
32 bit (e.g. FP64) or instructions with non-trivial execution controls<br>
(which will become somewhat more common in SIMD32 shaders).<br>
<br>
A bunch of FP64 piglit tests that spill registers are fixed by this<br>
series, and it considerably improves the piglit test pass rate with<br>
INTEL_DEBUG set to spill_fs to force the register allocator to spill<br>
all registers -- On SKL I only need an additional workaround to get it<br>
to survive a full spill_fs piglit run without hangs or regressions,<br>
but because the problem doesn't seem to be compiler-related I plan to<br>
address it separately after some additional investigation.<br>
<br>
You can find the same series in my Mesa tree:<br>
<br>
<a href="https://cgit.freedesktop.org/~currojerez/mesa/log/?h=i965-spilling-fixes" rel="noreferrer" target="_blank">https://cgit.freedesktop.org/~currojerez/mesa/log/?h=i965-spilling-fixes</a><br>
_______________________________________________<br>
mesa-dev mailing list<br>
<a href="mailto:mesa-dev@lists.freedesktop.org">mesa-dev@lists.freedesktop.org</a><br>
<a href="https://lists.freedesktop.org/mailman/listinfo/mesa-dev" rel="noreferrer" target="_blank">https://lists.freedesktop.org/mailman/listinfo/mesa-dev</a><br>
</blockquote></div><br></div>