1150384 - IonMonkey: MIPS: Fix build failure caused by Bug 1143011.

Status: RESOLVED FIXED

(Core :: JavaScript Engine: JIT, defect)

Description

[hev [:hev]]

/home/heiher/git/loongson/gecko-dev/js/src/jit/mips/Architecture-mips.cpp:109:17: error: ‘js::jit::FloatRegisterSet’ has no member named
 ‘addUnchecked’
             mod.addUnchecked((*iter).doubleOverlay());
/home/heiher/git/loongson/gecko-dev/js/src/jit/mips/Architecture-mips.cpp:111:17: error: ‘js::jit::FloatRegisterSet’ has no member named
 ‘addUnchecked’
             mod.addUnchecked(*iter);
/home/heiher/git/loongson/gecko-dev/js/src/jit/mips/BaselineIC-mips.cpp:84:58: error: conversion from ‘js::jit::AllocatableGeneralRegist
erSet {aka js::jit::AllocatableSet<js::jit::TypedRegisterSet<js::jit::Register> >}’ to non-scalar type ‘js::jit::GeneralRegisterSet {aka
 js::jit::TypedRegisterSet<js::jit::Register>}’ requested
     GeneralRegisterSet savedRegs = availableGeneralRegs(2);
....

Comment 1

[hev [:hev]]

Attachment: IonMonkey-MIPS-Fix-build-failure-caused-by-Bug-11430.patch

Comment 2

[Branislav Rankov [:rankov]]

Comment on attachment 8587206 [details] [diff] [review]
IonMonkey-MIPS-Fix-build-failure-caused-by-Bug-11430.patch

Review of attachment 8587206 [details] [diff] [review]:
-----------------------------------------------------------------

::: js/src/jit/mips/BaselineIC-mips.cpp
@@ +81,5 @@
>      Register scratchReg = R2.payloadReg();
>  
>      // DIV and MOD need an extra non-volatile ValueOperand to hold R0.
> +    AllocatableGeneralRegisterSet savedRegs(availableGeneralRegs(2));
> +    savedRegs.set() = GeneralRegisterSet::Intersect(GeneralRegisterSet::NonVolatile(), savedRegs.set());

nit: this line is too long. It should be less than 99 characters.

Comment 3

[Ryan VanderMeulen [:RyanVM]]

https://hg.mozilla.org/integration/mozilla-inbound/rev/e960d9b2e6fd

Comment 4

[Phil Ringnalda (:philor)]

https://hg.mozilla.org/mozilla-central/rev/e960d9b2e6fd