[compiler-rt] [scudo] Update secondary cache released pages bound. (PR #106466)

Joshua Baehring via llvm-commits llvm-commits at lists.llvm.org
Wed Aug 28 17:26:05 PDT 2024


https://github.com/JoshuaMBa updated https://github.com/llvm/llvm-project/pull/106466

>From a593986ef71bf2fa4cfa9c9e92bb644dd158822f Mon Sep 17 00:00:00 2001
From: Joshua Baehring <josh.baehring at yale.edu>
Date: Wed, 28 Aug 2024 13:35:48 -0400
Subject: [PATCH] [scudo] Update secondary cache released pages bound.

`MaxReleasedCachePages` has been set to 4.
---
 compiler-rt/lib/scudo/standalone/secondary.h | 3 +--
 1 file changed, 1 insertion(+), 2 deletions(-)

diff --git a/compiler-rt/lib/scudo/standalone/secondary.h b/compiler-rt/lib/scudo/standalone/secondary.h
index fba21c3cb6a09d..1a232b9b9fb2d7 100644
--- a/compiler-rt/lib/scudo/standalone/secondary.h
+++ b/compiler-rt/lib/scudo/standalone/secondary.h
@@ -79,8 +79,7 @@ struct CachedBlock {
   //        it suggests that beyond 4 pages, the release execution time is
   //        longer than the map execution time. In this way, the default
   //        is dependent on the platform.
-  //    TODO: set MaxReleasedCachePages back to 4U
-  static constexpr uptr MaxReleasedCachePages = 0U;
+  static constexpr uptr MaxReleasedCachePages = 4U;
 
   uptr CommitBase = 0;
   uptr CommitSize = 0;



More information about the llvm-commits mailing list