# HG changeset patch # User Doug Simon # Date 1419133995 -36000 # Node ID 5df121664e08b71751d279c44c756e0c44daf09a # Parent c968eefb992bbe07a966990293eeac8d20333fde added factory methods for creating sets diff -r c968eefb992b -r 5df121664e08 graal/com.oracle.graal.compiler.common/src/com/oracle/graal/compiler/common/CollectionsFactory.java --- a/graal/com.oracle.graal.compiler.common/src/com/oracle/graal/compiler/common/CollectionsFactory.java Sat Dec 20 15:56:59 2014 +0000 +++ b/graal/com.oracle.graal.compiler.common/src/com/oracle/graal/compiler/common/CollectionsFactory.java Sun Dec 21 13:53:15 2014 +1000 @@ -104,4 +104,21 @@ public static Map newIdentityMap(Map m) { return getMode() == STANDARD ? new IdentityHashMap<>(m) : new LinkedIdentityHashMap<>(m); } + + /** + * Creates a set. If the current thread is {@linkplain CollectionsFactory#getMode() using} + * {@link Mode#DETERMINISTIC_ITERATION_ORDER} collections, the returned set will have an + * iteration order determined by the order in which elements are inserted in the set. + */ + public static Set newSet() { + return CollectionsFactory.getMode() == Mode.STANDARD ? new HashSet<>() : new LinkedHashSet<>(); + } + + /** + * @see #newSet() + */ + public static Set newSet(Collection c) { + return CollectionsFactory.getMode() == Mode.STANDARD ? new HashSet<>(c) : new LinkedHashSet<>(c); + } + } diff -r c968eefb992b -r 5df121664e08 graal/com.oracle.graal.graph/src/com/oracle/graal/graph/Node.java --- a/graal/com.oracle.graal.graph/src/com/oracle/graal/graph/Node.java Sat Dec 20 15:56:59 2014 +0000 +++ b/graal/com.oracle.graal.graph/src/com/oracle/graal/graph/Node.java Sun Dec 21 13:53:15 2014 +1000 @@ -31,7 +31,6 @@ import sun.misc.*; import com.oracle.graal.compiler.common.*; -import com.oracle.graal.compiler.common.CollectionsFactory.Mode; import com.oracle.graal.graph.Graph.NodeEventListener; import com.oracle.graal.graph.iterators.*; import com.oracle.graal.graph.spi.*; @@ -209,20 +208,17 @@ } /** - * Creates a {@link Node} set. If the current thread is - * {@linkplain CollectionsFactory#getMode() using} {@link Mode#DETERMINISTIC_ITERATION_ORDER} - * collections, the returned set will have an iteration order determined by the order in which - * elements are inserted in the set. + * @see CollectionsFactory#newSet() */ public static Set newSet() { - return CollectionsFactory.getMode() == Mode.STANDARD ? new HashSet<>() : new LinkedHashSet<>(); + return CollectionsFactory.newSet(); } /** * @see #newSet() */ public static Set newSet(Collection c) { - return CollectionsFactory.getMode() == Mode.STANDARD ? new HashSet<>(c) : new LinkedHashSet<>(c); + return CollectionsFactory.newSet(c); } public static Map newMap() {