Centralize LifoAlloc for initial stencil and delazifications in CompilationInfoVector
Categories
(Core :: JavaScript Engine, task, P1)
Tracking
()
| Tracking | Status | |
|---|---|---|
| firefox85 | --- | fixed |
People
(Reporter: arai, Assigned: arai)
References
Details
Attachments
(5 files)
there can be thousands of delazifications stored into CompilationInfoVector, and
allocation/free cost of LifoAlloc inside CompilationStencil in delazifications isn't negligible.
moving LifoAlloc to CompilationInfo and reusing initial stencil's LifoAlloc from delazification improves the performance.
|
Assignee | |
Comment 1•4 years ago
|
||
These methods will be modified to take CompilationInput and CompilationStencil
separately in later patch.
Depends on D97114
|
|
Assignee | |
Comment 2•4 years ago
|
||
Depends on D97115
|
|
Assignee | |
Comment 3•4 years ago
|
||
Depends on D97116
|
|
Assignee | |
Comment 4•4 years ago
|
||
Depends on D97117
|
|
Assignee | |
Comment 5•4 years ago
|
||
Depends on D97118
Pushed by arai_a@mac.com: https://hg.mozilla.org/integration/autoland/rev/aa2f2b8408ee Part 1: Make CompilationInfo methods static. r=tcampbell https://hg.mozilla.org/integration/autoland/rev/3945960193ea Part 2: Take CompilationInput and CompilationStencil separately. r=tcampbell https://hg.mozilla.org/integration/autoland/rev/1d97945897ad Part 3: Use CompilationStencil as delazifications vector element. r=tcampbell https://hg.mozilla.org/integration/autoland/rev/63179c48aedc Part 4: Add cache for CompilationAtomCache for delazifications. r=tcampbell https://hg.mozilla.org/integration/autoland/rev/322ee18b6f51 Part 5: Move LifoAlloc to CompilationInfo and CompilationInfoVector, to share between all delazification. r=tcampbell
|
||
Comment 7•3 years ago
|
||
| bugherder | ||
https://hg.mozilla.org/mozilla-central/rev/aa2f2b8408ee
https://hg.mozilla.org/mozilla-central/rev/3945960193ea
https://hg.mozilla.org/mozilla-central/rev/1d97945897ad
https://hg.mozilla.org/mozilla-central/rev/63179c48aedc
https://hg.mozilla.org/mozilla-central/rev/322ee18b6f51
Description
•