1690863 - Remove gcOutput argument from ScopeStencil instantiation

Status: RESOLVED FIXED

(Core :: JavaScript Engine, task, P2)

Description

[Ted Campbell [:tcampbell]]

Refactor ScopeStencil::createScope to have a variant that does not use gcOutput as an argument. This allows constructing scopes at a finer granularity than the entire CompilationStencil.

Comment 1

[Ted Campbell [:tcampbell]]

Attachment: Bug 1690863 - Move Scope::canonical{Function,Module} initialization out of Scope. r?arai!

Move the Scope -> Function/Module initialization to the same place as the
Function/Module -> BaseScript initialization. This simplifies initial scope
creation.

Comment 2

[Ted Campbell [:tcampbell]]

Attachment: Bug 1690863 - Remove gcOutput argument from ScopeStencil::createSpecificScope. r?arai!

Instead, pass the enclosingScope argument explicitly. This makes it easier in
future to create a single scope independantly from the entire stencil which
gives more flexibility.

Depends on D104089

Comment 3

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/8ca896ce33af
Move Scope::canonical{Function,Module} initialization out of Scope. r=arai
https://hg.mozilla.org/integration/autoland/rev/244745c81b8f
Remove gcOutput argument from ScopeStencil::createSpecificScope. r=arai

Comment 4

[Bogdan Tara[:bogdan_tara | bogdant]]

https://hg.mozilla.org/mozilla-central/rev/8ca896ce33af
https://hg.mozilla.org/mozilla-central/rev/244745c81b8f