Support Generators in IonMonkey
Categories
(Core :: JavaScript Engine: JIT, enhancement, P3)
Tracking
()
Tracking | Status | |
---|---|---|
firefox53 | --- | affected |
People
(Reporter: till, Unassigned)
References
(Blocks 4 open bugs)
Details
Comment 1•8 years ago
|
||
Updated•8 years ago
|
Updated•5 years ago
|
Hi,
We are using a generator heavy approach in a new web component. Turns out it runs really slowly in Firefox. Found this jsperf (https://jsperf.com/generator-overhead-2) and tried it on Chrome, Safari and FF on my machine. Safari was roughly twice as fast as FF, Chrome seven times faster. Any hope of this improving soon? Would be great!
Regards,
Johan Isaksson
Comment 4•5 years ago
|
||
Any hope for this issue to be resolved? Is it on the radar at least (included in the roadmap)?
Comment 5•5 years ago
|
||
(In reply to Nickolay Platonov from comment #4)
Any hope for this issue to be resolved? Is it on the radar at least (included in the roadmap)?
From what I recall this was in the pipeline for December 2019, however, we since decided to simplify a lot SpiderMonkey to reduce the security risks related to IonBuilder usage. There is a project which is currently investigating the simplification of IonBuilder, and maybe replace it by WarpBuilder (Bug 1613592).
While this is not directly related, this is a higher priority at the moment.
I will raise this topic to the team, to see if we should raise the priority of generators in IonMonkey.
Comment 6•5 years ago
|
||
Thanks for the update! Yes, please draw some attention to this issue. We wrote a "modern" system, that uses generators a lot, and discovered the performance in FF is very bad (#1565549). In the same time its so powerful feature, arguably a best addition to JS spec in its history.
Comment 7•5 years ago
|
||
Yeah we expect this to be significantly easier with WarpBuilder in the future. Note though that Ion/Warp-compiling them is just part of it, generators also require more environment objects right now, and that could be a bigger issue than them not running in Ion.
Updated•2 years ago
|
Description
•