Need this for SS math-cordic.

Patch for m-c to move NaNCond and related functions.

Also fixes ARM FloatReg0: ReturnFloatReg was changed from d1 to d0 recently.

Comment on attachment 691821 [details] [diff] [review] Part 2: Double compare stub Review of attachment 691821 [details] [diff] [review]: ----------------------------------------------------------------- ::: js/src/ion/BaselineIC.h @@ +720,5 @@ > } > > + void ensureDouble(MacroAssembler &masm, const ValueOperand &source, FloatRegister dest, > + Label *failure); > + This really feels like it should go on MacroAssembler. The only baseline-specific part of ensureDouble is the usage of "ExtractTemp0", but since only one is needed, x64 can jut use its scratch reg for that, and x86 and arm don't need one. @@ +1087,5 @@ > { > friend class ICStubSpace; > > ICCompare_Int32(IonCode *stubCode) > + : ICStub(ICStub::Compare_Int32, stubCode) {} Nice catch. ::: js/src/ion/arm/BaselineIC-arm.cpp @@ +57,5 @@ > + Assembler::Condition cond = Assembler::ConditionFromDoubleCondition(doubleCond); > + > + masm.compareDouble(FloatReg0, FloatReg1); > + masm.ma_mov(Imm32(0), dest); > + masm.ma_mov(Imm32(1), dest, NoSetCond, cond); The first mov(Imm32(0), dest) should be NoSetCond as well.

https://hg.mozilla.org/projects/ionmonkey/rev/c255129f4994 (In reply to Kannan Vijayan [:djvj] from comment #3) > > The first mov(Imm32(0), dest) should be NoSetCond as well. NoSetCond is the default.