Mercurial > hg > truffle
diff graal/com.oracle.graal.phases.common/src/com/oracle/graal/phases/common/InliningUtil.java @ 13197:8569b9e047cd
change debug scopes implementation to prevent extra frames related to mechanism being added to call stack
author | Doug Simon <doug.simon@oracle.com> |
---|---|
date | Sat, 30 Nov 2013 01:16:55 +0100 |
parents | e343d4623e47 |
children | 210f58e992a1 |
line wrap: on
line diff
--- a/graal/com.oracle.graal.phases.common/src/com/oracle/graal/phases/common/InliningUtil.java Fri Nov 29 20:46:54 2013 +0100 +++ b/graal/com.oracle.graal.phases.common/src/com/oracle/graal/phases/common/InliningUtil.java Sat Nov 30 01:16:55 2013 +0100 @@ -30,7 +30,6 @@ import java.lang.reflect.*; import java.util.*; -import java.util.concurrent.*; import com.oracle.graal.api.code.*; import com.oracle.graal.api.code.Assumptions.Assumption; @@ -38,6 +37,7 @@ import com.oracle.graal.api.meta.JavaTypeProfile.ProfiledType; import com.oracle.graal.api.meta.ResolvedJavaType.Representation; import com.oracle.graal.debug.*; +import com.oracle.graal.debug.Debug.Scope; import com.oracle.graal.graph.*; import com.oracle.graal.graph.Graph.DuplicationReplacement; import com.oracle.graal.graph.Node.ValueNumberable; @@ -173,12 +173,9 @@ } public static void logInliningDecision(final String msg, final Object... args) { - Debug.scope(inliningDecisionsScopeString, new Runnable() { - - public void run() { - Debug.log(msg, args); - } - }); + try (Scope s = Debug.scope(inliningDecisionsScopeString)) { + Debug.log(msg, args); + } } private static boolean logNotInlinedMethod(Invoke invoke, String msg) { @@ -220,12 +217,9 @@ } public static boolean shouldLogInliningDecision() { - return Debug.scope(inliningDecisionsScopeString, new Callable<Boolean>() { - - public Boolean call() { - return Debug.isLogEnabled(); - } - }); + try (Scope s = Debug.scope(inliningDecisionsScopeString)) { + return Debug.isLogEnabled(); + } } private static String methodName(ResolvedJavaMethod method, Invoke invoke) {