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

Fix a serious code pessimization problem. If an inlined function has a single 

return, clone the 'ret' BB code into the block AFTER the inlined call, not the
other way around.

llvm-svn: 15030
This commit is contained in:
Chris Lattner 2004-07-20 05:45:24 +00:00
parent 11ffd59e37
commit 45b50d14c9
1 changed files with 6 additions and 5 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

11
llvm/lib/Transforms/Utils/InlineFunction.cpp
View File

@ -304,14 +304,15 @@ bool llvm::InlineFunction(CallSite CS) {
// Splice the code from the return block into the block that it will return
// to, which contains the code that was after the call.
BasicBlock *ReturnBB = Returns[0]->getParent();
ReturnBB->getInstList().splice(Returns[0], AfterCallBB->getInstList());
AfterCallBB->getInstList().splice(AfterCallBB->begin(),
ReturnBB->getInstList());
// Update PHI nodes that use the AfterCallBB to use the ReturnBB.
AfterCallBB->replaceAllUsesWith(ReturnBB);
// Update PHI nodes that use the ReturnBB to use the AfterCallBB.
ReturnBB->replaceAllUsesWith(AfterCallBB);
// Delete the return instruction now and empty AfterCallBB now.
// Delete the return instruction now and empty ReturnBB now.
Returns[0]->getParent()->getInstList().erase(Returns[0]);
Caller->getBasicBlockList().erase(AfterCallBB);
Caller->getBasicBlockList().erase(ReturnBB);
}
// Since we are now done with the Call/Invoke, we can delete it.