# HG changeset patch
# User Josef Eisl <josef.eisl@jku.at>
# Date 1445005397 -7200
# Node ID 1d8238228f815e658582eef9d8551bceb74ae845
# Parent  060324abb9942eaa922a26746f85340da14755bd
TraceRA: TraceLinearScanWalker: do not use spill state SpillInDominator.

diff -r 060324abb994 -r 1d8238228f81 graal/com.oracle.graal.lir/src/com/oracle/graal/lir/alloc/trace/TraceLinearScanWalker.java
--- 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;
             }