[DAGCombiner] Remove unnecessary check for visitSDIVLike/visitUDIVLike returning...
authorCraig Topper <craig.topper@intel.com>
Tue, 25 Sep 2018 23:52:07 +0000 (23:52 +0000)
committerCraig Topper <craig.topper@intel.com>
Tue, 25 Sep 2018 23:52:07 +0000 (23:52 +0000)
This shouldn't be possible and is a leftover from when we used to recursively call combine here.

llvm-svn: 343049

llvm/lib/CodeGen/SelectionDAG/DAGCombiner.cpp

index fe3dc39fa09ac42fbd4c873e7677bf9ec87d2193..ad8121babe4038043211da7140b7432128a1a47a 100644 (file)
@@ -3424,8 +3424,7 @@ SDValue DAGCombiner::visitREM(SDNode *N) {
   if (DAG.isKnownNeverZero(N1) && !TLI.isIntDivCheap(VT, Attr)) {
     SDValue OptimizedDiv =
         isSigned ? visitSDIVLike(N0, N1, N) : visitUDIVLike(N0, N1, N);
-    if (OptimizedDiv.getNode() && OptimizedDiv.getOpcode() != ISD::UDIVREM &&
-        OptimizedDiv.getOpcode() != ISD::SDIVREM) {
+    if (OptimizedDiv.getNode()) {
       SDValue Mul = DAG.getNode(ISD::MUL, DL, VT, OptimizedDiv, N1);
       SDValue Sub = DAG.getNode(ISD::SUB, DL, VT, N0, Mul);
       AddToWorklist(OptimizedDiv.getNode());