RFR: 8349835: C2: Simplify IGV property printing [v6]

Saranya Natarajan snatarajan at openjdk.org
Thu Nov 20 14:42:30 UTC 2025


On Wed, 19 Nov 2025 13:26:06 GMT, Roberto Castañeda Lozano <rcastanedalo at openjdk.org> wrote:

>> Saranya Natarajan has updated the pull request with a new target base due to a merge or a rebase. The pull request now contains nine commits:
>> 
>>  - Merge branch 'master' into JDK-8349835
>>  - testing and review on moving code to cpp
>>  - Merge branch 'master' into JDK-8349835
>>  - addressing review comments#2
>>  - fixing test failure
>>  - addressing review comments
>>  - changing int to bool in a struct
>>  - fix to failing test
>>  - initial fix
>
> Thanks for cleaning up this code and testing it thoroughly, Saranya. The changes look good to me overall, I just have a few minor suggestions.
> There may be more properties that could be printed using the new abstraction, but I am OK with addressing those separately.

Thank you for the review @robcasloz. I have now addressed these and filed [JDK-8372273](https://bugs.openjdk.org/browse/JDK-8372273) to address the rest of the node properties.

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

PR Comment: https://git.openjdk.org/jdk/pull/26902#issuecomment-3558401901


More information about the hotspot-compiler-dev mailing list