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

Status: RESOLVED WORKSFORME

(Core :: JavaScript Engine, defect)

Description

[Boris Zbarsky [:bzbarsky]]

Attachment: 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)

Comment 1

[Ryan VanderMeulen [:RyanVM]]

Current js shell results for attached testcase modified to loop 1000000 times:
Interp: 241
JM: 95
JM+TI: 89
d8: 51

Comment 2

[Till Schneidereit [:till]]

Current js shell results for attached testcase modified to loop 10000000 (note the additional 0) times:
Interp: 3639
Baseline: 1644
IM: 677
d8: 239

Comment 3

[André Bargull [:anba]]

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