nanojit: add OPCODE and IMM8 to i386 backend

RESOLVED FIXED

Status

Core Graveyard
Nanojit
RESOLVED FIXED
6 years ago
3 years ago

People

(Reporter: njn, Assigned: njn)

Tracking

Firefox Tracking Flags

(Not tracked)

Details

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

Attachments

(1 attachment)

(Assignee)

Description

6 years ago
Created attachment 517346 [details] [diff] [review]
patch (against TM 63014:513774b1e88e)

This patch gets rid of all naked _nIns writes, replacing them with functions like OPCODE and IMM8 which make the meaning of the bytes clearer.
Attachment #517346 - Flags: review?(rreitmai)

Updated

6 years ago
Attachment #517346 - Flags: review?(rreitmai) → review+
(Assignee)

Comment 1

6 years ago
http://hg.mozilla.org/projects/nanojit-central/rev/b0decf655268
http://hg.mozilla.org/tracemonkey/rev/b445bf49c96c
Whiteboard: fixed-in-nanojit, fixed-in-tracemonkey

Comment 2

6 years ago
changeset: 6083:43242d41caec
user:      Nicholas Nethercote <nnethercote@mozilla.com>
summary:   Bug 639418 - nanojit: add OPCODE and IMM8 to i386 backend.  r=rreitmai.

http://hg.mozilla.org/tamarin-redux/rev/43242d41caec
(Assignee)

Comment 3

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