replica
/
hanchenye-llvm-project
2
0
Fork 0
Code Issues Pull Requests Projects Releases Wiki Activity

[Coverage] Simplify r316141. NFC. 

llvm-svn: 316147
This commit is contained in:
Vedant Kumar 2017-10-19 06:16:23 +00:00
parent 3612d4b4f9
commit f5f153dd6f
2 changed files with 9 additions and 44 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
llvm/lib/ProfileData/Coverage/CoverageMapping.cpp
View File

@ -672,13 +672,13 @@ CoverageData CoverageMapping::getCoverageForExpansion(
}
LineCoverageStats::LineCoverageStats(
ArrayRef<const coverage::CoverageSegment *> LineSegments,
const coverage::CoverageSegment *WrappedSegment, unsigned Line)
ArrayRef<const CoverageSegment *> LineSegments,
const CoverageSegment *WrappedSegment, unsigned Line)
: ExecutionCount(0), HasMultipleRegions(false), Mapped(false), Line(Line),
LineSegments(LineSegments), WrappedSegment(WrappedSegment) {
// Find the minimum number of regions which start in this line.
unsigned MinRegionCount = 0;
auto isStartOfRegion = [](const coverage::CoverageSegment *S) {
auto isStartOfRegion = [](const CoverageSegment *S) {
return !S->IsGapRegion && S->HasCount && S->IsRegionEntry;
};
for (unsigned I = 0; I < LineSegments.size() && MinRegionCount < 2; ++I)

47
llvm/unittests/ProfileData/CoverageMappingTest.cpp
View File

@ -647,49 +647,14 @@ TEST_P(CoverageMappingTest, test_line_coverage_iterator) {
CoverageData Data = LoadedCoverage->getCoverageForFile("file1");
unsigned NumLineStats = 0;
unsigned Line = 0;
unsigned LineCounts[] = {20, 20, 20, 20, 10, 10, 10, 10, 10, 0, 0};
for (const auto &LCS : getLineCoverageStats(Data)) {
++NumLineStats;
(void)LCS;
ASSERT_EQ(Line + 1, LCS.getLine());
ASSERT_EQ(LineCounts[Line], LCS.getExecutionCount());
++Line;
}
ASSERT_EQ(11U, NumLineStats);
LineCoverageIterator LCI{Data};
ASSERT_EQ(1U, LCI->getLine());
ASSERT_EQ(20ULL, LCI->getExecutionCount());
++LCI;
ASSERT_EQ(2U, LCI->getLine());
ASSERT_EQ(20ULL, LCI->getExecutionCount());
++LCI;
ASSERT_EQ(3U, LCI->getLine());
ASSERT_EQ(20ULL, LCI->getExecutionCount());
++LCI;
ASSERT_EQ(4U, LCI->getLine());
ASSERT_EQ(20ULL, LCI->getExecutionCount());
++LCI;
ASSERT_EQ(5U, LCI->getLine());
ASSERT_EQ(10ULL, LCI->getExecutionCount());
++LCI;
ASSERT_EQ(6U, LCI->getLine());
ASSERT_EQ(10ULL, LCI->getExecutionCount());
++LCI;
ASSERT_EQ(7U, LCI->getLine());
ASSERT_EQ(10ULL, LCI->getExecutionCount());
++LCI;
ASSERT_EQ(8U, LCI->getLine());
ASSERT_EQ(10ULL, LCI->getExecutionCount());
++LCI;
ASSERT_EQ(9U, LCI->getLine());
ASSERT_EQ(10ULL, LCI->getExecutionCount());
++LCI;
ASSERT_EQ(10U, LCI->getLine());
ASSERT_EQ(0ULL, LCI->getExecutionCount());
++LCI;
ASSERT_EQ(11U, LCI->getLine());
ASSERT_EQ(0ULL, LCI->getExecutionCount());
++LCI;
ASSERT_EQ(LCI, LCI.getEnd());
ASSERT_EQ(11U, Line);
}
TEST_P(CoverageMappingTest, uncovered_function) {