[Mlir-commits] [mlir] d6f8f56 - [MLIR][Presburger] Silence -Wdangling-else warning (NFC)

Lorenzo Chelini llvmlistbot at llvm.org
Mon Jan 24 00:16:21 PST 2022


Author: Lorenzo Chelini
Date: 2022-01-24T09:16:05+01:00
New Revision: d6f8f56da04b2e975110c4a5ae15f00a22164e62

URL: https://github.com/llvm/llvm-project/commit/d6f8f56da04b2e975110c4a5ae15f00a22164e62
DIFF: https://github.com/llvm/llvm-project/commit/d6f8f56da04b2e975110c4a5ae15f00a22164e62.diff

LOG: [MLIR][Presburger] Silence -Wdangling-else warning (NFC)

gcc suggests explicit braces to avoid ambiguous ‘else’.

Added: 
    

Modified: 
    mlir/unittests/Analysis/Presburger/IntegerPolyhedronTest.cpp

Removed: 
    


################################################################################
diff  --git a/mlir/unittests/Analysis/Presburger/IntegerPolyhedronTest.cpp b/mlir/unittests/Analysis/Presburger/IntegerPolyhedronTest.cpp
index 5d1cfb4c6e781..cf68c92650548 100644
--- a/mlir/unittests/Analysis/Presburger/IntegerPolyhedronTest.cpp
+++ b/mlir/unittests/Analysis/Presburger/IntegerPolyhedronTest.cpp
@@ -636,8 +636,9 @@ static void checkDivisionRepresentation(
   // denominator for a division is zero, we ignore its dividend.
   EXPECT_TRUE(dividends.size() == expectedDividends.size());
   for (unsigned i = 0, e = dividends.size(); i < e; ++i)
-    if (denominators[i] != 0)
+    if (denominators[i] != 0) {
       EXPECT_TRUE(expectedDividends[i] == dividends[i]);
+    }
 }
 
 TEST(IntegerPolyhedronTest, computeLocalReprSimple) {


        


More information about the Mlir-commits mailing list