[K/N] Drop merging forward and real declaration in deserializer

^KT-59643

compiler/cli/src/org/jetbrains/kotlin/cli/metadata/K2MetadataKlibSerializer.kt

@@ -177,8 +177,7 @@ private class KlibMetadataDependencyContainer(
         KlibMetadataModuleDescriptorFactoryImpl(
             MetadataFactories.DefaultDescriptorFactory,
             MetadataFactories.DefaultPackageFragmentsFactory,
-            MetadataFactories.flexibleTypeDeserializer,
-            MetadataFactories.platformDependentTypeTransformer
+            MetadataFactories.flexibleTypeDeserializer
         )
     }
 
@@ -209,6 +208,5 @@ private class KlibMetadataDependencyContainer(
 private val MetadataFactories =
     KlibMetadataFactories(
         { DefaultBuiltIns.Instance },
-        NullFlexibleTypeDeserializer,
-        NativeTypeTransformer()
+        NullFlexibleTypeDeserializer
     )

compiler/ir/serialization.common/src/org/jetbrains/kotlin/backend/common/serialization/mangle/descriptor/DescriptorMangleComputer.kt

@@ -91,21 +91,10 @@ open class DescriptorMangleComputer(builder: StringBuilder, mode: MangleMode) :
         when (val type = wrappedType.unwrap()) {
             is SimpleType -> {
 
-                if (type is SupposititiousSimpleType) {
-                    val classId = type.overwrittenClass
-                    classId.packageFqName.let {
-                        if (!it.isRoot) {
-                            builder.appendSignature(it.asString())
-                            builder.appendSignature(MangleConstant.FQN_SEPARATOR)
-                        }
-                        builder.appendSignature(classId.relativeClassName.asString())
-                    }
-                } else {
-                    when (val classifier = type.constructor.declarationDescriptor) {
-                        is ClassDescriptor -> with(copy(MangleMode.FQNAME)) { classifier.visit() }
-                        is TypeParameterDescriptor -> tBuilder.mangleTypeParameterReference(classifier)
-                        else -> error("Unexpected classifier: $classifier")
-                    }
+                when (val classifier = type.constructor.declarationDescriptor) {
+                    is ClassDescriptor -> with(copy(MangleMode.FQNAME)) { classifier.visit() }
+                    is TypeParameterDescriptor -> tBuilder.mangleTypeParameterReference(classifier)
+                    else -> error("Unexpected classifier: $classifier")
                 }
 
                 mangleTypeArguments(tBuilder, type, null)

compiler/tests-common/tests/org/jetbrains/kotlin/test/KlibTestUtil.kt

@@ -27,7 +27,6 @@ import org.jetbrains.kotlin.library.impl.KotlinLibraryLayoutForWriter
 import org.jetbrains.kotlin.library.impl.KotlinLibraryWriterImpl
 import org.jetbrains.kotlin.library.metadata.KlibMetadataFactories
 import org.jetbrains.kotlin.library.metadata.KlibMetadataVersion
-import org.jetbrains.kotlin.library.metadata.NativeTypeTransformer
 import org.jetbrains.kotlin.library.metadata.NullFlexibleTypeDeserializer
 import org.jetbrains.kotlin.library.resolveSingleFileKlib
 import org.jetbrains.kotlin.name.Name
@@ -114,7 +113,7 @@ object KlibTestUtil {
             strategy = ToolingSingleFileKlibResolveStrategy
         )
 
-        val metadataFactories = KlibMetadataFactories({ DefaultBuiltIns.Instance }, NullFlexibleTypeDeserializer, NativeTypeTransformer())
+        val metadataFactories = KlibMetadataFactories({ DefaultBuiltIns.Instance }, NullFlexibleTypeDeserializer)
 
         val module = metadataFactories.DefaultDeserializedDescriptorFactory.createDescriptor(
             library = library,

compiler/util-klib-metadata/src/org/jetbrains/kotlin/library/metadata/KlibMetadataFactories.kt

@@ -6,7 +6,7 @@
 package org.jetbrains.kotlin.library.metadata
 
 import org.jetbrains.kotlin.builtins.KotlinBuiltIns
-import org.jetbrains.kotlin.descriptors.deserialization.PlatformDependentTypeTransformer
+import org.jetbrains.kotlin.descriptors.deserialization.*
 import org.jetbrains.kotlin.library.metadata.impl.KlibMetadataDeserializedPackageFragmentsFactoryImpl
 import org.jetbrains.kotlin.library.metadata.impl.KlibMetadataModuleDescriptorFactoryImpl
 import org.jetbrains.kotlin.library.metadata.impl.KlibModuleDescriptorFactoryImpl
@@ -19,13 +19,8 @@ import org.jetbrains.kotlin.storage.StorageManager
  */
 class KlibMetadataFactories(
     createBuiltIns: (StorageManager) -> KotlinBuiltIns,
-    val flexibleTypeDeserializer: FlexibleTypeDeserializer,
-    val platformDependentTypeTransformer: PlatformDependentTypeTransformer
+    val flexibleTypeDeserializer: FlexibleTypeDeserializer
 ) {
-
-    constructor(createBuiltIns:(StorageManager) -> KotlinBuiltIns, flexibleTypeDeserializer: FlexibleTypeDeserializer) :
-            this(createBuiltIns, flexibleTypeDeserializer, PlatformDependentTypeTransformer.None)
-
     /**
      * The default [KlibModuleDescriptorFactory] factory instance.
      */
@@ -58,8 +53,7 @@ class KlibMetadataFactories(
         KlibMetadataModuleDescriptorFactoryImpl(
             descriptorFactory,
             packageFragmentsFactory,
-            flexibleTypeDeserializer,
-            platformDependentTypeTransformer
+            flexibleTypeDeserializer
         )
 
     fun createDefaultKonanResolvedModuleDescriptorsFactory(

compiler/util-klib-metadata/src/org/jetbrains/kotlin/library/metadata/KlibMetadataModuleDescriptorFactory.kt

@@ -9,7 +9,6 @@ import org.jetbrains.kotlin.builtins.KotlinBuiltIns
 import org.jetbrains.kotlin.config.LanguageVersionSettings
 import org.jetbrains.kotlin.descriptors.ModuleDescriptor
 import org.jetbrains.kotlin.descriptors.PackageFragmentProvider
-import org.jetbrains.kotlin.descriptors.deserialization.PlatformDependentTypeTransformer
 import org.jetbrains.kotlin.descriptors.impl.ModuleDescriptorImpl
 import org.jetbrains.kotlin.incremental.components.LookupTracker
 import org.jetbrains.kotlin.library.KotlinLibrary
@@ -22,7 +21,6 @@ interface KlibMetadataModuleDescriptorFactory {
     val descriptorFactory: KlibModuleDescriptorFactory
     val packageFragmentsFactory: KlibMetadataDeserializedPackageFragmentsFactory
     val flexibleTypeDeserializer: FlexibleTypeDeserializer
-    val platformDependentTypeTransformer: PlatformDependentTypeTransformer
 
     fun createDescriptor(
         library: KotlinLibrary,

compiler/util-klib-metadata/src/org/jetbrains/kotlin/library/metadata/NativeTypeTransformer.kt

@@ -1,35 +0,0 @@
-/*
- * Copyright 2010-2020 JetBrains s.r.o. and Kotlin Programming Language contributors.
- * Use of this source code is governed by the Apache 2.0 license that can be found in the license/LICENSE.txt file.
- */
-
-package org.jetbrains.kotlin.library.metadata
-
-import org.jetbrains.kotlin.descriptors.ClassDescriptor
-import org.jetbrains.kotlin.descriptors.PackageFragmentDescriptor
-import org.jetbrains.kotlin.descriptors.deserialization.PlatformDependentTypeTransformer
-import org.jetbrains.kotlin.name.ClassId
-import org.jetbrains.kotlin.name.NativeForwardDeclarationKind
-import org.jetbrains.kotlin.types.SimpleType
-import org.jetbrains.kotlin.types.SupposititiousSimpleType
-
-class NativeTypeTransformer : PlatformDependentTypeTransformer {
-    override fun transformPlatformType(classId: ClassId, computedType: SimpleType): SimpleType {
-        val originalPackageFqn = classId.packageFqName
-        if (originalPackageFqn in forwardPackagesSet) {
-            // This hack is about keeping original class id written into proto which is required for correct IR linkage
-            val constructor = computedType.constructor
-            val classDescriptor = constructor.declarationDescriptor as ClassDescriptor
-            val realPackageFqn = (classDescriptor.containingDeclaration as PackageFragmentDescriptor).fqName
-            if (originalPackageFqn != realPackageFqn) {
-                return SupposititiousSimpleType(computedType, classId)
-            }
-        }
-
-        return computedType
-    }
-
-    companion object {
-        private val forwardPackagesSet = NativeForwardDeclarationKind.entries.map { it.packageFqName }.toSet()
-    }
-}

compiler/util-klib-metadata/src/org/jetbrains/kotlin/library/metadata/impl/KlibMetadataModuleDescriptorFactoryImpl.kt

@@ -10,7 +10,6 @@ import org.jetbrains.kotlin.builtins.functions.functionInterfacePackageFragmentP
 import org.jetbrains.kotlin.config.LanguageVersionSettings
 import org.jetbrains.kotlin.contracts.ContractDeserializerImpl
 import org.jetbrains.kotlin.descriptors.*
-import org.jetbrains.kotlin.descriptors.deserialization.PlatformDependentTypeTransformer
 import org.jetbrains.kotlin.descriptors.impl.CompositePackageFragmentProvider
 import org.jetbrains.kotlin.descriptors.impl.EmptyPackageFragmentDescriptor
 import org.jetbrains.kotlin.descriptors.impl.ModuleDescriptorImpl
@@ -33,8 +32,7 @@ private val ModuleDescriptorImpl.isStdlibModule
 class KlibMetadataModuleDescriptorFactoryImpl(
     override val descriptorFactory: KlibModuleDescriptorFactory,
     override val packageFragmentsFactory: KlibMetadataDeserializedPackageFragmentsFactory,
-    override val flexibleTypeDeserializer: FlexibleTypeDeserializer,
-    override val platformDependentTypeTransformer: PlatformDependentTypeTransformer
+    override val flexibleTypeDeserializer: FlexibleTypeDeserializer
 ) : KlibMetadataModuleDescriptorFactory {
 
     override fun createDescriptorOptionalBuiltIns(
@@ -173,7 +171,6 @@ class KlibMetadataModuleDescriptorFactoryImpl(
             ContractDeserializerImpl(configuration, storageManager),
             extensionRegistryLite = KlibMetadataSerializerProtocol.extensionRegistry,
             samConversionResolver = SamConversionResolverImpl(storageManager, samWithReceiverResolvers = emptyList()),
-            platformDependentTypeTransformer = platformDependentTypeTransformer,
             enumEntriesDeserializationSupport = enumEntriesDeserializationSupport,
         )