[JS IR BE] support multimodule tests

2018-09-17 14:17:29 +03:00
parent 2e709a81fa
commit 0660e4d616
70 changed files with 37 additions and 81 deletions
@@ -49,10 +49,11 @@ fun compile(
    files: List<KtFile>,
    configuration: CompilerConfiguration,
    export: FqName? = null,
-    dependencies: List<Result> = listOf()
+    dependencies: List<ModuleDescriptor> = listOf(),
+    irDependencyModules: List<IrModuleFragment> = listOf()
 ): Result {
    val analysisResult =
-        TopDownAnalyzerFacadeForJS.analyzeFiles(files, project, configuration, dependencies.mapNotNull { it.moduleDescriptor as? ModuleDescriptorImpl }, emptyList())
+        TopDownAnalyzerFacadeForJS.analyzeFiles(files, project, configuration, dependencies.mapNotNull { it as? ModuleDescriptorImpl }, emptyList())

    ProgressIndicatorAndCompilationCanceledStatus.checkCanceled()

@@ -61,8 +62,6 @@ fun compile(
    val psi2IrTranslator = Psi2IrTranslator(configuration.languageVersionSettings)
    val psi2IrContext = psi2IrTranslator.createGeneratorContext(analysisResult.moduleDescriptor, analysisResult.bindingContext)

-    val irDependencyModules = dependencies.map { it.moduleFragment.deepCopyWithSymbols() }
-
    irDependencyModules.forEach { psi2IrContext.symbolTable.loadModule(it)}

    val moduleFragment = psi2IrTranslator.generateModuleFragment(psi2IrContext, files)
@@ -6,15 +6,13 @@
 package org.jetbrains.kotlin.ir.backend.js.lower

 import org.jetbrains.kotlin.backend.common.FileLoweringPass
-import org.jetbrains.kotlin.builtins.PrimitiveType
-import org.jetbrains.kotlin.ir.UNDEFINED_OFFSET
 import org.jetbrains.kotlin.ir.backend.js.JsIrBackendContext
+import org.jetbrains.kotlin.ir.backend.js.ir.irCall
 import org.jetbrains.kotlin.ir.declarations.IrConstructor
 import org.jetbrains.kotlin.ir.declarations.IrFile
 import org.jetbrains.kotlin.ir.expressions.IrCall
 import org.jetbrains.kotlin.ir.expressions.IrExpression
 import org.jetbrains.kotlin.ir.expressions.impl.IrCallImpl
-import org.jetbrains.kotlin.ir.expressions.impl.IrConstImpl
 import org.jetbrains.kotlin.ir.symbols.IrClassSymbol
 import org.jetbrains.kotlin.ir.visitors.IrElementTransformerVoid
 import org.jetbrains.kotlin.ir.visitors.transformChildrenVoid
@@ -37,22 +37,31 @@ class SecondaryCtorLowering(val context: JsIrBackendContext) {

    fun getConstructorProcessorLowering() = object : DeclarationContainerLoweringPass {
        override fun lower(irDeclarationContainer: IrDeclarationContainer) {
-            irDeclarationContainer.declarations.transformFlat {
-                if (it is IrClass) {
-                    listOf(it) + lowerClass(it)
-                } else null
-            }
+            irDeclarationContainer.declarations.filterIsInstance<IrClass>().forEach { lowerClass(it) }
        }
    }::runOnFilePostfix

    fun getConstructorRedirectorLowering() = object : DeclarationContainerLoweringPass {
        override fun lower(irDeclarationContainer: IrDeclarationContainer) {
+            if (irDeclarationContainer is IrClass) {
+                updateConstructorDeclarations(irDeclarationContainer)
+            }
            for (it in irDeclarationContainer.declarations) {
                it.accept(CallsiteRedirectionTransformer(), null)
            }
        }
    }::runOnFilePostfix

+    private fun updateConstructorDeclarations(irClass: IrClass) {
+        irClass.declarations.transformFlat {
+            if (it is IrConstructor) {
+                oldCtorToNewMap[it.symbol]?.let { (newInit, newCreate) ->
+                    listOf(newInit.owner, newCreate.owner)
+                }
+            } else null
+        }
+    }
+
    private fun lowerClass(irClass: IrClass): List<IrSimpleFunction> {
        val className = irClass.name.asString()
        val oldConstructors = mutableListOf<IrConstructor>()
@@ -91,8 +100,6 @@ class SecondaryCtorLowering(val context: JsIrBackendContext) {
            }
        }

-        irClass.declarations.removeAll(oldConstructors)
-
        return newConstructors
    }

@@ -1,7 +1,7 @@
 // !LANGUAGE: +ReleaseCoroutines
 // !API_VERSION: 1.3
 // IGNORE_BACKEND: NATIVE
-// IGNORE_BACKEND: JS, JS_IR
+// IGNORE_BACKEND: JS
 // IGNORE_BACKEND: JVM_IR

 // WITH_RUNTIME
@@ -1,4 +1,3 @@
-// IGNORE_BACKEND: JS_IR
 // IGNORE_BACKEND: JVM_IR
 // IGNORE_BACKEND: NATIVE
 // WITH_COROUTINES
@@ -1,4 +1,3 @@
-// IGNORE_BACKEND: JS_IR
 // IGNORE_BACKEND: JVM_IR
 // MODULE: lib
 // FILE: lib.kt
@@ -1,4 +1,3 @@
-// IGNORE_BACKEND: JS_IR
 // IGNORE_BACKEND: JVM_IR
 // WITH_COROUTINES
 // WITH_RUNTIME
@@ -1,4 +1,3 @@
-// IGNORE_BACKEND: JS_IR
 // IGNORE_BACKEND: JVM_IR
 // WITH_COROUTINES
 // WITH_RUNTIME
@@ -1,4 +1,3 @@
-// IGNORE_BACKEND: JS_IR
 // IGNORE_BACKEND: JVM_IR
 // WITH_COROUTINES
 // WITH_RUNTIME
@@ -1,4 +1,3 @@
-// IGNORE_BACKEND: JS_IR
 // IGNORE_BACKEND: JVM_IR
 // IGNORE_BACKEND: NATIVE
 // WITH_COROUTINES
@@ -1,4 +1,3 @@
-// IGNORE_BACKEND: JS_IR
 // IGNORE_BACKEND: JVM_IR
 // IGNORE_BACKEND: NATIVE
 // WITH_RUNTIME
@@ -1,4 +1,3 @@
-// IGNORE_BACKEND: JS_IR
 // IGNORE_BACKEND: JVM_IR
 // WITH_RUNTIME
 // WITH_COROUTINES
@@ -1,5 +1,5 @@
 // WITH_RUNTIME
-// IGNORE_BACKEND: JVM_IR, JS_IR, NATIVE
+// IGNORE_BACKEND: JVM_IR, NATIVE

 inline class SnekDirection(val direction: Int) {
    companion object {
@@ -1,4 +1,3 @@
-// IGNORE_BACKEND: JS_IR
 // MODULE: lib
 // FILE: lib.kt
 class A {
@@ -1,4 +1,3 @@
-// IGNORE_BACKEND: JS_IR
 // MODULE: lib
 // FILE: lib.kt
@PublishedApi
@@ -1,5 +1,4 @@
 // IGNORE_BACKEND: JVM_IR
-// IGNORE_BACKEND: JS_IR
 // MODULE: lib
 // FILE: lib.kt

@@ -1,4 +1,3 @@
-// IGNORE_BACKEND: JS_IR
 // MODULE: lib
 // FILE: lib.kt

@@ -94,7 +94,7 @@ abstract class BasicBoxTest(
        doTest(filePath, "OK", MainCallParameters.noCall(), coroutinesPackage)
    }

-    fun doTest(filePath: String, expectedResult: String, mainCallParameters: MainCallParameters, coroutinesPackage: String = "") {
+    open fun doTest(filePath: String, expectedResult: String, mainCallParameters: MainCallParameters, coroutinesPackage: String = "") {
        val file = File(filePath)
        val outputDir = getOutputDir(file)
        var fileContent = KotlinTestUtils.doLoadFile(file)
@@ -8,6 +8,7 @@ package org.jetbrains.kotlin.js.test
 import org.jetbrains.kotlin.config.*
 import org.jetbrains.kotlin.ir.backend.js.Result
 import org.jetbrains.kotlin.ir.backend.js.compile
+import org.jetbrains.kotlin.js.config.JSConfigurationKeys
 import org.jetbrains.kotlin.js.config.JsConfig
 import org.jetbrains.kotlin.js.facade.MainCallParameters
 import org.jetbrains.kotlin.js.facade.TranslationUnit
@@ -116,6 +117,13 @@ abstract class BasicIrBoxTest(

    override var skipMinification = true

+    private val compilationCache = mutableMapOf<String, Result>()
+
+    override fun doTest(filePath: String, expectedResult: String, mainCallParameters: MainCallParameters, coroutinesPackage: String) {
+        compilationCache.clear()
+        super.doTest(filePath, expectedResult, mainCallParameters, coroutinesPackage)
+    }
+
    override fun translateFiles(
        units: List<TranslationUnit>,
        outputFile: File,
@@ -153,12 +161,19 @@ abstract class BasicIrBoxTest(
            runtimeFile.write(runtimeResult!!.generatedCode)
        }

+        val dependencyPaths = config.configuration[JSConfigurationKeys.LIBRARIES]!!
+        val dependencies = listOf(runtimeResult!!.moduleDescriptor) + dependencyPaths.mapNotNull { compilationCache[it]?.moduleDescriptor }
+        val irDependencies = listOf(runtimeResult!!.moduleFragment) + compilationCache.values.map { it.moduleFragment }
+
        val result = compile(
            config.project,
            filesToCompile,
            config.configuration,
            FqName((testPackage?.let { "$it." } ?: "") + testFunction),
-            listOf(runtimeResult!!))
+            dependencies,
+            irDependencies)
+
+        compilationCache[outputFile.absolutePath.let { it.substring(0, it.length - 2)} + "meta.js"] = result

        outputFile.write(result.generatedCode)
    }
@@ -4,7 +4,6 @@
 // PROPERTY_WRITE_COUNT: name=local$o count=2

 // Unmute when JS_IR supports release coroutines
-// IGNORE_BACKEND: JS_IR

 import kotlin.coroutines.*

@@ -1,7 +1,6 @@
 // EXPECTED_REACHABLE_NODES: 1250

 // Unmute when JS_IR supports release coroutines
-// IGNORE_BACKEND: JS_IR

 import kotlin.coroutines.*
 import kotlin.coroutines.intrinsics.*
@@ -1,4 +1,3 @@
-// IGNORE_BACKEND: JS_IR
 // EXPECTED_REACHABLE_NODES: 1289
 // MODULE: lib
 // FILE: lib.kt
@@ -1,4 +1,3 @@
-// IGNORE_BACKEND: JS_IR
 // EXPECTED_REACHABLE_NODES: 1291
 // MODULE: lib
 // FILE: lib.kt
@@ -1,4 +1,3 @@
-// IGNORE_BACKEND: JS_IR
 // EXPECTED_REACHABLE_NODES: 1292
 // MODULE: lib
 // FILE: lib.kt
@@ -1,4 +1,3 @@
-// IGNORE_BACKEND: JS_IR
 // EXPECTED_REACHABLE_NODES: 1283
 // MODULE: lib1
 // FILE: lib1.kt
@@ -1,4 +1,3 @@
-// IGNORE_BACKEND: JS_IR
 // EXPECTED_REACHABLE_NODES: 1283
 // MODULE: 1
 // FILE: lib1.kt
@@ -1,4 +1,3 @@
-// IGNORE_BACKEND: JS_IR
 // EXPECTED_REACHABLE_NODES: 1283
 // MODULE: lib
 // FILE: lib.kt
@@ -1,4 +1,3 @@
-// IGNORE_BACKEND: JS_IR
 // EXPECTED_REACHABLE_NODES: 1289
 // MODULE: lib
 // FILE: lib.kt
@@ -1,4 +1,3 @@
-// IGNORE_BACKEND: JS_IR
 // EXPECTED_REACHABLE_NODES: 1284
 // MODULE: lib
 // FILE: lib.kt
@@ -1,4 +1,3 @@
-// IGNORE_BACKEND: JS_IR
 // EXPECTED_REACHABLE_NODES: 1286
 // MODULE: lib
 // FILE: lib.kt
@@ -1,4 +1,3 @@
-// IGNORE_BACKEND: JS_IR
 // EXPECTED_REACHABLE_NODES: 1285
 // MODULE: lib
 // FILE: lib.kt
@@ -1,4 +1,3 @@
-// IGNORE_BACKEND: JS_IR
 // EXPECTED_REACHABLE_NODES: 1189

 // MODULE: lib
@@ -1,5 +1,4 @@
 // EXPECTED_REACHABLE_NODES: 1192
-// IGNORE_BACKEND: JS_IR

 // MODULE: lib
 // FILE: lib.kt
@@ -1,5 +1,4 @@
 // EXPECTED_REACHABLE_NODES: 1269
-// IGNORE_BACKEND: JS_IR

 // MODULE: lib
 // FILE: lib.kt
@@ -1,4 +1,3 @@
-// IGNORE_BACKEND: JS_IR
 // EXPECTED_REACHABLE_NODES: 1286
 // MODULE: lib
 // FILE: lib.kt
@@ -1,4 +1,3 @@
-// IGNORE_BACKEND: JS_IR
 // EXPECTED_REACHABLE_NODES: 1285
 // MODULE: lib
 // FILE: lib.kt
@@ -1,4 +1,3 @@
-// IGNORE_BACKEND: JS_IR
 // EXPECTED_REACHABLE_NODES: 1283
 // MODULE: lib
 // FILE: lib.kt
@@ -1,4 +1,3 @@
-// IGNORE_BACKEND: JS_IR
 // EXPECTED_REACHABLE_NODES: 1283
 // MODULE: lib
 // FILE: lib.kt
@@ -1,4 +1,3 @@
-// IGNORE_BACKEND: JS_IR
 // EXPECTED_REACHABLE_NODES: 1285
 // MODULE: lib
 // FILE: lib.kt
@@ -1,4 +1,3 @@
-// IGNORE_BACKEND: JS_IR
 // EXPECTED_REACHABLE_NODES: 1294
 // MODULE: lib
 // FILE: lib.kt
@@ -1,4 +1,3 @@
-// IGNORE_BACKEND: JS_IR
 // EXPECTED_REACHABLE_NODES: 5901
 // MODULE: lib
 // FILE: lib.kt
@@ -1,4 +1,3 @@
-// IGNORE_BACKEND: JS_IR
 // EXPECTED_REACHABLE_NODES: 1282
 // MODULE: lib
 // FILE: lib1.kt
@@ -1,4 +1,3 @@
-// IGNORE_BACKEND: JS_IR
 // EXPECTED_REACHABLE_NODES: 1289
 // MODULE: lib
 // FILE: lib.kt
@@ -1,4 +1,3 @@
-// IGNORE_BACKEND: JS_IR
 // EXPECTED_REACHABLE_NODES: 1229
 // MODULE: lib
 // FILE: lib.kt
@@ -1,4 +1,3 @@
-// IGNORE_BACKEND: JS_IR
 // EXPECTED_REACHABLE_NODES: 1286
 // MODULE: lib
 // FILE: lib.kt
@@ -1,4 +1,3 @@
-// IGNORE_BACKEND: JS_IR
 // EXPECTED_REACHABLE_NODES: 1288
 // MODULE: lib
 // FILE: lib.kt
@@ -1,4 +1,3 @@
-// IGNORE_BACKEND: JS_IR
 // EXPECTED_REACHABLE_NODES: 1304
 // MODULE: m3
 // FILE: m3.kt
@@ -1,4 +1,3 @@
-// IGNORE_BACKEND: JS_IR
 // EXPECTED_REACHABLE_NODES: 1300
 // MODULE: main(module2)
 // FILE: main.kt
@@ -1,4 +1,3 @@
-// IGNORE_BACKEND: JS_IR
 // EXPECTED_REACHABLE_NODES: 1283
 // MODULE: lib
 // FILE: lib.kt
@@ -1,4 +1,3 @@
-// IGNORE_BACKEND: JS_IR
 // EXPECTED_REACHABLE_NODES: 1283
 // MODULE: lib
 // FILE: lib.kt
@@ -1,4 +1,3 @@
-// IGNORE_BACKEND: JS_IR
 // EXPECTED_REACHABLE_NODES: 1283
 // MODULE: lib
 // FILE: lib.kt
@@ -1,4 +1,3 @@
-// IGNORE_BACKEND: JS_IR
 // EXPECTED_REACHABLE_NODES: 1302
 // MODULE: lib
 // FILE: lib.kt
@@ -1,4 +1,3 @@
-// IGNORE_BACKEND: JS_IR
 // EXPECTED_REACHABLE_NODES: 1291
 // MODULE: lib
 // FILE: lib.kt
@@ -1,4 +1,3 @@
-// IGNORE_BACKEND: JS_IR
 // EXPECTED_REACHABLE_NODES: 1284
 // MODULE: lib
 // FILE: lib.kt
@@ -1,4 +1,3 @@
-// IGNORE_BACKEND: JS_IR
 // EXPECTED_REACHABLE_NODES: 1282
 // MODULE: lib
 // FILE: lib.kt
@@ -1,4 +1,3 @@
-// IGNORE_BACKEND: JS_IR
 // EXPECTED_REACHABLE_NODES: 1319
 // MODULE: lib
 // FILE: lib.kt
@@ -1,4 +1,3 @@
-// IGNORE_BACKEND: JS_IR
 // EXPECTED_REACHABLE_NODES: 1283
 // MODULE: lib1
 // FILE: lib1.kt
@@ -1,4 +1,3 @@
-// IGNORE_BACKEND: JS_IR
 // EXPECTED_REACHABLE_NODES: 1283
 // MODULE: lib
 // FILE: lib.kt
@@ -1,4 +1,3 @@
-// IGNORE_BACKEND: JS_IR
 // EXPECTED_REACHABLE_NODES: 1281
 // MODULE: lib
 // FILE: lib1.kt
@@ -1,4 +1,3 @@
-// IGNORE_BACKEND: JS_IR
 // EXPECTED_REACHABLE_NODES: 1286
 // MODULE: lib
 // FILE: lib.kt
@@ -1,4 +1,3 @@
-// IGNORE_BACKEND: JS_IR
 // EXPECTED_REACHABLE_NODES: 1280
 // MODULE: lib
 // FILE: lib.kt
@@ -1,4 +1,3 @@
-// IGNORE_BACKEND: JS_IR
 // EXPECTED_REACHABLE_NODES: 1281
 // MODULE: lib
 // FILE: lib.kt
@@ -1,4 +1,3 @@
-// IGNORE_BACKEND: JS_IR
 // EXPECTED_REACHABLE_NODES: 1319
 // MODULE: lib
 // FILE: lib.kt
@@ -1,4 +1,3 @@
-// IGNORE_BACKEND: JS_IR
 // EXPECTED_REACHABLE_NODES: 1283
 // MODULE: main(module1)
 // FILE: main.kt
@@ -1,4 +1,3 @@
-// IGNORE_BACKEND: JS_IR
 // EXPECTED_REACHABLE_NODES: 1304
 // MODULE: lib1
 // FILE: lib1.kt
@@ -1,4 +1,3 @@
-// IGNORE_BACKEND: JS_IR
 // EXPECTED_REACHABLE_NODES: 1285
 // MODULE: main(module1)
 // FILE: main.kt
@@ -1,4 +1,3 @@
-// IGNORE_BACKEND: JS_IR
 // EXPECTED_REACHABLE_NODES: 1282
 // MODULE: #my-libr@ry
 // FILE: bar.kt
@@ -1,4 +1,3 @@
-// IGNORE_BACKEND: JS_IR
 // EXPECTED_REACHABLE_NODES: 1283
 // MODULE: module-1
 // FILE: bar.kt
@@ -1,4 +1,3 @@
-// IGNORE_BACKEND: JS_IR
 // EXPECTED_REACHABLE_NODES: 1283
 // MODULE: module1
 // FILE: bar.kt
@@ -1,4 +1,3 @@
-// IGNORE_BACKEND: JS_IR
 // EXPECTED_REACHABLE_NODES: 1287
 // MODULE: lib
 // FILE: lib.kt