# HG changeset patch # User Thomas Wuerthinger # Date 1306160608 -7200 # Node ID cc2b98e2b832e0bfba3fdec6204d469e5fe276af # Parent 99912abb3ff7b64d1fcd0f59a5bf6012be838128 Reduced moveToPhi usages to 1. diff -r 99912abb3ff7 -r cc2b98e2b832 graal/GraalCompiler/src/com/sun/c1x/gen/LIRGenerator.java --- a/graal/GraalCompiler/src/com/sun/c1x/gen/LIRGenerator.java Mon May 23 15:07:01 2011 +0200 +++ b/graal/GraalCompiler/src/com/sun/c1x/gen/LIRGenerator.java Mon May 23 16:23:28 2011 +0200 @@ -565,9 +565,6 @@ CiValue tag = load(x.value()); setNoResult(x); - // move values into phi locations - moveToPhi(x.stateAfter()); - if (x.numberOfCases() == 0 || x.numberOfCases() < C1XOptions.SequentialSwitchLimit) { int len = x.numberOfCases(); for (int i = 0; i < len; i++) { @@ -825,9 +822,6 @@ CiValue tag = value.result(); setNoResult(x); - // move values into phi locations - moveToPhi(x.stateAfter()); - // TODO: tune the defaults for the controls used to determine what kind of translation to use if (x.numberOfCases() == 0 || x.numberOfCases() <= C1XOptions.SequentialSwitchLimit) { int loKey = x.lowKey(); diff -r 99912abb3ff7 -r cc2b98e2b832 graal/GraalCompiler/src/com/sun/c1x/target/amd64/AMD64LIRGenerator.java --- a/graal/GraalCompiler/src/com/sun/c1x/target/amd64/AMD64LIRGenerator.java Mon May 23 15:07:01 2011 +0200 +++ b/graal/GraalCompiler/src/com/sun/c1x/target/amd64/AMD64LIRGenerator.java Mon May 23 16:23:28 2011 +0200 @@ -504,7 +504,6 @@ CiValue left = xin.result(); CiValue right = yin.result(); lir.cmp(cond, left, right); - moveToPhi(); if (x.x().kind.isFloat() || x.x().kind.isDouble()) { lir.branch(cond, right.kind, getLIRBlock(x.trueSuccessor()), getLIRBlock(x.unorderedSuccessor())); } else {