6502/llvm-6502
1
0
Fork 0
mirror of https://github.com/c64scene-ar/llvm-6502.git synced 2025-02-22 13:29:44 +00:00
Code Issues Projects Releases Wiki Activity

Let MemoryDependenceAnalysis take care of updating AliasAnalysis.

Browse Source 
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@39769 91177308-0d34-0410-b5e6-96231b3b80d8
This commit is contained in:
Owen Anderson 2007-07-12 00:06:21 +00:00
parent 680171f210
commit 1cb960a4bb
2 changed files with 2 additions and 5 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
lib/Analysis/MemoryDependenceAnalysis.cpp
View File

@ -255,4 +255,6 @@ void MemoryDependenceAnalysis::removeInstruction(Instruction* rem) {
reverseDep.erase(I); reverseDep.erase(I);
I = reverseDep.find(rem); I = reverseDep.find(rem);
} }
getAnalysis<AliasAnalysis>().deleteValue(rem);
} }

5
lib/Transforms/Scalar/FastDSE.cpp
View File

@ -70,7 +70,6 @@ namespace {
FunctionPass *llvm::createFastDeadStoreEliminationPass() { return new FDSE(); } FunctionPass *llvm::createFastDeadStoreEliminationPass() { return new FDSE(); }
bool FDSE::runOnBasicBlock(BasicBlock &BB) { bool FDSE::runOnBasicBlock(BasicBlock &BB) {
AliasAnalysis &AA = getAnalysis<AliasAnalysis>();
MemoryDependenceAnalysis& MD = getAnalysis<MemoryDependenceAnalysis>(); MemoryDependenceAnalysis& MD = getAnalysis<MemoryDependenceAnalysis>();
// Record the last-seen store to this pointer // Record the last-seen store to this pointer
@ -101,7 +100,6 @@ bool FDSE::runOnBasicBlock(BasicBlock &BB) {
// Remove it! // Remove it!
MD.removeInstruction(last); MD.removeInstruction(last);
AA.deleteValue(last);
// DCE instructions only used to calculate that store // DCE instructions only used to calculate that store
if (Instruction* D = dyn_cast<Instruction>(last->getOperand(0))) if (Instruction* D = dyn_cast<Instruction>(last->getOperand(0)))
@ -157,7 +155,6 @@ bool FDSE::handleFreeWithNonTrivialDependency(FreeInst* F, StoreInst* dependency
if (A == AliasAnalysis::MustAlias) { if (A == AliasAnalysis::MustAlias) {
// Remove it! // Remove it!
MD.removeInstruction(dependency); MD.removeInstruction(dependency);
AA.deleteValue(dependency);
// DCE instructions only used to calculate that store // DCE instructions only used to calculate that store
if (Instruction* D = dyn_cast<Instruction>(dependency->getOperand(0))) if (Instruction* D = dyn_cast<Instruction>(dependency->getOperand(0)))
@ -200,7 +197,6 @@ bool FDSE::handleEndBlock(BasicBlock& BB, SetVector<Instruction*>& possiblyDead)
if (deadPointers.count(S->getPointerOperand())){ if (deadPointers.count(S->getPointerOperand())){
// Remove it! // Remove it!
MD.removeInstruction(S); MD.removeInstruction(S);
AA.deleteValue(S);
// DCE instructions only used to calculate that store // DCE instructions only used to calculate that store
if (Instruction* D = dyn_cast<Instruction>(S->getOperand(0))) if (Instruction* D = dyn_cast<Instruction>(S->getOperand(0)))
@ -230,7 +226,6 @@ void FDSE::DeleteDeadInstructionChains(Instruction *I,
// Let the memory dependence know // Let the memory dependence know
getAnalysis<MemoryDependenceAnalysis>().removeInstruction(I); getAnalysis<MemoryDependenceAnalysis>().removeInstruction(I);
getAnalysis<AliasAnalysis>().deleteValue(I);
// See if this made any operands dead. We do it this way in case the // See if this made any operands dead. We do it this way in case the
// instruction uses the same operand twice. We don't want to delete a // instruction uses the same operand twice. We don't want to delete a