# HG changeset patch # User Thomas Wuerthinger # Date 1312986651 -7200 # Node ID e54641d0d012ac88ea93c1ee2649f689a88fdd61 # Parent c4f88f4ede5416a32c4b6c620afd5feb760dc36e Fix IGV to no longer access properties that are not available in the new node structure. diff -r c4f88f4ede54 -r e54641d0d012 src/share/tools/IdealGraphVisualizer/Graal/src/com/sun/hotspot/igv/graal/GraalGraphToTextConverter.java --- a/src/share/tools/IdealGraphVisualizer/Graal/src/com/sun/hotspot/igv/graal/GraalGraphToTextConverter.java Wed Aug 10 02:13:41 2011 +0200 +++ b/src/share/tools/IdealGraphVisualizer/Graal/src/com/sun/hotspot/igv/graal/GraalGraphToTextConverter.java Wed Aug 10 16:30:51 2011 +0200 @@ -122,8 +122,8 @@ // and for slots that are connected with multiple edges, by // from/to node - int succCount = Integer.parseInt(n.getProperties().get("successorCount")); InputEdge[] outgoing = outgoingEdges.get(n).toArray(new InputEdge[0]); + int succCount = outgoing.length - Integer.parseInt(n.getProperties().get("usageCount")); int i = 0; if (outgoing.length > 0 && outgoing[0].getFromIndex() < succCount) { diff -r c4f88f4ede54 -r e54641d0d012 src/share/tools/IdealGraphVisualizer/Graal/src/com/sun/hotspot/igv/graal/filters/GraalEdgeColorFilter.java --- a/src/share/tools/IdealGraphVisualizer/Graal/src/com/sun/hotspot/igv/graal/filters/GraalEdgeColorFilter.java Wed Aug 10 02:13:41 2011 +0200 +++ b/src/share/tools/IdealGraphVisualizer/Graal/src/com/sun/hotspot/igv/graal/filters/GraalEdgeColorFilter.java Wed Aug 10 16:30:51 2011 +0200 @@ -70,22 +70,6 @@ } } } - for (Figure f : figures) { - Properties p = f.getProperties(); - int predCount = Integer.parseInt(p.get("predecessorCount")); - int inputCount = Integer.parseInt(p.get("inputCount")); - int variableInputCount = Integer.parseInt(p.get("variableInputCount")); - if (p.get("memoryCheckpoint") != null) { - for (InputSlot is : f.getInputSlots()) { - if (is.getPosition() > predCount + inputCount - variableInputCount) { - is.setColor(memoryColor); - for (Connection c : is.getConnections()) { - c.setColor(memoryColor); - } - } - } - } - } } public Color getUsageColor() {