[turbofan] add more comments to compiler/graph-visualizer.cc.

[turbofan] add more comments to compiler/graph-visualizer.cc.

BUG=

Committed: https://crrev.com/b6a15350b2838b61c468373e0bdfd225f0a56f12
Cr-Commit-Position: refs/heads/master@{#36824}

[bgeron, 2016-06-07 17:43:09]

bgeron@google.com changed reviewers:
+ jarin@chromium.org

[bgeron, 2016-06-07 17:43:09]

Please take a look.

[Jarin, 2016-06-07 19:35:31]

lgtm with nits.

https://codereview.chromium.org/2040243002/diff/1/src/compiler/graph-visualiz...
File src/compiler/graph-visualizer.cc (right):

https://codereview.chromium.org/2040243002/diff/1/src/compiler/graph-visualiz...
src/compiler/graph-visualizer.cc:632: // the node itself.
Maybe you should hedge with "(if possible, cycles are broken arbitrarily)"

https://codereview.chromium.org/2040243002/diff/1/src/compiler/graph-visualiz...
src/compiler/graph-visualizer.cc:654: // Print the arguments
arguments -> inputs.

Please end sentences in comments with dots (here and below).

[bgeron, 2016-06-08 12:04:56]

Great feedback, thanks.

https://codereview.chromium.org/2040243002/diff/1/src/compiler/graph-visualiz...
File src/compiler/graph-visualizer.cc (right):

https://codereview.chromium.org/2040243002/diff/1/src/compiler/graph-visualiz...
src/compiler/graph-visualizer.cc:632: // the node itself.
On 2016/06/07 at 19:35:31, Jarin wrote:
> Maybe you should hedge with "(if possible, cycles are broken arbitrarily)"

Done.

https://codereview.chromium.org/2040243002/diff/1/src/compiler/graph-visualiz...
src/compiler/graph-visualizer.cc:654: // Print the arguments
On 2016/06/07 at 19:35:31, Jarin wrote:
> arguments -> inputs.
> 
> Please end sentences in comments with dots (here and below).

Done.

[bgeron, 2016-06-08 12:05:28]

The CQ bit was checked by bgeron@google.com

[bgeron, 2016-06-08 12:05:28]

The patchset sent to the CQ was uploaded after l-g-t-m from jarin@chromium.org
Link to the patchset: https://codereview.chromium.org/2040243002/#ps20001
(title: "Polish")

[commit-bot: I haz the power, 2016-06-08 12:05:36]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/2040243002/20001

[bgeron, 2016-06-08 12:09:58]

The CQ bit was checked by bgeron@google.com

[commit-bot: I haz the power, 2016-06-08 12:10:07]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/2040243002/20001

[commit-bot: I haz the power, 2016-06-08 12:30:41]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-06-08 12:31:34]

Description was changed from

==========
[turbofan] add more comments to compiler/graph-visualizer.cc.

BUG=
==========

to

==========
[turbofan] add more comments to compiler/graph-visualizer.cc.

BUG=

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

[commit-bot: I haz the power, 2016-06-08 12:31:35]

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