RFR: 8251464: make Node::dump(int depth) support indent
Andrew Dinn
adinn at redhat.com
Fri Sep 4 09:27:38 UTC 2020
Hi Tobias/Xin
On 04/09/2020 10:09, Tobias Hartmann wrote:
> thanks for making these changes. That looks good to me.
> But lets wait for some more opinions from other reviewers.
I'll start by noting that this is not a review, merely user feedback.
I use node dumps a lot when debugging C2 and, looking at the supplied
example, I don't find the indentation helpful -- in fact, I actually
find it slightly disrupts my reading the graph.
Also, most of the time I feed graph dumps through a sort process so that
nodes end up listed in id order, making it easier to track chains of
links in both directions. That re-ordering makes the indentation much
less useful.
Of course, this is only a report of my way of working. I'm not against
the patch per se, so long as it is easy to disable indentation (or have
emacs remove it).
regards,
Andrew Dinn
-----------
Red Hat Distinguished Engineer
Red Hat UK Ltd
Registered in England and Wales under Company Registration No. 03798903
Directors: Michael Cunningham, Michael ("Mike") O'Neill
More information about the hotspot-compiler-dev
mailing list