mirror of
https://github.com/c64scene-ar/llvm-6502.git
synced 2024-12-14 11:32:34 +00:00
fix some bugs (found by inspection) where LICM would not update
LICM correctly. When sinking an instruction, it should not add entries for the sunk instruction to the AST, it should remove the entry for the sunk instruction. The blocks being sunk to are not in the loop, so their instructions shouldn't be in the AST (yet)! git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@112447 91177308-0d34-0410-b5e6-96231b3b80d8
This commit is contained in:
parent
4282e32712
commit
98917503c7
@ -530,7 +530,6 @@ void LICM::sink(Instruction &I) {
|
||||
New = &I;
|
||||
} else {
|
||||
New = I.clone();
|
||||
CurAST->copyValue(&I, New);
|
||||
if (!I.getName().empty())
|
||||
New->setName(I.getName()+".le");
|
||||
ExitBlock->getInstList().insert(InsertPt, New);
|
||||
@ -563,6 +562,9 @@ void LICM::sink(Instruction &I) {
|
||||
if (I.getType()->isPointerTy())
|
||||
for (unsigned i = 0, e = NewPHIs.size(); i != e; ++i)
|
||||
CurAST->copyValue(NewPHIs[i], &I);
|
||||
|
||||
// Finally, remove the instruction from CurAST. It is no longer in the loop.
|
||||
CurAST->deleteValue(&I);
|
||||
}
|
||||
|
||||
/// hoist - When an instruction is found to only use loop invariant operands
|
||||
|
Loading…
Reference in New Issue
Block a user