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)

task

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.

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.

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
Status: NEW → RESOLVED
Closed: 2 years ago
Resolution: --- → FIXED
Target Milestone: --- → 111 Branch
See Also: → 1850814
See Also: → 1888768
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: