Closed Bug 1263769 Opened 8 years ago Closed 8 years ago

Sweep all WeakCaches in parallel

Categories

(Core :: JavaScript: GC, defect)

defect
Not set
normal

Tracking

()

RESOLVED FIXED
mozilla48
Tracking Status
firefox48 --- fixed

People

(Reporter: terrence, Assigned: terrence)

References

Details

Attachments

(1 file)

Given that we are already sweeping everything in parallel, we can't move stuff over to the new system without regressing things until we have equivalent parallelism there too. This wasn't actually that bad, surprisingly.
Attachment #8740185 - Flags: review?(sphink)
Blocks: 1263771
Blocks: 1263772
Blocks: 1263777
Comment on attachment 8740185 [details] [diff] [review]
sweep_weakcaches_in_parallel-v0.diff

Review of attachment 8740185 [details] [diff] [review]:
-----------------------------------------------------------------

This is nice.
Attachment #8740185 - Flags: review?(sphink) → review+
https://hg.mozilla.org/mozilla-central/rev/b23a6286c125
Status: ASSIGNED → RESOLVED
Closed: 8 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla48
Blocks: 1265483
Blocks: 1266552
Depends on: 1280132
You need to log in before you can comment on or make changes to this bug.