Closed Bug 1027674 Opened 10 years ago Closed 10 years ago

OdinMonkey: small Ion FFI optimization

Categories

(Core :: JavaScript Engine: JIT, defect)

All
Linux
defect
Not set
normal

Tracking

()

RESOLVED FIXED
mozilla33

People

(Reporter: luke, Assigned: luke)

Details

Attachments

(1 file)

Attached patch optimize-ffiSplinter Review
I noticed we can just bake in the Runtime* using the existing patching machinery instead of using two loads to get it from the AsmJSActivation.
Attachment #8442858 - Flags: review?(dtc-moz)
Comment on attachment 8442858 [details] [diff] [review]
optimize-ffi

Review of attachment 8442858 [details] [diff] [review]:
-----------------------------------------------------------------

Nice. Saves three instructions on the ARM.

::: js/src/jit/AsmJS.cpp
@@ +6708,4 @@
>          // The following is inlined:
> +        //   Activation *act = rt->mainThread.activation()->active_ = false;
> +        //   rt->mainThread.jitTop = prevJitTop_;
> +        //   rt->mainThread.jitJSContext = prevJitJSContext_;

This does not read well. Perhaps:
        //   Activation *act = rt->mainThread.activation();
        //   act->active_ = false;
        //   rt->mainThread.jitTop = act->prevJitTop_;
        //   rt->mainThread.jitJSContext = atc->prevJitJSContext_;
Attachment #8442858 - Flags: review?(dtc-moz) → review+
https://hg.mozilla.org/mozilla-central/rev/a43d41c1645a
Status: ASSIGNED → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla33
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: