Closed Bug 1744178 Opened 9 months ago Closed 8 months ago

Add public type for CompilationGCOuptut

Categories

(Core :: JavaScript Engine, task, P3)

task

Tracking

()

RESOLVED FIXED
97 Branch
Tracking Status
firefox97 --- fixed

People

(Reporter: arai, Assigned: arai)

References

Details

Attachments

(1 file)

In bug 1734098, I'm going to replace off-thread API purely on stencil, and let the consumer instantiate it on main thread.
In order to do that, we need to pass the CompilationGCOutput prepared in off-thread to the consumer, and let the consumer pass it back to instantiate API

Also, we could move the off-thread compilation/decode body to the embedding side, to utilize embedding's threading,
it will also require public type for pre-allocated/prepared CompilationGCOuptut that can be passed between threads.

In order to support allocating CompilationGCOutput off main thread with
stencil-based off-thread API, add public type that wraps CompilationGCOutput,
and make stencil-based off-thread API return pre-allocated CompilationGCOutput
if the options.allocateInstantiationStorage is true.

Pushed by arai_a@mac.com:
https://hg.mozilla.org/integration/autoland/rev/6ca5bceb693c
Add JS::InstantiationStorage. r=nbp

Backed out (bug 1744178, bug 1734098) for causing leaks.

Affected platform Linux 18.04 x64 WebRender asan opt

Push with failures

Failure log for wd leaks
Failure log for bc leaks
Other jobs have the same failure log as bc leaks.

Backout link

Also this V-swr bustages, failure log: https://treeherder.mozilla.org/logviewer?job_id=360715600&repo=autoland&lineNumber=71123
and
this py3 mch failures, failure log: https://treeherder.mozilla.org/logviewer?job_id=360720852&repo=autoland&lineNumber=181

Flags: needinfo?(arai.unmht)
Pushed by arai_a@mac.com:
https://hg.mozilla.org/integration/autoland/rev/5e5fbe48db65
Add JS::InstantiationStorage. r=nbp
Flags: needinfo?(arai.unmht)
Status: NEW → RESOLVED
Closed: 8 months ago
Resolution: --- → FIXED
Target Milestone: --- → 97 Branch
You need to log in before you can comment on or make changes to this bug.