Bug 1687094 Comment 2 Edit History

Note: The actual edited comment in the bug view page will always show the original commenter’s name and original timestamp.

Original comment by
on 
> b) keep `ParserAtomsTable` instance in `XDRIncrementalStencilEncoder`, and use it for all delazifications

This has the lifetime problem.
If we move `ParserAtomsTable` out of `CompilationStencil` and store it into encoder, we cannot keep the `CompilationStencil` instance on memory.
Currently `CompilationStencil` instance is discarded immediately after instantiation, the goal design is to allow the stencil live longer.

will look into (a), with introducing (or reviving?) atom map for stencil.
Revision 1 by
on 
> b) keep `ParserAtomsTable` instance in `XDRIncrementalStencilEncoder`, and use it for all delazifications

This has the lifetime problem.
If we move `ParserAtomsTable` out of `CompilationStencil` and store it into encoder, we cannot keep the `CompilationStencil` instance on memory.
Currently `CompilationStencil` instance is discarded immediately after instantiation, but the goal design is to allow the stencil live longer.

will look into (a), with introducing (or reviving?) atom map for stencil.

Back to Bug 1687094 Comment 2