<div dir="ltr">What Danny said here.<div><br></div><div>It feels like this patch is papering over failure to canonicalize. To make progress we need to either:</div><div><br></div><div>a) Show a super clear example where there *is no canonicalization*. That is, one usage wants one canonicalization and the other wants the other, so we have to support both. If you can show that we cannot canonicalize this because it is in tension with other canonicalization, that is exactly the motivation that folks have asked for to land this patch.</div><div><br></div><div>or...</div><div><br></div><div>b) Fix everything to canonicalize so this patch isn't necessary.</div></div><br><div class="gmail_quote"><div dir="ltr">On Wed, Apr 27, 2016 at 11:26 PM Daniel Berlin via llvm-commits <<a href="mailto:llvm-commits@lists.llvm.org">llvm-commits@lists.llvm.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Chandler explicitly requested we take a different approach, and while I could overrule him, not sure it's worth it for this patch.<div>I think we have spent more time on this thank it would take to track down every place generating it that ends up needing to be swapped and fixing it at the source</div><div><br><br><div class="gmail_quote"><div dir="ltr">On Wed, Apr 27, 2016, 12:23 PM Aditya Kumar <<a href="mailto:hiraditya@msn.com" target="_blank">hiraditya@msn.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">hiraditya added a comment.<br>
<br>
@dberlin<br>
Please approve the patch if you are fine with it.<br>
<br>
Thanks,<br>
<br>
<br>
Repository:<br>
  rL LLVM<br>
<br>
<a href="http://reviews.llvm.org/D18830" rel="noreferrer" target="_blank">http://reviews.llvm.org/D18830</a><br>
<br>
<br>
<br>
</blockquote></div></div>
_______________________________________________<br>
llvm-commits mailing list<br>
<a href="mailto:llvm-commits@lists.llvm.org" target="_blank">llvm-commits@lists.llvm.org</a><br>
<a href="http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-commits" rel="noreferrer" target="_blank">http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-commits</a><br>
</blockquote></div>