[Interpreter] Add ignition compile trace events.

[Interpreter] Add ignition compile trace events.

BUG=v8:4280
LOG=N

Committed: https://crrev.com/3b537f380f0bacd62bd7c4010312b14666bffd87
Cr-Commit-Position: refs/heads/master@{#34899}

[rmcilroy, 2016-03-18 11:15:20]

rmcilroy@chromium.org changed reviewers:
+ jochen@chromium.org

[rmcilroy, 2016-03-18 11:15:21]

Jochen, PTAL, thanks.

[rmcilroy, 2016-03-18 11:33:12]

The CQ bit was checked by rmcilroy@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-03-18 11:33:20]

Dry run: CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1811323002/1
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1811323002/1

[commit-bot: I haz the power, 2016-03-18 12:14:32]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-03-18 12:14:32]

Dry run: This issue passed the CQ dry run.

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

https://codereview.chromium.org/1811323002/diff/1/src/interpreter/interpreter.cc
File src/interpreter/interpreter.cc (right):

https://codereview.chromium.org/1811323002/diff/1/src/interpreter/interpreter...
src/interpreter/interpreter.cc:69: TimerEventScope<TimerEventCompileIgnition>
timer(info->isolate());
why both a timer event scope and a trace event?

[rmcilroy, 2016-03-18 12:42:45]

https://codereview.chromium.org/1811323002/diff/1/src/interpreter/interpreter.cc
File src/interpreter/interpreter.cc (right):

https://codereview.chromium.org/1811323002/diff/1/src/interpreter/interpreter...
src/interpreter/interpreter.cc:69: TimerEventScope<TimerEventCompileIgnition>
timer(info->isolate());
On 2016/03/18 12:40:31, jochen wrote:
> why both a timer event scope and a trace event?

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.

[jochen (gone - plz use gerrit), 2016-03-18 15:30:00]

*shrug*

it's not needed for telemetry, but lgtm either way

[rmcilroy, 2016-03-18 15:32:01]

The CQ bit was checked by rmcilroy@chromium.org

[commit-bot: I haz the power, 2016-03-18 15:32:06]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1811323002/1
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1811323002/1

[commit-bot: I haz the power, 2016-03-18 15:34:05]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-03-18 15:35:05]

Description was changed from

==========
[Interpreter] Add ignition compile trace events.

BUG=v8:4280
LOG=N
==========

to

==========
[Interpreter] Add ignition compile trace events.

BUG=v8:4280
LOG=N

Committed: https://crrev.com/3b537f380f0bacd62bd7c4010312b14666bffd87
Cr-Commit-Position: refs/heads/master@{#34899}
==========

[commit-bot: I haz the power, 2016-03-18 15:35:06]

Patchset 1 (id:??) landed as
https://crrev.com/3b537f380f0bacd62bd7c4010312b14666bffd87
Cr-Commit-Position: refs/heads/master@{#34899}