Revision: 23085
Author:   [email protected]
Date:     Tue Aug 12 15:33:05 2014 UTC
Log:      Version 3.28.71.1 (merged r23084)

Old space cannot be assumed to be iterable between GCs, even if swept precisely.

BUG=
[email protected]

Review URL: https://codereview.chromium.org/462093002
http://code.google.com/p/v8/source/detail?r=23085

Modified:
 /trunk/src/heap/mark-compact.cc
 /trunk/src/version.cc

=======================================
--- /trunk/src/heap/mark-compact.cc     Fri Aug  8 15:46:17 2014 UTC
+++ /trunk/src/heap/mark-compact.cc     Tue Aug 12 15:33:05 2014 UTC
@@ -2009,16 +2009,11 @@

static void DiscoverGreyObjectsInSpace(Heap* heap, MarkingDeque* marking_deque,
                                        PagedSpace* space) {
-  if (space->swept_precisely()) {
-    HeapObjectIterator it(space);
-    DiscoverGreyObjectsWithIterator(heap, marking_deque, &it);
-  } else {
-    PageIterator it(space);
-    while (it.has_next()) {
-      Page* p = it.next();
-      DiscoverGreyObjectsOnPage(marking_deque, p);
-      if (marking_deque->IsFull()) return;
-    }
+  PageIterator it(space);
+  while (it.has_next()) {
+    Page* p = it.next();
+    DiscoverGreyObjectsOnPage(marking_deque, p);
+    if (marking_deque->IsFull()) return;
   }
 }

=======================================
--- /trunk/src/version.cc       Tue Aug 12 06:42:13 2014 UTC
+++ /trunk/src/version.cc       Tue Aug 12 15:33:05 2014 UTC
@@ -35,7 +35,7 @@
 #define MAJOR_VERSION     3
 #define MINOR_VERSION     28
 #define BUILD_NUMBER      71
-#define PATCH_LEVEL       0
+#define PATCH_LEVEL       1
 // Use 1 for candidates and 0 otherwise.
 // (Boolean macro values are not supported by all preprocessors.)
 #define IS_CANDIDATE_VERSION 0

--
--
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.

Reply via email to