[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