Closed
Bug 1119783
Opened 9 years ago
Closed 9 years ago
IonMonkey MIPS: Fix latest build issues (unboxObject)
Categories
(Core :: JavaScript Engine: JIT, defect)
Tracking
()
RESOLVED
FIXED
mozilla38
People
(Reporter: rankov, Assigned: rankov)
Details
Attachments
(1 file, 1 obsolete file)
3.48 KB,
patch
|
rankov
:
review+
|
Details | Diff | Splinter Review |
No description provided.
Assignee | ||
Comment 1•9 years ago
|
||
Attachment #8546613 -
Flags: review?(jdemooij)
Comment 2•9 years ago
|
||
Comment on attachment 8546613 [details] [diff] [review] 1119783-Fix-build.patch Review of attachment 8546613 [details] [diff] [review]: ----------------------------------------------------------------- ::: js/src/jit/mips/MacroAssembler-mips.cpp @@ +2564,5 @@ > void > +MacroAssemblerMIPSCompat::unboxNonDouble(const BaseIndex &src, Register dest) > +{ > + computeScaledAddress(src, SecondScratchReg); > + ma_lw(dest, Address(SecondScratchReg, PAYLOAD_OFFSET)); We should either use src.offset + PAYLOAD_OFFSET or assert src.offset == 0, right?
Attachment #8546613 -
Flags: review?(jdemooij) → review+
Assignee | ||
Comment 3•9 years ago
|
||
(In reply to Jan de Mooij [:jandem] from comment #2) > Comment on attachment 8546613 [details] [diff] [review] > 1119783-Fix-build.patch > > Review of attachment 8546613 [details] [diff] [review]: > ----------------------------------------------------------------- > > ::: js/src/jit/mips/MacroAssembler-mips.cpp > @@ +2564,5 @@ > > void > > +MacroAssemblerMIPSCompat::unboxNonDouble(const BaseIndex &src, Register dest) > > +{ > > + computeScaledAddress(src, SecondScratchReg); > > + ma_lw(dest, Address(SecondScratchReg, PAYLOAD_OFFSET)); > > We should either use src.offset + PAYLOAD_OFFSET or assert src.offset == 0, > right? You are right. I will fix this.
QA Contact: branislav.rankov
Assignee | ||
Comment 4•9 years ago
|
||
Carry review from previous patch.
Assignee: nobody → branislav.rankov
Attachment #8546613 -
Attachment is obsolete: true
Status: NEW → ASSIGNED
Attachment #8547651 -
Flags: review+
Assignee | ||
Comment 5•9 years ago
|
||
https://hg.mozilla.org/integration/mozilla-inbound/rev/2f2b89a7f71a
https://hg.mozilla.org/mozilla-central/rev/2f2b89a7f71a
Status: ASSIGNED → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla38
You need to log in
before you can comment on or make changes to this bug.
Description
•