Cleanup BytecodeEmitter field initialization

RESOLVED FIXED in Firefox 67

Status

()

enhancement
P3
normal
RESOLVED FIXED
4 months ago
4 months ago

People

(Reporter: tcampbell, Assigned: tcampbell)

Tracking

unspecified
mozilla67
Points:
---

Firefox Tracking Flags

(firefox67 fixed)

Details

Attachments

(2 attachments, 1 obsolete attachment)

Assignee

Description

4 months ago

Migrate the giant constructor initialization list to field default values instead. Also some miscellaneous cleanups.

Assignee

Comment 3

4 months ago

Also replace a few packed bitfields with normal fields since this is a
stack type only.

Depends on D20885

Attachment #9046067 - Attachment is obsolete: true

Comment 4

4 months ago
Pushed by tcampbell@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/a69e03c02d34
Use mozilla::Maybe in BytecodeEmitter when reasonable. r=jorendorff
https://hg.mozilla.org/integration/autoland/rev/942730f2550d
Use default initializers for BytecodeEmitter fields. r=jorendorff

Comment 5

4 months ago
bugherder
Status: NEW → RESOLVED
Closed: 4 months ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla67
You need to log in before you can comment on or make changes to this bug.