azalea/kotlin-fork
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

JVM IR: do not use AsmUtil.genToString

Browse Source 
AsmUtil.genToString requires an instance of KotlinTypeMapper and calls
KotlinTypeMapper.mapToCallableMethod inside, which (in subsequent
commits) we're getting rid of in the IR backend
This commit is contained in:
Alexander Udalov
2019-08-23 18:44:23 +02:00
parent bf40b72451
commit e0823e20c7
1 changed files with 16 additions and 10 deletions
Download Patch File Download Diff File Expand all files Collapse all files
compiler/ir/backend.jvm/src/org/jetbrains/kotlin/backend/jvm/codegen/ExpressionCodegen.kt
+16 -10
View File
@@ -695,17 +695,17 @@ class ExpressionCodegen(
// Convert single arg to string.
val arg = expression.arguments[0]
val result = arg.accept(this, data).boxInlineClasses(arg.type).materialized
if (!arg.type.isString())
AsmUtil.genToString(StackValue.onStack(result.type), result.type, result.kotlinType, typeMapper.kotlinTypeMapper)
.put(expression.asmType, mv)
if (!arg.type.isString()) {
result.genToString(mv)
}
}
arity == 2 && expression.arguments[0].type.isStringClassType() -> {
// Call the stringPlus intrinsic
expression.arguments.forEach {
val result = it.accept(this, data).boxInlineClasses(it.type).materialized
val type = result.type
if (type.sort != Type.OBJECT)
AsmUtil.genToString(StackValue.onStack(type), type, result.kotlinType, typeMapper.kotlinTypeMapper).put(expression.asmType, mv)
for (argument in expression.arguments) {
val result = argument.accept(this, data).boxInlineClasses(argument.type).materialized
if (result.type.sort != Type.OBJECT) {
result.genToString(mv)
}
}
mv.invokestatic(
IrIntrinsicMethods.INTRINSICS_CLASS_NAME,
@@ -717,8 +717,8 @@ class ExpressionCodegen(
else -> {
// Use StringBuilder to concatenate.
genStringBuilderConstructor(mv)
expression.arguments.forEach {
genInvokeAppendMethod(mv, it.accept(this, data).boxInlineClasses(it.type).materialized.type, null)
for (argument in expression.arguments) {
genInvokeAppendMethod(mv, argument.accept(this, data).boxInlineClasses(argument.type).materialized.type, null)
}
mv.invokevirtual("java/lang/StringBuilder", "toString", "()Ljava/lang/String;", false)
}
@@ -726,6 +726,12 @@ class ExpressionCodegen(
return expression.onStack
}
private fun MaterialValue.genToString(v: InstructionAdapter) {
val asmType =
if (irType.getClass()?.isInline == true) OBJECT_TYPE else stringValueOfType(type)
v.invokestatic("java/lang/String", "valueOf", Type.getMethodDescriptor(AsmTypes.JAVA_STRING_TYPE, asmType), false)
}
override fun visitWhileLoop(loop: IrWhileLoop, data: BlockInfo): PromisedValue {
val continueLabel = markNewLabel()
val endLabel = Label()