Added a EvalCache that caches eval'ed scripts and compiled function boilerpla...

src/runtime.cc

Index: src/runtime.cc
===================================================================
--- src/runtime.cc	(revision 147)
+++ src/runtime.cc	(working copy)
@@ -3328,10 +3328,26 @@
   }
 
   // Compile eval() source.
+  bool is_global_context = context->IsGlobalContext();

[Kasper Lund, 2008/09/05 09:16:29]

I think you should move this code to the Compiler and use it for scripts too. It
would be nice to be able to just put it in compiler.cc:MakeFunction but that may
be a little to late because the Compile, CompileLazy, CompileEval functions
build script data structures and may do pre-parsing which should be avoided too.
You probably have to refactor it a bit.

[Feng Qian, 2008/09/09 03:42:50]

Done.

   Handle<String> source(String::cast(args[0]));
-  Handle<JSFunction> boilerplate =
-      Compiler::CompileEval(context->IsGlobalContext(), source);
-  if (boilerplate.is_null()) return Failure::Exception();
+  Object* obj = Heap::LookupEvalCache(is_global_context, *source);
+  if (obj->IsFailure()) return obj;
+
+  Handle<JSFunction> boilerplate;
+  if (!obj->IsJSFunction()) {
+    Counters::eval_cache_misses.Increment();
+    boilerplate = Compiler::CompileEval(is_global_context, source);
+    if (boilerplate.is_null()) return Failure::Exception();
+
+    Object* obj =
+        Heap::PutInEvalCache(is_global_context, *source, *boilerplate);
+    if (obj->IsFailure()) return obj;
+
+  } else {
+    Counters::eval_cache_hits.Increment();
+    boilerplate = Handle<JSFunction>(JSFunction::cast(obj));
+  }
+
   Handle<JSFunction> fun =
       Factory::NewFunctionFromBoilerplate(boilerplate, context);
   return *fun;