Open
Bug 856128
Opened 12 years ago
Updated 2 years ago
We're slow in the Chipmunk js logo smash demo
Categories
(Core :: JavaScript Engine, defect)
Tracking
()
NEW
People
(Reporter: ehsan.akhgari, Unassigned)
References
Details
Attachments
(1 file)
|
(deleted),
application/x-javascript
|
Details |
See the demo here: <http://dl.dropbox.com/u/2494815/demo/index.html>. V8 does a lot better!
|
Reporter | |
Updated•12 years ago
|
Summary: We're slow in the Chupmunk js logo smash demo → We're slow in the Chipmunk js logo smash demo
|
||
Comment 1•12 years ago
|
||
Shouldn't this additionally/rather block Bug 705294 and/or BC (if it applies) as JM is going away sooner than later?
|
||
Comment 2•11 years ago
|
||
Shell testcase; renders 500 frames and measures how long it takes. Instruments profile looks similar to what I see in the browser so hopefully fixing this will also help the real website.
d8 : 5600 ms
js : 15564 ms
There's some low-hanging fruit like Ion not compiling DELELEM, will start with that.
Assignee: general → jdemooij
Status: NEW → ASSIGNED
|
|
||
Comment 3•11 years ago
|
||
The main performance problem here is |delete| on an object with many properties inside a for-in loop. Filed bug 912079.
|
|
||
Comment 4•6 years ago
|
||
I just posted some patches for bug 912079. With that the shell testcase runs in 3.6 seconds or so, still a bit slower than d8 (~2.8 seconds) but way better than it was before.
Assignee: jdemooij → nobody
Status: ASSIGNED → NEW
|
||
Updated•2 years ago
|
Severity: normal → S3
You need to log in
before you can comment on or make changes to this bug.
Description
•