1655768 - Remove parserGlobal and mergeRealms

Status: RESOLVED FIXED

(Core :: JavaScript Engine, task, P2)

Description

[Kannan Vijayan [:djvj]]

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.

Comment 1

[Kannan Vijayan [:djvj]]

Attachment: remove-parseglobal-and-mergerealms.patch

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.

Comment 2

[Ted Campbell [:tcampbell]]

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.

Comment 3

[Tooru Fujisawa [:arai]]

Attachment: Bug 1655768 - Part 1: Remove RealmCreationOptions::setMergeable. r?nbp!

Depends on D131372

Comment 4

[Tooru Fujisawa [:arai]]

Attachment: Bug 1655768 - Part 2: Remove MergeRealms. r?jonco!

Depends on D131373

Comment 5

[Pulsebot]

Pushed by [e-mail]:
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

Comment 6

[Alexandru Michis [:malexandru]]

https://hg.mozilla.org/mozilla-central/rev/bac63667ef07
https://hg.mozilla.org/mozilla-central/rev/1b7538bc798b