Created attachment 527913 [details]
See attached shell testcase. I see times around 61ms in TM and 73ms in TI+JM.
Are you on 64-bit? I see these numbers (JM tip):
-m -n: 59 ms
-j: 71 ms
-m: 97 ms
-m -n: 99 ms
-j: 82 ms
-m: 154 ms
The problem is that we only inline jsop_mod on 32-bit, on 64-bit we always call a stub.. I will file a bug, we should try to fix this in TM for Firefox 6.
Filed bug 652321.
With the patch in bug 652321 applied to JM:
-m -n: 64 ms
-j: 81 ms
-m: 101 ms
> Are you on 64-bit?
Yes (note that the bug is filed on Mac).
Before landing, we need to make sure that this doesn't regress Mac when TI is pref'd off. If that checks out, this can be unhooked from LandTI.
This regressed terribly in the last month. From 97 ms (comment 1) to 408 ms.
parseInt returns doubles and this disables the inline paths for JSOP_MOD/JSOP_GETELEM.
With the patch in bug 662766 there's a large perf win here compared to the TM-branch, even without -n. I'll close this bug after landing the patch.
Some 64-bit numbers:
js -m -n : 64 ms
d8 : 77 ms
js -j : 82 ms
js -m -j -p : 82 ms
js -m : 111 ms
js -j : 144 ms
js -m -j -p : 144 ms
js -m : 355 ms
32-bit is a bit faster (59 ms with -m -n). Not much more to do here.