[PATCH] D18830: [llvm] GVN.cpp: Do not swap when both LHS and RHS are arguments.

Daniel Berlin via llvm-commits llvm-commits at lists.llvm.org
Wed Apr 27 20:26:35 PDT 2016


Chandler explicitly requested we take a different approach, and while I
could overrule him, not sure it's worth it for this patch.
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

On Wed, Apr 27, 2016, 12:23 PM Aditya Kumar <hiraditya at msn.com> wrote:

> hiraditya added a comment.
>
> @dberlin
> Please approve the patch if you are fine with it.
>
> Thanks,
>
>
> Repository:
>   rL LLVM
>
> http://reviews.llvm.org/D18830
>
>
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20160428/38c79727/attachment.html>


More information about the llvm-commits mailing list