diff options
author | Ramkumar Ramachandra <artagnon@gmail.com> | 2015-02-09 19:49:54 +0000 |
---|---|---|
committer | Ramkumar Ramachandra <artagnon@gmail.com> | 2015-02-09 19:49:54 +0000 |
commit | 010b77c3a2ee820f2b59bc11323b43088d62b7c6 (patch) | |
tree | 01c7c21bb21f7e595cb0a094364056b7399aa346 /llvm/lib/Analysis/MemDepPrinter.cpp | |
parent | bc4ab6ee7b2989cbf0ab85497465c0bab7e27754 (diff) | |
download | bcm5719-llvm-010b77c3a2ee820f2b59bc11323b43088d62b7c6.tar.gz bcm5719-llvm-010b77c3a2ee820f2b59bc11323b43088d62b7c6.zip |
MemDepPrinter: cleanup a few loops (NFC)
Make use of the newly introduced inst_range to clean up two loops. Clean
up a third one while at it.
Differential Revision: http://reviews.llvm.org/D7455
llvm-svn: 228596
Diffstat (limited to 'llvm/lib/Analysis/MemDepPrinter.cpp')
-rw-r--r-- | llvm/lib/Analysis/MemDepPrinter.cpp | 17 |
1 files changed, 8 insertions, 9 deletions
diff --git a/llvm/lib/Analysis/MemDepPrinter.cpp b/llvm/lib/Analysis/MemDepPrinter.cpp index ffc9fe64cc5..854197badfb 100644 --- a/llvm/lib/Analysis/MemDepPrinter.cpp +++ b/llvm/lib/Analysis/MemDepPrinter.cpp @@ -96,8 +96,8 @@ bool MemDepPrinter::runOnFunction(Function &F) { // All this code uses non-const interfaces because MemDep is not // const-friendly, though nothing is actually modified. - for (inst_iterator I = inst_begin(F), E = inst_end(F); I != E; ++I) { - Instruction *Inst = &*I; + for (auto &I: inst_range(F)) { + Instruction *Inst = &I; if (!Inst->mayReadFromMemory() && !Inst->mayWriteToMemory()) continue; @@ -135,8 +135,8 @@ bool MemDepPrinter::runOnFunction(Function &F) { } void MemDepPrinter::print(raw_ostream &OS, const Module *M) const { - for (const_inst_iterator I = inst_begin(*F), E = inst_end(*F); I != E; ++I) { - const Instruction *Inst = &*I; + for (auto &I: inst_range(*F)) { + const Instruction *Inst = &I; DepSetMap::const_iterator DI = Deps.find(Inst); if (DI == Deps.end()) @@ -144,11 +144,10 @@ void MemDepPrinter::print(raw_ostream &OS, const Module *M) const { const DepSet &InstDeps = DI->second; - for (DepSet::const_iterator I = InstDeps.begin(), E = InstDeps.end(); - I != E; ++I) { - const Instruction *DepInst = I->first.getPointer(); - DepType type = I->first.getInt(); - const BasicBlock *DepBB = I->second; + for (auto &I: InstDeps) { + const Instruction *DepInst = I.first.getPointer(); + DepType type = I.first.getInt(); + const BasicBlock *DepBB = I.second; OS << " "; OS << DepTypeStr[type]; |