annotate graal/com.oracle.max.graal.compiler/src/com/oracle/max/graal/compiler/phases/DeadCodeEliminationPhase.java @ 2910:7eec76f3e39e

adjust monitor index while inlining, renamed NodeWorklist to NodeFlood
author Lukas Stadler <lukas.stadler@jku.at>
date Wed, 08 Jun 2011 14:30:27 +0200
parents 7f584bf507ed
children 241798fa9d34
Ignore whitespace changes - Everywhere: Within whitespace: At end of lines:
rev   line source
2867
Lukas Stadler <lukas.stadler@jku.at>
parents:
diff changeset
1 /*
Lukas Stadler <lukas.stadler@jku.at>
parents:
diff changeset
2 * Copyright (c) 2011, Oracle and/or its affiliates. All rights reserved.
Lukas Stadler <lukas.stadler@jku.at>
parents:
diff changeset
3 * DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS FILE HEADER.
Lukas Stadler <lukas.stadler@jku.at>
parents:
diff changeset
4 *
Lukas Stadler <lukas.stadler@jku.at>
parents:
diff changeset
5 * This code is free software; you can redistribute it and/or modify it
Lukas Stadler <lukas.stadler@jku.at>
parents:
diff changeset
6 * under the terms of the GNU General Public License version 2 only, as
Lukas Stadler <lukas.stadler@jku.at>
parents:
diff changeset
7 * published by the Free Software Foundation.
Lukas Stadler <lukas.stadler@jku.at>
parents:
diff changeset
8 *
Lukas Stadler <lukas.stadler@jku.at>
parents:
diff changeset
9 * This code is distributed in the hope that it will be useful, but WITHOUT
Lukas Stadler <lukas.stadler@jku.at>
parents:
diff changeset
10 * ANY WARRANTY; without even the implied warranty of MERCHANTABILITY or
Lukas Stadler <lukas.stadler@jku.at>
parents:
diff changeset
11 * FITNESS FOR A PARTICULAR PURPOSE. See the GNU General Public License
Lukas Stadler <lukas.stadler@jku.at>
parents:
diff changeset
12 * version 2 for more details (a copy is included in the LICENSE file that
Lukas Stadler <lukas.stadler@jku.at>
parents:
diff changeset
13 * accompanied this code).
Lukas Stadler <lukas.stadler@jku.at>
parents:
diff changeset
14 *
Lukas Stadler <lukas.stadler@jku.at>
parents:
diff changeset
15 * You should have received a copy of the GNU General Public License version
Lukas Stadler <lukas.stadler@jku.at>
parents:
diff changeset
16 * 2 along with this work; if not, write to the Free Software Foundation,
Lukas Stadler <lukas.stadler@jku.at>
parents:
diff changeset
17 * Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301 USA.
Lukas Stadler <lukas.stadler@jku.at>
parents:
diff changeset
18 *
Lukas Stadler <lukas.stadler@jku.at>
parents:
diff changeset
19 * Please contact Oracle, 500 Oracle Parkway, Redwood Shores, CA 94065 USA
Lukas Stadler <lukas.stadler@jku.at>
parents:
diff changeset
20 * or visit www.oracle.com if you need additional information or have any
Lukas Stadler <lukas.stadler@jku.at>
parents:
diff changeset
21 * questions.
Lukas Stadler <lukas.stadler@jku.at>
parents:
diff changeset
22 */
2879
7f584bf507ed Renamed and moved phase subclasses.
Thomas Wuerthinger <thomas@wuerthinger.net>
parents: 2874
diff changeset
23 package com.oracle.max.graal.compiler.phases;
2867
Lukas Stadler <lukas.stadler@jku.at>
parents:
diff changeset
24
2874
d90bf514d647 Renamed packages.
Thomas Wuerthinger <thomas@wuerthinger.net>
parents: 2872
diff changeset
25 import com.oracle.max.graal.compiler.*;
d90bf514d647 Renamed packages.
Thomas Wuerthinger <thomas@wuerthinger.net>
parents: 2872
diff changeset
26 import com.oracle.max.graal.compiler.gen.*;
d90bf514d647 Renamed packages.
Thomas Wuerthinger <thomas@wuerthinger.net>
parents: 2872
diff changeset
27 import com.oracle.max.graal.compiler.ir.*;
d90bf514d647 Renamed packages.
Thomas Wuerthinger <thomas@wuerthinger.net>
parents: 2872
diff changeset
28 import com.oracle.max.graal.graph.*;
2867
Lukas Stadler <lukas.stadler@jku.at>
parents:
diff changeset
29
Lukas Stadler <lukas.stadler@jku.at>
parents:
diff changeset
30
2879
7f584bf507ed Renamed and moved phase subclasses.
Thomas Wuerthinger <thomas@wuerthinger.net>
parents: 2874
diff changeset
31 public class DeadCodeEliminationPhase extends Phase {
2867
Lukas Stadler <lukas.stadler@jku.at>
parents:
diff changeset
32
Lukas Stadler <lukas.stadler@jku.at>
parents:
diff changeset
33 private NodeBitMap alive;
2910
7eec76f3e39e adjust monitor index while inlining, renamed NodeWorklist to NodeFlood
Lukas Stadler <lukas.stadler@jku.at>
parents: 2879
diff changeset
34 private NodeFlood flood;
2867
Lukas Stadler <lukas.stadler@jku.at>
parents:
diff changeset
35 private Graph graph;
Lukas Stadler <lukas.stadler@jku.at>
parents:
diff changeset
36
Lukas Stadler <lukas.stadler@jku.at>
parents:
diff changeset
37 public int deletedNodeCount;
Lukas Stadler <lukas.stadler@jku.at>
parents:
diff changeset
38
Lukas Stadler <lukas.stadler@jku.at>
parents:
diff changeset
39 @Override
Lukas Stadler <lukas.stadler@jku.at>
parents:
diff changeset
40 protected void run(Graph graph) {
Lukas Stadler <lukas.stadler@jku.at>
parents:
diff changeset
41 this.graph = graph;
Lukas Stadler <lukas.stadler@jku.at>
parents:
diff changeset
42 this.alive = graph.createNodeBitMap();
2910
7eec76f3e39e adjust monitor index while inlining, renamed NodeWorklist to NodeFlood
Lukas Stadler <lukas.stadler@jku.at>
parents: 2879
diff changeset
43 this.flood = graph.createNodeFlood();
2867
Lukas Stadler <lukas.stadler@jku.at>
parents:
diff changeset
44
2910
7eec76f3e39e adjust monitor index while inlining, renamed NodeWorklist to NodeFlood
Lukas Stadler <lukas.stadler@jku.at>
parents: 2879
diff changeset
45 flood.add(graph.start());
2867
Lukas Stadler <lukas.stadler@jku.at>
parents:
diff changeset
46
Lukas Stadler <lukas.stadler@jku.at>
parents:
diff changeset
47 iterateSuccessors();
Lukas Stadler <lukas.stadler@jku.at>
parents:
diff changeset
48 disconnectCFGNodes();
Lukas Stadler <lukas.stadler@jku.at>
parents:
diff changeset
49
Lukas Stadler <lukas.stadler@jku.at>
parents:
diff changeset
50 iterateInputs();
Lukas Stadler <lukas.stadler@jku.at>
parents:
diff changeset
51 disconnectNonCFGNodes();
Lukas Stadler <lukas.stadler@jku.at>
parents:
diff changeset
52
Lukas Stadler <lukas.stadler@jku.at>
parents:
diff changeset
53 deleteCFGNodes();
Lukas Stadler <lukas.stadler@jku.at>
parents:
diff changeset
54 deleteNonCFGNodes();
Lukas Stadler <lukas.stadler@jku.at>
parents:
diff changeset
55
Lukas Stadler <lukas.stadler@jku.at>
parents:
diff changeset
56 new PhiSimplifier(graph);
Lukas Stadler <lukas.stadler@jku.at>
parents:
diff changeset
57
Lukas Stadler <lukas.stadler@jku.at>
parents:
diff changeset
58 if (C1XOptions.TraceDeadCodeElimination) {
Lukas Stadler <lukas.stadler@jku.at>
parents:
diff changeset
59 System.out.printf("dead code elimination: deleted %d nodes\n", deletedNodeCount);
Lukas Stadler <lukas.stadler@jku.at>
parents:
diff changeset
60 }
Lukas Stadler <lukas.stadler@jku.at>
parents:
diff changeset
61 }
Lukas Stadler <lukas.stadler@jku.at>
parents:
diff changeset
62
2868
6d24c27902a2 turned inlining into a phase, some node cloning fixes, added NodeWorklist
Lukas Stadler <lukas.stadler@jku.at>
parents: 2867
diff changeset
63 private static boolean isCFG(Node n) {
6d24c27902a2 turned inlining into a phase, some node cloning fixes, added NodeWorklist
Lukas Stadler <lukas.stadler@jku.at>
parents: 2867
diff changeset
64 return n != null && ((n instanceof Instruction) || n == n.graph().start());
6d24c27902a2 turned inlining into a phase, some node cloning fixes, added NodeWorklist
Lukas Stadler <lukas.stadler@jku.at>
parents: 2867
diff changeset
65 }
6d24c27902a2 turned inlining into a phase, some node cloning fixes, added NodeWorklist
Lukas Stadler <lukas.stadler@jku.at>
parents: 2867
diff changeset
66
2867
Lukas Stadler <lukas.stadler@jku.at>
parents:
diff changeset
67 private void iterateSuccessors() {
2910
7eec76f3e39e adjust monitor index while inlining, renamed NodeWorklist to NodeFlood
Lukas Stadler <lukas.stadler@jku.at>
parents: 2879
diff changeset
68 for (Node current : flood) {
2867
Lukas Stadler <lukas.stadler@jku.at>
parents:
diff changeset
69 for (Node successor : current.successors()) {
2910
7eec76f3e39e adjust monitor index while inlining, renamed NodeWorklist to NodeFlood
Lukas Stadler <lukas.stadler@jku.at>
parents: 2879
diff changeset
70 flood.add(successor);
2867
Lukas Stadler <lukas.stadler@jku.at>
parents:
diff changeset
71 }
Lukas Stadler <lukas.stadler@jku.at>
parents:
diff changeset
72 }
Lukas Stadler <lukas.stadler@jku.at>
parents:
diff changeset
73 }
Lukas Stadler <lukas.stadler@jku.at>
parents:
diff changeset
74
Lukas Stadler <lukas.stadler@jku.at>
parents:
diff changeset
75 private void disconnectCFGNodes() {
Lukas Stadler <lukas.stadler@jku.at>
parents:
diff changeset
76 for (Node node : graph.getNodes()) {
2910
7eec76f3e39e adjust monitor index while inlining, renamed NodeWorklist to NodeFlood
Lukas Stadler <lukas.stadler@jku.at>
parents: 2879
diff changeset
77 if (node != Node.Null && !flood.isMarked(node) && isCFG(node)) {
2867
Lukas Stadler <lukas.stadler@jku.at>
parents:
diff changeset
78 // iterate backwards so that the predecessor indexes in removePhiPredecessor are correct
Lukas Stadler <lukas.stadler@jku.at>
parents:
diff changeset
79 for (int i = node.successors().size() - 1; i >= 0; i--) {
Lukas Stadler <lukas.stadler@jku.at>
parents:
diff changeset
80 Node successor = node.successors().get(i);
2910
7eec76f3e39e adjust monitor index while inlining, renamed NodeWorklist to NodeFlood
Lukas Stadler <lukas.stadler@jku.at>
parents: 2879
diff changeset
81 if (successor != Node.Null && flood.isMarked(successor)) {
2867
Lukas Stadler <lukas.stadler@jku.at>
parents:
diff changeset
82 if (successor instanceof Merge) {
Lukas Stadler <lukas.stadler@jku.at>
parents:
diff changeset
83 ((Merge) successor).removePhiPredecessor(node);
Lukas Stadler <lukas.stadler@jku.at>
parents:
diff changeset
84 }
Lukas Stadler <lukas.stadler@jku.at>
parents:
diff changeset
85 }
Lukas Stadler <lukas.stadler@jku.at>
parents:
diff changeset
86 }
Lukas Stadler <lukas.stadler@jku.at>
parents:
diff changeset
87 node.successors().clearAll();
Lukas Stadler <lukas.stadler@jku.at>
parents:
diff changeset
88 node.inputs().clearAll();
Lukas Stadler <lukas.stadler@jku.at>
parents:
diff changeset
89 }
Lukas Stadler <lukas.stadler@jku.at>
parents:
diff changeset
90 }
Lukas Stadler <lukas.stadler@jku.at>
parents:
diff changeset
91 }
Lukas Stadler <lukas.stadler@jku.at>
parents:
diff changeset
92
Lukas Stadler <lukas.stadler@jku.at>
parents:
diff changeset
93 private void deleteCFGNodes() {
Lukas Stadler <lukas.stadler@jku.at>
parents:
diff changeset
94 for (Node node : graph.getNodes()) {
2910
7eec76f3e39e adjust monitor index while inlining, renamed NodeWorklist to NodeFlood
Lukas Stadler <lukas.stadler@jku.at>
parents: 2879
diff changeset
95 if (node != Node.Null && !flood.isMarked(node) && isCFG(node)) {
2867
Lukas Stadler <lukas.stadler@jku.at>
parents:
diff changeset
96 node.delete();
Lukas Stadler <lukas.stadler@jku.at>
parents:
diff changeset
97 deletedNodeCount++;
Lukas Stadler <lukas.stadler@jku.at>
parents:
diff changeset
98 }
Lukas Stadler <lukas.stadler@jku.at>
parents:
diff changeset
99 }
Lukas Stadler <lukas.stadler@jku.at>
parents:
diff changeset
100 }
Lukas Stadler <lukas.stadler@jku.at>
parents:
diff changeset
101
Lukas Stadler <lukas.stadler@jku.at>
parents:
diff changeset
102 private void iterateInputs() {
Lukas Stadler <lukas.stadler@jku.at>
parents:
diff changeset
103 for (Node node : graph.getNodes()) {
2910
7eec76f3e39e adjust monitor index while inlining, renamed NodeWorklist to NodeFlood
Lukas Stadler <lukas.stadler@jku.at>
parents: 2879
diff changeset
104 if (node != Node.Null && flood.isMarked(node)) {
2867
Lukas Stadler <lukas.stadler@jku.at>
parents:
diff changeset
105 for (Node input : node.inputs()) {
2910
7eec76f3e39e adjust monitor index while inlining, renamed NodeWorklist to NodeFlood
Lukas Stadler <lukas.stadler@jku.at>
parents: 2879
diff changeset
106 flood.add(input);
2867
Lukas Stadler <lukas.stadler@jku.at>
parents:
diff changeset
107 }
Lukas Stadler <lukas.stadler@jku.at>
parents:
diff changeset
108 }
Lukas Stadler <lukas.stadler@jku.at>
parents:
diff changeset
109 }
2910
7eec76f3e39e adjust monitor index while inlining, renamed NodeWorklist to NodeFlood
Lukas Stadler <lukas.stadler@jku.at>
parents: 2879
diff changeset
110 for (Node current : flood) {
2867
Lukas Stadler <lukas.stadler@jku.at>
parents:
diff changeset
111 for (Node input : current.inputs()) {
2910
7eec76f3e39e adjust monitor index while inlining, renamed NodeWorklist to NodeFlood
Lukas Stadler <lukas.stadler@jku.at>
parents: 2879
diff changeset
112 flood.add(input);
2867
Lukas Stadler <lukas.stadler@jku.at>
parents:
diff changeset
113 }
Lukas Stadler <lukas.stadler@jku.at>
parents:
diff changeset
114 }
Lukas Stadler <lukas.stadler@jku.at>
parents:
diff changeset
115 }
Lukas Stadler <lukas.stadler@jku.at>
parents:
diff changeset
116
Lukas Stadler <lukas.stadler@jku.at>
parents:
diff changeset
117 private void disconnectNonCFGNodes() {
Lukas Stadler <lukas.stadler@jku.at>
parents:
diff changeset
118 for (Node node : graph.getNodes()) {
2910
7eec76f3e39e adjust monitor index while inlining, renamed NodeWorklist to NodeFlood
Lukas Stadler <lukas.stadler@jku.at>
parents: 2879
diff changeset
119 if (node != Node.Null && !flood.isMarked(node) && !isCFG(node)) {
2867
Lukas Stadler <lukas.stadler@jku.at>
parents:
diff changeset
120 node.inputs().clearAll();
Lukas Stadler <lukas.stadler@jku.at>
parents:
diff changeset
121 }
Lukas Stadler <lukas.stadler@jku.at>
parents:
diff changeset
122 }
Lukas Stadler <lukas.stadler@jku.at>
parents:
diff changeset
123 }
Lukas Stadler <lukas.stadler@jku.at>
parents:
diff changeset
124
Lukas Stadler <lukas.stadler@jku.at>
parents:
diff changeset
125 private void deleteNonCFGNodes() {
Lukas Stadler <lukas.stadler@jku.at>
parents:
diff changeset
126 for (Node node : graph.getNodes()) {
2910
7eec76f3e39e adjust monitor index while inlining, renamed NodeWorklist to NodeFlood
Lukas Stadler <lukas.stadler@jku.at>
parents: 2879
diff changeset
127 if (node != Node.Null && !flood.isMarked(node) && !isCFG(node)) {
2867
Lukas Stadler <lukas.stadler@jku.at>
parents:
diff changeset
128 node.delete();
Lukas Stadler <lukas.stadler@jku.at>
parents:
diff changeset
129 deletedNodeCount++;
Lukas Stadler <lukas.stadler@jku.at>
parents:
diff changeset
130 }
Lukas Stadler <lukas.stadler@jku.at>
parents:
diff changeset
131 }
Lukas Stadler <lukas.stadler@jku.at>
parents:
diff changeset
132 }
Lukas Stadler <lukas.stadler@jku.at>
parents:
diff changeset
133 }