Closed
Bug 1202138
Opened 9 years ago
Closed 9 years ago
IonMonkey: MIPS32: Fix callAndPushReturnAddress with Label
Categories
(Core :: JavaScript Engine: JIT, defect)
Tracking
()
RESOLVED
FIXED
mozilla43
Tracking | Status | |
---|---|---|
firefox43 | --- | fixed |
People
(Reporter: hev, Assigned: hev)
Details
Attachments
(2 files)
850 bytes,
patch
|
nbp
:
review+
|
Details | Diff | Splinter Review |
819 bytes,
patch
|
hev
:
review+
cbook
:
checkin+
|
Details | Diff | Splinter Review |
In file included from /home/heiher/git/mips64/gecko-dev/js/src/mips32-release/js/src/Unified_cpp_js_src7.cpp:11:0: /home/heiher/git/mips64/gecko-dev/js/src/jit/mips32/MacroAssembler-mips32.cpp: In member function 'void js::jit::MacroAssembler::callAndPushReturnAddress(js::jit::Label*)': /home/heiher/git/mips64/gecko-dev/js/src/jit/mips32/MacroAssembler-mips32.cpp:3463:18: error: no matching function for call to 'js::jit::MacroAssembler::as_jalr(js::jit::Label*&)' as_jalr(label); ^
Assignee | ||
Updated•9 years ago
|
Assignee: nobody → r
Assignee | ||
Comment 1•9 years ago
|
||
Attachment #8657444 -
Flags: review?(nicolas.b.pierron)
Updated•9 years ago
|
Attachment #8657444 -
Flags: review?(nicolas.b.pierron) → review+
Comment 2•9 years ago
|
||
Comment on attachment 8657444 [details] [diff] [review] 0001-IonMonkey-MIPS32-Fix-callAndPushReturnAddress-with-L.patch Review of attachment 8657444 [details] [diff] [review]: ----------------------------------------------------------------- ::: js/src/jit/mips32/MacroAssembler-mips32.cpp @@ +3460,2 @@ > { > // Push return address during jalr delay slot. Update this comment as well.
Assignee | ||
Comment 3•9 years ago
|
||
Update comment based on Attachment 8657444 [details] [diff], please check-in this patch.
Assignee | ||
Updated•9 years ago
|
Attachment #8657803 -
Flags: checkin?
Assignee | ||
Updated•9 years ago
|
Keywords: checkin-needed
Assignee | ||
Updated•9 years ago
|
Attachment #8657803 -
Flags: review+
Updated•9 years ago
|
Attachment #8657803 -
Flags: checkin? → checkin+
Comment 5•9 years ago
|
||
https://hg.mozilla.org/mozilla-central/rev/1af3c0789763
Status: NEW → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla43
You need to log in
before you can comment on or make changes to this bug.
Description
•