Commit b9420be7 authored by Michael Lippautz's avatar Michael Lippautz Committed by V8 LUCI CQ

[heap] Remove to aggressive CHECK from inner marking function

The check whether worklists are empty sits after marking the
transitive closure, when it is guaranteed that no concurrent marker is
running anymore.

Bug: chromium:1325628
Change-Id: Ibfa7278df2181a0aa6c7e0f1d53d51e8afaa3352
Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3647830
Commit-Queue: Michael Lippautz <mlippautz@chromium.org>
Reviewed-by: 's avatarDominik Inführ <dinfuehr@chromium.org>
Cr-Commit-Position: refs/heads/main@{#80566}
parent b33179ae
...@@ -2266,7 +2266,6 @@ void MarkCompactCollector::MarkTransitiveClosureLinear() { ...@@ -2266,7 +2266,6 @@ void MarkCompactCollector::MarkTransitiveClosureLinear() {
ResetNewlyDiscovered(); ResetNewlyDiscovered();
ephemeron_marking_.newly_discovered.shrink_to_fit(); ephemeron_marking_.newly_discovered.shrink_to_fit();
CHECK(local_marking_worklists()->IsEmpty());
CHECK(weak_objects_.current_ephemerons.IsEmpty()); CHECK(weak_objects_.current_ephemerons.IsEmpty());
CHECK(weak_objects_.discovered_ephemerons.IsEmpty()); CHECK(weak_objects_.discovered_ephemerons.IsEmpty());
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment