mirror of
https://github.com/jlengrand/kotlin.git
synced 2026-05-08 15:53:19 +00:00
4fc1bd9ec5
Namely, anonymous objects defined in lambdas that have all captured
variables as loose fields instead of a single reference to the parent.
The question is, when a lambda inside an inline function defines an
anonymous object, and that object is not regenerated during codegen for
the inline function itself, but then has to be regenerated at call site
anyway, do we use an outer `this` or loose capture fields? For example,
before KT-28064:
inline fun f1(g: () -> Unit) = object { g() }
// -> f1$1 { $g: () -> Unit }
inline fun f2(g: () -> Unit) = f1 { object { g() } }
// -> f2$$inlined$f1$1 { $g: () -> Unit }
// f2$$inlined$f1$1$lambda$1 { this$0: f2$$inlined$f1$1 }
inline fun f3(g: () -> Unit) = f2 { object { g() } }
// -> f3$$inlined$f2$1 { $g: () -> Unit }
// f3$$inlined$f2$1$1 { this$0: f3$$inlined$f2$1 }
// f3$$inlined$f2$1$1$lambda$1 { this$0: f3$$inlined$f2$1$1 }
After KT-28064:
inline fun f2(g: () -> Unit) = f1 { object { g() } }
// -> f2$$inlined$f1$1 { $g: () -> Unit }
// f2$1$1 { $g: () -> Unit }
inline fun f3(g: () -> Unit) = f2 { object { g() } }
// -> f3$$inlined$f2$1 { $g: () -> Unit }
// f3$$inlined$f2$2 { ??? }
// f3$1$1 { $g: () -> Unit }
Should `???` be `this$0: f3$$inlined$f2$1` or `$g: () -> Unit`? This
commit chooses the latter for KT-28064 bytecode and keeps `this$0` when
inlining the old bytecode.
22 lines
411 B
Kotlin
Vendored
22 lines
411 B
Kotlin
Vendored
fun box(): String {
|
|
var encl1 = "fail";
|
|
test {
|
|
encl1 = "OK"
|
|
}
|
|
|
|
return encl1
|
|
}
|
|
|
|
inline fun test(crossinline s: () -> Unit) {
|
|
{
|
|
{
|
|
s()
|
|
}()
|
|
}()
|
|
}
|
|
|
|
// 3 INNERCLASS Kt10259_2Kt\$test\$1 null
|
|
// 2 INNERCLASS Kt10259_2Kt\$test\$1\$1
|
|
// 2 INNERCLASS Kt10259_2Kt\$box\$\$inlined\$test\$1\s
|
|
// 2 INNERCLASS Kt10259_2Kt\$box\$\$inlined\$test\$1\$1
|
|
// 9 INNERCLASS |