Mercurial > hg > graal-compiler

changeset 10609:5f016460f95b

Find changesets by keywords (author, files, the commit message), revision number or hash, or revset expression.
scheduling: remove duplicated code check is already done in CFG.commonDominator()
author Bernhard Urban <bernhard.urban@jku.at>
date Fri, 05 Jul 2013 13:28:18 +0200
parents eeb80dcd60d8
children 1db97e3de11c
files graal/com.oracle.graal.phases/src/com/oracle/graal/phases/schedule/SchedulePhase.java
diffstat 1 files changed, 2 insertions(+), 11 deletions(-) [+]
line wrap: on
line diff
--- a/graal/com.oracle.graal.phases/src/com/oracle/graal/phases/schedule/SchedulePhase.java	Thu Jul 04 15:30:38 2013 +0200
+++ b/graal/com.oracle.graal.phases/src/com/oracle/graal/phases/schedule/SchedulePhase.java	Fri Jul 05 13:28:18 2013 +0200
@@ -23,6 +23,7 @@
 package com.oracle.graal.phases.schedule;
 
 import static com.oracle.graal.api.meta.LocationIdentity.*;
+import static com.oracle.graal.nodes.cfg.ControlFlowGraph.*;
 import static com.oracle.graal.phases.GraalOptions.*;
 
 import java.util.*;
@@ -335,7 +336,7 @@
 
         @Override
         public void apply(Block newBlock) {
-            this.block = getCommonDominator(this.block, newBlock);
+            this.block = commonDominator(this.block, newBlock);
         }
     }
 
@@ -487,16 +488,6 @@
         // now true usages are ready
     }
 
-    private static Block getCommonDominator(Block a, Block b) {
-        if (a == null) {
-            return b;
-        }
-        if (b == null) {
-            return a;
-        }
-        return ControlFlowGraph.commonDominator(a, b);
-    }
-
     private void sortNodesWithinBlocks(StructuredGraph graph, SchedulingStrategy strategy) {
         NodeBitMap visited = graph.createNodeBitMap();
         for (Block b : cfg.getBlocks()) {