[llvm] [polly] [AST] Don't merge memory locations in AliasSetTracker (PR #65731)
Nikita Popov via llvm-commits
llvm-commits at lists.llvm.org
Tue Dec 12 00:36:20 PST 2023
================
@@ -307,7 +173,7 @@ class AliasSetTracker {
BatchAAResults &AA;
ilist<AliasSet> AliasSets;
- using PointerMapType = DenseMap<AssertingVH<Value>, AliasSet::PointerRec *>;
+ using PointerMapType = DenseMap<MemoryLocation, AliasSet *>;
----------------
nikic wrote:
I don't really get why this needs to be indexed by MemoryLocation. I'd still expect all MemoryLocations for the same Value to be in the same AliasSet, so it seems like the existing indexing should work fine...
https://github.com/llvm/llvm-project/pull/65731
More information about the llvm-commits
mailing list