[PATCH] D13724: Merge DAGCombiner::visitSREM and DAGCombiner::visitUREM (NFC)

A. Skrobov via llvm-commits llvm-commits at lists.llvm.org
Wed Oct 14 08:48:15 PDT 2015


tyomitch added inline comments.

================
Comment at: lib/CodeGen/SelectionDAG/DAGCombiner.cpp:2350
@@ +2349,3 @@
+            DAG.getNode(ISD::ADD, DL, VT, N1,
+                   DAG.getConstant(APInt::getAllOnesValue(VT.getSizeInBits()),
+                                   DL, VT));
----------------
jmolloy wrote:
> This looks like an unneeded/unintended change?
Yes, you're right, I'll undo this one.


http://reviews.llvm.org/D13724





More information about the llvm-commits mailing list