make clients have to know how to call getCallSiteDependencyFrom

instead of making getDependencyFrom do it.


git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@60647 91177308-0d34-0410-b5e6-96231b3b80d8
This commit is contained in:
Chris Lattner 2008-12-07 01:21:14 +00:00
parent 106c6ca7b0
commit d8dd934d16

View File

@ -139,6 +139,7 @@ getDependencyFrom(Instruction *QueryInst, BasicBlock::iterator ScanIt,
// FreeInsts erase the entire structure, not just a field.
MemSize = ~0UL;
} else if (isa<CallInst>(QueryInst) || isa<InvokeInst>(QueryInst)) {
assert(0 && "Should use getCallSiteDependencyFrom!");
return getCallSiteDependencyFrom(CallSite::get(QueryInst), ScanIt, BB);
} else {
// Otherwise, this is a vaarg or non-memory instruction, just return a
@ -233,7 +234,11 @@ MemDepResult MemoryDependenceAnalysis::getDependency(Instruction *QueryInst) {
}
// Do the scan.
LocalCache = getDependencyFrom(QueryInst, ScanPos, QueryInst->getParent());
if (!isa<CallInst>(QueryInst) && !isa<InvokeInst>(QueryInst))
LocalCache = getDependencyFrom(QueryInst, ScanPos, QueryInst->getParent());
else
LocalCache = getCallSiteDependencyFrom(CallSite::get(QueryInst), ScanPos,
QueryInst->getParent());
// Remember the result!
if (Instruction *I = LocalCache.getInst())
@ -341,7 +346,12 @@ MemoryDependenceAnalysis::getNonLocalDependency(Instruction *QueryInst) {
}
// Find out if this block has a local dependency for QueryInst.
MemDepResult Dep = getDependencyFrom(QueryInst, ScanPos, DirtyBB);
MemDepResult Dep;
if (!isa<CallInst>(QueryInst) && !isa<InvokeInst>(QueryInst))
Dep = getDependencyFrom(QueryInst, ScanPos, DirtyBB);
else
Dep = getCallSiteDependencyFrom(CallSite::get(QueryInst), ScanPos,
DirtyBB);
// If we had a dirty entry for the block, update it. Otherwise, just add
// a new entry.
@ -368,6 +378,7 @@ MemoryDependenceAnalysis::getNonLocalDependency(Instruction *QueryInst) {
return Cache;
}
/// removeInstruction - Remove an instruction from the dependence analysis,
/// updating the dependence of instructions that previously depended on it.
/// This method attempts to keep the cache coherent using the reverse map.