Status

defect
RESOLVED FIXED
8 years ago
5 years ago

People

(Reporter: njn, Assigned: njn)

Tracking

Firefox Tracking Flags

(Not tracked)

Details

(Whiteboard: fixed-in-nanojit, fixed-in-tracemonkey)

Attachments

(2 attachments)

Assignee

Description

8 years ago
Bug 620757 changed TM so it no longer traces table switches.  That means that support from them can be removed from Nanojit.  I think this includes SwitchInfo and LIR_jtbl.

Comment 1

8 years ago
TR uses LIR_jtbl, but LIR_jtbl doesn't use SwitchInfo.  LIR_xtbl does, and only on x86.
Assignee

Comment 2

8 years ago
My mistake, I meant LIR_xtbl all along! :)
Summary: nanojit: remove LIR_jtbl → nanojit: remove LIR_xtbl
Assignee

Comment 4

8 years ago
Attachment #516505 - Flags: review?(dmandelin)

Comment 5

8 years ago
Comment on attachment 516504 [details] [diff] [review]
NJ patch (against TM 62988:40f500ecdfd0)

Passed TR acceptance tests on mac x86 and x64, no surprises in patch.
Attachment #516504 - Flags: review?(edwsmith) → review+
Attachment #516505 - Flags: review?(dmandelin) → review+
Assignee

Comment 7

8 years ago
Apologies for omitting the bug number in the commit logs for the above patches :(
Assignee

Comment 8

8 years ago
http://hg.mozilla.org/mozilla-central/rev/de384e76ed18
Status: ASSIGNED → RESOLVED
Last Resolved: 8 years ago
Resolution: --- → FIXED
Component: Nanojit → Nanojit
Product: Core → Core Graveyard
You need to log in before you can comment on or make changes to this bug.