Mercurial > hg > graal-compiler

changeset 9705:de7319e48e48

Find changesets by keywords (author, files, the commit message), revision number or hash, or revset expression.
Temporary fixes for the merge.
author Christian Haeubl <haeubl@ssw.jku.at>
date Mon, 13 May 2013 17:43:42 +0200
parents f9a65a0e626b
children f44d7e24cebd
files graal/com.oracle.graal.phases.common/src/com/oracle/graal/phases/common/InliningPhase.java graal/com.oracle.graal.phases.common/src/com/oracle/graal/phases/common/InliningUtil.java
diffstat 2 files changed, 39 insertions(+), 24 deletions(-) [+]
line wrap: on
line diff
--- a/graal/com.oracle.graal.phases.common/src/com/oracle/graal/phases/common/InliningPhase.java	Mon May 13 17:11:31 2013 +0200
+++ b/graal/com.oracle.graal.phases.common/src/com/oracle/graal/phases/common/InliningPhase.java	Mon May 13 17:43:42 2013 +0200
@@ -90,20 +90,29 @@
     }
 
     public static void storeHighLevelStatistics(StructuredGraph graph) {
-        CompiledMethodInfo info = compiledMethodInfo(graph.method());
-        double summedUpProbabilityOfRemainingInvokes = sumUpInvokeProbabilities(graph);
-        info.setSummedUpProbabilityOfRemainingInvokes(summedUpProbabilityOfRemainingInvokes);
-        info.setHighLevelNodeCount(graph.getNodeCount());
+        ResolvedJavaMethod method = graph.method();
+        if (method != null) {
+            CompiledMethodInfo info = compiledMethodInfo(method);
+            double summedUpProbabilityOfRemainingInvokes = sumUpInvokeProbabilities(graph);
+            info.setSummedUpProbabilityOfRemainingInvokes(summedUpProbabilityOfRemainingInvokes);
+            info.setHighLevelNodeCount(graph.getNodeCount());
+        }
     }
 
     public static void storeMidLevelStatistics(StructuredGraph graph) {
-        CompiledMethodInfo info = compiledMethodInfo(graph.method());
-        info.setMidLevelNodeCount(graph.getNodeCount());
+        ResolvedJavaMethod method = graph.method();
+        if (method != null) {
+            CompiledMethodInfo info = compiledMethodInfo(graph.method());
+            info.setMidLevelNodeCount(graph.getNodeCount());
+        }
     }
 
     public static void storeLowLevelStatistics(StructuredGraph graph) {
-        CompiledMethodInfo info = compiledMethodInfo(graph.method());
-        info.setLowLevelNodeCount(graph.getNodeCount());
+        ResolvedJavaMethod method = graph.method();
+        if (method != null) {
+            CompiledMethodInfo info = compiledMethodInfo(graph.method());
+            info.setLowLevelNodeCount(graph.getNodeCount());
+        }
     }
 
     @Override
--- a/graal/com.oracle.graal.phases.common/src/com/oracle/graal/phases/common/InliningUtil.java	Mon May 13 17:11:31 2013 +0200
+++ b/graal/com.oracle.graal.phases.common/src/com/oracle/graal/phases/common/InliningUtil.java	Mon May 13 17:43:42 2013 +0200
@@ -42,8 +42,8 @@
 import com.oracle.graal.nodes.type.*;
 import com.oracle.graal.nodes.util.*;
 import com.oracle.graal.phases.*;
+import com.oracle.graal.phases.common.InliningPhase.InliningData;
 import com.oracle.graal.phases.tiers.*;
