[llvm-commits] [llvm] r118752 - /llvm/trunk/lib/Analysis/MemoryDependenceAnalysis.cpp

Dan Gohman gohman at apple.com
Wed Nov 10 16:42:22 PST 2010


Author: djg
Date: Wed Nov 10 18:42:22 2010
New Revision: 118752

URL: http://llvm.org/viewvc/llvm-project?rev=118752&view=rev
Log:
It's not necessary to clear out the Size and TBAATag at each of
these points.

Modified:
    llvm/trunk/lib/Analysis/MemoryDependenceAnalysis.cpp

Modified: llvm/trunk/lib/Analysis/MemoryDependenceAnalysis.cpp
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/Analysis/MemoryDependenceAnalysis.cpp?rev=118752&r1=118751&r2=118752&view=diff
==============================================================================
--- llvm/trunk/lib/Analysis/MemoryDependenceAnalysis.cpp (original)
+++ llvm/trunk/lib/Analysis/MemoryDependenceAnalysis.cpp Wed Nov 10 18:42:22 2010
@@ -858,11 +858,8 @@
   // 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<BasicBlock*, 32> Worklist;
   Worklist.push_back(StartBB);
@@ -986,8 +983,6 @@
         // 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 @@
     // 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 @@
     // 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 @@
       
       // 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();





More information about the llvm-commits mailing list