[llvm] [Offload] Make MemoryManager threshold ENV var size_t type. (PR #124063)
via llvm-commits
llvm-commits at lists.llvm.org
Wed Jan 22 20:35:00 PST 2025
llvmbot wrote:
<!--LLVM PR SUMMARY COMMENT-->
@llvm/pr-subscribers-offload
Author: None (hidekisaito)
<details>
<summary>Changes</summary>
---
Full diff: https://github.com/llvm/llvm-project/pull/124063.diff
2 Files Affected:
- (modified) offload/include/Shared/EnvironmentVar.h (+1)
- (modified) offload/plugins-nextgen/common/include/MemoryManager.h (+1-1)
``````````diff
diff --git a/offload/include/Shared/EnvironmentVar.h b/offload/include/Shared/EnvironmentVar.h
index 82f434e91a85b8..c1854774d572b6 100644
--- a/offload/include/Shared/EnvironmentVar.h
+++ b/offload/include/Shared/EnvironmentVar.h
@@ -128,6 +128,7 @@ using Int32Envar = Envar<int32_t>;
using Int64Envar = Envar<int64_t>;
using UInt32Envar = Envar<uint32_t>;
using UInt64Envar = Envar<uint64_t>;
+using SizeTEnvar = Envar<size_t>;
using StringEnvar = Envar<std::string>;
using BoolEnvar = Envar<bool>;
diff --git a/offload/plugins-nextgen/common/include/MemoryManager.h b/offload/plugins-nextgen/common/include/MemoryManager.h
index fe1989930b76ef..34e9e9618680f3 100644
--- a/offload/plugins-nextgen/common/include/MemoryManager.h
+++ b/offload/plugins-nextgen/common/include/MemoryManager.h
@@ -324,7 +324,7 @@ class MemoryManagerTy {
/// manager explicitly by setting the var to 0. If user doesn't specify
/// anything, returns <0, true>.
static std::pair<size_t, bool> getSizeThresholdFromEnv() {
- static UInt32Envar MemoryManagerThreshold(
+ static SizeTEnvar MemoryManagerThreshold(
"LIBOMPTARGET_MEMORY_MANAGER_THRESHOLD", 0);
size_t Threshold = MemoryManagerThreshold.get();
``````````
</details>
https://github.com/llvm/llvm-project/pull/124063
More information about the llvm-commits
mailing list