[compiler-rt] [scudo] Support setting default value of ReleaseToOsIntervalMs in config (PR #90256)

via llvm-commits llvm-commits at lists.llvm.org
Fri Apr 26 15:09:48 PDT 2024


================
@@ -89,6 +89,7 @@ PRIMARY_REQUIRED(const s32, MaxReleaseToOsIntervalMs)
 // Indicates support for offsetting the start of a region by a random number of
 // pages. This is only used if `EnableContiguousRegions` is enabled.
 PRIMARY_OPTIONAL(const bool, EnableRandomOffset, false)
+PRIMARY_OPTIONAL(const s32, DefaultReleaseToOsIntervalMs, 0)
----------------
ChiaHungDuan wrote:

I was thinking to default the value to 5000 (the one used in  release_to_os_interval_ms) when we are ready to deprecate the flag and now the zero means "not set" temporarily. But you're right, especially 0 is useful in many cases.

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


More information about the llvm-commits mailing list