Closed
Bug 1820277
Opened 1 year ago
Closed 1 year ago
Add --only-inline-selfhosted
Categories
(Core :: JavaScript Engine: JIT, enhancement, P1)
Core
JavaScript Engine: JIT
Tracking
()
RESOLVED
FIXED
112 Branch
Tracking | Status | |
---|---|---|
firefox112 | --- | fixed |
People
(Reporter: iain, Assigned: iain)
References
Details
(Whiteboard: [sp3])
Attachments
(1 file)
Jeff and Markus want to be able to disable inlining of non-self-hosted code to make it easier to compare V8 vs SM performance.
Assignee | ||
Comment 1•1 year ago
|
||
Jeff and Markus requested this to make it easier to compare profiles between V8 and SM.
Updated•1 year ago
|
Whiteboard: [sp3]
Updated•1 year ago
|
See Also: → https://mozilla-hub.atlassian.net/browse/SP3-35
Comment 2•1 year ago
|
||
This has been quite helpful when comparing SM to V8. On react-todomvc the performance difference between --only-inline-self-hosted on vs off is pretty negligible which suggests that most of inline wins are from inlinning self hosted code.
Assignee | ||
Comment 3•1 year ago
|
||
Doug has a WIP patch on bug 1819722 that improves our Speedometer score by (roughly speaking) using a simpler inlining strategy in cases where our current complexity doesn't buy us anything. It would be interesting to see if Jeff's observation in comment 2 is still true with that patch applied.
Pushed by iireland@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/ef2f20ce02d3 Add onlyInlineSelfHosted jitoption r=jandem
Comment 5•1 year ago
|
||
bugherder |
Status: NEW → RESOLVED
Closed: 1 year ago
status-firefox112:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → 112 Branch
You need to log in
before you can comment on or make changes to this bug.
Description
•