Deduplicate XDR atoms
Categories
(Core :: JavaScript Engine, task, P1)
Tracking
()
Tracking | Status | |
---|---|---|
firefox72 | --- | fixed |
People
(Reporter: iain, Assigned: iain)
References
Details
Attachments
(5 files)
We currently write out atoms once per script that uses them, which results in atoms being encoded multiple times per top-level script. We can deduplicate them across top-level scripts to save space / calls to AtomizeChars when decoding.
Our first attempt at landing this code in bug 1575370 had to be backed out, but I think we have the GC issues sorted out now.
Assignee | ||
Comment 1•5 years ago
|
||
Right now there BinAST is the only case, but subsequent patches will add GC pointers to XDR encoders and decoders.
Assignee | ||
Comment 2•5 years ago
|
||
The code that uses these fields is part of a subsequent patch. This patch is just getting the tracing right.
Depends on D48781
Assignee | ||
Comment 3•5 years ago
|
||
Depends on D48782
Assignee | ||
Comment 4•5 years ago
|
||
Depends on D48783
Assignee | ||
Comment 5•5 years ago
|
||
Depends on D48784
Comment 8•5 years ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/9d297847a452
https://hg.mozilla.org/mozilla-central/rev/8834ca679ef7
https://hg.mozilla.org/mozilla-central/rev/145c009565f9
https://hg.mozilla.org/mozilla-central/rev/46a8774eed22
https://hg.mozilla.org/mozilla-central/rev/cf26123bd822
Description
•