RFR: 8349835: C2: simplify IGV property printing [v6]
Christian Hagedorn
chagedorn at openjdk.org
Thu Nov 20 12:11:45 UTC 2025
On Mon, 17 Nov 2025 10:27:05 GMT, Saranya Natarajan <snatarajan at openjdk.org> wrote:
>> The code that prints node properties and live range properties is very verbose and repetitive and could be simplified by applying a refactoring suggested [here](https://github.com/openjdk/jdk/pull/23558#discussion_r1950785708).
>>
>> ### Fix
>> Implemented the suggested refactoring.
>>
>> ### Testing
>> Github Actions, Tier 1-3
>
> 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 the updates! Apart from Roberto's suggestions, it looks good!
-------------
Marked as reviewed by chagedorn (Reviewer).
PR Review: https://git.openjdk.org/jdk/pull/26902#pullrequestreview-3487497803
More information about the hotspot-compiler-dev
mailing list