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

Improve implicit widening conversion to unsigned integer

Browse Source
This commit is contained in:
Svyatoslav Scherbina
2018-08-24 16:36:15 +03:00
committed by SvyatoslavScherbina
parent 62bbfcebd8
commit 872deb205c
1 changed files with 2 additions and 5 deletions
Download Patch File Download Diff File Expand all files Collapse all files
backend.native/compiler/ir/backend.native/src/org/jetbrains/kotlin/backend/konan/llvm/IrToBitcode.kt
+2 -5
View File
@@ -1272,8 +1272,7 @@ internal class CodeGeneratorVisitor(val context: Context, val lifetimes: Map<IrE
private fun evaluateIntegerCoercion(value: IrTypeOperatorCall): LLVMValueRef { private fun evaluateIntegerCoercion(value: IrTypeOperatorCall): LLVMValueRef {
context.log{"evaluateIntegerCoercion : ${ir2string(value)}"} context.log{"evaluateIntegerCoercion : ${ir2string(value)}"}
val type = value.typeOperand val type = value.typeOperand
val typeIsUnsigned = type.isUnsignedInteger() assert(type.isPrimitiveInteger() || type.isUnsignedInteger())
assert(type.isPrimitiveInteger() || typeIsUnsigned)
val result = evaluateExpression(value.argument) val result = evaluateExpression(value.argument)
assert(value.argument.type.isInt()) assert(value.argument.type.isInt())
val llvmSrcType = codegen.getLLVMType(value.argument.type) val llvmSrcType = codegen.getLLVMType(value.argument.type)
@@ -1283,9 +1282,7 @@ internal class CodeGeneratorVisitor(val context: Context, val lifetimes: Map<IrE
return when { return when {
srcWidth == dstWidth -> result srcWidth == dstWidth -> result
srcWidth > dstWidth -> LLVMBuildTrunc(functionGenerationContext.builder, result, llvmDstType, "")!! srcWidth > dstWidth -> LLVMBuildTrunc(functionGenerationContext.builder, result, llvmDstType, "")!!
/* srcWidth < dstWidth */ else /* srcWidth < dstWidth */ -> LLVMBuildSExt(functionGenerationContext.builder, result, llvmDstType, "")!!
typeIsUnsigned -> LLVMBuildZExt(functionGenerationContext.builder, result, llvmDstType, "")!!
else -> LLVMBuildSExt(functionGenerationContext.builder, result, llvmDstType, "")!!
} }
} }