[llvm] r268998 - [SCEVExpander] Clang format expressions; NFC

Sanjoy Das via llvm-commits llvm-commits at lists.llvm.org
Mon May 9 17:32:31 PDT 2016


Author: sanjoy
Date: Mon May  9 19:32:31 2016
New Revision: 268998

URL: http://llvm.org/viewvc/llvm-project?rev=268998&view=rev
Log:
[SCEVExpander] Clang format expressions; NFC

The boolean expressions are somewhat hard to read otherwise.

Modified:
    llvm/trunk/lib/Analysis/ScalarEvolutionExpander.cpp

Modified: llvm/trunk/lib/Analysis/ScalarEvolutionExpander.cpp
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/Analysis/ScalarEvolutionExpander.cpp?rev=268998&r1=268997&r2=268998&view=diff
==============================================================================
--- llvm/trunk/lib/Analysis/ScalarEvolutionExpander.cpp (original)
+++ llvm/trunk/lib/Analysis/ScalarEvolutionExpander.cpp Mon May  9 19:32:31 2016
@@ -1777,8 +1777,8 @@ unsigned SCEVExpander::replaceCongruentI
     PHINode *&OrigPhiRef = ExprToIVMap[SE.getSCEV(Phi)];
     if (!OrigPhiRef) {
       OrigPhiRef = Phi;
-      if (Phi->getType()->isIntegerTy() && TTI
-          && TTI->isTruncateFree(Phi->getType(), Phis.back()->getType())) {
+      if (Phi->getType()->isIntegerTy() && TTI &&
+          TTI->isTruncateFree(Phi->getType(), Phis.back()->getType())) {
         // This phi can be freely truncated to the narrowest phi type. Map the
         // truncated expression to it so it will be reused for narrow types.
         const SCEV *TruncExpr =
@@ -1802,11 +1802,11 @@ unsigned SCEVExpander::replaceCongruentI
       // If this phi has the same width but is more canonical, replace the
       // original with it. As part of the "more canonical" determination,
       // respect a prior decision to use an IV chain.
-      if (OrigPhiRef->getType() == Phi->getType()
-          && !(ChainedPhis.count(Phi)
-               || isExpandedAddRecExprPHI(OrigPhiRef, OrigInc, L))
-          && (ChainedPhis.count(Phi)
-              || isExpandedAddRecExprPHI(Phi, IsomorphicInc, L))) {
+      if (OrigPhiRef->getType() == Phi->getType() &&
+          !(ChainedPhis.count(Phi) ||
+            isExpandedAddRecExprPHI(OrigPhiRef, OrigInc, L)) &&
+          (ChainedPhis.count(Phi) ||
+           isExpandedAddRecExprPHI(Phi, IsomorphicInc, L))) {
         std::swap(OrigPhiRef, Phi);
         std::swap(OrigInc, IsomorphicInc);
       }
@@ -1818,14 +1818,13 @@ unsigned SCEVExpander::replaceCongruentI
       // cycles that had postinc uses.
       const SCEV *TruncExpr = SE.getTruncateOrNoop(SE.getSCEV(OrigInc),
                                                    IsomorphicInc->getType());
-      if (OrigInc != IsomorphicInc
-          && TruncExpr == SE.getSCEV(IsomorphicInc)
-          && SE.LI.replacementPreservesLCSSAForm(IsomorphicInc, OrigInc)
-          && ((isa<PHINode>(OrigInc) && isa<PHINode>(IsomorphicInc))
-              || hoistIVInc(OrigInc, IsomorphicInc))) {
-        DEBUG_WITH_TYPE(DebugType, dbgs()
-                        << "INDVARS: Eliminated congruent iv.inc: "
-                        << *IsomorphicInc << '\n');
+      if (OrigInc != IsomorphicInc && TruncExpr == SE.getSCEV(IsomorphicInc) &&
+          SE.LI.replacementPreservesLCSSAForm(IsomorphicInc, OrigInc) &&
+          ((isa<PHINode>(OrigInc) && isa<PHINode>(IsomorphicInc)) ||
+           hoistIVInc(OrigInc, IsomorphicInc))) {
+        DEBUG_WITH_TYPE(DebugType,
+                        dbgs() << "INDVARS: Eliminated congruent iv.inc: "
+                               << *IsomorphicInc << '\n');
         Value *NewInc = OrigInc;
         if (OrigInc->getType() != IsomorphicInc->getType()) {
           Instruction *IP = nullptr;
@@ -1843,8 +1842,8 @@ unsigned SCEVExpander::replaceCongruentI
         DeadInsts.emplace_back(IsomorphicInc);
       }
     }
-    DEBUG_WITH_TYPE(DebugType, dbgs()
-                    << "INDVARS: Eliminated congruent iv: " << *Phi << '\n');
+    DEBUG_WITH_TYPE(DebugType, dbgs() << "INDVARS: Eliminated congruent iv: "
+                                      << *Phi << '\n');
     ++NumElim;
     Value *NewIV = OrigPhiRef;
     if (OrigPhiRef->getType() != Phi->getType()) {




More information about the llvm-commits mailing list