RFR: 8368675: IGV: nodes are wrongly marked as changed in the difference view
    Roberto Castañeda Lozano 
    rcastanedalo at openjdk.org
       
    Mon Sep 29 08:42:22 UTC 2025
    
    
  
On Fri, 26 Sep 2025 15:30:49 GMT, Damon Fenacci <dfenacci at openjdk.org> wrote:
>> This changeset refines IGV's node difference analysis to ignore changes in node properties that are derived by IGV, as opposed to generated by HotSpot. Derived properties include the label and color of each node. Ignoring changes in these properties prevents IGV from wrongly marking equal nodes as "changed" (colored in yellow) when showing the difference between two graphs:
>> 
>> <img width="2323" height="651" alt="before-after" src="https://github.com/user-attachments/assets/ea51c86d-4719-45c0-b615-e6e4b8aec023" />
>> 
>> **Testing:** tier1 and manual testing on a few graphs.
>
> src/utils/IdealGraphVisualizer/Data/src/main/java/com/sun/hotspot/igv/data/serialization/Parser.java line 84:
> 
>> 82:     public static final String TO_INDEX_PROPERTY = "toIndex";
>> 83:     public static final String TO_INDEX_ALT_PROPERTY = "index";
>> 84:     public static final String EDGE_LABEL_PROPERTY = "label";
> 
> changed to improve its "expressiveness"?
Yes, just to make it more explicit that this is not the same "label" as node labels.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/27515#discussion_r2387147906
    
    
More information about the hotspot-compiler-dev
mailing list