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

optimize LICM::hoist to use moveBefore. Correct its updating 

of AST to remove the hoisted instruction from the AST, since it
is no longer in the loop.

llvm-svn: 112448
This commit is contained in:
Chris Lattner 2010-08-29 18:03:33 +00:00
parent 1a1ed69435
commit 11f8ad8211
1 changed files with 5 additions and 6 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

9
llvm/lib/Transforms/Scalar/LICM.cpp
View File

@ -574,12 +574,11 @@ void LICM::hoist(Instruction &I) {
DEBUG(dbgs() << "LICM hoisting to " << Preheader->getName() << ": " DEBUG(dbgs() << "LICM hoisting to " << Preheader->getName() << ": "
<< I << "\n"); << I << "\n");
// Remove the instruction from its current basic block... but don't delete the // The instruction is no longer in this loop.
// instruction. CurAST->deleteValue(&I);
I.removeFromParent();
// Insert the new node in Preheader, before the terminator. // Move the new node to the Preheader, before its terminator.
Preheader->getInstList().insert(Preheader->getTerminator(), &I); I.moveBefore(Preheader->getTerminator());
if (isa<LoadInst>(I)) ++NumMovedLoads; if (isa<LoadInst>(I)) ++NumMovedLoads;
else if (isa<CallInst>(I)) ++NumMovedCalls; else if (isa<CallInst>(I)) ++NumMovedCalls;