[PATCH] D105634: [OpenMP] Use AAHeapToStack/AAHeapToShared analysis in SPMDization

[Johannes Doerfert via Phabricator via llvm-commits]

jdoerfert added a comment.

Test?



================
Comment at: llvm/lib/Transforms/IPO/OpenMPOpt.cpp:3474
+
+    auto &HeapToStackAA = A.getAAFor<AAHeapToStack>(
+        *this, IRPosition::function(*CB.getCaller()), DepClassTy::OPTIONAL);
----------------
jhuber6 wrote:
> For HeapToShared I used `lookupAAFor` because it returns null if the instance doesn't exist or is invalid. Not sure if that is an issue here.
heap2shared doesn't depend on heap2stack, it could but it doesn't. Here we want getorcreate because there is a real dependence


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D105634/new/

https://reviews.llvm.org/D105634