Closed Bug 1631272 Opened 6 years ago Closed 6 years ago

Cleanup Stencil data structures

Categories

(Core :: JavaScript Engine, task, P1)

task

Tracking

()

RESOLVED FIXED
mozilla77
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.

No longer depends on: 1631258

Move the argument to the init methods of the sub-classes instead. This lets
us initialize the fields of the stencil in one place.

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

The isSelfHosting, kind, and allocKind fields should be directly encoded into
the FunctionFlags.

Depends on D71473

Instead, encode them as ImmutableScriptFlags. The sets up to later combine
the FunctionCreationData with the ScriptStencil.

Depends on D71474

Pushed by tcampbell@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/6f0c8cf76eff Remove immutableScriptData argument of ScriptStencil constructor. r=mgaudet https://hg.mozilla.org/integration/autoland/rev/000589fb60ee Organize the ScriptStencil fields. r=mgaudet https://hg.mozilla.org/integration/autoland/rev/c19316d62320 Remove redundant FunctionCreationData fields. r=mgaudet https://hg.mozilla.org/integration/autoland/rev/9ac3134f48af Remove FunctionCreationData::generatorKind/asyncKind. r=mgaudet
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: