Remove parserGlobal and mergeRealms
Categories
(Core :: JavaScript Engine, task, P2)
Tracking
()
| Tracking | Status | |
|---|---|---|
| firefox96 | --- | fixed |
People
(Reporter: djvj, Assigned: arai)
References
(Blocks 1 open bug)
Details
Attachments
(3 files)
|
(deleted),
patch
|
Details | Diff | Splinter Review | |
|
(deleted),
text/x-phabricator-request
|
Details | |
|
(deleted),
text/x-phabricator-request
|
Details |
We're at the point where removing parseGlobal and mergeRealms should be feasible. There are likely rough edges and corner cases that need to be addressed, or unsurfaced bugs in the current deferred allocation code.. but the frontend architecture should mostly accept the changes.
This change must be applied on top of any atoms-deferral, so it's blocked by the parser atoms bug.
|
Reporter | |
Comment 1•4 years ago
|
||
Draft, "just go in and rip it out" patch. About 1% of jit-tests fail with this. This patch must be applied on top of the patches in bug 1592105.
|
||
Updated•4 years ago
|
|
||
Updated•4 years ago
|
|
|
||
Comment 2•4 years ago
|
||
For the MVP, we will leave this code in tree, but have a pref that lets us skip it and use stencil directly. See Bug 1662142.
|
|
||
Updated•4 years ago
|
|
|
||
Updated•3 years ago
|
|
Assignee | |
Updated•3 years ago
|
|
|
Assignee | |
Comment 3•3 years ago
|
||
Depends on D131372
|
|
Assignee | |
Comment 4•3 years ago
|
||
Depends on D131373
|
||
Comment 6•3 years ago
|
||
| bugherder | ||
https://hg.mozilla.org/mozilla-central/rev/bac63667ef07
https://hg.mozilla.org/mozilla-central/rev/1b7538bc798b
Description
•