We're slow in the Chipmunk js logo smash demo

NEW
Unassigned

Status

()

defect
6 years ago
a year ago

People

(Reporter: Ehsan, Unassigned)

Tracking

Trunk
x86
macOS
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

141.53 KB, application/x-javascript
Details
(Reporter)

Description

6 years ago
See the demo here: <http://dl.dropbox.com/u/2494815/demo/index.html>.  V8 does a lot better!
(Reporter)

Updated

6 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

6 years ago
Shouldn't this additionally/rather block Bug 705294 and/or BC (if it applies) as JM is going away sooner than later?
Posted file Shell testcase
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
Depends on: 911989
Depends on: 912079
The main performance problem here is |delete| on an object with many properties inside a for-in loop. Filed bug 912079.
Depends on: 912958
Depends on: 913424
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
You need to log in before you can comment on or make changes to this bug.