Closed
Bug 806646
Opened 12 years ago
Closed 6 months ago
IonMonkey: Fix v8-regexp performance
Categories
(Core :: JavaScript Engine, defect)
Core
JavaScript Engine
Tracking
()
RESOLVED
FIXED
People
(Reporter: dvander, Unassigned)
References
(Blocks 1 open bug)
Details
(Whiteboard: [ion:p1])
Our score on v8-regexp is about 1300, to Crankshaft and JSC's 3650. A 180% improvement would increase our overall v8 score by ~13-14%.
This depends on bug 740015 (updating Yarr again), but that alone doesn't change our score. Newer versions of Yarr have a special JIT mode that can match without capturing subexpressions, and we'll need to use that for applicable regex functions.
![]() |
Reporter | |
Updated•12 years ago
|
Whiteboard: [ion:p1]
Comment 1•11 years ago
|
||
Seems like a not-so-nice 50% regression during mid-november (possibly due to octane upgrade) put us back close to where we started a year ago.
Comment 2•11 years ago
|
||
Yes, octane-regexp changed to require creation of the results array. Our previous optimization, MatchOnly mode, therefore no longer applies.
Comment 3•11 years ago
|
||
AWFY says that Chrome x86 is 40% faster on Win8 than OSX while Firefox stays about the same. I wonder what trick they use to do that?
Assignee | ||
Updated•11 years ago
|
Assignee: general → nobody
Comment 4•5 years ago
|
||
Perhaps the keywords "meta" and/or "tracking" should be added to this bug.
Updated•2 years ago
|
Severity: normal → S3
Comment 5•6 months ago
|
||
:iain, is there anything to do here or the dependent bugs?
Flags: needinfo?(iireland)
Comment 6•6 months ago
|
||
Running locally it seems like we're about 6-7% slower than V8. Given that we don't really care about Octane these days, I think that's close enough that we don't need a metabug.
Status: NEW → RESOLVED
Closed: 6 months ago
Flags: needinfo?(iireland)
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•