Cleanup Stencil data structures
Categories
(Core :: JavaScript Engine, task, P1)
Tracking
()
| Tracking | Status | |
|---|---|---|
| firefox77 | --- | fixed |
People
(Reporter: tcampbell, Assigned: tcampbell)
References
Details
Attachments
(4 files)
Here is a series of patches to bring the Stencil data structures closer to their final form. This includes removing redundant fields and organizing more similar to BaseScript.
| Assignee | ||
Comment 1•6 years ago
|
||
Move the argument to the init methods of the sub-classes instead. This lets
us initialize the fields of the stencil in one place.
| Assignee | ||
Comment 2•6 years ago
|
||
Organize fields to better reflect the BaseScript layout. Also remove the
ngthings field which is redundant. Add override keyword for virtual methods
for safety.
Depends on D71472
| Assignee | ||
Comment 3•6 years ago
|
||
The isSelfHosting, kind, and allocKind fields should be directly encoded into
the FunctionFlags.
Depends on D71473
| Assignee | ||
Comment 4•6 years ago
|
||
Instead, encode them as ImmutableScriptFlags. The sets up to later combine
the FunctionCreationData with the ScriptStencil.
Depends on D71474
Comment 6•6 years ago
|
||
| bugherder | ||
https://hg.mozilla.org/mozilla-central/rev/6f0c8cf76eff
https://hg.mozilla.org/mozilla-central/rev/000589fb60ee
https://hg.mozilla.org/mozilla-central/rev/c19316d62320
https://hg.mozilla.org/mozilla-central/rev/9ac3134f48af
Description
•