RFR: 8310220: IGV: dump graph after each IGVN step at level 4 [v2]

[Christian Hagedorn]

On Wed, 21 Jun 2023 11:22:46 GMT, Roberto Castañeda Lozano <rcastanedalo at openjdk.org> wrote:

>> src/hotspot/share/opto/phaseX.cpp line 896:
>> 
>>> 894:   const Type* newtype = type_or_null(n);
>>> 895:   if (nn != n || oldtype != newtype) {
>>> 896:     C->print_method(PHASE_AFTER_ITER_GVN_STEP, 4, n);
>> 
>> Should we keep this at level 4 and move the parser generated dumps to a new level 5? I often add dumps for these steps during IGVN but I rarely ever need the parser generated dumps.
>
> Thanks for the feedback Christian, your suggestion makes sense. I will address this in a few weeks when I am back from vacation.

Sounds good, thanks Roberto and enjoy your vacation!

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

PR Review Comment: https://git.openjdk.org/jdk/pull/14537#discussion_r1236856172