Closed Bug 596076 Opened 9 years ago Closed 9 years ago

Enable Jaegermonkey method JIT for Fennec chrome

Categories

(Firefox for Android Graveyard :: General, defect)

defect
Not set

Tracking

(Not tracked)

RESOLVED WONTFIX

People

(Reporter: mbrubeck, Assigned: mbrubeck)

References

Details

Attachments

(1 file)

Attached patch patchSplinter Review
This patch sets javascript.options.methodjit.chrome to "true" for Fennec.

And it gets rid of the javascript.options.jit.* prefs which aren't used anymore.
Attachment #474901 - Flags: review?(mark.finkle)
Comment on attachment 474901 [details] [diff] [review]
patch

>diff -r cc98b92ce4e0 app/mobile.js

>-pref("javascript.options.jit.content", true);
>-pref("javascript.options.jit.chrome", true);

Change these to "tracejit". For example: javascript.options.tracejit.content"

>+pref("javascript.options.methodjit.chrome",  true);
> pref("javascript.options.mem.gc_frequency", 300);

r+ with the "jit" -> "tracejit"
Attachment #474901 - Flags: review?(mark.finkle) → review+
Pushed without the tracejit prefs, per IRC discussion. (They are already "true" in all.js.)
http://hg.mozilla.org/mobile-browser/rev/20c846ba2905
Status: ASSIGNED → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
Backed out because it seems to be crashing on ARM:
http://hg.mozilla.org/mobile-browser/rev/2f2287cd99b7
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
Oops, this was pushed again on 9/15 but I forgot to update the bug:
http://hg.mozilla.org/mobile-browser/pushloghtml?changeset=6c686e29333c

This might need to be backed out again to fix bug 597093.
Blocks: 597093
Status: REOPENED → RESOLVED
Closed: 9 years ago9 years ago
Resolution: --- → FIXED
Backed out again because of bug 597093:
http://hg.mozilla.org/mobile-browser/rev/cba192dabb64

We will not enable this again until the JS team is ready to turn it on by default.
Resolution: FIXED → WONTFIX
You need to log in before you can comment on or make changes to this bug.