From a71cd69f728ecac2776fc87d111106d629dc11f4 Mon Sep 17 00:00:00 2001 From: Konstantin Anisimov Date: Mon, 20 Feb 2017 16:52:47 +0700 Subject: [PATCH] Kotlin methods are excluded from inlining --- .../backend/konan/lower/FunctionInlining.kt | 26 +++++++++++++------ 1 file changed, 18 insertions(+), 8 deletions(-) diff --git a/backend.native/compiler/ir/backend.native/src/org/jetbrains/kotlin/backend/konan/lower/FunctionInlining.kt b/backend.native/compiler/ir/backend.native/src/org/jetbrains/kotlin/backend/konan/lower/FunctionInlining.kt index 5a9ddfee787..ac0d5c8eca1 100644 --- a/backend.native/compiler/ir/backend.native/src/org/jetbrains/kotlin/backend/konan/lower/FunctionInlining.kt +++ b/backend.native/compiler/ir/backend.native/src/org/jetbrains/kotlin/backend/konan/lower/FunctionInlining.kt @@ -6,7 +6,7 @@ import org.jetbrains.kotlin.backend.konan.ir.IrInlineFunctionBody import org.jetbrains.kotlin.backend.konan.ir.getArguments import org.jetbrains.kotlin.descriptors.FunctionDescriptor import org.jetbrains.kotlin.descriptors.ParameterDescriptor -import org.jetbrains.kotlin.descriptors.impl.TypeParameterDescriptorImpl +import org.jetbrains.kotlin.descriptors.TypeParameterDescriptor import org.jetbrains.kotlin.ir.IrElement import org.jetbrains.kotlin.ir.IrStatement import org.jetbrains.kotlin.ir.builders.Scope @@ -21,7 +21,6 @@ import org.jetbrains.kotlin.ir.util.DeepCopyIrTree import org.jetbrains.kotlin.ir.visitors.IrElementTransformerVoid import org.jetbrains.kotlin.types.KotlinType - //-----------------------------------------------------------------------------// internal class FunctionInlining(val context: Context): IrElementTransformerVoid() { @@ -30,6 +29,15 @@ internal class FunctionInlining(val context: Context): IrElementTransformerVoid( //-------------------------------------------------------------------------// + var fqName: String? = null + override fun visitFile(declaration: IrFile): IrFile { + fqName = declaration.packageFragmentDescriptor.fqName.asString() + println("akm_inline $fqName") + return super.visitFile(declaration) + } + + //-------------------------------------------------------------------------// + fun isLambdaExpression(expression: IrExpression) : Boolean { if (expression !is IrContainerExpressionBase) return false if (expression.origin != IrStatementOrigin.LAMBDA) return false @@ -68,7 +76,6 @@ internal class FunctionInlining(val context: Context): IrElementTransformerVoid( .functions[functionDescriptor.original] // Get FunctionDeclaration by FunctionDescriptor. if (functionDeclaration == null) return super.visitCall(irCall) // Function is declared in another module. - val copyFuncDeclaration = functionDeclaration.accept(DeepCopyIrTree(), // Create copy of the function. null) as IrFunction @@ -84,16 +91,18 @@ internal class FunctionInlining(val context: Context): IrElementTransformerVoid( inlineBody.accept(lambdaInliner, null) val transformer = ParametersTransformer(parameters, irCall) inlineBody.accept(transformer, null) // Replace parameters with expression. - return inlineBody + return super.visitBlock(inlineBody) } //-------------------------------------------------------------------------// override fun visitCall(expression: IrCall): IrExpression { - val functionDescriptor = expression.descriptor as FunctionDescriptor - if (!functionDescriptor.name.asString().contains("foo")) return super.visitCall(expression) + if(fqName!!.contains("kotlin")) return super.visitCall(expression) + + val functionDescriptor = expression.descriptor as FunctionDescriptor if (functionDescriptor.isInline) return inlineFunction(expression) // Return newly created IrInlineBody instead of IrCall. + return super.visitCall(expression) } @@ -194,8 +203,9 @@ internal class ParametersTransformer(val parameterToArgument: List