Filter out static forwarders for Java package private methods

Co-authored-by: Guillaume Martres <[email protected]>

Author: vasilmkd

compiler/src/dotty/tools/dotc/core/SymDenotations.scala

@@ -1388,7 +1388,7 @@ object SymDenotations {
     final def accessBoundary(base: Symbol)(using Context): Symbol =
       if (this.is(Private)) owner
       else if (this.isAllOf(StaticProtected)) defn.RootClass
-      else if (privateWithin.exists && !ctx.phase.erasedTypes) privateWithin
+      else if (privateWithin.exists && (!ctx.phase.erasedTypes || this.is(JavaDefined))) privateWithin
       else if (this.is(Protected)) base
       else defn.RootClass
 

tests/run/forwarder-java-package-private.check

@@ -0,0 +1,2 @@
+public static void MyClassValue.remove(java.lang.Class)
+public static java.lang.Object MyClassValue.get(java.lang.Class)

tests/run/forwarder-java-package-private.scala

@@ -0,0 +1,12 @@
+object MyClassValue extends ClassValue[String] {
+  override protected def computeValue(cls: Class[_]): String = ""
+}
+
+object Test {
+  def main(args: Array[String]): Unit = {
+    println(
+      Class.forName("MyClassValue").getMethods
+        .filter(m => (m.getModifiers & java.lang.reflect.Modifier.STATIC) != 0)
+        .mkString("\n"))
+  }
+}