# HG changeset patch # User Josef Eisl # Date 1413387108 -7200 # Node ID a2ad508b8723e8e1194eb69a146fe187fb874bed # Parent ad52f96395bba4a87fa1194041e4fc1330f2ce41 ValuePosition: reduce interface. diff -r ad52f96395bb -r a2ad508b8723 graal/com.oracle.graal.lir/src/com/oracle/graal/lir/CompositeValueClass.java --- a/graal/com.oracle.graal.lir/src/com/oracle/graal/lir/CompositeValueClass.java Wed Oct 15 17:24:59 2014 +0200 +++ b/graal/com.oracle.graal.lir/src/com/oracle/graal/lir/CompositeValueClass.java Wed Oct 15 17:31:48 2014 +0200 @@ -121,10 +121,6 @@ return result.toString(); } - EnumSet getFlags(ValuePosition pos) { - return values.getFlags(pos.getIndex()); - } - void copyValueArrays(CompositeValue compositeValue) { for (int i = values.getDirectCount(); i < values.getCount(); i++) { Value[] valueArray = values.getValueArray(compositeValue, i); diff -r ad52f96395bb -r a2ad508b8723 graal/com.oracle.graal.lir/src/com/oracle/graal/lir/ValuePosition.java --- a/graal/com.oracle.graal.lir/src/com/oracle/graal/lir/ValuePosition.java Wed Oct 15 17:24:59 2014 +0200 +++ b/graal/com.oracle.graal.lir/src/com/oracle/graal/lir/ValuePosition.java Wed Oct 15 17:31:48 2014 +0200 @@ -52,7 +52,8 @@ */ private final int subIndex; /** - * @see #getOuterPosition() + * The {@link ValuePosition} of the containing {@link CompositeValue} if this value is part of a + * {@link CompositeValue}, otherwise {@link #ROOT_VALUE_POSITION}. */ private final ValuePosition outerPosition; @@ -70,7 +71,7 @@ * @return True if the value denoted by this {@linkplain ValuePosition position} is part of a * {@link CompositeValue}. */ - public boolean isCompositePosition() { + private boolean isCompositePosition() { return outerPosition != ROOT_VALUE_POSITION; } @@ -110,14 +111,6 @@ } } - int getSubIndex() { - return subIndex; - } - - int getIndex() { - return index; - } - /** * @return The flags associated with the value denoted by this {@linkplain ValuePosition * position}. @@ -126,16 +119,6 @@ return values.getFlags(index); } - /** - * @return The {@link ValuePosition} of the containing {@link CompositeValue} if this value is - * part of a {@link CompositeValue}, otherwise {@link #ROOT_VALUE_POSITION}. - * - * @see #isCompositePosition() - */ - public ValuePosition getOuterPosition() { - return outerPosition; - } - @Override public String toString() { String str = "(" + index + (subIndex < 0 ? "" : "/" + subIndex) + ")";