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