[llvm] r340072 - [AST] Adapt Polly to AnalysisSetTracker changes. NFC.

Michael Kruse via llvm-commits llvm-commits at lists.llvm.org
Fri Aug 17 12:31:41 PDT 2018


Author: meinersbur
Date: Fri Aug 17 12:31:41 2018
New Revision: 340072

URL: http://llvm.org/viewvc/llvm-project?rev=340072&view=rev
Log:
[AST] Adapt Polly to AnalysisSetTracker changes. NFC.

The method AliasSetTracker::getAliasSetForPointer was removed and replaced by AliasSetTracker::getAliasSetFor for the restructuring in r339930. 

Since Polly uses AliasSetTracker::getAliasSetForPointer, a temporary fix has been committed in r339937 with a comment:

     Can someone from polly please migrate usage and then delete the wrapper?

This commit is doing exactly that.

Modified:
    llvm/trunk/include/llvm/Analysis/AliasSetTracker.h

Modified: llvm/trunk/include/llvm/Analysis/AliasSetTracker.h
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/include/llvm/Analysis/AliasSetTracker.h?rev=340072&r1=340071&r2=340072&view=diff
==============================================================================
--- llvm/trunk/include/llvm/Analysis/AliasSetTracker.h (original)
+++ llvm/trunk/include/llvm/Analysis/AliasSetTracker.h Fri Aug 17 12:31:41 2018
@@ -399,13 +399,6 @@ public:
   /// set is returned.  
   AliasSet &getAliasSetFor(const MemoryLocation &MemLoc);
 
-  AliasSet &getAliasSetForPointer(Value *P, LocationSize Size,
-                                  const AAMDNodes &AAInfo) {
-    // This adapter exists so that polly can be updated to the new API.  Once
-    // done, please delete this.
-    return getAliasSetFor(MemoryLocation(P, Size, AAInfo));
-  }
-
   /// Return true if the specified instruction "may" (or must) alias one of the
   /// members in any of the sets.
   bool containsUnknown(const Instruction *I) const;




More information about the llvm-commits mailing list