[Interpreter] Don't compile Api or Builtin id functions through the interpreter.

src/compiler.cc

Index: src/compiler.cc
diff --git a/src/compiler.cc b/src/compiler.cc
index 9730f576297ddd2eaf3e944e0b3dbd4a189e621f..20371a84feb0f927b78cf5dbcc5ae4bbb3f31eea 100644
--- a/src/compiler.cc
+++ b/src/compiler.cc
@@ -747,7 +747,9 @@ MUST_USE_RESULT static MaybeHandle<Code> GetUnoptimizedCodeCommon(
   SetExpectedNofPropertiesFromEstimate(shared, lit->expected_property_count());
   MaybeDisableOptimization(shared, lit->dont_optimize_reason());
 
-  if (FLAG_ignition && info->closure()->PassesFilter(FLAG_ignition_filter) &&
+  if (FLAG_ignition &&
+      !shared->IsApiFunction() && !shared->HasBuiltinFunctionId() &&

[Michael Starzinger, 2015/11/02 14:08:43]

Can SharedFunctionInfo::IsApiFunction() really happen here? This would really
surprise/concern me, because those function should never get anything resembling
a lazy-compile stub installed. I would vote for dropping that part of the
predicate.

[rmcilroy, 2015/11/02 17:55:38]

Yeah, this doesn't seem to happen, at least in the code I'm running. Dropped the
IsApiFunction() part.

+      info->closure()->PassesFilter(FLAG_ignition_filter) &&
       ScriptPassesFilter(FLAG_ignition_script_filter, info->script())) {
     // Compile bytecode for the interpreter.
     if (!GenerateBytecode(info)) return MaybeHandle<Code>();