[llvm] [polly] [AST] Don't merge memory locations in AliasSetTracker (PR #65731)

Bruno De Fraine via llvm-commits llvm-commits at lists.llvm.org
Tue Dec 12 08:47:11 PST 2023


================
@@ -671,7 +671,7 @@ class AccessAnalysis {
   /// Register a store.
   void addStore(MemoryLocation &Loc, Type *AccessTy) {
     Value *Ptr = const_cast<Value*>(Loc.Ptr);
-    AST.add(Ptr, LocationSize::beforeOrAfterPointer(), Loc.AATags);
+    AST.add(Loc.getWithNewSize(LocationSize::beforeOrAfterPointer()));
----------------
brunodf-snps wrote:

Actually, I think we can precommit this a little bit wider, and already update all callers of this particular add method. I've reviewed my branch history and the new commit that I'd propose to precommit is 048677aead99 (this replaces the commit mentioned above).

https://github.com/llvm/llvm-project/pull/65731


More information about the llvm-commits mailing list