Closed Bug 532775 Opened 12 years ago Closed 4 years ago

IM: Setting array length 2.8x as slow as v8 (dromaeo [] test)

Categories

(Core :: JavaScript Engine, defect)

defect
Not set
normal

Tracking

()

RESOLVED WORKSFORME

People

(Reporter: bzbarsky, Assigned: Waldo)

References

Details

Attachments

(1 file)

73 bytes, application/x-javascript
Details
Attached file Testcase
See attached testcase.  TMFLAGS=abort,stats shows:

trace stopped: 11147: non-native JSObjectOps::setProperty
Abort recording of tree /Users/bzbarsky/test.js:2@34 at /Users/bzbarsky/test.js:3@39: setprop.
recorder: started(1), aborted(1), completed(0), different header(0), trees trashed(0), slot promoted(0), unstable loop variable(0), breaks(0), returns(0), merged loop exits(0), unstableInnerCalls(0), blacklisted(0)
monitor: exits(0), timeouts(0), type mismatch(0), triggered(0), global mismatch(0), flushed(0)
Summary: Setting array length doesn't trace → Setting array length doesn't trace (dromaeo [] test)
Assignee: general → jwalden+bmo
Attachment #415964 - Attachment is patch: false
Attachment #415964 - Attachment mime type: text/plain → application/x-javascript
Current js shell results for attached testcase modified to loop 1000000 times:
Interp: 241
JM: 95
JM+TI: 89
d8: 51
Summary: Setting array length doesn't trace (dromaeo [] test) → JM+TI: Setting array length 2x slower than v8 (dromaeo [] test)
Current js shell results for attached testcase modified to loop 10000000 (note the additional 0) times:
Interp: 3639
Baseline: 1644
IM: 677
d8: 239
OS: Mac OS X → All
Hardware: x86 → All
Summary: JM+TI: Setting array length 2x slower than v8 (dromaeo [] test) → IM: Setting array length 2.8x as slow as v8 (dromaeo [] test)
Performance seems to be acceptable now, therefore resolving as WFM.


andre@VBdev:~$ time mozjs --no-baseline /tmp/t.js
real    0m4.217s
user    0m4.204s
sys     0m0.012s

andre@VBdev:~$ time mozjs --no-ion /tmp/t.js                                                          
real    0m0.677s
user    0m0.660s
sys     0m0.016s

andre@VBdev:~$ time mozjs /tmp/t.js                                                                   
real    0m0.363s
user    0m0.352s
sys     0m0.012s

andre@VBdev:~$ time v8 /tmp/t.js 
real    0m1.093s
user    0m1.068s
sys     0m0.020s
Status: NEW → RESOLVED
Closed: 4 years ago
Resolution: --- → WORKSFORME
You need to log in before you can comment on or make changes to this bug.