stylo: align binary representations of computed keyword values and eliminate the match statements

ASSIGNED
Assigned to

Status

()

enhancement
P3
normal
ASSIGNED
2 years ago
a year ago

People

(Reporter: bholley, Assigned: heycam)

Tracking

(Blocks 3 bugs)

unspecified
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(firefox57 wontfix, firefox58 wontfix, firefox59 ?)

Details

Attachments

(1 attachment)

Reporter

Description

2 years ago
We have a bunch of match statements right now that we can probably get rid of. They aren't showing up in the profiles I've looked at so far, so we may be able to ship without this. Getting it on file in any case.
Reporter

Updated

2 years ago
Priority: -- → P4
status-firefox57=wontfix unless someone thinks this bug should block 57
Priority: P4 → P3
Assignee

Updated

2 years ago
Assignee: nobody → cam
Status: NEW → ASSIGNED
Assignee

Comment 2

2 years ago
I have a patch that converts most keyword glue function match statements into assertions and assignments, but it only saves around 25 KiB in libxul.
Comment hidden (mozreview-request)
You need to log in before you can comment on or make changes to this bug.