<div dir="ltr">Hi,<br><br><div>It sounds like the best time to do this if at all possible is in MachineInst form, where we have MachineTraceMetrics and accurate register pressure information.</div><div><br></div><div>Is there a reason why you're not doing it there? I know it's more awkward, but it really seems we need accurate register pressure if we're not going to make it cripplingly conservative.</div><div><br></div><div>Cheers,</div><div><br></div><div>James</div></div><br><div class="gmail_quote">On Thu, 14 May 2015 at 21:17 Fiona Glaser <<a href="mailto:escha@apple.com">escha@apple.com</a>> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Could you help me try to understand what purpose this serves in terms of complementing the already-existing reassociate pass? That is, what benefit comes from doing it in the DAG in addition to the existing reassociate IR pass?<br>
<br>
<br>
<a href="http://reviews.llvm.org/D9780" target="_blank">http://reviews.llvm.org/D9780</a><br>
<br>
EMAIL PREFERENCES<br>
  <a href="http://reviews.llvm.org/settings/panel/emailpreferences/" target="_blank">http://reviews.llvm.org/settings/panel/emailpreferences/</a><br>
<br>
<br>
<br>
_______________________________________________<br>
llvm-commits mailing list<br>
<a href="mailto:llvm-commits@cs.uiuc.edu" target="_blank">llvm-commits@cs.uiuc.edu</a><br>
<a href="http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits" target="_blank">http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits</a><br>
</blockquote></div>