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