It's not necessary to clear out the Size and TBAATag at each of

these points.


git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@118752 91177308-0d34-0410-b5e6-96231b3b80d8
This commit is contained in:
Dan Gohman 2010-11-11 00:42:22 +00:00
parent ec9b4ac914
commit 8a66a202f6

View File

@ -858,11 +858,8 @@ getNonLocalPointerDepFromBB(const PHITransAddr &Pointer,
// otherwise it isn't. // otherwise it isn't.
if (Cache->empty()) if (Cache->empty())
CacheInfo->Pair = BBSkipFirstBlockPair(StartBB, SkipFirstBlock); CacheInfo->Pair = BBSkipFirstBlockPair(StartBB, SkipFirstBlock);
else { else
CacheInfo->Pair = BBSkipFirstBlockPair(); CacheInfo->Pair = BBSkipFirstBlockPair();
CacheInfo->Size = AliasAnalysis::UnknownSize;
CacheInfo->TBAATag = 0;
}
SmallVector<BasicBlock*, 32> Worklist; SmallVector<BasicBlock*, 32> Worklist;
Worklist.push_back(StartBB); Worklist.push_back(StartBB);
@ -986,8 +983,6 @@ getNonLocalPointerDepFromBB(const PHITransAddr &Pointer,
// cached value to do more work but not miss the phi trans failure. // cached value to do more work but not miss the phi trans failure.
NonLocalPointerInfo &NLPI = NonLocalPointerDeps[CacheKey]; NonLocalPointerInfo &NLPI = NonLocalPointerDeps[CacheKey];
NLPI.Pair = BBSkipFirstBlockPair(); NLPI.Pair = BBSkipFirstBlockPair();
NLPI.Size = AliasAnalysis::UnknownSize;
NLPI.TBAATag = 0;
continue; continue;
} }
@ -1015,8 +1010,6 @@ getNonLocalPointerDepFromBB(const PHITransAddr &Pointer,
// specific block queries) but we can't do the fastpath "return all // specific block queries) but we can't do the fastpath "return all
// results from the set" Clear out the indicator for this. // results from the set" Clear out the indicator for this.
CacheInfo->Pair = BBSkipFirstBlockPair(); CacheInfo->Pair = BBSkipFirstBlockPair();
CacheInfo->Size = AliasAnalysis::UnknownSize;
CacheInfo->TBAATag = 0;
SkipFirstBlock = false; SkipFirstBlock = false;
continue; continue;
@ -1034,8 +1027,6 @@ getNonLocalPointerDepFromBB(const PHITransAddr &Pointer,
// specific block queries) but we can't do the fastpath "return all // specific block queries) but we can't do the fastpath "return all
// results from the set". Clear out the indicator for this. // results from the set". Clear out the indicator for this.
CacheInfo->Pair = BBSkipFirstBlockPair(); CacheInfo->Pair = BBSkipFirstBlockPair();
CacheInfo->Size = AliasAnalysis::UnknownSize;
CacheInfo->TBAATag = 0;
// If *nothing* works, mark the pointer as being clobbered by the first // If *nothing* works, mark the pointer as being clobbered by the first
// instruction in this block. // instruction in this block.
@ -1252,8 +1243,6 @@ void MemoryDependenceAnalysis::removeInstruction(Instruction *RemInst) {
// The cache is not valid for any specific block anymore. // The cache is not valid for any specific block anymore.
NonLocalPointerDeps[P].Pair = BBSkipFirstBlockPair(); NonLocalPointerDeps[P].Pair = BBSkipFirstBlockPair();
NonLocalPointerDeps[P].Size = AliasAnalysis::UnknownSize;
NonLocalPointerDeps[P].TBAATag = 0;
// Update any entries for RemInst to use the instruction after it. // Update any entries for RemInst to use the instruction after it.
for (NonLocalDepInfo::iterator DI = NLPDI.begin(), DE = NLPDI.end(); for (NonLocalDepInfo::iterator DI = NLPDI.begin(), DE = NLPDI.end();