# HG changeset patch # User Thomas Wuerthinger # Date 1434719041 -7200 # Node ID 9015ec60f7294a40f551e3c759c9afa4d2019c99 # Parent 187b34bd0151bf8b46f58358274a269cec0ea69a Make names of printed IGV graphs shorter. diff -r 187b34bd0151 -r 9015ec60f729 graal/com.oracle.graal.java/src/com/oracle/graal/java/BytecodeParser.java --- a/graal/com.oracle.graal.java/src/com/oracle/graal/java/BytecodeParser.java Fri Jun 19 12:48:12 2015 +0200 +++ b/graal/com.oracle.graal.java/src/com/oracle/graal/java/BytecodeParser.java Fri Jun 19 15:04:01 2015 +0200 @@ -491,7 +491,6 @@ } // Remove redundant begin nodes. - Debug.dump(graph, "Before removing redundant begins"); for (BeginNode beginNode : graph.getNodes(BeginNode.TYPE)) { Node predecessor = beginNode.predecessor(); if (predecessor instanceof ControlSplitNode) { diff -r 187b34bd0151 -r 9015ec60f729 graal/com.oracle.graal.lir/src/com/oracle/graal/lir/phases/LIRPhase.java --- a/graal/com.oracle.graal.lir/src/com/oracle/graal/lir/phases/LIRPhase.java Fri Jun 19 12:48:12 2015 +0200 +++ b/graal/com.oracle.graal.lir/src/com/oracle/graal/lir/phases/LIRPhase.java Fri Jun 19 15:04:01 2015 +0200 @@ -121,7 +121,7 @@ try (DebugCloseable a = timer.start(); DebugCloseable c = memUseTracker.start()) { run(target, lirGenRes, codeEmittingOrder, linearScanOrder, context); if (dumpLIR && Debug.isDumpEnabled(PHASE_DUMP_LEVEL)) { - Debug.dump(PHASE_DUMP_LEVEL, lirGenRes.getLIR(), "After phase %s", getName()); + Debug.dump(PHASE_DUMP_LEVEL, lirGenRes.getLIR(), "%s", getName()); } } } catch (Throwable e) { diff -r 187b34bd0151 -r 9015ec60f729 graal/com.oracle.graal.loop/src/com/oracle/graal/loop/phases/LoopFullUnrollPhase.java --- a/graal/com.oracle.graal.loop/src/com/oracle/graal/loop/phases/LoopFullUnrollPhase.java Fri Jun 19 12:48:12 2015 +0200 +++ b/graal/com.oracle.graal.loop/src/com/oracle/graal/loop/phases/LoopFullUnrollPhase.java Fri Jun 19 15:04:01 2015 +0200 @@ -51,7 +51,7 @@ Debug.log("FullUnroll %s", loop); LoopTransformations.fullUnroll(loop, context, canonicalizer); FULLY_UNROLLED_LOOPS.increment(); - Debug.dump(graph, "After fullUnroll %s", loop); + Debug.dump(graph, "FullUnroll %s", loop); peeled = true; break; } diff -r 187b34bd0151 -r 9015ec60f729 graal/com.oracle.graal.loop/src/com/oracle/graal/loop/phases/LoopPeelingPhase.java --- a/graal/com.oracle.graal.loop/src/com/oracle/graal/loop/phases/LoopPeelingPhase.java Fri Jun 19 12:48:12 2015 +0200 +++ b/graal/com.oracle.graal.loop/src/com/oracle/graal/loop/phases/LoopPeelingPhase.java Fri Jun 19 15:04:01 2015 +0200 @@ -37,7 +37,7 @@ if (LoopPolicies.shouldPeel(loop, data.getCFG())) { Debug.log("Peeling %s", loop); LoopTransformations.peel(loop); - Debug.dump(graph, "After peeling %s", loop); + Debug.dump(graph, "Peeling %s", loop); } } data.deleteUnusedNodes(); diff -r 187b34bd0151 -r 9015ec60f729 graal/com.oracle.graal.phases/src/com/oracle/graal/phases/BasePhase.java --- a/graal/com.oracle.graal.phases/src/com/oracle/graal/phases/BasePhase.java Fri Jun 19 12:48:12 2015 +0200 +++ b/graal/com.oracle.graal.phases/src/com/oracle/graal/phases/BasePhase.java Fri Jun 19 15:04:01 2015 +0200 @@ -135,7 +135,7 @@ apply(graph, context, true); } - public final void apply(final StructuredGraph graph, final C context, final boolean dumpGraph) { + protected final void apply(final StructuredGraph graph, final C context, final boolean dumpGraph) { try (DebugCloseable a = timer.start(); Scope s = Debug.scope(getClass(), this); DebugCloseable c = memUseTracker.start()) { if (dumpGraph && Debug.isDumpEnabled(BEFORE_PHASE_DUMP_LEVEL)) { Debug.dump(BEFORE_PHASE_DUMP_LEVEL, graph, "Before phase %s", getName()); @@ -144,14 +144,14 @@ executionCount.increment(); inputNodesCount.add(graph.getNodeCount()); if (dumpGraph && Debug.isDumpEnabled(PHASE_DUMP_LEVEL)) { - Debug.dump(PHASE_DUMP_LEVEL, graph, "After phase %s", getName()); + Debug.dump(PHASE_DUMP_LEVEL, graph, "%s", getName()); } if (Fingerprint.ENABLED) { String graphDesc = graph.method() == null ? graph.name : graph.method().format("%H.%n(%p)"); Fingerprint.submit("After phase %s nodes in %s are %s", getName(), graphDesc, graph.getNodes().snapshot()); } if (Debug.isVerifyEnabled()) { - Debug.verify(graph, "After phase %s", getName()); + Debug.verify(graph, "%s", getName()); } assert graph.verify(); } catch (Throwable t) { diff -r 187b34bd0151 -r 9015ec60f729 graal/com.oracle.graal.virtual/src/com/oracle/graal/virtual/phases/ea/EffectsPhase.java --- a/graal/com.oracle.graal.virtual/src/com/oracle/graal/virtual/phases/ea/EffectsPhase.java Fri Jun 19 12:48:12 2015 +0200 +++ b/graal/com.oracle.graal.virtual/src/com/oracle/graal/virtual/phases/ea/EffectsPhase.java Fri Jun 19 15:04:01 2015 +0200 @@ -98,7 +98,7 @@ } if (Debug.isDumpEnabled()) { - Debug.dump(graph, "after " + getName() + " iteration"); + Debug.dump(graph, getName() + " iteration"); } new DeadCodeEliminationPhase(Required).apply(graph);