changeset 5074:ad00d1d02ed2

change MergeableState to use List<T> instead of Collection<T>
author Lukas Stadler <lukas.stadler@jku.at>
date Wed, 14 Mar 2012 17:07:30 +0100
parents 2db1ad9dd385
children 47e317d6c71d
files graal/com.oracle.graal.compiler/src/com/oracle/graal/compiler/graph/MergeableState.java graal/com.oracle.graal.compiler/src/com/oracle/graal/compiler/phases/ComputeProbabilityPhase.java graal/com.oracle.graal.compiler/src/com/oracle/graal/compiler/phases/EscapeAnalysisPhase.java graal/com.oracle.graal.compiler/src/com/oracle/graal/compiler/schedule/UnscheduleNodes.java graal/com.oracle.graal.compiler/src/com/oracle/graal/compiler/types/PropagateTypesPhase.java
diffstat 5 files changed, 12 insertions(+), 12 deletions(-) [+]
line wrap: on
line diff
--- a/graal/com.oracle.graal.compiler/src/com/oracle/graal/compiler/graph/MergeableState.java	Wed Mar 14 16:57:18 2012 +0100
+++ b/graal/com.oracle.graal.compiler/src/com/oracle/graal/compiler/graph/MergeableState.java	Wed Mar 14 17:07:30 2012 +0100
@@ -28,8 +28,8 @@
 
 public interface MergeableState <T> {
     T clone();
-    boolean merge(MergeNode merge, Collection<T> withStates);
+    boolean merge(MergeNode merge, List<T> withStates);
     void loopBegin(LoopBeginNode loopBegin);
-    void loopEnds(LoopBeginNode loopBegin, Collection<T> loopEndStates);
+    void loopEnds(LoopBeginNode loopBegin, List<T> loopEndStates);
     void afterSplit(FixedNode node);
 }
--- a/graal/com.oracle.graal.compiler/src/com/oracle/graal/compiler/phases/ComputeProbabilityPhase.java	Wed Mar 14 16:57:18 2012 +0100
+++ b/graal/com.oracle.graal.compiler/src/com/oracle/graal/compiler/phases/ComputeProbabilityPhase.java	Wed Mar 14 17:07:30 2012 +0100
@@ -180,7 +180,7 @@
         }
 
         @Override
-        public boolean merge(MergeNode merge, Collection<Probability> withStates) {
+        public boolean merge(MergeNode merge, List<Probability> withStates) {
             if (merge.forwardEndCount() > 1) {
                 HashSet<LoopInfo> intersection = new HashSet<>(loops);
                 for (Probability other : withStates) {
@@ -223,7 +223,7 @@
         }
 
         @Override
-        public void loopEnds(LoopBeginNode loopBegin, Collection<Probability> loopEndStates) {
+        public void loopEnds(LoopBeginNode loopBegin, List<Probability> loopEndStates) {
             assert loopInfo != null;
             List<LoopEndNode> loopEnds = loopBegin.orderedLoopEnds();
             int i = 0;
@@ -291,7 +291,7 @@
         }
 
         @Override
-        public boolean merge(MergeNode merge, Collection<LoopCount> withStates) {
+        public boolean merge(MergeNode merge, List<LoopCount> withStates) {
             assert merge.forwardEndCount() == withStates.size() + 1;
             if (merge.forwardEndCount() > 1) {
                 Set<LoopInfo> loops = mergeLoops.get(merge);
@@ -311,7 +311,7 @@
         }
 
         @Override
-        public void loopEnds(LoopBeginNode loopBegin, Collection<LoopCount> loopEndStates) {
+        public void loopEnds(LoopBeginNode loopBegin, List<LoopCount> loopEndStates) {
             // nothing to do...
         }
 
--- a/graal/com.oracle.graal.compiler/src/com/oracle/graal/compiler/phases/EscapeAnalysisPhase.java	Wed Mar 14 16:57:18 2012 +0100
+++ b/graal/com.oracle.graal.compiler/src/com/oracle/graal/compiler/phases/EscapeAnalysisPhase.java	Wed Mar 14 17:07:30 2012 +0100
@@ -76,7 +76,7 @@
         }
 
         @Override
-        public boolean merge(MergeNode merge, Collection<BlockExitState> withStates) {
+        public boolean merge(MergeNode merge, List<BlockExitState> withStates) {
             PhiNode vobjPhi = null;
             PhiNode[] valuePhis = new PhiNode[fieldState.length];
             for (BlockExitState other : withStates) {
@@ -129,7 +129,7 @@
         }
 
         @Override
-        public void loopEnds(LoopBeginNode loopBegin, Collection<BlockExitState> loopEndStates) {
+        public void loopEnds(LoopBeginNode loopBegin, List<BlockExitState> loopEndStates) {
             while (!(virtualObjectField instanceof PhiNode)) {
                 virtualObjectField = ((VirtualObjectFieldNode) virtualObjectField).lastState();
             }
--- a/graal/com.oracle.graal.compiler/src/com/oracle/graal/compiler/schedule/UnscheduleNodes.java	Wed Mar 14 16:57:18 2012 +0100
+++ b/graal/com.oracle.graal.compiler/src/com/oracle/graal/compiler/schedule/UnscheduleNodes.java	Wed Mar 14 17:07:30 2012 +0100
@@ -33,7 +33,7 @@
     public FixedWithNextNode last;
 
     @Override
-    public boolean merge(MergeNode merge, Collection<UnscheduleState> withStates) {
+    public boolean merge(MergeNode merge, List<UnscheduleState> withStates) {
         last = null;
         return true;
     }
@@ -44,7 +44,7 @@
     }
 
     @Override
-    public void loopEnds(LoopBeginNode loop, Collection<UnscheduleState> loopEndStates) {
+    public void loopEnds(LoopBeginNode loop, List<UnscheduleState> loopEndStates) {
         last = null;
     }
 
--- a/graal/com.oracle.graal.compiler/src/com/oracle/graal/compiler/types/PropagateTypesPhase.java	Wed Mar 14 16:57:18 2012 +0100
+++ b/graal/com.oracle.graal.compiler/src/com/oracle/graal/compiler/types/PropagateTypesPhase.java	Wed Mar 14 17:07:30 2012 +0100
@@ -118,7 +118,7 @@
         }
 
         @Override
-        public boolean merge(MergeNode merge, Collection<TypeInfo> withStates) {
+        public boolean merge(MergeNode merge, List<TypeInfo> withStates) {
             if (merge.forwardEndCount() > 1) {
                 HashMap<ValueNode, PiNodeList> newPiNodes = new HashMap<>();
                 for (Entry<ValueNode, PiNodeList> entry : piNodes.entrySet()) {
@@ -148,7 +148,7 @@
         }
 
         @Override
-        public void loopEnds(LoopBeginNode loop, Collection<TypeInfo> loopEndStates) {
+        public void loopEnds(LoopBeginNode loop, List<TypeInfo> loopEndStates) {
         }
 
         @Override