Simplify the unboxing functions.

sjrd · sjrd · commit 0deaf4fca80e · 2024-05-27T14:23:52.000+02:00
We can rely on the `ToNumber(v)` conversions performed by
`ToWebAssemblyValue` instead of doing `null` handling ourselves.
diff --git a/linker/shared/src/main/scala/org/scalajs/linker/backend/wasmemitter/LoaderContent.scala b/linker/shared/src/main/scala/org/scalajs/linker/backend/wasmemitter/LoaderContent.scala
@@ -102,13 +102,13 @@ const scalaJSHelpers = {
   bF: (x) => x,
   bD: (x) => x,
 
-  // Unboxes (downcast, null is converted to the zero of the type)
-  uZ: (x) => x | 0,
-  uB: (x) => (x << 24) >> 24,
-  uS: (x) => (x << 16) >> 16,
-  uI: (x) => x | 0,
-  uF: (x) => Math.fround(x),
-  uD: (x) => +x,
+  // Unboxes (downcast, null is converted to the zero of the type as part of ToWebAssemblyValue)
+  uZ: (x) => x, // ToInt32 turns false into 0 and true into 1, so this is also an identity
+  uB: (x) => x,
+  uS: (x) => x,
+  uI: (x) => x,
+  uF: (x) => x,
+  uD: (x) => x,
 
   // Type tests
   tZ: (x) => typeof x === 'boolean',
