RFR: 8280076: Unify IGV and IR printing

openjdk-notifier[bot] duke at openjdk.java.net
Tue Jan 18 08:04:54 UTC 2022


On Mon, 17 Jan 2022 12:22:32 GMT, Nils Eliasson <neliasso at openjdk.org> wrote:

> This patch adds support for controlling printing of the textual ideal representation (PrintIdeal) in the same way as the printing of IGV graphs currently works. The new flag PrintIdealLevel uses the same phases and levels as the PrintIdealGraphLevel flag. This new flag also gets added to CompileCommand and CompilerDirectives. 
> 
> The motivation for this change is to add support for testing during different phases with the TestIR framework.
> 
> The old flag (PrintIdeal) is kept intact for now.
> 
> This patch is based upon https://github.com/openjdk/jdk/pull/7088
> 
> Please review,
> Nils

The dependent pull request has now been integrated, and the target branch of this pull request has been updated. This means that changes from the dependent pull request can start to show up as belonging to this pull request, which may be confusing for reviewers. To remedy this situation, simply merge the latest changes from the new target branch into this pull request by running commands similar to these in the local repository for your personal fork:


git checkout igv_print_cleanup
git fetch https://git.openjdk.java.net/jdk master
git merge FETCH_HEAD
# if there are conflicts, follow the instructions given by git merge
git commit -m "Merge master"
git push

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

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


More information about the hotspot-compiler-dev mailing list