changeset 18923:837d2b18a171

No need to clean up deleted phis.
author Thomas Wuerthinger <thomas.wuerthinger@oracle.com>
date Thu, 22 Jan 2015 16:25:04 +0100
parents fede93375dcb
children 6338dfaf6a44
files graal/com.oracle.graal.java/src/com/oracle/graal/java/GraphBuilderPhase.java graal/com.oracle.graal.java/src/com/oracle/graal/java/HIRFrameStateBuilder.java
diffstat 2 files changed, 0 insertions(+), 10 deletions(-) [+]
line wrap: on
line diff
--- a/graal/com.oracle.graal.java/src/com/oracle/graal/java/GraphBuilderPhase.java	Thu Jan 22 16:24:36 2015 +0100
+++ b/graal/com.oracle.graal.java/src/com/oracle/graal/java/GraphBuilderPhase.java	Thu Jan 22 16:25:04 2015 +0100
@@ -1150,7 +1150,6 @@
                     parser.setCurrentFrameState(frameState);
                     currentBlock = block;
 
-                    frameState.cleanupDeletedPhis();
                     if (lastInstr instanceof MergeNode) {
                         int bci = block.startBci;
                         if (block instanceof ExceptionDispatchBlock) {
--- a/graal/com.oracle.graal.java/src/com/oracle/graal/java/HIRFrameStateBuilder.java	Thu Jan 22 16:24:36 2015 +0100
+++ b/graal/com.oracle.graal.java/src/com/oracle/graal/java/HIRFrameStateBuilder.java	Thu Jan 22 16:25:04 2015 +0100
@@ -326,15 +326,6 @@
         return phi;
     }
 
-    public void cleanupDeletedPhis() {
-        for (int i = 0; i < localsSize(); i++) {
-            if (localAt(i) != null && localAt(i).isDeleted()) {
-                assert localAt(i) instanceof ValuePhiNode || localAt(i) instanceof ProxyNode : "Only phi and value proxies can be deleted during parsing: " + localAt(i);
-                storeLocal(i, null);
-            }
-        }
-    }
-
     /**
      * Adds a locked monitor to this frame state.
      *