RFR: 8350006: IGV: show memory slices as type information [v2]

Roberto Castañeda Lozano rcastanedalo at openjdk.org
Mon Feb 17 13:46:10 UTC 2025


On Fri, 14 Feb 2025 12:47:07 GMT, Quan Anh Mai <qamai at openjdk.org> wrote:

> In `MemNode::dump_adr_type`, we have this:
> 
> ```
>   ciField* field = atp->field();
>   if (field) {
>     st->print(", name=");
>     field->print_name_on(st);
>   }
>   st->print(", idx=%d;", atp->index());
> ```
> 
> By details, I mean the things like `field->print_name_on(st)` here. It would be good if we can know all the details about the memory slice from looking at the IGV, we can have it being an option that can be toggled if you are afraid that it would be too long.

The latest commit (af144195) dumps more information about the alias type ("slice") related to each node. I did not add this additional information to the type info shown by the "Show types" for compactness, but the new properties can be easily inspected in the Properties window. If you really want to show them on every node, you can e.g. set the "Node Text" field in the IGV Options window to something like `[idx] [name] ([alias_index] : [alias_field][alias_element])`. This would produce something like this (note the additional "(12 next)" text in the label of 444 LoadN):

![node-labels](https://github.com/user-attachments/assets/bc63338e-cbd1-49b3-a662-c1c7354878d8)

-------------

PR Comment: https://git.openjdk.org/jdk/pull/23621#issuecomment-2663181170


More information about the hotspot-compiler-dev mailing list