Integrated: 8280026: Cleanup of IGV printing

Nils Eliasson neliasso at openjdk.java.net
Tue Jan 18 08:01:32 UTC 2022


On Fri, 14 Jan 2022 15:15:30 GMT, Nils Eliasson <neliasso at openjdk.org> wrote:

> This is a clean up of the code that controls the printing of IGV graphs. This is preparatory work for another patch.
> 
> * Default params removed when they are only used in the non-default case.
> * Moved code around so that methods with similar concern are kept together
> * Move some code from hpp to cpp file
> * Changed name of print_method with actual implementation to print_method_impl to clarify intent
> * Removed the print_method that was used with a node idx. I once introduced it for debugging - but now there are also other versions that pass a node.
> 
> Please revew,
> Nils

This pull request has now been integrated.

Changeset: 48c5f3c7
Author:    Nils Eliasson <neliasso at openjdk.org>
URL:       https://git.openjdk.java.net/jdk/commit/48c5f3c71519826d3510e5ac55980d10be763b17
Stats:     91 lines in 3 files changed: 34 ins; 40 del; 17 mod

8280026: Cleanup of IGV printing

Reviewed-by: chagedorn, thartmann

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

PR: https://git.openjdk.java.net/jdk/pull/7088


More information about the hotspot-compiler-dev mailing list