[jdk11u-dev] RFR: 8139046: Compiler Control: IGVPrintLevel directive should set PrintIdealGraph

[Xin Liu]

On Fri, 20 May 2022 12:56:12 GMT, Martin Doerr <mdoerr at openjdk.org> wrote:

> Backport of [JDK-8139046](https://bugs.openjdk.java.net/browse/JDK-8139046)
> Enables dumping of C2 intermediate representation for a selected method: `-XX:PrintIdealGraphFile=igv.xml -XX:CompileCommand=option,TestKlass::testMethod,intx,IGVPrintLevel,3`
> Doesn't apply cleanly:
> 
> - compile.hpp: Had to take `should_print` from https://github.com/openjdk/jdk/commit/75e9d0a2901c149fc0acf3ed042ddedc689d8d23
> - compilerDefinitions.cpp, compile.cpp, idealGraphPrinter.cpp, idealGraphPrinter.hpp: Had to do resolve hunks manually.

This code change looks good to me.  I am not a reviewer. we still need other reviewers to approve it. 

This patch is for developer-oriented and only impacts debug-build. Is it necessary to backport to jdk11u? I think all developers of Hotspot/C2 have moved to jdk-tip.

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

Marked as reviewed by xliu (no project role).

PR: https://git.openjdk.java.net/jdk11u-dev/pull/1102