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

Use EXPECT_EQ in the unittests instead of plain assert 

This addresses post-review comments from Duncan P. N. Exon Smith to r261485.

llvm-svn: 261514
This commit is contained in:
Tobias Grosser 2016-02-22 07:20:40 +00:00
parent 452417ac12
commit 946ca0a946
1 changed files with 3 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

5
llvm/unittests/Analysis/ScalarEvolutionTest.cpp
View File

@ -255,12 +255,13 @@ TEST_F(ScalarEvolutionsTest, SimplifiedPHI) {
ScalarEvolution SE = buildSE(*F);
auto *S1 = SE.getSCEV(PN);
auto *S2 = SE.getSCEV(PN);
assert(isa<SCEVConstant>(S1) && "Expected a SCEV Constant");
auto *ZeroConst = SE.getConstant(Ty, 0);
// At some point, only the first call to getSCEV returned the simplified
// SCEVConstant and later calls just returned a SCEVUnknown referencing the
// PHI node.
assert(S1 == S2 && "Expected identical SCEV values");
EXPECT_EQ(S1, ZeroConst);
EXPECT_EQ(S1, S2);
}
} // end anonymous namespace