Scala.js: Do not emit definitions for primitive methods.

Author: sjrd

compiler/src/dotty/tools/backend/jvm/scalaPrimitives.scala

@@ -397,6 +397,9 @@ class DottyPrimitives(ictx: Context) {
     primitives.toMap
   }
 
+  def isPrimitive(sym: Symbol): Boolean =
+    primitives.contains(sym)
+
   def isPrimitive(fun: Tree): Boolean =
     given Context = ictx
     primitives.contains(fun.symbol)
@@ -407,4 +410,3 @@ class DottyPrimitives(ictx: Context) {
             case _ => true
         })
 }
-

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

@@ -791,9 +791,9 @@ class JSCodeGen()(using genCtx: Context) {
             toIRType(param.info), mutable = false, rest = false)
       }
 
-      /*if (primitives.isPrimitive(sym)) {
+      if (primitives.isPrimitive(sym)) {
         None
-      } else*/ if (sym.is(Deferred)) {
+      } else if (sym.is(Deferred)) {
         Some(js.MethodDef(js.MemberFlags.empty, methodName, originalName,
             jsParams, toIRType(patchedResultType(sym)), None)(
             OptimizerHints.empty, None))

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

@@ -59,6 +59,9 @@ class JSPrimitives(ictx: Context) extends DottyPrimitives(ictx) {
   override def getPrimitive(app: Apply, tpe: Type)(using Context): Int =
     jsPrimitives.getOrElse(app.fun.symbol, super.getPrimitive(app, tpe))
 
+  override def isPrimitive(sym: Symbol): Boolean =
+    jsPrimitives.contains(sym) || super.isPrimitive(sym)
+
   override def isPrimitive(fun: Tree): Boolean =
     jsPrimitives.contains(fun.symbol(using ictx)) || super.isPrimitive(fun)