changeset 15854:3ce7f1c32353

[inlining] readability in CallsiteHolder constructor, part 1
author Miguel Garcia <miguel.m.garcia@oracle.com>
date Wed, 21 May 2014 19:49:48 +0200
parents 241044995c87
children 1a8a29431872
files graal/com.oracle.graal.phases.common/src/com/oracle/graal/phases/common/inlining/walker/CallsiteHolder.java
diffstat 1 files changed, 6 insertions(+), 7 deletions(-) [+]
line wrap: on
line diff
--- a/graal/com.oracle.graal.phases.common/src/com/oracle/graal/phases/common/inlining/walker/CallsiteHolder.java	Thu May 22 16:51:32 2014 +0200
+++ b/graal/com.oracle.graal.phases.common/src/com/oracle/graal/phases/common/inlining/walker/CallsiteHolder.java	Wed May 21 19:49:48 2014 +0200
@@ -51,15 +51,14 @@
 
     public CallsiteHolder(StructuredGraph graph, double probability, double relevance) {
         this.graph = graph;
-        if (graph == null) {
-            this.remainingInvokes = new LinkedList<>();
-        } else {
-            LinkedList<Invoke> invokes = new InliningIterator(graph).apply();
-            assert invokes.size() == count(graph.getInvokes());
-            this.remainingInvokes = invokes;
-        }
         this.probability = probability;
         this.relevance = relevance;
+        if (graph == null) {
+            remainingInvokes = new LinkedList<>();
+        } else {
+            remainingInvokes = new InliningIterator(graph).apply();
+            assert remainingInvokes.size() == count(graph.getInvokes());
+        }
 
         if (graph != null && !remainingInvokes.isEmpty()) {
             probabilities = new FixedNodeProbabilityCache();