String.prototype.includes should be SIMD-optimized
Categories
(Core :: JavaScript: Standard Library, enhancement, P1)
Tracking
()
Tracking | Status | |
---|---|---|
firefox104 | --- | fixed |
People
(Reporter: alexical, Assigned: alexical)
References
Details
Attachments
(2 files)
Setting up this bug to track existing work regarding making String.prototype.includes
and String.prototype.indexOf
's faster through SIMDification.
Assignee | ||
Comment 1•3 years ago
|
||
Assignee | ||
Comment 2•3 years ago
|
||
Depends on D150067
Updated•3 years ago
|
Updated•3 years ago
|
Comment 4•3 years ago
|
||
Backed out for causing spidermonkey bustages on SIMD.cpp
Comment 6•3 years ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/05b8d4030ff8
https://hg.mozilla.org/mozilla-central/rev/bcfc4b4d52b9
Assignee | ||
Updated•3 years ago
|
Comment 7•3 years ago
|
||
Did this cause these big improvements on Jetstream2 -AWFY?
https://arewefastyet.com/win10/benchmarks/raptor-desktop-jetstream2?numDays=60
regexp-Average
regexp-First
regexp-Geometric
regexp-Worst
Assignee | ||
Comment 8•3 years ago
•
|
||
(In reply to Mayank Bansal from comment #7)
Did this cause these big improvements on Jetstream2 -AWFY?
https://arewefastyet.com/win10/benchmarks/raptor-desktop-jetstream2?numDays=60
regexp-Average
regexp-First
regexp-Geometric
regexp-Worst
Just tested it locally and yes, this patch appears to be the cause. \o/
I'm also seeing a difference locally on regex-dna-SP., though I think it's more pronounced locally because I have the AVX2 stuff from bug 1779807 applied (which I should be able to land today.)
Description
•