Closed
Bug 658502
Opened 13 years ago
Closed 6 years ago
SH4: use REGNUM() to access the Register number in asm_output() parameters
Categories
(Tamarin Graveyard :: Baseline JIT (CodegenLIR), defect, P3)
Tracking
(Not tracked)
RESOLVED
WONTFIX
Q1 12 - Brannan
People
(Reporter: cedric.vincent, Assigned: cedric.vincent)
Details
(Whiteboard: loose-end,fixed-in-nanojit,fixed-in-tr)
Attachments
(1 file)
User-Agent: Opera/9.80 (X11; Linux x86_64; U; en) Presto/2.7.62 Version/11.01
Build Identifier:
This patch fixes the following compilation error(s) when NJ_VERBOSE is
defined but DEBUG isn't:
In member function ‘void nanojit::Assembler::SH4_add_imm(int, nanojit::Register)’:
error: request for member ‘n’ in ‘Rx’, which is of non-class type ‘nanojit::Register’
...
Reported and fixed by: Anandha-kumar CHINNAIYAN <anandha.chinnaiyan@st.com>
Reproducible: Always
|
Assignee | |
Comment 1•13 years ago
|
||
Cédric, do you need to have this pushed?
Flags: flashplayer-qrb+
Flags: flashplayer-bug-
Target Milestone: --- → Future
|
|
Assignee | |
Comment 3•13 years ago
|
||
(In reply to Dan Smith from comment #2) > Cédric, do you need to have this pushed? Yes, please. Thanks.
Rick, land Cedric's change.
Flags: flashplayer-injection-
Priority: -- → P3
Whiteboard: loose-end
Target Milestone: Future → Q1 12 - Brannan
|
||
Comment 5•13 years ago
|
||
cedric http://hg.mozilla.org/projects/nanojit-central/rev/dbdf2176179e
Whiteboard: loose-end → loose-end,fixed-in-nanojit
|
||
Comment 6•13 years ago
|
||
changeset: 6627:3182a0d42af2 user: =?UTF-8?q?C=C3=A9dric=20VINCENT?= <cedric.vincent@st.com> summary: Bug 658502 SH4: use REGNUM() to access the Register number in http://hg.mozilla.org/tamarin-redux/rev/3182a0d42af2
Whiteboard: loose-end,fixed-in-nanojit → loose-end,fixed-in-nanojit,fixed-in-tr
|
||
Comment 7•6 years ago
|
||
Tamarin is a dead project now. Mass WONTFIX.
Status: UNCONFIRMED → RESOLVED
Closed: 6 years ago
Resolution: --- → WONTFIX
|
|
||
Comment 8•6 years ago
|
||
Tamarin isn't maintained anymore. WONTFIX remaining bugs.
You need to log in
before you can comment on or make changes to this bug.
Description
•