[PATCH] D120079: [OpenMP] Add an option to limit shared memory usage in OpenMPOpt

[Johannes Doerfert via Phabricator via llvm-commits]

jdoerfert accepted this revision.
jdoerfert added a comment.
This revision is now accepted and ready to land.

LG, two nits



================
Comment at: llvm/lib/Transforms/IPO/OpenMPOpt.cpp:3083
+
+  unsigned SharedMemoryUsed = 0;
 };
----------------
Add doxygen docs plz.


================
Comment at: llvm/lib/Transforms/IPO/OpenMPOpt.cpp:3010
+          SharedMemoryLimit)
+        continue;
+
----------------
jdoerfert wrote:
> What happens in non-assert builds, are statistics available to be used?
Add a debug message though.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D120079