- Consolidate code into the old generation.

vm/gc_sweeper.cc

Index: vm/gc_sweeper.cc
===================================================================
--- vm/gc_sweeper.cc	(revision 14048)
+++ vm/gc_sweeper.cc	(working copy)
@@ -22,6 +22,7 @@
     return 0;
   }
 
+  bool is_executable = (page->type() == HeapPage::kExecutable);
   uword current = page->object_start();
   uword end = page->object_end();
 
@@ -51,6 +52,9 @@
         free_end += next_obj->Size();
       }
       obj_size = free_end - current;
+      if (is_executable) {
+        memset(reinterpret_cast<void*>(current), 0xcc, obj_size);

[cshapiro, 2012/10/26 04:58:07]

Can we put this behind an compile time flag, like what is done in the scavenger?

#if defined(DEBUG)
      if (is_executable) {
        memset(reinterpret_cast<void*>(current), 0xcc, obj_size);
      }
#endif  // DEBUG
 

[Ivan Posva, 2012/10/26 05:00:45]

I left this specifically in for now while this is new to ensure that we do not
end up jumping into free'd code. I will add a TODO to move this under DEBUG
though.

+      }
       freelist->Free(current, obj_size);
     }
     current += obj_size;