1641419 - Defer JSFunction/BaseScript allocation in BinAST

Status: RESOLVED FIXED

(Core :: JavaScript Engine, task, P2)

Description

[Ted Campbell [:tcampbell]]

Now that normal scripts are deferred, it is a much smaller task to also support BinAST while it's fate is decided.

Comment 1

[Ted Campbell [:tcampbell]]

Attachment: Bug 1641419 - Fix BinAST lazy BaseScript SourceExtent handling. r?mgaudet

Use consistent SourceExtent values with non-lazy BinAST scripts. Change the
accessors on js::BaseScript to be consistent with SharedContext. This will be
merged in the next patch. This patch also fixes a typo in index computation.

See BinASTParserPerTokenizer::makeEmptyFunctionNode(..) for how non-lazy
BinAST scripts initialize this. Note that toStringStart is always 0.

These changes are largely superficial because nothing tests for these values
and BinAST is not enabled by default. Consistency is still helpful though.

Comment 2

[Ted Campbell [:tcampbell]]

Attachment: Bug 1641419 - Defer BaseScript/JSFunction allocation in BinAST. r?mgaudet

Store appropriate data on the FunctionBox and let the machinery for
non-BinAST scripts do the Stencil instantiation for us.

Depends on D77188

Comment 3

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/06d56b6fdbbf
Fix BinAST lazy BaseScript SourceExtent handling. r=mgaudet
https://hg.mozilla.org/integration/autoland/rev/4525a2e75012
Defer BaseScript/JSFunction allocation in BinAST. r=mgaudet

Comment 4

[Raul Gurzau (:RaulG)]

https://hg.mozilla.org/mozilla-central/rev/06d56b6fdbbf
https://hg.mozilla.org/mozilla-central/rev/4525a2e75012