<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On Tue, Jan 13, 2015 at 1:08 PM, 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">I would have no clue what this patch did if I just read the subject. How about:<br>
<br>
> i965/fs: Allow constant propagation between different types.<br></blockquote><div><br></div><div>Sure.<br> <br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
Also, this should go in separately from NIR, since it affects things<br>
(in Synmark) now:<br>
<br>
instructions in affected programs:     2288 -> 2144 (-6.29%) <br></blockquote><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
It allows us to copy propagate a 0x00000000UD into a 0.0F in a<br>
load_payload, which then lets us combine 4 load_payloads.<br></blockquote><div><br></div><div>I'll add that and get it pushed.  Thanks for reviewing.<br></div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<span class=""><br>
On Mon, Dec 15, 2014 at 10:04 PM, Jason Ekstrand <<a href="mailto:jason@jlekstrand.net">jason@jlekstrand.net</a>> wrote:<br>
> Because NIR is typeless, it treats all constants as uint32 values and<br>
> reinterprets them when they are used later.  This commit allows those<br>
> values to be properly propagated.<br>
> ---<br>
<br>
</span>Reviewed-by: Matt Turner <<a href="mailto:mattst88@gmail.com">mattst88@gmail.com</a>><br>
</blockquote></div><br></div></div>