# HG changeset patch # User Miguel Garcia # Date 1401721534 -7200 # Node ID a27eceb10d33319c16ddf2a62800aaf58c517b2b # Parent 66fe80527060b0f3d3a0711c2f5b668cc507340a [inliner] readability diff -r 66fe80527060 -r a27eceb10d33 graal/com.oracle.graal.phases.common/src/com/oracle/graal/phases/common/inlining/info/elem/InlineableGraph.java --- a/graal/com.oracle.graal.phases.common/src/com/oracle/graal/phases/common/inlining/info/elem/InlineableGraph.java Mon Jun 02 17:00:17 2014 +0200 +++ b/graal/com.oracle.graal.phases.common/src/com/oracle/graal/phases/common/inlining/info/elem/InlineableGraph.java Mon Jun 02 17:05:34 2014 +0200 @@ -43,7 +43,7 @@ public InlineableGraph(final ResolvedJavaMethod method, final Invoke invoke, final HighTierContext context, CanonicalizerPhase canonicalizer) { this.graph = buildGraph(method, context, canonicalizer); - specializeGraphToArguments(this.graph, invoke, context, canonicalizer); + specializeGraphToArguments(invoke, context, canonicalizer); } /** @@ -75,13 +75,13 @@ * @return true iff one or more parameters newGraph were specialized to account for * a constant argument, or an argument with a more specific stamp. */ - private static boolean specializeGraphToArguments(final StructuredGraph newGraph, final Invoke invoke, final HighTierContext context, CanonicalizerPhase canonicalizer) { - try (Debug.Scope s = Debug.scope("InlineGraph", newGraph)) { + private boolean specializeGraphToArguments(final Invoke invoke, final HighTierContext context, CanonicalizerPhase canonicalizer) { + try (Debug.Scope s = Debug.scope("InlineGraph", graph)) { - ArrayList parameterUsages = replaceParamsWithMoreInformativeArguments(invoke, newGraph, context); + ArrayList parameterUsages = replaceParamsWithMoreInformativeArguments(invoke, graph, context); if (parameterUsages != null && OptCanonicalizer.getValue()) { assert !parameterUsages.isEmpty() : "The caller didn't have more information about arguments after all"; - canonicalizer.applyIncremental(newGraph, context, parameterUsages); + canonicalizer.applyIncremental(graph, context, parameterUsages); return true; } else { // TODO (chaeubl): if args are not more concrete, inlining should be avoided