Closed
Bug 1692624
Opened 4 years ago
Closed 4 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•4 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•4 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•4 years ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/a20ff9a6cd4d
https://hg.mozilla.org/mozilla-central/rev/17402bf45cac
Status: NEW → RESOLVED
Closed: 4 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
•