[Interpreter] Add ignition compile trace events.

src/interpreter/interpreter.cc

 // Copyright 2015 the V8 project authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "src/interpreter/interpreter.h"
 
 #include "src/ast/prettyprinter.h"
 #include "src/code-factory.h"
 #include "src/compiler.h"
 #include "src/factory.h"
@@ skipping 48 matching lines @@
 }
 
 // static
 int Interpreter::InterruptBudget() {
   // TODO(ignition): Tune code size multiplier.
   const int kCodeSizeMultiplier = 32;
   return FLAG_interrupt_budget * kCodeSizeMultiplier;
 }
 
 bool Interpreter::MakeBytecode(CompilationInfo* info) {
+  TimerEventScope<TimerEventCompileIgnition> timer(info->isolate());

[jochen (gone - plz use gerrit), 2016/03/18 12:40:31]

why both a timer event scope and a trace event?

[rmcilroy, 2016/03/18 12:42:44]

I was making it the same as FCG - I thought we might want to scope timer events
during ignition compile in the same way as FCG does. Happy to remove this if you
don't think it's useful though.

+  TRACE_EVENT0("v8", "V8.CompileIgnition");
+
   if (FLAG_print_bytecode || FLAG_print_source || FLAG_print_ast) {
     OFStream os(stdout);
     base::SmartArrayPointer<char> name = info->GetDebugName();
     os << "[generating bytecode for function: " << info->GetDebugName().get()
        << "]" << std::endl
        << std::flush;
   }
 
 #ifdef DEBUG
   if (info->parse_info() && FLAG_print_source) {
@@ skipping 1841 matching lines @@
   Node* index = __ LoadRegister(index_reg);
   Node* one = __ SmiConstant(Smi::FromInt(1));
   Node* result = __ SmiAdd(index, one);
   __ SetAccumulator(result);
   __ Dispatch();
 }
 
 }  // namespace interpreter
 }  // namespace internal
 }  // namespace v8