[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