Mark RuntimeLong.toDouble as fully inline.

sjrd · sjrd · commit c8c8050aa7e5 · 2025-06-21T23:11:15.000+02:00
Its implementation is shorter than many of the methods we already
inline, like additions and multiplications. We do pay a little
code size cost for this change.

This will be particularly useful for the following commit, which
will rely on folding to happen within `toDouble`, once inlined at
call site.
diff --git a/linker-private-library/src/main/scala/org/scalajs/linker/runtime/RuntimeLong.scala b/linker-private-library/src/main/scala/org/scalajs/linker/runtime/RuntimeLong.scala
@@ -695,10 +695,9 @@ object RuntimeLong {
     a.lo
 
   @inline
-  def toDouble(a: RuntimeLong): Double =
-    toDouble(a.lo, a.hi)
-
-  private def toDouble(lo: Int, hi: Int): Double = {
+  def toDouble(a: RuntimeLong): Double = {
+    val lo = a.lo
+    val hi = a.hi
     if (hi < 0) {
       // We do asUint() on the hi part specifically for MinValue
       val neg = inline_negate(lo, hi)
diff --git a/project/Build.scala b/project/Build.scala
@@ -2060,17 +2060,17 @@ object Build {
               ))
             } else {
               Some(ExpectedSizes(
-                  fastLink = 425000 to 426000,
-                  fullLink = 282000 to 283000,
-                  fastLinkGz = 60000 to 61000,
+                  fastLink = 426000 to 427000,
+                  fullLink = 283000 to 284000,
+                  fastLinkGz = 61000 to 62000,
                   fullLinkGz = 43000 to 44000,
               ))
             }
 
           case `default213Version` =>
             if (!useMinifySizes) {
               Some(ExpectedSizes(
-                  fastLink = 442000 to 443000,
+                  fastLink = 443000 to 444000,
                   fullLink = 90000 to 91000,
                   fastLinkGz = 57000 to 58000,
                   fullLinkGz = 24000 to 25000,
