[Interpreter] List top dispatch sources and destinations for a given Ignition bytecode.

[Interpreter] List top dispatch sources and destinations for a given Ignition bytecode.

This commit introduces a new mode for bytecode_dispatches_report.py
which reports the top sources of dispatches to a given bytecode and
the top destinations of dispatches from the same bytecode.

The bytecode name is passed with --top-dispatches-for-bytecode
(short form: -f), while the number of sources and destinations to
show is controlled with -n.

BUG=v8:4899
LOG=N

Committed: https://crrev.com/101e076b34a0b99e9584695109354230819bd1cb
Cr-Commit-Position: refs/heads/master@{#36284}

[Stefano Sanfilippo, 2016-05-16 15:34:24]

Patchset #1 (id:1) has been deleted

[Stefano Sanfilippo, 2016-05-16 15:34:30]

Patchset #1 (id:20001) has been deleted

[Stefano Sanfilippo, 2016-05-16 15:39:39]

Description was changed from

==========
[Interpreter] List top dispatch sources and destinations for a given bytecode.

BUG=v8:4899
LOG=N
==========

to

==========
------------------------------------------------------------------------
[Interpreter] List top dispatch sources and destinations for bytecodes.

This commit introduces a new mode for bytecode_dispatches_report.py,
which reports the top sources of dispatches to a given bytecode, and
the top destinations of dispatches from the same bytecode.

The number of sources and destinations to show is controlled with -n,
while the bytecode name is passed with 
--top-dispatch-sources-destinations-for (short form, -f).

BUG=v8:4899
LOG=N
==========

[Stefano Sanfilippo, 2016-05-16 15:41:29]

Patchset #1 (id:40001) has been deleted

[Stefano Sanfilippo, 2016-05-16 15:42:37]

Description was changed from

==========
------------------------------------------------------------------------
[Interpreter] List top dispatch sources and destinations for bytecodes.

This commit introduces a new mode for bytecode_dispatches_report.py,
which reports the top sources of dispatches to a given bytecode, and
the top destinations of dispatches from the same bytecode.

The number of sources and destinations to show is controlled with -n,
while the bytecode name is passed with 
--top-dispatch-sources-destinations-for (short form, -f).

BUG=v8:4899
LOG=N
==========

to

==========
[Interpreter] List top dispatch sources and destinations for a given Ignition
bytecode.

This commit introduces a new mode for bytecode_dispatches_report.py,
which reports the top sources of dispatches to a given bytecode, and
the top destinations of dispatches from the same bytecode.

The number of sources and destinations to show is controlled with -n,
while the bytecode name is passed with 
--top-dispatch-sources-destinations-for (short form, -f).

BUG=v8:4899
LOG=N
==========

[Stefano Sanfilippo, 2016-05-16 15:42:57]

Description was changed from

==========
[Interpreter] List top dispatch sources and destinations for a given Ignition
bytecode.

This commit introduces a new mode for bytecode_dispatches_report.py,
which reports the top sources of dispatches to a given bytecode, and
the top destinations of dispatches from the same bytecode.

The number of sources and destinations to show is controlled with -n,
while the bytecode name is passed with 
--top-dispatch-sources-destinations-for (short form, -f).

BUG=v8:4899
LOG=N
==========

to

==========
[Interpreter] List top dispatch sources and destinations for a given Ignition
bytecode.

This commit introduces a new mode for bytecode_dispatches_report.py,
which reports the top sources of dispatches to a given bytecode, and
the top destinations of dispatches from the same bytecode.

The number of sources and destinations to show is controlled with -n,
while the bytecode name is passed with 
--top-dispatch-sources-destinations-for (short form: -f).

BUG=v8:4899
LOG=N
==========

[Stefano Sanfilippo, 2016-05-16 15:44:45]

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

[rmcilroy, 2016-05-17 13:21:29]

Nice, LGTM with one suggestion. Thanks.

https://codereview.chromium.org/1979233002/diff/60001/tools/ignition/bytecode...
File tools/ignition/bytecode_dispatches_report.py (right):

https://codereview.chromium.org/1979233002/diff/60001/tools/ignition/bytecode...
tools/ignition/bytecode_dispatches_report.py:206:
"--top-dispatch-sources-destinations-for", "-f",
just --top-dispatches-for-bytecode would be a clearer flag name I think.

[Stefano Sanfilippo, 2016-05-17 13:52:00]

Patchset #2 (id:80001) has been deleted

[Stefano Sanfilippo, 2016-05-17 13:53:02]

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

[commit-bot: I haz the power, 2016-05-17 13:53:07]

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

[Stefano Sanfilippo, 2016-05-17 13:55:53]

Description was changed from

==========
[Interpreter] List top dispatch sources and destinations for a given Ignition
bytecode.

This commit introduces a new mode for bytecode_dispatches_report.py,
which reports the top sources of dispatches to a given bytecode, and
the top destinations of dispatches from the same bytecode.

The number of sources and destinations to show is controlled with -n,
while the bytecode name is passed with 
--top-dispatch-sources-destinations-for (short form: -f).

BUG=v8:4899
LOG=N
==========

to

==========
[Interpreter] List top dispatch sources and destinations for a given Ignition
bytecode.

This commit introduces a new mode for bytecode_dispatches_report.py
which reports the top sources of dispatches to a given bytecode and
the top destinations of dispatches from the same bytecode.

The number of sources and destinations to show is controlled with -n,
while the bytecode name is passed with 
--top-dispatch-sources-destinations-for (short form: -f).

BUG=v8:4899
LOG=N
==========

[Stefano Sanfilippo, 2016-05-17 13:59:09]

Description was changed from

==========
[Interpreter] List top dispatch sources and destinations for a given Ignition
bytecode.

This commit introduces a new mode for bytecode_dispatches_report.py
which reports the top sources of dispatches to a given bytecode and
the top destinations of dispatches from the same bytecode.

The number of sources and destinations to show is controlled with -n,
while the bytecode name is passed with 
--top-dispatch-sources-destinations-for (short form: -f).

BUG=v8:4899
LOG=N
==========

to

==========
[Interpreter] List top dispatch sources and destinations for a given Ignition
bytecode.

This commit introduces a new mode for bytecode_dispatches_report.py
which reports the top sources of dispatches to a given bytecode and
the top destinations of dispatches from the same bytecode.

The number of sources and destinations to show is controlled with -n,
while the bytecode name is passed with --top-dispatches-for-bytecode 
(short form: -f).

BUG=v8:4899
LOG=N
==========

[Stefano Sanfilippo, 2016-05-17 14:01:19]

Description was changed from

==========
[Interpreter] List top dispatch sources and destinations for a given Ignition
bytecode.

This commit introduces a new mode for bytecode_dispatches_report.py
which reports the top sources of dispatches to a given bytecode and
the top destinations of dispatches from the same bytecode.

The number of sources and destinations to show is controlled with -n,
while the bytecode name is passed with --top-dispatches-for-bytecode 
(short form: -f).

BUG=v8:4899
LOG=N
==========

to

==========
[Interpreter] List top dispatch sources and destinations for a given Ignition
bytecode.

This commit introduces a new mode for bytecode_dispatches_report.py
which reports the top sources of dispatches to a given bytecode and
the top destinations of dispatches from the same bytecode.

The bytecode name is passed with --top-dispatches-for-bytecode
(short form: -f), while the number of sources and destinations to
show is controlled with -n.

BUG=v8:4899
LOG=N
==========

[Stefano Sanfilippo, 2016-05-17 14:05:38]

https://codereview.chromium.org/1979233002/diff/60001/tools/ignition/bytecode...
File tools/ignition/bytecode_dispatches_report.py (right):

https://codereview.chromium.org/1979233002/diff/60001/tools/ignition/bytecode...
tools/ignition/bytecode_dispatches_report.py:206:
"--top-dispatch-sources-destinations-for", "-f",
On 2016/05/17 13:21:29, rmcilroy wrote:
> just --top-dispatches-for-bytecode would be a clearer flag name I think.

Done.

[commit-bot: I haz the power, 2016-05-17 14:22:53]

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

[commit-bot: I haz the power, 2016-05-17 14:22:54]

Dry run: This issue passed the CQ dry run.

[Stefano Sanfilippo, 2016-05-17 14:41:21]

The CQ bit was checked by ssanfilippo@chromium.org

[Stefano Sanfilippo, 2016-05-17 14:41:22]

The patchset sent to the CQ was uploaded after l-g-t-m from
rmcilroy@chromium.org
Link to the patchset: https://codereview.chromium.org/1979233002/#ps100001
(title: "Rename flag.")

[commit-bot: I haz the power, 2016-05-17 14:41:28]

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

[commit-bot: I haz the power, 2016-05-17 14:42:50]

Description was changed from

==========
[Interpreter] List top dispatch sources and destinations for a given Ignition
bytecode.

This commit introduces a new mode for bytecode_dispatches_report.py
which reports the top sources of dispatches to a given bytecode and
the top destinations of dispatches from the same bytecode.

The bytecode name is passed with --top-dispatches-for-bytecode
(short form: -f), while the number of sources and destinations to
show is controlled with -n.

BUG=v8:4899
LOG=N
==========

to

==========
[Interpreter] List top dispatch sources and destinations for a given Ignition
bytecode.

This commit introduces a new mode for bytecode_dispatches_report.py
which reports the top sources of dispatches to a given bytecode and
the top destinations of dispatches from the same bytecode.

The bytecode name is passed with --top-dispatches-for-bytecode
(short form: -f), while the number of sources and destinations to
show is controlled with -n.

BUG=v8:4899
LOG=N
==========

[commit-bot: I haz the power, 2016-05-17 14:42:54]

Committed patchset #2 (id:100001)

[commit-bot: I haz the power, 2016-05-17 14:44:12]

Description was changed from

==========
[Interpreter] List top dispatch sources and destinations for a given Ignition
bytecode.

This commit introduces a new mode for bytecode_dispatches_report.py
which reports the top sources of dispatches to a given bytecode and
the top destinations of dispatches from the same bytecode.

The bytecode name is passed with --top-dispatches-for-bytecode
(short form: -f), while the number of sources and destinations to
show is controlled with -n.

BUG=v8:4899
LOG=N
==========

to

==========
[Interpreter] List top dispatch sources and destinations for a given Ignition
bytecode.

This commit introduces a new mode for bytecode_dispatches_report.py
which reports the top sources of dispatches to a given bytecode and
the top destinations of dispatches from the same bytecode.

The bytecode name is passed with --top-dispatches-for-bytecode
(short form: -f), while the number of sources and destinations to
show is controlled with -n.

BUG=v8:4899
LOG=N

Committed: https://crrev.com/101e076b34a0b99e9584695109354230819bd1cb
Cr-Commit-Position: refs/heads/master@{#36284}
==========

[commit-bot: I haz the power, 2016-05-17 14:44:13]

Patchset 2 (id:??) landed as
https://crrev.com/101e076b34a0b99e9584695109354230819bd1cb
Cr-Commit-Position: refs/heads/master@{#36284}