[turbofan] Add performance counters for escape analysis

[turbofan] Add performance counters for escape analysis

R=bmeurer@chromium.org
BUG=v8:4586
LOG=n

Committed: https://crrev.com/89654538e7b5227e5a893064ad31c325d7bede00
Cr-Commit-Position: refs/heads/master@{#33107}

[sigurds, 2016-01-04 16:25:50]

Please let me know if you think these counters are useful

[sigurds, 2016-01-04 17:52:54]

Description was changed from

==========
[turbofan] Add performance counters for escape analysis

R=bmeurer@chromium.org
==========

to

==========
[turbofan] Add performance counters for escape analysis

R=bmeurer@chromium.org
==========

[sigurds, 2016-01-04 17:52:54]

sigurds@chromium.org changed reviewers:
+ mstarzinger@chromium.org

[sigurds, 2016-01-04 18:10:31]

Description was changed from

==========
[turbofan] Add performance counters for escape analysis

R=bmeurer@chromium.org
==========

to

==========
[turbofan] Add performance counters for escape analysis

R=bmeurer@chromium.org
BUG=v8:4586
LOG=n
==========

[Benedikt Meurer, 2016-01-05 04:55:58]

LGTM with nits.

https://codereview.chromium.org/1559003002/diff/40001/src/compiler/escape-ana...
File src/compiler/escape-analysis-reducer.cc (right):

https://codereview.chromium.org/1559003002/diff/40001/src/compiler/escape-ana...
src/compiler/escape-analysis-reducer.cc:53: jsgraph()
Nit: Add EscapeAnalysisReducer::isolate() and/or
EscapeAnalysisReducer::counters() accessors.

https://codereview.chromium.org/1559003002/diff/40001/src/compiler/escape-ana...
src/compiler/escape-analysis-reducer.cc:88: jsgraph()
Nit: Add EscapeAnalysisReducer::isolate() and/or
EscapeAnalysisReducer::counters() accessors.

[sigurds, 2016-01-05 09:22:26]

FYI: Addressed comments.

https://codereview.chromium.org/1559003002/diff/40001/src/compiler/escape-ana...
File src/compiler/escape-analysis-reducer.cc (right):

https://codereview.chromium.org/1559003002/diff/40001/src/compiler/escape-ana...
src/compiler/escape-analysis-reducer.cc:53: jsgraph()
On 2016/01/05 04:55:58, Benedikt Meurer wrote:
> Nit: Add EscapeAnalysisReducer::isolate() and/or
> EscapeAnalysisReducer::counters() accessors.

Done.

https://codereview.chromium.org/1559003002/diff/40001/src/compiler/escape-ana...
src/compiler/escape-analysis-reducer.cc:88: jsgraph()
On 2016/01/05 04:55:58, Benedikt Meurer wrote:
> Nit: Add EscapeAnalysisReducer::isolate() and/or
> EscapeAnalysisReducer::counters() accessors.

Done.

[sigurds, 2016-01-05 10:19:10]

The CQ bit was checked by sigurds@chromium.org

[sigurds, 2016-01-05 10:19:10]

The patchset sent to the CQ was uploaded after l-g-t-m from bmeurer@chromium.org
Link to the patchset: https://codereview.chromium.org/1559003002/#ps80001
(title: "Rebase")

[commit-bot: I haz the power, 2016-01-05 10:19:21]

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

[commit-bot: I haz the power, 2016-01-05 10:30:16]

Description was changed from

==========
[turbofan] Add performance counters for escape analysis

R=bmeurer@chromium.org
BUG=v8:4586
LOG=n
==========

to

==========
[turbofan] Add performance counters for escape analysis

R=bmeurer@chromium.org
BUG=v8:4586
LOG=n
==========

[commit-bot: I haz the power, 2016-01-05 10:30:16]

Committed patchset #5 (id:80001)

[commit-bot: I haz the power, 2016-01-05 10:30:28]

Description was changed from

==========
[turbofan] Add performance counters for escape analysis

R=bmeurer@chromium.org
BUG=v8:4586
LOG=n
==========

to

==========
[turbofan] Add performance counters for escape analysis

R=bmeurer@chromium.org
BUG=v8:4586
LOG=n

Committed: https://crrev.com/89654538e7b5227e5a893064ad31c325d7bede00
Cr-Commit-Position: refs/heads/master@{#33107}
==========

[commit-bot: I haz the power, 2016-01-05 10:30:29]

Patchset 5 (id:??) landed as
https://crrev.com/89654538e7b5227e5a893064ad31c325d7bede00
Cr-Commit-Position: refs/heads/master@{#33107}