[GR-71114] Make AnnotationValueSupport.declaredAnnotations an LRU cache

PullRequest: graal/22519

Author: dougxc

compiler/src/jdk.graal.compiler/src/jdk/graal/compiler/annotation/AnnotationValueSupport.java

@@ -39,7 +39,6 @@
 
 import jdk.graal.compiler.core.common.LibGraalSupport;
 import jdk.graal.compiler.debug.GraalError;
-import jdk.graal.compiler.util.EconomicHashMap;
 import jdk.vm.ci.meta.ResolvedJavaMethod;
 import jdk.vm.ci.meta.ResolvedJavaType;
 import jdk.vm.ci.meta.UnresolvedJavaType;
@@ -153,11 +152,24 @@ public static AnnotationValue getAnnotationValue(Annotated annotated, Class<? ex
 
     /**
      * Cache for {@link #getAnnotationValue}. Building libgraal-ee shows that this cache grows to
-     * about 3K entries and there are about 30K accesses so no need to optimize further with an LRU
-     * cache.
+     * about 3K entries so the LRU cache is sized just above that (4096). This cache must not grow
+     * too large as there are Native Image tests that build numerous images in the one JVM process.
      */
     @LibGraalSupport.HostedOnly //
-    private static final Map<Annotated, Map<ResolvedJavaType, AnnotationValue>> declaredAnnotations = LibGraalSupport.INSTANCE == null ? Collections.synchronizedMap(new EconomicHashMap<>()) : null;
+    private static final Map<Annotated, Map<ResolvedJavaType, AnnotationValue>> declaredAnnotations;
+    static {
+        final int cacheMaxSize = 4096;
+        if (LibGraalSupport.INSTANCE == null) {
+            declaredAnnotations = Collections.synchronizedMap(new java.util.LinkedHashMap<>(cacheMaxSize, 0.75f, true) {
+                @Override
+                protected boolean removeEldestEntry(Map.Entry<Annotated, Map<ResolvedJavaType, AnnotationValue>> eldest) {
+                    return size() > cacheMaxSize;
+                }
+            });
+        } else {
+            declaredAnnotations = null;
+        }
+    }
 
     @LibGraalSupport.HostedOnly
     private static AnnotationValue getAnnotationValue0(Annotated annotated, Class<? extends Annotation> annotationType, boolean inherited) {

substratevm/src/com.oracle.svm.util/src/com/oracle/svm/util/AnnotatedObjectAccess.java

@@ -522,7 +522,7 @@ private static Annotated unwrap(Annotated element, List<AnnotationValue> injecte
         while (cur instanceof AnnotatedWrapper wrapper) {
             if (injectedAnnotationsCollector != null) {
                 List<AnnotationValue> injectedAnnotations = wrapper.getInjectedAnnotations();
-                if (injectedAnnotations != null) {
+                if (!injectedAnnotations.isEmpty()) {
                     injectedAnnotationsCollector.addAll(injectedAnnotations);
                 }
             }