Closed
Bug 1814928
Opened 2 years ago
Closed 2 years ago
Experiment with putting a one element cache in front of the whole cell buffer for JIT callers
Categories
(Core :: JavaScript: GC, task)
Core
JavaScript: GC
Tracking
()
RESOLVED
FIXED
111 Branch
Tracking | Status | |
---|---|---|
firefox111 | --- | fixed |
People
(Reporter: jonco, Assigned: jonco)
References
Details
Attachments
(2 files)
Calling into the VM for the post write barrier adds overhead. We can probably skip a lot of these calls by adding a single element cache which we can check from JIT code before calling into the VM.
Assignee | ||
Comment 1•2 years ago
|
||
This patch reduces the time taken by the workload in bug 1814796 by about 10%.
This only checks the cache when called from optimised code. I didn't add the
check for baseline or cache IR.
Assignee | ||
Comment 2•2 years ago
|
||
There's a bunch of condtional code in this function and it's clearer to split
out a separate function for globals.
Depends on D168825
Pushed by jcoppeard@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/4e6cd201a7c1
Add a single element cache in front of the whole cell buffer for use from JIT code r=jandem
Comment 4•2 years ago
|
||
bugherder |
Status: NEW → RESOLVED
Closed: 2 years ago
status-firefox111:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → 111 Branch
You need to log in
before you can comment on or make changes to this bug.
Description
•