1
0
mirror of https://github.com/RPCS3/llvm-mirror.git synced 2024-10-20 19:42:54 +02:00

Let MemoryDependenceAnalysis take care of updating AliasAnalysis.

llvm-svn: 39769
This commit is contained in:
Owen Anderson 2007-07-12 00:06:21 +00:00
parent 9a299aa63a
commit b6211f5b50
2 changed files with 2 additions and 5 deletions

View File

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

View File

@ -70,7 +70,6 @@ namespace {
FunctionPass *llvm::createFastDeadStoreEliminationPass() { return new FDSE(); }
bool FDSE::runOnBasicBlock(BasicBlock &BB) {
AliasAnalysis &AA = getAnalysis<AliasAnalysis>();
MemoryDependenceAnalysis& MD = getAnalysis<MemoryDependenceAnalysis>();
// Record the last-seen store to this pointer
@ -101,7 +100,6 @@ bool FDSE::runOnBasicBlock(BasicBlock &BB) {
// Remove it!
MD.removeInstruction(last);
AA.deleteValue(last);
// DCE instructions only used to calculate that store
if (Instruction* D = dyn_cast<Instruction>(last->getOperand(0)))
@ -157,7 +155,6 @@ bool FDSE::handleFreeWithNonTrivialDependency(FreeInst* F, StoreInst* dependency
if (A == AliasAnalysis::MustAlias) {
// Remove it!
MD.removeInstruction(dependency);
AA.deleteValue(dependency);
// DCE instructions only used to calculate that store
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())){
// Remove it!
MD.removeInstruction(S);
AA.deleteValue(S);
// DCE instructions only used to calculate that store
if (Instruction* D = dyn_cast<Instruction>(S->getOperand(0)))
@ -230,7 +226,6 @@ void FDSE::DeleteDeadInstructionChains(Instruction *I,
// Let the memory dependence know
getAnalysis<MemoryDependenceAnalysis>().removeInstruction(I);
getAnalysis<AliasAnalysis>().deleteValue(I);
// 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