[llvm] r367740 - Add brackets to remove warnings about ambiguous 'else's.

Bill Wendling via llvm-commits llvm-commits at lists.llvm.org
Fri Aug 2 17:26:10 PDT 2019


Author: void
Date: Fri Aug  2 17:26:10 2019
New Revision: 367740

URL: http://llvm.org/viewvc/llvm-project?rev=367740&view=rev
Log:
Add brackets to remove warnings about ambiguous 'else's.

Modified:
    llvm/trunk/unittests/Support/AlignmentTest.cpp

Modified: llvm/trunk/unittests/Support/AlignmentTest.cpp
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/unittests/Support/AlignmentTest.cpp?rev=367740&r1=367739&r2=367740&view=diff
==============================================================================
--- llvm/trunk/unittests/Support/AlignmentTest.cpp (original)
+++ llvm/trunk/unittests/Support/AlignmentTest.cpp Fri Aug  2 17:26:10 2019
@@ -83,8 +83,9 @@ TEST(AlignmentTest, AlignTo) {
     // Test MaybeAlign
     EXPECT_EQ(alignTo(T.offset, A), T.rounded);
     // Test Align
-    if (A)
+    if (A) {
       EXPECT_EQ(alignTo(T.offset, A.getValue()), T.rounded);
+    }
   }
 }
 
@@ -112,12 +113,15 @@ TEST(AlignmentTest, MinAlign) {
   for (const auto &T : kTests) {
     EXPECT_EQ(commonAlignment(MaybeAlign(T.A), MaybeAlign(T.B)), T.MinAlign);
     EXPECT_EQ(MinAlign(T.A, T.B), T.MinAlign);
-    if (T.A)
+    if (T.A) {
       EXPECT_EQ(commonAlignment(Align(T.A), MaybeAlign(T.B)), T.MinAlign);
-    if (T.B)
+    }
+    if (T.B) {
       EXPECT_EQ(commonAlignment(MaybeAlign(T.A), Align(T.B)), T.MinAlign);
-    if (T.A && T.B)
+    }
+    if (T.A && T.B) {
       EXPECT_EQ(commonAlignment(Align(T.A), Align(T.B)), T.MinAlign);
+    }
   }
 }
 
@@ -155,8 +159,9 @@ TEST(AlignmentTest, isAligned) {
     // Test MaybeAlign
     EXPECT_EQ(isAligned(A, T.offset), T.isAligned);
     // Test Align
-    if (A)
+    if (A) {
       EXPECT_EQ(isAligned(A.getValue(), T.offset), T.isAligned);
+    }
   }
 }
 




More information about the llvm-commits mailing list