Closed
Bug 1431434
Opened 6 years ago
Closed 6 years ago
Merge SweepRegExps with SweepMisc
Categories
(Core :: JavaScript: GC, enhancement, P3)
Core
JavaScript: GC
Tracking
()
RESOLVED
FIXED
mozilla61
People
(Reporter: anba, Assigned: sumi29, Mentored)
Details
(Keywords: good-first-bug)
Attachments
(1 file)
https://mozilla.logbot.info/jsapi/20180118#c14152420 SweepRegExps is no longer performing any complicated computations (bug 1345177), so we should merge it with SweepMisc to avoid any additional AutoRunParallelTask overhead.
Updated•6 years ago
|
Keywords: good-first-bug
Priority: -- → P3
Updated•6 years ago
|
Mentor: jcoppeard
Comment hidden (mozreview-request) |
Comment 2•6 years ago
|
||
mozreview-review |
Comment on attachment 8971733 [details] Bug 1431434 - Merge SweepRegExps with SweepMisc; https://reviewboard.mozilla.org/r/240496/#review246262 Looks good. Thanks for this fix!
Attachment #8971733 -
Flags: review?(jcoppeard) → review+
Comment 3•6 years ago
|
||
Sumit, I assume from comments on other bugs that you don't have commit access, so I'll land this.
Assignee: nobody → sumi29
Pushed by jcoppeard@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/a2409475579a Merge SweepRegExps with SweepMisc; r=jonco
Comment 5•6 years ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/a2409475579a
Status: NEW → RESOLVED
Closed: 6 years ago
status-firefox61:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla61
Updated•6 years ago
|
status-firefox60:
--- → wontfix
You need to log in
before you can comment on or make changes to this bug.
Description
•