annotate graal/com.oracle.max.graal.compiler/src/com/oracle/max/graal/compiler/phases/DeadCodeEliminationPhase.java @ 2886:c3b8968233fa

Removed counting of deleted nodes for each phase.
author Thomas Wuerthinger <thomas@wuerthinger.net>
date Wed, 08 Jun 2011 12:10:57 +0200
parents 7f584bf507ed
children 224412c24426
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;
2868
6d24c27902a2 turned inlining into a phase, some node cloning fixes, added NodeWorklist
Lukas Stadler <lukas.stadler@jku.at>
parents: 2867
diff changeset
34 private NodeWorklist worklist;
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 @Override
Lukas Stadler <lukas.stadler@jku.at>
parents:
diff changeset
38 protected void run(Graph graph) {
Lukas Stadler <lukas.stadler@jku.at>
parents:
diff changeset
39 this.graph = graph;
Lukas Stadler <lukas.stadler@jku.at>
parents:
diff changeset
40 this.alive = graph.createNodeBitMap();
2868
6d24c27902a2 turned inlining into a phase, some node cloning fixes, added NodeWorklist
Lukas Stadler <lukas.stadler@jku.at>
parents: 2867
diff changeset
41 this.worklist = graph.createNodeWorklist();
2867
Lukas Stadler <lukas.stadler@jku.at>
parents:
diff changeset
42
2868
6d24c27902a2 turned inlining into a phase, some node cloning fixes, added NodeWorklist
Lukas Stadler <lukas.stadler@jku.at>
parents: 2867
diff changeset
43 worklist.add(graph.start());
2867
Lukas Stadler <lukas.stadler@jku.at>
parents:
diff changeset
44
Lukas Stadler <lukas.stadler@jku.at>
parents:
diff changeset
45 iterateSuccessors();
Lukas Stadler <lukas.stadler@jku.at>
parents:
diff changeset
46 disconnectCFGNodes();
Lukas Stadler <lukas.stadler@jku.at>
parents:
diff changeset
47
Lukas Stadler <lukas.stadler@jku.at>
parents:
diff changeset
48 iterateInputs();
Lukas Stadler <lukas.stadler@jku.at>
parents:
diff changeset
49 disconnectNonCFGNodes();
Lukas Stadler <lukas.stadler@jku.at>
parents:
diff changeset
50
Lukas Stadler <lukas.stadler@jku.at>
parents:
diff changeset
51 deleteCFGNodes();
Lukas Stadler <lukas.stadler@jku.at>
parents:
diff changeset
52 deleteNonCFGNodes();
Lukas Stadler <lukas.stadler@jku.at>
parents:
diff changeset
53
Lukas Stadler <lukas.stadler@jku.at>
parents:
diff changeset
54 new PhiSimplifier(graph);
Lukas Stadler <lukas.stadler@jku.at>
parents:
diff changeset
55
Lukas Stadler <lukas.stadler@jku.at>
parents:
diff changeset
56 if (C1XOptions.TraceDeadCodeElimination) {
2886
c3b8968233fa Removed counting of deleted nodes for each phase.
Thomas Wuerthinger <thomas@wuerthinger.net>
parents: 2879
diff changeset
57 System.out.printf("dead code elimination finished\n");
2867
Lukas Stadler <lukas.stadler@jku.at>
parents:
diff changeset
58 }
Lukas Stadler <lukas.stadler@jku.at>
parents:
diff changeset
59 }
Lukas Stadler <lukas.stadler@jku.at>
parents:
diff changeset
60
2868
6d24c27902a2 turned inlining into a phase, some node cloning fixes, added NodeWorklist
Lukas Stadler <lukas.stadler@jku.at>
parents: 2867
diff changeset
61 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
62 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
63 }
6d24c27902a2 turned inlining into a phase, some node cloning fixes, added NodeWorklist
Lukas Stadler <lukas.stadler@jku.at>
parents: 2867
diff changeset
64
2867
Lukas Stadler <lukas.stadler@jku.at>
parents:
diff changeset
65 private void iterateSuccessors() {
2868
6d24c27902a2 turned inlining into a phase, some node cloning fixes, added NodeWorklist
Lukas Stadler <lukas.stadler@jku.at>
parents: 2867
diff changeset
66 for (Node current : worklist) {
2867
Lukas Stadler <lukas.stadler@jku.at>
parents:
diff changeset
67 for (Node successor : current.successors()) {
2868
6d24c27902a2 turned inlining into a phase, some node cloning fixes, added NodeWorklist
Lukas Stadler <lukas.stadler@jku.at>
parents: 2867
diff changeset
68 worklist.add(successor);
2867
Lukas Stadler <lukas.stadler@jku.at>
parents:
diff changeset
69 }
Lukas Stadler <lukas.stadler@jku.at>
parents:
diff changeset
70 }
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 private void disconnectCFGNodes() {
Lukas Stadler <lukas.stadler@jku.at>
parents:
diff changeset
74 for (Node node : graph.getNodes()) {
2868
6d24c27902a2 turned inlining into a phase, some node cloning fixes, added NodeWorklist
Lukas Stadler <lukas.stadler@jku.at>
parents: 2867
diff changeset
75 if (node != Node.Null && !worklist.isMarked(node) && isCFG(node)) {
2867
Lukas Stadler <lukas.stadler@jku.at>
parents:
diff changeset
76 // iterate backwards so that the predecessor indexes in removePhiPredecessor are correct
Lukas Stadler <lukas.stadler@jku.at>
parents:
diff changeset
77 for (int i = node.successors().size() - 1; i >= 0; i--) {
Lukas Stadler <lukas.stadler@jku.at>
parents:
diff changeset
78 Node successor = node.successors().get(i);
2868
6d24c27902a2 turned inlining into a phase, some node cloning fixes, added NodeWorklist
Lukas Stadler <lukas.stadler@jku.at>
parents: 2867
diff changeset
79 if (successor != Node.Null && worklist.isMarked(successor)) {
2867
Lukas Stadler <lukas.stadler@jku.at>
parents:
diff changeset
80 if (successor instanceof Merge) {
Lukas Stadler <lukas.stadler@jku.at>
parents:
diff changeset
81 ((Merge) successor).removePhiPredecessor(node);
Lukas Stadler <lukas.stadler@jku.at>
parents:
diff changeset
82 }
Lukas Stadler <lukas.stadler@jku.at>
parents:
diff changeset
83 }
Lukas Stadler <lukas.stadler@jku.at>
parents:
diff changeset
84 }
Lukas Stadler <lukas.stadler@jku.at>
parents:
diff changeset
85 node.successors().clearAll();
Lukas Stadler <lukas.stadler@jku.at>
parents:
diff changeset
86 node.inputs().clearAll();
Lukas Stadler <lukas.stadler@jku.at>
parents:
diff changeset
87 }
Lukas Stadler <lukas.stadler@jku.at>
parents:
diff changeset
88 }
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 private void deleteCFGNodes() {
Lukas Stadler <lukas.stadler@jku.at>
parents:
diff changeset
92 for (Node node : graph.getNodes()) {
2868
6d24c27902a2 turned inlining into a phase, some node cloning fixes, added NodeWorklist
Lukas Stadler <lukas.stadler@jku.at>
parents: 2867
diff changeset
93 if (node != Node.Null && !worklist.isMarked(node) && isCFG(node)) {
2867
Lukas Stadler <lukas.stadler@jku.at>
parents:
diff changeset
94 node.delete();
Lukas Stadler <lukas.stadler@jku.at>
parents:
diff changeset
95 }
Lukas Stadler <lukas.stadler@jku.at>
parents:
diff changeset
96 }
Lukas Stadler <lukas.stadler@jku.at>
parents:
diff changeset
97 }
Lukas Stadler <lukas.stadler@jku.at>
parents:
diff changeset
98
Lukas Stadler <lukas.stadler@jku.at>
parents:
diff changeset
99 private void iterateInputs() {
Lukas Stadler <lukas.stadler@jku.at>
parents:
diff changeset
100 for (Node node : graph.getNodes()) {
2868
6d24c27902a2 turned inlining into a phase, some node cloning fixes, added NodeWorklist
Lukas Stadler <lukas.stadler@jku.at>
parents: 2867
diff changeset
101 if (node != Node.Null && worklist.isMarked(node)) {
2867
Lukas Stadler <lukas.stadler@jku.at>
parents:
diff changeset
102 for (Node input : node.inputs()) {
2868
6d24c27902a2 turned inlining into a phase, some node cloning fixes, added NodeWorklist
Lukas Stadler <lukas.stadler@jku.at>
parents: 2867
diff changeset
103 worklist.add(input);
2867
Lukas Stadler <lukas.stadler@jku.at>
parents:
diff changeset
104 }
Lukas Stadler <lukas.stadler@jku.at>
parents:
diff changeset
105 }
Lukas Stadler <lukas.stadler@jku.at>
parents:
diff changeset
106 }
2868
6d24c27902a2 turned inlining into a phase, some node cloning fixes, added NodeWorklist
Lukas Stadler <lukas.stadler@jku.at>
parents: 2867
diff changeset
107 for (Node current : worklist) {
2867
Lukas Stadler <lukas.stadler@jku.at>
parents:
diff changeset
108 for (Node input : current.inputs()) {
2868
6d24c27902a2 turned inlining into a phase, some node cloning fixes, added NodeWorklist
Lukas Stadler <lukas.stadler@jku.at>
parents: 2867
diff changeset
109 worklist.add(input);
2867
Lukas Stadler <lukas.stadler@jku.at>
parents:
diff changeset
110 }
Lukas Stadler <lukas.stadler@jku.at>
parents:
diff changeset
111 }
Lukas Stadler <lukas.stadler@jku.at>
parents:
diff changeset
112 }
Lukas Stadler <lukas.stadler@jku.at>
parents:
diff changeset
113
Lukas Stadler <lukas.stadler@jku.at>
parents:
diff changeset
114 private void disconnectNonCFGNodes() {
Lukas Stadler <lukas.stadler@jku.at>
parents:
diff changeset
115 for (Node node : graph.getNodes()) {
2868
6d24c27902a2 turned inlining into a phase, some node cloning fixes, added NodeWorklist
Lukas Stadler <lukas.stadler@jku.at>
parents: 2867
diff changeset
116 if (node != Node.Null && !worklist.isMarked(node) && !isCFG(node)) {
2867
Lukas Stadler <lukas.stadler@jku.at>
parents:
diff changeset
117 node.inputs().clearAll();
Lukas Stadler <lukas.stadler@jku.at>
parents:
diff changeset
118 }
Lukas Stadler <lukas.stadler@jku.at>
parents:
diff changeset
119 }
Lukas Stadler <lukas.stadler@jku.at>
parents:
diff changeset
120 }
Lukas Stadler <lukas.stadler@jku.at>
parents:
diff changeset
121
Lukas Stadler <lukas.stadler@jku.at>
parents:
diff changeset
122 private void deleteNonCFGNodes() {
Lukas Stadler <lukas.stadler@jku.at>
parents:
diff changeset
123 for (Node node : graph.getNodes()) {
2868
6d24c27902a2 turned inlining into a phase, some node cloning fixes, added NodeWorklist
Lukas Stadler <lukas.stadler@jku.at>
parents: 2867
diff changeset
124 if (node != Node.Null && !worklist.isMarked(node) && !isCFG(node)) {
2867
Lukas Stadler <lukas.stadler@jku.at>
parents:
diff changeset
125 node.delete();
Lukas Stadler <lukas.stadler@jku.at>
parents:
diff changeset
126 }
Lukas Stadler <lukas.stadler@jku.at>
parents:
diff changeset
127 }
Lukas Stadler <lukas.stadler@jku.at>
parents:
diff changeset
128 }
Lukas Stadler <lukas.stadler@jku.at>
parents:
diff changeset
129 }