RFR: 8288897: Clean up node dump code [v4]

Emanuel Peter epeter at openjdk.org
Fri Jul 8 15:49:23 UTC 2022


On Fri, 24 Jun 2022 09:59:55 GMT, Christian Hagedorn <chagedorn at openjdk.org> wrote:

>> Emanuel Peter has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   implementing Christians review suggestions
>
> Otherwise, nice cleanup! I think it's the right thing to remove unused and unmaintained `dump` methods and reduce code duplication.
> 
> Have you checked that the printed node order with `dump(X)` is the same as before? I'm not sure if that is a strong requirement. I'm just thinking about `PrintIdeal` with which we do:
> https://github.com/openjdk/jdk/blob/17aacde50fb971bc686825772e29f6bfecadabda/src/hotspot/share/opto/compile.cpp#L554
> 
> Some tools/scripts might depend on the previous order of `dump(X)`. But I'm currently not aware of any such order-dependent processing. For the IR framework, the node order does not matter and if I see that correctly, the dump of an individual node is the same as before. So, it should be fine.

@chhagedorn 
> Have you checked that the printed node order with `dump(X)` is the same as before? I'm not sure if that is a strong requirement.

I did try to make sure that the output of `dump` stays equivalent. As far as I manually inspected, they are. The visit order is the same, and the same nodes are dumped.

-------------

PR: https://git.openjdk.org/jdk/pull/9234


More information about the hotspot-compiler-dev mailing list