================
@@ -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
_______________________________________________
llvm-branch-commits mailing list
[email protected]
https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-branch-commits
