Mercurial > hg > graal-compiler

changeset 22949:1d8238228f81

Find changesets by keywords (author, files, the commit message), revision number or hash, or revset expression.
TraceRA: TraceLinearScanWalker: do not use spill state SpillInDominator.
author Josef Eisl <josef.eisl@jku.at>
date Fri, 16 Oct 2015 16:23:17 +0200
parents 060324abb994
children 16c3ddbe2ebc
files graal/com.oracle.graal.lir/src/com/oracle/graal/lir/alloc/trace/TraceLinearScanWalker.java
diffstat 1 files changed, 4 insertions(+), 14 deletions(-) [+]
line wrap: on
line diff
--- a/graal/com.oracle.graal.lir/src/com/oracle/graal/lir/alloc/trace/TraceLinearScanWalker.java	Thu Nov 05 10:22:28 2015 +0100
+++ b/graal/com.oracle.graal.lir/src/com/oracle/graal/lir/alloc/trace/TraceLinearScanWalker.java	Fri Oct 16 16:23:17 2015 +0200
@@ -569,13 +569,8 @@
                      * The loop depth of the spilling position is higher then the loop depth at the
                      * definition of the interval. Move write to memory out of loop.
                      */
-                    if (com.oracle.graal.lir.alloc.lsra.LinearScan.Options.LIROptLSRAOptimizeSpillPosition.getValue()) {
-                        // find best spill position in dominator the tree
-                        interval.setSpillState(SpillState.SpillInDominator);
-                    } else {
-                        // store at definition of the interval
-                        interval.setSpillState(SpillState.StoreAtDefinition);
-                    }
+                    // store at definition of the interval
+                    interval.setSpillState(SpillState.StoreAtDefinition);
                 } else {
                     /*
                      * The interval is currently spilled only once, so for now there is no reason to
@@ -587,13 +582,8 @@
             }
 
             case OneSpillStore: {
-                if (com.oracle.graal.lir.alloc.lsra.LinearScan.Options.LIROptLSRAOptimizeSpillPosition.getValue()) {
-                    // the interval is spilled more then once
-                    interval.setSpillState(SpillState.SpillInDominator);
-                } else {
-                    // It is better to store it to memory at the definition.
-                    interval.setSpillState(SpillState.StoreAtDefinition);
-                }
+                // It is better to store it to memory at the definition.
+                interval.setSpillState(SpillState.StoreAtDefinition);
                 break;
             }