ARM bustage fix for bug 507089. r=me.
authorNicholas Nethercote <nnethercote@mozilla.com>
Mon, 22 Mar 2010 12:18:47 -0700
changeset 40347 30e0a4ed98b5adea50bdd7aeb9ce475025a71731
parent 40346 c55d266e9f79ee3c84c5844ad1b42abdd593aa74
child 40348 656054962a9ebaac6236bd4daefbbfef34fbaef4
push id12610
push userrsayre@mozilla.com
push dateMon, 05 Apr 2010 17:26:41 +0000
treeherdermozilla-central@1942c0b4e101 [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewersme
bugs507089
milestone1.9.3a3pre
ARM bustage fix for bug 507089. r=me.
js/src/nanojit/NativeARM.cpp
--- a/js/src/nanojit/NativeARM.cpp
+++ b/js/src/nanojit/NativeARM.cpp
@@ -857,19 +857,16 @@ Assembler::asm_call(LInsp ins)
     // function call.
     NanoAssert(_config.arm_vfp || ins->isop(LIR_icall));
 
     // If we're using VFP, and the return type is a double, it'll come back in
     // R0/R1. We need to either place it in the result fp reg, or store it.
     // See comments above for more details as to why this is necessary here
     // for floating point calls, but not for integer calls.
     if (_config.arm_vfp && ins->isUsed()) {
-        // Determine the size (and type) of the instruction result.
-        ArgType rsize = (ArgType)(ci->_typesig & ARGTYPE_MASK);
-
         // If the result size is a floating-point value, treat the result
         // specially, as described previously.
         if (ci->returnType() == ARGTYPE_F) {
             Register rr = ins->deprecated_getReg();
 
             NanoAssert(ins->opcode() == LIR_fcall);
 
             if (!deprecated_isKnownReg(rr)) {