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

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
This commit is contained in:
Ramkumar Ramachandra 2015-02-09 19:49:54 +00:00
parent bc4ab6ee7b
commit 010b77c3a2
1 changed files with 8 additions and 9 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

17
llvm/lib/Analysis/MemDepPrinter.cpp
View File

@ -96,8 +96,8 @@ bool MemDepPrinter::runOnFunction(Function &F) {
// All this code uses non-const interfaces because MemDep is not // All this code uses non-const interfaces because MemDep is not
// const-friendly, though nothing is actually modified. // const-friendly, though nothing is actually modified.
for (inst_iterator I = inst_begin(F), E = inst_end(F); I != E; ++I) { for (auto &I: inst_range(F)) {
Instruction *Inst = &*I; Instruction *Inst = &I;
if (!Inst->mayReadFromMemory() && !Inst->mayWriteToMemory()) if (!Inst->mayReadFromMemory() && !Inst->mayWriteToMemory())
continue; continue;
@ -135,8 +135,8 @@ bool MemDepPrinter::runOnFunction(Function &F) {
} }
void MemDepPrinter::print(raw_ostream &OS, const Module *M) const { 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) { for (auto &I: inst_range(*F)) {
const Instruction *Inst = &*I; const Instruction *Inst = &I;
DepSetMap::const_iterator DI = Deps.find(Inst); DepSetMap::const_iterator DI = Deps.find(Inst);
if (DI == Deps.end()) if (DI == Deps.end())
@ -144,11 +144,10 @@ void MemDepPrinter::print(raw_ostream &OS, const Module *M) const {
const DepSet &InstDeps = DI->second; const DepSet &InstDeps = DI->second;
for (DepSet::const_iterator I = InstDeps.begin(), E = InstDeps.end(); for (auto &I: InstDeps) {
I != E; ++I) { const Instruction *DepInst = I.first.getPointer();
const Instruction *DepInst = I->first.getPointer(); DepType type = I.first.getInt();
DepType type = I->first.getInt(); const BasicBlock *DepBB = I.second;
const BasicBlock *DepBB = I->second;
OS << " "; OS << " ";
OS << DepTypeStr[type]; OS << DepTypeStr[type];