[PATCH] X86ISelLowering has seemingly redundant code, remove it

David Majnemer david.majnemer at gmail.com
Wed May 1 19:19:57 PDT 2013


X86ISelLowering has support to treat:
(icmp ne (and (xor %flags, -1), (shl 1, flag)), 0)

as if it were actually:
(icmp eq (and %flags, (shl 1, flag)), 0)

However, r179386 has code at the InstCombine level to handle this.
This makes me believe that we can remove this target specific optimization.

-- 
David Majnemer
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20130501/b86e1681/attachment.html>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: r179386-supersedes-r169339.diff
Type: application/octet-stream
Size: 1643 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20130501/b86e1681/attachment.obj>


More information about the llvm-commits mailing list