darkroom demo got way slower than old days

NEW
Unassigned

Status

()

Core
JavaScript Engine
5 years ago
3 years ago

People

(Reporter: vlad, Unassigned)

Tracking

Trunk
x86_64
Windows 7
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

(Whiteboard: [js:t], URL)

We used to be a faster than Chrome at this at one point:

http://conduit.bitops.com/~vladimir/misc/darkroom/darkroom.html

Now, not so much -- we get around 16fps on Windows, and Chrome gets 90+ fps (basically instant manipulation).  I want to say that the last time it was fast was back in the tracemonkey days before the other monkeys hopped on board, but I'm not sure.
(In reply to Vladimir Vukicevic [:vlad] [:vladv] from comment #0)
> I want to say that the last time it was fast was back in the 
> tracemonkey days before the other monkeys hopped on board, 
> but I'm not sure.

I believe that's correct. This should be really good code for Ion to optimize for (once it's landed). Ion is already about 20% faster than trunk, but it should be possible to do as well as Chrome.
Whiteboard: [js:t]
Just tried this with a build from the IonMonkey branch; no real change.  I'm pretty sure it's running in Ion jit mode -- on a non-ion build, if I disable methodjit.content perf plummets to < 1fps, but stays rouhgly the same on the IonMonkey branch. (Granted, that pref might not do exactly the same thing on IM branch, but still.)
(In reply to Vladimir Vukicevic [:vlad] [:vladv] from comment #2)
> Just tried this with a build from the IonMonkey branch; no real change.  I'm
> pretty sure it's running in Ion jit mode -- on a non-ion build, if I disable
> methodjit.content perf plummets to < 1fps, but stays rouhgly the same on the
> IonMonkey branch. (Granted, that pref might not do exactly the same thing on
> IM branch, but still.)

20% is no 4x, so that sounds about right. It probably just needs some kind of tuning or a new optimization or two. Right now they're focused on landing, but after that they will be able to dig in to bugs like this one.
(Assignee)

Updated

3 years ago
Assignee: general → nobody
You need to log in before you can comment on or make changes to this bug.