Closed Bug 1633464 Opened 4 years ago Closed 4 years ago

Use a tail-call to enter the Ion invalidation trampoline

Categories

(Core :: JavaScript Engine: JIT, task, P3)

task

Tracking

()

RESOLVED FIXED
mozilla77
Tracking Status
firefox77 --- fixed

People

(Reporter: tcampbell, Assigned: tcampbell)

Details

Attachments

(1 file)

https://searchfox.org/mozilla-central/rev/55a4faa52f72918efa51150d127aebdc057dc6cf/js/src/jit/x64/Trampoline-x64.cpp#363

We currently generate a call and then immediately strip the return address off the stack. Instead we should just use a jump to simplify. Architectures with link-registers should also use a jmp, but they don't have the same stack traffic.

On x86/x64 platforms, we were using a call and then immediately popping and
ignoring the return value, so use a jump instead. For architectures that use
a link-register, we no longer clobber it but it was already ignored.

Pushed by tcampbell@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/2407418c8476
Use a jump to enter Ion invalidation trampoline. r=jandem
Status: NEW → RESOLVED
Closed: 4 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla77
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: