# HG changeset patch # User Bernhard Urban # Date 1387292457 -3600 # Node ID 413040ab993ee49993b67db5e29eae101dfbadb5 # Parent 1480cfe9746290cda003dad9b17f5b93f6aed27c remove some usages of HashSet diff -r 1480cfe97462 -r 413040ab993e graal/com.oracle.graal.hotspot/src/com/oracle/graal/hotspot/meta/HotSpotResolvedJavaField.java --- a/graal/com.oracle.graal.hotspot/src/com/oracle/graal/hotspot/meta/HotSpotResolvedJavaField.java Tue Dec 17 18:14:02 2013 +0100 +++ b/graal/com.oracle.graal.hotspot/src/com/oracle/graal/hotspot/meta/HotSpotResolvedJavaField.java Tue Dec 17 16:00:57 2013 +0100 @@ -117,7 +117,7 @@ return false; } - private static final Set notEmbeddable = new HashSet<>(); + private static final List notEmbeddable = new ArrayList<>(); private static void addResolvedToSet(Field field) { MetaAccessProvider metaAccess = runtime().getHostProviders().getMetaAccess(); diff -r 1480cfe97462 -r 413040ab993e graal/com.oracle.graal.java/src/com/oracle/graal/java/BciBlockMapping.java --- a/graal/com.oracle.graal.java/src/com/oracle/graal/java/BciBlockMapping.java Tue Dec 17 18:14:02 2013 +0100 +++ b/graal/com.oracle.graal.java/src/com/oracle/graal/java/BciBlockMapping.java Tue Dec 17 16:00:57 2013 +0100 @@ -434,7 +434,7 @@ predecessor.successors.add(sux); } - private final HashSet jsrVisited = new HashSet<>(); + private final ArrayList jsrVisited = new ArrayList<>(); private void createJsrAlternatives(Block block) { jsrVisited.add(block); diff -r 1480cfe97462 -r 413040ab993e graal/com.oracle.graal.phases.common/src/com/oracle/graal/phases/common/TailDuplicationPhase.java --- a/graal/com.oracle.graal.phases.common/src/com/oracle/graal/phases/common/TailDuplicationPhase.java Tue Dec 17 18:14:02 2013 +0100 +++ b/graal/com.oracle.graal.phases.common/src/com/oracle/graal/phases/common/TailDuplicationPhase.java Tue Dec 17 16:00:57 2013 +0100 @@ -87,17 +87,22 @@ if (fixedNodeCount < TailDuplicationTrivialSize.getValue()) { return true; } - HashSet improvements = new HashSet<>(); + ArrayList improvements = null; for (PhiNode phi : merge.phis()) { Stamp phiStamp = phi.stamp(); for (ValueNode input : phi.values()) { if (!input.stamp().equals(phiStamp)) { - improvements.add(phi); + if (improvements == null) { + improvements = new ArrayList<>(); + } + if (!improvements.contains(phi)) { + improvements.add(phi); + } break; } } } - if (improvements.isEmpty()) { + if (improvements == null) { return false; } FixedNode current = merge; diff -r 1480cfe97462 -r 413040ab993e graal/com.oracle.graal.virtual/src/com/oracle/graal/virtual/phases/ea/GraphEffectList.java --- a/graal/com.oracle.graal.virtual/src/com/oracle/graal/virtual/phases/ea/GraphEffectList.java Tue Dec 17 18:14:02 2013 +0100 +++ b/graal/com.oracle.graal.virtual/src/com/oracle/graal/virtual/phases/ea/GraphEffectList.java Tue Dec 17 16:00:57 2013 +0100 @@ -344,7 +344,7 @@ commit.getLocks().addAll(locks); assert commit.usages().filter(AllocatedObjectNode.class).count() == commit.usages().count(); - HashSet materializedValues = new HashSet<>(commit.usages().filter(AllocatedObjectNode.class).snapshot()); + List materializedValues = commit.usages().filter(AllocatedObjectNode.class).snapshot(); for (int i = 0; i < commit.getValues().size(); i++) { if (materializedValues.contains(commit.getValues().get(i))) { commit.getValues().set(i, ((AllocatedObjectNode) commit.getValues().get(i)).getVirtualObject());