Closed Bug 1635464 Opened 4 years ago Closed 4 years ago

Box the ParseError to reduce the size of Result type.

Categories

(Core :: JavaScript Engine, enhancement, P1)

Product:
Core
Component:
JavaScript Engine
Type:
enhancement

Tracking

()

Status:
RESOLVED FIXED
Milestone:
mozilla78
Tracking Flags:
Tracking Status
firefox78 --- fixed

People

(Reporter: nbp, Assigned: nbp)

References

(Blocks 1 open bug)

Details

(Whiteboard: [smooshmonkey-mvp])

Attachments

(1 file)

Bug 1635464 - Deref BoxedParseError to read the content.
47 bytes, text/x-phabricator-request
Details | Review

Integration of: https://github.com/mozilla-spidermonkey/jsparagus/pull/505

The linked pull request changed the Result type to use a boxed ParseError, in order to reduce the overhead caused by the size of the Result type.

This bug is about modifying SmooshMonkey code to unbox the content of the error case, to read the ParseError.

Whiteboard: [smooshmonkey-mvp]
Severity: -- → N/A
Priority: -- → P1
Pushed by npierron@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/3ca0da71c4ce Deref BoxedParseError to read the content. r=arai

https://hg.mozilla.org/mozilla-central/rev/3ca0da71c4ce

Status: ASSIGNED → RESOLVED
Closed: 4 years ago
status-firefox78: --- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla78
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: