diff options
| author | Gabor Greif <ggreif@gmail.com> | 2009-01-23 19:40:15 +0000 |
|---|---|---|
| committer | Gabor Greif <ggreif@gmail.com> | 2009-01-23 19:40:15 +0000 |
| commit | eb61fcf2a1f949b70264c6a86ea7559f7ac3ed76 (patch) | |
| tree | ea7b87b3cff62ed43ff01bfb64872ab6ab866431 /llvm/lib/Transforms/Utils/LCSSA.cpp | |
| parent | 617001d84279f075a09e236f7002d58572c1192b (diff) | |
| download | bcm5719-llvm-eb61fcf2a1f949b70264c6a86ea7559f7ac3ed76.tar.gz bcm5719-llvm-eb61fcf2a1f949b70264c6a86ea7559f7ac3ed76.zip | |
Simplify the logic of getting hold of a PHI predecessor block.
There is now a direct way from value-use-iterator to incoming block in PHINode's API.
This way we avoid the iterator->index->iterator trip, and especially the costly
getOperandNo() invocation. Additionally there is now an assertion that the iterator
really refers to one of the PHI's Uses.
llvm-svn: 62869
Diffstat (limited to 'llvm/lib/Transforms/Utils/LCSSA.cpp')
| -rw-r--r-- | llvm/lib/Transforms/Utils/LCSSA.cpp | 6 |
1 files changed, 2 insertions, 4 deletions
diff --git a/llvm/lib/Transforms/Utils/LCSSA.cpp b/llvm/lib/Transforms/Utils/LCSSA.cpp index 38558886115..9cd7e69557a 100644 --- a/llvm/lib/Transforms/Utils/LCSSA.cpp +++ b/llvm/lib/Transforms/Utils/LCSSA.cpp @@ -175,8 +175,7 @@ void LCSSA::ProcessInstruction(Instruction *Instr, UI != E;) { BasicBlock *UserBB = cast<Instruction>(*UI)->getParent(); if (PHINode *P = dyn_cast<PHINode>(*UI)) { - unsigned OperandNo = UI.getOperandNo(); - UserBB = P->getIncomingBlock(OperandNo/2); + UserBB = P->getIncomingBlock(UI); } // If the user is in the loop, don't rewrite it! @@ -212,8 +211,7 @@ void LCSSA::getLoopValuesUsedOutsideLoop(Loop *L, ++UI) { BasicBlock *UserBB = cast<Instruction>(*UI)->getParent(); if (PHINode* p = dyn_cast<PHINode>(*UI)) { - unsigned OperandNo = UI.getOperandNo(); - UserBB = p->getIncomingBlock(OperandNo/2); + UserBB = p->getIncomingBlock(UI); } if (*BB != UserBB && !inLoop(UserBB)) { |
