[PATCH] D23000: [X86] Improve 64-bit shifts on 32-bit targets (PR14593)

Eli Friedman via llvm-commits llvm-commits at lists.llvm.org
Sun Jul 31 12:15:51 PDT 2016


eli.friedman accepted this revision.
eli.friedman added a comment.
This revision is now accepted and ready to land.

LGTM.


================
Comment at: lib/Target/X86/X86ISelLowering.cpp:28679
@@ +28678,3 @@
+        }
+        // Test for ADD( Y, Y ) as an equivalent to SHL( Y, 1 ).
+        if (InnerShift == ISD::SHL && Op1.getOpcode() == ISD::ADD &&
----------------
If DAGCombine doesn't transform Y+Y to Y<<1, it should.  (instcombine definitely does this.)  Not really important for now, though.


Repository:
  rL LLVM

https://reviews.llvm.org/D23000





More information about the llvm-commits mailing list