JVM: Do not implicitly expose functions with @JvmName annotation

if the functions return inline class.

 #KT-78358
 #KT-78386 Fixed

Author: ilmirus

compiler/ir/backend.jvm/src/org/jetbrains/kotlin/backend/jvm/ir/JvmIrUtils.kt

@@ -61,6 +61,7 @@ import org.jetbrains.kotlin.name.JvmStandardClassIds.JVM_SYNTHETIC_ANNOTATION_FQ
 import org.jetbrains.kotlin.name.Name
 import org.jetbrains.kotlin.psi.KtFile
 import org.jetbrains.kotlin.resolve.DescriptorUtils
+import org.jetbrains.kotlin.resolve.JVM_NAME_ANNOTATION_FQ_NAME
 import org.jetbrains.kotlin.resolve.jvm.AsmTypes
 import org.jetbrains.kotlin.resolve.multiplatform.OptionalAnnotationUtil
 import org.jetbrains.kotlin.resolve.source.PsiSourceElement
@@ -249,23 +250,30 @@ val IrDeclaration.isStaticMultiFieldValueClassReplacement: Boolean
             || origin == JvmLoweredDeclarationOrigin.STATIC_MULTI_FIELD_VALUE_CLASS_CONSTRUCTOR
 
 fun IrDeclaration.shouldBeExposedByAnnotationOrFlag(languageVersionSettings: LanguageVersionSettings): Boolean {
-    if (!isFunctionWhichCanBeExposed()) return false
+    val isPropagatedOrImplicit = propagatedOrImplicitJvmExposeBoxed(languageVersionSettings)
+    val isExplicit = hasAnnotation(JVM_EXPOSE_BOXED_ANNOTATION_FQ_NAME)
+
+    if (!(isExplicit || isPropagatedOrImplicit)) return false
+    if (!isFunctionWhichCanBeExposed(isPropagatedOrImplicit && !isExplicit)) return false
     if (hasAnnotation(JVM_SYNTHETIC_ANNOTATION_FQ_NAME)) return false
 
-    return getAnnotation(JVM_EXPOSE_BOXED_ANNOTATION_FQ_NAME) != null ||
-            parentClassOrNull?.getAnnotation(JVM_EXPOSE_BOXED_ANNOTATION_FQ_NAME) != null ||
-            languageVersionSettings.supportsFeature(LanguageFeature.ImplicitJvmExposeBoxed)
+    return true
 }
 
+private fun IrDeclaration.propagatedOrImplicitJvmExposeBoxed(languageVersionSettings: LanguageVersionSettings): Boolean =
+    parentClassOrNull?.getAnnotation(JVM_EXPOSE_BOXED_ANNOTATION_FQ_NAME) != null ||
+            languageVersionSettings.supportsFeature(LanguageFeature.ImplicitJvmExposeBoxed)
+
 // Do not duplicate function without inline classes in parameters, since it would lead to CONFLICTING_JVM_DECLARATIONS
