<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Tue, Sep 6, 2016 at 11:14 AM, Eric Anholt <span dir="ltr"><<a href="mailto:eric@anholt.net" target="_blank">eric@anholt.net</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="">Tapani Pälli <<a href="mailto:tapani.palli@intel.com">tapani.palli@intel.com</a>> writes:<br>
<br>
> This change makes copy propagation pass faster. Complete link time<br>
> spent in test case attached to bug 94477 goes down to ~400 secs from<br>
> over 500 secs on my HSW machine. Does not fix the actual issue but<br>
> brings down the total. No regressions seen in CI.<br>
><br>
> Signed-off-by: Tapani Pälli <<a href="mailto:tapani.palli@intel.com">tapani.palli@intel.com</a>><br>
> ---<br>
><br>
> Next I'll attempt to make similar change to opt_copy_propagation_elements.<br>
<br>
</span>This all looks correct, though it makes me sad to see work being done on<br>
these passes instead of just removing them from being called by NIR<br>
drivers.<br></blockquote><div><br></div><div>Sadly, I think copy-prop is one of the ones that will never go away since we need it for evaluating GLSL stuff that we have to be able to validate at link time.  That said, if the 500 seconds is coming because of channel expressions.... That one needs to die.<br></div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
Reviewed-by: Eric Anholt <<a href="mailto:eric@anholt.net">eric@anholt.net</a>><br>
<br>______________________________<wbr>_________________<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/<wbr>mailman/listinfo/mesa-dev</a><br>
<br></blockquote></div><br></div></div>