[compiler-rt] [scudo] Update secondary cache time-based release logic (PR #107507)

via llvm-commits llvm-commits at lists.llvm.org
Mon Sep 23 13:44:47 PDT 2024


================
@@ -334,9 +336,6 @@ class MapAllocatorCache {
       }
 
       insert(Entry);
----------------
ChiaHungDuan wrote:

The case I'm saying is the moment when we want to remove the time record in the `CacheEntry`. Which means we  may need another argument to tell the page status but anyway, it's not a problem here so we can do it as what you suggest. BTW, I think the issue was the cases mentioned above, would you prepare a reapply CL for this?

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


More information about the llvm-commits mailing list