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

Bruno De Fraine via llvm-commits llvm-commits at lists.llvm.org
Thu Dec 21 03:28:36 PST 2023


================
@@ -284,16 +150,16 @@ class AliasSet : public ilist_node<AliasSet> {
 
   void removeFromTracker(AliasSetTracker &AST);
 
-  void addPointer(AliasSetTracker &AST, PointerRec &Entry, LocationSize Size,
-                  const AAMDNodes &AAInfo, bool KnownMustAlias = false,
-                  bool SkipSizeUpdate = false);
+  bool isMustAliasMergeWith(AliasSet &AS, BatchAAResults &BatchAA) const;
----------------
brunodf-snps wrote:

> Actually, the auxiliary function was only introduced to exit elegantly from a nested loop.

This was perhaps unnecessarily heavy. I still removed the function, and replaced it by an immediately-invoked lambda, so the logic is directly visible in `AliasSet::mergeSetIn`.

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


More information about the llvm-commits mailing list