Do not remove accessors to early

Accessors that are added to packages were removed to early from `accessedBy`.
This resulted in a new symbol beeing entered for the same accessor.

Fixes #13476

Author: nicolasstucki

compiler/src/dotty/tools/dotc/transform/AccessProxies.scala

@@ -31,12 +31,9 @@ abstract class AccessProxies {
    */
   protected def passReceiverAsArg(accessorName: Name)(using Context): Boolean = false
 
-  /** The accessor definitions that need to be added to class `cls`
-   *  As a side-effect, this method removes entries from the `accessedBy` map.
-   *  So a second call of the same method will yield the empty list.
-   */
+  /** The accessor definitions that need to be added to class `cls` */
   private def accessorDefs(cls: Symbol)(using Context): Iterator[DefDef] =
-    for (accessor <- cls.info.decls.iterator; accessed <- accessedBy.remove(accessor).toOption) yield
+    for accessor <- cls.info.decls.iterator; accessed <- accessedBy.get(accessor) yield
       DefDef(accessor.asTerm, prefss => {
         def numTypeParams = accessed.info match {
           case info: PolyType => info.paramNames.length

tests/pos/i13476.scala

@@ -0,0 +1,7 @@
+private object Foo:
+  inline def d(arg : Int): Unit = {}
+  transparent inline def haha() : Unit = {}
+
+export Foo.*
+
+@main def main : Unit = haha()