Closed Bug 1468422 Opened 6 years ago Closed 2 years ago

Investigate allowing collection of atoms zone sooner after off-thread parsing

Categories

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

61 Branch
enhancement

Tracking

()

RESOLVED WORKSFORME

People

(Reporter: jonco, Unassigned)

References

Details

Currently we disallow collection of the atoms zone until zones used by off-thread parsing have been merged to their target zones (JSContext::canCollectAtoms() returns |!runtime()->hasHelperThreadZones()|).

I think this is overly conservative.  We cannot collect atoms while off-thread parses are running, but it should be fine to allow this when they have finished, even if the zones have not been merged yet.

We would need to mark such zones so atoms they reference would be kept alive.  There might be some complication if we merged them during an incremental GC though.
See Also: → 1687973

Off-thread parsing has been removed so this isn't an issue any more.

Status: NEW → RESOLVED
Closed: 2 years ago
Resolution: --- → WORKSFORME
You need to log in before you can comment on or make changes to this bug.