-private fun IrDeclaration.isFunctionWhichCanBeExposed(): Boolean {
+private fun IrDeclaration.isFunctionWhichCanBeExposed(isPropagatedOrImplicit: Boolean): Boolean {
     if (this !is IrFunction || origin == IrDeclarationOrigin.GENERATED_SINGLE_FIELD_VALUE_CLASS_MEMBER) return false
     // No sense in exposing suspend functions - they cannot be called from Java in normal way anyway
     if (isSuspend) return false
     // Cannot expose open or abstract - @JvmName problem
     if (isOverridable) return false
     if (parameters.any { it.type.isInlineClassType() }) return true
-    if (!returnType.isInlineClassType()) return false
+    // If the function is annotated with @JvmName, it is unmangled, so, we cannot implicitly expose it.
+    if (!returnType.isInlineClassType() || (isPropagatedOrImplicit && hasAnnotation(JVM_NAME_ANNOTATION_FQ_NAME))) return false
     // It is not explicitly annotated, global and returns inline class, do not expose it, since otherwise
     // it would lead to ambiguous call on Java side
     // WARNING: Do not use parentAsClass here, otherwise, it leads to ICE in some obscure cases, see KT-78551

compiler/testData/asJava/lightClasses/lightClassByPsi/jvmExposeBoxed/directive/methodReturnWithJvmName.lib.java

@@ -1,8 +1,4 @@
 public final class Foo /* Foo*/ {
-  @kotlin.jvm.JvmExposeBoxed(jvmName = "foo")
-  @org.jetbrains.annotations.NotNull()
-  public final StringWrapper foo();//  foo()
-
   @kotlin.jvm.JvmName(name = "foo")
   @org.jetbrains.annotations.NotNull()
   public final java.lang.String foo();//  foo()

compiler/testData/asJava/lightClasses/lightClassByPsi/jvmExposeBoxed/featureInteraction/jvmNameVariable.lib.java

@@ -1,12 +1,4 @@
 public final class Foo /* Foo*/ {
-  @kotlin.jvm.JvmExposeBoxed(jvmName = "foo11")
-  @org.jetbrains.annotations.NotNull()
-  public final StringWrapper foo11();//  foo11()
-
-  @kotlin.jvm.JvmExposeBoxed(jvmName = "foo41")
-  @org.jetbrains.annotations.NotNull()
-  public final StringWrapper foo41(int);//  foo41(int)
-
   @kotlin.jvm.JvmExposeBoxed(jvmName = "fooGetter22")
   @org.jetbrains.annotations.NotNull()
   public final StringWrapper fooGetter22();//  fooGetter22()

compiler/testData/asJava/lightClasses/lightClassByPsi/jvmExposeBoxed/featureInteraction/jvmNameVariableDirective.lib.java

@@ -1,12 +1,4 @@
 public final class Foo /* Foo*/ {
-  @kotlin.jvm.JvmExposeBoxed(jvmName = "foo11")
-  @org.jetbrains.annotations.NotNull()
-  public final StringWrapper foo11();//  foo11()
-
-  @kotlin.jvm.JvmExposeBoxed(jvmName = "foo41")
-  @org.jetbrains.annotations.NotNull()
-  public final StringWrapper foo41(int);//  foo41(int)
-
   @kotlin.jvm.JvmExposeBoxed(jvmName = "fooGetter22")
   @org.jetbrains.annotations.NotNull()
   public final StringWrapper fooGetter22();//  fooGetter22()

compiler/testData/asJava/lightClasses/lightClassByPsi/jvmExposeBoxed/featureInteraction/jvmOverloadsReturnTypeJvmNameDirective.lib.java

@@ -1,19 +1,4 @@
 public final class Baz /* Baz*/ {
-  @kotlin.jvm.JvmExposeBoxed(jvmName = "jvmMemberLevel")
-  @kotlin.jvm.JvmOverloads()
-  @org.jetbrains.annotations.NotNull()
-  public final IntWrapper jvmMemberLevel();//  jvmMemberLevel()
-
-  @kotlin.jvm.JvmExposeBoxed(jvmName = "jvmMemberLevel")
-  @kotlin.jvm.JvmOverloads()
-  @org.jetbrains.annotations.NotNull()
-  public final IntWrapper jvmMemberLevel(int);//  jvmMemberLevel(int)
-
-  @kotlin.jvm.JvmExposeBoxed(jvmName = "jvmMemberLevel")
-  @kotlin.jvm.JvmOverloads()
-  @org.jetbrains.annotations.NotNull()
-  public final IntWrapper jvmMemberLevel(int, int);//  jvmMemberLevel(int, int)
-
   @kotlin.jvm.JvmName(name = "jvmMemberLevel")
   @kotlin.jvm.JvmOverloads()
   public final int jvmMemberLevel();//  jvmMemberLevel()

compiler/testData/codegen/box/inlineClasses/jvmExposeBoxed/directive/methodReturnWithJvmName.txt

@@ -8,7 +8,6 @@ public final class BoxKt {
 public final class Foo {
     // source: 'IC.kt'
     public method <init>(): void
-    public final @kotlin.jvm.JvmExposeBoxed(jvmName="foo") @org.jetbrains.annotations.NotNull method foo(): StringWrapper
     public final @kotlin.jvm.JvmName(name="foo") @org.jetbrains.annotations.NotNull method foo(): java.lang.String
 }