FIR: introduce separate companion object resolve context
Before this commit, during the resolve of companion objects we used the same context than for any nested class. However, during companion object resolve we should not have companion object receiver itself accessible in any case (in particular, it should not be accessible in constructor). So in this commit we introduced separate context for this purpose.
This commit is contained in:
+36
@@ -0,0 +1,36 @@
|
||||
FILE: selfReferenceToCompanionObject.kt
|
||||
public abstract class Base : R|kotlin/Any| {
|
||||
public constructor(fn: R|() -> kotlin/String|): R|Base| {
|
||||
super<R|kotlin/Any|>()
|
||||
}
|
||||
|
||||
public final val fn: R|() -> kotlin/String| = R|<local>/fn|
|
||||
public get(): R|() -> kotlin/String|
|
||||
|
||||
}
|
||||
public final class Host : R|kotlin/Any| {
|
||||
public constructor(): R|Host| {
|
||||
super<R|kotlin/Any|>()
|
||||
}
|
||||
|
||||
public final companion object Companion : R|Base| {
|
||||
private constructor(): R|Host.Companion| {
|
||||
super<R|Base|>(R|kotlin/run|<R|() -> kotlin/String|>(<L> = run@fun <anonymous>(): R|() -> kotlin/String| <kind=EXACTLY_ONCE> {
|
||||
^ run@fun <anonymous>(): R|kotlin/String| {
|
||||
^ Q|Host|.R|/Host.Companion.ok|()
|
||||
}
|
||||
|
||||
}
|
||||
))
|
||||
}
|
||||
|
||||
public final fun ok(): R|kotlin/String| {
|
||||
^ok String(OK)
|
||||
}
|
||||
|
||||
}
|
||||
|
||||
}
|
||||
public final fun box(): R|kotlin/String| {
|
||||
^box Q|Host.Companion|.R|/Base.fn|.R|SubstitutionOverride<kotlin/Function0.invoke: R|kotlin/String|>|()
|
||||
}
|
||||
Vendored
+9
@@ -0,0 +1,9 @@
|
||||
abstract class Base(val fn: () -> String)
|
||||
|
||||
class Host {
|
||||
companion object : Base(run { { Host.ok() } }) {
|
||||
fun ok() = "OK"
|
||||
}
|
||||
}
|
||||
|
||||
fun box() = Host.Companion.fn()
|
||||
+6
@@ -4889,6 +4889,12 @@ public class FirDiagnosticTestGenerated extends AbstractFirDiagnosticTest {
|
||||
runTest("compiler/fir/analysis-tests/testData/resolveWithStdlib/problems/receiverResolutionInLambda.kt");
|
||||
}
|
||||
|
||||
@Test
|
||||
@TestMetadata("selfReferenceToCompanionObject.kt")
|
||||
public void testSelfReferenceToCompanionObject() throws Exception {
|
||||
runTest("compiler/fir/analysis-tests/testData/resolveWithStdlib/problems/selfReferenceToCompanionObject.kt");
|
||||
}
|
||||
|
||||
@Test
|
||||
@TestMetadata("SpecialCallsWithLambdas.kt")
|
||||
public void testSpecialCallsWithLambdas() throws Exception {
|
||||
|
||||
@@ -143,6 +143,7 @@ typealias TowerDataContextForAnonymousFunctions = Map<FirAnonymousFunctionSymbol
|
||||
|
||||
class FirTowerDataContextsForClassParts(
|
||||
val forNestedClasses: FirTowerDataContext,
|
||||
val forCompanionObject: FirTowerDataContext,
|
||||
val forConstructorHeaders: FirTowerDataContext,
|
||||
val primaryConstructorPureParametersScope: FirLocalScope?,
|
||||
val primaryConstructorAllParametersScope: FirLocalScope?,
|
||||
|
||||
+3
@@ -82,6 +82,9 @@ class BodyResolveContext(
|
||||
fun getTowerDataContextForStaticNestedClassesUnsafe(): FirTowerDataContext =
|
||||
firTowerDataContextsForClassParts().forNestedClasses
|
||||
|
||||
fun getTowerDataContextForCompanionUnsafe(): FirTowerDataContext =
|
||||
firTowerDataContextsForClassParts().forCompanionObject
|
||||
|
||||
fun getTowerDataContextForConstructorResolution(): FirTowerDataContext =
|
||||
firTowerDataContextsForClassParts().forConstructorHeaders
|
||||
|
||||
|
||||
+16
-12
@@ -348,7 +348,11 @@ open class FirDeclarationsResolveTransformer(transformer: FirBodyResolveTransfor
|
||||
return context.withTowerDataCleanup {
|
||||
if (!regularClass.isInner && context.containerIfAny is FirRegularClass) {
|
||||
context.replaceTowerDataContext(
|
||||
context.getTowerDataContextForStaticNestedClassesUnsafe()
|
||||
if (regularClass.isCompanion) {
|
||||
context.getTowerDataContextForCompanionUnsafe()
|
||||
} else {
|
||||
context.getTowerDataContextForStaticNestedClassesUnsafe()
|
||||
}
|
||||
)
|
||||
}
|
||||
|
||||
@@ -816,17 +820,16 @@ open class FirDeclarationsResolveTransformer(transformer: FirBodyResolveTransfor
|
||||
): T = context.withTowerDataCleanup {
|
||||
val towerElementsForClass = components.collectTowerDataElementsForClass(owner, type)
|
||||
|
||||
val staticsAndCompanion =
|
||||
context.towerDataContext
|
||||
.addNonLocalTowerDataElements(towerElementsForClass.superClassesStaticsAndCompanionReceivers)
|
||||
.run {
|
||||
if (towerElementsForClass.companionReceiver != null)
|
||||
addReceiver(null, towerElementsForClass.companionReceiver)
|
||||
else
|
||||
this
|
||||
}
|
||||
.addNonLocalScopeIfNotNull(towerElementsForClass.companionStaticScope)
|
||||
.addNonLocalScopeIfNotNull(towerElementsForClass.staticScope)
|
||||
val base = context.towerDataContext.addNonLocalTowerDataElements(towerElementsForClass.superClassesStaticsAndCompanionReceivers)
|
||||
val statics = base
|
||||
.addNonLocalScopeIfNotNull(towerElementsForClass.companionStaticScope)
|
||||
.addNonLocalScopeIfNotNull(towerElementsForClass.staticScope)
|
||||
|
||||
val companionReceiver = towerElementsForClass.companionReceiver
|
||||
val staticsAndCompanion = if (companionReceiver == null) statics else base
|
||||
.addReceiver(null, companionReceiver)
|
||||
.addNonLocalScopeIfNotNull(towerElementsForClass.companionStaticScope)
|
||||
.addNonLocalScopeIfNotNull(towerElementsForClass.staticScope)
|
||||
|
||||
val typeParameterScope = (owner as? FirRegularClass)?.let(this::createTypeParameterScope)
|
||||
|
||||
@@ -856,6 +859,7 @@ open class FirDeclarationsResolveTransformer(transformer: FirBodyResolveTransfor
|
||||
|
||||
val newContexts = FirTowerDataContextsForClassParts(
|
||||
newTowerDataContextForStaticNestedClasses,
|
||||
statics,
|
||||
scopeForConstructorHeader,
|
||||
primaryConstructorPureParametersScope,
|
||||
primaryConstructorAllParametersScope
|
||||
|
||||
-1
@@ -1,4 +1,3 @@
|
||||
// IGNORE_BACKEND_FIR: JVM_IR
|
||||
abstract class Base(val fn: () -> String)
|
||||
|
||||
class Host {
|
||||
|
||||
-1
@@ -1,4 +1,3 @@
|
||||
// IGNORE_BACKEND_FIR: JVM_IR
|
||||
abstract class Base(val fn: () -> String)
|
||||
|
||||
interface Host {
|
||||
|
||||
-19
@@ -1,19 +0,0 @@
|
||||
open class S(val a: Any, val b: Any, val c: Any) {}
|
||||
|
||||
interface A {
|
||||
companion object : S(prop1, prop2, func()) {
|
||||
val prop1 = 1
|
||||
val prop2: Int
|
||||
get() = 1
|
||||
fun func() {}
|
||||
}
|
||||
}
|
||||
|
||||
class B {
|
||||
companion object : S(prop1, prop2, func()) {
|
||||
val prop1 = 1
|
||||
val prop2: Int
|
||||
get() = 1
|
||||
fun func() {}
|
||||
}
|
||||
}
|
||||
Vendored
+1
@@ -1,3 +1,4 @@
|
||||
// FIR_IDENTICAL
|
||||
open class S(val a: Any, val b: Any, val c: Any) {}
|
||||
|
||||
interface A {
|
||||
|
||||
Reference in New Issue
Block a user