# HG changeset patch # User Doug Simon # Date 1384115658 -3600 # Node ID b5cf28c97430d8e189c41baf2ad11514d78a56fb # Parent a7eee87c7897abd85c4d563d7c601267a5e5dfde replaced global snippet and template node metrics with per-snippet and per-template metrics diff -r a7eee87c7897 -r b5cf28c97430 graal/com.oracle.graal.replacements/src/com/oracle/graal/replacements/ReplacementsImpl.java --- a/graal/com.oracle.graal.replacements/src/com/oracle/graal/replacements/ReplacementsImpl.java Sun Nov 10 21:31:18 2013 +0100 +++ b/graal/com.oracle.graal.replacements/src/com/oracle/graal/replacements/ReplacementsImpl.java Sun Nov 10 21:34:18 2013 +0100 @@ -83,8 +83,6 @@ private static final boolean UseSnippetGraphCache = Boolean.parseBoolean(System.getProperty("graal.useSnippetGraphCache", "true")); private static final DebugTimer SnippetPreparationTime = Debug.timer("SnippetPreparationTime"); - private static final DebugMetric SnippetGraphsNodeCount = Debug.metric("SnippetGraphsNodeCount"); - public StructuredGraph getSnippet(ResolvedJavaMethod method) { assert method.getAnnotation(Snippet.class) != null : "Snippet must be annotated with @" + Snippet.class.getSimpleName(); assert !Modifier.isAbstract(method.getModifiers()) && !Modifier.isNative(method.getModifiers()) : "Snippet must not be abstract or native"; @@ -93,7 +91,7 @@ if (graph == null) { try (TimerCloseable a = SnippetPreparationTime.start()) { StructuredGraph newGraph = makeGraph(method, null, inliningPolicy(method), method.getAnnotation(Snippet.class).removeAllFrameStates()); - SnippetGraphsNodeCount.add(newGraph.getNodeCount()); + Debug.metric("SnippetNodeCount[" + method.getName() + "]").add(newGraph.getNodeCount()); if (!UseSnippetGraphCache) { return newGraph; } diff -r a7eee87c7897 -r b5cf28c97430 graal/com.oracle.graal.replacements/src/com/oracle/graal/replacements/SnippetTemplate.java --- a/graal/com.oracle.graal.replacements/src/com/oracle/graal/replacements/SnippetTemplate.java Sun Nov 10 21:31:18 2013 +0100 +++ b/graal/com.oracle.graal.replacements/src/com/oracle/graal/replacements/SnippetTemplate.java Sun Nov 10 21:34:18 2013 +0100 @@ -360,7 +360,6 @@ private static final DebugTimer SnippetTemplateCreationTime = Debug.timer("SnippetTemplateCreationTime"); private static final DebugMetric SnippetTemplates = Debug.metric("SnippetTemplateCount"); - private static final DebugMetric SnippetTemplatesNodeCount = Debug.metric("SnippetTemplatesNodeCount"); private static final String MAX_TEMPLATES_PER_SNIPPET_PROPERTY_NAME = "graal.maxTemplatesPerSnippet"; private static final boolean UseSnippetTemplateCache = Boolean.parseBoolean(System.getProperty("graal.useSnippetTemplateCache", "true")); @@ -661,7 +660,7 @@ this.stampNodes = curStampNodes; this.returnNode = retNode; - SnippetTemplatesNodeCount.add(nodes.size()); + Debug.metric(debugValueName("SnippetTemplateNodeCount", args)).add(nodes.size()); args.info.notifyNewTemplate(); }