Move codegen tests on old language versions under oldLanguageVersions/

This directory is skipped in JVM IR test generator, so they won't show up as failed anymore. Note that only failing tests are moved to oldLanguageVersions/. Tests which already pass are still being run. It may be useful not to break them in case we _do_ need to support some pre-1.3 language feature switches in JVM IR.
2019-10-28 18:00:47 +01:00
parent c3729c8189
commit 1978db9d0e
33 changed files with 942 additions and 418 deletions
@@ -0,0 +1,29 @@
+// !LANGUAGE: -ReleaseCoroutines
+// IGNORE_BACKEND: JVM_IR
+// WITH_COROUTINES
+// TREAT_AS_ONE_FILE
+
+import helpers.*
+import kotlin.coroutines.experimental.*
+import kotlin.coroutines.experimental.intrinsics.*
+
+suspend fun suspendHere() = ""
+
+fun builder(c: suspend () -> Unit) {
+    c.startCoroutine(EmptyContinuation)
+}
+
+fun box(): String {
+
+    for (i in 1..3) {
+        builder {
+            if (suspendHere() != "OK") throw java.lang.RuntimeException("fail 1")
+        }
+    }
+
+    return "OK"
+}
+
+// 2 GETSTATIC kotlin/Unit.INSTANCE
+// 1 GETSTATIC helpers/EmptyContinuation.Companion
+// 3 GETSTATIC
@@ -0,0 +1,22 @@
+// !LANGUAGE: -InlineConstVals
+// IGNORE_BACKEND: JVM_IR
+// FILE: Foo.kt
+
+private const val OUTER_PRIVATE = 20
+
+class Foo {
+    companion object {
+        private const val LOCAL_PRIVATE = 20
+    }
+
+    fun foo() {
+        // Access to the property use getstatic on the backed field
+        LOCAL_PRIVATE
+        // Access to the property requires an invokestatic
+        OUTER_PRIVATE
+    }
+}
+
+// 1 INVOKESTATIC
+// 1 PUTSTATIC
+// 2 GETSTATIC
@@ -0,0 +1,34 @@
+// !LANGUAGE: -ReleaseCoroutines
+// IGNORE_BACKEND: JVM_IR
+// WITH_COROUTINES
+
+import helpers.*
+import kotlin.coroutines.experimental.*
+import kotlin.coroutines.experimental.intrinsics.*
+
+suspend fun dummy() {}
+
+fun builder(c: suspend () -> Unit) {
+    c.startCoroutine(EmptyContinuation)
+}
+
+interface SuspendRunnable {
+    suspend fun run(): String
+}
+
+inline fun inlineMe(crossinline c: suspend () -> String) = object : SuspendRunnable {
+    override suspend fun run() = c()
+}
+
+fun box(): String {
+    var res = "FAIL"
+    builder {
+        res = inlineMe { dummy(); "OK" }.run()
+    }
+    return res
+}
+
+// Test for continuation of 'run' transformation.
+// Since continuation is not suspend lambda, it should not have state-machine.
+// @CrossinlineSuspendContinuation_1_2Kt$box$1$doResume$$inlined$inlineMe$1$1.class:
+// 0 TABLESWITCH
@@ -0,0 +1,19 @@
+// !LANGUAGE: -ReleaseCoroutines
+// IGNORE_BACKEND: JVM_IR
+// WITH_COROUTINES
+
+import helpers.*
+// TREAT_AS_ONE_FILE
+import kotlin.coroutines.experimental.*
+import kotlin.coroutines.experimental.intrinsics.*
+suspend fun suspendHere(): String = suspendCoroutineUninterceptedOrReturn { x ->
+    x.resume("OK")
+}
+
+suspend fun suspendThere(param: Int, param2: String, param3: Long): String {
+    val a = suspendHere()
+    val b = suspendHere()
+    return a + b
+}
+
+// 0 ASTORE 4
@@ -0,0 +1,49 @@
+// !LANGUAGE: -ReleaseCoroutines
+// IGNORE_BACKEND: JVM_IR
+// WITH_COROUTINES
+
+import helpers.*
+// TREAT_AS_ONE_FILE
+import kotlin.coroutines.experimental.*
+import kotlin.coroutines.experimental.intrinsics.*
+suspend fun suspendHere(): String = suspendCoroutineUninterceptedOrReturn { x ->
+    x.resume("OK")
+}
+
+fun builder(c: suspend () -> Unit) {
+    c.startCoroutine(EmptyContinuation)
+}
+
+fun box(): String {
+    var result = "fail 1"
+
+    builder {
+        // Initialize var with Int value
+        try {
+            var i: String = "abc"
+            i = "123"
+        } finally { }
+
+        // This variable should take the same slot as 'i' had
+        var s: String
+
+        // We shout not spill 's' to continuation field because it's not effectively initialized
+        if (suspendHere() == "OK") {
+            s = "OK"
+        }
+        else {
+            s = "fail 2"
+        }
+
+        result = s
+    }
+
+    return result
+}
+
+// 1 LOCALVARIABLE i Ljava/lang/String; L.* 3
+// 1 LOCALVARIABLE s Ljava/lang/String; L.* 3
+// 0 PUTFIELD VarValueConflictsWithTableSameSort_1_2Kt\$box\$1.L\$0 : Ljava/lang/Object;
+/* 1 load in try/finally */
+/* 1 load in result = s */
+// 2 ALOAD 3
@@ -0,0 +1,50 @@
+// !LANGUAGE: -ReleaseCoroutines
+// IGNORE_BACKEND: JVM_IR
+// WITH_COROUTINES
+
+import helpers.*
+// TREAT_AS_ONE_FILE
+import kotlin.coroutines.experimental.*
+import kotlin.coroutines.experimental.intrinsics.*
+suspend fun suspendHere(): String = suspendCoroutineUninterceptedOrReturn { x ->
+    x.resume("OK")
+}
+
+fun builder(c: suspend () -> Unit) {
+    c.startCoroutine(EmptyContinuation)
+}
+
+val nonConstOne = 1
+
+fun box(): String {
+    var result = "fail 1"
+    builder {
+        // Initialize var with Int value
+        for (i in 1..nonConstOne) {
+            if ("".length > 0) continue
+        }
+
+        // This variable should take the same slot as 'i' had
+        var s: String
+
+        // We should not spill 's' to continuation field because it's not initialized
+        // More precisely it contains a value of wrong type (it conflicts with contents of local var table),
+        // so an attempt of spilling may lead to problems on Android
+        if (suspendHere() == "OK") {
+            s = "OK"
+        }
+        else {
+            s = "fail 2"
+        }
+
+        result = s
+    }
+
+    return result
+}
+
+// 1 LOCALVARIABLE i I L.* 3
+// 1 LOCALVARIABLE s Ljava/lang/String; L.* 3
+// 0 PUTFIELD VarValueConflictsWithTableKt\$box\$1.I\$0 : I
+/* 2 loads in cycle */
+// 2 ILOAD 3
@@ -0,0 +1,24 @@
+// !LANGUAGE: -InlineConstVals
+// IGNORE_BACKEND: JVM_IR
+// FILE: first/Foo.java
+
+package first;
+
+public class Foo {
+    protected static final int FOO = 42;
+}
+
+// FILE: bar.kt
+
+package second
+
+import first.Foo
+
+class Bar : Foo() {
+    fun bar() = FOO
+}
+
+// @second/BarKt.class
+// 1 INVOKESTATIC
+// 0 GETSTATIC
+// 1 BIPUSH 42