Do not call memset() to initialize StubCache.

src/isolate.cc

Index: src/isolate.cc
diff --git a/src/isolate.cc b/src/isolate.cc
index 128136faa0b83d47ba0ed90fa972cf04282e1d2f..7445e40f0d10d07c492509dc58b9ab12ef9831af 100644
--- a/src/isolate.cc
+++ b/src/isolate.cc
@@ -1630,7 +1630,7 @@ Isolate::~Isolate() {
 
   delete transcendental_cache_;
   transcendental_cache_ = NULL;
-  delete stub_cache_;
+  free(stub_cache_);

[Sven Panne, 2012/02/27 13:03:29]

Leave this as-is.

   stub_cache_ = NULL;
   delete stats_table_;
   stats_table_ = NULL;
@@ -1779,7 +1779,7 @@ bool Isolate::Init(Deserializer* des) {
   global_handles_ = new GlobalHandles(this);
   bootstrapper_ = new Bootstrapper();
   handle_scope_implementer_ = new HandleScopeImplementer(this);
-  stub_cache_ = new StubCache(this);
+  stub_cache_ = new (calloc(1, sizeof(StubCache))) StubCache(this);

[Sven Panne, 2012/02/27 13:03:29]

I think we can remove this line completely and replace it with something
below...

[ulan, 2012/02/27 16:13:02]

Unfortunately, builtins.SetUp() assumes that stub_cache->isolate_ is
initialized.

   regexp_stack_ = new RegExpStack();
   regexp_stack_->isolate_ = this;