Closed Bug 1655768 Opened 1 year ago Closed 8 days ago

Remove parserGlobal and mergeRealms

Categories

(Core :: JavaScript Engine, task, P2)

task

Tracking

()

RESOLVED FIXED
96 Branch
Tracking Status
firefox96 --- fixed

People

(Reporter: djvj, Assigned: arai)

References

(Blocks 1 open bug)

Details

Attachments

(3 files)

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.

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.

Assignee: nobody → kvijayan
Severity: -- → N/A
Priority: -- → P2
Blocks: stencil-mvp
No longer blocks: 1544117

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.

Blocks: stencil-backlog
No longer blocks: stencil-mvp
Blocks: 1687973
No longer blocks: 1687973
Assignee: kvijayan → nobody
Assignee: nobody → arai.unmht
See Also: → 1687973

Depends on D131373

Blocks: 1611437
Pushed by arai_a@mac.com:
https://hg.mozilla.org/integration/autoland/rev/bac63667ef07
Part 1: Remove RealmCreationOptions::setMergeable. r=nbp
https://hg.mozilla.org/integration/autoland/rev/1b7538bc798b
Part 2: Remove MergeRealms. r=jonco
Status: NEW → RESOLVED
Closed: 8 days ago
Resolution: --- → FIXED
Target Milestone: --- → 96 Branch
You need to log in before you can comment on or make changes to this bug.