-import com.oracle.graal.phases.common.InliningPhase.InliningData;
 
 public class InliningUtil {
 
@@ -267,7 +267,7 @@
 
         @Override
         public StructuredGraph graph() {
-            return (StructuredGraph) invoke.asNode().graph();
+            return invoke.asNode().graph();
         }
 
         @Override
@@ -510,8 +510,8 @@
 
         private double[] computeMethodProbabilities() {
             double[] result = new double[concretes.size()];
-            for (int i = 0; i < typesToConcretes.length; i++) {
-                int concrete = typesToConcretes[i];
+            for (int i = 0; i < typesToConcretes.size(); i++) {
+                int concrete = typesToConcretes.get(i);
                 double probability = ptypes.get(i).getProbability();
                 result[concrete] += probability;
             }
@@ -564,9 +564,9 @@
             // receiver null check must be the first node
             InliningUtil.receiverNullCheck(invoke);
             if (hasSingleMethod()) {
-                inlineSingleMethod(graph(), assumptions);
+                inlineSingleMethod(graph(), runtime, assumptions);
             } else {
-                inlineMultipleMethods(graph(), assumptions);
+                inlineMultipleMethods(graph(), runtime, assumptions);
             }
         }
 
@@ -578,7 +578,7 @@
             return notRecordedTypeProbability > 0;
         }
 
-        private void inlineMultipleMethods(StructuredGraph graph, Assumptions assumptions) {
+        private void inlineMultipleMethods(StructuredGraph graph, MetaAccessProvider runtime, Assumptions assumptions) {
             int numberOfMethods = concretes.size();
             FixedNode continuation = invoke.next();
 
@@ -684,11 +684,15 @@
                     }
                     current = ((FixedWithNextNode) current).next();
                 } while (current instanceof FixedWithNextNode);
-                if (opportunities > 0) {
-                    metricInliningTailDuplication.increment();
-                    Debug.log("MultiTypeGuardInlineInfo starting tail duplication (%d opportunities)", opportunities);
-                    TailDuplicationPhase.tailDuplicate(returnMerge, TailDuplicationPhase.TRUE_DECISION, replacementNodes, new HighTierContext(runtime, assumptions, replacements));
-                }
+
+                // TEMP:
+// if (opportunities > 0) {
+// metricInliningTailDuplication.increment();
+// Debug.log("MultiTypeGuardInlineInfo starting tail duplication (%d opportunities)",
+// opportunities);
+// TailDuplicationPhase.tailDuplicate(returnMerge, TailDuplicationPhase.TRUE_DECISION,
+// replacementNodes, new HighTierContext(runtime, assumptions, replacements));
+// }
             }
         }
 
@@ -725,7 +729,7 @@
             return result;
         }
 
-        private void inlineSingleMethod(StructuredGraph graph, Assumptions assumptions) {
+        private void inlineSingleMethod(StructuredGraph graph, MetaAccessProvider runtime, Assumptions assumptions) {
             assert concretes.size() == 1 && inlineableElements.length == 1 && ptypes.size() > 1 && !shouldFallbackToInvoke() && notRecordedTypeProbability == 0;
 
             AbstractBeginNode calleeEntryNode = graph.add(new BeginNode());
@@ -904,9 +908,9 @@
         @Override
         public void tryToDevirtualizeInvoke(MetaAccessProvider runtime, Assumptions assumptions) {
             if (hasSingleMethod()) {
-                tryToDevirtualizeSingleMethod(graph());
+                tryToDevirtualizeSingleMethod(graph(), runtime);
             } else {
-                tryToDevirtualizeMultipleMethods(graph());
+                tryToDevirtualizeMultipleMethods(graph(), runtime);
             }
         }
 
@@ -1075,7 +1079,9 @@
         return new ExactInlineInfo(invoke, targetMethod);
     }
 
-    private static InlineInfo getTypeCheckedInlineInfo(InliningData data, Invoke invoke, int maxNumberOfMethods, Replacements replacements, ResolvedJavaMethod targetMethod, OptimisticOptimizations optimisticOpts) {
+    private static InlineInfo getTypeCheckedInlineInfo(InliningData data, Invoke invoke, int maxNumberOfMethods, Replacements replacements, ResolvedJavaMethod targetMethod,
+                    OptimisticOptimizations optimisticOpts) {
+        JavaTypeProfile typeProfile;
         ValueNode receiver = invoke.callTarget().arguments().get(0);
         if (receiver instanceof TypeProfileProxyNode) {
             TypeProfileProxyNode typeProfileProxyNode = (TypeProfileProxyNode) receiver;