Scala.js: Fix references to fields in other compilation units.

Author: sjrd

compiler/src/dotty/tools/backend/sjs/JSEncoding.scala

@@ -1,13 +1,17 @@
 package dotty.tools.backend.sjs
 
+import scala.annotation.tailrec
+
 import scala.collection.mutable
 
 import dotty.tools.FatalError
 
 import dotty.tools.dotc.core._
+import Decorators._
 import Periods._
 import SymDenotations._
 import Contexts._
+import Flags._
 import Types._
 import Symbols._
 import Denotations._
@@ -103,11 +107,17 @@ object JSEncoding {
     js.Ident(localNames.localSymbolName(sym), Some(sym.unexpandedName.decoded))
   }
 
+  private[this] var allRefClassesCache: Set[Symbol] = _
   private def allRefClasses(implicit ctx: Context): Set[Symbol] = {
-    //TODO
-    /*(Set(ObjectRefClass, VolatileObjectRefClass) ++
-        refClass.values ++ volatileRefClass.values)*/
-    Set()
+    if (allRefClassesCache == null) {
+      val baseNames = List("Object", "Boolean", "Character", "Byte", "Short",
+          "Int", "Long", "Float", "Double")
+      val fullNames = baseNames.flatMap { base =>
+        List(s"scala.runtime.${base}Ref", s"scala.runtime.Volatile${base}Ref")
+      }
+      allRefClassesCache = fullNames.map(name => ctx.requiredClass(name)).toSet
+    }
+    allRefClassesCache
   }
 
   def encodeFieldSym(sym: Symbol)(
@@ -120,14 +130,19 @@ object JSEncoding {
       if (name0.charAt(name0.length()-1) != ' ') name0
       else name0.substring(0, name0.length()-1)
 
+    @tailrec
+    def superClassCount(sym: Symbol, acc: Int): Int =
+      if (sym == defn.ObjectClass) acc
+      else superClassCount(sym.asClass.superClass, acc + 1)
+
     /* We have to special-case fields of Ref types (IntRef, ObjectRef, etc.)
      * because they are emitted as private by our .scala source files, but
      * they are considered public at use site since their symbols come from
      * Java-emitted .class files.
      */
     val idSuffix =
       if (sym.is(Flags.Private) || allRefClasses.contains(sym.owner))
-        sym.owner.asClass.baseClasses.size.toString
+        superClassCount(sym.owner, 0).toString
       else
         "f"
 

project/Build.scala

@@ -948,6 +948,7 @@ object Build {
         (
           (dir / "shared/src/test/scala/org/scalajs/testsuite/compiler" ** "IntTest.scala").get
           ++ (dir / "shared/src/test/scala/org/scalajs/testsuite/javalib/lang" ** "IntegerTest.scala").get
+          ++ (dir / "shared/src/test/scala/org/scalajs/testsuite/javalib/lang" ** "ObjectTest.scala").get
           ++ (dir / "shared/src/test/require-jdk8/org/scalajs/testsuite/javalib/util" ** "Base64Test.scala").get
           ++ (dir / "shared/src/test/scala/org/scalajs/testsuite/utils" ** "*.scala").get
         )