Closed Bug 1820277 Opened 1 year ago Closed 1 year ago

Add --only-inline-selfhosted

Categories

(Core :: JavaScript Engine: JIT, enhancement, P1)

enhancement

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.

Jeff and Markus requested this to make it easier to compare profiles between V8 and SM.

Whiteboard: [sp3]

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.

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
Status: NEW → RESOLVED
Closed: 1 year ago
Resolution: --- → FIXED
Target Milestone: --- → 112 Branch
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: