[llvm-branch-commits] [llvm] [openmp] [offload] Pool host and shared allocations (PR #214754)

[Robert Imschweiler via llvm-branch-commits]

================
@@ -359,6 +365,8 @@ class MemoryManagerTy {
   /// threshold and the second element represents whether user disables memory
   /// manager explicitly by setting the var to 0. If user doesn't specify
   /// anything, returns <0, true>.
+  /// Note that this only affects the device memory manager, not the manager for
----------------
ro-i wrote:

I thought about it. Imho, if we want a control for the other pools, we should use different env vars. Either one for each pool, or one for device and one for host/shared. Because the sizes that you'd like for the device memory (GB sth), might diverge significantly from the sizes you'd want as pinned host memory. I felt like changing the semantics of this env var might be very confusing or even hurtful for users.

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