[compiler-rt] [scudo] Added LRU eviction policy to secondary cache. (PR #99409)

Christopher Ferris via llvm-commits llvm-commits at lists.llvm.org
Mon Jul 22 22:08:46 PDT 2024


================
@@ -188,10 +195,11 @@ template <typename Config> class MapAllocatorCache {
     Str->append("Stats: CacheRetrievalStats: SuccessRate: %u/%u "
                 "(%zu.%02zu%%)\n",
                 SuccessfulRetrieves, CallsToRetrieve, Integral, Fractional);
-    for (CachedBlock Entry : Entries) {
-      if (!Entry.isValid())
-        continue;
-      Str->append("StartBlockAddress: 0x%zx, EndBlockAddress: 0x%zx, "
+    Str->append("Cache Entry Dump (Most Recent -> Least Recent):\n");
----------------
cferris1000 wrote:

This is such a small nit, but I'd probably use Info instead of Dump. This is mostly an aesthetic thing though.

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


More information about the llvm-commits mailing list