# HG changeset patch # User Josef Eisl # Date 1430835879 -7200 # Node ID b2b3c514a391d89b4584e0ddd249202b4faeb22f # Parent a2430e146460b728fde6ee7a66e1f7e4d43cf952 MoveResolver: don't call check from constructor. diff -r a2430e146460 -r b2b3c514a391 graal/com.oracle.graal.lir/src/com/oracle/graal/lir/alloc/lsra/LinearScan.java --- a/graal/com.oracle.graal.lir/src/com/oracle/graal/lir/alloc/lsra/LinearScan.java Thu Apr 30 15:39:11 2015 +0200 +++ b/graal/com.oracle.graal.lir/src/com/oracle/graal/lir/alloc/lsra/LinearScan.java Tue May 05 16:24:39 2015 +0200 @@ -202,7 +202,9 @@ } protected MoveResolver createMoveResolver() { - return new MoveResolver(this); + MoveResolver moveResolver = new MoveResolver(this); + assert moveResolver.checkEmpty(); + return moveResolver; } public static boolean isVariableOrRegister(Value value) { diff -r a2430e146460 -r b2b3c514a391 graal/com.oracle.graal.lir/src/com/oracle/graal/lir/alloc/lsra/MoveResolver.java --- a/graal/com.oracle.graal.lir/src/com/oracle/graal/lir/alloc/lsra/MoveResolver.java Thu Apr 30 15:39:11 2015 +0200 +++ b/graal/com.oracle.graal.lir/src/com/oracle/graal/lir/alloc/lsra/MoveResolver.java Tue May 05 16:24:39 2015 +0200 @@ -90,7 +90,6 @@ this.insertIdx = -1; this.insertionBuffer = new LIRInsertionBuffer(); this.registerBlocked = new int[allocator.registers.length]; - assert checkEmpty(); } boolean checkEmpty() { diff -r a2430e146460 -r b2b3c514a391 graal/com.oracle.graal.lir/src/com/oracle/graal/lir/alloc/lsra/SSALinearScan.java --- a/graal/com.oracle.graal.lir/src/com/oracle/graal/lir/alloc/lsra/SSALinearScan.java Thu Apr 30 15:39:11 2015 +0200 +++ b/graal/com.oracle.graal.lir/src/com/oracle/graal/lir/alloc/lsra/SSALinearScan.java Tue May 05 16:24:39 2015 +0200 @@ -54,7 +54,9 @@ @Override protected MoveResolver createMoveResolver() { - return new SSAMoveResolver(this); + SSAMoveResolver moveResolver = new SSAMoveResolver(this); + assert moveResolver.checkEmpty(); + return moveResolver; } @Override diff -r a2430e146460 -r b2b3c514a391 graal/com.oracle.graal.lir/src/com/oracle/graal/lir/alloc/lsra/SSAMoveResolver.java --- a/graal/com.oracle.graal.lir/src/com/oracle/graal/lir/alloc/lsra/SSAMoveResolver.java Thu Apr 30 15:39:11 2015 +0200 +++ b/graal/com.oracle.graal.lir/src/com/oracle/graal/lir/alloc/lsra/SSAMoveResolver.java Tue May 05 16:24:39 2015 +0200 @@ -32,7 +32,7 @@ final class SSAMoveResolver extends MoveResolver { - private int[] stackBlocked = null; + private int[] stackBlocked; SSAMoveResolver(LinearScan allocator) { super(allocator); @@ -41,10 +41,8 @@ @Override boolean checkEmpty() { - if (stackBlocked != null) { - for (int i = 0; i < stackBlocked.length; i++) { - assert stackBlocked[i] == 0 : "stack map must be empty before and after processing"; - } + for (int i = 0; i < stackBlocked.length; i++) { + assert stackBlocked[i] == 0 : "stack map must be empty before and after processing"; } return super.checkEmpty(); }