From 41849c8beb68ac41b1bd8e4d4d491a702274af26 Mon Sep 17 00:00:00 2001 From: Mikhail Glukhikh Date: Thu, 21 Oct 2021 15:44:55 +0300 Subject: [PATCH] FIR: get rid of effectively unused AccessorSymbol --- .../org/jetbrains/kotlin/fir/backend/ConversionUtils.kt | 3 +-- .../org/jetbrains/kotlin/fir/symbols/SyntheticSymbol.kt | 6 ------ .../kotlin/fir/symbols/impl/FirFunctionSymbol.kt | 8 ++------ 3 files changed, 3 insertions(+), 14 deletions(-) diff --git a/compiler/fir/fir2ir/src/org/jetbrains/kotlin/fir/backend/ConversionUtils.kt b/compiler/fir/fir2ir/src/org/jetbrains/kotlin/fir/backend/ConversionUtils.kt index 5bc70d1766c..e14fb5c5738 100644 --- a/compiler/fir/fir2ir/src/org/jetbrains/kotlin/fir/backend/ConversionUtils.kt +++ b/compiler/fir/fir2ir/src/org/jetbrains/kotlin/fir/backend/ConversionUtils.kt @@ -37,7 +37,6 @@ import org.jetbrains.kotlin.fir.resolve.providers.symbolProvider import org.jetbrains.kotlin.fir.scopes.FirTypeScope import org.jetbrains.kotlin.fir.scopes.ProcessorAction import org.jetbrains.kotlin.fir.scopes.unsubstitutedScope -import org.jetbrains.kotlin.fir.symbols.AccessorSymbol import org.jetbrains.kotlin.fir.symbols.FirBasedSymbol import org.jetbrains.kotlin.fir.symbols.impl.* import org.jetbrains.kotlin.fir.types.* @@ -452,7 +451,7 @@ internal fun FirReference.statementOrigin(): IrStatementOrigin? { return when (this) { is FirPropertyFromParameterResolvedNamedReference -> IrStatementOrigin.INITIALIZE_PROPERTY_FROM_PARAMETER is FirResolvedNamedReference -> when (val symbol = resolvedSymbol) { - is AccessorSymbol, is FirSyntheticPropertySymbol -> IrStatementOrigin.GET_PROPERTY + is FirAccessorSymbol -> IrStatementOrigin.GET_PROPERTY is FirNamedFunctionSymbol -> when { symbol.callableId.isInvoke() -> IrStatementOrigin.INVOKE diff --git a/compiler/fir/tree/src/org/jetbrains/kotlin/fir/symbols/SyntheticSymbol.kt b/compiler/fir/tree/src/org/jetbrains/kotlin/fir/symbols/SyntheticSymbol.kt index 0b06b3dd330..0b2e66edb43 100644 --- a/compiler/fir/tree/src/org/jetbrains/kotlin/fir/symbols/SyntheticSymbol.kt +++ b/compiler/fir/tree/src/org/jetbrains/kotlin/fir/symbols/SyntheticSymbol.kt @@ -5,11 +5,5 @@ package org.jetbrains.kotlin.fir.symbols -import org.jetbrains.kotlin.name.CallableId - interface SyntheticSymbol -interface AccessorSymbol { - val callableId: CallableId // it's an id of related property (synthetic or not) - val accessorId: CallableId // it's an id of accessor function -} \ No newline at end of file diff --git a/compiler/fir/tree/src/org/jetbrains/kotlin/fir/symbols/impl/FirFunctionSymbol.kt b/compiler/fir/tree/src/org/jetbrains/kotlin/fir/symbols/impl/FirFunctionSymbol.kt index a3f861a094d..32ad52caaf5 100644 --- a/compiler/fir/tree/src/org/jetbrains/kotlin/fir/symbols/impl/FirFunctionSymbol.kt +++ b/compiler/fir/tree/src/org/jetbrains/kotlin/fir/symbols/impl/FirFunctionSymbol.kt @@ -11,8 +11,6 @@ import org.jetbrains.kotlin.fir.declarations.* import org.jetbrains.kotlin.fir.expressions.FirDelegatedConstructorCall import org.jetbrains.kotlin.fir.references.FirControlFlowGraphReference import org.jetbrains.kotlin.fir.references.FirResolvedNamedReference -import org.jetbrains.kotlin.fir.symbols.AccessorSymbol -import org.jetbrains.kotlin.fir.symbols.FirBasedSymbol import org.jetbrains.kotlin.fir.symbols.ensureResolved import org.jetbrains.kotlin.name.CallableId import org.jetbrains.kotlin.name.FqName @@ -84,8 +82,8 @@ class FirConstructorSymbol( open class FirAccessorSymbol( callableId: CallableId, - override val accessorId: CallableId -) : FirPropertySymbol(callableId), AccessorSymbol + open val accessorId: CallableId +) : FirPropertySymbol(callableId) // ------------------------ unnamed ------------------------ @@ -99,6 +97,4 @@ class FirAnonymousFunctionSymbol : FirFunctionWithoutNameSymbol(Name.identifier("accessor")) -class FirPropertyFieldDeclarationSymbol() : FirBasedSymbol() - class FirErrorFunctionSymbol : FirFunctionWithoutNameSymbol(Name.identifier("error"))