Add size to cache stats

Author: mbovel

compiler/src/dotty/tools/dotc/core/Caches.scala

@@ -43,11 +43,11 @@ object Caches:
       s"${this.getClass.getSimpleName}(stats() = ${stats()})"
 
   /** Statistics about a cache */
-  final case class CacheStats(total: Long, misses: Long, uncached: Long):
+  final case class CacheStats(total: Long, misses: Long, size: Long, uncached: Long):
     val hits: Long = total - misses - uncached
 
     override def toString: String =
-      s"(total = $total, hits = $hits, misses = $misses, uncached = $uncached)"
+      s"(total = $total, hits = $hits, misses = $misses, size = $size, uncached = $uncached)"
 
   /** A no-op cache implementation that does not cache anything. */
   final class NoopCache[K, V] extends Cache[K, V]:
@@ -61,7 +61,7 @@ object Caches:
       false
 
     def stats(): CacheStats =
-      CacheStats(total, misses = 0, uncached = total)
+      CacheStats(total, misses = 0, size = 0, uncached = total)
 
   private def noStamp[K](key: K): Option[Unit] = Some(())
 
@@ -95,7 +95,7 @@ object Caches:
       getStamp(key).isDefined
 
     def stats(): CacheStats =
-      CacheStats(total, misses, uncached)
+      CacheStats(total, misses, map.size, uncached)
 
   /** A thread-safe cache implementation based on a Java [[ConcurrentHashMap]].
    *
@@ -128,7 +128,7 @@ object Caches:
       getStamp(key).isDefined
 
     def stats(): CacheStats =
-      CacheStats(total.longValue(), misses.longValue(), uncached.longValue())
+      CacheStats(total.longValue(), misses.longValue(), map.size(), uncached.longValue())
 
   /** A cache where keys are [[AbstractFile]]s.
    *
@@ -206,7 +206,8 @@ object Caches:
       CacheStats(
         total = baseStats.total + uncached.longValue(),
         misses = baseStats.misses,
-        uncached = baseStats.uncached + uncached.longValue()
+        size = baseStats.size,
+        uncached = baseStats.uncached + uncached.longValue(),
       )
 
     def mightContain(key: K): Boolean =