# HG changeset patch # User Miguel Garcia # Date 1400148970 -7200 # Node ID 6da6cba882f60638d2fdc247ff23ae5ed16a3f1f # Parent c80794ec690bb4d343d9e64db9974aa9048dbce5 [inlining] access levels in AbstractInliningPolicy back to what they were diff -r c80794ec690b -r 6da6cba882f6 graal/com.oracle.graal.phases.common/src/com/oracle/graal/phases/common/inlining/InliningUtil.java --- a/graal/com.oracle.graal.phases.common/src/com/oracle/graal/phases/common/inlining/InliningUtil.java Thu May 15 11:22:08 2014 +0200 +++ b/graal/com.oracle.graal.phases.common/src/com/oracle/graal/phases/common/inlining/InliningUtil.java Thu May 15 12:16:10 2014 +0200 @@ -28,7 +28,6 @@ import static com.oracle.graal.compiler.common.type.StampFactory.*; import java.util.*; -import java.util.function.*; import com.oracle.graal.api.code.*; import com.oracle.graal.api.code.Assumptions.Assumption; diff -r c80794ec690b -r 6da6cba882f6 graal/com.oracle.graal.phases.common/src/com/oracle/graal/phases/common/inlining/policy/AbstractInliningPolicy.java --- a/graal/com.oracle.graal.phases.common/src/com/oracle/graal/phases/common/inlining/policy/AbstractInliningPolicy.java Thu May 15 11:22:08 2014 +0200 +++ b/graal/com.oracle.graal.phases.common/src/com/oracle/graal/phases/common/inlining/policy/AbstractInliningPolicy.java Thu May 15 12:16:10 2014 +0200 @@ -45,19 +45,19 @@ this.hints = hints; } - public double computeMaximumSize(double relevance, int configuredMaximum) { + protected double computeMaximumSize(double relevance, int configuredMaximum) { double inlineRatio = Math.min(RelevanceCapForInlining.getValue(), relevance); return configuredMaximum * inlineRatio; } - public double getInliningBonus(InlineInfo info) { + protected double getInliningBonus(InlineInfo info) { if (hints != null && hints.containsKey(info.invoke())) { return hints.get(info.invoke()); } return 1; } - public boolean isIntrinsic(Replacements replacements, InlineInfo info) { + protected boolean isIntrinsic(Replacements replacements, InlineInfo info) { if (AlwaysInlineIntrinsics.getValue()) { return onlyIntrinsics(replacements, info); } else { @@ -86,7 +86,7 @@ return true; } - public static int previousLowLevelGraphSize(InlineInfo info) { + protected static int previousLowLevelGraphSize(InlineInfo info) { int size = 0; for (int i = 0; i < info.numberOfMethods(); i++) { ResolvedJavaMethod m = info.methodAt(i); @@ -99,7 +99,7 @@ return size; } - public static int determineNodeCount(InlineInfo info) { + protected static int determineNodeCount(InlineInfo info) { int nodes = 0; for (int i = 0; i < info.numberOfMethods(); i++) { InliningUtil.Inlineable elem = info.inlineableElementAt(i); @@ -110,7 +110,7 @@ return nodes; } - public static double determineInvokeProbability(ToDoubleFunction probabilities, InlineInfo info) { + protected static double determineInvokeProbability(ToDoubleFunction probabilities, InlineInfo info) { double invokeProbability = 0; for (int i = 0; i < info.numberOfMethods(); i++) { InliningUtil.Inlineable callee = info.inlineableElementAt(i);