# HG changeset patch # User Lukas Stadler # Date 1364488629 -3600 # Node ID 516e35a8eed83748bfa36d55deaace9f9d2e8d74 # Parent c7672a325fafa7cae55d2ef0861dc9d194dc3d2e rename early read elimination option and enable it diff -r c7672a325faf -r 516e35a8eed8 graal/com.oracle.graal.compiler.test/src/com/oracle/graal/compiler/test/ea/IterativeInliningTest.java --- a/graal/com.oracle.graal.compiler.test/src/com/oracle/graal/compiler/test/ea/IterativeInliningTest.java Wed Apr 03 10:08:42 2013 +0200 +++ b/graal/com.oracle.graal.compiler.test/src/com/oracle/graal/compiler/test/ea/IterativeInliningTest.java Thu Mar 28 17:37:09 2013 +0100 @@ -101,7 +101,7 @@ private void processMethod(final String snippet) { graph = parse(snippet); new ComputeProbabilityPhase().apply(graph); - GraalOptions.PEAReadCache = true; + GraalOptions.OptEarlyReadElimination = true; new IterativeInliningPhase(runtime(), new Assumptions(false), null, getDefaultPhasePlan(), OptimisticOptimizations.ALL).apply(graph); } } diff -r c7672a325faf -r 516e35a8eed8 graal/com.oracle.graal.compiler.test/src/com/oracle/graal/compiler/test/ea/PEAReadEliminationTest.java --- a/graal/com.oracle.graal.compiler.test/src/com/oracle/graal/compiler/test/ea/PEAReadEliminationTest.java Wed Apr 03 10:08:42 2013 +0200 +++ b/graal/com.oracle.graal.compiler.test/src/com/oracle/graal/compiler/test/ea/PEAReadEliminationTest.java Thu Mar 28 17:37:09 2013 +0100 @@ -204,7 +204,7 @@ new ComputeProbabilityPhase().apply(graph); Assumptions assumptions = new Assumptions(false); new InliningPhase(runtime(), null, assumptions, null, getDefaultPhasePlan(), OptimisticOptimizations.ALL).apply(graph); - GraalOptions.PEAReadCache = true; + GraalOptions.OptEarlyReadElimination = true; new PartialEscapeAnalysisPhase(runtime(), assumptions, false).apply(graph); } } diff -r c7672a325faf -r 516e35a8eed8 graal/com.oracle.graal.phases/src/com/oracle/graal/phases/GraalOptions.java --- a/graal/com.oracle.graal.phases/src/com/oracle/graal/phases/GraalOptions.java Wed Apr 03 10:08:42 2013 +0200 +++ b/graal/com.oracle.graal.phases/src/com/oracle/graal/phases/GraalOptions.java Thu Mar 28 17:37:09 2013 +0100 @@ -68,7 +68,6 @@ public static int EscapeAnalysisIterations = 2; public static String EscapeAnalyzeOnly = null; public static int MaximumEscapeAnalysisArrayLength = 32; - public static boolean PEAReadCache = ____; public static boolean PEAInliningHints = ____; public static double TailDuplicationProbability = 0.5; @@ -191,6 +190,7 @@ public static boolean OptAssumptions = true; public static boolean OptConvertDeoptsToGuards = true; public static boolean OptReadElimination = true; + public static boolean OptEarlyReadElimination = true; public static boolean OptCanonicalizer = true; public static boolean OptScheduleOutOfLoops = true; public static boolean OptEliminateGuards = true; diff -r c7672a325faf -r 516e35a8eed8 graal/com.oracle.graal.virtual/src/com/oracle/graal/virtual/phases/ea/PartialEscapeAnalysisPhase.java --- a/graal/com.oracle.graal.virtual/src/com/oracle/graal/virtual/phases/ea/PartialEscapeAnalysisPhase.java Wed Apr 03 10:08:42 2013 +0200 +++ b/graal/com.oracle.graal.virtual/src/com/oracle/graal/virtual/phases/ea/PartialEscapeAnalysisPhase.java Thu Mar 28 17:37:09 2013 +0100 @@ -76,7 +76,7 @@ return; } - if (!GraalOptions.PEAReadCache) { + if (!GraalOptions.OptEarlyReadElimination) { boolean analyzableNodes = false; for (Node node : graph.getNodes()) { if (node instanceof VirtualizableAllocation) { diff -r c7672a325faf -r 516e35a8eed8 graal/com.oracle.graal.virtual/src/com/oracle/graal/virtual/phases/ea/PartialEscapeClosure.java --- a/graal/com.oracle.graal.virtual/src/com/oracle/graal/virtual/phases/ea/PartialEscapeClosure.java Wed Apr 03 10:08:42 2013 +0200 +++ b/graal/com.oracle.graal.virtual/src/com/oracle/graal/virtual/phases/ea/PartialEscapeClosure.java Thu Mar 28 17:37:09 2013 +0100 @@ -110,7 +110,7 @@ trace("%s ", node); deleted = false; } - if (GraalOptions.PEAReadCache) { + if (GraalOptions.OptEarlyReadElimination) { if (!deleted) { if (node instanceof StoreFieldNode) { METRIC_STOREFIELD_RECORDED.increment();