[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