Reviewers: Michael Starzinger,

Description:
Fix release build.  Add comment accidentally omitted from
preceeding change (stress-compaction).

Please review this at https://chromiumcodereview.appspot.com/10161027/

SVN Base: http://v8.googlecode.com/svn/branches/bleeding_edge/

Affected files:
  M     src/heap.cc
  M     src/spaces-inl.h


Index: src/heap.cc
===================================================================
--- src/heap.cc (revision 11431)
+++ src/heap.cc (working copy)
@@ -6142,9 +6142,11 @@


 void Heap::TearDown() {
+#ifdef DEBUG
   if (FLAG_verify_heap) {
     Verify();
   }
+#endif
   if (FLAG_print_cumulative_gc_stat) {
     PrintF("\n\n");
     PrintF("gc_count=%d ", gc_count_);
Index: src/spaces-inl.h
===================================================================
--- src/spaces-inl.h    (revision 11431)
+++ src/spaces-inl.h    (working copy)
@@ -296,6 +296,8 @@
 MaybeObject* NewSpace::AllocateRaw(int size_in_bytes) {
   Address old_top = allocation_info_.top;
 #ifdef DEBUG
+  // If we are stressing compaction we waste some memory in new space
+  // in order to get more frequent GCs.
   if (FLAG_stress_compaction && !HEAP->linear_allocation()) {
     if (allocation_info_.limit - old_top >= size_in_bytes * 4) {
       int filler_size = size_in_bytes * 4;


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

Reply via email to