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

Johannes Doerfert via llvm-commits llvm-commits at lists.llvm.org
Mon Oct 9 13:15:58 PDT 2023


================
@@ -307,7 +173,7 @@ class AliasSetTracker {
   BatchAAResults &AA;
   ilist<AliasSet> AliasSets;
 
-  using PointerMapType = DenseMap<AssertingVH<Value>, AliasSet::PointerRec *>;
+  using PointerMapType = DenseMap<MemoryLocation, AliasSet *>;
----------------
jdoerfert wrote:

A MemoryLocation is not super small, I think 7 or 8 pointer sized objects, roughly. Unsure if this could cause issues at some point.

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


More information about the llvm-commits mailing list