[Interpreter] Fix runtime-profiler ticks for Interpreted functions.

src/compiler.cc

Index: src/compiler.cc
diff --git a/src/compiler.cc b/src/compiler.cc
index a8cf14c620079bdd7a202972aad68873a5d56311..28a4a2c3f7ff32e520b01313d360bf302c66f891 100644
--- a/src/compiler.cc
+++ b/src/compiler.cc
@@ -643,6 +643,9 @@ MaybeHandle<Code> GetOptimizedCode(Handle<JSFunction> function,
   if (shared->is_compiled()) {

[Michael Starzinger, 2016/11/14 15:36:49]

question: I know this code was here before, but would using the
{SharedFunctionInfo::HasBaselineCode} predicate be more accurate here? If you
feel we shouldn't touch it as part of this CL then that's totally fine, just
wanted to throw out my thoughts here.

[rmcilroy, 2016/11/14 17:24:26]

Hmm, yeah that makes sense. I'll change it here and see what happens. Also made
this an if/else, so that we only reset the shared profiler ticks if there isn't
baseline code (so only affect the Ignition->TF, not the FCG->Crankshaft path)

     shared->code()->set_profiler_ticks(0);
   }
+  if (shared->HasBytecodeArray()) {
+    shared->set_profiler_ticks(0);
+  }
 
   VMState<COMPILER> state(isolate);
   DCHECK(!isolate->has_pending_exception());
@@ -735,6 +738,9 @@ CompilationJob::Status FinalizeOptimizedCompilationJob(CompilationJob* job) {
 
   Handle<SharedFunctionInfo> shared = info->shared_info();
   shared->code()->set_profiler_ticks(0);

[Michael Starzinger, 2016/11/14 15:36:49]

Likewise.

[rmcilroy, 2016/11/14 17:24:26]

Done.

+  if (shared->HasBytecodeArray()) {
+    shared->set_profiler_ticks(0);
+  }
 
   DCHECK(!shared->HasDebugInfo());