<div dir="ltr">On 12 August 2013 13:11, Kenneth Graunke <span dir="ltr"><<a href="mailto:kenneth@whitecape.org" target="_blank">kenneth@whitecape.org</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">Hello,<br>
<br>
This surprisingly large series fixes bugs in the data flow algorithm<br>
for copy propagation.  As far as I can tell, there were a myriad of<br>
issues.  The updated algorithm seems to follow the textbook and<br>
online materials much more closely.<br>
<br>
No Piglit regressions on Ivybridge.<br>
<br>
shader-db statistics are more or less a wash:<br>
<br>
total instructions in shared programs: 1569184 -> 1569423 (0.02%)<br>
instructions in affected programs:     44961 -> 45200 (0.53%)<br>
<br>
A bunch of things are helped by a small margin, while other things are hurt<br>
by a small margin (a couple of extra MOVs).  The few hurt shaders I looked<br>
into appeared to suffer from compute-to-MRF not handling control flow.<br>
There may be other reasons as well.<br>
<br>
I'd still like to land it, though, as the old algorithm seems to be<br>
pretty broken.<br>
<br>
<br>
--Ken<br></blockquote><div><br></div><div>I sent comments on patch 2, 7, and 13.  The remainder are:<br><br>Reviewed-by: Paul Berry <<a href="mailto:stereotype441@gmail.com">stereotype441@gmail.com</a>><br></div></div>
</div></div>