Native: RedundantCoercionsCleaner should rewrite nested returns too

kotlin-native/backend.native/compiler/ir/backend.native/src/org/jetbrains/kotlin/backend/konan/lower/RedundantCoercionsCleaner.kt

@@ -163,10 +163,11 @@ internal class RedundantCoercionsCleaner(val context: Context) : FileLoweringPas
                             }
                             transformedReturnableBlock.transformChildrenVoid(object: IrElementTransformerVoid() {
                                 override fun visitExpression(expression: IrExpression): IrExpression {
+                                    expression.transformChildrenVoid()
                                     foldedReturnableBlockValues[expression]?.let {
                                         return it.getFullExpression(coercion, cast)
                                     }
-                                    return super.visitExpression(expression)
+                                    return expression
                                 }
 
                                 override fun visitReturn(expression: IrReturn): IrExpression {

kotlin-native/backend.native/tests/build.gradle

@@ -3586,6 +3586,10 @@ task inline_coercionToUnit(type: KonanLocalTest) {
     source = "codegen/inline/coercionToUnit.kt"
 }
 
+task inline_redundantCoercionsCleaner(type: KonanLocalTest) {
+    source = "codegen/inline/redundantCoercionsCleaner.kt"
+}
+
 task classDeclarationInsideInline(type: KonanLocalTest) {
     source = "codegen/inline/classDeclarationInsideInline.kt"
     goldValue = "test1: 1.0\n1\ntest2\n"

kotlin-native/backend.native/tests/codegen/inline/redundantCoercionsCleaner.kt

@@ -0,0 +1,17 @@
+package codegen.inline.redundantCoercionsCleaner
+
+import kotlin.test.*
+
+inline fun runAndThrow(action: () -> Unit): Nothing {
+    action()
+    throw Exception()
+}
+
+inline fun foo(): Int = runAndThrow {
+    return 1
+}
+
+@Test fun runTest() {
+    val result: Any = foo()
+    assertEquals(1, result)
+}