FIR: cache accessor symbols in JavaClassUseSiteMemberScope

compiler/fir/java/src/org/jetbrains/kotlin/fir/java/scopes/JavaClassUseSiteMemberScope.kt

@@ -43,6 +43,7 @@ class JavaClassUseSiteMemberScope(
 ) {
     private val typeParameterStack = klass.javaTypeParameterStack
     private val specialFunctions = hashMapOf<Name, Collection<FirNamedFunctionSymbol>>()
+    private val accessorByNameMap = hashMapOf<Name, FirAccessorSymbol>()
 
     override fun getCallableNames(): Set<Name> {
         return declaredMemberScope.getContainingCallableNamesIfPresent() + superTypesScope.getCallableNames()
@@ -57,16 +58,18 @@ class JavaClassUseSiteMemberScope(
         setterSymbol: FirNamedFunctionSymbol?,
         syntheticPropertyName: Name,
     ): FirAccessorSymbol {
-        return buildSyntheticProperty {
+        return accessorByNameMap.getOrPut(syntheticPropertyName) {
-            session = this@JavaClassUseSiteMemberScope.session
+            buildSyntheticProperty {
-            name = syntheticPropertyName
+                session = this@JavaClassUseSiteMemberScope.session
-            symbol = FirAccessorSymbol(
+                name = syntheticPropertyName
-                accessorId = getterSymbol.callableId,
+                symbol = FirAccessorSymbol(
-                callableId = CallableId(getterSymbol.callableId.packageName, getterSymbol.callableId.className, syntheticPropertyName)
+                    accessorId = getterSymbol.callableId,
-            )
+                    callableId = CallableId(getterSymbol.callableId.packageName, getterSymbol.callableId.className, syntheticPropertyName)
-            delegateGetter = getterSymbol.fir
+                )
-            delegateSetter = setterSymbol?.fir
+                delegateGetter = getterSymbol.fir
-        }.symbol
+                delegateSetter = setterSymbol?.fir
+            }.symbol
+        }
     }
 
     override fun processPropertiesByName(name: Name, processor: (FirVariableSymbol<*>) -> Unit) {

compiler/testData/codegen/box/bridges/kt2498.kt

@@ -1,6 +1,5 @@
 // DONT_TARGET_EXACT_BACKEND: WASM
 // WASM_MUTE_REASON: STDLIB_COLLECTIONS
-// IGNORE_BACKEND_FIR: JVM_IR
 // KJS_WITH_FULL_RUNTIME
 // IGNORE_BACKEND: NATIVE
 

compiler/testData/codegen/box/collections/inheritFromHashtable.kt

@@ -1,4 +1,3 @@
-// IGNORE_BACKEND_FIR: JVM_IR
 // TARGET_BACKEND: JVM
 // WITH_RUNTIME
 // FULL_JDK

compiler/testData/codegen/box/collections/kt41123.kt

@@ -2,7 +2,6 @@
 // WASM_MUTE_REASON: STDLIB_COLLECTIONS
 // WITH_RUNTIME
 // KJS_WITH_FULL_RUNTIME
-// IGNORE_BACKEND_FIR: JVM_IR
 
 open class A : HashMap<String, String>()
 

compiler/testData/codegen/box/properties/collectionSize.kt

@@ -1,4 +1,3 @@
-// IGNORE_BACKEND_FIR: JVM_IR
 // TARGET_BACKEND: JVM
 
 // FILE: Test.java

compiler/testData/codegen/box/specialBuiltins/commonBridgesTarget.kt

@@ -1,6 +1,5 @@
 // DONT_TARGET_EXACT_BACKEND: WASM
 // WASM_MUTE_REASON: STDLIB_COLLECTIONS
-// IGNORE_BACKEND_FIR: JVM_IR
 // KJS_WITH_FULL_RUNTIME
 // IGNORE_BACKEND: NATIVE
 

compiler/testData/codegen/box/specialBuiltins/entrySetSOE.kt

@@ -1,6 +1,5 @@
 // DONT_TARGET_EXACT_BACKEND: WASM
 // WASM_MUTE_REASON: STDLIB_COLLECTIONS
-// IGNORE_BACKEND_FIR: JVM_IR
 // KJS_WITH_FULL_RUNTIME
 // IGNORE_BACKEND: NATIVE
 

compiler/testData/codegen/bytecodeText/builtinFunctions/size.kt

@@ -1,4 +1,3 @@
-// IGNORE_BACKEND_FIR: JVM_IR
 abstract class A1 : Collection<String> {
     override val size: Int get() = 1
 }