1796081 - Refactor WeakMap a little in preparation for parallel marking

Status: RESOLVED FIXED

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

Comment 1

[Jon Coppeard (:jonco)]

Attachment: Bug 1796081 - Part 1: Don't trace weak map values that are in the nursery r?sfink

We don't need to trace weakmap values that are in the nursery because these
will be kept alive in minor GC by the store buffer entry pushed by HeapPtr.

Comment 2

[Jon Coppeard (:jonco)]

Attachment: Bug 1796081 - Part 2: Only check the key color once when populating the ephemeron table r?sfink

Currently the table population is done separately from marking the entry which
means we check the key color twice, potentially getting different results if
another thread is marking at the same time. This could result in us not adding
ephemeron table entries when necessary.

The patch consolidates these two paths.

Depends on D159677

Comment 3

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/c7898b0dbad3
Part 1: Don't trace weak map values that are in the nursery r=sfink
https://hg.mozilla.org/integration/autoland/rev/894ad5982bff
Part 2: Only check the key color once when populating the ephemeron table r=sfink

Comment 4

[Iulian Moraru]

https://hg.mozilla.org/mozilla-central/rev/c7898b0dbad3
https://hg.mozilla.org/mozilla-central/rev/894ad5982bff