From 8a66a202f6c110273a7570b77e0b8bcb660fd92f Mon Sep 17 00:00:00 2001 From: Dan Gohman Date: Thu, 11 Nov 2010 00:42:22 +0000 Subject: [PATCH] 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 --- lib/Analysis/MemoryDependenceAnalysis.cpp | 13 +------------ 1 file changed, 1 insertion(+), 12 deletions(-) diff --git a/lib/Analysis/MemoryDependenceAnalysis.cpp b/lib/Analysis/MemoryDependenceAnalysis.cpp index f29ff4a7314..96af281c395 100644 --- a/lib/Analysis/MemoryDependenceAnalysis.cpp +++ b/lib/Analysis/MemoryDependenceAnalysis.cpp @@ -858,11 +858,8 @@ getNonLocalPointerDepFromBB(const PHITransAddr &Pointer, // otherwise it isn't. if (Cache->empty()) CacheInfo->Pair = BBSkipFirstBlockPair(StartBB, SkipFirstBlock); - else { + else CacheInfo->Pair = BBSkipFirstBlockPair(); - CacheInfo->Size = AliasAnalysis::UnknownSize; - CacheInfo->TBAATag = 0; - } SmallVector Worklist; 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. NonLocalPointerInfo &NLPI = NonLocalPointerDeps[CacheKey]; NLPI.Pair = BBSkipFirstBlockPair(); - NLPI.Size = AliasAnalysis::UnknownSize; - NLPI.TBAATag = 0; continue; } @@ -1015,8 +1010,6 @@ getNonLocalPointerDepFromBB(const PHITransAddr &Pointer, // specific block queries) but we can't do the fastpath "return all // results from the set" Clear out the indicator for this. CacheInfo->Pair = BBSkipFirstBlockPair(); - CacheInfo->Size = AliasAnalysis::UnknownSize; - CacheInfo->TBAATag = 0; SkipFirstBlock = false; continue; @@ -1034,8 +1027,6 @@ getNonLocalPointerDepFromBB(const PHITransAddr &Pointer, // specific block queries) but we can't do the fastpath "return all // results from the set". Clear out the indicator for this. CacheInfo->Pair = BBSkipFirstBlockPair(); - CacheInfo->Size = AliasAnalysis::UnknownSize; - CacheInfo->TBAATag = 0; // If *nothing* works, mark the pointer as being clobbered by the first // instruction in this block. @@ -1252,8 +1243,6 @@ void MemoryDependenceAnalysis::removeInstruction(Instruction *RemInst) { // The cache is not valid for any specific block anymore. NonLocalPointerDeps[P].Pair = BBSkipFirstBlockPair(); - NonLocalPointerDeps[P].Size = AliasAnalysis::UnknownSize; - NonLocalPointerDeps[P].TBAATag = 0; // Update any entries for RemInst to use the instruction after it. for (NonLocalDepInfo::iterator DI = NLPDI.begin(), DE = NLPDI.end();