https://llvm.org/bugs/show_bug.cgi?id=20134

Sanjay Patel <spatel+l...@rotateright.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|ASSIGNED                    |RESOLVED
         Resolution|---                         |FIXED

--- Comment #16 from Sanjay Patel <spatel+l...@rotateright.com> ---
Thanks for working on this, Duncan. 

For the record, I did review your patch and tried all kinds of related things (
details in http://reviews.llvm.org/D13757 ), but I settled on an x86 DAG
solution because it caught a lot of x86-specific cases with a simple patch. 

But in the long-term, the consensus is that it would be better to improve
CodeGenPrepare or generalize AddressTypePromotion to do more of these
transforms.

Fix committed here:
http://reviews.llvm.org/rL250560

-- 
You are receiving this mail because:
You are on the CC list for the bug.
_______________________________________________
llvm-bugs mailing list
llvm-bugs@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-bugs

Reply via email to