If you think a bug might affect users in the 57 release, please set the correct tracking and status flags for Release Management.

abcasm: make OP_label explicit, not implicit

VERIFIED FIXED

Status

Tamarin
Tools
VERIFIED FIXED
9 years ago
8 years ago

People

(Reporter: Tom Harwood, Assigned: Tom Harwood)

Tracking

Details

Attachments

(1 attachment)

(Assignee)

Description

9 years ago
Created attachment 364995 [details] [diff] [review]
Proposed patch

The assembler's L1: style labels should only guide the assembler's internal calculations.  If the program needs a label, then it should be explicitly coded.

Attached patch:
- never generates an OP_label by assembler-internal calculations.
- supports label as an instruction so it can be coded into the program.
- modifies the two testcases that need a label on a backwards branch.
(Assignee)

Updated

9 years ago
Attachment #364995 - Flags: review?(edwsmith)

Updated

9 years ago
Attachment #364995 - Flags: review?(edwsmith) → review+
(Assignee)

Updated

9 years ago
Status: NEW → RESOLVED
Last Resolved: 9 years ago
Resolution: --- → FIXED

Comment 1

8 years ago
http://hg.mozilla.org/tamarin-redux/rev/2c333f557e38
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.