1665387 - Support merging delazification CompilationStencil into top-level CompilationStencil

Status: RESOLVED WONTFIX

(Core :: JavaScript Engine, task, P1)

Description

[Tooru Fujisawa [:arai]]

incremental encoding + decoding will be the following steps:

1. encode stencil for top-level
2. encode stencils for delazification
3. linearize 1+2 just by concatenation, with number of delazifications
4. decode stencil for top-level
5. decode stencils for delazification
6. merge stencils for delazification into top-level
7. instantiate the merged top-level stencil

Step 6 requires merging function CompilationStencil into top-level script CompilationStencil.

Comment 1

[Ted Campbell [:tcampbell]]

The merge process probably follows this part of the instantiate code: https://searchfox.org/mozilla-central/rev/30e70f2fe80c97bfbfcd975e68538cefd7f58b2a/js/src/frontend/Stencil.cpp#576-582,588-590,594

That includes:

• Replacing the ScriptStencil::gcThings list while updating the indices of functions, scopes, etc
• Attach ImmutableScriptData
• For inner-functions of the delazified function:
  • Updating ScriptStencil::isSingletonFunction flag
  • Updating ScriptStencil::functionAtom
  • Updating HAS_INFERRED_NAME, HAS_GUESSED_ATOM function flags
  • Updating ScriptStencil::lazyFunctionEnclosingScopeIndex_
  • Updating ScriptStencil::scriptStencil.memberInitializers
  • Updating TreatAsRunOnce immutableFlag

Comment 2

[Tooru Fujisawa [:arai]]

Attachment: Bug 1665387 - Support merging delazification CompilationStencil into top-level CompilationStencil. r?tcampbell!

Depends on D90573

Comment 3

[Tooru Fujisawa [:arai]]

latest patch in bug 1662274 doesn't requrie merging