[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