Reviewers: Michael Starzinger,
Message:
Concurrent and parallel sweeping clears the mark bits which makes evacuation
verification invalid. If we would sweep precisely, we could always iterate
over
the old space pages. We have to investigate that option. Temporarily, we
have to
disable evacuation verification.
Description:
Temporarily disable evacuation verification for parallel-concurrently swept
pages.
BUG=
Please review this at https://codereview.chromium.org/204443002/
SVN Base: https://v8.googlecode.com/svn/branches/bleeding_edge
Affected files (+4, -0 lines):
M src/mark-compact.cc
Index: src/mark-compact.cc
diff --git a/src/mark-compact.cc b/src/mark-compact.cc
index
24139307b72df0a3be9c0a279f1c82d1643a0c1e..9c862a96e49237c25b5cafd1e0fd06f5c231d3e1
100644
--- a/src/mark-compact.cc
+++ b/src/mark-compact.cc
@@ -227,6 +227,10 @@ static void VerifyEvacuation(NewSpace* space) {
static void VerifyEvacuation(PagedSpace* space) {
+ // TODO(hpayer): Bring back VerifyEvacuation for parallel-concurrently
+ // swept pages.
+ if ((FLAG_concurrent_sweeping || FLAG_parallel_sweeping) &&
+ space->was_swept_conservatively()) return;
PageIterator it(space);
while (it.has_next()) {
--
--
v8-dev mailing list
[email protected]
http://groups.google.com/group/v8-dev
---
You received this message because you are subscribed to the Google Groups "v8-dev" group.
To unsubscribe from this group and stop receiving emails from it, send an email
to [email protected].
For more options, visit https://groups.google.com/d/optout.
