Closed Bug 1589717 Opened 3 months ago Closed 3 months ago

GCVector<T>::sweep depends on T supported unbarrieredGet method

Categories

(Core :: JavaScript: GC, task, P1)

task

Tracking

()

RESOLVED FIXED
mozilla72
Tracking Status
firefox72 --- fixed

People

(Reporter: jonco, Assigned: jonco)

References

Details

Attachments

(1 file)

No description provided.

We can simplify this method by using std::remove_if, which does the shifting down of removed elements for us. We just need to make sure all our wrapper classes support moving assignment.

Pushed by jcoppeard@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/84ecc410ce65
Simplify GCVector::sweep and allow use with element types that don't have barriers r=sfink
Status: NEW → RESOLVED
Closed: 3 months ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla72
Regressions: 1593260
You need to log in before you can comment on or make changes to this bug.