1773335 - Remove uses of `cx->tempLifoAlloc()` from the Parser and HelperThreads

Status: RESOLVED FIXED

(Core :: JavaScript Engine, task, P3)

Description

[Nicolas B. Pierron [:nbp]]

Currently the parser makes use of cx->tempLifoAlloc() to allocate its AST and all temporary structures which are not part of its output. Using a LifoAlloc is fine, but relying strictly on the JSContext is a dependency that we want to remove.

We should remplace the cx->tempLifoAlloc() within the Parser by an argument which can either be coming from the JSContext, in case of main-thread parsing, or from a newly initialized LifoAlloc, in the case of the HelperThread.

See cx->tempLifoAlloc() references.

Comment 1

[Bryan Thrall [:bthrall]]

Attachment: Bug 1773335 - Pass LifoAlloc to CompileGlobalScriptToStencil() as parameter r?arai

Comment 2

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/b004f2382dc6
Pass LifoAlloc to CompileGlobalScriptToStencil() as parameter r=arai

Comment 3

[Norisz Fay [:noriszfay]]

https://hg.mozilla.org/mozilla-central/rev/b004f2382dc6