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