RFR: 8349835: C2: simplify IGV property printing
Damon Fenacci
dfenacci at openjdk.org
Mon Sep 22 06:58:21 UTC 2025
On Fri, 22 Aug 2025 13:28:22 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
src/hotspot/share/opto/idealGraphPrinter.cpp line 261:
> 259: void IdealGraphPrinter::print_prop_record(const IdealGraphPrintRecord rec[], int size) {
> 260: for ( int i = 0; i < size; i++ ) {
> 261: if (rec[i]._cond != 0) {
As for the comment below it might be more consistent to only use `rec[i]._cond` as the condition.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/26902#discussion_r2366898344
More information about the hotspot-compiler-dev
mailing list