RFR: 8288897: Clean up node dump code [v4]
Emanuel Peter
epeter at openjdk.org
Thu Jul 14 08:56:00 UTC 2022
On Wed, 13 Jul 2022 22:07:50 GMT, Xin Liu <xliu at openjdk.org> wrote:
>> Emanuel Peter has updated the pull request incrementally with one additional commit since the last revision:
>>
>> implementing Christians review suggestions
>
> src/hotspot/share/opto/node.cpp line 2671:
>
>> 2669: // only_data: whether to regard data edges only during traversal.
>> 2670: static void collect_nodes_i(GrowableArray<Node*>* queue, const Node* start, int direction, uint depth, bool include_start, bool only_ctrl, bool only_data) {
>> 2671: bool indent = depth <= PrintIdealIndentThreshold;
>
> hi, @eme64 ,
> Could you also delete PrintIdealIndentThreshold from c2_globals.hpp?
> This is like a hack to let node::dump() indent. I don't think it's quite useful. it can be done in gdb pretty-print
Oh thank you so much, I overlooked this! Will delete it.
-------------
PR: https://git.openjdk.org/jdk/pull/9234
More information about the hotspot-compiler-dev
mailing list