# HG changeset patch # User Manuel Rigger # Date 1453202424 -3600 # Node ID 2160e7da7fb0e144fdfde8dae3d760043f7e5af7 # Parent 0653faea03276a8ec728916b40ccc7ba992d22fc remove redundant modifiers. diff -r 0653faea0327 -r 2160e7da7fb0 graal/com.oracle.graal.asm.aarch64.test/src/com/oracle/graal/asm/aarch64/test/TestProtectedAssembler.java --- a/graal/com.oracle.graal.asm.aarch64.test/src/com/oracle/graal/asm/aarch64/test/TestProtectedAssembler.java Tue Jan 19 10:55:27 2016 +0100 +++ b/graal/com.oracle.graal.asm.aarch64.test/src/com/oracle/graal/asm/aarch64/test/TestProtectedAssembler.java Tue Jan 19 12:20:24 2016 +0100 @@ -36,7 +36,7 @@ */ class TestProtectedAssembler extends AArch64Assembler { - public TestProtectedAssembler(TargetDescription target) { + TestProtectedAssembler(TargetDescription target) { super(target); } diff -r 0653faea0327 -r 2160e7da7fb0 graal/com.oracle.graal.asm.aarch64/src/com/oracle/graal/asm/aarch64/AArch64Assembler.java --- a/graal/com.oracle.graal.asm.aarch64/src/com/oracle/graal/asm/aarch64/AArch64Assembler.java Tue Jan 19 10:55:27 2016 +0100 +++ b/graal/com.oracle.graal.asm.aarch64/src/com/oracle/graal/asm/aarch64/AArch64Assembler.java Tue Jan 19 12:20:24 2016 +0100 @@ -55,7 +55,7 @@ * Specifies whether immediate can be represented in all cases (YES), as a 64bit instruction * (SIXTY_FOUR_BIT_ONLY) or not at all (NO). */ - static enum Representable { + enum Representable { YES, SIXTY_FOUR_BIT_ONLY, NO @@ -141,13 +141,13 @@ public final long imm; public final int encoding; - public Immediate(long imm, boolean is64, int s, int r) { + Immediate(long imm, boolean is64, int s, int r) { this.imm = imm; this.encoding = computeEncoding(is64, s, r); } // Used to be able to binary search for an immediate in the table. - public Immediate(long imm) { + Immediate(long imm) { this(imm, false, 0, 0); } @@ -254,7 +254,7 @@ public final int width; public final boolean isGeneral; - private InstructionType(int encoding, int width, boolean isGeneral) { + InstructionType(int encoding, int width, boolean isGeneral) { this.encoding = encoding; this.width = width; this.isGeneral = isGeneral; @@ -468,7 +468,7 @@ public final int encoding; - private Instruction(int encoding) { + Instruction(int encoding) { this.encoding = encoding; } @@ -482,7 +482,7 @@ public final int encoding; - private ShiftType(int encoding) { + ShiftType(int encoding) { this.encoding = encoding; } } @@ -499,7 +499,7 @@ public final int encoding; - private ExtendType(int encoding) { + ExtendType(int encoding) { this.encoding = encoding; } } @@ -576,7 +576,7 @@ public final int encoding; - private ConditionFlag(int encoding) { + ConditionFlag(int encoding) { this.encoding = encoding; } @@ -2357,7 +2357,7 @@ private final int encoding; - private SystemHint(int encoding) { + SystemHint(int encoding) { this.encoding = encoding; } } @@ -2401,7 +2401,7 @@ public final int encoding; public final String optionName; - private BarrierKind(int encoding, String optionName) { + BarrierKind(int encoding, String optionName) { this.encoding = encoding; this.optionName = optionName; } diff -r 0653faea0327 -r 2160e7da7fb0 graal/com.oracle.graal.asm.aarch64/src/com/oracle/graal/asm/aarch64/AArch64MacroAssembler.java --- a/graal/com.oracle.graal.asm.aarch64/src/com/oracle/graal/asm/aarch64/AArch64MacroAssembler.java Tue Jan 19 10:55:27 2016 +0100 +++ b/graal/com.oracle.graal.asm.aarch64/src/com/oracle/graal/asm/aarch64/AArch64MacroAssembler.java Tue Jan 19 12:20:24 2016 +0100 @@ -1106,7 +1106,7 @@ public final int encoding; - private PatchLabelKind(int encoding) { + PatchLabelKind(int encoding) { this.encoding = encoding; } @@ -1242,7 +1242,7 @@ public final int encoding; - private AArch64ExceptionCode(int encoding) { + AArch64ExceptionCode(int encoding) { this.encoding = encoding; } } diff -r 0653faea0327 -r 2160e7da7fb0 graal/com.oracle.graal.asm.amd64.test/src/com/oracle/graal/asm/amd64/test/IncrementDecrementMacroTest.java --- a/graal/com.oracle.graal.asm.amd64.test/src/com/oracle/graal/asm/amd64/test/IncrementDecrementMacroTest.java Tue Jan 19 10:55:27 2016 +0100 +++ b/graal/com.oracle.graal.asm.amd64.test/src/com/oracle/graal/asm/amd64/test/IncrementDecrementMacroTest.java Tue Jan 19 12:20:24 2016 +0100 @@ -60,7 +60,7 @@ private static class IncrementCodeGenTest implements CodeGenTest { final int value; - public IncrementCodeGenTest(int value) { + IncrementCodeGenTest(int value) { this.value = value; } @@ -97,7 +97,7 @@ private static class DecrementCodeGenTest implements CodeGenTest { final int value; - public DecrementCodeGenTest(int value) { + DecrementCodeGenTest(int value) { this.value = value; } diff -r 0653faea0327 -r 2160e7da7fb0 graal/com.oracle.graal.asm.amd64/src/com/oracle/graal/asm/amd64/AMD64Address.java --- a/graal/com.oracle.graal.asm.amd64/src/com/oracle/graal/asm/amd64/AMD64Address.java Tue Jan 19 10:55:27 2016 +0100 +++ b/graal/com.oracle.graal.asm.amd64/src/com/oracle/graal/asm/amd64/AMD64Address.java Tue Jan 19 12:20:24 2016 +0100 @@ -85,7 +85,7 @@ Times4(4, 2), Times8(8, 3); - private Scale(int value, int log2) { + Scale(int value, int log2) { this.value = value; this.log2 = log2; } diff -r 0653faea0327 -r 2160e7da7fb0 graal/com.oracle.graal.asm.amd64/src/com/oracle/graal/asm/amd64/AMD64Assembler.java --- a/graal/com.oracle.graal.asm.amd64/src/com/oracle/graal/asm/amd64/AMD64Assembler.java Tue Jan 19 10:55:27 2016 +0100 +++ b/graal/com.oracle.graal.asm.amd64/src/com/oracle/graal/asm/amd64/AMD64Assembler.java Tue Jan 19 12:20:24 2016 +0100 @@ -102,7 +102,7 @@ private final int value; private final String operator; - private ConditionFlag(int value, String operator) { + ConditionFlag(int value, String operator) { this.value = value; this.operator = operator; } @@ -189,7 +189,7 @@ /** * The x86 operand sizes. */ - public static enum OperandSize { + public enum OperandSize { BYTE(1) { @Override protected void emitImmediate(AMD64Assembler asm, int imm) { @@ -233,19 +233,19 @@ private final int bytes; private final boolean xmm; - private OperandSize(int bytes) { + OperandSize(int bytes) { this(bytes, 0); } - private OperandSize(int bytes, int sizePrefix) { + OperandSize(int bytes, int sizePrefix) { this(bytes, sizePrefix, false); } - private OperandSize(int bytes, boolean xmm) { + OperandSize(int bytes, boolean xmm) { this(bytes, 0, xmm); } - private OperandSize(int bytes, int sizePrefix, boolean xmm) { + OperandSize(int bytes, int sizePrefix, boolean xmm) { this.sizePrefix = sizePrefix; this.bytes = bytes; this.xmm = xmm; @@ -274,7 +274,7 @@ /** * Operand size and register type constraints. */ - private static enum OpAssertion { + private enum OpAssertion { ByteAssertion(CPU, CPU, BYTE), IntegerAssertion(CPU, CPU, WORD, DWORD, QWORD), No16BitAssertion(CPU, CPU, DWORD, QWORD), @@ -291,7 +291,7 @@ private final RegisterCategory inputCategory; private final OperandSize[] allowedSizes; - private OpAssertion(RegisterCategory resultCategory, RegisterCategory inputCategory, OperandSize... allowedSizes) { + OpAssertion(RegisterCategory resultCategory, RegisterCategory inputCategory, OperandSize... allowedSizes) { this.resultCategory = resultCategory; this.inputCategory = inputCategory; this.allowedSizes = allowedSizes; @@ -353,7 +353,7 @@ /** * Get RXB bits for register-memory instruction. The R bit extends the ModRM.reg field. There * are two cases for the memory operand:
- * ModRM.rm contains the base register: In that case, B extends the ModRM.rm field and X = 0.
+ * ModRM.rm contains the base register: In that case, B extends the ModRM.rm field and X = 0.
* There is an SIB byte: In that case, X extends SIB.index and B extends SIB.base. */ protected static int getRXB(Register reg, AMD64Address rm) { diff -r 0653faea0327 -r 2160e7da7fb0 graal/com.oracle.graal.asm.sparc/src/com/oracle/graal/asm/sparc/SPARCAssembler.java --- a/graal/com.oracle.graal.asm.sparc/src/com/oracle/graal/asm/sparc/SPARCAssembler.java Tue Jan 19 10:55:27 2016 +0100 +++ b/graal/com.oracle.graal.asm.sparc/src/com/oracle/graal/asm/sparc/SPARCAssembler.java Tue Jan 19 12:20:24 2016 +0100 @@ -209,7 +209,7 @@ } } - public static enum Ops { + public enum Ops { // @formatter:off BranchOp(0b00), CallOp(0b01), @@ -219,7 +219,7 @@ private final int value; - private Ops(int value) { + Ops(int value) { this.value = value; } @@ -233,7 +233,7 @@ } } - public static enum Op2s { + public enum Op2s { // @formatter:off Illtrap(0b000), Bpr (0b011), @@ -247,7 +247,7 @@ private final int value; - private Op2s(int value) { + Op2s(int value) { this.value = value; } @@ -265,7 +265,7 @@ private static final int UNARY = 4; private static final int VOID_IN = 8; - public static enum Op3s { + public enum Op3s { // @formatter:off Add(0x00, "add", ArithOp, BINARY | COMMUTATIVE), And(0x01, "and", ArithOp, BINARY | COMMUTATIVE), @@ -380,11 +380,11 @@ private final Ops op; private final int flags; - private Op3s(int value, String name, Ops op) { + Op3s(int value, String name, Ops op) { this(value, name, op, 0); } - private Op3s(int value, String name, Ops op, int flags) { + Op3s(int value, String name, Ops op, int flags) { this.value = value; this.operator = name; this.op = op; @@ -429,7 +429,7 @@ } } - public static enum Opfs { + public enum Opfs { // @formatter:off Fmovs(0b0_0000_0001, "fmovs", Fpop1, UNARY), @@ -495,7 +495,7 @@ private final Op3s op3; private final int flags; - private Opfs(int value, String op, Op3s op3, int flags) { + Opfs(int value, String op, Op3s op3, int flags) { this.value = value; this.operator = op; this.op3 = op3; @@ -523,7 +523,7 @@ } } - public static enum OpfLow { + public enum OpfLow { Fmovscc(0b00_0001, "fmovscc", Fpop2), Fmovdcc(0b00_0010, "fmovdcc", Fpop2); @@ -531,7 +531,7 @@ private final String operator; private final Op3s op3; - private OpfLow(int value, String op, Op3s op3) { + OpfLow(int value, String op, Op3s op3) { this.value = value; this.operator = op; this.op3 = op3; @@ -579,7 +579,7 @@ private final int value; private final String operator; - private MembarMask(int value, String op) { + MembarMask(int value, String op) { this.value = value; this.operator = op; } @@ -617,7 +617,7 @@ private final String operator; private boolean isFloat; - private CC(int value, String op, boolean isFloat) { + CC(int value, String op, boolean isFloat) { this.value = value; this.operator = op; this.isFloat = isFloat; @@ -693,11 +693,11 @@ private final String operator; private boolean forCBcond = false; - private ConditionFlag(int value, String op) { + ConditionFlag(int value, String op) { this(value, op, false); } - private ConditionFlag(int value, String op, boolean cbcond) { + ConditionFlag(int value, String op, boolean cbcond) { this.value = value; this.operator = op; this.forCBcond = cbcond; @@ -803,7 +803,7 @@ private final int value; private final String operator; - private RCondition(int value, String op) { + RCondition(int value, String op) { this.value = value; this.operator = op; } @@ -836,7 +836,7 @@ private final int value; - private Asi(int value) { + Asi(int value) { this.value = value; } @@ -863,7 +863,7 @@ private final int value; - private Fcn(int value) { + Fcn(int value) { this.value = value; } diff -r 0653faea0327 -r 2160e7da7fb0 graal/com.oracle.graal.asm.sparc/src/com/oracle/graal/asm/sparc/SPARCInstructionCounter.java --- a/graal/com.oracle.graal.asm.sparc/src/com/oracle/graal/asm/sparc/SPARCInstructionCounter.java Tue Jan 19 10:55:27 2016 +0100 +++ b/graal/com.oracle.graal.asm.sparc/src/com/oracle/graal/asm/sparc/SPARCInstructionCounter.java Tue Jan 19 12:20:24 2016 +0100 @@ -29,6 +29,7 @@ public class SPARCInstructionCounter implements InstructionCounter { // Use a treemap to keep the order in the output private static final TreeMap INSTRUCTION_MATCHER = new TreeMap<>(); + static { // @formatter:off INSTRUCTION_MATCHER.put("nop", new SPARCInstructionMatch(0xFFFF_FFFF, 0x0100_0000)); @@ -37,6 +38,7 @@ INSTRUCTION_MATCHER.put("all", new SPARCInstructionMatch(0x0, 0x0)); // @formatter:on } + private final SPARCAssembler asm; public SPARCInstructionCounter(SPARCAssembler asm) { @@ -83,7 +85,7 @@ private final int[] op3b03; private final int op; - public OP3LowBitsMatcher(int op, int... op3b03) { + OP3LowBitsMatcher(int op, int... op3b03) { super(0, 0); this.op = op; this.op3b03 = op3b03; @@ -108,7 +110,7 @@ private final int mask; private final int[] patterns; - public SPARCInstructionMatch(int mask, int... patterns) { + SPARCInstructionMatch(int mask, int... patterns) { super(); this.mask = mask; this.patterns = patterns; diff -r 0653faea0327 -r 2160e7da7fb0 graal/com.oracle.graal.asm/src/com/oracle/graal/asm/Buffer.java --- a/graal/com.oracle.graal.asm/src/com/oracle/graal/asm/Buffer.java Tue Jan 19 10:55:27 2016 +0100 +++ b/graal/com.oracle.graal.asm/src/com/oracle/graal/asm/Buffer.java Tue Jan 19 12:20:24 2016 +0100 @@ -33,7 +33,7 @@ protected ByteBuffer data; - public Buffer(ByteOrder order) { + Buffer(ByteOrder order) { data = ByteBuffer.allocate(AsmOptions.InitialCodeBufferSize); data.order(order); } diff -r 0653faea0327 -r 2160e7da7fb0 graal/com.oracle.graal.code/src/com/oracle/graal/code/HexCodeFile.java --- a/graal/com.oracle.graal.code/src/com/oracle/graal/code/HexCodeFile.java Tue Jan 19 10:55:27 2016 +0100 +++ b/graal/com.oracle.graal.code/src/com/oracle/graal/code/HexCodeFile.java Tue Jan 19 12:20:24 2016 +0100 @@ -327,7 +327,7 @@ final int line; final int col; - public InputPos(int line, int col) { + InputPos(int line, int col) { this.line = line; this.col = col; } diff -r 0653faea0327 -r 2160e7da7fb0 graal/com.oracle.graal.compiler.amd64.test/src/com/oracle/graal/compiler/amd64/test/ConstantStackMoveTest.java --- a/graal/com.oracle.graal.compiler.amd64.test/src/com/oracle/graal/compiler/amd64/test/ConstantStackMoveTest.java Tue Jan 19 10:55:27 2016 +0100 +++ b/graal/com.oracle.graal.compiler.amd64.test/src/com/oracle/graal/compiler/amd64/test/ConstantStackMoveTest.java Tue Jan 19 12:20:24 2016 +0100 @@ -45,7 +45,7 @@ private static class LoadConstantStackSpec extends LIRTestSpecification { protected final Object primitive; - public LoadConstantStackSpec(Object primitive) { + LoadConstantStackSpec(Object primitive) { this.primitive = primitive; } @@ -63,7 +63,7 @@ } private static final class LoadConstantStackSpecByte extends LoadConstantStackSpec { - public LoadConstantStackSpecByte(byte primitive) { + LoadConstantStackSpecByte(byte primitive) { super(primitive); } @@ -73,7 +73,7 @@ } private static final class LoadConstantStackSpecShort extends LoadConstantStackSpec { - public LoadConstantStackSpecShort(short primitive) { + LoadConstantStackSpecShort(short primitive) { super(primitive); } @@ -83,7 +83,7 @@ } private static final class LoadConstantStackSpecInteger extends LoadConstantStackSpec { - public LoadConstantStackSpecInteger(int primitive) { + LoadConstantStackSpecInteger(int primitive) { super(primitive); } @@ -93,7 +93,7 @@ } private static final class LoadConstantStackSpecLong extends LoadConstantStackSpec { - public LoadConstantStackSpecLong(long primitive) { + LoadConstantStackSpecLong(long primitive) { super(primitive); } @@ -103,7 +103,7 @@ } private static final class LoadConstantStackSpecFloat extends LoadConstantStackSpec { - public LoadConstantStackSpecFloat(float primitive) { + LoadConstantStackSpecFloat(float primitive) { super(primitive); } @@ -113,7 +113,7 @@ } private static final class LoadConstantStackSpecDouble extends LoadConstantStackSpec { - public LoadConstantStackSpecDouble(double primitive) { + LoadConstantStackSpecDouble(double primitive) { super(primitive); } diff -r 0653faea0327 -r 2160e7da7fb0 graal/com.oracle.graal.compiler.common/src/com/oracle/graal/compiler/common/LinkedIdentityHashMap.java --- a/graal/com.oracle.graal.compiler.common/src/com/oracle/graal/compiler/common/LinkedIdentityHashMap.java Tue Jan 19 10:55:27 2016 +0100 +++ b/graal/com.oracle.graal.compiler.common/src/com/oracle/graal/compiler/common/LinkedIdentityHashMap.java Tue Jan 19 12:20:24 2016 +0100 @@ -41,16 +41,16 @@ private final LinkedHashMap, V> map; - public LinkedIdentityHashMap() { + LinkedIdentityHashMap() { map = new LinkedHashMap<>(); } - public LinkedIdentityHashMap(Map m) { + LinkedIdentityHashMap(Map m) { map = new LinkedHashMap<>(m.size()); putAll(m); } - public LinkedIdentityHashMap(int expectedMaxSize) { + LinkedIdentityHashMap(int expectedMaxSize) { map = new LinkedHashMap<>(expectedMaxSize); } @@ -61,7 +61,7 @@ static final class Id { final T object; - public Id(T object) { + Id(T object) { assert object != null; this.object = object; } diff -r 0653faea0327 -r 2160e7da7fb0 graal/com.oracle.graal.compiler.common/src/com/oracle/graal/compiler/common/calc/Condition.java --- a/graal/com.oracle.graal.compiler.common/src/com/oracle/graal/compiler/common/calc/Condition.java Tue Jan 19 10:55:27 2016 +0100 +++ b/graal/com.oracle.graal.compiler.common/src/com/oracle/graal/compiler/common/calc/Condition.java Tue Jan 19 12:20:24 2016 +0100 @@ -84,7 +84,7 @@ public final String operator; - private Condition(String operator) { + Condition(String operator) { this.operator = operator; } diff -r 0653faea0327 -r 2160e7da7fb0 graal/com.oracle.graal.compiler.common/src/com/oracle/graal/compiler/common/cfg/AbstractControlFlowGraph.java --- a/graal/com.oracle.graal.compiler.common/src/com/oracle/graal/compiler/common/cfg/AbstractControlFlowGraph.java Tue Jan 19 10:55:27 2016 +0100 +++ b/graal/com.oracle.graal.compiler.common/src/com/oracle/graal/compiler/common/cfg/AbstractControlFlowGraph.java Tue Jan 19 12:20:24 2016 +0100 @@ -81,7 +81,7 @@ Iterator dominated; Frame parent; - public Frame(int myNumber, T block, Iterator dominated, Frame parent) { + Frame(int myNumber, T block, Iterator dominated, Frame parent) { super(); this.myNumber = myNumber; this.maxNumber = myNumber; diff -r 0653faea0327 -r 2160e7da7fb0 graal/com.oracle.graal.compiler.match.processor/src/com/oracle/graal/compiler/match/processor/MatchProcessor.java --- a/graal/com.oracle.graal.compiler.match.processor/src/com/oracle/graal/compiler/match/processor/MatchProcessor.java Tue Jan 19 10:55:27 2016 +0100 +++ b/graal/com.oracle.graal.compiler.match.processor/src/com/oracle/graal/compiler/match/processor/MatchProcessor.java Tue Jan 19 12:20:24 2016 +0100 @@ -619,7 +619,7 @@ private final String matchPattern; private final MethodInvokerItem invoker; - public MatchRuleItem(String matchPattern, MethodInvokerItem invoker) { + MatchRuleItem(String matchPattern, MethodInvokerItem invoker) { this.matchPattern = matchPattern; this.invoker = invoker; } @@ -675,7 +675,7 @@ */ Set requiredPackages = new HashSet<>(); - public MatchRuleDescriptor(TypeElement topDeclaringType) { + MatchRuleDescriptor(TypeElement topDeclaringType) { this.topDeclaringType = topDeclaringType; } } diff -r 0653faea0327 -r 2160e7da7fb0 graal/com.oracle.graal.compiler.test/src/com/oracle/graal/compiler/test/AllocSpy.java --- a/graal/com.oracle.graal.compiler.test/src/com/oracle/graal/compiler/test/AllocSpy.java Tue Jan 19 10:55:27 2016 +0100 +++ b/graal/com.oracle.graal.compiler.test/src/com/oracle/graal/compiler/test/AllocSpy.java Tue Jan 19 12:20:24 2016 +0100 @@ -62,6 +62,7 @@ static ThreadLocal current = new ThreadLocal<>(); private static final boolean ENABLED; + static { boolean enabled = false; try { @@ -296,7 +297,7 @@ private int count; private final Object value; - public CountedValue(int count, Object value) { + CountedValue(int count, Object value) { this.count = count; this.value = value; } diff -r 0653faea0327 -r 2160e7da7fb0 graal/com.oracle.graal.compiler.test/src/com/oracle/graal/compiler/test/ConditionalEliminationTest2.java --- a/graal/com.oracle.graal.compiler.test/src/com/oracle/graal/compiler/test/ConditionalEliminationTest2.java Tue Jan 19 10:55:27 2016 +0100 +++ b/graal/com.oracle.graal.compiler.test/src/com/oracle/graal/compiler/test/ConditionalEliminationTest2.java Tue Jan 19 12:20:24 2016 +0100 @@ -47,14 +47,14 @@ final String name; - public Entry(String name) { + Entry(String name) { this.name = name; } } static class EntryWithNext extends Entry { - public EntryWithNext(String name, Entry next) { + EntryWithNext(String name, Entry next) { super(name); this.next = next; } diff -r 0653faea0327 -r 2160e7da7fb0 graal/com.oracle.graal.compiler.test/src/com/oracle/graal/compiler/test/ConstantArrayReadFoldingTest.java --- a/graal/com.oracle.graal.compiler.test/src/com/oracle/graal/compiler/test/ConstantArrayReadFoldingTest.java Tue Jan 19 10:55:27 2016 +0100 +++ b/graal/com.oracle.graal.compiler.test/src/com/oracle/graal/compiler/test/ConstantArrayReadFoldingTest.java Tue Jan 19 12:20:24 2016 +0100 @@ -38,7 +38,7 @@ public final double ceiling; public double weight; - private E(double ceiling) { + E(double ceiling) { this.ceiling = ceiling; } } diff -r 0653faea0327 -r 2160e7da7fb0 graal/com.oracle.graal.compiler.test/src/com/oracle/graal/compiler/test/CountedLoopTest.java --- a/graal/com.oracle.graal.compiler.test/src/com/oracle/graal/compiler/test/CountedLoopTest.java Tue Jan 19 10:55:27 2016 +0100 +++ b/graal/com.oracle.graal.compiler.test/src/com/oracle/graal/compiler/test/CountedLoopTest.java Tue Jan 19 12:20:24 2016 +0100 @@ -219,7 +219,7 @@ private final IVProperty property; @Input private ValueNode iv; - public IVPropertyNode(IVProperty property, ValueNode iv) { + protected IVPropertyNode(IVProperty property, ValueNode iv) { super(TYPE, iv.stamp().unrestricted()); this.property = property; this.iv = iv; diff -r 0653faea0327 -r 2160e7da7fb0 graal/com.oracle.graal.compiler.test/src/com/oracle/graal/compiler/test/FinalizableSubclassTest.java --- a/graal/com.oracle.graal.compiler.test/src/com/oracle/graal/compiler/test/FinalizableSubclassTest.java Tue Jan 19 10:55:27 2016 +0100 +++ b/graal/com.oracle.graal.compiler.test/src/com/oracle/graal/compiler/test/FinalizableSubclassTest.java Tue Jan 19 12:20:24 2016 +0100 @@ -122,7 +122,7 @@ private final String replaceTo; private HashMap> cache = new HashMap<>(); - public ClassTemplateLoader() { + ClassTemplateLoader() { loaderInstance++; replaceTo = String.format("%04d", loaderInstance); } diff -r 0653faea0327 -r 2160e7da7fb0 graal/com.oracle.graal.compiler.test/src/com/oracle/graal/compiler/test/FindUniqueConcreteMethodBugTest.java --- a/graal/com.oracle.graal.compiler.test/src/com/oracle/graal/compiler/test/FindUniqueConcreteMethodBugTest.java Tue Jan 19 10:55:27 2016 +0100 +++ b/graal/com.oracle.graal.compiler.test/src/com/oracle/graal/compiler/test/FindUniqueConcreteMethodBugTest.java Tue Jan 19 12:20:24 2016 +0100 @@ -94,7 +94,7 @@ static class PersonImpl extends AbstractPerson { public String name; - public PersonImpl(String name) { + PersonImpl(String name) { this.name = name; } @@ -106,7 +106,7 @@ static class TenantImpl extends AbstractPerson implements Tenant { public int id; - public TenantImpl(int id) { + TenantImpl(int id) { this.id = id; } diff -r 0653faea0327 -r 2160e7da7fb0 graal/com.oracle.graal.compiler.test/src/com/oracle/graal/compiler/test/MemoryScheduleTest.java --- a/graal/com.oracle.graal.compiler.test/src/com/oracle/graal/compiler/test/MemoryScheduleTest.java Tue Jan 19 10:55:27 2016 +0100 +++ b/graal/com.oracle.graal.compiler.test/src/com/oracle/graal/compiler/test/MemoryScheduleTest.java Tue Jan 19 12:20:24 2016 +0100 @@ -72,7 +72,7 @@ */ public class MemoryScheduleTest extends GraphScheduleTest { - private static enum TestMode { + private enum TestMode { WITH_FRAMESTATES, WITHOUT_FRAMESTATES, INLINED_WITHOUT_FRAMESTATES diff -r 0653faea0327 -r 2160e7da7fb0 graal/com.oracle.graal.compiler.test/src/com/oracle/graal/compiler/test/NodePosIteratorTest.java --- a/graal/com.oracle.graal.compiler.test/src/com/oracle/graal/compiler/test/NodePosIteratorTest.java Tue Jan 19 10:55:27 2016 +0100 +++ b/graal/com.oracle.graal.compiler.test/src/com/oracle/graal/compiler/test/NodePosIteratorTest.java Tue Jan 19 12:20:24 2016 +0100 @@ -50,7 +50,7 @@ @Input ConstantNode i1; @Input FloatingNode i2; - public TestNode() { + protected TestNode() { super(TYPE); } diff -r 0653faea0327 -r 2160e7da7fb0 graal/com.oracle.graal.compiler.test/src/com/oracle/graal/compiler/test/OptionsVerifierTest.java --- a/graal/com.oracle.graal.compiler.test/src/com/oracle/graal/compiler/test/OptionsVerifierTest.java Tue Jan 19 10:55:27 2016 +0100 +++ b/graal/com.oracle.graal.compiler.test/src/com/oracle/graal/compiler/test/OptionsVerifierTest.java Tue Jan 19 12:20:24 2016 +0100 @@ -77,7 +77,7 @@ static class Classpath implements AutoCloseable { private final Map entries = new LinkedHashMap<>(); - public Classpath() throws IOException { + Classpath() throws IOException { String[] names = (System.getProperty("sun.boot.class.path") + File.pathSeparatorChar + System.getProperty("java.class.path")).split(File.pathSeparator); for (String n : names) { File path = new File(n); diff -r 0653faea0327 -r 2160e7da7fb0 graal/com.oracle.graal.compiler.test/src/com/oracle/graal/compiler/test/backend/AllocatorTest.java --- a/graal/com.oracle.graal.compiler.test/src/com/oracle/graal/compiler/test/backend/AllocatorTest.java Tue Jan 19 10:55:27 2016 +0100 +++ b/graal/com.oracle.graal.compiler.test/src/com/oracle/graal/compiler/test/backend/AllocatorTest.java Tue Jan 19 12:20:24 2016 +0100 @@ -67,7 +67,7 @@ public int regRegMoves; public int spillMoves; - public RegisterStats(LIR lir) { + RegisterStats(LIR lir) { this.lir = lir; for (AbstractBlockBase block : lir.codeEmittingOrder()) { diff -r 0653faea0327 -r 2160e7da7fb0 graal/com.oracle.graal.compiler.test/src/com/oracle/graal/compiler/test/ea/EscapeAnalysisTest.java --- a/graal/com.oracle.graal.compiler.test/src/com/oracle/graal/compiler/test/ea/EscapeAnalysisTest.java Tue Jan 19 10:55:27 2016 +0100 +++ b/graal/com.oracle.graal.compiler.test/src/com/oracle/graal/compiler/test/ea/EscapeAnalysisTest.java Tue Jan 19 12:20:24 2016 +0100 @@ -229,7 +229,7 @@ protected Integer value; - public MyException(Integer value) { + MyException(Integer value) { super((Throwable) null); this.value = value; } diff -r 0653faea0327 -r 2160e7da7fb0 graal/com.oracle.graal.compiler.test/src/com/oracle/graal/compiler/test/inlining/InliningTest.java --- a/graal/com.oracle.graal.compiler.test/src/com/oracle/graal/compiler/test/inlining/InliningTest.java Tue Jan 19 10:55:27 2016 +0100 +++ b/graal/com.oracle.graal.compiler.test/src/com/oracle/graal/compiler/test/inlining/InliningTest.java Tue Jan 19 12:20:24 2016 +0100 @@ -329,7 +329,7 @@ protected int value; - public SuperClass(int value) { + SuperClass(int value) { this.value = value; } @@ -360,7 +360,7 @@ private static class SubClassA extends SuperClass implements SingleImplementorInterface { - public SubClassA(int value) { + SubClassA(int value) { super(value); } @@ -377,7 +377,7 @@ private static class SubClassB extends SuperClass { - public SubClassB(int value) { + SubClassB(int value) { super(value); } @@ -394,7 +394,7 @@ private static class SubClassC extends SuperClass { - public SubClassC(int value) { + SubClassC(int value) { super(value); } @@ -411,7 +411,7 @@ private static final class FinalSubClass extends SuperClass { - public FinalSubClass(int value) { + FinalSubClass(int value) { super(value); } diff -r 0653faea0327 -r 2160e7da7fb0 graal/com.oracle.graal.compiler/src/com/oracle/graal/compiler/gen/InstructionPrinter.java --- a/graal/com.oracle.graal.compiler/src/com/oracle/graal/compiler/gen/InstructionPrinter.java Tue Jan 19 10:55:27 2016 +0100 +++ b/graal/com.oracle.graal.compiler/src/com/oracle/graal/compiler/gen/InstructionPrinter.java Tue Jan 19 12:20:24 2016 +0100 @@ -69,7 +69,7 @@ final int position; final String label; - private InstructionLineColumn(int position, String label) { + InstructionLineColumn(int position, String label) { this.position = position; this.label = label; } diff -r 0653faea0327 -r 2160e7da7fb0 graal/com.oracle.graal.debug/src/com/oracle/graal/debug/DebugFilter.java --- a/graal/com.oracle.graal.debug/src/com/oracle/graal/debug/DebugFilter.java Tue Jan 19 10:55:27 2016 +0100 +++ b/graal/com.oracle.graal.debug/src/com/oracle/graal/debug/DebugFilter.java Tue Jan 19 12:20:24 2016 +0100 @@ -39,11 +39,11 @@ * {@code level = 0} means disabled and a {@code level > 0} means enabled. *

* A filter is a list of comma-separated terms of the form {@code [:]}. - * {@code } is interpreted as a glob pattern if it contains a "*" or "?" character. + * {@code } is interpreted as a glob pattern if it contains a "*" or "?" character. * Otherwise, it is interpreted as a substring. If {@code } is empty, it matches every - * scope. If {@code :} is omitted, it defaults to {@link Debug#DEFAULT_LOG_LEVEL}. The term - * {@code ~} is a shorthand for {@code :0} to disable a debug facility for a - * pattern. + * scope. If {@code : + * } is omitted, it defaults to {@link Debug#DEFAULT_LOG_LEVEL}. The term {@code ~} + * is a shorthand for {@code :0} to disable a debug facility for a pattern. *

* The resulting log level of a scope is determined by the last matching term. If no term * matches, the log level is 0 (disabled). A filter with no terms matches every scope with a log @@ -55,29 +55,29 @@ *

  • (empty string)
    * Matches any scope with log level {@link Debug#DEFAULT_LOG_LEVEL}. * - *
  • {@code :1}
    + *
  • {@code :1}
    * Matches any scope with log level 1. * - *
  • {@code *}
    + *
  • {@code *}
    * Matches any scope with log level {@link Debug#DEFAULT_LOG_LEVEL}. * - *
  • {@code CodeGen,CodeInstall}
    + *
  • {@code CodeGen,CodeInstall}
    * Matches scopes containing "CodeGen" or "CodeInstall", both with log level * {@link Debug#DEFAULT_LOG_LEVEL}. * - *
  • {@code CodeGen:2,CodeInstall:1}
    + *
  • {@code CodeGen:2,CodeInstall:1}
    * Matches scopes containing "CodeGen" with log level 2, or "CodeInstall" with log level 1. * - *
  • {@code :1,Dead:2}
    + *
  • {@code :1,Dead:2}
    * Matches scopes containing "Dead" with log level 2, and all other scopes with log level 1. * - *
  • {@code :1,Dead:0}
    + *
  • {@code :1,Dead:0}
    * Matches all scopes with log level 1, except those containing "Dead". * - *
  • {@code Code*}
    + *
  • {@code Code*}
    * Matches scopes starting with "Code" with log level {@link Debug#DEFAULT_LOG_LEVEL}. * - *
  • {@code Code,~Dead}
    + *
  • {@code Code,~Dead}
    * Matches scopes containing "Code" but not "Dead", with log level {@link Debug#DEFAULT_LOG_LEVEL}. * */ @@ -158,7 +158,7 @@ private final Pattern pattern; public final int level; - public Term(String filter, int level) { + Term(String filter, int level) { this.level = level; if (filter.isEmpty()) { this.pattern = null; diff -r 0653faea0327 -r 2160e7da7fb0 graal/com.oracle.graal.debug/src/com/oracle/graal/debug/DebugHistogram.java --- a/graal/com.oracle.graal.debug/src/com/oracle/graal/debug/DebugHistogram.java Tue Jan 19 10:55:27 2016 +0100 +++ b/graal/com.oracle.graal.debug/src/com/oracle/graal/debug/DebugHistogram.java Tue Jan 19 12:20:24 2016 +0100 @@ -46,7 +46,7 @@ * A value and a frequency. The ordering imposed by {@link #compareTo(CountedValue)} places * values with higher frequencies first. */ - public class CountedValue implements Comparable { + class CountedValue implements Comparable { private long count; private final Object value; diff -r 0653faea0327 -r 2160e7da7fb0 graal/com.oracle.graal.debug/src/com/oracle/graal/debug/internal/TimerImpl.java --- a/graal/com.oracle.graal.debug/src/com/oracle/graal/debug/internal/TimerImpl.java Tue Jan 19 10:55:27 2016 +0100 +++ b/graal/com.oracle.graal.debug/src/com/oracle/graal/debug/internal/TimerImpl.java Tue Jan 19 12:20:24 2016 +0100 @@ -44,7 +44,7 @@ static class FlatTimer extends DebugValue implements DebugTimer { private TimerImpl accm; - public FlatTimer(String name, boolean conditional) { + FlatTimer(String name, boolean conditional) { super(name + "_Flat", conditional); } @@ -115,7 +115,7 @@ private final class SystemNanosTimer extends AbstractTimer { - public SystemNanosTimer(TimerImpl timer) { + SystemNanosTimer(TimerImpl timer) { super(timer); } @@ -127,7 +127,7 @@ private final class CpuTimeTimer extends AbstractTimer { - public CpuTimeTimer(TimerImpl timer) { + CpuTimeTimer(TimerImpl timer) { super(timer); } diff -r 0653faea0327 -r 2160e7da7fb0 graal/com.oracle.graal.graph.test/src/com/oracle/graal/graph/test/NodeUsagesTests.java --- a/graal/com.oracle.graal.graph.test/src/com/oracle/graal/graph/test/NodeUsagesTests.java Tue Jan 19 10:55:27 2016 +0100 +++ b/graal/com.oracle.graal.graph.test/src/com/oracle/graal/graph/test/NodeUsagesTests.java Tue Jan 19 12:20:24 2016 +0100 @@ -53,7 +53,7 @@ @Input Def in1; @Input Def in2; - public Use(Def in0, Def in1, Def in2) { + protected Use(Def in0, Def in1, Def in2) { super(TYPE); this.in0 = in0; this.in1 = in1; diff -r 0653faea0327 -r 2160e7da7fb0 graal/com.oracle.graal.graph.test/src/com/oracle/graal/graph/test/NodeValidationChecksTest.java --- a/graal/com.oracle.graal.graph.test/src/com/oracle/graal/graph/test/NodeValidationChecksTest.java Tue Jan 19 10:55:27 2016 +0100 +++ b/graal/com.oracle.graal.graph.test/src/com/oracle/graal/graph/test/NodeValidationChecksTest.java Tue Jan 19 12:20:24 2016 +0100 @@ -39,7 +39,7 @@ @Input TestNode input; @Successor TestNode successor; - public TestNode(TestNode input, TestNode successor) { + protected TestNode(TestNode input, TestNode successor) { super(TYPE); this.input = input; this.successor = successor; diff -r 0653faea0327 -r 2160e7da7fb0 graal/com.oracle.graal.graph.test/src/com/oracle/graal/graph/test/TypedNodeIteratorTest.java --- a/graal/com.oracle.graal.graph.test/src/com/oracle/graal/graph/test/TypedNodeIteratorTest.java Tue Jan 19 10:55:27 2016 +0100 +++ b/graal/com.oracle.graal.graph.test/src/com/oracle/graal/graph/test/TypedNodeIteratorTest.java Tue Jan 19 12:20:24 2016 +0100 @@ -46,7 +46,7 @@ public static final NodeClass TYPE = NodeClass.create(TestNode.class); protected final String name; - public TestNode(String name) { + protected TestNode(String name) { super(TYPE); this.name = name; } diff -r 0653faea0327 -r 2160e7da7fb0 graal/com.oracle.graal.graph.test/src/com/oracle/graal/graph/test/TypedNodeIteratorTest2.java --- a/graal/com.oracle.graal.graph.test/src/com/oracle/graal/graph/test/TypedNodeIteratorTest2.java Tue Jan 19 10:55:27 2016 +0100 +++ b/graal/com.oracle.graal.graph.test/src/com/oracle/graal/graph/test/TypedNodeIteratorTest2.java Tue Jan 19 12:20:24 2016 +0100 @@ -42,7 +42,7 @@ public static final NodeClass TYPE = NodeClass.create(NodeA.class); protected final String name; - public NodeA(String name) { + protected NodeA(String name) { this(TYPE, name); } @@ -60,7 +60,7 @@ static class NodeB extends NodeA implements IterableNodeType { public static final NodeClass TYPE = NodeClass.create(NodeB.class); - public NodeB(String name) { + protected NodeB(String name) { this(TYPE, name); } @@ -74,7 +74,7 @@ static class NodeC extends NodeB { public static final NodeClass TYPE = NodeClass.create(NodeC.class); - public NodeC(String name) { + protected NodeC(String name) { this(TYPE, name); } @@ -88,7 +88,7 @@ static final class NodeD extends NodeC { public static final NodeClass TYPE = NodeClass.create(NodeD.class); - public NodeD(String name) { + protected NodeD(String name) { super(TYPE, name); } diff -r 0653faea0327 -r 2160e7da7fb0 graal/com.oracle.graal.graph/src/com/oracle/graal/graph/Graph.java --- a/graal/com.oracle.graal.graph/src/com/oracle/graal/graph/Graph.java Tue Jan 19 10:55:27 2016 +0100 +++ b/graal/com.oracle.graal.graph/src/com/oracle/graal/graph/Graph.java Tue Jan 19 12:20:24 2016 +0100 @@ -127,7 +127,7 @@ private final Node node; - public CacheEntry(Node node) { + CacheEntry(Node node) { assert node.getNodeClass().valueNumberable(); assert node.getNodeClass().isLeafNode(); this.node = node; @@ -747,7 +747,7 @@ public static final NodeClass TYPE = NodeClass.create(PlaceHolderNode.class); - public PlaceHolderNode() { + protected PlaceHolderNode() { super(TYPE); } @@ -1046,7 +1046,7 @@ private final Map map; - public MapReplacement(Map map) { + MapReplacement(Map map) { this.map = map; } diff -r 0653faea0327 -r 2160e7da7fb0 graal/com.oracle.graal.graph/src/com/oracle/graal/graph/GraphNodeIterator.java --- a/graal/com.oracle.graal.graph/src/com/oracle/graal/graph/GraphNodeIterator.java Tue Jan 19 10:55:27 2016 +0100 +++ b/graal/com.oracle.graal.graph/src/com/oracle/graal/graph/GraphNodeIterator.java Tue Jan 19 12:20:24 2016 +0100 @@ -32,11 +32,11 @@ private final Graph graph; private int index; - public GraphNodeIterator(Graph graph) { + GraphNodeIterator(Graph graph) { this(graph, 0); } - public GraphNodeIterator(Graph graph, int index) { + GraphNodeIterator(Graph graph, int index) { this.graph = graph; this.index = index - 1; forward(); diff -r 0653faea0327 -r 2160e7da7fb0 graal/com.oracle.graal.graph/src/com/oracle/graal/graph/NodeBitMap.java --- a/graal/com.oracle.graal.graph/src/com/oracle/graal/graph/NodeBitMap.java Tue Jan 19 10:55:27 2016 +0100 +++ b/graal/com.oracle.graal.graph/src/com/oracle/graal/graph/NodeBitMap.java Tue Jan 19 12:20:24 2016 +0100 @@ -169,7 +169,7 @@ private Iterator nodes; private Node nextNode; - public MarkedNodeIterator(NodeBitMap visited, Iterator nodes) { + MarkedNodeIterator(NodeBitMap visited, Iterator nodes) { this.visited = visited; this.nodes = nodes; forward(); diff -r 0653faea0327 -r 2160e7da7fb0 graal/com.oracle.graal.graph/src/com/oracle/graal/graph/NodeFlood.java --- a/graal/com.oracle.graal.graph/src/com/oracle/graal/graph/NodeFlood.java Tue Jan 19 10:55:27 2016 +0100 +++ b/graal/com.oracle.graal.graph/src/com/oracle/graal/graph/NodeFlood.java Tue Jan 19 12:20:24 2016 +0100 @@ -71,7 +71,7 @@ private final Queue queue; - public QueueConsumingIterator(Queue queue) { + QueueConsumingIterator(Queue queue) { this.queue = queue; } @@ -102,7 +102,7 @@ private Iterator nodes; private Node nextNode; - public UnmarkedNodeIterator(NodeBitMap visited, Iterator nodes) { + UnmarkedNodeIterator(NodeBitMap visited, Iterator nodes) { this.visited = visited; this.nodes = nodes; forward(); diff -r 0653faea0327 -r 2160e7da7fb0 graal/com.oracle.graal.graph/src/com/oracle/graal/graph/NodeIdAccessor.java --- a/graal/com.oracle.graal.graph/src/com/oracle/graal/graph/NodeIdAccessor.java Tue Jan 19 10:55:27 2016 +0100 +++ b/graal/com.oracle.graal.graph/src/com/oracle/graal/graph/NodeIdAccessor.java Tue Jan 19 12:20:24 2016 +0100 @@ -29,7 +29,7 @@ final Graph graph; final int epoch; - public NodeIdAccessor(Graph graph) { + NodeIdAccessor(Graph graph) { this.graph = graph; this.epoch = graph.compressions; } diff -r 0653faea0327 -r 2160e7da7fb0 graal/com.oracle.graal.graph/src/com/oracle/graal/graph/NodeUsageIterator.java --- a/graal/com.oracle.graal.graph/src/com/oracle/graal/graph/NodeUsageIterator.java Tue Jan 19 10:55:27 2016 +0100 +++ b/graal/com.oracle.graal.graph/src/com/oracle/graal/graph/NodeUsageIterator.java Tue Jan 19 12:20:24 2016 +0100 @@ -46,7 +46,7 @@ } } - public NodeUsageIterator(Node node) { + NodeUsageIterator(Node node) { this.node = node; advance(); } diff -r 0653faea0327 -r 2160e7da7fb0 graal/com.oracle.graal.graph/src/com/oracle/graal/graph/NodeUsageWithModCountIterator.java --- a/graal/com.oracle.graal.graph/src/com/oracle/graal/graph/NodeUsageWithModCountIterator.java Tue Jan 19 10:55:27 2016 +0100 +++ b/graal/com.oracle.graal.graph/src/com/oracle/graal/graph/NodeUsageWithModCountIterator.java Tue Jan 19 12:20:24 2016 +0100 @@ -26,7 +26,7 @@ class NodeUsageWithModCountIterator extends NodeUsageIterator { - public NodeUsageWithModCountIterator(Node n) { + NodeUsageWithModCountIterator(Node n) { super(n); } diff -r 0653faea0327 -r 2160e7da7fb0 graal/com.oracle.graal.graph/src/com/oracle/graal/graph/TypedGraphNodeIterator.java --- a/graal/com.oracle.graal.graph/src/com/oracle/graal/graph/TypedGraphNodeIterator.java Tue Jan 19 10:55:27 2016 +0100 +++ b/graal/com.oracle.graal.graph/src/com/oracle/graal/graph/TypedGraphNodeIterator.java Tue Jan 19 12:20:24 2016 +0100 @@ -35,7 +35,7 @@ private int currentIdIndex; private boolean needsForward; - public TypedGraphNodeIterator(NodeClass clazz, Graph graph) { + TypedGraphNodeIterator(NodeClass clazz, Graph graph) { this.graph = graph; ids = clazz.iterableIds(); currentIdIndex = 0; diff -r 0653faea0327 -r 2160e7da7fb0 graal/com.oracle.graal.hotspot.aarch64/src/com/oracle/graal/hotspot/aarch64/AArch64HotSpotDirectStaticCallOp.java --- a/graal/com.oracle.graal.hotspot.aarch64/src/com/oracle/graal/hotspot/aarch64/AArch64HotSpotDirectStaticCallOp.java Tue Jan 19 10:55:27 2016 +0100 +++ b/graal/com.oracle.graal.hotspot.aarch64/src/com/oracle/graal/hotspot/aarch64/AArch64HotSpotDirectStaticCallOp.java Tue Jan 19 12:20:24 2016 +0100 @@ -48,7 +48,7 @@ private final InvokeKind invokeKind; private final HotSpotVMConfig config; - public AArch64HotSpotDirectStaticCallOp(ResolvedJavaMethod target, Value result, Value[] parameters, Value[] temps, LIRFrameState state, InvokeKind invokeKind, HotSpotVMConfig config) { + AArch64HotSpotDirectStaticCallOp(ResolvedJavaMethod target, Value result, Value[] parameters, Value[] temps, LIRFrameState state, InvokeKind invokeKind, HotSpotVMConfig config) { super(TYPE, target, result, parameters, temps, state); assert invokeKind.isDirect(); this.invokeKind = invokeKind; diff -r 0653faea0327 -r 2160e7da7fb0 graal/com.oracle.graal.hotspot.aarch64/src/com/oracle/graal/hotspot/aarch64/AArch64HotSpotDirectVirtualCallOp.java --- a/graal/com.oracle.graal.hotspot.aarch64/src/com/oracle/graal/hotspot/aarch64/AArch64HotSpotDirectVirtualCallOp.java Tue Jan 19 10:55:27 2016 +0100 +++ b/graal/com.oracle.graal.hotspot.aarch64/src/com/oracle/graal/hotspot/aarch64/AArch64HotSpotDirectVirtualCallOp.java Tue Jan 19 12:20:24 2016 +0100 @@ -48,7 +48,7 @@ private final InvokeKind invokeKind; private final HotSpotVMConfig config; - public AArch64HotSpotDirectVirtualCallOp(ResolvedJavaMethod target, Value result, Value[] parameters, Value[] temps, LIRFrameState state, InvokeKind invokeKind, HotSpotVMConfig config) { + AArch64HotSpotDirectVirtualCallOp(ResolvedJavaMethod target, Value result, Value[] parameters, Value[] temps, LIRFrameState state, InvokeKind invokeKind, HotSpotVMConfig config) { super(TYPE, target, result, parameters, temps, state); assert invokeKind.isIndirect(); this.invokeKind = invokeKind; diff -r 0653faea0327 -r 2160e7da7fb0 graal/com.oracle.graal.hotspot.aarch64/src/com/oracle/graal/hotspot/aarch64/AArch64IndirectCallOp.java --- a/graal/com.oracle.graal.hotspot.aarch64/src/com/oracle/graal/hotspot/aarch64/AArch64IndirectCallOp.java Tue Jan 19 10:55:27 2016 +0100 +++ b/graal/com.oracle.graal.hotspot.aarch64/src/com/oracle/graal/hotspot/aarch64/AArch64IndirectCallOp.java Tue Jan 19 12:20:24 2016 +0100 @@ -59,7 +59,7 @@ private final HotSpotVMConfig config; - public AArch64IndirectCallOp(ResolvedJavaMethod callTarget, Value result, Value[] parameters, Value[] temps, Value metaspaceMethod, Value targetAddress, LIRFrameState state, HotSpotVMConfig config) { + AArch64IndirectCallOp(ResolvedJavaMethod callTarget, Value result, Value[] parameters, Value[] temps, Value metaspaceMethod, Value targetAddress, LIRFrameState state, HotSpotVMConfig config) { super(TYPE, callTarget, result, parameters, temps, targetAddress, state); this.metaspaceMethod = metaspaceMethod; this.config = config; diff -r 0653faea0327 -r 2160e7da7fb0 graal/com.oracle.graal.hotspot.amd64.test/src/com/oracle/graal/hotspot/amd64/test/DataPatchInConstantsTest.java --- a/graal/com.oracle.graal.hotspot.amd64.test/src/com/oracle/graal/hotspot/amd64/test/DataPatchInConstantsTest.java Tue Jan 19 10:55:27 2016 +0100 +++ b/graal/com.oracle.graal.hotspot.amd64.test/src/com/oracle/graal/hotspot/amd64/test/DataPatchInConstantsTest.java Tue Jan 19 12:20:24 2016 +0100 @@ -168,7 +168,7 @@ @Input protected ValueNode input; - public LoadThroughPatchNode(ValueNode input) { + protected LoadThroughPatchNode(ValueNode input) { super(TYPE, input.stamp()); this.input = input; } diff -r 0653faea0327 -r 2160e7da7fb0 graal/com.oracle.graal.hotspot.amd64/src/com/oracle/graal/hotspot/amd64/AMD64DeoptimizationStub.java --- a/graal/com.oracle.graal.hotspot.amd64/src/com/oracle/graal/hotspot/amd64/AMD64DeoptimizationStub.java Tue Jan 19 10:55:27 2016 +0100 +++ b/graal/com.oracle.graal.hotspot.amd64/src/com/oracle/graal/hotspot/amd64/AMD64DeoptimizationStub.java Tue Jan 19 12:20:24 2016 +0100 @@ -47,7 +47,7 @@ private RegisterConfig registerConfig; - public AMD64DeoptimizationStub(HotSpotProviders providers, TargetDescription target, HotSpotVMConfig config, HotSpotForeignCallLinkage linkage) { + AMD64DeoptimizationStub(HotSpotProviders providers, TargetDescription target, HotSpotVMConfig config, HotSpotForeignCallLinkage linkage) { super(providers, target, linkage); registerConfig = new AMD64HotSpotRegisterConfig(target.arch, config, new Register[]{rbx, rcx, rdx, rsi, rdi, r8, r9, r10, r11, r13, r14}); } diff -r 0653faea0327 -r 2160e7da7fb0 graal/com.oracle.graal.hotspot.amd64/src/com/oracle/graal/hotspot/amd64/AMD64HotSpotCRuntimeCallEpilogueOp.java --- a/graal/com.oracle.graal.hotspot.amd64/src/com/oracle/graal/hotspot/amd64/AMD64HotSpotCRuntimeCallEpilogueOp.java Tue Jan 19 10:55:27 2016 +0100 +++ b/graal/com.oracle.graal.hotspot.amd64/src/com/oracle/graal/hotspot/amd64/AMD64HotSpotCRuntimeCallEpilogueOp.java Tue Jan 19 12:20:24 2016 +0100 @@ -39,7 +39,7 @@ private final int threadLastJavaFpOffset; private final Register thread; - public AMD64HotSpotCRuntimeCallEpilogueOp(int threadLastJavaSpOffset, int threadLastJavaFpOffset, Register thread) { + AMD64HotSpotCRuntimeCallEpilogueOp(int threadLastJavaSpOffset, int threadLastJavaFpOffset, Register thread) { super(TYPE); this.threadLastJavaSpOffset = threadLastJavaSpOffset; this.threadLastJavaFpOffset = threadLastJavaFpOffset; diff -r 0653faea0327 -r 2160e7da7fb0 graal/com.oracle.graal.hotspot.amd64/src/com/oracle/graal/hotspot/amd64/AMD64HotSpotCRuntimeCallPrologueOp.java --- a/graal/com.oracle.graal.hotspot.amd64/src/com/oracle/graal/hotspot/amd64/AMD64HotSpotCRuntimeCallPrologueOp.java Tue Jan 19 10:55:27 2016 +0100 +++ b/graal/com.oracle.graal.hotspot.amd64/src/com/oracle/graal/hotspot/amd64/AMD64HotSpotCRuntimeCallPrologueOp.java Tue Jan 19 12:20:24 2016 +0100 @@ -39,7 +39,7 @@ private final int threadLastJavaSpOffset; private final Register thread; - public AMD64HotSpotCRuntimeCallPrologueOp(int threadLastJavaSpOffset, Register thread) { + AMD64HotSpotCRuntimeCallPrologueOp(int threadLastJavaSpOffset, Register thread) { super(TYPE); this.threadLastJavaSpOffset = threadLastJavaSpOffset; this.thread = thread; diff -r 0653faea0327 -r 2160e7da7fb0 graal/com.oracle.graal.hotspot.amd64/src/com/oracle/graal/hotspot/amd64/AMD64HotSpotLIRGenerator.java --- a/graal/com.oracle.graal.hotspot.amd64/src/com/oracle/graal/hotspot/amd64/AMD64HotSpotLIRGenerator.java Tue Jan 19 10:55:27 2016 +0100 +++ b/graal/com.oracle.graal.hotspot.amd64/src/com/oracle/graal/hotspot/amd64/AMD64HotSpotLIRGenerator.java Tue Jan 19 12:20:24 2016 +0100 @@ -127,7 +127,7 @@ */ final StackSlot reservedSlot; - public SaveRbp(NoOp placeholder) { + SaveRbp(NoOp placeholder) { this.placeholder = placeholder; AMD64FrameMapBuilder frameMapBuilder = (AMD64FrameMapBuilder) getResult().getFrameMapBuilder(); this.reservedSlot = frameMapBuilder.allocateRBPSpillSlot(); @@ -174,7 +174,7 @@ @Alive({OperandFlag.STACK, OperandFlag.UNINITIALIZED}) private AllocatableValue slot; - public RescueSlotDummyOp(FrameMapBuilder frameMapBuilder, LIRKind kind) { + RescueSlotDummyOp(FrameMapBuilder frameMapBuilder, LIRKind kind) { super(TYPE); slot = frameMapBuilder.allocateSpillSlot(kind); } diff -r 0653faea0327 -r 2160e7da7fb0 graal/com.oracle.graal.hotspot.amd64/src/com/oracle/graal/hotspot/amd64/AMD64HotSpotLeaveCurrentStackFrameOp.java --- a/graal/com.oracle.graal.hotspot.amd64/src/com/oracle/graal/hotspot/amd64/AMD64HotSpotLeaveCurrentStackFrameOp.java Tue Jan 19 10:55:27 2016 +0100 +++ b/graal/com.oracle.graal.hotspot.amd64/src/com/oracle/graal/hotspot/amd64/AMD64HotSpotLeaveCurrentStackFrameOp.java Tue Jan 19 12:20:24 2016 +0100 @@ -47,7 +47,7 @@ private final SaveRegistersOp saveRegisterOp; - public AMD64HotSpotLeaveCurrentStackFrameOp(SaveRegistersOp saveRegisterOp) { + AMD64HotSpotLeaveCurrentStackFrameOp(SaveRegistersOp saveRegisterOp) { super(TYPE); this.saveRegisterOp = saveRegisterOp; } diff -r 0653faea0327 -r 2160e7da7fb0 graal/com.oracle.graal.hotspot.amd64/src/com/oracle/graal/hotspot/amd64/AMD64HotSpotLeaveDeoptimizedStackFrameOp.java --- a/graal/com.oracle.graal.hotspot.amd64/src/com/oracle/graal/hotspot/amd64/AMD64HotSpotLeaveDeoptimizedStackFrameOp.java Tue Jan 19 10:55:27 2016 +0100 +++ b/graal/com.oracle.graal.hotspot.amd64/src/com/oracle/graal/hotspot/amd64/AMD64HotSpotLeaveDeoptimizedStackFrameOp.java Tue Jan 19 12:20:24 2016 +0100 @@ -43,7 +43,7 @@ @Use(REG) AllocatableValue frameSize; @Use(REG) AllocatableValue framePointer; - public AMD64HotSpotLeaveDeoptimizedStackFrameOp(AllocatableValue frameSize, AllocatableValue initialInfo) { + AMD64HotSpotLeaveDeoptimizedStackFrameOp(AllocatableValue frameSize, AllocatableValue initialInfo) { super(TYPE); this.frameSize = frameSize; this.framePointer = initialInfo; diff -r 0653faea0327 -r 2160e7da7fb0 graal/com.oracle.graal.hotspot.amd64/src/com/oracle/graal/hotspot/amd64/AMD64HotSpotRegisterAllocationConfig.java --- a/graal/com.oracle.graal.hotspot.amd64/src/com/oracle/graal/hotspot/amd64/AMD64HotSpotRegisterAllocationConfig.java Tue Jan 19 10:55:27 2016 +0100 +++ b/graal/com.oracle.graal.hotspot.amd64/src/com/oracle/graal/hotspot/amd64/AMD64HotSpotRegisterAllocationConfig.java Tue Jan 19 12:20:24 2016 +0100 @@ -80,7 +80,7 @@ }; // @formatter:on - public AMD64HotSpotRegisterAllocationConfig(RegisterConfig registerConfig) { + AMD64HotSpotRegisterAllocationConfig(RegisterConfig registerConfig) { super(registerConfig); } diff -r 0653faea0327 -r 2160e7da7fb0 graal/com.oracle.graal.hotspot.amd64/src/com/oracle/graal/hotspot/amd64/AMD64HotSpotStrategySwitchOp.java --- a/graal/com.oracle.graal.hotspot.amd64/src/com/oracle/graal/hotspot/amd64/AMD64HotSpotStrategySwitchOp.java Tue Jan 19 10:55:27 2016 +0100 +++ b/graal/com.oracle.graal.hotspot.amd64/src/com/oracle/graal/hotspot/amd64/AMD64HotSpotStrategySwitchOp.java Tue Jan 19 12:20:24 2016 +0100 @@ -39,7 +39,7 @@ final class AMD64HotSpotStrategySwitchOp extends AMD64ControlFlow.StrategySwitchOp { public static final LIRInstructionClass TYPE = LIRInstructionClass.create(AMD64HotSpotStrategySwitchOp.class); - public AMD64HotSpotStrategySwitchOp(SwitchStrategy strategy, LabelRef[] keyTargets, LabelRef defaultTarget, Value key, Value scratch) { + AMD64HotSpotStrategySwitchOp(SwitchStrategy strategy, LabelRef[] keyTargets, LabelRef defaultTarget, Value key, Value scratch) { super(TYPE, strategy, keyTargets, defaultTarget, key, scratch); } diff -r 0653faea0327 -r 2160e7da7fb0 graal/com.oracle.graal.hotspot.amd64/src/com/oracle/graal/hotspot/amd64/AMD64UncommonTrapStub.java --- a/graal/com.oracle.graal.hotspot.amd64/src/com/oracle/graal/hotspot/amd64/AMD64UncommonTrapStub.java Tue Jan 19 10:55:27 2016 +0100 +++ b/graal/com.oracle.graal.hotspot.amd64/src/com/oracle/graal/hotspot/amd64/AMD64UncommonTrapStub.java Tue Jan 19 12:20:24 2016 +0100 @@ -47,7 +47,7 @@ private RegisterConfig registerConfig; - public AMD64UncommonTrapStub(HotSpotProviders providers, TargetDescription target, HotSpotVMConfig config, HotSpotForeignCallLinkage linkage) { + AMD64UncommonTrapStub(HotSpotProviders providers, TargetDescription target, HotSpotVMConfig config, HotSpotForeignCallLinkage linkage) { super(providers, target, linkage); Register[] allocatable = new Register[]{rbx, rcx, rdx, rsi, rdi, r8, r9, r10, r11, r13, r14}; registerConfig = new AMD64HotSpotRegisterConfig(target.arch, config, allocatable); diff -r 0653faea0327 -r 2160e7da7fb0 graal/com.oracle.graal.hotspot.sparc/src/com/oracle/graal/hotspot/sparc/SPARCDeoptimizationStub.java --- a/graal/com.oracle.graal.hotspot.sparc/src/com/oracle/graal/hotspot/sparc/SPARCDeoptimizationStub.java Tue Jan 19 10:55:27 2016 +0100 +++ b/graal/com.oracle.graal.hotspot.sparc/src/com/oracle/graal/hotspot/sparc/SPARCDeoptimizationStub.java Tue Jan 19 12:20:24 2016 +0100 @@ -45,7 +45,7 @@ private RegisterConfig registerConfig; - public SPARCDeoptimizationStub(HotSpotProviders providers, TargetDescription target, HotSpotForeignCallLinkage linkage, HotSpotVMConfig config) { + SPARCDeoptimizationStub(HotSpotProviders providers, TargetDescription target, HotSpotForeignCallLinkage linkage, HotSpotVMConfig config) { super(providers, target, linkage); // This is basically the maximum we can spare. All other G and O register are used. Register[] allocatable = new Register[]{g1, g3, g4, g5, o0, o1, o2, o3, o4}; diff -r 0653faea0327 -r 2160e7da7fb0 graal/com.oracle.graal.hotspot.sparc/src/com/oracle/graal/hotspot/sparc/SPARCHotSpotBackend.java --- a/graal/com.oracle.graal.hotspot.sparc/src/com/oracle/graal/hotspot/sparc/SPARCHotSpotBackend.java Tue Jan 19 10:55:27 2016 +0100 +++ b/graal/com.oracle.graal.hotspot.sparc/src/com/oracle/graal/hotspot/sparc/SPARCHotSpotBackend.java Tue Jan 19 12:20:24 2016 +0100 @@ -111,7 +111,7 @@ private static final ConcurrentHashMap metrics = new ConcurrentHashMap<>(); private final String suffix; - public SizeEstimateStatistics(String suffix) { + SizeEstimateStatistics(String suffix) { super(); this.suffix = suffix; } diff -r 0653faea0327 -r 2160e7da7fb0 graal/com.oracle.graal.hotspot.sparc/src/com/oracle/graal/hotspot/sparc/SPARCHotSpotCRuntimeCallEpilogueOp.java --- a/graal/com.oracle.graal.hotspot.sparc/src/com/oracle/graal/hotspot/sparc/SPARCHotSpotCRuntimeCallEpilogueOp.java Tue Jan 19 10:55:27 2016 +0100 +++ b/graal/com.oracle.graal.hotspot.sparc/src/com/oracle/graal/hotspot/sparc/SPARCHotSpotCRuntimeCallEpilogueOp.java Tue Jan 19 12:20:24 2016 +0100 @@ -50,7 +50,7 @@ private final Register thread; @Use({REG, STACK}) protected Value threadTemp; - public SPARCHotSpotCRuntimeCallEpilogueOp(int threadLastJavaSpOffset, int threadLastJavaPcOffset, int threadJavaFrameAnchorFlagsOffset, Register thread, Value threadTemp) { + SPARCHotSpotCRuntimeCallEpilogueOp(int threadLastJavaSpOffset, int threadLastJavaPcOffset, int threadJavaFrameAnchorFlagsOffset, Register thread, Value threadTemp) { super(TYPE, SIZE); this.threadLastJavaSpOffset = threadLastJavaSpOffset; this.threadLastJavaPcOffset = threadLastJavaPcOffset; diff -r 0653faea0327 -r 2160e7da7fb0 graal/com.oracle.graal.hotspot.sparc/src/com/oracle/graal/hotspot/sparc/SPARCHotSpotCRuntimeCallPrologueOp.java --- a/graal/com.oracle.graal.hotspot.sparc/src/com/oracle/graal/hotspot/sparc/SPARCHotSpotCRuntimeCallPrologueOp.java Tue Jan 19 10:55:27 2016 +0100 +++ b/graal/com.oracle.graal.hotspot.sparc/src/com/oracle/graal/hotspot/sparc/SPARCHotSpotCRuntimeCallPrologueOp.java Tue Jan 19 12:20:24 2016 +0100 @@ -52,7 +52,7 @@ @Def({REG, STACK}) protected Value threadTemp; @Temp({REG}) protected AllocatableValue spScratch; - public SPARCHotSpotCRuntimeCallPrologueOp(int threadLastJavaSpOffset, Register thread, Register stackPointer, Value threadTemp, AllocatableValue spScratch) { + SPARCHotSpotCRuntimeCallPrologueOp(int threadLastJavaSpOffset, Register thread, Register stackPointer, Value threadTemp, AllocatableValue spScratch) { super(TYPE, SIZE); this.threadLastJavaSpOffset = threadLastJavaSpOffset; this.thread = thread; diff -r 0653faea0327 -r 2160e7da7fb0 graal/com.oracle.graal.hotspot.sparc/src/com/oracle/graal/hotspot/sparc/SPARCHotSpotCounterOp.java --- a/graal/com.oracle.graal.hotspot.sparc/src/com/oracle/graal/hotspot/sparc/SPARCHotSpotCounterOp.java Tue Jan 19 10:55:27 2016 +0100 +++ b/graal/com.oracle.graal.hotspot.sparc/src/com/oracle/graal/hotspot/sparc/SPARCHotSpotCounterOp.java Tue Jan 19 12:20:24 2016 +0100 @@ -117,7 +117,7 @@ private final Register countersArrayReg; private final SPARCMacroAssembler masm; - public IncrementEmitter(Register countersArrayReg, SPARCMacroAssembler masm) { + IncrementEmitter(Register countersArrayReg, SPARCMacroAssembler masm) { super(); this.countersArrayReg = countersArrayReg; this.masm = masm; diff -r 0653faea0327 -r 2160e7da7fb0 graal/com.oracle.graal.hotspot.sparc/src/com/oracle/graal/hotspot/sparc/SPARCHotSpotLeaveCurrentStackFrameOp.java --- a/graal/com.oracle.graal.hotspot.sparc/src/com/oracle/graal/hotspot/sparc/SPARCHotSpotLeaveCurrentStackFrameOp.java Tue Jan 19 10:55:27 2016 +0100 +++ b/graal/com.oracle.graal.hotspot.sparc/src/com/oracle/graal/hotspot/sparc/SPARCHotSpotLeaveCurrentStackFrameOp.java Tue Jan 19 12:20:24 2016 +0100 @@ -46,7 +46,7 @@ final class SPARCHotSpotLeaveCurrentStackFrameOp extends SPARCLIRInstruction { public static final LIRInstructionClass TYPE = LIRInstructionClass.create(SPARCHotSpotLeaveCurrentStackFrameOp.class); - public SPARCHotSpotLeaveCurrentStackFrameOp() { + SPARCHotSpotLeaveCurrentStackFrameOp() { super(TYPE); } diff -r 0653faea0327 -r 2160e7da7fb0 graal/com.oracle.graal.hotspot.sparc/src/com/oracle/graal/hotspot/sparc/SPARCHotSpotStrategySwitchOp.java --- a/graal/com.oracle.graal.hotspot.sparc/src/com/oracle/graal/hotspot/sparc/SPARCHotSpotStrategySwitchOp.java Tue Jan 19 10:55:27 2016 +0100 +++ b/graal/com.oracle.graal.hotspot.sparc/src/com/oracle/graal/hotspot/sparc/SPARCHotSpotStrategySwitchOp.java Tue Jan 19 12:20:24 2016 +0100 @@ -52,7 +52,7 @@ final class SPARCHotSpotStrategySwitchOp extends SPARCControlFlow.StrategySwitchOp { public static final LIRInstructionClass TYPE = LIRInstructionClass.create(SPARCHotSpotStrategySwitchOp.class); - public SPARCHotSpotStrategySwitchOp(Value constantTableBase, SwitchStrategy strategy, LabelRef[] keyTargets, LabelRef defaultTarget, Value key, Value scratch) { + SPARCHotSpotStrategySwitchOp(Value constantTableBase, SwitchStrategy strategy, LabelRef[] keyTargets, LabelRef defaultTarget, Value key, Value scratch) { super(TYPE, constantTableBase, strategy, keyTargets, defaultTarget, key, scratch); } diff -r 0653faea0327 -r 2160e7da7fb0 graal/com.oracle.graal.hotspot.sparc/src/com/oracle/graal/hotspot/sparc/SPARCUncommonTrapStub.java --- a/graal/com.oracle.graal.hotspot.sparc/src/com/oracle/graal/hotspot/sparc/SPARCUncommonTrapStub.java Tue Jan 19 10:55:27 2016 +0100 +++ b/graal/com.oracle.graal.hotspot.sparc/src/com/oracle/graal/hotspot/sparc/SPARCUncommonTrapStub.java Tue Jan 19 12:20:24 2016 +0100 @@ -45,7 +45,7 @@ private RegisterConfig registerConfig; - public SPARCUncommonTrapStub(HotSpotProviders providers, TargetDescription target, HotSpotForeignCallLinkage linkage, HotSpotVMConfig config) { + SPARCUncommonTrapStub(HotSpotProviders providers, TargetDescription target, HotSpotForeignCallLinkage linkage, HotSpotVMConfig config) { super(providers, target, linkage); // This is basically the maximum we can spare. All other G and O register are used. Register[] allocatable = new Register[]{g1, g3, g4, g5, o0, o1, o2, o3, o4}; diff -r 0653faea0327 -r 2160e7da7fb0 graal/com.oracle.graal.hotspot.test/src/com/oracle/graal/hotspot/test/CompressedOopTest.java --- a/graal/com.oracle.graal.hotspot.test/src/com/oracle/graal/hotspot/test/CompressedOopTest.java Tue Jan 19 10:55:27 2016 +0100 +++ b/graal/com.oracle.graal.hotspot.test/src/com/oracle/graal/hotspot/test/CompressedOopTest.java Tue Jan 19 12:20:24 2016 +0100 @@ -392,7 +392,7 @@ public Object[] a = new Object[10]; - public ArrayContainer() { + ArrayContainer() { for (int i = 0; i < 10; i++) { a[i] = new Object(); } @@ -403,7 +403,7 @@ public HashMap a = new HashMap<>(); - public HashMapContainer() { + HashMapContainer() { for (int i = 0; i < 10; i++) { a.put(new Object(), new Object()); } @@ -414,7 +414,7 @@ public char[] value = new char[5]; - public StringContainer1() { + StringContainer1() { value[0] = 'T'; value[1] = 'e'; value[2] = 's'; @@ -428,7 +428,7 @@ public char[] value = new char[6]; - public StringContainer2() { + StringContainer2() { value[0] = 'S'; value[1] = 't'; value[2] = 'r'; @@ -443,7 +443,7 @@ public final Object a = new Object(); public final Object b = new Object(); - public ConstantContainer() { + ConstantContainer() { } } diff -r 0653faea0327 -r 2160e7da7fb0 graal/com.oracle.graal.hotspot.test/src/com/oracle/graal/hotspot/test/JVMCIInfopointErrorTest.java --- a/graal/com.oracle.graal.hotspot.test/src/com/oracle/graal/hotspot/test/JVMCIInfopointErrorTest.java Tue Jan 19 10:55:27 2016 +0100 +++ b/graal/com.oracle.graal.hotspot.test/src/com/oracle/graal/hotspot/test/JVMCIInfopointErrorTest.java Tue Jan 19 12:20:24 2016 +0100 @@ -71,7 +71,7 @@ @Def({REG, STACK}) AllocatableValue value; - public ValueDef(AllocatableValue value) { + ValueDef(AllocatableValue value) { super(TYPE); this.value = value; } @@ -86,7 +86,7 @@ @Use({REG, STACK}) AllocatableValue value; - public ValueUse(AllocatableValue value) { + ValueUse(AllocatableValue value) { super(TYPE); this.value = value; } @@ -102,7 +102,7 @@ private final TestSpec spec; - public TestNode(TestSpec spec) { + protected TestNode(TestSpec spec) { super(TYPE, StampFactory.forVoid()); this.spec = spec; } diff -r 0653faea0327 -r 2160e7da7fb0 graal/com.oracle.graal.hotspot.test/src/com/oracle/graal/hotspot/test/MemoryUsageBenchmark.java --- a/graal/com.oracle.graal.hotspot.test/src/com/oracle/graal/hotspot/test/MemoryUsageBenchmark.java Tue Jan 19 10:55:27 2016 +0100 +++ b/graal/com.oracle.graal.hotspot.test/src/com/oracle/graal/hotspot/test/MemoryUsageBenchmark.java Tue Jan 19 12:20:24 2016 +0100 @@ -105,7 +105,7 @@ private final long start; private final String name; - public MemoryUsageCloseable(String name) { + MemoryUsageCloseable(String name) { this.name = name; this.start = getCurrentThreadAllocatedBytes(); } diff -r 0653faea0327 -r 2160e7da7fb0 graal/com.oracle.graal.hotspot/src/com/oracle/graal/hotspot/CompileTheWorld.java --- a/graal/com.oracle.graal.hotspot/src/com/oracle/graal/hotspot/CompileTheWorld.java Tue Jan 19 10:55:27 2016 +0100 +++ b/graal/com.oracle.graal.hotspot/src/com/oracle/graal/hotspot/CompileTheWorld.java Tue Jan 19 12:20:24 2016 +0100 @@ -299,7 +299,7 @@ abstract static class ClassPathEntry implements Closeable { final String name; - public ClassPathEntry(String name) { + ClassPathEntry(String name) { this.name = name; } @@ -329,7 +329,7 @@ private final File dir; - public DirClassPathEntry(String name) { + DirClassPathEntry(String name) { super(name); dir = new File(name); assert dir.isDirectory(); @@ -373,7 +373,7 @@ private final JarFile jarFile; - public JarClassPathEntry(String name) throws IOException { + JarClassPathEntry(String name) throws IOException { super(name); jarFile = new JarFile(name); } @@ -412,7 +412,7 @@ private final File jimage; - public ImageClassPathEntry(String name) { + ImageClassPathEntry(String name) { super(name); jimage = new File(name); assert jimage.isFile(); diff -r 0653faea0327 -r 2160e7da7fb0 graal/com.oracle.graal.hotspot/src/com/oracle/graal/hotspot/DebugValuesPrinter.java --- a/graal/com.oracle.graal.hotspot/src/com/oracle/graal/hotspot/DebugValuesPrinter.java Tue Jan 19 10:55:27 2016 +0100 +++ b/graal/com.oracle.graal.hotspot/src/com/oracle/graal/hotspot/DebugValuesPrinter.java Tue Jan 19 12:20:24 2016 +0100 @@ -153,7 +153,7 @@ final DebugValueMap map; private boolean printed; - public DebugValueScope(DebugValueScope parent, DebugValueMap map) { + DebugValueScope(DebugValueScope parent, DebugValueMap map) { this.parent = parent; this.map = map; this.level = parent == null ? 0 : parent.level + 1; diff -r 0653faea0327 -r 2160e7da7fb0 graal/com.oracle.graal.hotspot/src/com/oracle/graal/hotspot/HotSpotForeignCallLinkage.java --- a/graal/com.oracle.graal.hotspot/src/com/oracle/graal/hotspot/HotSpotForeignCallLinkage.java Tue Jan 19 10:55:27 2016 +0100 +++ b/graal/com.oracle.graal.hotspot/src/com/oracle/graal/hotspot/HotSpotForeignCallLinkage.java Tue Jan 19 12:20:24 2016 +0100 @@ -39,7 +39,7 @@ * call will always destroy {@link HotSpotForeignCallLinkage#getOutgoingCallingConvention() its} * {@linkplain ForeignCallLinkage#getTemporaries() temporary} registers. */ - public enum RegisterEffect { + enum RegisterEffect { DESTROYS_REGISTERS, PRESERVES_REGISTERS } @@ -49,7 +49,7 @@ * {@code JavaFrameAnchor} prologue and epilogue is required around the call. A leaf function * does not lock, GC or throw exceptions. */ - public enum Transition { + enum Transition { /** * A call to a leaf function that is guaranteed to not use floating point registers and will * never have its caller stack inspected by the VM. That is, {@code JavaFrameAnchor} diff -r 0653faea0327 -r 2160e7da7fb0 graal/com.oracle.graal.hotspot/src/com/oracle/graal/hotspot/HotSpotInstructionProfiling.java --- a/graal/com.oracle.graal.hotspot/src/com/oracle/graal/hotspot/HotSpotInstructionProfiling.java Tue Jan 19 10:55:27 2016 +0100 +++ b/graal/com.oracle.graal.hotspot/src/com/oracle/graal/hotspot/HotSpotInstructionProfiling.java Tue Jan 19 12:20:24 2016 +0100 @@ -66,7 +66,7 @@ private final LIRInsertionBuffer buffer; private final String compilationUnitName; - public Analyzer(TargetDescription target, String compilationUnitName, LIR lir, BenchmarkCounterFactory counterFactory) { + Analyzer(TargetDescription target, String compilationUnitName, LIR lir, BenchmarkCounterFactory counterFactory) { this.target = target; this.lir = lir; this.compilationUnitName = compilationUnitName; diff -r 0653faea0327 -r 2160e7da7fb0 graal/com.oracle.graal.hotspot/src/com/oracle/graal/hotspot/debug/BenchmarkCounters.java --- a/graal/com.oracle.graal.hotspot/src/com/oracle/graal/hotspot/debug/BenchmarkCounters.java Tue Jan 19 10:55:27 2016 +0100 +++ b/graal/com.oracle.graal.hotspot/src/com/oracle/graal/hotspot/debug/BenchmarkCounters.java Tue Jan 19 12:20:24 2016 +0100 @@ -113,7 +113,7 @@ public final String group; public final AtomicLong staticCounters; - public Counter(int index, String group, AtomicLong staticCounters) { + Counter(int index, String group, AtomicLong staticCounters) { this.index = index; this.group = group; this.staticCounters = staticCounters; @@ -288,7 +288,7 @@ private final byte[][] patterns; private final int[] positions; - public CallbackOutputStream(PrintStream delegate, String... patterns) { + CallbackOutputStream(PrintStream delegate, String... patterns) { this.delegate = delegate; this.positions = new int[patterns.length]; this.patterns = new byte[patterns.length][]; diff -r 0653faea0327 -r 2160e7da7fb0 graal/com.oracle.graal.hotspot/src/com/oracle/graal/hotspot/meta/HotSpotInvocationPlugins.java --- a/graal/com.oracle.graal.hotspot/src/com/oracle/graal/hotspot/meta/HotSpotInvocationPlugins.java Tue Jan 19 10:55:27 2016 +0100 +++ b/graal/com.oracle.graal.hotspot/src/com/oracle/graal/hotspot/meta/HotSpotInvocationPlugins.java Tue Jan 19 12:20:24 2016 +0100 @@ -47,7 +47,7 @@ final class HotSpotInvocationPlugins extends InvocationPlugins { final HotSpotVMConfig config; - public HotSpotInvocationPlugins(HotSpotVMConfig config, MetaAccessProvider metaAccess) { + HotSpotInvocationPlugins(HotSpotVMConfig config, MetaAccessProvider metaAccess) { super(metaAccess); this.config = config; } diff -r 0653faea0327 -r 2160e7da7fb0 graal/com.oracle.graal.hotspot/src/com/oracle/graal/hotspot/meta/HotSpotWordOperationPlugin.java --- a/graal/com.oracle.graal.hotspot/src/com/oracle/graal/hotspot/meta/HotSpotWordOperationPlugin.java Tue Jan 19 10:55:27 2016 +0100 +++ b/graal/com.oracle.graal.hotspot/src/com/oracle/graal/hotspot/meta/HotSpotWordOperationPlugin.java Tue Jan 19 12:20:24 2016 +0100 @@ -61,7 +61,7 @@ * operations}. */ class HotSpotWordOperationPlugin extends WordOperationPlugin { - public HotSpotWordOperationPlugin(SnippetReflectionProvider snippetReflection, WordTypes wordTypes) { + HotSpotWordOperationPlugin(SnippetReflectionProvider snippetReflection, WordTypes wordTypes) { super(snippetReflection, wordTypes); } diff -r 0653faea0327 -r 2160e7da7fb0 graal/com.oracle.graal.hotspot/src/com/oracle/graal/hotspot/replacements/NewObjectSnippets.java --- a/graal/com.oracle.graal.hotspot/src/com/oracle/graal/hotspot/replacements/NewObjectSnippets.java Tue Jan 19 10:55:27 2016 +0100 +++ b/graal/com.oracle.graal.hotspot/src/com/oracle/graal/hotspot/replacements/NewObjectSnippets.java Tue Jan 19 12:20:24 2016 +0100 @@ -126,7 +126,7 @@ public static final LocationIdentity INIT_LOCATION = NamedLocationIdentity.mutable("Initialization"); - static enum ProfileMode { + enum ProfileMode { AllocatingMethods, InstanceOrArray, AllocatedTypes, diff -r 0653faea0327 -r 2160e7da7fb0 graal/com.oracle.graal.hotspot/src/com/oracle/graal/hotspot/word/HotSpotOperation.java --- a/graal/com.oracle.graal.hotspot/src/com/oracle/graal/hotspot/word/HotSpotOperation.java Tue Jan 19 10:55:27 2016 +0100 +++ b/graal/com.oracle.graal.hotspot/src/com/oracle/graal/hotspot/word/HotSpotOperation.java Tue Jan 19 12:20:24 2016 +0100 @@ -31,7 +31,7 @@ @Target(ElementType.METHOD) public @interface HotSpotOperation { - public enum HotspotOpcode { + enum HotspotOpcode { FROM_POINTER, TO_KLASS_POINTER, TO_METHOD_POINTER, diff -r 0653faea0327 -r 2160e7da7fb0 graal/com.oracle.graal.java/src/com/oracle/graal/java/BytecodeParser.java --- a/graal/com.oracle.graal.java/src/com/oracle/graal/java/BytecodeParser.java Tue Jan 19 10:55:27 2016 +0100 +++ b/graal/com.oracle.graal.java/src/com/oracle/graal/java/BytecodeParser.java Tue Jan 19 12:20:24 2016 +0100 @@ -444,7 +444,7 @@ * * @param parser the parsing context of the intrinsic */ - public IntrinsicScope(BytecodeParser parser) { + IntrinsicScope(BytecodeParser parser) { this.parser = parser; assert parser.parent == null; assert parser.bci() == 0; @@ -457,7 +457,7 @@ * @param parser the parsing context of the (non-intrinsic) method calling the intrinsic * @param args the arguments to the call */ - public IntrinsicScope(BytecodeParser parser, JavaKind[] argSlotKinds, ValueNode[] args) { + IntrinsicScope(BytecodeParser parser, JavaKind[] argSlotKinds, ValueNode[] args) { assert !parser.parsingIntrinsic(); this.parser = parser; mark = parser.getGraph().getMark(); @@ -533,7 +533,7 @@ FixedNode fixed; FrameStateBuilder state; - public Target(FixedNode fixed, FrameStateBuilder state) { + Target(FixedNode fixed, FrameStateBuilder state) { this.fixed = fixed; this.state = state; } @@ -1409,7 +1409,7 @@ final int nodeCount; final Mark mark; - public InvocationPluginAssertions(InvocationPlugin plugin, ValueNode[] args, ResolvedJavaMethod targetMethod, JavaKind resultType) { + InvocationPluginAssertions(InvocationPlugin plugin, ValueNode[] args, ResolvedJavaMethod targetMethod, JavaKind resultType) { guarantee(assertionsEnabled(), "%s should only be loaded and instantiated if assertions are enabled", getClass().getSimpleName()); this.plugin = plugin; this.targetMethod = targetMethod; @@ -3243,7 +3243,7 @@ final int blockIndex; int actualIndex; - public SuccessorInfo(int blockSuccessorIndex) { + SuccessorInfo(int blockSuccessorIndex) { this.blockIndex = blockSuccessorIndex; actualIndex = -1; } diff -r 0653faea0327 -r 2160e7da7fb0 graal/com.oracle.graal.jtt/src/com/oracle/graal/jtt/lang/ProcessEnvironment_init.java --- a/graal/com.oracle.graal.jtt/src/com/oracle/graal/jtt/lang/ProcessEnvironment_init.java Tue Jan 19 10:55:27 2016 +0100 +++ b/graal/com.oracle.graal.jtt/src/com/oracle/graal/jtt/lang/ProcessEnvironment_init.java Tue Jan 19 12:20:24 2016 +0100 @@ -55,7 +55,7 @@ private static final class StringEnvironment extends HashMap { @SuppressWarnings("unused") - public StringEnvironment(HashMap theenvironment) { + StringEnvironment(HashMap theenvironment) { } } diff -r 0653faea0327 -r 2160e7da7fb0 graal/com.oracle.graal.jtt/src/com/oracle/graal/jtt/loop/LoopNewInstance.java --- a/graal/com.oracle.graal.jtt/src/com/oracle/graal/jtt/loop/LoopNewInstance.java Tue Jan 19 10:55:27 2016 +0100 +++ b/graal/com.oracle.graal.jtt/src/com/oracle/graal/jtt/loop/LoopNewInstance.java Tue Jan 19 12:20:24 2016 +0100 @@ -48,7 +48,7 @@ private boolean exists; - public Blop() { + Blop() { if (!exists) { count++; } diff -r 0653faea0327 -r 2160e7da7fb0 graal/com.oracle.graal.jtt/src/com/oracle/graal/jtt/optimize/ConditionalElimination01.java --- a/graal/com.oracle.graal.jtt/src/com/oracle/graal/jtt/optimize/ConditionalElimination01.java Tue Jan 19 10:55:27 2016 +0100 +++ b/graal/com.oracle.graal.jtt/src/com/oracle/graal/jtt/optimize/ConditionalElimination01.java Tue Jan 19 12:20:24 2016 +0100 @@ -35,7 +35,7 @@ private static class A { - public A(int y) { + A(int y) { this.y = y; } diff -r 0653faea0327 -r 2160e7da7fb0 graal/com.oracle.graal.jtt/src/com/oracle/graal/jtt/optimize/ConditionalElimination02.java --- a/graal/com.oracle.graal.jtt/src/com/oracle/graal/jtt/optimize/ConditionalElimination02.java Tue Jan 19 10:55:27 2016 +0100 +++ b/graal/com.oracle.graal.jtt/src/com/oracle/graal/jtt/optimize/ConditionalElimination02.java Tue Jan 19 12:20:24 2016 +0100 @@ -36,7 +36,7 @@ private static class A { - public A(int y) { + A(int y) { this.y = y; } diff -r 0653faea0327 -r 2160e7da7fb0 graal/com.oracle.graal.jtt/src/com/oracle/graal/jtt/threads/Thread_isInterrupted05.java --- a/graal/com.oracle.graal.jtt/src/com/oracle/graal/jtt/threads/Thread_isInterrupted05.java Tue Jan 19 10:55:27 2016 +0100 +++ b/graal/com.oracle.graal.jtt/src/com/oracle/graal/jtt/threads/Thread_isInterrupted05.java Tue Jan 19 12:20:24 2016 +0100 @@ -48,7 +48,7 @@ Throwable throwable; - public WaitInterruptee() { + WaitInterruptee() { super("WaitInterruptee"); } diff -r 0653faea0327 -r 2160e7da7fb0 graal/com.oracle.graal.lir.aarch64/src/com/oracle/graal/lir/aarch64/AArch64ControlFlow.java --- a/graal/com.oracle.graal.lir.aarch64/src/com/oracle/graal/lir/aarch64/AArch64ControlFlow.java Tue Jan 19 10:55:27 2016 +0100 +++ b/graal/com.oracle.graal.lir.aarch64/src/com/oracle/graal/lir/aarch64/AArch64ControlFlow.java Tue Jan 19 12:20:24 2016 +0100 @@ -190,7 +190,7 @@ private final AArch64MacroAssembler masm; private final CompilationResultBuilder crb; - public SwitchClosure(CompilationResultBuilder crb, AArch64MacroAssembler masm) { + SwitchClosure(CompilationResultBuilder crb, AArch64MacroAssembler masm) { super(crb, masm, keyTargets, defaultTarget); this.masm = masm; this.crb = crb; diff -r 0653faea0327 -r 2160e7da7fb0 graal/com.oracle.graal.lir.aarch64/src/com/oracle/graal/lir/aarch64/AArch64Move.java --- a/graal/com.oracle.graal.lir.aarch64/src/com/oracle/graal/lir/aarch64/AArch64Move.java Tue Jan 19 10:55:27 2016 +0100 +++ b/graal/com.oracle.graal.lir.aarch64/src/com/oracle/graal/lir/aarch64/AArch64Move.java Tue Jan 19 12:20:24 2016 +0100 @@ -244,7 +244,7 @@ @Use({COMPOSITE}) protected AArch64AddressValue addressValue; @State protected LIRFrameState state; - public MemOp(LIRInstructionClass c, AArch64Kind kind, AArch64AddressValue address, LIRFrameState state) { + MemOp(LIRInstructionClass c, AArch64Kind kind, AArch64AddressValue address, LIRFrameState state) { super(c); this.kind = kind; this.addressValue = address; diff -r 0653faea0327 -r 2160e7da7fb0 graal/com.oracle.graal.lir.jtt/src/com/oracle/graal/lir/jtt/LIRTest.java --- a/graal/com.oracle.graal.lir.jtt/src/com/oracle/graal/lir/jtt/LIRTest.java Tue Jan 19 10:55:27 2016 +0100 +++ b/graal/com.oracle.graal.lir.jtt/src/com/oracle/graal/lir/jtt/LIRTest.java Tue Jan 19 12:20:24 2016 +0100 @@ -65,7 +65,7 @@ @Input protected NodeInputList values; public final SnippetReflectionProvider snippetReflection; - public LIRTestNode(SnippetReflectionProvider snippetReflection, JavaKind kind, ValueNode opsNode, ValueNode[] values) { + protected LIRTestNode(SnippetReflectionProvider snippetReflection, JavaKind kind, ValueNode opsNode, ValueNode[] values) { super(TYPE, StampFactory.forKind(kind)); this.opsNode = opsNode; this.values = new NodeInputList<>(this, values); @@ -107,7 +107,7 @@ @Input protected ValueNode name; public final SnippetReflectionProvider snippetReflection; - public LIRValueNode(SnippetReflectionProvider snippetReflection, JavaKind kind, ValueNode opsNode, ValueNode name) { + protected LIRValueNode(SnippetReflectionProvider snippetReflection, JavaKind kind, ValueNode opsNode, ValueNode name) { super(TYPE, StampFactory.forKind(kind)); this.opsNode = opsNode; this.name = name; diff -r 0653faea0327 -r 2160e7da7fb0 graal/com.oracle.graal.lir.sparc/src/com/oracle/graal/lir/sparc/SPARCBitManipulationOp.java --- a/graal/com.oracle.graal.lir.sparc/src/com/oracle/graal/lir/sparc/SPARCBitManipulationOp.java Tue Jan 19 10:55:27 2016 +0100 +++ b/graal/com.oracle.graal.lir.sparc/src/com/oracle/graal/lir/sparc/SPARCBitManipulationOp.java Tue Jan 19 12:20:24 2016 +0100 @@ -51,7 +51,7 @@ final SizeEstimate size; - private IntrinsicOpcode(SizeEstimate size) { + IntrinsicOpcode(SizeEstimate size) { this.size = size; } } diff -r 0653faea0327 -r 2160e7da7fb0 graal/com.oracle.graal.lir.sparc/src/com/oracle/graal/lir/sparc/SPARCLIRInstructionMixin.java --- a/graal/com.oracle.graal.lir.sparc/src/com/oracle/graal/lir/sparc/SPARCLIRInstructionMixin.java Tue Jan 19 10:55:27 2016 +0100 +++ b/graal/com.oracle.graal.lir.sparc/src/com/oracle/graal/lir/sparc/SPARCLIRInstructionMixin.java Tue Jan 19 12:20:24 2016 +0100 @@ -39,16 +39,18 @@ * pessimistic estimate of emitted SPARC instructions and emitted bytes into the constant * section. */ - public static class SizeEstimate { + class SizeEstimate { /** * Cache the first size definition (with just 0 as constant size). */ private static final SizeEstimate[] cache = new SizeEstimate[5]; + static { for (int i = 0; i < cache.length; i++) { cache[i] = new SizeEstimate(i, 0); } } + public final int instructionSize; public final int constantSize; @@ -79,7 +81,7 @@ } } - public static class SPARCLIRInstructionMixinStore { + class SPARCLIRInstructionMixinStore { public final SizeEstimate estimate; public SPARCDelayedControlTransfer delayedControlTransfer = SPARCDelayedControlTransfer.DUMMY; diff -r 0653faea0327 -r 2160e7da7fb0 graal/com.oracle.graal.lir.test/src/com/oracle/graal/lir/test/CompositeValueReplacementTest1.java --- a/graal/com.oracle.graal.lir.test/src/com/oracle/graal/lir/test/CompositeValueReplacementTest1.java Tue Jan 19 10:55:27 2016 +0100 +++ b/graal/com.oracle.graal.lir.test/src/com/oracle/graal/lir/test/CompositeValueReplacementTest1.java Tue Jan 19 12:20:24 2016 +0100 @@ -53,7 +53,7 @@ @Component({REG, OperandFlag.ILLEGAL}) protected Value value; - public TestCompositeValue(Value value) { + TestCompositeValue(Value value) { super(LIRKind.Illegal); this.value = value; } @@ -123,7 +123,7 @@ @Use({COMPOSITE}) protected TestCompositeValue compValue; - public TestOp(TestCompositeValue compValue) { + TestOp(TestCompositeValue compValue) { super(TYPE); this.compValue = compValue; } diff -r 0653faea0327 -r 2160e7da7fb0 graal/com.oracle.graal.lir.test/src/com/oracle/graal/lir/test/GenericValueMapTest.java --- a/graal/com.oracle.graal.lir.test/src/com/oracle/graal/lir/test/GenericValueMapTest.java Tue Jan 19 10:55:27 2016 +0100 +++ b/graal/com.oracle.graal.lir.test/src/com/oracle/graal/lir/test/GenericValueMapTest.java Tue Jan 19 12:20:24 2016 +0100 @@ -37,7 +37,7 @@ public class GenericValueMapTest { - private static enum DummyKind implements PlatformKind { + private enum DummyKind implements PlatformKind { Long; private EnumKey key = new EnumKey<>(this); diff -r 0653faea0327 -r 2160e7da7fb0 graal/com.oracle.graal.lir.test/src/com/oracle/graal/lir/test/alloc/trace/TraceGlobalMoveResolutionMappingTest.java --- a/graal/com.oracle.graal.lir.test/src/com/oracle/graal/lir/test/alloc/trace/TraceGlobalMoveResolutionMappingTest.java Tue Jan 19 10:55:27 2016 +0100 +++ b/graal/com.oracle.graal.lir.test/src/com/oracle/graal/lir/test/alloc/trace/TraceGlobalMoveResolutionMappingTest.java Tue Jan 19 12:20:24 2016 +0100 @@ -133,7 +133,7 @@ private static final Register r0 = new Register(0, 0, "r0", CPU); private static final Register r1 = new Register(1, 1, "r1", CPU); - private static enum DummyPlatformKind implements PlatformKind { + private enum DummyPlatformKind implements PlatformKind { Long; private EnumKey key = new EnumKey<>(this); diff -r 0653faea0327 -r 2160e7da7fb0 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 Tue Jan 19 10:55:27 2016 +0100 +++ b/graal/com.oracle.graal.lir/src/com/oracle/graal/lir/CompositeValueClass.java Tue Jan 19 12:20:24 2016 +0100 @@ -80,7 +80,7 @@ private static class CompositeValueFieldsScanner extends LIRFieldsScanner { - public CompositeValueFieldsScanner(FieldsScanner.CalcOffset calc) { + CompositeValueFieldsScanner(FieldsScanner.CalcOffset calc) { super(calc); valueAnnotations.put(CompositeValue.Component.class, new OperandModeAnnotation()); } diff -r 0653faea0327 -r 2160e7da7fb0 graal/com.oracle.graal.lir/src/com/oracle/graal/lir/EdgeMoveOptimizer.java --- a/graal/com.oracle.graal.lir/src/com/oracle/graal/lir/EdgeMoveOptimizer.java Tue Jan 19 10:55:27 2016 +0100 +++ b/graal/com.oracle.graal.lir/src/com/oracle/graal/lir/EdgeMoveOptimizer.java Tue Jan 19 12:20:24 2016 +0100 @@ -78,7 +78,7 @@ private final List> edgeInstructionSeqences; private LIR ir; - public Optimizer(LIR ir) { + Optimizer(LIR ir) { this.ir = ir; edgeInstructionSeqences = new ArrayList<>(4); } diff -r 0653faea0327 -r 2160e7da7fb0 graal/com.oracle.graal.lir/src/com/oracle/graal/lir/LIRInstructionClass.java --- a/graal/com.oracle.graal.lir/src/com/oracle/graal/lir/LIRInstructionClass.java Tue Jan 19 10:55:27 2016 +0100 +++ b/graal/com.oracle.graal.lir/src/com/oracle/graal/lir/LIRInstructionClass.java Tue Jan 19 12:20:24 2016 +0100 @@ -100,7 +100,7 @@ */ private FieldsScanner.FieldInfo opcodeField; - public LIRInstructionFieldsScanner(FieldsScanner.CalcOffset calc) { + LIRInstructionFieldsScanner(FieldsScanner.CalcOffset calc) { super(calc); valueAnnotations.put(LIRInstruction.Use.class, new OperandModeAnnotation()); diff -r 0653faea0327 -r 2160e7da7fb0 graal/com.oracle.graal.lir/src/com/oracle/graal/lir/LIRIntrospection.java --- a/graal/com.oracle.graal.lir/src/com/oracle/graal/lir/LIRIntrospection.java Tue Jan 19 10:55:27 2016 +0100 +++ b/graal/com.oracle.graal.lir/src/com/oracle/graal/lir/LIRIntrospection.java Tue Jan 19 12:20:24 2016 +0100 @@ -55,7 +55,7 @@ private static final Class STACK_SLOT_CLASS = StackSlot.class; private static final Class VALUE_ARRAY_CLASS = Value[].class; - public LIRIntrospection(Class clazz) { + LIRIntrospection(Class clazz) { super(clazz); } diff -r 0653faea0327 -r 2160e7da7fb0 graal/com.oracle.graal.lir/src/com/oracle/graal/lir/RedundantMoveElimination.java --- a/graal/com.oracle.graal.lir/src/com/oracle/graal/lir/RedundantMoveElimination.java Tue Jan 19 10:55:27 2016 +0100 +++ b/graal/com.oracle.graal.lir/src/com/oracle/graal/lir/RedundantMoveElimination.java Tue Jan 19 12:20:24 2016 +0100 @@ -123,7 +123,7 @@ */ static final int INIT_VALUE = 0; - public Optimization(FrameMap frameMap) { + Optimization(FrameMap frameMap) { this.frameMap = frameMap; } diff -r 0653faea0327 -r 2160e7da7fb0 graal/com.oracle.graal.lir/src/com/oracle/graal/lir/SwitchStrategy.java --- a/graal/com.oracle.graal.lir/src/com/oracle/graal/lir/SwitchStrategy.java Tue Jan 19 10:55:27 2016 +0100 +++ b/graal/com.oracle.graal.lir/src/com/oracle/graal/lir/SwitchStrategy.java Tue Jan 19 12:20:24 2016 +0100 @@ -158,7 +158,7 @@ private final int[] keyCounts = new int[keyProbabilities.length]; private final LabelRef[] keyTargets; - public EffortClosure(LabelRef[] keyTargets) { + EffortClosure(LabelRef[] keyTargets) { this.keyTargets = keyTargets; } diff -r 0653faea0327 -r 2160e7da7fb0 graal/com.oracle.graal.lir/src/com/oracle/graal/lir/alloc/lsra/Interval.java --- a/graal/com.oracle.graal.lir/src/com/oracle/graal/lir/alloc/lsra/Interval.java Tue Jan 19 10:55:27 2016 +0100 +++ b/graal/com.oracle.graal.lir/src/com/oracle/graal/lir/alloc/lsra/Interval.java Tue Jan 19 12:20:24 2016 +0100 @@ -64,7 +64,7 @@ public final Interval first; public final Interval second; - public Pair(Interval first, Interval second) { + Pair(Interval first, Interval second) { this.first = first; this.second = second; } @@ -90,7 +90,7 @@ */ public Interval stack; - public RegisterBindingLists(Interval fixed, Interval any, Interval stack) { + RegisterBindingLists(Interval fixed, Interval any, Interval stack) { this.fixed = fixed; this.any = any; this.stack = stack; diff -r 0653faea0327 -r 2160e7da7fb0 graal/com.oracle.graal.lir/src/com/oracle/graal/lir/alloc/lsra/LinearScanIntervalDumper.java --- a/graal/com.oracle.graal.lir/src/com/oracle/graal/lir/alloc/lsra/LinearScanIntervalDumper.java Tue Jan 19 10:55:27 2016 +0100 +++ b/graal/com.oracle.graal.lir/src/com/oracle/graal/lir/alloc/lsra/LinearScanIntervalDumper.java Tue Jan 19 12:20:24 2016 +0100 @@ -32,7 +32,7 @@ class LinearScanIntervalDumper implements IntervalDumper { private final Interval[] intervals; - public LinearScanIntervalDumper(Interval[] intervals) { + LinearScanIntervalDumper(Interval[] intervals) { this.intervals = intervals; } diff -r 0653faea0327 -r 2160e7da7fb0 graal/com.oracle.graal.lir/src/com/oracle/graal/lir/alloc/lsra/LinearScanOptimizeSpillPositionPhase.java --- a/graal/com.oracle.graal.lir/src/com/oracle/graal/lir/alloc/lsra/LinearScanOptimizeSpillPositionPhase.java Tue Jan 19 10:55:27 2016 +0100 +++ b/graal/com.oracle.graal.lir/src/com/oracle/graal/lir/alloc/lsra/LinearScanOptimizeSpillPositionPhase.java Tue Jan 19 12:20:24 2016 +0100 @@ -122,7 +122,7 @@ /* * The spill block is the begin of the first split child (aka the value is on the * stack). - * + * * The problem is that if spill block has more than one predecessor, the values at the * end of the predecessors might differ. Therefore, we would need a spill move in all * predecessors. To avoid this we spill in the dominator. @@ -185,7 +185,7 @@ Range range; AbstractBlockBase block; - public IntervalBlockIterator(Interval interval) { + IntervalBlockIterator(Interval interval) { range = interval.first(); block = allocator.blockForId(range.from); } diff -r 0653faea0327 -r 2160e7da7fb0 graal/com.oracle.graal.lir/src/com/oracle/graal/lir/alloc/lsra/ssi/SSILinearScanResolveDataFlowPhase.java --- a/graal/com.oracle.graal.lir/src/com/oracle/graal/lir/alloc/lsra/ssi/SSILinearScanResolveDataFlowPhase.java Tue Jan 19 10:55:27 2016 +0100 +++ b/graal/com.oracle.graal.lir/src/com/oracle/graal/lir/alloc/lsra/ssi/SSILinearScanResolveDataFlowPhase.java Tue Jan 19 12:20:24 2016 +0100 @@ -97,7 +97,7 @@ final int toId; final int fromId; - public MyPhiValueVisitor(MoveResolver moveResolver, AbstractBlockBase toBlock, AbstractBlockBase fromBlock) { + MyPhiValueVisitor(MoveResolver moveResolver, AbstractBlockBase toBlock, AbstractBlockBase fromBlock) { this.moveResolver = moveResolver; toId = allocator.getFirstLirInstructionId(toBlock); fromId = allocator.getLastLirInstructionId(fromBlock); diff -r 0653faea0327 -r 2160e7da7fb0 graal/com.oracle.graal.lir/src/com/oracle/graal/lir/alloc/trace/TraceGlobalMoveResolver.java --- a/graal/com.oracle.graal.lir/src/com/oracle/graal/lir/alloc/trace/TraceGlobalMoveResolver.java Tue Jan 19 10:55:27 2016 +0100 +++ b/graal/com.oracle.graal.lir/src/com/oracle/graal/lir/alloc/trace/TraceGlobalMoveResolver.java Tue Jan 19 12:20:24 2016 +0100 @@ -135,7 +135,7 @@ return frameMapBuilder.getRegisterConfig().getAllocatableRegisters(); } - public TraceGlobalMoveResolver(LIRGenerationResult res, MoveFactory spillMoveFactory, Architecture arch) { + TraceGlobalMoveResolver(LIRGenerationResult res, MoveFactory spillMoveFactory, Architecture arch) { this.mappingFrom = new ArrayList<>(8); this.mappingFromStack = new ArrayList<>(8); diff -r 0653faea0327 -r 2160e7da7fb0 graal/com.oracle.graal.lir/src/com/oracle/graal/lir/alloc/trace/lsra/FixedInterval.java --- a/graal/com.oracle.graal.lir/src/com/oracle/graal/lir/alloc/trace/lsra/FixedInterval.java Tue Jan 19 10:55:27 2016 +0100 +++ b/graal/com.oracle.graal.lir/src/com/oracle/graal/lir/alloc/trace/lsra/FixedInterval.java Tue Jan 19 12:20:24 2016 +0100 @@ -38,7 +38,7 @@ public FixedInterval fixed; - public FixedList(FixedInterval fixed) { + FixedList(FixedInterval fixed) { this.fixed = fixed; } diff -r 0653faea0327 -r 2160e7da7fb0 graal/com.oracle.graal.lir/src/com/oracle/graal/lir/alloc/trace/lsra/TraceInterval.java --- a/graal/com.oracle.graal.lir/src/com/oracle/graal/lir/alloc/trace/lsra/TraceInterval.java Tue Jan 19 10:55:27 2016 +0100 +++ b/graal/com.oracle.graal.lir/src/com/oracle/graal/lir/alloc/trace/lsra/TraceInterval.java Tue Jan 19 12:20:24 2016 +0100 @@ -62,7 +62,7 @@ */ public TraceInterval any; - public AnyList(TraceInterval any) { + AnyList(TraceInterval any) { this.any = any; } diff -r 0653faea0327 -r 2160e7da7fb0 graal/com.oracle.graal.lir/src/com/oracle/graal/lir/alloc/trace/lsra/TraceIntervalDumper.java --- a/graal/com.oracle.graal.lir/src/com/oracle/graal/lir/alloc/trace/lsra/TraceIntervalDumper.java Tue Jan 19 10:55:27 2016 +0100 +++ b/graal/com.oracle.graal.lir/src/com/oracle/graal/lir/alloc/trace/lsra/TraceIntervalDumper.java Tue Jan 19 12:20:24 2016 +0100 @@ -32,7 +32,7 @@ private final FixedInterval[] fixedIntervals; private final TraceInterval[] intervals; - public TraceIntervalDumper(FixedInterval[] fixedIntervals, TraceInterval[] intervals) { + TraceIntervalDumper(FixedInterval[] fixedIntervals, TraceInterval[] intervals) { this.fixedIntervals = fixedIntervals; this.intervals = intervals; } diff -r 0653faea0327 -r 2160e7da7fb0 graal/com.oracle.graal.lir/src/com/oracle/graal/lir/alloc/trace/lsra/TraceLinearScanResolveDataFlowPhase.java --- a/graal/com.oracle.graal.lir/src/com/oracle/graal/lir/alloc/trace/lsra/TraceLinearScanResolveDataFlowPhase.java Tue Jan 19 10:55:27 2016 +0100 +++ b/graal/com.oracle.graal.lir/src/com/oracle/graal/lir/alloc/trace/lsra/TraceLinearScanResolveDataFlowPhase.java Tue Jan 19 12:20:24 2016 +0100 @@ -199,7 +199,7 @@ final int toId; final int fromId; - public MyPhiValueVisitor(TraceLocalMoveResolver moveResolver, AbstractBlockBase toBlock, AbstractBlockBase fromBlock) { + MyPhiValueVisitor(TraceLocalMoveResolver moveResolver, AbstractBlockBase toBlock, AbstractBlockBase fromBlock) { this.moveResolver = moveResolver; toId = allocator.getFirstLirInstructionId(toBlock); fromId = allocator.getLastLirInstructionId(fromBlock); diff -r 0653faea0327 -r 2160e7da7fb0 graal/com.oracle.graal.lir/src/com/oracle/graal/lir/asm/CompilationResultBuilder.java --- a/graal/com.oracle.graal.lir/src/com/oracle/graal/lir/asm/CompilationResultBuilder.java Tue Jan 19 10:55:27 2016 +0100 +++ b/graal/com.oracle.graal.lir/src/com/oracle/graal/lir/asm/CompilationResultBuilder.java Tue Jan 19 12:20:24 2016 +0100 @@ -84,7 +84,7 @@ public final int codeOffset; public final LabelRef exceptionEdge; - public ExceptionInfo(int pcOffset, LabelRef exceptionEdge) { + ExceptionInfo(int pcOffset, LabelRef exceptionEdge) { this.codeOffset = pcOffset; this.exceptionEdge = exceptionEdge; } diff -r 0653faea0327 -r 2160e7da7fb0 graal/com.oracle.graal.lir/src/com/oracle/graal/lir/constopt/DefUseTree.java --- a/graal/com.oracle.graal.lir/src/com/oracle/graal/lir/constopt/DefUseTree.java Tue Jan 19 10:55:27 2016 +0100 +++ b/graal/com.oracle.graal.lir/src/com/oracle/graal/lir/constopt/DefUseTree.java Tue Jan 19 12:20:24 2016 +0100 @@ -42,7 +42,7 @@ private final AbstractBlockBase block; private final List uses; - public DefUseTree(LIRInstruction instruction, AbstractBlockBase block) { + DefUseTree(LIRInstruction instruction, AbstractBlockBase block) { assert instruction instanceof LoadConstantOp : "Not a LoadConstantOp: " + instruction; this.instruction = (LoadConstantOp) instruction; this.block = block; diff -r 0653faea0327 -r 2160e7da7fb0 graal/com.oracle.graal.lir/src/com/oracle/graal/lir/constopt/UseEntry.java --- a/graal/com.oracle.graal.lir/src/com/oracle/graal/lir/constopt/UseEntry.java Tue Jan 19 10:55:27 2016 +0100 +++ b/graal/com.oracle.graal.lir/src/com/oracle/graal/lir/constopt/UseEntry.java Tue Jan 19 12:20:24 2016 +0100 @@ -37,7 +37,7 @@ private final LIRInstruction instruction; private final Value value; - public UseEntry(AbstractBlockBase block, LIRInstruction instruction, Value value) { + UseEntry(AbstractBlockBase block, LIRInstruction instruction, Value value) { this.block = block; this.instruction = instruction; this.value = value; diff -r 0653faea0327 -r 2160e7da7fb0 graal/com.oracle.graal.lir/src/com/oracle/graal/lir/constopt/VariableMap.java --- a/graal/com.oracle.graal.lir/src/com/oracle/graal/lir/constopt/VariableMap.java Tue Jan 19 10:55:27 2016 +0100 +++ b/graal/com.oracle.graal.lir/src/com/oracle/graal/lir/constopt/VariableMap.java Tue Jan 19 12:20:24 2016 +0100 @@ -37,7 +37,7 @@ private final ArrayList content; - public VariableMap() { + VariableMap() { content = new ArrayList<>(); } diff -r 0653faea0327 -r 2160e7da7fb0 graal/com.oracle.graal.lir/src/com/oracle/graal/lir/dfa/MarkBasePointersPhase.java --- a/graal/com.oracle.graal.lir/src/com/oracle/graal/lir/dfa/MarkBasePointersPhase.java Tue Jan 19 10:55:27 2016 +0100 +++ b/graal/com.oracle.graal.lir/src/com/oracle/graal/lir/dfa/MarkBasePointersPhase.java Tue Jan 19 12:20:24 2016 +0100 @@ -54,7 +54,7 @@ private final IndexedValueMap variables; - public BasePointersSet() { + BasePointersSet() { variables = new IndexedValueMap(); } diff -r 0653faea0327 -r 2160e7da7fb0 graal/com.oracle.graal.lir/src/com/oracle/graal/lir/dfa/RegStackValueSet.java --- a/graal/com.oracle.graal.lir/src/com/oracle/graal/lir/dfa/RegStackValueSet.java Tue Jan 19 10:55:27 2016 +0100 +++ b/graal/com.oracle.graal.lir/src/com/oracle/graal/lir/dfa/RegStackValueSet.java Tue Jan 19 12:20:24 2016 +0100 @@ -49,7 +49,7 @@ private final IndexedValueMap stack; private Set extraStack; - public RegStackValueSet(FrameMap frameMap) { + RegStackValueSet(FrameMap frameMap) { this.frameMap = frameMap; registers = new IndexedValueMap(); stack = new IndexedValueMap(); diff -r 0653faea0327 -r 2160e7da7fb0 graal/com.oracle.graal.lir/src/com/oracle/graal/lir/dfa/UniqueWorkList.java --- a/graal/com.oracle.graal.lir/src/com/oracle/graal/lir/dfa/UniqueWorkList.java Tue Jan 19 10:55:27 2016 +0100 +++ b/graal/com.oracle.graal.lir/src/com/oracle/graal/lir/dfa/UniqueWorkList.java Tue Jan 19 12:20:24 2016 +0100 @@ -37,7 +37,7 @@ private static final long serialVersionUID = 8009554570990975712L; BitSet valid; - public UniqueWorkList(int size) { + UniqueWorkList(int size) { this.valid = new BitSet(size); } diff -r 0653faea0327 -r 2160e7da7fb0 graal/com.oracle.graal.lir/src/com/oracle/graal/lir/gen/LIRGeneratorTool.java --- a/graal/com.oracle.graal.lir/src/com/oracle/graal/lir/gen/LIRGeneratorTool.java Tue Jan 19 10:55:27 2016 +0100 +++ b/graal/com.oracle.graal.lir/src/com/oracle/graal/lir/gen/LIRGeneratorTool.java Tue Jan 19 12:20:24 2016 +0100 @@ -80,7 +80,7 @@ LIRInstruction createLoad(AllocatableValue result, Constant input); } - public abstract class BlockScope implements AutoCloseable { + abstract class BlockScope implements AutoCloseable { public abstract AbstractBlockBase getCurrentBlock(); diff -r 0653faea0327 -r 2160e7da7fb0 graal/com.oracle.graal.lir/src/com/oracle/graal/lir/profiling/MoveProfiling.java --- a/graal/com.oracle.graal.lir/src/com/oracle/graal/lir/profiling/MoveProfiling.java Tue Jan 19 10:55:27 2016 +0100 +++ b/graal/com.oracle.graal.lir/src/com/oracle/graal/lir/profiling/MoveProfiling.java Tue Jan 19 12:20:24 2016 +0100 @@ -59,7 +59,7 @@ new Analyzer(target, lirGenRes.getLIR(), counterFactory).run(); } - private static enum MoveType { + private enum MoveType { REG2REG("Reg", "Reg"), STACK2REG("Reg", "Stack"), CONST2REG("Reg", "Const"), @@ -114,7 +114,7 @@ private final LIRInsertionBuffer buffer; private final int[] cnt; - public Analyzer(TargetDescription target, LIR lir, BenchmarkCounterFactory counterFactory) { + Analyzer(TargetDescription target, LIR lir, BenchmarkCounterFactory counterFactory) { this.target = target; this.lir = lir; this.counterFactory = counterFactory; diff -r 0653faea0327 -r 2160e7da7fb0 graal/com.oracle.graal.lir/src/com/oracle/graal/lir/stackslotalloc/LSStackSlotAllocator.java --- a/graal/com.oracle.graal.lir/src/com/oracle/graal/lir/stackslotalloc/LSStackSlotAllocator.java Tue Jan 19 10:55:27 2016 +0100 +++ b/graal/com.oracle.graal.lir/src/com/oracle/graal/lir/stackslotalloc/LSStackSlotAllocator.java Tue Jan 19 12:20:24 2016 +0100 @@ -262,7 +262,7 @@ current.setLocation(location); } - private static enum SlotSize { + private enum SlotSize { Size1, Size2, Size4, diff -r 0653faea0327 -r 2160e7da7fb0 graal/com.oracle.graal.lir/src/com/oracle/graal/lir/stackslotalloc/StackIntervalDumper.java --- a/graal/com.oracle.graal.lir/src/com/oracle/graal/lir/stackslotalloc/StackIntervalDumper.java Tue Jan 19 10:55:27 2016 +0100 +++ b/graal/com.oracle.graal.lir/src/com/oracle/graal/lir/stackslotalloc/StackIntervalDumper.java Tue Jan 19 12:20:24 2016 +0100 @@ -30,7 +30,7 @@ class StackIntervalDumper implements IntervalDumper { private final StackInterval[] intervals; - public StackIntervalDumper(StackInterval[] intervals) { + StackIntervalDumper(StackInterval[] intervals) { this.intervals = intervals; } diff -r 0653faea0327 -r 2160e7da7fb0 graal/com.oracle.graal.microbenchmarks/src/com/oracle/graal/microbenchmarks/graal/ConditionalEliminationBenchmark.java --- a/graal/com.oracle.graal.microbenchmarks/src/com/oracle/graal/microbenchmarks/graal/ConditionalEliminationBenchmark.java Tue Jan 19 10:55:27 2016 +0100 +++ b/graal/com.oracle.graal.microbenchmarks/src/com/oracle/graal/microbenchmarks/graal/ConditionalEliminationBenchmark.java Tue Jan 19 12:20:24 2016 +0100 @@ -82,13 +82,13 @@ static class Entry { final String name; - public Entry(String name) { + Entry(String name) { this.name = name; } } static class EntryWithNext extends Entry { - public EntryWithNext(String name, Entry next) { + EntryWithNext(String name, Entry next) { super(name); this.next = next; } diff -r 0653faea0327 -r 2160e7da7fb0 graal/com.oracle.graal.microbenchmarks/src/com/oracle/graal/microbenchmarks/graal/GraphCopyBenchmark.java --- a/graal/com.oracle.graal.microbenchmarks/src/com/oracle/graal/microbenchmarks/graal/GraphCopyBenchmark.java Tue Jan 19 10:55:27 2016 +0100 +++ b/graal/com.oracle.graal.microbenchmarks/src/com/oracle/graal/microbenchmarks/graal/GraphCopyBenchmark.java Tue Jan 19 12:20:24 2016 +0100 @@ -86,13 +86,13 @@ static class Entry { final String name; - public Entry(String name) { + Entry(String name) { this.name = name; } } static class EntryWithNext extends Entry { - public EntryWithNext(String name, Entry next) { + EntryWithNext(String name, Entry next) { super(name); this.next = next; } diff -r 0653faea0327 -r 2160e7da7fb0 graal/com.oracle.graal.nodes/src/com/oracle/graal/nodes/AbstractBeginNode.java --- a/graal/com.oracle.graal.nodes/src/com/oracle/graal/nodes/AbstractBeginNode.java Tue Jan 19 10:55:27 2016 +0100 +++ b/graal/com.oracle.graal.nodes/src/com/oracle/graal/nodes/AbstractBeginNode.java Tue Jan 19 12:20:24 2016 +0100 @@ -114,7 +114,7 @@ private FixedNode current; - public BlockNodeIterator(FixedNode next) { + BlockNodeIterator(FixedNode next) { this.current = next; } diff -r 0653faea0327 -r 2160e7da7fb0 graal/com.oracle.graal.nodes/src/com/oracle/graal/nodes/CallTargetNode.java --- a/graal/com.oracle.graal.nodes/src/com/oracle/graal/nodes/CallTargetNode.java Tue Jan 19 10:55:27 2016 +0100 +++ b/graal/com.oracle.graal.nodes/src/com/oracle/graal/nodes/CallTargetNode.java Tue Jan 19 12:20:24 2016 +0100 @@ -45,7 +45,7 @@ Static(true), Virtual(false); - private InvokeKind(boolean direct) { + InvokeKind(boolean direct) { this.direct = direct; } diff -r 0653faea0327 -r 2160e7da7fb0 graal/com.oracle.graal.nodes/src/com/oracle/graal/nodes/GraphEncoder.java --- a/graal/com.oracle.graal.nodes/src/com/oracle/graal/nodes/GraphEncoder.java Tue Jan 19 10:55:27 2016 +0100 +++ b/graal/com.oracle.graal.nodes/src/com/oracle/graal/nodes/GraphEncoder.java Tue Jan 19 12:20:24 2016 +0100 @@ -112,7 +112,9 @@ public static final int NULL_ORDER_ID = 0; /** The orderId of the {@link StructuredGraph#start() start node} of the encoded graph. */ public static final int START_NODE_ORDER_ID = 1; - /** The orderId of the first actual node after the {@link StructuredGraph#start() start node}. */ + /** + * The orderId of the first actual node after the {@link StructuredGraph#start() start node}. + */ public static final int FIRST_NODE_ORDER_ID = 2; /** @@ -294,7 +296,7 @@ protected final NodeMap orderIds; protected int nextOrderId; - public NodeOrder(StructuredGraph graph) { + NodeOrder(StructuredGraph graph) { this.orderIds = new NodeMap<>(graph); this.nextOrderId = START_NODE_ORDER_ID; @@ -537,7 +539,7 @@ public final F first; public final S second; - public Pair(F first, S second) { + Pair(F first, S second) { this.first = first; this.second = second; } diff -r 0653faea0327 -r 2160e7da7fb0 graal/com.oracle.graal.nodes/src/com/oracle/graal/nodes/IfNode.java --- a/graal/com.oracle.graal.nodes/src/com/oracle/graal/nodes/IfNode.java Tue Jan 19 10:55:27 2016 +0100 +++ b/graal/com.oracle.graal.nodes/src/com/oracle/graal/nodes/IfNode.java Tue Jan 19 12:20:24 2016 +0100 @@ -401,7 +401,7 @@ final ResolvedJavaType type; double probability; - public MutableProfiledType(ResolvedJavaType type, double probability) { + MutableProfiledType(ResolvedJavaType type, double probability) { this.type = type; this.probability = probability; } diff -r 0653faea0327 -r 2160e7da7fb0 graal/com.oracle.graal.nodes/src/com/oracle/graal/nodes/PhiNode.java --- a/graal/com.oracle.graal.nodes/src/com/oracle/graal/nodes/PhiNode.java Tue Jan 19 10:55:27 2016 +0100 +++ b/graal/com.oracle.graal.nodes/src/com/oracle/graal/nodes/PhiNode.java Tue Jan 19 12:20:24 2016 +0100 @@ -153,7 +153,7 @@ public static final NodeClass TYPE = NodeClass.create(MultipleValuesNode.class); - public MultipleValuesNode() { + protected MultipleValuesNode() { super(TYPE, null); } diff -r 0653faea0327 -r 2160e7da7fb0 graal/com.oracle.graal.nodes/src/com/oracle/graal/nodes/StructuredGraph.java --- a/graal/com.oracle.graal.nodes/src/com/oracle/graal/nodes/StructuredGraph.java Tue Jan 19 10:55:27 2016 +0100 +++ b/graal/com.oracle.graal.nodes/src/com/oracle/graal/nodes/StructuredGraph.java Tue Jan 19 12:20:24 2016 +0100 @@ -65,7 +65,7 @@ * framestates}. The stage of a graph progresses monotonously. * */ - public static enum GuardsStage { + public enum GuardsStage { /** * During this stage, there can be {@link FloatingNode floating} {@link DeoptimizingNode} * such as {@link GuardNode GuardNodes}. New {@link DeoptimizingNode DeoptimizingNodes} can @@ -171,7 +171,7 @@ */ private final List inlinedMethods = new ArrayList<>(); - private static enum UnsafeAccessState { + private enum UnsafeAccessState { NO_ACCESS, HAS_ACCESS, DISABLED diff -r 0653faea0327 -r 2160e7da7fb0 graal/com.oracle.graal.nodes/src/com/oracle/graal/nodes/cfg/Block.java --- a/graal/com.oracle.graal.nodes/src/com/oracle/graal/nodes/cfg/Block.java Tue Jan 19 10:55:27 2016 +0100 +++ b/graal/com.oracle.graal.nodes/src/com/oracle/graal/nodes/cfg/Block.java Tue Jan 19 12:20:24 2016 +0100 @@ -124,7 +124,7 @@ private FixedNode cur; - public NodeIterator() { + NodeIterator() { cur = getBeginNode(); } diff -r 0653faea0327 -r 2160e7da7fb0 graal/com.oracle.graal.nodes/src/com/oracle/graal/nodes/cfg/ControlFlowGraph.java --- a/graal/com.oracle.graal.nodes/src/com/oracle/graal/nodes/cfg/ControlFlowGraph.java Tue Jan 19 10:55:27 2016 +0100 +++ b/graal/com.oracle.graal.nodes/src/com/oracle/graal/nodes/cfg/ControlFlowGraph.java Tue Jan 19 12:20:24 2016 +0100 @@ -337,7 +337,7 @@ final Loop loop; final Frame parent; - public Frame(Iterator blocks, Loop loop, Frame parent) { + Frame(Iterator blocks, Loop loop, Frame parent) { this.blocks = blocks; this.loop = loop; this.parent = parent; diff -r 0653faea0327 -r 2160e7da7fb0 graal/com.oracle.graal.nodes/src/com/oracle/graal/nodes/graphbuilderconf/GraphBuilderConfiguration.java --- a/graal/com.oracle.graal.nodes/src/com/oracle/graal/nodes/graphbuilderconf/GraphBuilderConfiguration.java Tue Jan 19 10:55:27 2016 +0100 +++ b/graal/com.oracle.graal.nodes/src/com/oracle/graal/nodes/graphbuilderconf/GraphBuilderConfiguration.java Tue Jan 19 12:20:24 2016 +0100 @@ -148,7 +148,7 @@ private final boolean clearNonLiveLocals; private final Plugins plugins; - public static enum DebugInfoMode { + public enum DebugInfoMode { SafePointsOnly, /** * This mode inserts {@link SimpleInfopointNode}s in places where no safepoints would be diff -r 0653faea0327 -r 2160e7da7fb0 graal/com.oracle.graal.nodes/src/com/oracle/graal/nodes/graphbuilderconf/InlineInvokePlugin.java --- a/graal/com.oracle.graal.nodes/src/com/oracle/graal/nodes/graphbuilderconf/InlineInvokePlugin.java Tue Jan 19 10:55:27 2016 +0100 +++ b/graal/com.oracle.graal.nodes/src/com/oracle/graal/nodes/graphbuilderconf/InlineInvokePlugin.java Tue Jan 19 12:20:24 2016 +0100 @@ -39,7 +39,7 @@ /** * Result of a {@link #shouldInlineInvoke inlining decision}. */ - public static class InlineInfo { + class InlineInfo { /** * Denotes a call site that must not be inlined and should be implemented by a node that diff -r 0653faea0327 -r 2160e7da7fb0 graal/com.oracle.graal.nodes/src/com/oracle/graal/nodes/graphbuilderconf/InvocationPlugins.java --- a/graal/com.oracle.graal.nodes/src/com/oracle/graal/nodes/graphbuilderconf/InvocationPlugins.java Tue Jan 19 10:55:27 2016 +0100 +++ b/graal/com.oracle.graal.nodes/src/com/oracle/graal/nodes/graphbuilderconf/InvocationPlugins.java Tue Jan 19 12:20:24 2016 +0100 @@ -94,7 +94,7 @@ private final String name; private Class resolved; - public OptionalLazySymbol(String name) { + OptionalLazySymbol(String name) { this.name = name; } diff -r 0653faea0327 -r 2160e7da7fb0 graal/com.oracle.graal.nodes/src/com/oracle/graal/nodes/memory/HeapAccess.java --- a/graal/com.oracle.graal.nodes/src/com/oracle/graal/nodes/memory/HeapAccess.java Tue Jan 19 10:55:27 2016 +0100 +++ b/graal/com.oracle.graal.nodes/src/com/oracle/graal/nodes/memory/HeapAccess.java Tue Jan 19 12:20:24 2016 +0100 @@ -30,7 +30,7 @@ /** * The types of (write/read) barriers attached to stores. */ - public enum BarrierType { + enum BarrierType { /** * Primitive stores which do not necessitate barriers. */ diff -r 0653faea0327 -r 2160e7da7fb0 graal/com.oracle.graal.nodes/src/com/oracle/graal/nodes/memory/MemoryCheckpoint.java --- a/graal/com.oracle.graal.nodes/src/com/oracle/graal/nodes/memory/MemoryCheckpoint.java Tue Jan 19 10:55:27 2016 +0100 +++ b/graal/com.oracle.graal.nodes/src/com/oracle/graal/nodes/memory/MemoryCheckpoint.java Tue Jan 19 12:20:24 2016 +0100 @@ -61,7 +61,7 @@ } - public class TypeAssertion { + class TypeAssertion { public static boolean correctType(Node node) { return !(node instanceof MemoryCheckpoint) || (node instanceof MemoryCheckpoint.Single ^ node instanceof MemoryCheckpoint.Multi); diff -r 0653faea0327 -r 2160e7da7fb0 graal/com.oracle.graal.nodes/src/com/oracle/graal/nodes/util/GraphUtil.java --- a/graal/com.oracle.graal.nodes/src/com/oracle/graal/nodes/util/GraphUtil.java Tue Jan 19 10:55:27 2016 +0100 +++ b/graal/com.oracle.graal.nodes/src/com/oracle/graal/nodes/util/GraphUtil.java Tue Jan 19 12:20:24 2016 +0100 @@ -501,7 +501,7 @@ static class OriginalValueSearch { ValueNode result; - public OriginalValueSearch(ValueNode proxy) { + OriginalValueSearch(ValueNode proxy) { NodeWorkList worklist = proxy.graph().createNodeWorkList(); worklist.add(proxy); for (Node node : worklist) { @@ -585,7 +585,7 @@ private final ConstantReflectionProvider constantReflection; private final boolean canonicalizeReads; - public DefaultSimplifierTool(MetaAccessProvider metaAccess, ConstantReflectionProvider constantReflection, boolean canonicalizeReads) { + DefaultSimplifierTool(MetaAccessProvider metaAccess, ConstantReflectionProvider constantReflection, boolean canonicalizeReads) { this.metaAccess = metaAccess; this.constantReflection = constantReflection; this.canonicalizeReads = canonicalizeReads; diff -r 0653faea0327 -r 2160e7da7fb0 graal/com.oracle.graal.options.processor/src/com/oracle/graal/options/processor/OptionProcessor.java --- a/graal/com.oracle.graal.options.processor/src/com/oracle/graal/options/processor/OptionProcessor.java Tue Jan 19 10:55:27 2016 +0100 +++ b/graal/com.oracle.graal.options.processor/src/com/oracle/graal/options/processor/OptionProcessor.java Tue Jan 19 12:20:24 2016 +0100 @@ -286,7 +286,7 @@ final String declaringClass; final VariableElement field; - public OptionInfo(String name, String help, String type, String declaringClass, VariableElement field) { + OptionInfo(String name, String help, String type, String declaringClass, VariableElement field) { this.name = name; this.help = help; this.type = type; @@ -311,7 +311,7 @@ final List options = new ArrayList<>(); final Set originatingElements = new HashSet<>(); - public OptionsInfo(Element topDeclaringType) { + OptionsInfo(Element topDeclaringType) { this.topDeclaringType = topDeclaringType; } } diff -r 0653faea0327 -r 2160e7da7fb0 graal/com.oracle.graal.options/src/com/oracle/graal/options/OptionValue.java --- a/graal/com.oracle.graal.options/src/com/oracle/graal/options/OptionValue.java Tue Jan 19 10:55:27 2016 +0100 +++ b/graal/com.oracle.graal.options/src/com/oracle/graal/options/OptionValue.java Tue Jan 19 12:20:24 2016 +0100 @@ -348,7 +348,7 @@ private final OptionValue option; private final Object value; - public SingleOverrideScope(OptionValue option, Object value) { + SingleOverrideScope(OptionValue option, Object value) { if (option instanceof StableOptionValue) { throw new IllegalArgumentException("Cannot override stable option " + option); } @@ -388,7 +388,7 @@ final OverrideScope parent; final Map, Object> overrides; - public MultipleOverridesScope(OverrideScope parent, OptionValue option, Object value) { + MultipleOverridesScope(OverrideScope parent, OptionValue option, Object value) { this.parent = parent; this.overrides = new HashMap<>(); if (parent != null) { diff -r 0653faea0327 -r 2160e7da7fb0 graal/com.oracle.graal.phases.common/src/com/oracle/graal/phases/common/DominatorConditionalEliminationPhase.java --- a/graal/com.oracle.graal.phases.common/src/com/oracle/graal/phases/common/DominatorConditionalEliminationPhase.java Tue Jan 19 10:55:27 2016 +0100 +++ b/graal/com.oracle.graal.phases.common/src/com/oracle/graal/phases/common/DominatorConditionalEliminationPhase.java Tue Jan 19 12:20:24 2016 +0100 @@ -103,7 +103,7 @@ private final Stamp stamp; private final ValueNode guard; - public InfoElement(Stamp stamp, ValueNode guard) { + InfoElement(Stamp stamp, ValueNode guard) { this.stamp = stamp; this.guard = guard; } @@ -125,7 +125,7 @@ private static final class Info { private final ArrayList infos; - public Info() { + Info() { infos = new ArrayList<>(); } @@ -186,7 +186,7 @@ private final LogicNode condition; private final DeoptimizingGuard guard; - public PendingTest(LogicNode condition, DeoptimizingGuard guard) { + PendingTest(LogicNode condition, DeoptimizingGuard guard) { this.condition = condition; this.guard = guard; } @@ -204,7 +204,7 @@ */ private Deque pendingTests; - public Instance(StructuredGraph graph, Function> blockToNodes, Function nodeToBlock) { + Instance(StructuredGraph graph, Function> blockToNodes, Function nodeToBlock) { map = graph.createNodeMap(); loopExits = new ArrayDeque<>(); this.blockToNodes = blockToNodes; @@ -219,7 +219,7 @@ public class InstanceFrame extends LoweringPhase.Frame { List undoOperations = new ArrayList<>(); - public InstanceFrame(Block block, InstanceFrame parent) { + InstanceFrame(Block block, InstanceFrame parent) { super(block, parent); } @@ -379,7 +379,7 @@ public final F first; public final S second; - public Pair(F first, S second) { + Pair(F first, S second) { this.first = first; this.second = second; } diff -r 0653faea0327 -r 2160e7da7fb0 graal/com.oracle.graal.phases.common/src/com/oracle/graal/phases/common/GuardLoweringPhase.java --- a/graal/com.oracle.graal.phases.common/src/com/oracle/graal/phases/common/GuardLoweringPhase.java Tue Jan 19 10:55:27 2016 +0100 +++ b/graal/com.oracle.graal.phases.common/src/com/oracle/graal/phases/common/GuardLoweringPhase.java Tue Jan 19 12:20:24 2016 +0100 @@ -206,7 +206,7 @@ private final Block block; private boolean useGuardIdAsDebugId; - public LowerGuards(Block block, boolean useGuardIdAsDebugId) { + LowerGuards(Block block, boolean useGuardIdAsDebugId) { this.block = block; this.useGuardIdAsDebugId = useGuardIdAsDebugId; } diff -r 0653faea0327 -r 2160e7da7fb0 graal/com.oracle.graal.phases.common/src/com/oracle/graal/phases/common/LoweringPhase.java --- a/graal/com.oracle.graal.phases.common/src/com/oracle/graal/phases/common/LoweringPhase.java Tue Jan 19 10:55:27 2016 +0100 +++ b/graal/com.oracle.graal.phases.common/src/com/oracle/graal/phases/common/LoweringPhase.java Tue Jan 19 12:20:24 2016 +0100 @@ -85,7 +85,7 @@ public static final NodeClass TYPE = NodeClass.create(DummyGuardHandle.class); @Input(InputType.Guard) GuardingNode guard; - public DummyGuardHandle(GuardingNode guard) { + protected DummyGuardHandle(GuardingNode guard) { super(TYPE, StampFactory.forVoid()); this.guard = guard; } @@ -112,7 +112,7 @@ private AnchoringNode guardAnchor; private FixedWithNextNode lastFixedNode; - public LoweringToolImpl(PhaseContext context, AnchoringNode guardAnchor, NodeBitMap activeGuards, FixedWithNextNode lastFixedNode) { + LoweringToolImpl(PhaseContext context, AnchoringNode guardAnchor, NodeBitMap activeGuards, FixedWithNextNode lastFixedNode) { this.context = context; this.guardAnchor = guardAnchor; this.activeGuards = activeGuards; @@ -316,7 +316,7 @@ private final NodeBitMap activeGuards; private AnchoringNode anchor; - public ProcessFrame(Block block, NodeBitMap activeGuards, AnchoringNode anchor, ProcessFrame parent) { + ProcessFrame(Block block, NodeBitMap activeGuards, AnchoringNode anchor, ProcessFrame parent) { super(block, parent); this.activeGuards = activeGuards; this.anchor = anchor; @@ -460,7 +460,7 @@ * if (alwaysReachedBlock != null && alwaysReachedBlock.getDominator() == block) { * processBlock(alwaysReachedBlock); * } - * + * * // Now go for the other dominators. * for (Block dominated : block.getDominated()) { * if (dominated != alwaysReachedBlock) { diff -r 0653faea0327 -r 2160e7da7fb0 graal/com.oracle.graal.phases.common/src/com/oracle/graal/phases/common/ValueAnchorCleanupPhase.java --- a/graal/com.oracle.graal.phases.common/src/com/oracle/graal/phases/common/ValueAnchorCleanupPhase.java Tue Jan 19 10:55:27 2016 +0100 +++ b/graal/com.oracle.graal.phases.common/src/com/oracle/graal/phases/common/ValueAnchorCleanupPhase.java Tue Jan 19 12:20:24 2016 +0100 @@ -47,11 +47,11 @@ private final Set anchoredValues; - public State() { + State() { anchoredValues = Node.newSet(); } - public State(State other) { + State(State other) { anchoredValues = Node.newSet(other.anchoredValues); } @@ -71,7 +71,7 @@ private static class CleanupValueAnchorsClosure extends SinglePassNodeIterator { - public CleanupValueAnchorsClosure(StartNode start) { + CleanupValueAnchorsClosure(StartNode start) { super(start, new State()); } diff -r 0653faea0327 -r 2160e7da7fb0 graal/com.oracle.graal.phases.common/src/com/oracle/graal/phases/common/inlining/walker/ComputeInliningRelevance.java --- a/graal/com.oracle.graal.phases.common/src/com/oracle/graal/phases/common/inlining/walker/ComputeInliningRelevance.java Tue Jan 19 10:55:27 2016 +0100 +++ b/graal/com.oracle.graal.phases.common/src/com/oracle/graal/phases/common/inlining/walker/ComputeInliningRelevance.java Tue Jan 19 12:20:24 2016 +0100 @@ -165,7 +165,7 @@ */ private double scopeRelevanceWithinParent = UNINITIALIZED; - public Scope(FixedNode start, Scope parent) { + Scope(FixedNode start, Scope parent) { this.start = start; this.parent = parent; } diff -r 0653faea0327 -r 2160e7da7fb0 graal/com.oracle.graal.phases.common/src/com/oracle/graal/phases/common/instrumentation/ExtractInstrumentationPhase.java --- a/graal/com.oracle.graal.phases.common/src/com/oracle/graal/phases/common/instrumentation/ExtractInstrumentationPhase.java Tue Jan 19 10:55:27 2016 +0100 +++ b/graal/com.oracle.graal.phases.common/src/com/oracle/graal/phases/common/instrumentation/ExtractInstrumentationPhase.java Tue Jan 19 12:20:24 2016 +0100 @@ -86,7 +86,7 @@ protected ValueNode target; protected NodeBitMap nodes; - public Instrumentation(InstrumentationBeginNode begin) { + Instrumentation(InstrumentationBeginNode begin) { this.begin = begin; resolveInstrumentationNodes(); diff -r 0653faea0327 -r 2160e7da7fb0 graal/com.oracle.graal.phases.common/src/com/oracle/graal/phases/common/instrumentation/HighTierReconcileInstrumentationPhase.java --- a/graal/com.oracle.graal.phases.common/src/com/oracle/graal/phases/common/instrumentation/HighTierReconcileInstrumentationPhase.java Tue Jan 19 10:55:27 2016 +0100 +++ b/graal/com.oracle.graal.phases.common/src/com/oracle/graal/phases/common/instrumentation/HighTierReconcileInstrumentationPhase.java Tue Jan 19 12:20:24 2016 +0100 @@ -88,7 +88,7 @@ protected CommitAllocationNode commit; protected FixedWithNextNode insertingLocation; - public InstrumentationAggregation(CommitAllocationNode commit) { + InstrumentationAggregation(CommitAllocationNode commit) { this.commit = commit; this.insertingLocation = commit; } diff -r 0653faea0327 -r 2160e7da7fb0 graal/com.oracle.graal.phases/src/com/oracle/graal/phases/OptimisticOptimizations.java --- a/graal/com.oracle.graal.phases/src/com/oracle/graal/phases/OptimisticOptimizations.java Tue Jan 19 10:55:27 2016 +0100 +++ b/graal/com.oracle.graal.phases/src/com/oracle/graal/phases/OptimisticOptimizations.java Tue Jan 19 12:20:24 2016 +0100 @@ -38,7 +38,7 @@ public static final OptimisticOptimizations NONE = new OptimisticOptimizations(EnumSet.noneOf(Optimization.class)); private static final DebugMetric disabledOptimisticOptsMetric = Debug.metric("DisabledOptimisticOpts"); - public static enum Optimization { + public enum Optimization { RemoveNeverExecutedCode, UseTypeCheckedInlining, UseTypeCheckHints, diff -r 0653faea0327 -r 2160e7da7fb0 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 Tue Jan 19 10:55:27 2016 +0100 +++ b/graal/com.oracle.graal.phases/src/com/oracle/graal/phases/schedule/SchedulePhase.java Tue Jan 19 12:20:24 2016 +0100 @@ -75,7 +75,7 @@ public final class SchedulePhase extends Phase { - public static enum SchedulingStrategy { + public enum SchedulingStrategy { EARLIEST, LATEST, LATEST_OUT_OF_LOOPS, diff -r 0653faea0327 -r 2160e7da7fb0 graal/com.oracle.graal.printer/src/com/oracle/graal/printer/BinaryGraphPrinter.java --- a/graal/com.oracle.graal.printer/src/com/oracle/graal/printer/BinaryGraphPrinter.java Tue Jan 19 10:55:27 2016 +0100 +++ b/graal/com.oracle.graal.printer/src/com/oracle/graal/printer/BinaryGraphPrinter.java Tue Jan 19 12:20:24 2016 +0100 @@ -104,7 +104,7 @@ private char nextId; private static final long serialVersionUID = -2676889957907285681L; - public ConstantPool() { + ConstantPool() { super(50, 0.65f); availableIds = new LinkedList<>(); } diff -r 0653faea0327 -r 2160e7da7fb0 graal/com.oracle.graal.printer/src/com/oracle/graal/printer/CFGPrinter.java --- a/graal/com.oracle.graal.printer/src/com/oracle/graal/printer/CFGPrinter.java Tue Jan 19 10:55:27 2016 +0100 +++ b/graal/com.oracle.graal.printer/src/com/oracle/graal/printer/CFGPrinter.java Tue Jan 19 12:20:24 2016 +0100 @@ -81,7 +81,7 @@ * * @param out where the output generated via this printer shown be written */ - public CFGPrinter(OutputStream out) { + CFGPrinter(OutputStream out) { super(out); } diff -r 0653faea0327 -r 2160e7da7fb0 graal/com.oracle.graal.replacements.aarch64/src/com/oracle/graal/replacements/aarch64/AArch64FloatArithmeticSnippets.java --- a/graal/com.oracle.graal.replacements.aarch64/src/com/oracle/graal/replacements/aarch64/AArch64FloatArithmeticSnippets.java Tue Jan 19 10:55:27 2016 +0100 +++ b/graal/com.oracle.graal.replacements.aarch64/src/com/oracle/graal/replacements/aarch64/AArch64FloatArithmeticSnippets.java Tue Jan 19 12:20:24 2016 +0100 @@ -122,7 +122,7 @@ public static final NodeClass TYPE = NodeClass.create(SafeFloatRemNode.class); @SuppressWarnings("unused") - public SafeFloatRemNode(JavaKind kind, ValueNode x, ValueNode y, boolean isStrictFP) { + protected SafeFloatRemNode(JavaKind kind, ValueNode x, ValueNode y, boolean isStrictFP) { super(TYPE, ArithmeticOpTable::getRem, x, y); } diff -r 0653faea0327 -r 2160e7da7fb0 graal/com.oracle.graal.replacements.aarch64/src/com/oracle/graal/replacements/aarch64/AArch64IntegerArithmeticSnippets.java --- a/graal/com.oracle.graal.replacements.aarch64/src/com/oracle/graal/replacements/aarch64/AArch64IntegerArithmeticSnippets.java Tue Jan 19 10:55:27 2016 +0100 +++ b/graal/com.oracle.graal.replacements.aarch64/src/com/oracle/graal/replacements/aarch64/AArch64IntegerArithmeticSnippets.java Tue Jan 19 12:20:24 2016 +0100 @@ -195,7 +195,7 @@ public static final NodeClass TYPE = NodeClass.create(SafeIntegerDivNode.class); @SuppressWarnings("unused") - public SafeIntegerDivNode(JavaKind kind, ValueNode x, ValueNode y) { + protected SafeIntegerDivNode(JavaKind kind, ValueNode x, ValueNode y) { super(x, y); } } @@ -205,7 +205,7 @@ public static final NodeClass TYPE = NodeClass.create(SafeIntegerRemNode.class); @SuppressWarnings("unused") - public SafeIntegerRemNode(JavaKind kind, ValueNode x, ValueNode y) { + protected SafeIntegerRemNode(JavaKind kind, ValueNode x, ValueNode y) { super(x, y); } } @@ -215,7 +215,7 @@ public static final NodeClass TYPE = NodeClass.create(SafeUnsignedDivNode.class); @SuppressWarnings("unused") - public SafeUnsignedDivNode(JavaKind kind, ValueNode x, ValueNode y) { + protected SafeUnsignedDivNode(JavaKind kind, ValueNode x, ValueNode y) { super(x, y); } } @@ -225,7 +225,7 @@ public static final NodeClass TYPE = NodeClass.create(SafeUnsignedRemNode.class); @SuppressWarnings("unused") - public SafeUnsignedRemNode(JavaKind kind, ValueNode x, ValueNode y) { + protected SafeUnsignedRemNode(JavaKind kind, ValueNode x, ValueNode y) { super(x, y); } } diff -r 0653faea0327 -r 2160e7da7fb0 graal/com.oracle.graal.replacements.test/src/com/oracle/graal/replacements/test/DerivedOopTest.java --- a/graal/com.oracle.graal.replacements.test/src/com/oracle/graal/replacements/test/DerivedOopTest.java Tue Jan 19 10:55:27 2016 +0100 +++ b/graal/com.oracle.graal.replacements.test/src/com/oracle/graal/replacements/test/DerivedOopTest.java Tue Jan 19 12:20:24 2016 +0100 @@ -74,7 +74,7 @@ public Pointers beforeGC; public Pointers afterGC; - public Result() { + Result() { beforeGC = new Pointers(); afterGC = new Pointers(); } diff -r 0653faea0327 -r 2160e7da7fb0 graal/com.oracle.graal.replacements.test/src/com/oracle/graal/replacements/test/EdgesTest.java --- a/graal/com.oracle.graal.replacements.test/src/com/oracle/graal/replacements/test/EdgesTest.java Tue Jan 19 10:55:27 2016 +0100 +++ b/graal/com.oracle.graal.replacements.test/src/com/oracle/graal/replacements/test/EdgesTest.java Tue Jan 19 12:20:24 2016 +0100 @@ -55,7 +55,7 @@ @Input ConstantNode i1; @Input FloatingNode i2; - public TestNode() { + protected TestNode() { super(TYPE); } diff -r 0653faea0327 -r 2160e7da7fb0 graal/com.oracle.graal.replacements.test/src/com/oracle/graal/replacements/test/MonitorTest.java --- a/graal/com.oracle.graal.replacements.test/src/com/oracle/graal/replacements/test/MonitorTest.java Tue Jan 19 10:55:27 2016 +0100 +++ b/graal/com.oracle.graal.replacements.test/src/com/oracle/graal/replacements/test/MonitorTest.java Tue Jan 19 12:20:24 2016 +0100 @@ -165,11 +165,11 @@ final char[] data; - public Chars(int size) { + Chars(int size) { this.data = new char[size]; } - public Chars(char[] data) { + Chars(char[] data) { this.data = data; } } diff -r 0653faea0327 -r 2160e7da7fb0 graal/com.oracle.graal.replacements.test/src/com/oracle/graal/replacements/test/NewInstanceTest.java --- a/graal/com.oracle.graal.replacements.test/src/com/oracle/graal/replacements/test/NewInstanceTest.java Tue Jan 19 10:55:27 2016 +0100 +++ b/graal/com.oracle.graal.replacements.test/src/com/oracle/graal/replacements/test/NewInstanceTest.java Tue Jan 19 12:20:24 2016 +0100 @@ -144,7 +144,7 @@ String name = "o1"; HashMap map = new HashMap<>(); - public SomeObject() { + SomeObject() { map.put(name, this.getClass()); } diff -r 0653faea0327 -r 2160e7da7fb0 graal/com.oracle.graal.replacements.test/src/com/oracle/graal/replacements/test/SubstitutionsTest.java --- a/graal/com.oracle.graal.replacements.test/src/com/oracle/graal/replacements/test/SubstitutionsTest.java Tue Jan 19 10:55:27 2016 +0100 +++ b/graal/com.oracle.graal.replacements.test/src/com/oracle/graal/replacements/test/SubstitutionsTest.java Tue Jan 19 12:20:24 2016 +0100 @@ -55,7 +55,7 @@ static class TestMemory extends FixedWithNextNode implements MemoryNode { private static final NodeClass TYPE = NodeClass.create(TestMemory.class); - public TestMemory() { + protected TestMemory() { super(TYPE, StampFactory.forVoid()); } @@ -69,7 +69,7 @@ @Input(Memory) MemoryNode memory; - public TestGuard(ValueNode memory) { + protected TestGuard(ValueNode memory) { super(TYPE, StampFactory.forVoid()); this.memory = (MemoryNode) memory; } @@ -84,7 +84,7 @@ @Input(Guard) GuardingNode guard; - public TestValue(ValueNode guard) { + protected TestValue(ValueNode guard) { super(TYPE, StampFactory.forKind(JavaKind.Int)); this.guard = (GuardingNode) guard; } diff -r 0653faea0327 -r 2160e7da7fb0 graal/com.oracle.graal.replacements.test/src/com/oracle/graal/replacements/test/UnsafeSubstitutionsTest.java --- a/graal/com.oracle.graal.replacements.test/src/com/oracle/graal/replacements/test/UnsafeSubstitutionsTest.java Tue Jan 19 10:55:27 2016 +0100 +++ b/graal/com.oracle.graal.replacements.test/src/com/oracle/graal/replacements/test/UnsafeSubstitutionsTest.java Tue Jan 19 12:20:24 2016 +0100 @@ -356,7 +356,7 @@ final String a = "a"; final String b; - public MyObject(String b) { + MyObject(String b) { this.b = b; Thread.dumpStack(); } diff -r 0653faea0327 -r 2160e7da7fb0 graal/com.oracle.graal.replacements.verifier/src/com/oracle/graal/replacements/verifier/InjectedDependencies.java --- a/graal/com.oracle.graal.replacements.verifier/src/com/oracle/graal/replacements/verifier/InjectedDependencies.java Tue Jan 19 10:55:27 2016 +0100 +++ b/graal/com.oracle.graal.replacements.verifier/src/com/oracle/graal/replacements/verifier/InjectedDependencies.java Tue Jan 19 12:20:24 2016 +0100 @@ -85,13 +85,13 @@ private final String type; private final Dependency generateMember; - private WellKnownDependency(String expr, String type) { + WellKnownDependency(String expr, String type) { this.expr = expr; this.type = type; this.generateMember = null; } - private WellKnownDependency(Dependency generateMember) { + WellKnownDependency(Dependency generateMember) { this.expr = generateMember.name; this.type = generateMember.type; this.generateMember = generateMember; diff -r 0653faea0327 -r 2160e7da7fb0 graal/com.oracle.graal.replacements/src/com/oracle/graal/replacements/PEGraphDecoder.java --- a/graal/com.oracle.graal.replacements/src/com/oracle/graal/replacements/PEGraphDecoder.java Tue Jan 19 10:55:27 2016 +0100 +++ b/graal/com.oracle.graal.replacements/src/com/oracle/graal/replacements/PEGraphDecoder.java Tue Jan 19 12:20:24 2016 +0100 @@ -332,7 +332,7 @@ static class ExceptionPlaceholderNode extends ValueNode { public static final NodeClass TYPE = NodeClass.create(ExceptionPlaceholderNode.class); - public ExceptionPlaceholderNode() { + protected ExceptionPlaceholderNode() { super(TYPE, StampFactory.object()); } } @@ -433,7 +433,9 @@ ValueNode[] arguments = callTarget.arguments().toArray(new ValueNode[0]); FixedWithNextNode invokePredecessor = (FixedWithNextNode) invoke.asNode().predecessor(); - /* Remove invoke from graph so that invocation plugin can append nodes to the predecessor. */ + /* + * Remove invoke from graph so that invocation plugin can append nodes to the predecessor. + */ invoke.asNode().replaceAtPredecessor(null); PEMethodScope inlineScope = new PEMethodScope(methodScope.graph, methodScope, loopScope, null, targetMethod, invokeData, methodScope.inliningDepth + 1, methodScope.loopExplosionPlugin, diff -r 0653faea0327 -r 2160e7da7fb0 graal/com.oracle.graal.replacements/src/com/oracle/graal/replacements/ReplacementsImpl.java --- a/graal/com.oracle.graal.replacements/src/com/oracle/graal/replacements/ReplacementsImpl.java Tue Jan 19 10:55:27 2016 +0100 +++ b/graal/com.oracle.graal.replacements/src/com/oracle/graal/replacements/ReplacementsImpl.java Tue Jan 19 12:20:24 2016 +0100 @@ -694,7 +694,7 @@ final Class returnType; final Class[] parameters; - public JavaSignature(Class returnType, Class[] parameters) { + JavaSignature(Class returnType, Class[] parameters) { this.parameters = parameters; this.returnType = returnType; } diff -r 0653faea0327 -r 2160e7da7fb0 graal/com.oracle.graal.replacements/src/com/oracle/graal/replacements/SnippetCounterNode.java --- a/graal/com.oracle.graal.replacements/src/com/oracle/graal/replacements/SnippetCounterNode.java Tue Jan 19 10:55:27 2016 +0100 +++ b/graal/com.oracle.graal.replacements/src/com/oracle/graal/replacements/SnippetCounterNode.java Tue Jan 19 12:20:24 2016 +0100 @@ -145,7 +145,7 @@ private final SnippetInfo add = snippet(SnippetCounterSnippets.class, "add", SNIPPET_COUNTER_LOCATION); - public Templates(Providers providers, SnippetReflectionProvider snippetReflection, TargetDescription target) { + Templates(Providers providers, SnippetReflectionProvider snippetReflection, TargetDescription target) { super(providers, snippetReflection, target); } diff -r 0653faea0327 -r 2160e7da7fb0 graal/com.oracle.graal.replacements/src/com/oracle/graal/replacements/SnippetTemplate.java --- a/graal/com.oracle.graal.replacements/src/com/oracle/graal/replacements/SnippetTemplate.java Tue Jan 19 10:55:27 2016 +0100 +++ b/graal/com.oracle.graal.replacements/src/com/oracle/graal/replacements/SnippetTemplate.java Tue Jan 19 12:20:24 2016 +0100 @@ -153,7 +153,7 @@ * Lazily constructed parts of {@link SnippetInfo}. */ static class Lazy { - public Lazy(ResolvedJavaMethod method) { + Lazy(ResolvedJavaMethod method) { int count = method.getSignature().getParameterCount(false); constantParameters = new boolean[count]; varargsParameters = new boolean[count]; @@ -469,7 +469,7 @@ public static final NodeClass TYPE = NodeClass.create(VarargsPlaceholderNode.class); protected final Varargs varargs; - public VarargsPlaceholderNode(Varargs varargs, MetaAccessProvider metaAccess) { + protected VarargsPlaceholderNode(Varargs varargs, MetaAccessProvider metaAccess) { super(TYPE, StampFactory.exactNonNull(metaAccess.lookupJavaType(varargs.componentType).getArrayClass())); this.varargs = varargs; } @@ -613,7 +613,7 @@ private static final long serialVersionUID = 1L; private final int maxCacheSize; - public LRUCache(int initialCapacity, int maxCacheSize) { + LRUCache(int initialCapacity, int maxCacheSize) { super(initialCapacity, 0.75F, true); this.maxCacheSize = maxCacheSize; } @@ -1135,7 +1135,7 @@ private final LocationIdentity locationIdentity; private final MemoryNode lastLocationAccess; - public MemoryInputMap(ValueNode replacee) { + MemoryInputMap(ValueNode replacee) { if (replacee instanceof MemoryAccess) { MemoryAccess access = (MemoryAccess) replacee; locationIdentity = access.getLocationIdentity(); @@ -1169,7 +1169,7 @@ private final Map duplicates; - public MemoryOutputMap(ValueNode replacee, Map duplicates) { + MemoryOutputMap(ValueNode replacee, Map duplicates) { super(replacee); this.duplicates = duplicates; } diff -r 0653faea0327 -r 2160e7da7fb0 graal/com.oracle.graal.truffle.hotspot/src/com/oracle/graal/truffle/hotspot/HotSpotTruffleRuntime.java --- a/graal/com.oracle.graal.truffle.hotspot/src/com/oracle/graal/truffle/hotspot/HotSpotTruffleRuntime.java Tue Jan 19 10:55:27 2016 +0100 +++ b/graal/com.oracle.graal.truffle.hotspot/src/com/oracle/graal/truffle/hotspot/HotSpotTruffleRuntime.java Tue Jan 19 12:20:24 2016 +0100 @@ -110,7 +110,7 @@ static class Lazy extends BackgroundCompileQueue { private StackIntrospection stackIntrospection; - public Lazy(HotSpotTruffleRuntime runtime) { + Lazy(HotSpotTruffleRuntime runtime) { runtime.installDefaultListeners(); } diff -r 0653faea0327 -r 2160e7da7fb0 graal/com.oracle.graal.truffle.test/src/com/oracle/graal/truffle/test/InstrumentationPETestLanguage.java --- a/graal/com.oracle.graal.truffle.test/src/com/oracle/graal/truffle/test/InstrumentationPETestLanguage.java Tue Jan 19 10:55:27 2016 +0100 +++ b/graal/com.oracle.graal.truffle.test/src/com/oracle/graal/truffle/test/InstrumentationPETestLanguage.java Tue Jan 19 12:20:24 2016 +0100 @@ -50,7 +50,7 @@ public static final InstrumentationPETestLanguage INSTANCE = new InstrumentationPETestLanguage(); - static enum InstrumentTestTag implements SyntaxTag { + enum InstrumentTestTag implements SyntaxTag { ADD_TAG("addition", "test language addition node"), @@ -59,7 +59,7 @@ private final String name; private final String description; - private InstrumentTestTag(String name, String description) { + InstrumentTestTag(String name, String description) { this.name = name; this.description = description; } @@ -163,7 +163,7 @@ @Child private TestLanguageNode child; @Child private EventHandlerNode eventHandlerNode; - public TestLanguageWrapperNode(TestLanguageNode child) { + TestLanguageWrapperNode(TestLanguageNode child) { assert !(child instanceof TestLanguageWrapperNode); this.child = child; } @@ -211,7 +211,7 @@ static class TestValueNode extends TestLanguageNode { private final int value; - public TestValueNode(int value) { + TestValueNode(int value) { this.value = value; } @@ -228,7 +228,7 @@ @Child private TestLanguageNode leftChild; @Child private TestLanguageNode rightChild; - public TestAdditionNode(TestValueNode leftChild, TestValueNode rightChild) { + TestAdditionNode(TestValueNode leftChild, TestValueNode rightChild) { this.leftChild = insert(leftChild); this.rightChild = insert(rightChild); } @@ -253,7 +253,7 @@ * newly created AST. Global registry is not used, since that would interfere with other * tests run in the same environment. */ - public TestRootNode(String name, TestLanguageNode body) { + TestRootNode(String name, TestLanguageNode body) { super(InstrumentationPETestLanguage.class, null, null); this.name = name; this.body = body; diff -r 0653faea0327 -r 2160e7da7fb0 graal/com.oracle.graal.truffle.test/src/com/oracle/graal/truffle/test/TruffleStampTest.java --- a/graal/com.oracle.graal.truffle.test/src/com/oracle/graal/truffle/test/TruffleStampTest.java Tue Jan 19 10:55:27 2016 +0100 +++ b/graal/com.oracle.graal.truffle.test/src/com/oracle/graal/truffle/test/TruffleStampTest.java Tue Jan 19 12:20:24 2016 +0100 @@ -163,7 +163,7 @@ private final Object type; - public TestTypedObject(Object type) { + TestTypedObject(Object type) { this.type = type; } diff -r 0653faea0327 -r 2160e7da7fb0 graal/com.oracle.graal.truffle.test/src/com/oracle/graal/truffle/test/builtins/SLGenerateDummyNodesBuiltin.java --- a/graal/com.oracle.graal.truffle.test/src/com/oracle/graal/truffle/test/builtins/SLGenerateDummyNodesBuiltin.java Tue Jan 19 10:55:27 2016 +0100 +++ b/graal/com.oracle.graal.truffle.test/src/com/oracle/graal/truffle/test/builtins/SLGenerateDummyNodesBuiltin.java Tue Jan 19 12:20:24 2016 +0100 @@ -77,7 +77,7 @@ @Child private SLDummyNode left; @Child private SLDummyNode right; - public SLDummyNode(SLDummyNode left, SLDummyNode right) { + SLDummyNode(SLDummyNode left, SLDummyNode right) { super(null); this.left = left; this.right = right; diff -r 0653faea0327 -r 2160e7da7fb0 graal/com.oracle.graal.truffle/src/com/oracle/graal/truffle/DefaultTruffleStamp.java --- a/graal/com.oracle.graal.truffle/src/com/oracle/graal/truffle/DefaultTruffleStamp.java Tue Jan 19 10:55:27 2016 +0100 +++ b/graal/com.oracle.graal.truffle/src/com/oracle/graal/truffle/DefaultTruffleStamp.java Tue Jan 19 12:20:24 2016 +0100 @@ -142,7 +142,7 @@ private final Class clazz; private final Object type; - public InstanceStamp(Object instance) { + InstanceStamp(Object instance) { this.instance = instance; this.type = instance != null ? getTypeIdentifier(instance) : NO_TYPE; this.clazz = instance != null ? instance.getClass() : NO_CLASS; @@ -218,7 +218,7 @@ private final Class clazz; private final Object type; - public TypeStamp(Class clazz, Object type) { + TypeStamp(Class clazz, Object type) { this.clazz = clazz; this.type = type; assert type != NO_TYPE; @@ -281,7 +281,7 @@ private final Class clazz; - public ClassStamp(Class clazz) { + ClassStamp(Class clazz) { this.clazz = clazz; } @@ -336,13 +336,13 @@ private final TruffleStamp[] stampArray; private final int length; - public ArrayStamp(TruffleStamp stamp) { + ArrayStamp(TruffleStamp stamp) { this.stampArray = new TruffleStamp[MAX_STAMPED_ARGUMENTS]; Arrays.fill(this.stampArray, stamp); this.length = UNINITIALIZED_LENGTH; } - public ArrayStamp(TruffleStamp[] profiledTypes, int length) { + ArrayStamp(TruffleStamp[] profiledTypes, int length) { this.stampArray = profiledTypes; this.length = length; } diff -r 0653faea0327 -r 2160e7da7fb0 graal/com.oracle.graal.truffle/src/com/oracle/graal/truffle/OptimizedOSRLoopNode.java --- a/graal/com.oracle.graal.truffle/src/com/oracle/graal/truffle/OptimizedOSRLoopNode.java Tue Jan 19 10:55:27 2016 +0100 +++ b/graal/com.oracle.graal.truffle/src/com/oracle/graal/truffle/OptimizedOSRLoopNode.java Tue Jan 19 12:20:24 2016 +0100 @@ -212,7 +212,7 @@ @Child private OptimizedOSRLoopNode loopNode; - public OSRRootNode(OptimizedOSRLoopNode loop) { + OSRRootNode(OptimizedOSRLoopNode loop) { super(TruffleLanguage.class, loop.getSourceSection(), loop.getRootNode().getFrameDescriptor()); this.loopNode = loop; } diff -r 0653faea0327 -r 2160e7da7fb0 graal/com.oracle.graal.truffle/src/com/oracle/graal/truffle/PartialEvaluator.java --- a/graal/com.oracle.graal.truffle/src/com/oracle/graal/truffle/PartialEvaluator.java Tue Jan 19 10:55:27 2016 +0100 +++ b/graal/com.oracle.graal.truffle/src/com/oracle/graal/truffle/PartialEvaluator.java Tue Jan 19 12:20:24 2016 +0100 @@ -184,7 +184,7 @@ private final Object receiver; - public InterceptReceiverPlugin(Object receiver) { + InterceptReceiverPlugin(Object receiver) { this.receiver = receiver; } @@ -202,7 +202,7 @@ private OptimizedDirectCallNode lastDirectCallNode; private final ReplacementsImpl replacements; - public PEInlineInvokePlugin(TruffleInlining inlining, ReplacementsImpl replacements) { + PEInlineInvokePlugin(TruffleInlining inlining, ReplacementsImpl replacements) { this.inlining = new ArrayDeque<>(); this.inlining.push(inlining); this.replacements = replacements; @@ -259,7 +259,7 @@ private final InvocationPlugins invocationPlugins; private final LoopExplosionPlugin loopExplosionPlugin; - public ParsingInlineInvokePlugin(ReplacementsImpl replacements, InvocationPlugins invocationPlugins, LoopExplosionPlugin loopExplosionPlugin) { + ParsingInlineInvokePlugin(ReplacementsImpl replacements, InvocationPlugins invocationPlugins, LoopExplosionPlugin loopExplosionPlugin) { this.replacements = replacements; this.invocationPlugins = invocationPlugins; this.loopExplosionPlugin = loopExplosionPlugin; diff -r 0653faea0327 -r 2160e7da7fb0 graal/com.oracle.graal.truffle/src/com/oracle/graal/truffle/ReadOnlyFrame.java --- a/graal/com.oracle.graal.truffle/src/com/oracle/graal/truffle/ReadOnlyFrame.java Tue Jan 19 10:55:27 2016 +0100 +++ b/graal/com.oracle.graal.truffle/src/com/oracle/graal/truffle/ReadOnlyFrame.java Tue Jan 19 12:20:24 2016 +0100 @@ -34,7 +34,7 @@ class ReadOnlyFrame implements Frame { private final Frame delegate; - public ReadOnlyFrame(Frame delegate) { + ReadOnlyFrame(Frame delegate) { this.delegate = delegate; } diff -r 0653faea0327 -r 2160e7da7fb0 graal/com.oracle.graal.truffle/src/com/oracle/graal/truffle/TruffleCompilationResultBuilderFactory.java --- a/graal/com.oracle.graal.truffle/src/com/oracle/graal/truffle/TruffleCompilationResultBuilderFactory.java Tue Jan 19 10:55:27 2016 +0100 +++ b/graal/com.oracle.graal.truffle/src/com/oracle/graal/truffle/TruffleCompilationResultBuilderFactory.java Tue Jan 19 12:20:24 2016 +0100 @@ -56,7 +56,7 @@ */ private final List validAssumptions; - public TruffleCompilationResultBuilderFactory(StructuredGraph graph, List validAssumptions) { + TruffleCompilationResultBuilderFactory(StructuredGraph graph, List validAssumptions) { this.graph = graph; this.validAssumptions = validAssumptions; } diff -r 0653faea0327 -r 2160e7da7fb0 graal/com.oracle.graal.truffle/src/com/oracle/graal/truffle/TruffleExpansionLogger.java --- a/graal/com.oracle.graal.truffle/src/com/oracle/graal/truffle/TruffleExpansionLogger.java Tue Jan 19 10:55:27 2016 +0100 +++ b/graal/com.oracle.graal.truffle/src/com/oracle/graal/truffle/TruffleExpansionLogger.java Tue Jan 19 12:20:24 2016 +0100 @@ -109,7 +109,7 @@ private final int parentBci; private final List children = new ArrayList<>(); - public ExpansionTree(ExpansionTree parent, ResolvedJavaType targetReceiverType, ResolvedJavaMethod targetMethod, int parentBci) { + ExpansionTree(ExpansionTree parent, ResolvedJavaType targetReceiverType, ResolvedJavaMethod targetMethod, int parentBci) { this.parent = parent; this.targetReceiverType = targetReceiverType; this.targetMethod = targetMethod; diff -r 0653faea0327 -r 2160e7da7fb0 graal/com.oracle.graal.truffle/src/com/oracle/graal/truffle/TruffleInlining.java --- a/graal/com.oracle.graal.truffle/src/com/oracle/graal/truffle/TruffleInlining.java Tue Jan 19 10:55:27 2016 +0100 +++ b/graal/com.oracle.graal.truffle/src/com/oracle/graal/truffle/TruffleInlining.java Tue Jan 19 12:20:24 2016 +0100 @@ -227,7 +227,7 @@ private final NodeVisitor visitor; private boolean continueTraverse = true; - public CallTreeNodeVisitorImpl(OptimizedCallTarget target, NodeVisitor visitor) { + CallTreeNodeVisitorImpl(OptimizedCallTarget target, NodeVisitor visitor) { stack.add(target.getInlining()); this.visitor = visitor; } @@ -268,7 +268,7 @@ private List inliningDecisionStack = new ArrayList<>(); private List> iteratorStack = new ArrayList<>(); - public CallTreeNodeIterator(OptimizedCallTarget target) { + CallTreeNodeIterator(OptimizedCallTarget target) { inliningDecisionStack.add(target.getInlining()); iteratorStack.add(NodeUtil.makeRecursiveIterator(target.getRootNode())); } diff -r 0653faea0327 -r 2160e7da7fb0 graal/com.oracle.graal.truffle/src/com/oracle/graal/truffle/TruffleTreeDumpHandler.java --- a/graal/com.oracle.graal.truffle/src/com/oracle/graal/truffle/TruffleTreeDumpHandler.java Tue Jan 19 10:55:27 2016 +0100 +++ b/graal/com.oracle.graal.truffle/src/com/oracle/graal/truffle/TruffleTreeDumpHandler.java Tue Jan 19 12:20:24 2016 +0100 @@ -79,7 +79,7 @@ class InliningGraphPrintHandler implements GraphPrintHandler { private final TruffleInlining inlining; - public InliningGraphPrintHandler(TruffleInlining inlining) { + InliningGraphPrintHandler(TruffleInlining inlining) { this.inlining = inlining; } diff -r 0653faea0327 -r 2160e7da7fb0 graal/com.oracle.graal.truffle/src/com/oracle/graal/truffle/debug/CompilationStatisticsListener.java --- a/graal/com.oracle.graal.truffle/src/com/oracle/graal/truffle/debug/CompilationStatisticsListener.java Tue Jan 19 10:55:27 2016 +0100 +++ b/graal/com.oracle.graal.truffle/src/com/oracle/graal/truffle/debug/CompilationStatisticsListener.java Tue Jan 19 12:20:24 2016 +0100 @@ -343,7 +343,7 @@ private int callCountDirectNotCloned; private int loopCount; - public CallTargetNodeStatistics(OptimizedCallTarget target) { + CallTargetNodeStatistics(OptimizedCallTarget target) { target.accept((CallTreeNodeVisitor) this::visitNode, true); } diff -r 0653faea0327 -r 2160e7da7fb0 graal/com.oracle.graal.truffle/src/com/oracle/graal/truffle/debug/HistogramInlineInvokePlugin.java --- a/graal/com.oracle.graal.truffle/src/com/oracle/graal/truffle/debug/HistogramInlineInvokePlugin.java Tue Jan 19 10:55:27 2016 +0100 +++ b/graal/com.oracle.graal.truffle/src/com/oracle/graal/truffle/debug/HistogramInlineInvokePlugin.java Tue Jan 19 12:20:24 2016 +0100 @@ -105,7 +105,7 @@ private final IntSummaryStatistics shallowCount = new IntSummaryStatistics(); private final IntSummaryStatistics callCount = new IntSummaryStatistics(); - public MethodStatistics(ResolvedJavaMethod method) { + MethodStatistics(ResolvedJavaMethod method) { this.method = method; } @@ -143,7 +143,7 @@ private int deepNodeCount; private int callCount; - public MethodStatistic(MethodStatistic parent, ResolvedJavaMethod method, int nodeCountBefore, int callsBefore) { + MethodStatistic(MethodStatistic parent, ResolvedJavaMethod method, int nodeCountBefore, int callsBefore) { this.parent = parent; this.method = method; this.callCount = callsBefore; diff -r 0653faea0327 -r 2160e7da7fb0 graal/com.oracle.graal.truffle/src/com/oracle/graal/truffle/substitutions/TruffleGraphBuilderPlugins.java --- a/graal/com.oracle.graal.truffle/src/com/oracle/graal/truffle/substitutions/TruffleGraphBuilderPlugins.java Tue Jan 19 10:55:27 2016 +0100 +++ b/graal/com.oracle.graal.truffle/src/com/oracle/graal/truffle/substitutions/TruffleGraphBuilderPlugins.java Tue Jan 19 12:20:24 2016 +0100 @@ -444,7 +444,7 @@ private final JavaKind returnKind; - public CustomizedUnsafeLoadPlugin(JavaKind returnKind) { + CustomizedUnsafeLoadPlugin(JavaKind returnKind) { this.returnKind = returnKind; } @@ -469,7 +469,7 @@ private final JavaKind kind; - public CustomizedUnsafeStorePlugin(JavaKind kind) { + CustomizedUnsafeStorePlugin(JavaKind kind) { this.kind = kind; } diff -r 0653faea0327 -r 2160e7da7fb0 graal/com.oracle.graal.virtual/src/com/oracle/graal/virtual/phases/ea/PEReadEliminationBlockState.java --- a/graal/com.oracle.graal.virtual/src/com/oracle/graal/virtual/phases/ea/PEReadEliminationBlockState.java Tue Jan 19 10:55:27 2016 +0100 +++ b/graal/com.oracle.graal.virtual/src/com/oracle/graal/virtual/phases/ea/PEReadEliminationBlockState.java Tue Jan 19 12:20:24 2016 +0100 @@ -45,7 +45,7 @@ public final ValueNode object; public final int index; - public ReadCacheEntry(LocationIdentity identity, ValueNode object, int index) { + ReadCacheEntry(LocationIdentity identity, ValueNode object, int index) { this.identity = identity; this.object = object; this.index = index; diff -r 0653faea0327 -r 2160e7da7fb0 graal/com.oracle.graal.virtual/src/com/oracle/graal/virtual/phases/ea/PEReadEliminationClosure.java --- a/graal/com.oracle.graal.virtual/src/com/oracle/graal/virtual/phases/ea/PEReadEliminationClosure.java Tue Jan 19 10:55:27 2016 +0100 +++ b/graal/com.oracle.graal.virtual/src/com/oracle/graal/virtual/phases/ea/PEReadEliminationClosure.java Tue Jan 19 12:20:24 2016 +0100 @@ -66,6 +66,7 @@ public class PEReadEliminationClosure extends PartialEscapeClosure { private static final EnumMap UNBOX_LOCATIONS; + static { UNBOX_LOCATIONS = new EnumMap<>(JavaKind.class); for (JavaKind kind : JavaKind.values()) { @@ -284,7 +285,7 @@ private class ReadEliminationMergeProcessor extends MergeProcessor { - public ReadEliminationMergeProcessor(Block mergeBlock) { + ReadEliminationMergeProcessor(Block mergeBlock) { super(mergeBlock); } diff -r 0653faea0327 -r 2160e7da7fb0 graal/com.oracle.graal.virtual/src/com/oracle/graal/virtual/phases/ea/ReadEliminationBlockState.java --- a/graal/com.oracle.graal.virtual/src/com/oracle/graal/virtual/phases/ea/ReadEliminationBlockState.java Tue Jan 19 10:55:27 2016 +0100 +++ b/graal/com.oracle.graal.virtual/src/com/oracle/graal/virtual/phases/ea/ReadEliminationBlockState.java Tue Jan 19 12:20:24 2016 +0100 @@ -39,7 +39,7 @@ public final ValueNode object; public final T identity; - public CacheEntry(ValueNode object, T identity) { + CacheEntry(ValueNode object, T identity) { this.object = object; this.identity = identity; } @@ -71,7 +71,7 @@ static class LoadCacheEntry extends CacheEntry { - public LoadCacheEntry(ValueNode object, LocationIdentity identity) { + LoadCacheEntry(ValueNode object, LocationIdentity identity) { super(object, identity); } @@ -96,7 +96,7 @@ private LocationIdentity locationIdentity; - public UnsafeLoadCacheEntry(ValueNode object, ValueNode location, LocationIdentity locationIdentity) { + UnsafeLoadCacheEntry(ValueNode object, ValueNode location, LocationIdentity locationIdentity) { super(object, location); this.locationIdentity = locationIdentity; } diff -r 0653faea0327 -r 2160e7da7fb0 graal/com.oracle.graal.virtual/src/com/oracle/graal/virtual/phases/ea/ReadEliminationClosure.java --- a/graal/com.oracle.graal.virtual/src/com/oracle/graal/virtual/phases/ea/ReadEliminationClosure.java Tue Jan 19 10:55:27 2016 +0100 +++ b/graal/com.oracle.graal.virtual/src/com/oracle/graal/virtual/phases/ea/ReadEliminationClosure.java Tue Jan 19 12:20:24 2016 +0100 @@ -220,7 +220,7 @@ private final HashMap materializedPhis = CollectionsFactory.newMap(); - public ReadEliminationMergeProcessor(Block mergeBlock) { + ReadEliminationMergeProcessor(Block mergeBlock) { super(mergeBlock); }