[DAG] visitREM - merge buildOptimizedSREM into if(). NFCI.
authorSimon Pilgrim <llvm-dev@redking.me.uk>
Fri, 6 May 2022 14:17:00 +0000 (15:17 +0100)
committerSimon Pilgrim <llvm-dev@redking.me.uk>
Fri, 6 May 2022 14:39:17 +0000 (15:39 +0100)
llvm/lib/CodeGen/SelectionDAG/DAGCombiner.cpp

index 711ead2..a2fd45b 100644 (file)
@@ -4579,10 +4579,10 @@ SDValue DAGCombiner::visitREM(SDNode *N) {
   if (DAG.isKnownNeverZero(N1) && !TLI.isIntDivCheap(VT, Attr)) {
     if (isSigned) {
       // check if we can build faster implementation for srem
-      SDValue OptimizedRem = buildOptimizedSREM(N0, N1, N);
-      if (OptimizedRem.getNode())
+      if (SDValue OptimizedRem = buildOptimizedSREM(N0, N1, N))
         return OptimizedRem;
     }
+
     SDValue OptimizedDiv =
         isSigned ? visitSDIVLike(N0, N1, N) : visitUDIVLike(N0, N1, N);
     if (OptimizedDiv.getNode() && OptimizedDiv.getNode() != N) {