Open Bug 1071218 Opened 10 years ago Updated 2 years ago

Refactor DebugAPI::sweepAll and JitRuntime::TraceWeakJitcodeGlobalTable to work per sweep group

Categories

(Core :: JavaScript: GC, defect)

defect

Tracking

()

People

(Reporter: terrence, Unassigned)

References

(Blocks 1 open bug)

Details

They appear to currently be visiting the entire runtime for each zone group. Not a huge issue as they are generally small, but should be cleaned up to match the style of the rest of sweeping.
Summary: Refactor WatchpointMap::sweep and Debugger::sweepAll to work per zone group → Refactor WatchpointMap::sweep and Debugger::sweepAll to work per sweep group

The WatchpointMap doesn't exist any more. This problem does apply to JitcodeGlobalTable though which is per-runtime and not per-zone. This is blocking us from asserting that we don't call IsAboutToBeFinalized on things that are not in a zone that's being swept, which would be a nice check to have.

Summary: Refactor WatchpointMap::sweep and Debugger::sweepAll to work per sweep group → Refactor Debugger::sweepAll and JitRuntime::SweepJitcodeGlobalTable to work per sweep group
Summary: Refactor Debugger::sweepAll and JitRuntime::SweepJitcodeGlobalTable to work per sweep group → Refactor DebugAPI::sweepAll and JitRuntime::TraceWeakJitcodeGlobalTable to work per sweep group
Severity: normal → S3
You need to log in before you can comment on or make changes to this bug.