# HG changeset patch # User Lukas Stadler # Date 1372053693 -7200 # Node ID dcbdf71c450741cfce91dfd9265e359f593af34f # Parent 175a4900c2302eb2ad0cd93c61f9483178cfec77 remove scheduledNext from ScheduledNode diff -r 175a4900c230 -r dcbdf71c4507 graal/com.oracle.graal.nodes/src/com/oracle/graal/nodes/FixedWithNextNode.java --- a/graal/com.oracle.graal.nodes/src/com/oracle/graal/nodes/FixedWithNextNode.java Mon Jun 24 02:19:14 2013 +0200 +++ b/graal/com.oracle.graal.nodes/src/com/oracle/graal/nodes/FixedWithNextNode.java Mon Jun 24 08:01:33 2013 +0200 @@ -30,13 +30,15 @@ */ public abstract class FixedWithNextNode extends FixedNode { + @Successor private FixedNode next; + public FixedNode next() { - assert scheduledNext() == null || scheduledNext() instanceof FixedNode : "next() cannot be used while the graph is scheduled"; - return (FixedNode) scheduledNext(); + return next; } public void setNext(FixedNode x) { - setScheduledNext(x); + updatePredecessor(next, x); + next = x; } public FixedWithNextNode(Stamp stamp) { diff -r 175a4900c230 -r dcbdf71c4507 graal/com.oracle.graal.nodes/src/com/oracle/graal/nodes/ScheduledNode.java --- a/graal/com.oracle.graal.nodes/src/com/oracle/graal/nodes/ScheduledNode.java Mon Jun 24 02:19:14 2013 +0200 +++ b/graal/com.oracle.graal.nodes/src/com/oracle/graal/nodes/ScheduledNode.java Mon Jun 24 08:01:33 2013 +0200 @@ -26,17 +26,6 @@ public abstract class ScheduledNode extends Node { - @Successor private ScheduledNode scheduledNext; // the immediate successor of the current node - - public ScheduledNode scheduledNext() { - return scheduledNext; - } - - public void setScheduledNext(ScheduledNode x) { - updatePredecessor(scheduledNext, x); - scheduledNext = x; - } - @Override public StructuredGraph graph() { return (StructuredGraph) super.graph(); diff -r 175a4900c230 -r dcbdf71c4507 graal/com.oracle.graal.phases/src/com/oracle/graal/phases/schedule/SchedulePhase.java --- a/graal/com.oracle.graal.phases/src/com/oracle/graal/phases/schedule/SchedulePhase.java Mon Jun 24 02:19:14 2013 +0200 +++ b/graal/com.oracle.graal.phases/src/com/oracle/graal/phases/schedule/SchedulePhase.java Mon Jun 24 08:01:33 2013 +0200 @@ -209,25 +209,6 @@ sortNodesWithinBlocks(graph, selectedStrategy); } - /** - * Sets {@link ScheduledNode#scheduledNext} on all scheduled nodes in all blocks using the - * scheduling built by {@link #run(StructuredGraph)}. This method should thus only be called - * when run has been successfully executed. - */ - public void scheduleGraph() { - assert blockToNodesMap != null : "cannot set scheduledNext before run has been executed"; - for (Block block : cfg.getBlocks()) { - List nodeList = blockToNodesMap.get(block); - ScheduledNode last = null; - for (ScheduledNode node : nodeList) { - if (last != null) { - last.setScheduledNext(node); - } - last = node; - } - } - } - public ControlFlowGraph getCFG() { return cfg; } diff -r 175a4900c230 -r dcbdf71c4507 graal/com.oracle.graal.truffle/src/com/oracle/graal/truffle/nodes/IntegerExactArithmeticSplitNode.java --- a/graal/com.oracle.graal.truffle/src/com/oracle/graal/truffle/nodes/IntegerExactArithmeticSplitNode.java Mon Jun 24 02:19:14 2013 +0200 +++ b/graal/com.oracle.graal.truffle/src/com/oracle/graal/truffle/nodes/IntegerExactArithmeticSplitNode.java Mon Jun 24 08:01:33 2013 +0200 @@ -48,7 +48,7 @@ @Override public double probability(AbstractBeginNode successor) { - return successor == scheduledNext() ? 1 : 0; + return successor == next ? 1 : 0; } @Override