Bug 1047332: Remove unused src2 variable in ARM's move emitter; r=mjrosenb
authorBenjamin Bouvier <benj@benj.me>
Mon, 04 Aug 2014 10:45:55 +0200
changeset 197568 13f79445761e899e38f7aca9187f5a5c015c5142
parent 197567 b61ba0db784ac71b06096e3b765becc3da3301d5
child 197569 b1552199cc7cee534308d4c82823699be3e42104
push id47165
push userbenj@benj.me
push dateMon, 04 Aug 2014 08:46:42 +0000
treeherdermozilla-inbound@b1552199cc7c [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewersmjrosenb
bugs1047332
milestone34.0a1
first release with
nightly linux32
nightly linux64
nightly mac
nightly win32
nightly win64
last release without
nightly linux32
nightly linux64
nightly mac
nightly win32
nightly win64
Bug 1047332: Remove unused src2 variable in ARM's move emitter; r=mjrosenb
js/src/jit/arm/MoveEmitter-arm.cpp
--- a/js/src/jit/arm/MoveEmitter-arm.cpp
+++ b/js/src/jit/arm/MoveEmitter-arm.cpp
@@ -117,17 +117,16 @@ MoveEmitterARM::breakCycle(const MoveOpe
             // Since it is uncertain if the load will be aligned or not
             // just fill both of them with the same value.
             masm.ma_vstr(temp, cycleSlot(slotId, 0));
             masm.ma_vstr(temp, cycleSlot(slotId, 4));
         } else {
             FloatRegister src = to.floatReg();
             // Just always store the largest possible size. Currently, this is
             // a double. When SIMD is added, two doubles will need to be stored.
-            FloatRegister src2 = src.doubleOverlay();
             masm.ma_vstr(src.doubleOverlay(), cycleSlot(slotId, 0));
         }
         break;
       case MoveOp::DOUBLE:
         if (to.isMemory()) {
             FloatRegister temp = ScratchDoubleReg;
             masm.ma_vldr(toOperand(to, true), temp);
             masm.ma_vstr(temp, cycleSlot(slotId, 0));