From 78d62dcb1e808691cb8021a945336f86ec65887b Mon Sep 17 00:00:00 2001 From: Georgy Bronnikov Date: Tue, 16 Mar 2021 14:13:37 +0300 Subject: [PATCH] JVM_IR: use ConcurrentHashMap in codegen where appropriate --- .../state/BuilderFactoryForDuplicateClassNameDiagnostics.kt | 3 ++- .../src/org/jetbrains/kotlin/backend/jvm/JvmBackendContext.kt | 2 +- 2 files changed, 3 insertions(+), 2 deletions(-) diff --git a/compiler/backend/src/org/jetbrains/kotlin/codegen/state/BuilderFactoryForDuplicateClassNameDiagnostics.kt b/compiler/backend/src/org/jetbrains/kotlin/codegen/state/BuilderFactoryForDuplicateClassNameDiagnostics.kt index 2f3c681961b..1826f4f06b1 100644 --- a/compiler/backend/src/org/jetbrains/kotlin/codegen/state/BuilderFactoryForDuplicateClassNameDiagnostics.kt +++ b/compiler/backend/src/org/jetbrains/kotlin/codegen/state/BuilderFactoryForDuplicateClassNameDiagnostics.kt @@ -22,6 +22,7 @@ import org.jetbrains.kotlin.diagnostics.DiagnosticSink import org.jetbrains.kotlin.renderer.DescriptorRenderer import org.jetbrains.kotlin.resolve.jvm.diagnostics.ErrorsJvm import org.jetbrains.kotlin.resolve.jvm.diagnostics.JvmDeclarationOrigin +import java.util.concurrent.ConcurrentHashMap class BuilderFactoryForDuplicateClassNameDiagnostics( @@ -29,7 +30,7 @@ class BuilderFactoryForDuplicateClassNameDiagnostics( private val diagnostics: DiagnosticSink ) : ClassNameCollectionClassBuilderFactory(builderFactory) { - private val className = hashMapOf () + private val className = ConcurrentHashMap() override fun handleClashingNames(internalName: String, origin: JvmDeclarationOrigin) { val another = className.getOrPut(internalName, { origin }) diff --git a/compiler/ir/backend.jvm/src/org/jetbrains/kotlin/backend/jvm/JvmBackendContext.kt b/compiler/ir/backend.jvm/src/org/jetbrains/kotlin/backend/jvm/JvmBackendContext.kt index 76deb542ad3..657ca8b313f 100644 --- a/compiler/ir/backend.jvm/src/org/jetbrains/kotlin/backend/jvm/JvmBackendContext.kt +++ b/compiler/ir/backend.jvm/src/org/jetbrains/kotlin/backend/jvm/JvmBackendContext.kt @@ -91,7 +91,7 @@ class JvmBackendContext( internal val isEnclosedInConstructor = ConcurrentHashMap.newKeySet() - internal val classCodegens = mutableMapOf() + internal val classCodegens = ConcurrentHashMap() val localDelegatedProperties = ConcurrentHashMap>()