RFR: 8324950: IGV: save the state to a file [v25]
Tobias Holenstein
tholenstein at openjdk.org
Tue Apr 16 15:36:44 UTC 2024
On Mon, 15 Apr 2024 09:01:22 GMT, Roberto Castañeda Lozano <rcastanedalo at openjdk.org> wrote:
> There is an issue with saved difference graph states. If I open [diff.zip](https://github.com/openjdk/jdk/files/14976481/diff.zip) (which I just created by importing some graphs, opening one of them, and diffing it against another one), I get the following assertion error:
>
> ```
> [INFO] java.lang.AssertionError
> [INFO] at com.sun.hotspot.igv.util.RangeSliderModel.setPositions(RangeSliderModel.java:101)
> [INFO] at com.sun.hotspot.igv.coordinator.OutlineTopComponent.lambda$loadContext$2(OutlineTopComponent.java:481)
> [INFO] at java.desktop/java.awt.event.InvocationEvent.dispatch(InvocationEvent.java:318)
> [INFO] at java.desktop/java.awt.EventQueue.dispatchEventImpl(EventQueue.java:773)
> [INFO] at java.desktop/java.awt.EventQueue$4.run(EventQueue.java:720)
> [INFO] at java.desktop/java.awt.EventQueue$4.run(EventQueue.java:714)
> [INFO] at java.base/java.security.AccessController.doPrivileged(AccessController.java:399)
> [INFO] at java.base/java.security.ProtectionDomain$JavaSecurityAccessImpl.doIntersectionPrivilege(ProtectionDomain.java:86)
> [INFO] at java.desktop/java.awt.EventQueue.dispatchEvent(EventQueue.java:742)
> [INFO] at org.netbeans.core.TimableEventQueue.dispatchEvent(TimableEventQueue.java:136)
> [INFO] [catch] at java.desktop/java.awt.EventDispatchThread.pumpOneEventForFilters(EventDispatchThread.java:203)
> [INFO] at java.desktop/java.awt.EventDispatchThread.pumpEventsForFilter(EventDispatchThread.java:124)
> [INFO] at java.desktop/java.awt.EventDispatchThread.pumpEventsForHierarchy(EventDispatchThread.java:113)
> [INFO] at java.desktop/java.awt.EventDispatchThread.pumpEvents(EventDispatchThread.java:109)
> [INFO] at java.desktop/java.awt.EventDispatchThread.pumpEvents(EventDispatchThread.java:101)
> [INFO] at java.desktop/java.awt.EventDispatchThread.run(EventDispatchThread.java:90)
> ```
>
> I imagine fully supporting saving and restoring the diff state would require quite a lot of additional complexity, both in IGV and in the XML files. Maybe this is not a very important use case, and we could just not support it?
Thanks for catching that! I fixed it. Difference graph are supported as long as they are in the same group. A difference graph from two different groups is just not saved to the xml.
-------------
PR Comment: https://git.openjdk.org/jdk/pull/17630#issuecomment-2059379353
More information about the hotspot-compiler-dev
mailing list