Simplify JIT event logging.

Simplify JIT event logging.

R=jgruber@chromium.org
BUG=v8:5117
Committed: https://crrev.com/bcb5b3d550c744e119e412c46e03f9800f1b619e
Cr-Commit-Position: refs/heads/master@{#37435}

[jgruber, 2016-06-30 08:49:19]

LGTMwC

https://codereview.chromium.org/2111793002/diff/1/src/source-position-table.h
File src/source-position-table.h (right):

https://codereview.chromium.org/2111793002/diff/1/src/source-position-table.h...
src/source-position-table.h:11: #include "src/log.h"
Can we move this to the .cc file?

https://codereview.chromium.org/2111793002/diff/1/src/source-position-table.h...
src/source-position-table.h:37: void StartJitLogging();
Is this used anywhere?

https://codereview.chromium.org/2111793002/diff/1/src/source-position-table.h...
src/source-position-table.h:54: // over the lifetime of a PositionsRecorder
s/PositionsRecorder/SourcePositionTableBuilder/

[jgruber, 2016-06-30 08:49:32]

lgtm

[Yang, 2016-06-30 09:00:27]

https://codereview.chromium.org/2111793002/diff/1/src/source-position-table.h
File src/source-position-table.h (right):

https://codereview.chromium.org/2111793002/diff/1/src/source-position-table.h...
src/source-position-table.h:11: #include "src/log.h"
On 2016/06/30 08:49:18, jgruber wrote:
> Can we move this to the .cc file?

Done.

https://codereview.chromium.org/2111793002/diff/1/src/source-position-table.h...
src/source-position-table.h:37: void StartJitLogging();
On 2016/06/30 08:49:18, jgruber wrote:
> Is this used anywhere?

Done.

https://codereview.chromium.org/2111793002/diff/1/src/source-position-table.h...
src/source-position-table.h:54: // over the lifetime of a PositionsRecorder
On 2016/06/30 08:49:18, jgruber wrote:
> s/PositionsRecorder/SourcePositionTableBuilder/

Done.

[Yang, 2016-06-30 09:06:51]

The CQ bit was checked by yangguo@chromium.org

[Yang, 2016-06-30 09:06:51]

The patchset sent to the CQ was uploaded after l-g-t-m from jgruber@chromium.org
Link to the patchset: https://codereview.chromium.org/2111793002/#ps40001
(title: "fix forward declaration")

[commit-bot: I haz the power, 2016-06-30 09:06:54]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-06-30 09:11:04]

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

[commit-bot: I haz the power, 2016-06-30 09:11:05]

Try jobs failed on following builders:
  v8_presubmit on master.tryserver.v8 (JOB_FAILED,
http://build.chromium.org/p/tryserver.v8/builders/v8_presubmit/builds/18512)

[Benedikt Meurer, 2016-06-30 09:26:28]

bmeurer@chromium.org changed reviewers:
+ bmeurer@chromium.org

[Benedikt Meurer, 2016-06-30 09:26:28]

LGTM (gummi-gestempelt)

[Yang, 2016-06-30 10:33:09]

The CQ bit was checked by yangguo@chromium.org

[commit-bot: I haz the power, 2016-06-30 10:33:16]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-06-30 11:17:14]

Committed patchset #3 (id:40001)

[commit-bot: I haz the power, 2016-06-30 11:17:34]

Description was changed from

==========
Simplify JIT event logging.

R=jgruber@chromium.org
BUG=v8:5117
==========

to

==========
Simplify JIT event logging.

R=jgruber@chromium.org
BUG=v8:5117

Committed: https://crrev.com/bcb5b3d550c744e119e412c46e03f9800f1b619e
Cr-Commit-Position: refs/heads/master@{#37435}
==========

[commit-bot: I haz the power, 2016-06-30 11:17:35]

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