mirror of
https://github.com/c64scene-ar/llvm-6502.git
synced 2024-08-19 19:29:31 +00:00
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. git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@15030 91177308-0d34-0410-b5e6-96231b3b80d8
This commit is contained in:
parent
59a20773f8
commit
adfd32f8ee
@ -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.
|
||||
|
Loading…
Reference in New Issue
Block a user