[llvm] r281331 - Don't use else if after return. Tidy comments. NFC.

Chad Rosier via llvm-commits llvm-commits at lists.llvm.org
Tue Sep 13 06:08:54 PDT 2016


Author: mcrosier
Date: Tue Sep 13 08:08:53 2016
New Revision: 281331

URL: http://llvm.org/viewvc/llvm-project?rev=281331&view=rev
Log:
Don't use else if after return. Tidy comments. NFC.

Modified:
    llvm/trunk/lib/Transforms/Scalar/LoopInterchange.cpp

Modified: llvm/trunk/lib/Transforms/Scalar/LoopInterchange.cpp
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/Transforms/Scalar/LoopInterchange.cpp?rev=281331&r1=281330&r2=281331&view=diff
==============================================================================
--- llvm/trunk/lib/Transforms/Scalar/LoopInterchange.cpp (original)
+++ llvm/trunk/lib/Transforms/Scalar/LoopInterchange.cpp Tue Sep 13 08:08:53 2016
@@ -254,9 +254,8 @@ static bool validDepInterchange(CharMatr
 
 // Checks if it is legal to interchange 2 loops.
 // [Theorem] A permutation of the loops in a perfect nest is legal if and only
-// if
-// the direction matrix, after the same permutation is applied to its columns,
-// has no ">" direction as the leftmost non-"=" direction in any row.
+// if the direction matrix, after the same permutation is applied to its
+// columns, has no ">" direction as the leftmost non-"=" direction in any row.
 static bool isLegalToInterChangeLoops(CharMatrix &DepMatrix,
                                       unsigned InnerLoopId,
                                       unsigned OuterLoopId) {
@@ -268,8 +267,7 @@ static bool isLegalToInterChangeLoops(Ch
     char OuterDep = DepMatrix[Row][OuterLoopId];
     if (InnerDep == '*' || OuterDep == '*')
       return false;
-    else if (!validDepInterchange(DepMatrix, Row, OuterLoopId, InnerDep,
-                                  OuterDep))
+    if (!validDepInterchange(DepMatrix, Row, OuterLoopId, InnerDep, OuterDep))
       return false;
   }
   return true;




More information about the llvm-commits mailing list