6502/llvm-6502
1
0
Fork 0
mirror of https://github.com/c64scene-ar/llvm-6502.git synced 2025-06-14 14:24:05 +00:00
Code Issues Projects Releases Wiki Activity

Improve encapsulation in the Loop and LoopInfo classes by eliminating the

Browse Source 
getSubLoops/getTopLevelLoops methods, replacing them with iterator-based
accessors.


git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@10714 91177308-0d34-0410-b5e6-96231b3b80d8
This commit is contained in:
Chris Lattner
2004-01-08 00:09:44 +00:00
parent f789d409fa
commit 329c1c6c94
5 changed files with 40 additions and 35 deletions
Download Patch File Download Diff File Expand all files Collapse all files

31
lib/Transforms/Utils/LoopSimplify.cpp
View File

@ -91,8 +91,8 @@ bool LoopSimplify::runOnFunction(Function &F) {
bool Changed = false;
LoopInfo &LI = getAnalysis<LoopInfo>();
for (unsigned i = 0, e = LI.getTopLevelLoops().size(); i != e; ++i)
Changed |= ProcessLoop(LI.getTopLevelLoops()[i]);
for (LoopInfo::iterator I = LI.begin(), E = LI.end(); I != E; ++I)
Changed |= ProcessLoop(*I);
return Changed;
}
@ -136,9 +136,8 @@ bool LoopSimplify::ProcessLoop(Loop *L) {
Changed = true;
}
const std::vector<Loop*> &SubLoops = L->getSubLoops();
for (unsigned i = 0, e = SubLoops.size(); i != e; ++i)
Changed |= ProcessLoop(SubLoops[i]);
for (Loop::iterator I = L->begin(), E = L->end(); I != E; ++I)
Changed |= ProcessLoop(*I);
return Changed;
}
@ -227,9 +226,8 @@ BasicBlock *LoopSimplify::SplitBlockPredecessors(BasicBlock *BB,
static void ChangeExitBlock(Loop *L, BasicBlock *OldExit, BasicBlock *NewExit) {
if (L->hasExitBlock(OldExit)) {
L->changeExitBlock(OldExit, NewExit);
const std::vector<Loop*> &SubLoops = L->getSubLoops();
for (unsigned i = 0, e = SubLoops.size(); i != e; ++i)
ChangeExitBlock(SubLoops[i], OldExit, NewExit);
for (Loop::iterator I = L->begin(), E = L->end(); I != E; ++I)
ChangeExitBlock(*I, OldExit, NewExit);
}
}
@ -266,16 +264,19 @@ void LoopSimplify::InsertPreheaderForLoop(Loop *L) {
// is a sibling loop, ie, one with the same parent loop, or one if it's
// children.
//
const std::vector<Loop*> *ParentSubLoops;
if (Loop *Parent = L->getParentLoop())
ParentSubLoops = &Parent->getSubLoops();
else // Must check top-level loops...
ParentSubLoops = &getAnalysis<LoopInfo>().getTopLevelLoops();
LoopInfo::iterator ParentLoops, ParentLoopsE;
if (Loop *Parent = L->getParentLoop()) {
ParentLoops = Parent->begin();
ParentLoopsE = Parent->end();
} else { // Must check top-level loops...
ParentLoops = getAnalysis<LoopInfo>().begin();
ParentLoopsE = getAnalysis<LoopInfo>().end();
}
// Loop over all sibling loops, performing the substitution (recursively to
// include child loops)...
for (unsigned i = 0, e = ParentSubLoops->size(); i != e; ++i)
ChangeExitBlock((*ParentSubLoops)[i], Header, NewBB);
for (; ParentLoops != ParentLoopsE; ++ParentLoops)
ChangeExitBlock(*ParentLoops, Header, NewBB);
DominatorSet &DS = getAnalysis<DominatorSet>(); // Update dominator info
{