Revision: 13099
Author:   [email protected]
Date:     Fri Nov 30 01:42:20 2012
Log:      Revert "Allow incremental marking when expose_gc is turned on."

This reverts r13072 from bleeding edge.

[email protected]

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

Modified:
 /branches/bleeding_edge/src/extensions/gc-extension.cc
 /branches/bleeding_edge/src/heap.cc
 /branches/bleeding_edge/src/incremental-marking.cc

=======================================
--- /branches/bleeding_edge/src/extensions/gc-extension.cc Wed Nov 28 00:43:10 2012 +++ /branches/bleeding_edge/src/extensions/gc-extension.cc Fri Nov 30 01:42:20 2012
@@ -43,7 +43,7 @@
   if (args[0]->BooleanValue()) {
     HEAP->CollectGarbage(NEW_SPACE, "gc extension");
   } else {
- HEAP->CollectAllGarbage(Heap::kAbortIncrementalMarkingMask, "gc extension");
+    HEAP->CollectAllGarbage(Heap::kNoGCFlags, "gc extension");
   }
   return v8::Undefined();
 }
=======================================
--- /branches/bleeding_edge/src/heap.cc Wed Nov 28 23:38:00 2012
+++ /branches/bleeding_edge/src/heap.cc Fri Nov 30 01:42:20 2012
@@ -5323,7 +5323,8 @@
       AgeInlineCaches();
     }
     int mark_sweep_time = Min(TimeMarkSweepWouldTakeInMs(), 1000);
-    if (hint >= mark_sweep_time && incremental_marking()->IsStopped()) {
+    if (hint >= mark_sweep_time && !FLAG_expose_gc &&
+        incremental_marking()->IsStopped()) {
       HistogramTimerScope scope(isolate_->counters()->gc_context());
       CollectAllGarbage(kReduceMemoryFootprintMask,
                         "idle notification: contexts disposed");
@@ -5338,7 +5339,7 @@
     return false;
   }

-  if (!FLAG_incremental_marking || Serializer::enabled()) {
+ if (!FLAG_incremental_marking || FLAG_expose_gc || Serializer::enabled()) {
     return IdleGlobalGC();
   }

=======================================
--- /branches/bleeding_edge/src/incremental-marking.cc Thu Nov 29 07:13:49 2012 +++ /branches/bleeding_edge/src/incremental-marking.cc Fri Nov 30 01:42:20 2012
@@ -491,7 +491,8 @@
   static const intptr_t kActivationThreshold = 0;
 #endif

-  return FLAG_incremental_marking &&
+  return !FLAG_expose_gc &&
+      FLAG_incremental_marking &&
       !Serializer::enabled() &&
       heap_->PromotedSpaceSizeOfObjects() > kActivationThreshold;
 }

--
v8-dev mailing list
[email protected]
http://groups.google.com/group/v8-dev

Reply via email to