Closed
Bug 1658720
Opened 4 years ago
Closed 4 years ago
Combine ScriptStencil topLevel/funcData structures
Categories
(Core :: JavaScript Engine, task, P2)
Core
JavaScript Engine
Tracking
()
RESOLVED
FIXED
82 Branch
| Tracking | Status | |
|---|---|---|
| firefox82 | --- | fixed |
People
(Reporter: tcampbell, Assigned: tcampbell)
References
Details
Attachments
(1 file)
Now that we've unified the stencil data for BaseScript / JSFunction and are preparing to factor out the GC results, it may be helpful to combine the topLevel and funcData fields. This would resolve some confusion when compiling standalone functions that have two stencils but only one is used.
|
Assignee | |
Comment 1•4 years ago
|
||
This should also let us remove isStandaloneFunction
|
|
Assignee | |
Comment 2•4 years ago
|
||
Instead, rename funcData to scriptData and reserve index 0 for top-level.
|
||
Updated•4 years ago
|
Attachment #9170023 -
Attachment description: Bug 1658720 - Remove CompilationInfo::topLevel → Bug 1658720 - Remove CompilationInfo::topLevel. r?arai
Pushed by arai_a@mac.com: https://hg.mozilla.org/integration/autoland/rev/d612e1883efc Remove CompilationInfo::topLevel. r=arai
|
||
Comment 4•4 years ago
|
||
| bugherder | ||
Status: NEW → RESOLVED
Closed: 4 years ago
status-firefox82:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → 82 Branch
You need to log in
before you can comment on or make changes to this bug.
Description
•