Closed
Bug 1180184
Opened 9 years ago
Closed 9 years ago
Template strings are slow because the JITs don't support JSOP_TOSTRING
Categories
(Core :: JavaScript Engine: JIT, defect)
Core
JavaScript Engine: JIT
Tracking
()
RESOLVED
FIXED
mozilla42
Tracking | Status | |
---|---|---|
firefox42 | --- | fixed |
People
(Reporter: jandem, Assigned: evilpie)
References
(Blocks 1 open bug)
Details
Attachments
(1 file)
1.97 KB,
patch
|
jandem
:
review+
|
Details | Diff | Splinter Review |
According to http://kpdecker.github.io/six-speed/, our template strings are 865x slower than the non-template-string alternative. The problem is that JSOP_TOSTRING is only implemented in the interpreter.
![]() |
||
Comment 1•9 years ago
|
||
Could we make intrinsic_ToString/MToString work with all value types? Because that would be the obvious thing to do in Ion if it were made to work. Baseline could probably just do a vmcall to ToString, right?
Assignee | ||
Comment 2•9 years ago
|
||
I haven't looked into implementing this for Ion yet, but at least this should get baseline support going.
Attachment #8632225 -
Flags: review?(jdemooij)
Reporter | ||
Comment 3•9 years ago
|
||
Comment on attachment 8632225 [details] [diff] [review] baseline-tostring Review of attachment 8632225 [details] [diff] [review]: ----------------------------------------------------------------- Thanks! Want to file a follow-up bug for Ion, if you close this one? I was wondering about using an IC, but considering template strings aren't used much (yet) this is fine.
Attachment #8632225 -
Flags: review?(jdemooij) → review+
Comment 5•9 years ago
|
||
https://hg.mozilla.org/mozilla-central/rev/37ad44083b99
Status: NEW → RESOLVED
Closed: 9 years ago
status-firefox42:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla42
You need to log in
before you can comment on or make changes to this bug.
Description
•