maxjhandsome
/
llvm-project
forked from OSchip/llvm-project
1
0
Fork 0
Code Issues Pull Requests Packages Releases Wiki Activity

Add brackets to remove warnings about ambiguous 'else's. 

llvm-svn: 367740
This commit is contained in:
Bill Wendling 2019-08-03 00:26:10 +00:00
parent c835164a47
commit 383650c7f1
1 changed files with 10 additions and 5 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

15
llvm/unittests/Support/AlignmentTest.cpp
View File

@ -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);
}
}
}