Bug 881382 - Cleanup some ARM code to be more consistent with the corresponding x86 code. r=mjrosenb
authorDan Gohman <sunfish@google.com>
Tue, 11 Jun 2013 18:50:27 -0700
changeset 146225 f2a96f9d1ef64e68cfdefeb2d3c7417086a586a1
parent 146224 e78d02f5e0f63fa426f3ba4eba6cf4c78181bbc0
child 146226 216620b31732b1c0f27eeadc51d9d58f880875e3
push id2697
push userbbajaj@mozilla.com
push dateMon, 05 Aug 2013 18:49:53 +0000
treeherdermozilla-beta@dfec938c7b63 [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewersmjrosenb
bugs881382
milestone24.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 881382 - Cleanup some ARM code to be more consistent with the corresponding x86 code. r=mjrosenb
js/src/ion/arm/Lowering-arm.cpp
--- a/js/src/ion/arm/Lowering-arm.cpp
+++ b/js/src/ion/arm/Lowering-arm.cpp
@@ -36,19 +36,17 @@ LIRGeneratorARM::useBoxFixed(LInstructio
     lir->setOperand(n, LUse(reg1, mir->virtualRegister()));
     lir->setOperand(n + 1, LUse(reg2, VirtualRegisterOfPayload(mir)));
     return true;
 }
 
 bool
 LIRGeneratorARM::lowerConstantDouble(double d, MInstruction *mir)
 {
-    uint32_t index;
-    LDouble *lir = new LDouble(d);
-    return define(lir, mir);
+    return define(new LDouble(d), mir);
 }
 
 bool
 LIRGeneratorARM::visitConstant(MConstant *ins)
 {
     if (ins->type() == MIRType_Double) {
         LDouble *lir = new LDouble(ins->value().toDouble());
         return define(lir, ins);