Simplify JS class-field-initializer parsing
Categories
(Core :: JavaScript Engine, task, P2)
Tracking
()
Tracking | Status | |
---|---|---|
firefox87 | --- | fixed |
People
(Reporter: tcampbell, Assigned: tcampbell)
References
(Blocks 1 open bug)
Details
Attachments
(2 files)
Class constructor scripts currently track the member-initializer data of the class body and save it for bytecode generation. This is currently computed when compiling the enclosing script, while we can now simply do it during initial parse.
This will be much simpler to reason about and allows moving the data to script-extra stencil section to save more memory. As well, this opens the path to allowing syntax parsing of class fields.
Assignee | ||
Comment 1•4 years ago
|
||
Assignee | ||
Comment 2•4 years ago
|
||
Depends on D102822
Updated•4 years ago
|
Updated•4 years ago
|
Assignee | ||
Comment 3•4 years ago
|
||
Unsurprisingly, supporting lazy parsing as well is way more complex and out of scope.
Assignee | ||
Updated•4 years ago
|
Comment 5•4 years ago
|
||
Backed out as per request (for potentially causing perf issues)
backout: https://hg.mozilla.org/integration/autoland/rev/0a991f0d6563d41838b983b3285bedbdfb8bcc89
push: https://treeherder.mozilla.org/jobs?repo=autoland&revision=991388ec272dd61462630b76f4ec86ff4de30721
Comment 7•4 years ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/b2385073866d
https://hg.mozilla.org/mozilla-central/rev/c9a4a9f6e6cf
Assignee | ||
Updated•4 years ago
|
Description
•