nicolas/llvm-project
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Use llvm::reverse to make a bunch of loops use foreach. NFC.

Browse Source 
In llvm commit r243581, a reverse range adapter was added which allows
us to change code such as

  for (auto I = Fields.rbegin(), E = Fields.rend(); I != E; ++I) {

in to

  for (const FieldDecl *I : llvm::reverse(Fields))

This commit changes a few of the places in clang which are eligible to use
this new adapter.

llvm-svn: 243663
This commit is contained in:
Pete Cooper
2015-07-30 17:22:52 +00:00
parent 7a0c3a92c0
commit 57d3f14502
9 changed files with 24 additions and 34 deletions
Download Patch File Download Diff File Expand all files Collapse all files

12
clang/lib/CodeGen/CoverageMappingGen.cpp
View File

@@ -496,12 +496,12 @@ struct CounterCoverageMappingBuilder
llvm::SmallSet<SourceLocation, 8> StartLocs;
Optional<Counter> ParentCounter;
for (auto I = RegionStack.rbegin(), E = RegionStack.rend(); I != E; ++I) {
if (!I->hasStartLoc())
for (SourceMappingRegion &I : llvm::reverse(RegionStack)) {
if (!I.hasStartLoc())
continue;
SourceLocation Loc = I->getStartLoc();
SourceLocation Loc = I.getStartLoc();
if (!isNestedIn(Loc, ParentFile)) {
ParentCounter = I->getCounter();
ParentCounter = I.getCounter();
break;
}
@@ -510,11 +510,11 @@ struct CounterCoverageMappingBuilder
// correct count. We avoid creating redundant regions by stopping once
// we've seen this region.
if (StartLocs.insert(Loc).second)
SourceRegions.emplace_back(I->getCounter(), Loc,
SourceRegions.emplace_back(I.getCounter(), Loc,
getEndOfFileOrMacro(Loc));
Loc = getIncludeOrExpansionLoc(Loc);
}
I->setStartLoc(getPreciseTokenLocEnd(Loc));
I.setStartLoc(getPreciseTokenLocEnd(Loc));
}
if (ParentCounter) {