Bug 1874502: Use ScratchDoubleScope in moveValue. r=jandem, a=RyanVM

Differential Revision: https://phabricator.services.mozilla.com/D198479
goastian · Feb 28, 2024 · 1575b6b · 1575b6b
1 parent 937d938
commit 1575b6b
Show file tree

Hide file tree

Showing 2 changed files with 9 additions and 1 deletion.
diff --git a/js/src/jit-test/tests/ion/bug1874502.js b/js/src/jit-test/tests/ion/bug1874502.js
@@ -0,0 +1,8 @@
+// |jit-test| --no-threads; --fast-warmup
+
+function f(x) {
+    Math.fround(function () { x; });
+  }
+  for (let i = 0; i < 30; i++) {
+    f(Math.fround(1));
+  }
diff --git a/js/src/jit/arm/MacroAssembler-arm.cpp b/js/src/jit/arm/MacroAssembler-arm.cpp
@@ -4592,7 +4592,7 @@ void MacroAssembler::moveValue(const TypedOrValueRegister& src,
     return;
   }
 
-  ScratchFloat32Scope scratch(*this);
+  ScratchDoubleScope scratch(*this);
   FloatRegister freg = reg.fpu();
   if (type == MIRType::Float32) {
     convertFloat32ToDouble(freg, scratch);