JVM KT-49613 use adapter for indy reference to protected constructor

compiler/fir/fir2ir/tests-gen/org/jetbrains/kotlin/test/runners/codegen/FirBlackBoxCodegenTestGenerated.java

@@ -22666,6 +22666,12 @@ public class FirBlackBoxCodegenTestGenerated extends AbstractFirBlackBoxCodegenT
                     runTest("compiler/testData/codegen/box/invokedynamic/sam/functionRefToJavaInterface/kt46408.kt");
                 }
 
+                @Test
+                @TestMetadata("kt49613.kt")
+                public void testKt49613() throws Exception {
+                    runTest("compiler/testData/codegen/box/invokedynamic/sam/functionRefToJavaInterface/kt49613.kt");
+                }
+
                 @Test
                 @TestMetadata("localFunction1.kt")
                 public void testLocalFunction1() throws Exception {

compiler/ir/backend.jvm/lower/src/org/jetbrains/kotlin/backend/jvm/lower/TypeOperatorLowering.kt

@@ -553,7 +553,7 @@ private class TypeOperatorLowering(private val context: JvmBackendContext) : Fil
         targetRef: IrFunctionReference
     ): IrCall {
         fun fail(message: String): Nothing =
-            throw AssertionError("$message, irFunRef:\n${targetRef.dump()}")
+            throw AssertionError("$message, targetRef:\n${targetRef.dump()}")
 
         val dynamicCallArguments = ArrayList<IrExpression>()
 

compiler/ir/backend.jvm/lower/src/org/jetbrains/kotlin/backend/jvm/lower/indy/LambdaMetafactoryArguments.kt

@@ -145,9 +145,8 @@ internal class LambdaMetafactoryArgumentsBuilder(
             functionHazard = true
         }
 
-        if (implFun is IrConstructor && DescriptorVisibilities.isPrivate(implFun.visibility)) {
+        if (isConstructorRequiringAccessor(implFun)) {
             // Kotlin generates constructor accessors differently from Java.
-            // We don't do exact accessibility check here, just
             functionHazard = true
         }
 
@@ -214,6 +213,15 @@ internal class LambdaMetafactoryArgumentsBuilder(
             ?: MetafactoryArgumentsResult.Failure.FunctionHazard
     }
 
+    private fun isConstructorRequiringAccessor(implFun: IrFunction): Boolean {
+        if (implFun !is IrConstructor) return false
+        // We don't do exact accessibility check here:
+        // constructor will be called by a class generated by LambdaMetafactory at runtime.
+        val visibility = implFun.visibility
+        return visibility == DescriptorVisibilities.PROTECTED ||
+                DescriptorVisibilities.isPrivate(visibility)
+    }
+
     private val javaIoSerializableFqn =
         FqName("java.io").child(Name.identifier("Serializable"))
 

compiler/testData/codegen/box/invokedynamic/sam/functionRefToJavaInterface/kt49613.kt

@@ -0,0 +1,28 @@
+// TARGET_BACKEND: JVM
+// JVM_TARGET: 1.8
+// SAM_CONVERSIONS: INDY
+// FILE: kt49613.kt
+
+interface GetStep {
+    fun get(): Any
+}
+
+class Outer protected constructor(val ok: Any) {
+    constructor(): this("xxx")
+
+    val obj = object : GetStep {
+        override fun get() = Step(::Outer)
+    }
+}
+
+fun box(): String {
+    val s = Outer().obj.get() as Step
+    val t = s.step("OK") as Outer
+    return t.ok as String
+}
+
+// FILE: Step.java
+
+public interface Step {
+    Object step(String string);
+}

compiler/tests-common-new/tests-gen/org/jetbrains/kotlin/test/runners/codegen/BlackBoxCodegenTestGenerated.java

@@ -22504,6 +22504,12 @@ public class BlackBoxCodegenTestGenerated extends AbstractBlackBoxCodegenTest {
                     runTest("compiler/testData/codegen/box/invokedynamic/sam/functionRefToJavaInterface/kt46408.kt");
                 }
 
+                @Test
+                @TestMetadata("kt49613.kt")
+                public void testKt49613() throws Exception {
+                    runTest("compiler/testData/codegen/box/invokedynamic/sam/functionRefToJavaInterface/kt49613.kt");
+                }
+
                 @Test
                 @TestMetadata("localFunction1.kt")
                 public void testLocalFunction1() throws Exception {

compiler/tests-common-new/tests-gen/org/jetbrains/kotlin/test/runners/codegen/IrBlackBoxCodegenTestGenerated.java

@@ -22666,6 +22666,12 @@ public class IrBlackBoxCodegenTestGenerated extends AbstractIrBlackBoxCodegenTes
                     runTest("compiler/testData/codegen/box/invokedynamic/sam/functionRefToJavaInterface/kt46408.kt");
                 }
 
+                @Test
+                @TestMetadata("kt49613.kt")
+                public void testKt49613() throws Exception {
+                    runTest("compiler/testData/codegen/box/invokedynamic/sam/functionRefToJavaInterface/kt49613.kt");
+                }
+
                 @Test
                 @TestMetadata("localFunction1.kt")
                 public void testLocalFunction1() throws Exception {

compiler/tests-gen/org/jetbrains/kotlin/codegen/LightAnalysisModeTestGenerated.java

@@ -18840,6 +18840,11 @@ public class LightAnalysisModeTestGenerated extends AbstractLightAnalysisModeTes
                     runTest("compiler/testData/codegen/box/invokedynamic/sam/functionRefToJavaInterface/kt46408.kt");
                 }
 
+                @TestMetadata("kt49613.kt")
+                public void testKt49613() throws Exception {
+                    runTest("compiler/testData/codegen/box/invokedynamic/sam/functionRefToJavaInterface/kt49613.kt");
+                }
+
                 @TestMetadata("localFunction1.kt")
                 public void testLocalFunction1() throws Exception {
                     runTest("compiler/testData/codegen/box/invokedynamic/sam/functionRefToJavaInterface/localFunction1.kt");