Closed
Bug 1692624
Opened 3 years ago
Closed 3 years ago
Remove frontend::SharedContext::stencil_ field
Categories
(Core :: JavaScript Engine, task, P3)
Core
JavaScript Engine
Tracking
()
RESOLVED
FIXED
87 Branch
| Tracking | Status | |
|---|---|---|
| firefox87 | --- | fixed |
People
(Reporter: tcampbell, Assigned: tcampbell)
References
(Blocks 1 open bug)
Details
Attachments
(2 files)
This field is almost unused now with most data being on CompilationState. If we change asm.js handling, we can remove from the SharedContext entirely.
|
Assignee | |
Comment 1•3 years ago
|
||
The asm.js modules are the last set of data directly tracked on the stencil
and not on the parser state. By migrating this, we can later remove a bit of
plumbing that copies the stencil refernce around.
|
|
Assignee | |
Comment 2•3 years ago
|
||
This has no significant uses left, so just remove it.
Depends on D105090
Pushed by tcampbell@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/a20ff9a6cd4d Track asmJS modules on frontend::CompilationState. r=arai https://hg.mozilla.org/integration/autoland/rev/17402bf45cac Remove SharedContext::stencil_ field. r=arai
|
||
Comment 4•3 years ago
|
||
| bugherder | ||
https://hg.mozilla.org/mozilla-central/rev/a20ff9a6cd4d
https://hg.mozilla.org/mozilla-central/rev/17402bf45cac
Status: NEW → RESOLVED
Closed: 3 years ago
status-firefox87:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → 87 Branch
You need to log in
before you can comment on or make changes to this bug.
Description
•