annotate src/share/vm/opto/phaseX.cpp @ 23874:c42cb5db3601

Merge
author asaha
date Tue, 01 Mar 2016 15:19:31 -0800
parents 70649f10b88c 94ec11846b18
children a96cf90239c6
Ignore whitespace changes - Everywhere: Within whitespace: At end of lines:
rev   line source
0
a61af66fc99e Initial load
duke
parents:
diff changeset
1 /*
17937
78bbf4d43a14 8037816: Fix for 8036122 breaks build with Xcode5/clang
drchase
parents: 17671
diff changeset
2 * Copyright (c) 1997, 2014, Oracle and/or its affiliates. All rights reserved.
0
a61af66fc99e Initial load
duke
parents:
diff changeset
3 * DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS FILE HEADER.
a61af66fc99e Initial load
duke
parents:
diff changeset
4 *
a61af66fc99e Initial load
duke
parents:
diff changeset
5 * This code is free software; you can redistribute it and/or modify it
a61af66fc99e Initial load
duke
parents:
diff changeset
6 * under the terms of the GNU General Public License version 2 only, as
a61af66fc99e Initial load
duke
parents:
diff changeset
7 * published by the Free Software Foundation.
a61af66fc99e Initial load
duke
parents:
diff changeset
8 *
a61af66fc99e Initial load
duke
parents:
diff changeset
9 * This code is distributed in the hope that it will be useful, but WITHOUT
a61af66fc99e Initial load
duke
parents:
diff changeset
10 * ANY WARRANTY; without even the implied warranty of MERCHANTABILITY or
a61af66fc99e Initial load
duke
parents:
diff changeset
11 * FITNESS FOR A PARTICULAR PURPOSE. See the GNU General Public License
a61af66fc99e Initial load
duke
parents:
diff changeset
12 * version 2 for more details (a copy is included in the LICENSE file that
a61af66fc99e Initial load
duke
parents:
diff changeset
13 * accompanied this code).
a61af66fc99e Initial load
duke
parents:
diff changeset
14 *
a61af66fc99e Initial load
duke
parents:
diff changeset
15 * You should have received a copy of the GNU General Public License version
a61af66fc99e Initial load
duke
parents:
diff changeset
16 * 2 along with this work; if not, write to the Free Software Foundation,
a61af66fc99e Initial load
duke
parents:
diff changeset
17 * Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301 USA.
a61af66fc99e Initial load
duke
parents:
diff changeset
18 *
1552
c18cbe5936b8 6941466: Oracle rebranding changes for Hotspot repositories
trims
parents: 1017
diff changeset
19 * Please contact Oracle, 500 Oracle Parkway, Redwood Shores, CA 94065 USA
c18cbe5936b8 6941466: Oracle rebranding changes for Hotspot repositories
trims
parents: 1017
diff changeset
20 * or visit www.oracle.com if you need additional information or have any
c18cbe5936b8 6941466: Oracle rebranding changes for Hotspot repositories
trims
parents: 1017
diff changeset
21 * questions.
0
a61af66fc99e Initial load
duke
parents:
diff changeset
22 *
a61af66fc99e Initial load
duke
parents:
diff changeset
23 */
a61af66fc99e Initial load
duke
parents:
diff changeset
24
1972
f95d63e2154a 6989984: Use standard include model for Hospot
stefank
parents: 1826
diff changeset
25 #include "precompiled.hpp"
f95d63e2154a 6989984: Use standard include model for Hospot
stefank
parents: 1826
diff changeset
26 #include "memory/allocation.inline.hpp"
f95d63e2154a 6989984: Use standard include model for Hospot
stefank
parents: 1826
diff changeset
27 #include "opto/block.hpp"
f95d63e2154a 6989984: Use standard include model for Hospot
stefank
parents: 1826
diff changeset
28 #include "opto/callnode.hpp"
f95d63e2154a 6989984: Use standard include model for Hospot
stefank
parents: 1826
diff changeset
29 #include "opto/cfgnode.hpp"
f95d63e2154a 6989984: Use standard include model for Hospot
stefank
parents: 1826
diff changeset
30 #include "opto/connode.hpp"
f95d63e2154a 6989984: Use standard include model for Hospot
stefank
parents: 1826
diff changeset
31 #include "opto/idealGraphPrinter.hpp"
f95d63e2154a 6989984: Use standard include model for Hospot
stefank
parents: 1826
diff changeset
32 #include "opto/loopnode.hpp"
f95d63e2154a 6989984: Use standard include model for Hospot
stefank
parents: 1826
diff changeset
33 #include "opto/machnode.hpp"
f95d63e2154a 6989984: Use standard include model for Hospot
stefank
parents: 1826
diff changeset
34 #include "opto/opcodes.hpp"
f95d63e2154a 6989984: Use standard include model for Hospot
stefank
parents: 1826
diff changeset
35 #include "opto/phaseX.hpp"
f95d63e2154a 6989984: Use standard include model for Hospot
stefank
parents: 1826
diff changeset
36 #include "opto/regalloc.hpp"
f95d63e2154a 6989984: Use standard include model for Hospot
stefank
parents: 1826
diff changeset
37 #include "opto/rootnode.hpp"
0
a61af66fc99e Initial load
duke
parents:
diff changeset
38
a61af66fc99e Initial load
duke
parents:
diff changeset
39 //=============================================================================
a61af66fc99e Initial load
duke
parents:
diff changeset
40 #define NODE_HASH_MINIMUM_SIZE 255
a61af66fc99e Initial load
duke
parents:
diff changeset
41 //------------------------------NodeHash---------------------------------------
a61af66fc99e Initial load
duke
parents:
diff changeset
42 NodeHash::NodeHash(uint est_max_size) :
a61af66fc99e Initial load
duke
parents:
diff changeset
43 _max( round_up(est_max_size < NODE_HASH_MINIMUM_SIZE ? NODE_HASH_MINIMUM_SIZE : est_max_size) ),
a61af66fc99e Initial load
duke
parents:
diff changeset
44 _a(Thread::current()->resource_area()),
a61af66fc99e Initial load
duke
parents:
diff changeset
45 _table( NEW_ARENA_ARRAY( _a , Node* , _max ) ), // (Node**)_a->Amalloc(_max * sizeof(Node*)) ),
a61af66fc99e Initial load
duke
parents:
diff changeset
46 _inserts(0), _insert_limit( insert_limit() ),
a61af66fc99e Initial load
duke
parents:
diff changeset
47 _look_probes(0), _lookup_hits(0), _lookup_misses(0),
a61af66fc99e Initial load
duke
parents:
diff changeset
48 _total_insert_probes(0), _total_inserts(0),
a61af66fc99e Initial load
duke
parents:
diff changeset
49 _insert_probes(0), _grows(0) {
a61af66fc99e Initial load
duke
parents:
diff changeset
50 // _sentinel must be in the current node space
6804
e626685e9f6c 7193318: C2: remove number of inputs requirement from Node's new operator
kvn
parents: 6244
diff changeset
51 _sentinel = new (Compile::current()) ProjNode(NULL, TypeFunc::Control);
0
a61af66fc99e Initial load
duke
parents:
diff changeset
52 memset(_table,0,sizeof(Node*)*_max);
a61af66fc99e Initial load
duke
parents:
diff changeset
53 }
a61af66fc99e Initial load
duke
parents:
diff changeset
54
a61af66fc99e Initial load
duke
parents:
diff changeset
55 //------------------------------NodeHash---------------------------------------
a61af66fc99e Initial load
duke
parents:
diff changeset
56 NodeHash::NodeHash(Arena *arena, uint est_max_size) :
a61af66fc99e Initial load
duke
parents:
diff changeset
57 _max( round_up(est_max_size < NODE_HASH_MINIMUM_SIZE ? NODE_HASH_MINIMUM_SIZE : est_max_size) ),
a61af66fc99e Initial load
duke
parents:
diff changeset
58 _a(arena),
a61af66fc99e Initial load
duke
parents:
diff changeset
59 _table( NEW_ARENA_ARRAY( _a , Node* , _max ) ),
a61af66fc99e Initial load
duke
parents:
diff changeset
60 _inserts(0), _insert_limit( insert_limit() ),
a61af66fc99e Initial load
duke
parents:
diff changeset
61 _look_probes(0), _lookup_hits(0), _lookup_misses(0),
a61af66fc99e Initial load
duke
parents:
diff changeset
62 _delete_probes(0), _delete_hits(0), _delete_misses(0),
a61af66fc99e Initial load
duke
parents:
diff changeset
63 _total_insert_probes(0), _total_inserts(0),
a61af66fc99e Initial load
duke
parents:
diff changeset
64 _insert_probes(0), _grows(0) {
a61af66fc99e Initial load
duke
parents:
diff changeset
65 // _sentinel must be in the current node space
6804
e626685e9f6c 7193318: C2: remove number of inputs requirement from Node's new operator
kvn
parents: 6244
diff changeset
66 _sentinel = new (Compile::current()) ProjNode(NULL, TypeFunc::Control);
0
a61af66fc99e Initial load
duke
parents:
diff changeset
67 memset(_table,0,sizeof(Node*)*_max);
a61af66fc99e Initial load
duke
parents:
diff changeset
68 }
a61af66fc99e Initial load
duke
parents:
diff changeset
69
a61af66fc99e Initial load
duke
parents:
diff changeset
70 //------------------------------NodeHash---------------------------------------
a61af66fc99e Initial load
duke
parents:
diff changeset
71 NodeHash::NodeHash(NodeHash *nh) {
a61af66fc99e Initial load
duke
parents:
diff changeset
72 debug_only(_table = (Node**)badAddress); // interact correctly w/ operator=
a61af66fc99e Initial load
duke
parents:
diff changeset
73 // just copy in all the fields
a61af66fc99e Initial load
duke
parents:
diff changeset
74 *this = *nh;
a61af66fc99e Initial load
duke
parents:
diff changeset
75 // nh->_sentinel must be in the current node space
a61af66fc99e Initial load
duke
parents:
diff changeset
76 }
a61af66fc99e Initial load
duke
parents:
diff changeset
77
7473
d092d1b31229 8005071: Incremental inlining for JSR 292
roland
parents: 7196
diff changeset
78 void NodeHash::replace_with(NodeHash *nh) {
d092d1b31229 8005071: Incremental inlining for JSR 292
roland
parents: 7196
diff changeset
79 debug_only(_table = (Node**)badAddress); // interact correctly w/ operator=
d092d1b31229 8005071: Incremental inlining for JSR 292
roland
parents: 7196
diff changeset
80 // just copy in all the fields
d092d1b31229 8005071: Incremental inlining for JSR 292
roland
parents: 7196
diff changeset
81 *this = *nh;
d092d1b31229 8005071: Incremental inlining for JSR 292
roland
parents: 7196
diff changeset
82 // nh->_sentinel must be in the current node space
d092d1b31229 8005071: Incremental inlining for JSR 292
roland
parents: 7196
diff changeset
83 }
d092d1b31229 8005071: Incremental inlining for JSR 292
roland
parents: 7196
diff changeset
84
0
a61af66fc99e Initial load
duke
parents:
diff changeset
85 //------------------------------hash_find--------------------------------------
a61af66fc99e Initial load
duke
parents:
diff changeset
86 // Find in hash table
a61af66fc99e Initial load
duke
parents:
diff changeset
87 Node *NodeHash::hash_find( const Node *n ) {
a61af66fc99e Initial load
duke
parents:
diff changeset
88 // ((Node*)n)->set_hash( n->hash() );
a61af66fc99e Initial load
duke
parents:
diff changeset
89 uint hash = n->hash();
a61af66fc99e Initial load
duke
parents:
diff changeset
90 if (hash == Node::NO_HASH) {
a61af66fc99e Initial load
duke
parents:
diff changeset
91 debug_only( _lookup_misses++ );
a61af66fc99e Initial load
duke
parents:
diff changeset
92 return NULL;
a61af66fc99e Initial load
duke
parents:
diff changeset
93 }
a61af66fc99e Initial load
duke
parents:
diff changeset
94 uint key = hash & (_max-1);
a61af66fc99e Initial load
duke
parents:
diff changeset
95 uint stride = key | 0x01;
a61af66fc99e Initial load
duke
parents:
diff changeset
96 debug_only( _look_probes++ );
a61af66fc99e Initial load
duke
parents:
diff changeset
97 Node *k = _table[key]; // Get hashed value
a61af66fc99e Initial load
duke
parents:
diff changeset
98 if( !k ) { // ?Miss?
a61af66fc99e Initial load
duke
parents:
diff changeset
99 debug_only( _lookup_misses++ );
a61af66fc99e Initial load
duke
parents:
diff changeset
100 return NULL; // Miss!
a61af66fc99e Initial load
duke
parents:
diff changeset
101 }
a61af66fc99e Initial load
duke
parents:
diff changeset
102
a61af66fc99e Initial load
duke
parents:
diff changeset
103 int op = n->Opcode();
a61af66fc99e Initial load
duke
parents:
diff changeset
104 uint req = n->req();
a61af66fc99e Initial load
duke
parents:
diff changeset
105 while( 1 ) { // While probing hash table
a61af66fc99e Initial load
duke
parents:
diff changeset
106 if( k->req() == req && // Same count of inputs
a61af66fc99e Initial load
duke
parents:
diff changeset
107 k->Opcode() == op ) { // Same Opcode
a61af66fc99e Initial load
duke
parents:
diff changeset
108 for( uint i=0; i<req; i++ )
a61af66fc99e Initial load
duke
parents:
diff changeset
109 if( n->in(i)!=k->in(i)) // Different inputs?
a61af66fc99e Initial load
duke
parents:
diff changeset
110 goto collision; // "goto" is a speed hack...
a61af66fc99e Initial load
duke
parents:
diff changeset
111 if( n->cmp(*k) ) { // Check for any special bits
a61af66fc99e Initial load
duke
parents:
diff changeset
112 debug_only( _lookup_hits++ );
a61af66fc99e Initial load
duke
parents:
diff changeset
113 return k; // Hit!
a61af66fc99e Initial load
duke
parents:
diff changeset
114 }
a61af66fc99e Initial load
duke
parents:
diff changeset
115 }
a61af66fc99e Initial load
duke
parents:
diff changeset
116 collision:
a61af66fc99e Initial load
duke
parents:
diff changeset
117 debug_only( _look_probes++ );
a61af66fc99e Initial load
duke
parents:
diff changeset
118 key = (key + stride/*7*/) & (_max-1); // Stride through table with relative prime
a61af66fc99e Initial load
duke
parents:
diff changeset
119 k = _table[key]; // Get hashed value
a61af66fc99e Initial load
duke
parents:
diff changeset
120 if( !k ) { // ?Miss?
a61af66fc99e Initial load
duke
parents:
diff changeset
121 debug_only( _lookup_misses++ );
a61af66fc99e Initial load
duke
parents:
diff changeset
122 return NULL; // Miss!
a61af66fc99e Initial load
duke
parents:
diff changeset
123 }
a61af66fc99e Initial load
duke
parents:
diff changeset
124 }
a61af66fc99e Initial load
duke
parents:
diff changeset
125 ShouldNotReachHere();
a61af66fc99e Initial load
duke
parents:
diff changeset
126 return NULL;
a61af66fc99e Initial load
duke
parents:
diff changeset
127 }
a61af66fc99e Initial load
duke
parents:
diff changeset
128
a61af66fc99e Initial load
duke
parents:
diff changeset
129 //------------------------------hash_find_insert-------------------------------
a61af66fc99e Initial load
duke
parents:
diff changeset
130 // Find in hash table, insert if not already present
a61af66fc99e Initial load
duke
parents:
diff changeset
131 // Used to preserve unique entries in hash table
a61af66fc99e Initial load
duke
parents:
diff changeset
132 Node *NodeHash::hash_find_insert( Node *n ) {
a61af66fc99e Initial load
duke
parents:
diff changeset
133 // n->set_hash( );
a61af66fc99e Initial load
duke
parents:
diff changeset
134 uint hash = n->hash();
a61af66fc99e Initial load
duke
parents:
diff changeset
135 if (hash == Node::NO_HASH) {
a61af66fc99e Initial load
duke
parents:
diff changeset
136 debug_only( _lookup_misses++ );
a61af66fc99e Initial load
duke
parents:
diff changeset
137 return NULL;
a61af66fc99e Initial load
duke
parents:
diff changeset
138 }
a61af66fc99e Initial load
duke
parents:
diff changeset
139 uint key = hash & (_max-1);
a61af66fc99e Initial load
duke
parents:
diff changeset
140 uint stride = key | 0x01; // stride must be relatively prime to table siz
a61af66fc99e Initial load
duke
parents:
diff changeset
141 uint first_sentinel = 0; // replace a sentinel if seen.
a61af66fc99e Initial load
duke
parents:
diff changeset
142 debug_only( _look_probes++ );
a61af66fc99e Initial load
duke
parents:
diff changeset
143 Node *k = _table[key]; // Get hashed value
a61af66fc99e Initial load
duke
parents:
diff changeset
144 if( !k ) { // ?Miss?
a61af66fc99e Initial load
duke
parents:
diff changeset
145 debug_only( _lookup_misses++ );
a61af66fc99e Initial load
duke
parents:
diff changeset
146 _table[key] = n; // Insert into table!
a61af66fc99e Initial load
duke
parents:
diff changeset
147 debug_only(n->enter_hash_lock()); // Lock down the node while in the table.
a61af66fc99e Initial load
duke
parents:
diff changeset
148 check_grow(); // Grow table if insert hit limit
a61af66fc99e Initial load
duke
parents:
diff changeset
149 return NULL; // Miss!
a61af66fc99e Initial load
duke
parents:
diff changeset
150 }
a61af66fc99e Initial load
duke
parents:
diff changeset
151 else if( k == _sentinel ) {
a61af66fc99e Initial load
duke
parents:
diff changeset
152 first_sentinel = key; // Can insert here
a61af66fc99e Initial load
duke
parents:
diff changeset
153 }
a61af66fc99e Initial load
duke
parents:
diff changeset
154
a61af66fc99e Initial load
duke
parents:
diff changeset
155 int op = n->Opcode();
a61af66fc99e Initial load
duke
parents:
diff changeset
156 uint req = n->req();
a61af66fc99e Initial load
duke
parents:
diff changeset
157 while( 1 ) { // While probing hash table
a61af66fc99e Initial load
duke
parents:
diff changeset
158 if( k->req() == req && // Same count of inputs
a61af66fc99e Initial load
duke
parents:
diff changeset
159 k->Opcode() == op ) { // Same Opcode
a61af66fc99e Initial load
duke
parents:
diff changeset
160 for( uint i=0; i<req; i++ )
a61af66fc99e Initial load
duke
parents:
diff changeset
161 if( n->in(i)!=k->in(i)) // Different inputs?
a61af66fc99e Initial load
duke
parents:
diff changeset
162 goto collision; // "goto" is a speed hack...
a61af66fc99e Initial load
duke
parents:
diff changeset
163 if( n->cmp(*k) ) { // Check for any special bits
a61af66fc99e Initial load
duke
parents:
diff changeset
164 debug_only( _lookup_hits++ );
a61af66fc99e Initial load
duke
parents:
diff changeset
165 return k; // Hit!
a61af66fc99e Initial load
duke
parents:
diff changeset
166 }
a61af66fc99e Initial load
duke
parents:
diff changeset
167 }
a61af66fc99e Initial load
duke
parents:
diff changeset
168 collision:
a61af66fc99e Initial load
duke
parents:
diff changeset
169 debug_only( _look_probes++ );
a61af66fc99e Initial load
duke
parents:
diff changeset
170 key = (key + stride) & (_max-1); // Stride through table w/ relative prime
a61af66fc99e Initial load
duke
parents:
diff changeset
171 k = _table[key]; // Get hashed value
a61af66fc99e Initial load
duke
parents:
diff changeset
172 if( !k ) { // ?Miss?
a61af66fc99e Initial load
duke
parents:
diff changeset
173 debug_only( _lookup_misses++ );
a61af66fc99e Initial load
duke
parents:
diff changeset
174 key = (first_sentinel == 0) ? key : first_sentinel; // ?saw sentinel?
a61af66fc99e Initial load
duke
parents:
diff changeset
175 _table[key] = n; // Insert into table!
a61af66fc99e Initial load
duke
parents:
diff changeset
176 debug_only(n->enter_hash_lock()); // Lock down the node while in the table.
a61af66fc99e Initial load
duke
parents:
diff changeset
177 check_grow(); // Grow table if insert hit limit
a61af66fc99e Initial load
duke
parents:
diff changeset
178 return NULL; // Miss!
a61af66fc99e Initial load
duke
parents:
diff changeset
179 }
a61af66fc99e Initial load
duke
parents:
diff changeset
180 else if( first_sentinel == 0 && k == _sentinel ) {
a61af66fc99e Initial load
duke
parents:
diff changeset
181 first_sentinel = key; // Can insert here
a61af66fc99e Initial load
duke
parents:
diff changeset
182 }
a61af66fc99e Initial load
duke
parents:
diff changeset
183
a61af66fc99e Initial load
duke
parents:
diff changeset
184 }
a61af66fc99e Initial load
duke
parents:
diff changeset
185 ShouldNotReachHere();
a61af66fc99e Initial load
duke
parents:
diff changeset
186 return NULL;
a61af66fc99e Initial load
duke
parents:
diff changeset
187 }
a61af66fc99e Initial load
duke
parents:
diff changeset
188
a61af66fc99e Initial load
duke
parents:
diff changeset
189 //------------------------------hash_insert------------------------------------
a61af66fc99e Initial load
duke
parents:
diff changeset
190 // Insert into hash table
a61af66fc99e Initial load
duke
parents:
diff changeset
191 void NodeHash::hash_insert( Node *n ) {
a61af66fc99e Initial load
duke
parents:
diff changeset
192 // // "conflict" comments -- print nodes that conflict
a61af66fc99e Initial load
duke
parents:
diff changeset
193 // bool conflict = false;
a61af66fc99e Initial load
duke
parents:
diff changeset
194 // n->set_hash();
a61af66fc99e Initial load
duke
parents:
diff changeset
195 uint hash = n->hash();
a61af66fc99e Initial load
duke
parents:
diff changeset
196 if (hash == Node::NO_HASH) {
a61af66fc99e Initial load
duke
parents:
diff changeset
197 return;
a61af66fc99e Initial load
duke
parents:
diff changeset
198 }
a61af66fc99e Initial load
duke
parents:
diff changeset
199 check_grow();
a61af66fc99e Initial load
duke
parents:
diff changeset
200 uint key = hash & (_max-1);
a61af66fc99e Initial load
duke
parents:
diff changeset
201 uint stride = key | 0x01;
a61af66fc99e Initial load
duke
parents:
diff changeset
202
a61af66fc99e Initial load
duke
parents:
diff changeset
203 while( 1 ) { // While probing hash table
a61af66fc99e Initial load
duke
parents:
diff changeset
204 debug_only( _insert_probes++ );
a61af66fc99e Initial load
duke
parents:
diff changeset
205 Node *k = _table[key]; // Get hashed value
a61af66fc99e Initial load
duke
parents:
diff changeset
206 if( !k || (k == _sentinel) ) break; // Found a slot
a61af66fc99e Initial load
duke
parents:
diff changeset
207 assert( k != n, "already inserted" );
a61af66fc99e Initial load
duke
parents:
diff changeset
208 // if( PrintCompilation && PrintOptoStatistics && Verbose ) { tty->print(" conflict: "); k->dump(); conflict = true; }
a61af66fc99e Initial load
duke
parents:
diff changeset
209 key = (key + stride) & (_max-1); // Stride through table w/ relative prime
a61af66fc99e Initial load
duke
parents:
diff changeset
210 }
a61af66fc99e Initial load
duke
parents:
diff changeset
211 _table[key] = n; // Insert into table!
a61af66fc99e Initial load
duke
parents:
diff changeset
212 debug_only(n->enter_hash_lock()); // Lock down the node while in the table.
a61af66fc99e Initial load
duke
parents:
diff changeset
213 // if( conflict ) { n->dump(); }
a61af66fc99e Initial load
duke
parents:
diff changeset
214 }
a61af66fc99e Initial load
duke
parents:
diff changeset
215
a61af66fc99e Initial load
duke
parents:
diff changeset
216 //------------------------------hash_delete------------------------------------
605
98cb887364d3 6810672: Comment typos
twisti
parents: 305
diff changeset
217 // Replace in hash table with sentinel
0
a61af66fc99e Initial load
duke
parents:
diff changeset
218 bool NodeHash::hash_delete( const Node *n ) {
a61af66fc99e Initial load
duke
parents:
diff changeset
219 Node *k;
a61af66fc99e Initial load
duke
parents:
diff changeset
220 uint hash = n->hash();
a61af66fc99e Initial load
duke
parents:
diff changeset
221 if (hash == Node::NO_HASH) {
a61af66fc99e Initial load
duke
parents:
diff changeset
222 debug_only( _delete_misses++ );
a61af66fc99e Initial load
duke
parents:
diff changeset
223 return false;
a61af66fc99e Initial load
duke
parents:
diff changeset
224 }
a61af66fc99e Initial load
duke
parents:
diff changeset
225 uint key = hash & (_max-1);
a61af66fc99e Initial load
duke
parents:
diff changeset
226 uint stride = key | 0x01;
a61af66fc99e Initial load
duke
parents:
diff changeset
227 debug_only( uint counter = 0; );
605
98cb887364d3 6810672: Comment typos
twisti
parents: 305
diff changeset
228 for( ; /* (k != NULL) && (k != _sentinel) */; ) {
0
a61af66fc99e Initial load
duke
parents:
diff changeset
229 debug_only( counter++ );
a61af66fc99e Initial load
duke
parents:
diff changeset
230 debug_only( _delete_probes++ );
a61af66fc99e Initial load
duke
parents:
diff changeset
231 k = _table[key]; // Get hashed value
a61af66fc99e Initial load
duke
parents:
diff changeset
232 if( !k ) { // Miss?
a61af66fc99e Initial load
duke
parents:
diff changeset
233 debug_only( _delete_misses++ );
a61af66fc99e Initial load
duke
parents:
diff changeset
234 #ifdef ASSERT
a61af66fc99e Initial load
duke
parents:
diff changeset
235 if( VerifyOpto ) {
a61af66fc99e Initial load
duke
parents:
diff changeset
236 for( uint i=0; i < _max; i++ )
a61af66fc99e Initial load
duke
parents:
diff changeset
237 assert( _table[i] != n, "changed edges with rehashing" );
a61af66fc99e Initial load
duke
parents:
diff changeset
238 }
a61af66fc99e Initial load
duke
parents:
diff changeset
239 #endif
a61af66fc99e Initial load
duke
parents:
diff changeset
240 return false; // Miss! Not in chain
a61af66fc99e Initial load
duke
parents:
diff changeset
241 }
a61af66fc99e Initial load
duke
parents:
diff changeset
242 else if( n == k ) {
a61af66fc99e Initial load
duke
parents:
diff changeset
243 debug_only( _delete_hits++ );
a61af66fc99e Initial load
duke
parents:
diff changeset
244 _table[key] = _sentinel; // Hit! Label as deleted entry
a61af66fc99e Initial load
duke
parents:
diff changeset
245 debug_only(((Node*)n)->exit_hash_lock()); // Unlock the node upon removal from table.
a61af66fc99e Initial load
duke
parents:
diff changeset
246 return true;
a61af66fc99e Initial load
duke
parents:
diff changeset
247 }
a61af66fc99e Initial load
duke
parents:
diff changeset
248 else {
a61af66fc99e Initial load
duke
parents:
diff changeset
249 // collision: move through table with prime offset
a61af66fc99e Initial load
duke
parents:
diff changeset
250 key = (key + stride/*7*/) & (_max-1);
a61af66fc99e Initial load
duke
parents:
diff changeset
251 assert( counter <= _insert_limit, "Cycle in hash-table");
a61af66fc99e Initial load
duke
parents:
diff changeset
252 }
a61af66fc99e Initial load
duke
parents:
diff changeset
253 }
a61af66fc99e Initial load
duke
parents:
diff changeset
254 ShouldNotReachHere();
a61af66fc99e Initial load
duke
parents:
diff changeset
255 return false;
a61af66fc99e Initial load
duke
parents:
diff changeset
256 }
a61af66fc99e Initial load
duke
parents:
diff changeset
257
a61af66fc99e Initial load
duke
parents:
diff changeset
258 //------------------------------round_up---------------------------------------
a61af66fc99e Initial load
duke
parents:
diff changeset
259 // Round up to nearest power of 2
a61af66fc99e Initial load
duke
parents:
diff changeset
260 uint NodeHash::round_up( uint x ) {
a61af66fc99e Initial load
duke
parents:
diff changeset
261 x += (x>>2); // Add 25% slop
a61af66fc99e Initial load
duke
parents:
diff changeset
262 if( x <16 ) return 16; // Small stuff
a61af66fc99e Initial load
duke
parents:
diff changeset
263 uint i=16;
a61af66fc99e Initial load
duke
parents:
diff changeset
264 while( i < x ) i <<= 1; // Double to fit
a61af66fc99e Initial load
duke
parents:
diff changeset
265 return i; // Return hash table size
a61af66fc99e Initial load
duke
parents:
diff changeset
266 }
a61af66fc99e Initial load
duke
parents:
diff changeset
267
a61af66fc99e Initial load
duke
parents:
diff changeset
268 //------------------------------grow-------------------------------------------
a61af66fc99e Initial load
duke
parents:
diff changeset
269 // Grow _table to next power of 2 and insert old entries
a61af66fc99e Initial load
duke
parents:
diff changeset
270 void NodeHash::grow() {
a61af66fc99e Initial load
duke
parents:
diff changeset
271 // Record old state
a61af66fc99e Initial load
duke
parents:
diff changeset
272 uint old_max = _max;
a61af66fc99e Initial load
duke
parents:
diff changeset
273 Node **old_table = _table;
a61af66fc99e Initial load
duke
parents:
diff changeset
274 // Construct new table with twice the space
a61af66fc99e Initial load
duke
parents:
diff changeset
275 _grows++;
a61af66fc99e Initial load
duke
parents:
diff changeset
276 _total_inserts += _inserts;
a61af66fc99e Initial load
duke
parents:
diff changeset
277 _total_insert_probes += _insert_probes;
a61af66fc99e Initial load
duke
parents:
diff changeset
278 _inserts = 0;
a61af66fc99e Initial load
duke
parents:
diff changeset
279 _insert_probes = 0;
a61af66fc99e Initial load
duke
parents:
diff changeset
280 _max = _max << 1;
a61af66fc99e Initial load
duke
parents:
diff changeset
281 _table = NEW_ARENA_ARRAY( _a , Node* , _max ); // (Node**)_a->Amalloc( _max * sizeof(Node*) );
a61af66fc99e Initial load
duke
parents:
diff changeset
282 memset(_table,0,sizeof(Node*)*_max);
a61af66fc99e Initial load
duke
parents:
diff changeset
283 _insert_limit = insert_limit();
a61af66fc99e Initial load
duke
parents:
diff changeset
284 // Insert old entries into the new table
a61af66fc99e Initial load
duke
parents:
diff changeset
285 for( uint i = 0; i < old_max; i++ ) {
a61af66fc99e Initial load
duke
parents:
diff changeset
286 Node *m = *old_table++;
a61af66fc99e Initial load
duke
parents:
diff changeset
287 if( !m || m == _sentinel ) continue;
a61af66fc99e Initial load
duke
parents:
diff changeset
288 debug_only(m->exit_hash_lock()); // Unlock the node upon removal from old table.
a61af66fc99e Initial load
duke
parents:
diff changeset
289 hash_insert(m);
a61af66fc99e Initial load
duke
parents:
diff changeset
290 }
a61af66fc99e Initial load
duke
parents:
diff changeset
291 }
a61af66fc99e Initial load
duke
parents:
diff changeset
292
a61af66fc99e Initial load
duke
parents:
diff changeset
293 //------------------------------clear------------------------------------------
a61af66fc99e Initial load
duke
parents:
diff changeset
294 // Clear all entries in _table to NULL but keep storage
a61af66fc99e Initial load
duke
parents:
diff changeset
295 void NodeHash::clear() {
a61af66fc99e Initial load
duke
parents:
diff changeset
296 #ifdef ASSERT
a61af66fc99e Initial load
duke
parents:
diff changeset
297 // Unlock all nodes upon removal from table.
a61af66fc99e Initial load
duke
parents:
diff changeset
298 for (uint i = 0; i < _max; i++) {
a61af66fc99e Initial load
duke
parents:
diff changeset
299 Node* n = _table[i];
a61af66fc99e Initial load
duke
parents:
diff changeset
300 if (!n || n == _sentinel) continue;
a61af66fc99e Initial load
duke
parents:
diff changeset
301 n->exit_hash_lock();
a61af66fc99e Initial load
duke
parents:
diff changeset
302 }
a61af66fc99e Initial load
duke
parents:
diff changeset
303 #endif
a61af66fc99e Initial load
duke
parents:
diff changeset
304
a61af66fc99e Initial load
duke
parents:
diff changeset
305 memset( _table, 0, _max * sizeof(Node*) );
a61af66fc99e Initial load
duke
parents:
diff changeset
306 }
a61af66fc99e Initial load
duke
parents:
diff changeset
307
a61af66fc99e Initial load
duke
parents:
diff changeset
308 //-----------------------remove_useless_nodes----------------------------------
a61af66fc99e Initial load
duke
parents:
diff changeset
309 // Remove useless nodes from value table,
a61af66fc99e Initial load
duke
parents:
diff changeset
310 // implementation does not depend on hash function
a61af66fc99e Initial load
duke
parents:
diff changeset
311 void NodeHash::remove_useless_nodes(VectorSet &useful) {
a61af66fc99e Initial load
duke
parents:
diff changeset
312
a61af66fc99e Initial load
duke
parents:
diff changeset
313 // Dead nodes in the hash table inherited from GVN should not replace
a61af66fc99e Initial load
duke
parents:
diff changeset
314 // existing nodes, remove dead nodes.
a61af66fc99e Initial load
duke
parents:
diff changeset
315 uint max = size();
a61af66fc99e Initial load
duke
parents:
diff changeset
316 Node *sentinel_node = sentinel();
a61af66fc99e Initial load
duke
parents:
diff changeset
317 for( uint i = 0; i < max; ++i ) {
a61af66fc99e Initial load
duke
parents:
diff changeset
318 Node *n = at(i);
a61af66fc99e Initial load
duke
parents:
diff changeset
319 if(n != NULL && n != sentinel_node && !useful.test(n->_idx)) {
a61af66fc99e Initial load
duke
parents:
diff changeset
320 debug_only(n->exit_hash_lock()); // Unlock the node when removed
a61af66fc99e Initial load
duke
parents:
diff changeset
321 _table[i] = sentinel_node; // Replace with placeholder
a61af66fc99e Initial load
duke
parents:
diff changeset
322 }
a61af66fc99e Initial load
duke
parents:
diff changeset
323 }
a61af66fc99e Initial load
duke
parents:
diff changeset
324 }
a61af66fc99e Initial load
duke
parents:
diff changeset
325
17671
de95063c0e34 8027422: assert(_gvn.type(obj)->higher_equal(tjp)) failed: cast_up is no longer needed
roland
parents: 17467
diff changeset
326
de95063c0e34 8027422: assert(_gvn.type(obj)->higher_equal(tjp)) failed: cast_up is no longer needed
roland
parents: 17467
diff changeset
327 void NodeHash::check_no_speculative_types() {
de95063c0e34 8027422: assert(_gvn.type(obj)->higher_equal(tjp)) failed: cast_up is no longer needed
roland
parents: 17467
diff changeset
328 #ifdef ASSERT
de95063c0e34 8027422: assert(_gvn.type(obj)->higher_equal(tjp)) failed: cast_up is no longer needed
roland
parents: 17467
diff changeset
329 uint max = size();
de95063c0e34 8027422: assert(_gvn.type(obj)->higher_equal(tjp)) failed: cast_up is no longer needed
roland
parents: 17467
diff changeset
330 Node *sentinel_node = sentinel();
de95063c0e34 8027422: assert(_gvn.type(obj)->higher_equal(tjp)) failed: cast_up is no longer needed
roland
parents: 17467
diff changeset
331 for (uint i = 0; i < max; ++i) {
de95063c0e34 8027422: assert(_gvn.type(obj)->higher_equal(tjp)) failed: cast_up is no longer needed
roland
parents: 17467
diff changeset
332 Node *n = at(i);
de95063c0e34 8027422: assert(_gvn.type(obj)->higher_equal(tjp)) failed: cast_up is no longer needed
roland
parents: 17467
diff changeset
333 if(n != NULL && n != sentinel_node && n->is_Type()) {
de95063c0e34 8027422: assert(_gvn.type(obj)->higher_equal(tjp)) failed: cast_up is no longer needed
roland
parents: 17467
diff changeset
334 TypeNode* tn = n->as_Type();
de95063c0e34 8027422: assert(_gvn.type(obj)->higher_equal(tjp)) failed: cast_up is no longer needed
roland
parents: 17467
diff changeset
335 const Type* t = tn->type();
de95063c0e34 8027422: assert(_gvn.type(obj)->higher_equal(tjp)) failed: cast_up is no longer needed
roland
parents: 17467
diff changeset
336 const Type* t_no_spec = t->remove_speculative();
de95063c0e34 8027422: assert(_gvn.type(obj)->higher_equal(tjp)) failed: cast_up is no longer needed
roland
parents: 17467
diff changeset
337 assert(t == t_no_spec, "dead node in hash table or missed node during speculative cleanup");
de95063c0e34 8027422: assert(_gvn.type(obj)->higher_equal(tjp)) failed: cast_up is no longer needed
roland
parents: 17467
diff changeset
338 }
de95063c0e34 8027422: assert(_gvn.type(obj)->higher_equal(tjp)) failed: cast_up is no longer needed
roland
parents: 17467
diff changeset
339 }
de95063c0e34 8027422: assert(_gvn.type(obj)->higher_equal(tjp)) failed: cast_up is no longer needed
roland
parents: 17467
diff changeset
340 #endif
de95063c0e34 8027422: assert(_gvn.type(obj)->higher_equal(tjp)) failed: cast_up is no longer needed
roland
parents: 17467
diff changeset
341 }
de95063c0e34 8027422: assert(_gvn.type(obj)->higher_equal(tjp)) failed: cast_up is no longer needed
roland
parents: 17467
diff changeset
342
0
a61af66fc99e Initial load
duke
parents:
diff changeset
343 #ifndef PRODUCT
a61af66fc99e Initial load
duke
parents:
diff changeset
344 //------------------------------dump-------------------------------------------
a61af66fc99e Initial load
duke
parents:
diff changeset
345 // Dump statistics for the hash table
a61af66fc99e Initial load
duke
parents:
diff changeset
346 void NodeHash::dump() {
a61af66fc99e Initial load
duke
parents:
diff changeset
347 _total_inserts += _inserts;
a61af66fc99e Initial load
duke
parents:
diff changeset
348 _total_insert_probes += _insert_probes;
4064
670a74b863fc 7107042: assert(no_dead_loop) failed: dead loop detected
kvn
parents: 3958
diff changeset
349 if (PrintCompilation && PrintOptoStatistics && Verbose && (_inserts > 0)) {
670a74b863fc 7107042: assert(no_dead_loop) failed: dead loop detected
kvn
parents: 3958
diff changeset
350 if (WizardMode) {
670a74b863fc 7107042: assert(no_dead_loop) failed: dead loop detected
kvn
parents: 3958
diff changeset
351 for (uint i=0; i<_max; i++) {
670a74b863fc 7107042: assert(no_dead_loop) failed: dead loop detected
kvn
parents: 3958
diff changeset
352 if (_table[i])
670a74b863fc 7107042: assert(no_dead_loop) failed: dead loop detected
kvn
parents: 3958
diff changeset
353 tty->print("%d/%d/%d ",i,_table[i]->hash()&(_max-1),_table[i]->_idx);
670a74b863fc 7107042: assert(no_dead_loop) failed: dead loop detected
kvn
parents: 3958
diff changeset
354 }
0
a61af66fc99e Initial load
duke
parents:
diff changeset
355 }
a61af66fc99e Initial load
duke
parents:
diff changeset
356 tty->print("\nGVN Hash stats: %d grows to %d max_size\n", _grows, _max);
a61af66fc99e Initial load
duke
parents:
diff changeset
357 tty->print(" %d/%d (%8.1f%% full)\n", _inserts, _max, (double)_inserts/_max*100.0);
a61af66fc99e Initial load
duke
parents:
diff changeset
358 tty->print(" %dp/(%dh+%dm) (%8.2f probes/lookup)\n", _look_probes, _lookup_hits, _lookup_misses, (double)_look_probes/(_lookup_hits+_lookup_misses));
a61af66fc99e Initial load
duke
parents:
diff changeset
359 tty->print(" %dp/%di (%8.2f probes/insert)\n", _total_insert_probes, _total_inserts, (double)_total_insert_probes/_total_inserts);
a61af66fc99e Initial load
duke
parents:
diff changeset
360 // sentinels increase lookup cost, but not insert cost
a61af66fc99e Initial load
duke
parents:
diff changeset
361 assert((_lookup_misses+_lookup_hits)*4+100 >= _look_probes, "bad hash function");
a61af66fc99e Initial load
duke
parents:
diff changeset
362 assert( _inserts+(_inserts>>3) < _max, "table too full" );
a61af66fc99e Initial load
duke
parents:
diff changeset
363 assert( _inserts*3+100 >= _insert_probes, "bad hash function" );
a61af66fc99e Initial load
duke
parents:
diff changeset
364 }
a61af66fc99e Initial load
duke
parents:
diff changeset
365 }
a61af66fc99e Initial load
duke
parents:
diff changeset
366
a61af66fc99e Initial load
duke
parents:
diff changeset
367 Node *NodeHash::find_index(uint idx) { // For debugging
a61af66fc99e Initial load
duke
parents:
diff changeset
368 // Find an entry by its index value
a61af66fc99e Initial load
duke
parents:
diff changeset
369 for( uint i = 0; i < _max; i++ ) {
a61af66fc99e Initial load
duke
parents:
diff changeset
370 Node *m = _table[i];
a61af66fc99e Initial load
duke
parents:
diff changeset
371 if( !m || m == _sentinel ) continue;
a61af66fc99e Initial load
duke
parents:
diff changeset
372 if( m->_idx == (uint)idx ) return m;
a61af66fc99e Initial load
duke
parents:
diff changeset
373 }
a61af66fc99e Initial load
duke
parents:
diff changeset
374 return NULL;
a61af66fc99e Initial load
duke
parents:
diff changeset
375 }
a61af66fc99e Initial load
duke
parents:
diff changeset
376 #endif
a61af66fc99e Initial load
duke
parents:
diff changeset
377
a61af66fc99e Initial load
duke
parents:
diff changeset
378 #ifdef ASSERT
a61af66fc99e Initial load
duke
parents:
diff changeset
379 NodeHash::~NodeHash() {
a61af66fc99e Initial load
duke
parents:
diff changeset
380 // Unlock all nodes upon destruction of table.
a61af66fc99e Initial load
duke
parents:
diff changeset
381 if (_table != (Node**)badAddress) clear();
a61af66fc99e Initial load
duke
parents:
diff changeset
382 }
a61af66fc99e Initial load
duke
parents:
diff changeset
383
a61af66fc99e Initial load
duke
parents:
diff changeset
384 void NodeHash::operator=(const NodeHash& nh) {
a61af66fc99e Initial load
duke
parents:
diff changeset
385 // Unlock all nodes upon replacement of table.
a61af66fc99e Initial load
duke
parents:
diff changeset
386 if (&nh == this) return;
a61af66fc99e Initial load
duke
parents:
diff changeset
387 if (_table != (Node**)badAddress) clear();
a61af66fc99e Initial load
duke
parents:
diff changeset
388 memcpy(this, &nh, sizeof(*this));
a61af66fc99e Initial load
duke
parents:
diff changeset
389 // Do not increment hash_lock counts again.
a61af66fc99e Initial load
duke
parents:
diff changeset
390 // Instead, be sure we never again use the source table.
a61af66fc99e Initial load
duke
parents:
diff changeset
391 ((NodeHash*)&nh)->_table = (Node**)badAddress;
a61af66fc99e Initial load
duke
parents:
diff changeset
392 }
a61af66fc99e Initial load
duke
parents:
diff changeset
393
a61af66fc99e Initial load
duke
parents:
diff changeset
394
a61af66fc99e Initial load
duke
parents:
diff changeset
395 #endif
a61af66fc99e Initial load
duke
parents:
diff changeset
396
a61af66fc99e Initial load
duke
parents:
diff changeset
397
a61af66fc99e Initial load
duke
parents:
diff changeset
398 //=============================================================================
a61af66fc99e Initial load
duke
parents:
diff changeset
399 //------------------------------PhaseRemoveUseless-----------------------------
a61af66fc99e Initial load
duke
parents:
diff changeset
400 // 1) Use a breadthfirst walk to collect useful nodes reachable from root.
23471
70649f10b88c 8129847: Compiling methods generated by Nashorn triggers high memory usage in C2
zmajo
parents: 23421
diff changeset
401 PhaseRemoveUseless::PhaseRemoveUseless(PhaseGVN *gvn, Unique_Node_List *worklist, PhaseNumber phase_num) : Phase(phase_num),
0
a61af66fc99e Initial load
duke
parents:
diff changeset
402 _useful(Thread::current()->resource_area()) {
a61af66fc99e Initial load
duke
parents:
diff changeset
403
a61af66fc99e Initial load
duke
parents:
diff changeset
404 // Implementation requires 'UseLoopSafepoints == true' and an edge from root
a61af66fc99e Initial load
duke
parents:
diff changeset
405 // to each SafePointNode at a backward branch. Inserted in add_safepoint().
a61af66fc99e Initial load
duke
parents:
diff changeset
406 if( !UseLoopSafepoints || !OptoRemoveUseless ) return;
a61af66fc99e Initial load
duke
parents:
diff changeset
407
a61af66fc99e Initial load
duke
parents:
diff changeset
408 // Identify nodes that are reachable from below, useful.
a61af66fc99e Initial load
duke
parents:
diff changeset
409 C->identify_useful_nodes(_useful);
7196
2aff40cb4703 7092905: C2: Keep track of the number of dead nodes
bharadwaj
parents: 6842
diff changeset
410 // Update dead node list
2aff40cb4703 7092905: C2: Keep track of the number of dead nodes
bharadwaj
parents: 6842
diff changeset
411 C->update_dead_node_list(_useful);
0
a61af66fc99e Initial load
duke
parents:
diff changeset
412
a61af66fc99e Initial load
duke
parents:
diff changeset
413 // Remove all useless nodes from PhaseValues' recorded types
a61af66fc99e Initial load
duke
parents:
diff changeset
414 // Must be done before disconnecting nodes to preserve hash-table-invariant
a61af66fc99e Initial load
duke
parents:
diff changeset
415 gvn->remove_useless_nodes(_useful.member_set());
a61af66fc99e Initial load
duke
parents:
diff changeset
416
a61af66fc99e Initial load
duke
parents:
diff changeset
417 // Remove all useless nodes from future worklist
a61af66fc99e Initial load
duke
parents:
diff changeset
418 worklist->remove_useless_nodes(_useful.member_set());
a61af66fc99e Initial load
duke
parents:
diff changeset
419
a61af66fc99e Initial load
duke
parents:
diff changeset
420 // Disconnect 'useless' nodes that are adjacent to useful nodes
a61af66fc99e Initial load
duke
parents:
diff changeset
421 C->remove_useless_nodes(_useful);
a61af66fc99e Initial load
duke
parents:
diff changeset
422
a61af66fc99e Initial load
duke
parents:
diff changeset
423 // Remove edges from "root" to each SafePoint at a backward branch.
a61af66fc99e Initial load
duke
parents:
diff changeset
424 // They were inserted during parsing (see add_safepoint()) to make infinite
a61af66fc99e Initial load
duke
parents:
diff changeset
425 // loops without calls or exceptions visible to root, i.e., useful.
a61af66fc99e Initial load
duke
parents:
diff changeset
426 Node *root = C->root();
a61af66fc99e Initial load
duke
parents:
diff changeset
427 if( root != NULL ) {
a61af66fc99e Initial load
duke
parents:
diff changeset
428 for( uint i = root->req(); i < root->len(); ++i ) {
a61af66fc99e Initial load
duke
parents:
diff changeset
429 Node *n = root->in(i);
a61af66fc99e Initial load
duke
parents:
diff changeset
430 if( n != NULL && n->is_SafePoint() ) {
a61af66fc99e Initial load
duke
parents:
diff changeset
431 root->rm_prec(i);
a61af66fc99e Initial load
duke
parents:
diff changeset
432 --i;
a61af66fc99e Initial load
duke
parents:
diff changeset
433 }
a61af66fc99e Initial load
duke
parents:
diff changeset
434 }
a61af66fc99e Initial load
duke
parents:
diff changeset
435 }
a61af66fc99e Initial load
duke
parents:
diff changeset
436 }
a61af66fc99e Initial load
duke
parents:
diff changeset
437
23471
70649f10b88c 8129847: Compiling methods generated by Nashorn triggers high memory usage in C2
zmajo
parents: 23421
diff changeset
438 //=============================================================================
70649f10b88c 8129847: Compiling methods generated by Nashorn triggers high memory usage in C2
zmajo
parents: 23421
diff changeset
439 //------------------------------PhaseRenumberLive------------------------------
70649f10b88c 8129847: Compiling methods generated by Nashorn triggers high memory usage in C2
zmajo
parents: 23421
diff changeset
440 // First, remove useless nodes (equivalent to identifying live nodes).
70649f10b88c 8129847: Compiling methods generated by Nashorn triggers high memory usage in C2
zmajo
parents: 23421
diff changeset
441 // Then, renumber live nodes.
70649f10b88c 8129847: Compiling methods generated by Nashorn triggers high memory usage in C2
zmajo
parents: 23421
diff changeset
442 //
70649f10b88c 8129847: Compiling methods generated by Nashorn triggers high memory usage in C2
zmajo
parents: 23421
diff changeset
443 // The set of live nodes is returned by PhaseRemoveUseless in the _useful structure.
70649f10b88c 8129847: Compiling methods generated by Nashorn triggers high memory usage in C2
zmajo
parents: 23421
diff changeset
444 // If the number of live nodes is 'x' (where 'x' == _useful.size()), then the
70649f10b88c 8129847: Compiling methods generated by Nashorn triggers high memory usage in C2
zmajo
parents: 23421
diff changeset
445 // PhaseRenumberLive updates the node ID of each node (the _idx field) with a unique
70649f10b88c 8129847: Compiling methods generated by Nashorn triggers high memory usage in C2
zmajo
parents: 23421
diff changeset
446 // value in the range [0, x).
70649f10b88c 8129847: Compiling methods generated by Nashorn triggers high memory usage in C2
zmajo
parents: 23421
diff changeset
447 //
70649f10b88c 8129847: Compiling methods generated by Nashorn triggers high memory usage in C2
zmajo
parents: 23421
diff changeset
448 // At the end of the PhaseRenumberLive phase, the compiler's count of unique nodes is
70649f10b88c 8129847: Compiling methods generated by Nashorn triggers high memory usage in C2
zmajo
parents: 23421
diff changeset
449 // updated to 'x' and the list of dead nodes is reset (as there are no dead nodes).
70649f10b88c 8129847: Compiling methods generated by Nashorn triggers high memory usage in C2
zmajo
parents: 23421
diff changeset
450 //
70649f10b88c 8129847: Compiling methods generated by Nashorn triggers high memory usage in C2
zmajo
parents: 23421
diff changeset
451 // The PhaseRenumberLive phase updates two data structures with the new node IDs.
70649f10b88c 8129847: Compiling methods generated by Nashorn triggers high memory usage in C2
zmajo
parents: 23421
diff changeset
452 // (1) The worklist is used by the PhaseIterGVN phase to identify nodes that must be
70649f10b88c 8129847: Compiling methods generated by Nashorn triggers high memory usage in C2
zmajo
parents: 23421
diff changeset
453 // processed. A new worklist (with the updated node IDs) is returned in 'new_worklist'.
70649f10b88c 8129847: Compiling methods generated by Nashorn triggers high memory usage in C2
zmajo
parents: 23421
diff changeset
454 // (2) Type information (the field PhaseGVN::_types) maps type information to each
70649f10b88c 8129847: Compiling methods generated by Nashorn triggers high memory usage in C2
zmajo
parents: 23421
diff changeset
455 // node ID. The mapping is updated to use the new node IDs as well. Updated type
70649f10b88c 8129847: Compiling methods generated by Nashorn triggers high memory usage in C2
zmajo
parents: 23421
diff changeset
456 // information is returned in PhaseGVN::_types.
70649f10b88c 8129847: Compiling methods generated by Nashorn triggers high memory usage in C2
zmajo
parents: 23421
diff changeset
457 //
70649f10b88c 8129847: Compiling methods generated by Nashorn triggers high memory usage in C2
zmajo
parents: 23421
diff changeset
458 // The PhaseRenumberLive phase does not preserve the order of elements in the worklist.
70649f10b88c 8129847: Compiling methods generated by Nashorn triggers high memory usage in C2
zmajo
parents: 23421
diff changeset
459 //
70649f10b88c 8129847: Compiling methods generated by Nashorn triggers high memory usage in C2
zmajo
parents: 23421
diff changeset
460 // Other data structures used by the compiler are not updated. The hash table for value
70649f10b88c 8129847: Compiling methods generated by Nashorn triggers high memory usage in C2
zmajo
parents: 23421
diff changeset
461 // numbering (the field PhaseGVN::_table) is not updated because computing the hash
70649f10b88c 8129847: Compiling methods generated by Nashorn triggers high memory usage in C2
zmajo
parents: 23421
diff changeset
462 // values is not based on node IDs. The field PhaseGVN::_nodes is not updated either
70649f10b88c 8129847: Compiling methods generated by Nashorn triggers high memory usage in C2
zmajo
parents: 23421
diff changeset
463 // because it is empty wherever PhaseRenumberLive is used.
70649f10b88c 8129847: Compiling methods generated by Nashorn triggers high memory usage in C2
zmajo
parents: 23421
diff changeset
464 PhaseRenumberLive::PhaseRenumberLive(PhaseGVN* gvn,
70649f10b88c 8129847: Compiling methods generated by Nashorn triggers high memory usage in C2
zmajo
parents: 23421
diff changeset
465 Unique_Node_List* worklist, Unique_Node_List* new_worklist,
70649f10b88c 8129847: Compiling methods generated by Nashorn triggers high memory usage in C2
zmajo
parents: 23421
diff changeset
466 PhaseNumber phase_num) :
70649f10b88c 8129847: Compiling methods generated by Nashorn triggers high memory usage in C2
zmajo
parents: 23421
diff changeset
467 PhaseRemoveUseless(gvn, worklist, Remove_Useless_And_Renumber_Live) {
70649f10b88c 8129847: Compiling methods generated by Nashorn triggers high memory usage in C2
zmajo
parents: 23421
diff changeset
468
70649f10b88c 8129847: Compiling methods generated by Nashorn triggers high memory usage in C2
zmajo
parents: 23421
diff changeset
469 assert(RenumberLiveNodes, "RenumberLiveNodes must be set to true for node renumbering to take place");
70649f10b88c 8129847: Compiling methods generated by Nashorn triggers high memory usage in C2
zmajo
parents: 23421
diff changeset
470 assert(C->live_nodes() == _useful.size(), "the number of live nodes must match the number of useful nodes");
70649f10b88c 8129847: Compiling methods generated by Nashorn triggers high memory usage in C2
zmajo
parents: 23421
diff changeset
471 assert(gvn->nodes_size() == 0, "GVN must not contain any nodes at this point");
70649f10b88c 8129847: Compiling methods generated by Nashorn triggers high memory usage in C2
zmajo
parents: 23421
diff changeset
472
70649f10b88c 8129847: Compiling methods generated by Nashorn triggers high memory usage in C2
zmajo
parents: 23421
diff changeset
473 uint old_unique_count = C->unique();
70649f10b88c 8129847: Compiling methods generated by Nashorn triggers high memory usage in C2
zmajo
parents: 23421
diff changeset
474 uint live_node_count = C->live_nodes();
70649f10b88c 8129847: Compiling methods generated by Nashorn triggers high memory usage in C2
zmajo
parents: 23421
diff changeset
475 uint worklist_size = worklist->size();
70649f10b88c 8129847: Compiling methods generated by Nashorn triggers high memory usage in C2
zmajo
parents: 23421
diff changeset
476
70649f10b88c 8129847: Compiling methods generated by Nashorn triggers high memory usage in C2
zmajo
parents: 23421
diff changeset
477 // Storage for the updated type information.
70649f10b88c 8129847: Compiling methods generated by Nashorn triggers high memory usage in C2
zmajo
parents: 23421
diff changeset
478 Type_Array new_type_array(C->comp_arena());
70649f10b88c 8129847: Compiling methods generated by Nashorn triggers high memory usage in C2
zmajo
parents: 23421
diff changeset
479
70649f10b88c 8129847: Compiling methods generated by Nashorn triggers high memory usage in C2
zmajo
parents: 23421
diff changeset
480 // Iterate over the set of live nodes.
70649f10b88c 8129847: Compiling methods generated by Nashorn triggers high memory usage in C2
zmajo
parents: 23421
diff changeset
481 uint current_idx = 0; // The current new node ID. Incremented after every assignment.
70649f10b88c 8129847: Compiling methods generated by Nashorn triggers high memory usage in C2
zmajo
parents: 23421
diff changeset
482 for (uint i = 0; i < _useful.size(); i++) {
70649f10b88c 8129847: Compiling methods generated by Nashorn triggers high memory usage in C2
zmajo
parents: 23421
diff changeset
483 Node* n = _useful.at(i);
70649f10b88c 8129847: Compiling methods generated by Nashorn triggers high memory usage in C2
zmajo
parents: 23421
diff changeset
484 const Type* type = gvn->type_or_null(n);
70649f10b88c 8129847: Compiling methods generated by Nashorn triggers high memory usage in C2
zmajo
parents: 23421
diff changeset
485 new_type_array.map(current_idx, type);
70649f10b88c 8129847: Compiling methods generated by Nashorn triggers high memory usage in C2
zmajo
parents: 23421
diff changeset
486
70649f10b88c 8129847: Compiling methods generated by Nashorn triggers high memory usage in C2
zmajo
parents: 23421
diff changeset
487 bool in_worklist = false;
70649f10b88c 8129847: Compiling methods generated by Nashorn triggers high memory usage in C2
zmajo
parents: 23421
diff changeset
488 if (worklist->member(n)) {
70649f10b88c 8129847: Compiling methods generated by Nashorn triggers high memory usage in C2
zmajo
parents: 23421
diff changeset
489 in_worklist = true;
70649f10b88c 8129847: Compiling methods generated by Nashorn triggers high memory usage in C2
zmajo
parents: 23421
diff changeset
490 }
70649f10b88c 8129847: Compiling methods generated by Nashorn triggers high memory usage in C2
zmajo
parents: 23421
diff changeset
491
70649f10b88c 8129847: Compiling methods generated by Nashorn triggers high memory usage in C2
zmajo
parents: 23421
diff changeset
492 n->set_idx(current_idx); // Update node ID.
70649f10b88c 8129847: Compiling methods generated by Nashorn triggers high memory usage in C2
zmajo
parents: 23421
diff changeset
493
70649f10b88c 8129847: Compiling methods generated by Nashorn triggers high memory usage in C2
zmajo
parents: 23421
diff changeset
494 if (in_worklist) {
70649f10b88c 8129847: Compiling methods generated by Nashorn triggers high memory usage in C2
zmajo
parents: 23421
diff changeset
495 new_worklist->push(n);
70649f10b88c 8129847: Compiling methods generated by Nashorn triggers high memory usage in C2
zmajo
parents: 23421
diff changeset
496 }
70649f10b88c 8129847: Compiling methods generated by Nashorn triggers high memory usage in C2
zmajo
parents: 23421
diff changeset
497
70649f10b88c 8129847: Compiling methods generated by Nashorn triggers high memory usage in C2
zmajo
parents: 23421
diff changeset
498 current_idx++;
70649f10b88c 8129847: Compiling methods generated by Nashorn triggers high memory usage in C2
zmajo
parents: 23421
diff changeset
499 }
70649f10b88c 8129847: Compiling methods generated by Nashorn triggers high memory usage in C2
zmajo
parents: 23421
diff changeset
500
70649f10b88c 8129847: Compiling methods generated by Nashorn triggers high memory usage in C2
zmajo
parents: 23421
diff changeset
501 assert(worklist_size == new_worklist->size(), "the new worklist must have the same size as the original worklist");
70649f10b88c 8129847: Compiling methods generated by Nashorn triggers high memory usage in C2
zmajo
parents: 23421
diff changeset
502 assert(live_node_count == current_idx, "all live nodes must be processed");
70649f10b88c 8129847: Compiling methods generated by Nashorn triggers high memory usage in C2
zmajo
parents: 23421
diff changeset
503
70649f10b88c 8129847: Compiling methods generated by Nashorn triggers high memory usage in C2
zmajo
parents: 23421
diff changeset
504 // Replace the compiler's type information with the updated type information.
70649f10b88c 8129847: Compiling methods generated by Nashorn triggers high memory usage in C2
zmajo
parents: 23421
diff changeset
505 gvn->replace_types(new_type_array);
70649f10b88c 8129847: Compiling methods generated by Nashorn triggers high memory usage in C2
zmajo
parents: 23421
diff changeset
506
70649f10b88c 8129847: Compiling methods generated by Nashorn triggers high memory usage in C2
zmajo
parents: 23421
diff changeset
507 // Update the unique node count of the compilation to the number of currently live nodes.
70649f10b88c 8129847: Compiling methods generated by Nashorn triggers high memory usage in C2
zmajo
parents: 23421
diff changeset
508 C->set_unique(live_node_count);
70649f10b88c 8129847: Compiling methods generated by Nashorn triggers high memory usage in C2
zmajo
parents: 23421
diff changeset
509
70649f10b88c 8129847: Compiling methods generated by Nashorn triggers high memory usage in C2
zmajo
parents: 23421
diff changeset
510 // Set the dead node count to 0 and reset dead node list.
70649f10b88c 8129847: Compiling methods generated by Nashorn triggers high memory usage in C2
zmajo
parents: 23421
diff changeset
511 C->reset_dead_node_list();
70649f10b88c 8129847: Compiling methods generated by Nashorn triggers high memory usage in C2
zmajo
parents: 23421
diff changeset
512 }
70649f10b88c 8129847: Compiling methods generated by Nashorn triggers high memory usage in C2
zmajo
parents: 23421
diff changeset
513
0
a61af66fc99e Initial load
duke
parents:
diff changeset
514
a61af66fc99e Initial load
duke
parents:
diff changeset
515 //=============================================================================
a61af66fc99e Initial load
duke
parents:
diff changeset
516 //------------------------------PhaseTransform---------------------------------
a61af66fc99e Initial load
duke
parents:
diff changeset
517 PhaseTransform::PhaseTransform( PhaseNumber pnum ) : Phase(pnum),
a61af66fc99e Initial load
duke
parents:
diff changeset
518 _arena(Thread::current()->resource_area()),
a61af66fc99e Initial load
duke
parents:
diff changeset
519 _nodes(_arena),
a61af66fc99e Initial load
duke
parents:
diff changeset
520 _types(_arena)
a61af66fc99e Initial load
duke
parents:
diff changeset
521 {
a61af66fc99e Initial load
duke
parents:
diff changeset
522 init_con_caches();
a61af66fc99e Initial load
duke
parents:
diff changeset
523 #ifndef PRODUCT
a61af66fc99e Initial load
duke
parents:
diff changeset
524 clear_progress();
a61af66fc99e Initial load
duke
parents:
diff changeset
525 clear_transforms();
a61af66fc99e Initial load
duke
parents:
diff changeset
526 set_allow_progress(true);
a61af66fc99e Initial load
duke
parents:
diff changeset
527 #endif
a61af66fc99e Initial load
duke
parents:
diff changeset
528 // Force allocation for currently existing nodes
a61af66fc99e Initial load
duke
parents:
diff changeset
529 _types.map(C->unique(), NULL);
a61af66fc99e Initial load
duke
parents:
diff changeset
530 }
a61af66fc99e Initial load
duke
parents:
diff changeset
531
a61af66fc99e Initial load
duke
parents:
diff changeset
532 //------------------------------PhaseTransform---------------------------------
a61af66fc99e Initial load
duke
parents:
diff changeset
533 PhaseTransform::PhaseTransform( Arena *arena, PhaseNumber pnum ) : Phase(pnum),
a61af66fc99e Initial load
duke
parents:
diff changeset
534 _arena(arena),
a61af66fc99e Initial load
duke
parents:
diff changeset
535 _nodes(arena),
a61af66fc99e Initial load
duke
parents:
diff changeset
536 _types(arena)
a61af66fc99e Initial load
duke
parents:
diff changeset
537 {
a61af66fc99e Initial load
duke
parents:
diff changeset
538 init_con_caches();
a61af66fc99e Initial load
duke
parents:
diff changeset
539 #ifndef PRODUCT
a61af66fc99e Initial load
duke
parents:
diff changeset
540 clear_progress();
a61af66fc99e Initial load
duke
parents:
diff changeset
541 clear_transforms();
a61af66fc99e Initial load
duke
parents:
diff changeset
542 set_allow_progress(true);
a61af66fc99e Initial load
duke
parents:
diff changeset
543 #endif
a61af66fc99e Initial load
duke
parents:
diff changeset
544 // Force allocation for currently existing nodes
a61af66fc99e Initial load
duke
parents:
diff changeset
545 _types.map(C->unique(), NULL);
a61af66fc99e Initial load
duke
parents:
diff changeset
546 }
a61af66fc99e Initial load
duke
parents:
diff changeset
547
a61af66fc99e Initial load
duke
parents:
diff changeset
548 //------------------------------PhaseTransform---------------------------------
a61af66fc99e Initial load
duke
parents:
diff changeset
549 // Initialize with previously generated type information
a61af66fc99e Initial load
duke
parents:
diff changeset
550 PhaseTransform::PhaseTransform( PhaseTransform *pt, PhaseNumber pnum ) : Phase(pnum),
a61af66fc99e Initial load
duke
parents:
diff changeset
551 _arena(pt->_arena),
a61af66fc99e Initial load
duke
parents:
diff changeset
552 _nodes(pt->_nodes),
a61af66fc99e Initial load
duke
parents:
diff changeset
553 _types(pt->_types)
a61af66fc99e Initial load
duke
parents:
diff changeset
554 {
a61af66fc99e Initial load
duke
parents:
diff changeset
555 init_con_caches();
a61af66fc99e Initial load
duke
parents:
diff changeset
556 #ifndef PRODUCT
a61af66fc99e Initial load
duke
parents:
diff changeset
557 clear_progress();
a61af66fc99e Initial load
duke
parents:
diff changeset
558 clear_transforms();
a61af66fc99e Initial load
duke
parents:
diff changeset
559 set_allow_progress(true);
a61af66fc99e Initial load
duke
parents:
diff changeset
560 #endif
a61af66fc99e Initial load
duke
parents:
diff changeset
561 }
a61af66fc99e Initial load
duke
parents:
diff changeset
562
a61af66fc99e Initial load
duke
parents:
diff changeset
563 void PhaseTransform::init_con_caches() {
a61af66fc99e Initial load
duke
parents:
diff changeset
564 memset(_icons,0,sizeof(_icons));
a61af66fc99e Initial load
duke
parents:
diff changeset
565 memset(_lcons,0,sizeof(_lcons));
a61af66fc99e Initial load
duke
parents:
diff changeset
566 memset(_zcons,0,sizeof(_zcons));
a61af66fc99e Initial load
duke
parents:
diff changeset
567 }
a61af66fc99e Initial load
duke
parents:
diff changeset
568
a61af66fc99e Initial load
duke
parents:
diff changeset
569
a61af66fc99e Initial load
duke
parents:
diff changeset
570 //--------------------------------find_int_type--------------------------------
a61af66fc99e Initial load
duke
parents:
diff changeset
571 const TypeInt* PhaseTransform::find_int_type(Node* n) {
a61af66fc99e Initial load
duke
parents:
diff changeset
572 if (n == NULL) return NULL;
a61af66fc99e Initial load
duke
parents:
diff changeset
573 // Call type_or_null(n) to determine node's type since we might be in
a61af66fc99e Initial load
duke
parents:
diff changeset
574 // parse phase and call n->Value() may return wrong type.
a61af66fc99e Initial load
duke
parents:
diff changeset
575 // (For example, a phi node at the beginning of loop parsing is not ready.)
a61af66fc99e Initial load
duke
parents:
diff changeset
576 const Type* t = type_or_null(n);
a61af66fc99e Initial load
duke
parents:
diff changeset
577 if (t == NULL) return NULL;
a61af66fc99e Initial load
duke
parents:
diff changeset
578 return t->isa_int();
a61af66fc99e Initial load
duke
parents:
diff changeset
579 }
a61af66fc99e Initial load
duke
parents:
diff changeset
580
a61af66fc99e Initial load
duke
parents:
diff changeset
581
a61af66fc99e Initial load
duke
parents:
diff changeset
582 //-------------------------------find_long_type--------------------------------
a61af66fc99e Initial load
duke
parents:
diff changeset
583 const TypeLong* PhaseTransform::find_long_type(Node* n) {
a61af66fc99e Initial load
duke
parents:
diff changeset
584 if (n == NULL) return NULL;
a61af66fc99e Initial load
duke
parents:
diff changeset
585 // (See comment above on type_or_null.)
a61af66fc99e Initial load
duke
parents:
diff changeset
586 const Type* t = type_or_null(n);
a61af66fc99e Initial load
duke
parents:
diff changeset
587 if (t == NULL) return NULL;
a61af66fc99e Initial load
duke
parents:
diff changeset
588 return t->isa_long();
a61af66fc99e Initial load
duke
parents:
diff changeset
589 }
a61af66fc99e Initial load
duke
parents:
diff changeset
590
a61af66fc99e Initial load
duke
parents:
diff changeset
591
a61af66fc99e Initial load
duke
parents:
diff changeset
592 #ifndef PRODUCT
a61af66fc99e Initial load
duke
parents:
diff changeset
593 void PhaseTransform::dump_old2new_map() const {
a61af66fc99e Initial load
duke
parents:
diff changeset
594 _nodes.dump();
a61af66fc99e Initial load
duke
parents:
diff changeset
595 }
a61af66fc99e Initial load
duke
parents:
diff changeset
596
a61af66fc99e Initial load
duke
parents:
diff changeset
597 void PhaseTransform::dump_new( uint nidx ) const {
a61af66fc99e Initial load
duke
parents:
diff changeset
598 for( uint i=0; i<_nodes.Size(); i++ )
a61af66fc99e Initial load
duke
parents:
diff changeset
599 if( _nodes[i] && _nodes[i]->_idx == nidx ) {
a61af66fc99e Initial load
duke
parents:
diff changeset
600 _nodes[i]->dump();
a61af66fc99e Initial load
duke
parents:
diff changeset
601 tty->cr();
a61af66fc99e Initial load
duke
parents:
diff changeset
602 tty->print_cr("Old index= %d",i);
a61af66fc99e Initial load
duke
parents:
diff changeset
603 return;
a61af66fc99e Initial load
duke
parents:
diff changeset
604 }
a61af66fc99e Initial load
duke
parents:
diff changeset
605 tty->print_cr("Node %d not found in the new indices", nidx);
a61af66fc99e Initial load
duke
parents:
diff changeset
606 }
a61af66fc99e Initial load
duke
parents:
diff changeset
607
a61af66fc99e Initial load
duke
parents:
diff changeset
608 //------------------------------dump_types-------------------------------------
a61af66fc99e Initial load
duke
parents:
diff changeset
609 void PhaseTransform::dump_types( ) const {
a61af66fc99e Initial load
duke
parents:
diff changeset
610 _types.dump();
a61af66fc99e Initial load
duke
parents:
diff changeset
611 }
a61af66fc99e Initial load
duke
parents:
diff changeset
612
a61af66fc99e Initial load
duke
parents:
diff changeset
613 //------------------------------dump_nodes_and_types---------------------------
a61af66fc99e Initial load
duke
parents:
diff changeset
614 void PhaseTransform::dump_nodes_and_types(const Node *root, uint depth, bool only_ctrl) {
a61af66fc99e Initial load
duke
parents:
diff changeset
615 VectorSet visited(Thread::current()->resource_area());
a61af66fc99e Initial load
duke
parents:
diff changeset
616 dump_nodes_and_types_recur( root, depth, only_ctrl, visited );
a61af66fc99e Initial load
duke
parents:
diff changeset
617 }
a61af66fc99e Initial load
duke
parents:
diff changeset
618
a61af66fc99e Initial load
duke
parents:
diff changeset
619 //------------------------------dump_nodes_and_types_recur---------------------
a61af66fc99e Initial load
duke
parents:
diff changeset
620 void PhaseTransform::dump_nodes_and_types_recur( const Node *n, uint depth, bool only_ctrl, VectorSet &visited) {
a61af66fc99e Initial load
duke
parents:
diff changeset
621 if( !n ) return;
a61af66fc99e Initial load
duke
parents:
diff changeset
622 if( depth == 0 ) return;
a61af66fc99e Initial load
duke
parents:
diff changeset
623 if( visited.test_set(n->_idx) ) return;
a61af66fc99e Initial load
duke
parents:
diff changeset
624 for( uint i=0; i<n->len(); i++ ) {
a61af66fc99e Initial load
duke
parents:
diff changeset
625 if( only_ctrl && !(n->is_Region()) && i != TypeFunc::Control ) continue;
a61af66fc99e Initial load
duke
parents:
diff changeset
626 dump_nodes_and_types_recur( n->in(i), depth-1, only_ctrl, visited );
a61af66fc99e Initial load
duke
parents:
diff changeset
627 }
a61af66fc99e Initial load
duke
parents:
diff changeset
628 n->dump();
a61af66fc99e Initial load
duke
parents:
diff changeset
629 if (type_or_null(n) != NULL) {
a61af66fc99e Initial load
duke
parents:
diff changeset
630 tty->print(" "); type(n)->dump(); tty->cr();
a61af66fc99e Initial load
duke
parents:
diff changeset
631 }
a61af66fc99e Initial load
duke
parents:
diff changeset
632 }
a61af66fc99e Initial load
duke
parents:
diff changeset
633
a61af66fc99e Initial load
duke
parents:
diff changeset
634 #endif
a61af66fc99e Initial load
duke
parents:
diff changeset
635
a61af66fc99e Initial load
duke
parents:
diff changeset
636
a61af66fc99e Initial load
duke
parents:
diff changeset
637 //=============================================================================
a61af66fc99e Initial load
duke
parents:
diff changeset
638 //------------------------------PhaseValues------------------------------------
a61af66fc99e Initial load
duke
parents:
diff changeset
639 // Set minimum table size to "255"
a61af66fc99e Initial load
duke
parents:
diff changeset
640 PhaseValues::PhaseValues( Arena *arena, uint est_max_size ) : PhaseTransform(arena, GVN), _table(arena, est_max_size) {
a61af66fc99e Initial load
duke
parents:
diff changeset
641 NOT_PRODUCT( clear_new_values(); )
a61af66fc99e Initial load
duke
parents:
diff changeset
642 }
a61af66fc99e Initial load
duke
parents:
diff changeset
643
a61af66fc99e Initial load
duke
parents:
diff changeset
644 //------------------------------PhaseValues------------------------------------
a61af66fc99e Initial load
duke
parents:
diff changeset
645 // Set minimum table size to "255"
a61af66fc99e Initial load
duke
parents:
diff changeset
646 PhaseValues::PhaseValues( PhaseValues *ptv ) : PhaseTransform( ptv, GVN ),
a61af66fc99e Initial load
duke
parents:
diff changeset
647 _table(&ptv->_table) {
a61af66fc99e Initial load
duke
parents:
diff changeset
648 NOT_PRODUCT( clear_new_values(); )
a61af66fc99e Initial load
duke
parents:
diff changeset
649 }
a61af66fc99e Initial load
duke
parents:
diff changeset
650
a61af66fc99e Initial load
duke
parents:
diff changeset
651 //------------------------------PhaseValues------------------------------------
a61af66fc99e Initial load
duke
parents:
diff changeset
652 // Used by +VerifyOpto. Clear out hash table but copy _types array.
a61af66fc99e Initial load
duke
parents:
diff changeset
653 PhaseValues::PhaseValues( PhaseValues *ptv, const char *dummy ) : PhaseTransform( ptv, GVN ),
a61af66fc99e Initial load
duke
parents:
diff changeset
654 _table(ptv->arena(),ptv->_table.size()) {
a61af66fc99e Initial load
duke
parents:
diff changeset
655 NOT_PRODUCT( clear_new_values(); )
a61af66fc99e Initial load
duke
parents:
diff changeset
656 }
a61af66fc99e Initial load
duke
parents:
diff changeset
657
a61af66fc99e Initial load
duke
parents:
diff changeset
658 //------------------------------~PhaseValues-----------------------------------
a61af66fc99e Initial load
duke
parents:
diff changeset
659 #ifndef PRODUCT
a61af66fc99e Initial load
duke
parents:
diff changeset
660 PhaseValues::~PhaseValues() {
a61af66fc99e Initial load
duke
parents:
diff changeset
661 _table.dump();
a61af66fc99e Initial load
duke
parents:
diff changeset
662
a61af66fc99e Initial load
duke
parents:
diff changeset
663 // Statistics for value progress and efficiency
a61af66fc99e Initial load
duke
parents:
diff changeset
664 if( PrintCompilation && Verbose && WizardMode ) {
a61af66fc99e Initial load
duke
parents:
diff changeset
665 tty->print("\n%sValues: %d nodes ---> %d/%d (%d)",
a61af66fc99e Initial load
duke
parents:
diff changeset
666 is_IterGVN() ? "Iter" : " ", C->unique(), made_progress(), made_transforms(), made_new_values());
a61af66fc99e Initial load
duke
parents:
diff changeset
667 if( made_transforms() != 0 ) {
a61af66fc99e Initial load
duke
parents:
diff changeset
668 tty->print_cr(" ratio %f", made_progress()/(float)made_transforms() );
a61af66fc99e Initial load
duke
parents:
diff changeset
669 } else {
a61af66fc99e Initial load
duke
parents:
diff changeset
670 tty->cr();
a61af66fc99e Initial load
duke
parents:
diff changeset
671 }
a61af66fc99e Initial load
duke
parents:
diff changeset
672 }
a61af66fc99e Initial load
duke
parents:
diff changeset
673 }
a61af66fc99e Initial load
duke
parents:
diff changeset
674 #endif
a61af66fc99e Initial load
duke
parents:
diff changeset
675
a61af66fc99e Initial load
duke
parents:
diff changeset
676 //------------------------------makecon----------------------------------------
a61af66fc99e Initial load
duke
parents:
diff changeset
677 ConNode* PhaseTransform::makecon(const Type *t) {
a61af66fc99e Initial load
duke
parents:
diff changeset
678 assert(t->singleton(), "must be a constant");
a61af66fc99e Initial load
duke
parents:
diff changeset
679 assert(!t->empty() || t == Type::TOP, "must not be vacuous range");
a61af66fc99e Initial load
duke
parents:
diff changeset
680 switch (t->base()) { // fast paths
a61af66fc99e Initial load
duke
parents:
diff changeset
681 case Type::Half:
a61af66fc99e Initial load
duke
parents:
diff changeset
682 case Type::Top: return (ConNode*) C->top();
a61af66fc99e Initial load
duke
parents:
diff changeset
683 case Type::Int: return intcon( t->is_int()->get_con() );
a61af66fc99e Initial load
duke
parents:
diff changeset
684 case Type::Long: return longcon( t->is_long()->get_con() );
a61af66fc99e Initial load
duke
parents:
diff changeset
685 }
a61af66fc99e Initial load
duke
parents:
diff changeset
686 if (t->is_zero_type())
a61af66fc99e Initial load
duke
parents:
diff changeset
687 return zerocon(t->basic_type());
a61af66fc99e Initial load
duke
parents:
diff changeset
688 return uncached_makecon(t);
a61af66fc99e Initial load
duke
parents:
diff changeset
689 }
a61af66fc99e Initial load
duke
parents:
diff changeset
690
a61af66fc99e Initial load
duke
parents:
diff changeset
691 //--------------------------uncached_makecon-----------------------------------
a61af66fc99e Initial load
duke
parents:
diff changeset
692 // Make an idealized constant - one of ConINode, ConPNode, etc.
a61af66fc99e Initial load
duke
parents:
diff changeset
693 ConNode* PhaseValues::uncached_makecon(const Type *t) {
a61af66fc99e Initial load
duke
parents:
diff changeset
694 assert(t->singleton(), "must be a constant");
a61af66fc99e Initial load
duke
parents:
diff changeset
695 ConNode* x = ConNode::make(C, t);
a61af66fc99e Initial load
duke
parents:
diff changeset
696 ConNode* k = (ConNode*)hash_find_insert(x); // Value numbering
a61af66fc99e Initial load
duke
parents:
diff changeset
697 if (k == NULL) {
a61af66fc99e Initial load
duke
parents:
diff changeset
698 set_type(x, t); // Missed, provide type mapping
a61af66fc99e Initial load
duke
parents:
diff changeset
699 GrowableArray<Node_Notes*>* nna = C->node_note_array();
a61af66fc99e Initial load
duke
parents:
diff changeset
700 if (nna != NULL) {
a61af66fc99e Initial load
duke
parents:
diff changeset
701 Node_Notes* loc = C->locate_node_notes(nna, x->_idx, true);
a61af66fc99e Initial load
duke
parents:
diff changeset
702 loc->clear(); // do not put debug info on constants
a61af66fc99e Initial load
duke
parents:
diff changeset
703 }
a61af66fc99e Initial load
duke
parents:
diff changeset
704 } else {
a61af66fc99e Initial load
duke
parents:
diff changeset
705 x->destruct(); // Hit, destroy duplicate constant
a61af66fc99e Initial load
duke
parents:
diff changeset
706 x = k; // use existing constant
a61af66fc99e Initial load
duke
parents:
diff changeset
707 }
a61af66fc99e Initial load
duke
parents:
diff changeset
708 return x;
a61af66fc99e Initial load
duke
parents:
diff changeset
709 }
a61af66fc99e Initial load
duke
parents:
diff changeset
710
a61af66fc99e Initial load
duke
parents:
diff changeset
711 //------------------------------intcon-----------------------------------------
a61af66fc99e Initial load
duke
parents:
diff changeset
712 // Fast integer constant. Same as "transform(new ConINode(TypeInt::make(i)))"
a61af66fc99e Initial load
duke
parents:
diff changeset
713 ConINode* PhaseTransform::intcon(int i) {
a61af66fc99e Initial load
duke
parents:
diff changeset
714 // Small integer? Check cache! Check that cached node is not dead
a61af66fc99e Initial load
duke
parents:
diff changeset
715 if (i >= _icon_min && i <= _icon_max) {
a61af66fc99e Initial load
duke
parents:
diff changeset
716 ConINode* icon = _icons[i-_icon_min];
a61af66fc99e Initial load
duke
parents:
diff changeset
717 if (icon != NULL && icon->in(TypeFunc::Control) != NULL)
a61af66fc99e Initial load
duke
parents:
diff changeset
718 return icon;
a61af66fc99e Initial load
duke
parents:
diff changeset
719 }
a61af66fc99e Initial load
duke
parents:
diff changeset
720 ConINode* icon = (ConINode*) uncached_makecon(TypeInt::make(i));
a61af66fc99e Initial load
duke
parents:
diff changeset
721 assert(icon->is_Con(), "");
a61af66fc99e Initial load
duke
parents:
diff changeset
722 if (i >= _icon_min && i <= _icon_max)
a61af66fc99e Initial load
duke
parents:
diff changeset
723 _icons[i-_icon_min] = icon; // Cache small integers
a61af66fc99e Initial load
duke
parents:
diff changeset
724 return icon;
a61af66fc99e Initial load
duke
parents:
diff changeset
725 }
a61af66fc99e Initial load
duke
parents:
diff changeset
726
a61af66fc99e Initial load
duke
parents:
diff changeset
727 //------------------------------longcon----------------------------------------
a61af66fc99e Initial load
duke
parents:
diff changeset
728 // Fast long constant.
a61af66fc99e Initial load
duke
parents:
diff changeset
729 ConLNode* PhaseTransform::longcon(jlong l) {
a61af66fc99e Initial load
duke
parents:
diff changeset
730 // Small integer? Check cache! Check that cached node is not dead
a61af66fc99e Initial load
duke
parents:
diff changeset
731 if (l >= _lcon_min && l <= _lcon_max) {
a61af66fc99e Initial load
duke
parents:
diff changeset
732 ConLNode* lcon = _lcons[l-_lcon_min];
a61af66fc99e Initial load
duke
parents:
diff changeset
733 if (lcon != NULL && lcon->in(TypeFunc::Control) != NULL)
a61af66fc99e Initial load
duke
parents:
diff changeset
734 return lcon;
a61af66fc99e Initial load
duke
parents:
diff changeset
735 }
a61af66fc99e Initial load
duke
parents:
diff changeset
736 ConLNode* lcon = (ConLNode*) uncached_makecon(TypeLong::make(l));
a61af66fc99e Initial load
duke
parents:
diff changeset
737 assert(lcon->is_Con(), "");
a61af66fc99e Initial load
duke
parents:
diff changeset
738 if (l >= _lcon_min && l <= _lcon_max)
a61af66fc99e Initial load
duke
parents:
diff changeset
739 _lcons[l-_lcon_min] = lcon; // Cache small integers
a61af66fc99e Initial load
duke
parents:
diff changeset
740 return lcon;
a61af66fc99e Initial load
duke
parents:
diff changeset
741 }
a61af66fc99e Initial load
duke
parents:
diff changeset
742
a61af66fc99e Initial load
duke
parents:
diff changeset
743 //------------------------------zerocon-----------------------------------------
a61af66fc99e Initial load
duke
parents:
diff changeset
744 // Fast zero or null constant. Same as "transform(ConNode::make(Type::get_zero_type(bt)))"
a61af66fc99e Initial load
duke
parents:
diff changeset
745 ConNode* PhaseTransform::zerocon(BasicType bt) {
a61af66fc99e Initial load
duke
parents:
diff changeset
746 assert((uint)bt <= _zcon_max, "domain check");
a61af66fc99e Initial load
duke
parents:
diff changeset
747 ConNode* zcon = _zcons[bt];
a61af66fc99e Initial load
duke
parents:
diff changeset
748 if (zcon != NULL && zcon->in(TypeFunc::Control) != NULL)
a61af66fc99e Initial load
duke
parents:
diff changeset
749 return zcon;
a61af66fc99e Initial load
duke
parents:
diff changeset
750 zcon = (ConNode*) uncached_makecon(Type::get_zero_type(bt));
a61af66fc99e Initial load
duke
parents:
diff changeset
751 _zcons[bt] = zcon;
a61af66fc99e Initial load
duke
parents:
diff changeset
752 return zcon;
a61af66fc99e Initial load
duke
parents:
diff changeset
753 }
a61af66fc99e Initial load
duke
parents:
diff changeset
754
a61af66fc99e Initial load
duke
parents:
diff changeset
755
a61af66fc99e Initial load
duke
parents:
diff changeset
756
a61af66fc99e Initial load
duke
parents:
diff changeset
757 //=============================================================================
a61af66fc99e Initial load
duke
parents:
diff changeset
758 //------------------------------transform--------------------------------------
a61af66fc99e Initial load
duke
parents:
diff changeset
759 // Return a node which computes the same function as this node, but in a
41
874b2c4f43d1 6667605: (Escape Analysis) inline java constructors when EA is on
kvn
parents: 0
diff changeset
760 // faster or cheaper fashion.
0
a61af66fc99e Initial load
duke
parents:
diff changeset
761 Node *PhaseGVN::transform( Node *n ) {
41
874b2c4f43d1 6667605: (Escape Analysis) inline java constructors when EA is on
kvn
parents: 0
diff changeset
762 return transform_no_reclaim(n);
0
a61af66fc99e Initial load
duke
parents:
diff changeset
763 }
a61af66fc99e Initial load
duke
parents:
diff changeset
764
a61af66fc99e Initial load
duke
parents:
diff changeset
765 //------------------------------transform--------------------------------------
a61af66fc99e Initial load
duke
parents:
diff changeset
766 // Return a node which computes the same function as this node, but
a61af66fc99e Initial load
duke
parents:
diff changeset
767 // in a faster or cheaper fashion.
a61af66fc99e Initial load
duke
parents:
diff changeset
768 Node *PhaseGVN::transform_no_reclaim( Node *n ) {
a61af66fc99e Initial load
duke
parents:
diff changeset
769 NOT_PRODUCT( set_transforms(); )
a61af66fc99e Initial load
duke
parents:
diff changeset
770
a61af66fc99e Initial load
duke
parents:
diff changeset
771 // Apply the Ideal call in a loop until it no longer applies
a61af66fc99e Initial load
duke
parents:
diff changeset
772 Node *k = n;
a61af66fc99e Initial load
duke
parents:
diff changeset
773 NOT_PRODUCT( uint loop_count = 0; )
a61af66fc99e Initial load
duke
parents:
diff changeset
774 while( 1 ) {
a61af66fc99e Initial load
duke
parents:
diff changeset
775 Node *i = k->Ideal(this, /*can_reshape=*/false);
a61af66fc99e Initial load
duke
parents:
diff changeset
776 if( !i ) break;
a61af66fc99e Initial load
duke
parents:
diff changeset
777 assert( i->_idx >= k->_idx, "Idealize should return new nodes, use Identity to return old nodes" );
a61af66fc99e Initial load
duke
parents:
diff changeset
778 k = i;
a61af66fc99e Initial load
duke
parents:
diff changeset
779 assert(loop_count++ < K, "infinite loop in PhaseGVN::transform");
a61af66fc99e Initial load
duke
parents:
diff changeset
780 }
a61af66fc99e Initial load
duke
parents:
diff changeset
781 NOT_PRODUCT( if( loop_count != 0 ) { set_progress(); } )
a61af66fc99e Initial load
duke
parents:
diff changeset
782
a61af66fc99e Initial load
duke
parents:
diff changeset
783
a61af66fc99e Initial load
duke
parents:
diff changeset
784 // If brand new node, make space in type array.
a61af66fc99e Initial load
duke
parents:
diff changeset
785 ensure_type_or_null(k);
a61af66fc99e Initial load
duke
parents:
diff changeset
786
a61af66fc99e Initial load
duke
parents:
diff changeset
787 // Since I just called 'Value' to compute the set of run-time values
a61af66fc99e Initial load
duke
parents:
diff changeset
788 // for this Node, and 'Value' is non-local (and therefore expensive) I'll
a61af66fc99e Initial load
duke
parents:
diff changeset
789 // cache Value. Later requests for the local phase->type of this Node can
a61af66fc99e Initial load
duke
parents:
diff changeset
790 // use the cached Value instead of suffering with 'bottom_type'.
a61af66fc99e Initial load
duke
parents:
diff changeset
791 const Type *t = k->Value(this); // Get runtime Value set
a61af66fc99e Initial load
duke
parents:
diff changeset
792 assert(t != NULL, "value sanity");
a61af66fc99e Initial load
duke
parents:
diff changeset
793 if (type_or_null(k) != t) {
a61af66fc99e Initial load
duke
parents:
diff changeset
794 #ifndef PRODUCT
a61af66fc99e Initial load
duke
parents:
diff changeset
795 // Do not count initial visit to node as a transformation
a61af66fc99e Initial load
duke
parents:
diff changeset
796 if (type_or_null(k) == NULL) {
a61af66fc99e Initial load
duke
parents:
diff changeset
797 inc_new_values();
a61af66fc99e Initial load
duke
parents:
diff changeset
798 set_progress();
a61af66fc99e Initial load
duke
parents:
diff changeset
799 }
a61af66fc99e Initial load
duke
parents:
diff changeset
800 #endif
a61af66fc99e Initial load
duke
parents:
diff changeset
801 set_type(k, t);
a61af66fc99e Initial load
duke
parents:
diff changeset
802 // If k is a TypeNode, capture any more-precise type permanently into Node
a61af66fc99e Initial load
duke
parents:
diff changeset
803 k->raise_bottom_type(t);
a61af66fc99e Initial load
duke
parents:
diff changeset
804 }
a61af66fc99e Initial load
duke
parents:
diff changeset
805
a61af66fc99e Initial load
duke
parents:
diff changeset
806 if( t->singleton() && !k->is_Con() ) {
a61af66fc99e Initial load
duke
parents:
diff changeset
807 NOT_PRODUCT( set_progress(); )
a61af66fc99e Initial load
duke
parents:
diff changeset
808 return makecon(t); // Turn into a constant
a61af66fc99e Initial load
duke
parents:
diff changeset
809 }
a61af66fc99e Initial load
duke
parents:
diff changeset
810
a61af66fc99e Initial load
duke
parents:
diff changeset
811 // Now check for Identities
a61af66fc99e Initial load
duke
parents:
diff changeset
812 Node *i = k->Identity(this); // Look for a nearby replacement
a61af66fc99e Initial load
duke
parents:
diff changeset
813 if( i != k ) { // Found? Return replacement!
a61af66fc99e Initial load
duke
parents:
diff changeset
814 NOT_PRODUCT( set_progress(); )
a61af66fc99e Initial load
duke
parents:
diff changeset
815 return i;
a61af66fc99e Initial load
duke
parents:
diff changeset
816 }
a61af66fc99e Initial load
duke
parents:
diff changeset
817
a61af66fc99e Initial load
duke
parents:
diff changeset
818 // Global Value Numbering
a61af66fc99e Initial load
duke
parents:
diff changeset
819 i = hash_find_insert(k); // Insert if new
a61af66fc99e Initial load
duke
parents:
diff changeset
820 if( i && (i != k) ) {
a61af66fc99e Initial load
duke
parents:
diff changeset
821 // Return the pre-existing node
a61af66fc99e Initial load
duke
parents:
diff changeset
822 NOT_PRODUCT( set_progress(); )
a61af66fc99e Initial load
duke
parents:
diff changeset
823 return i;
a61af66fc99e Initial load
duke
parents:
diff changeset
824 }
a61af66fc99e Initial load
duke
parents:
diff changeset
825
a61af66fc99e Initial load
duke
parents:
diff changeset
826 // Return Idealized original
a61af66fc99e Initial load
duke
parents:
diff changeset
827 return k;
a61af66fc99e Initial load
duke
parents:
diff changeset
828 }
a61af66fc99e Initial load
duke
parents:
diff changeset
829
a61af66fc99e Initial load
duke
parents:
diff changeset
830 #ifdef ASSERT
a61af66fc99e Initial load
duke
parents:
diff changeset
831 //------------------------------dead_loop_check--------------------------------
605
98cb887364d3 6810672: Comment typos
twisti
parents: 305
diff changeset
832 // Check for a simple dead loop when a data node references itself directly
0
a61af66fc99e Initial load
duke
parents:
diff changeset
833 // or through an other data node excluding cons and phis.
a61af66fc99e Initial load
duke
parents:
diff changeset
834 void PhaseGVN::dead_loop_check( Node *n ) {
a61af66fc99e Initial load
duke
parents:
diff changeset
835 // Phi may reference itself in a loop
a61af66fc99e Initial load
duke
parents:
diff changeset
836 if (n != NULL && !n->is_dead_loop_safe() && !n->is_CFG()) {
a61af66fc99e Initial load
duke
parents:
diff changeset
837 // Do 2 levels check and only data inputs.
a61af66fc99e Initial load
duke
parents:
diff changeset
838 bool no_dead_loop = true;
a61af66fc99e Initial load
duke
parents:
diff changeset
839 uint cnt = n->req();
a61af66fc99e Initial load
duke
parents:
diff changeset
840 for (uint i = 1; i < cnt && no_dead_loop; i++) {
a61af66fc99e Initial load
duke
parents:
diff changeset
841 Node *in = n->in(i);
a61af66fc99e Initial load
duke
parents:
diff changeset
842 if (in == n) {
a61af66fc99e Initial load
duke
parents:
diff changeset
843 no_dead_loop = false;
a61af66fc99e Initial load
duke
parents:
diff changeset
844 } else if (in != NULL && !in->is_dead_loop_safe()) {
a61af66fc99e Initial load
duke
parents:
diff changeset
845 uint icnt = in->req();
a61af66fc99e Initial load
duke
parents:
diff changeset
846 for (uint j = 1; j < icnt && no_dead_loop; j++) {
a61af66fc99e Initial load
duke
parents:
diff changeset
847 if (in->in(j) == n || in->in(j) == in)
a61af66fc99e Initial load
duke
parents:
diff changeset
848 no_dead_loop = false;
a61af66fc99e Initial load
duke
parents:
diff changeset
849 }
a61af66fc99e Initial load
duke
parents:
diff changeset
850 }
a61af66fc99e Initial load
duke
parents:
diff changeset
851 }
a61af66fc99e Initial load
duke
parents:
diff changeset
852 if (!no_dead_loop) n->dump(3);
a61af66fc99e Initial load
duke
parents:
diff changeset
853 assert(no_dead_loop, "dead loop detected");
a61af66fc99e Initial load
duke
parents:
diff changeset
854 }
a61af66fc99e Initial load
duke
parents:
diff changeset
855 }
a61af66fc99e Initial load
duke
parents:
diff changeset
856 #endif
a61af66fc99e Initial load
duke
parents:
diff changeset
857
a61af66fc99e Initial load
duke
parents:
diff changeset
858 //=============================================================================
a61af66fc99e Initial load
duke
parents:
diff changeset
859 //------------------------------PhaseIterGVN-----------------------------------
a61af66fc99e Initial load
duke
parents:
diff changeset
860 // Initialize hash table to fresh and clean for +VerifyOpto
113
ba764ed4b6f2 6420645: Create a vm that uses compressed oops for up to 32gb heapsizes
coleenp
parents: 65
diff changeset
861 PhaseIterGVN::PhaseIterGVN( PhaseIterGVN *igvn, const char *dummy ) : PhaseGVN(igvn,dummy), _worklist( ),
23421
c1091733abe6 8011858: Use Compile::live_nodes() instead of Compile::unique() in appropriate places
zmajo
parents: 23043
diff changeset
862 _stack(C->live_nodes() >> 1),
113
ba764ed4b6f2 6420645: Create a vm that uses compressed oops for up to 32gb heapsizes
coleenp
parents: 65
diff changeset
863 _delay_transform(false) {
0
a61af66fc99e Initial load
duke
parents:
diff changeset
864 }
a61af66fc99e Initial load
duke
parents:
diff changeset
865
a61af66fc99e Initial load
duke
parents:
diff changeset
866 //------------------------------PhaseIterGVN-----------------------------------
a61af66fc99e Initial load
duke
parents:
diff changeset
867 // Initialize with previous PhaseIterGVN info; used by PhaseCCP
a61af66fc99e Initial load
duke
parents:
diff changeset
868 PhaseIterGVN::PhaseIterGVN( PhaseIterGVN *igvn ) : PhaseGVN(igvn),
113
ba764ed4b6f2 6420645: Create a vm that uses compressed oops for up to 32gb heapsizes
coleenp
parents: 65
diff changeset
869 _worklist( igvn->_worklist ),
6244
611e8a669a2c 7147464: Java crashed while executing method with over 8k of dneg operations
dlong
parents: 4064
diff changeset
870 _stack( igvn->_stack ),
113
ba764ed4b6f2 6420645: Create a vm that uses compressed oops for up to 32gb heapsizes
coleenp
parents: 65
diff changeset
871 _delay_transform(igvn->_delay_transform)
0
a61af66fc99e Initial load
duke
parents:
diff changeset
872 {
a61af66fc99e Initial load
duke
parents:
diff changeset
873 }
a61af66fc99e Initial load
duke
parents:
diff changeset
874
a61af66fc99e Initial load
duke
parents:
diff changeset
875 //------------------------------PhaseIterGVN-----------------------------------
a61af66fc99e Initial load
duke
parents:
diff changeset
876 // Initialize with previous PhaseGVN info from Parser
a61af66fc99e Initial load
duke
parents:
diff changeset
877 PhaseIterGVN::PhaseIterGVN( PhaseGVN *gvn ) : PhaseGVN(gvn),
113
ba764ed4b6f2 6420645: Create a vm that uses compressed oops for up to 32gb heapsizes
coleenp
parents: 65
diff changeset
878 _worklist(*C->for_igvn()),
23421
c1091733abe6 8011858: Use Compile::live_nodes() instead of Compile::unique() in appropriate places
zmajo
parents: 23043
diff changeset
879 // TODO: Before incremental inlining it was allocated only once and it was fine. Now that
c1091733abe6 8011858: Use Compile::live_nodes() instead of Compile::unique() in appropriate places
zmajo
parents: 23043
diff changeset
880 // the constructor is used in incremental inlining, this consumes too much memory:
c1091733abe6 8011858: Use Compile::live_nodes() instead of Compile::unique() in appropriate places
zmajo
parents: 23043
diff changeset
881 // _stack(C->live_nodes() >> 1),
c1091733abe6 8011858: Use Compile::live_nodes() instead of Compile::unique() in appropriate places
zmajo
parents: 23043
diff changeset
882 // So, as a band-aid, we replace this by:
c1091733abe6 8011858: Use Compile::live_nodes() instead of Compile::unique() in appropriate places
zmajo
parents: 23043
diff changeset
883 _stack(C->comp_arena(), 32),
113
ba764ed4b6f2 6420645: Create a vm that uses compressed oops for up to 32gb heapsizes
coleenp
parents: 65
diff changeset
884 _delay_transform(false)
0
a61af66fc99e Initial load
duke
parents:
diff changeset
885 {
a61af66fc99e Initial load
duke
parents:
diff changeset
886 uint max;
a61af66fc99e Initial load
duke
parents:
diff changeset
887
a61af66fc99e Initial load
duke
parents:
diff changeset
888 // Dead nodes in the hash table inherited from GVN were not treated as
a61af66fc99e Initial load
duke
parents:
diff changeset
889 // roots during def-use info creation; hence they represent an invisible
a61af66fc99e Initial load
duke
parents:
diff changeset
890 // use. Clear them out.
a61af66fc99e Initial load
duke
parents:
diff changeset
891 max = _table.size();
a61af66fc99e Initial load
duke
parents:
diff changeset
892 for( uint i = 0; i < max; ++i ) {
a61af66fc99e Initial load
duke
parents:
diff changeset
893 Node *n = _table.at(i);
a61af66fc99e Initial load
duke
parents:
diff changeset
894 if(n != NULL && n != _table.sentinel() && n->outcnt() == 0) {
a61af66fc99e Initial load
duke
parents:
diff changeset
895 if( n->is_top() ) continue;
a61af66fc99e Initial load
duke
parents:
diff changeset
896 assert( false, "Parse::remove_useless_nodes missed this node");
a61af66fc99e Initial load
duke
parents:
diff changeset
897 hash_delete(n);
a61af66fc99e Initial load
duke
parents:
diff changeset
898 }
a61af66fc99e Initial load
duke
parents:
diff changeset
899 }
a61af66fc99e Initial load
duke
parents:
diff changeset
900
a61af66fc99e Initial load
duke
parents:
diff changeset
901 // Any Phis or Regions on the worklist probably had uses that could not
a61af66fc99e Initial load
duke
parents:
diff changeset
902 // make more progress because the uses were made while the Phis and Regions
a61af66fc99e Initial load
duke
parents:
diff changeset
903 // were in half-built states. Put all uses of Phis and Regions on worklist.
a61af66fc99e Initial load
duke
parents:
diff changeset
904 max = _worklist.size();
a61af66fc99e Initial load
duke
parents:
diff changeset
905 for( uint j = 0; j < max; j++ ) {
a61af66fc99e Initial load
duke
parents:
diff changeset
906 Node *n = _worklist.at(j);
a61af66fc99e Initial load
duke
parents:
diff changeset
907 uint uop = n->Opcode();
a61af66fc99e Initial load
duke
parents:
diff changeset
908 if( uop == Op_Phi || uop == Op_Region ||
a61af66fc99e Initial load
duke
parents:
diff changeset
909 n->is_Type() ||
a61af66fc99e Initial load
duke
parents:
diff changeset
910 n->is_Mem() )
a61af66fc99e Initial load
duke
parents:
diff changeset
911 add_users_to_worklist(n);
a61af66fc99e Initial load
duke
parents:
diff changeset
912 }
a61af66fc99e Initial load
duke
parents:
diff changeset
913 }
a61af66fc99e Initial load
duke
parents:
diff changeset
914
a61af66fc99e Initial load
duke
parents:
diff changeset
915
a61af66fc99e Initial load
duke
parents:
diff changeset
916 #ifndef PRODUCT
a61af66fc99e Initial load
duke
parents:
diff changeset
917 void PhaseIterGVN::verify_step(Node* n) {
a61af66fc99e Initial load
duke
parents:
diff changeset
918 _verify_window[_verify_counter % _verify_window_size] = n;
a61af66fc99e Initial load
duke
parents:
diff changeset
919 ++_verify_counter;
a61af66fc99e Initial load
duke
parents:
diff changeset
920 ResourceMark rm;
a61af66fc99e Initial load
duke
parents:
diff changeset
921 ResourceArea *area = Thread::current()->resource_area();
a61af66fc99e Initial load
duke
parents:
diff changeset
922 VectorSet old_space(area), new_space(area);
a61af66fc99e Initial load
duke
parents:
diff changeset
923 if (C->unique() < 1000 ||
a61af66fc99e Initial load
duke
parents:
diff changeset
924 0 == _verify_counter % (C->unique() < 10000 ? 10 : 100)) {
a61af66fc99e Initial load
duke
parents:
diff changeset
925 ++_verify_full_passes;
a61af66fc99e Initial load
duke
parents:
diff changeset
926 Node::verify_recur(C->root(), -1, old_space, new_space);
a61af66fc99e Initial load
duke
parents:
diff changeset
927 }
a61af66fc99e Initial load
duke
parents:
diff changeset
928 const int verify_depth = 4;
a61af66fc99e Initial load
duke
parents:
diff changeset
929 for ( int i = 0; i < _verify_window_size; i++ ) {
a61af66fc99e Initial load
duke
parents:
diff changeset
930 Node* n = _verify_window[i];
a61af66fc99e Initial load
duke
parents:
diff changeset
931 if ( n == NULL ) continue;
a61af66fc99e Initial load
duke
parents:
diff changeset
932 if( n->in(0) == NodeSentinel ) { // xform_idom
a61af66fc99e Initial load
duke
parents:
diff changeset
933 _verify_window[i] = n->in(1);
a61af66fc99e Initial load
duke
parents:
diff changeset
934 --i; continue;
a61af66fc99e Initial load
duke
parents:
diff changeset
935 }
a61af66fc99e Initial load
duke
parents:
diff changeset
936 // Typical fanout is 1-2, so this call visits about 6 nodes.
a61af66fc99e Initial load
duke
parents:
diff changeset
937 Node::verify_recur(n, verify_depth, old_space, new_space);
a61af66fc99e Initial load
duke
parents:
diff changeset
938 }
a61af66fc99e Initial load
duke
parents:
diff changeset
939 }
a61af66fc99e Initial load
duke
parents:
diff changeset
940 #endif
a61af66fc99e Initial load
duke
parents:
diff changeset
941
a61af66fc99e Initial load
duke
parents:
diff changeset
942
a61af66fc99e Initial load
duke
parents:
diff changeset
943 //------------------------------init_worklist----------------------------------
a61af66fc99e Initial load
duke
parents:
diff changeset
944 // Initialize worklist for each node.
a61af66fc99e Initial load
duke
parents:
diff changeset
945 void PhaseIterGVN::init_worklist( Node *n ) {
a61af66fc99e Initial load
duke
parents:
diff changeset
946 if( _worklist.member(n) ) return;
a61af66fc99e Initial load
duke
parents:
diff changeset
947 _worklist.push(n);
a61af66fc99e Initial load
duke
parents:
diff changeset
948 uint cnt = n->req();
a61af66fc99e Initial load
duke
parents:
diff changeset
949 for( uint i =0 ; i < cnt; i++ ) {
a61af66fc99e Initial load
duke
parents:
diff changeset
950 Node *m = n->in(i);
a61af66fc99e Initial load
duke
parents:
diff changeset
951 if( m ) init_worklist(m);
a61af66fc99e Initial load
duke
parents:
diff changeset
952 }
a61af66fc99e Initial load
duke
parents:
diff changeset
953 }
a61af66fc99e Initial load
duke
parents:
diff changeset
954
a61af66fc99e Initial load
duke
parents:
diff changeset
955 //------------------------------optimize---------------------------------------
a61af66fc99e Initial load
duke
parents:
diff changeset
956 void PhaseIterGVN::optimize() {
a61af66fc99e Initial load
duke
parents:
diff changeset
957 debug_only(uint num_processed = 0;);
a61af66fc99e Initial load
duke
parents:
diff changeset
958 #ifndef PRODUCT
a61af66fc99e Initial load
duke
parents:
diff changeset
959 {
a61af66fc99e Initial load
duke
parents:
diff changeset
960 _verify_counter = 0;
a61af66fc99e Initial load
duke
parents:
diff changeset
961 _verify_full_passes = 0;
a61af66fc99e Initial load
duke
parents:
diff changeset
962 for ( int i = 0; i < _verify_window_size; i++ ) {
a61af66fc99e Initial load
duke
parents:
diff changeset
963 _verify_window[i] = NULL;
a61af66fc99e Initial load
duke
parents:
diff changeset
964 }
a61af66fc99e Initial load
duke
parents:
diff changeset
965 }
a61af66fc99e Initial load
duke
parents:
diff changeset
966 #endif
a61af66fc99e Initial load
duke
parents:
diff changeset
967
1826
56601ef83436 6916062: assert(_inserts <= _insert_limit,"hash table overflow") in NodeHash::hash_insert
kvn
parents: 1621
diff changeset
968 #ifdef ASSERT
56601ef83436 6916062: assert(_inserts <= _insert_limit,"hash table overflow") in NodeHash::hash_insert
kvn
parents: 1621
diff changeset
969 Node* prev = NULL;
56601ef83436 6916062: assert(_inserts <= _insert_limit,"hash table overflow") in NodeHash::hash_insert
kvn
parents: 1621
diff changeset
970 uint rep_cnt = 0;
56601ef83436 6916062: assert(_inserts <= _insert_limit,"hash table overflow") in NodeHash::hash_insert
kvn
parents: 1621
diff changeset
971 #endif
56601ef83436 6916062: assert(_inserts <= _insert_limit,"hash table overflow") in NodeHash::hash_insert
kvn
parents: 1621
diff changeset
972 uint loop_count = 0;
56601ef83436 6916062: assert(_inserts <= _insert_limit,"hash table overflow") in NodeHash::hash_insert
kvn
parents: 1621
diff changeset
973
0
a61af66fc99e Initial load
duke
parents:
diff changeset
974 // Pull from worklist; transform node;
a61af66fc99e Initial load
duke
parents:
diff changeset
975 // If node has changed: update edge info and put uses on worklist.
a61af66fc99e Initial load
duke
parents:
diff changeset
976 while( _worklist.size() ) {
3958
075ea0ed9e7c 7081842: assert(Compile::current()->unique() < (uint)MaxNodeLimit) failed: Node limit exceeded
kvn
parents: 1972
diff changeset
977 if (C->check_node_count(NodeLimitFudgeFactor * 2,
075ea0ed9e7c 7081842: assert(Compile::current()->unique() < (uint)MaxNodeLimit) failed: Node limit exceeded
kvn
parents: 1972
diff changeset
978 "out of nodes optimizing method")) {
075ea0ed9e7c 7081842: assert(Compile::current()->unique() < (uint)MaxNodeLimit) failed: Node limit exceeded
kvn
parents: 1972
diff changeset
979 return;
075ea0ed9e7c 7081842: assert(Compile::current()->unique() < (uint)MaxNodeLimit) failed: Node limit exceeded
kvn
parents: 1972
diff changeset
980 }
0
a61af66fc99e Initial load
duke
parents:
diff changeset
981 Node *n = _worklist.pop();
10278
6f3fd5150b67 6934604: enable parts of EliminateAutoBox by default
kvn
parents: 8868
diff changeset
982 if (++loop_count >= K * C->live_nodes()) {
1826
56601ef83436 6916062: assert(_inserts <= _insert_limit,"hash table overflow") in NodeHash::hash_insert
kvn
parents: 1621
diff changeset
983 debug_only(n->dump(4);)
56601ef83436 6916062: assert(_inserts <= _insert_limit,"hash table overflow") in NodeHash::hash_insert
kvn
parents: 1621
diff changeset
984 assert(false, "infinite loop in PhaseIterGVN::optimize");
56601ef83436 6916062: assert(_inserts <= _insert_limit,"hash table overflow") in NodeHash::hash_insert
kvn
parents: 1621
diff changeset
985 C->record_method_not_compilable("infinite loop in PhaseIterGVN::optimize");
56601ef83436 6916062: assert(_inserts <= _insert_limit,"hash table overflow") in NodeHash::hash_insert
kvn
parents: 1621
diff changeset
986 return;
56601ef83436 6916062: assert(_inserts <= _insert_limit,"hash table overflow") in NodeHash::hash_insert
kvn
parents: 1621
diff changeset
987 }
56601ef83436 6916062: assert(_inserts <= _insert_limit,"hash table overflow") in NodeHash::hash_insert
kvn
parents: 1621
diff changeset
988 #ifdef ASSERT
56601ef83436 6916062: assert(_inserts <= _insert_limit,"hash table overflow") in NodeHash::hash_insert
kvn
parents: 1621
diff changeset
989 if (n == prev) {
56601ef83436 6916062: assert(_inserts <= _insert_limit,"hash table overflow") in NodeHash::hash_insert
kvn
parents: 1621
diff changeset
990 if (++rep_cnt > 3) {
56601ef83436 6916062: assert(_inserts <= _insert_limit,"hash table overflow") in NodeHash::hash_insert
kvn
parents: 1621
diff changeset
991 n->dump(4);
56601ef83436 6916062: assert(_inserts <= _insert_limit,"hash table overflow") in NodeHash::hash_insert
kvn
parents: 1621
diff changeset
992 assert(false, "loop in Ideal transformation");
56601ef83436 6916062: assert(_inserts <= _insert_limit,"hash table overflow") in NodeHash::hash_insert
kvn
parents: 1621
diff changeset
993 }
56601ef83436 6916062: assert(_inserts <= _insert_limit,"hash table overflow") in NodeHash::hash_insert
kvn
parents: 1621
diff changeset
994 } else {
56601ef83436 6916062: assert(_inserts <= _insert_limit,"hash table overflow") in NodeHash::hash_insert
kvn
parents: 1621
diff changeset
995 rep_cnt = 0;
56601ef83436 6916062: assert(_inserts <= _insert_limit,"hash table overflow") in NodeHash::hash_insert
kvn
parents: 1621
diff changeset
996 }
56601ef83436 6916062: assert(_inserts <= _insert_limit,"hash table overflow") in NodeHash::hash_insert
kvn
parents: 1621
diff changeset
997 prev = n;
56601ef83436 6916062: assert(_inserts <= _insert_limit,"hash table overflow") in NodeHash::hash_insert
kvn
parents: 1621
diff changeset
998 #endif
0
a61af66fc99e Initial load
duke
parents:
diff changeset
999 if (TraceIterativeGVN && Verbose) {
a61af66fc99e Initial load
duke
parents:
diff changeset
1000 tty->print(" Pop ");
a61af66fc99e Initial load
duke
parents:
diff changeset
1001 NOT_PRODUCT( n->dump(); )
a61af66fc99e Initial load
duke
parents:
diff changeset
1002 debug_only(if( (num_processed++ % 100) == 0 ) _worklist.print_set();)
a61af66fc99e Initial load
duke
parents:
diff changeset
1003 }
a61af66fc99e Initial load
duke
parents:
diff changeset
1004
a61af66fc99e Initial load
duke
parents:
diff changeset
1005 if (n->outcnt() != 0) {
a61af66fc99e Initial load
duke
parents:
diff changeset
1006
a61af66fc99e Initial load
duke
parents:
diff changeset
1007 #ifndef PRODUCT
a61af66fc99e Initial load
duke
parents:
diff changeset
1008 uint wlsize = _worklist.size();
a61af66fc99e Initial load
duke
parents:
diff changeset
1009 const Type* oldtype = type_or_null(n);
a61af66fc99e Initial load
duke
parents:
diff changeset
1010 #endif //PRODUCT
a61af66fc99e Initial load
duke
parents:
diff changeset
1011
a61af66fc99e Initial load
duke
parents:
diff changeset
1012 Node *nn = transform_old(n);
a61af66fc99e Initial load
duke
parents:
diff changeset
1013
a61af66fc99e Initial load
duke
parents:
diff changeset
1014 #ifndef PRODUCT
a61af66fc99e Initial load
duke
parents:
diff changeset
1015 if (TraceIterativeGVN) {
a61af66fc99e Initial load
duke
parents:
diff changeset
1016 const Type* newtype = type_or_null(n);
a61af66fc99e Initial load
duke
parents:
diff changeset
1017 if (nn != n) {
a61af66fc99e Initial load
duke
parents:
diff changeset
1018 // print old node
a61af66fc99e Initial load
duke
parents:
diff changeset
1019 tty->print("< ");
a61af66fc99e Initial load
duke
parents:
diff changeset
1020 if (oldtype != newtype && oldtype != NULL) {
a61af66fc99e Initial load
duke
parents:
diff changeset
1021 oldtype->dump();
a61af66fc99e Initial load
duke
parents:
diff changeset
1022 }
a61af66fc99e Initial load
duke
parents:
diff changeset
1023 do { tty->print("\t"); } while (tty->position() < 16);
a61af66fc99e Initial load
duke
parents:
diff changeset
1024 tty->print("<");
a61af66fc99e Initial load
duke
parents:
diff changeset
1025 n->dump();
a61af66fc99e Initial load
duke
parents:
diff changeset
1026 }
a61af66fc99e Initial load
duke
parents:
diff changeset
1027 if (oldtype != newtype || nn != n) {
a61af66fc99e Initial load
duke
parents:
diff changeset
1028 // print new node and/or new type
a61af66fc99e Initial load
duke
parents:
diff changeset
1029 if (oldtype == NULL) {
a61af66fc99e Initial load
duke
parents:
diff changeset
1030 tty->print("* ");
a61af66fc99e Initial load
duke
parents:
diff changeset
1031 } else if (nn != n) {
a61af66fc99e Initial load
duke
parents:
diff changeset
1032 tty->print("> ");
a61af66fc99e Initial load
duke
parents:
diff changeset
1033 } else {
a61af66fc99e Initial load
duke
parents:
diff changeset
1034 tty->print("= ");
a61af66fc99e Initial load
duke
parents:
diff changeset
1035 }
a61af66fc99e Initial load
duke
parents:
diff changeset
1036 if (newtype == NULL) {
a61af66fc99e Initial load
duke
parents:
diff changeset
1037 tty->print("null");
a61af66fc99e Initial load
duke
parents:
diff changeset
1038 } else {
a61af66fc99e Initial load
duke
parents:
diff changeset
1039 newtype->dump();
a61af66fc99e Initial load
duke
parents:
diff changeset
1040 }
a61af66fc99e Initial load
duke
parents:
diff changeset
1041 do { tty->print("\t"); } while (tty->position() < 16);
a61af66fc99e Initial load
duke
parents:
diff changeset
1042 nn->dump();
a61af66fc99e Initial load
duke
parents:
diff changeset
1043 }
a61af66fc99e Initial load
duke
parents:
diff changeset
1044 if (Verbose && wlsize < _worklist.size()) {
a61af66fc99e Initial load
duke
parents:
diff changeset
1045 tty->print(" Push {");
a61af66fc99e Initial load
duke
parents:
diff changeset
1046 while (wlsize != _worklist.size()) {
a61af66fc99e Initial load
duke
parents:
diff changeset
1047 Node* pushed = _worklist.at(wlsize++);
a61af66fc99e Initial load
duke
parents:
diff changeset
1048 tty->print(" %d", pushed->_idx);
a61af66fc99e Initial load
duke
parents:
diff changeset
1049 }
a61af66fc99e Initial load
duke
parents:
diff changeset
1050 tty->print_cr(" }");
a61af66fc99e Initial load
duke
parents:
diff changeset
1051 }
a61af66fc99e Initial load
duke
parents:
diff changeset
1052 }
a61af66fc99e Initial load
duke
parents:
diff changeset
1053 if( VerifyIterativeGVN && nn != n ) {
a61af66fc99e Initial load
duke
parents:
diff changeset
1054 verify_step((Node*) NULL); // ignore n, it might be subsumed
a61af66fc99e Initial load
duke
parents:
diff changeset
1055 }
a61af66fc99e Initial load
duke
parents:
diff changeset
1056 #endif
a61af66fc99e Initial load
duke
parents:
diff changeset
1057 } else if (!n->is_top()) {
a61af66fc99e Initial load
duke
parents:
diff changeset
1058 remove_dead_node(n);
a61af66fc99e Initial load
duke
parents:
diff changeset
1059 }
a61af66fc99e Initial load
duke
parents:
diff changeset
1060 }
a61af66fc99e Initial load
duke
parents:
diff changeset
1061
a61af66fc99e Initial load
duke
parents:
diff changeset
1062 #ifndef PRODUCT
a61af66fc99e Initial load
duke
parents:
diff changeset
1063 C->verify_graph_edges();
a61af66fc99e Initial load
duke
parents:
diff changeset
1064 if( VerifyOpto && allow_progress() ) {
a61af66fc99e Initial load
duke
parents:
diff changeset
1065 // Must turn off allow_progress to enable assert and break recursion
a61af66fc99e Initial load
duke
parents:
diff changeset
1066 C->root()->verify();
a61af66fc99e Initial load
duke
parents:
diff changeset
1067 { // Check if any progress was missed using IterGVN
a61af66fc99e Initial load
duke
parents:
diff changeset
1068 // Def-Use info enables transformations not attempted in wash-pass
a61af66fc99e Initial load
duke
parents:
diff changeset
1069 // e.g. Region/Phi cleanup, ...
a61af66fc99e Initial load
duke
parents:
diff changeset
1070 // Null-check elision -- may not have reached fixpoint
a61af66fc99e Initial load
duke
parents:
diff changeset
1071 // do not propagate to dominated nodes
a61af66fc99e Initial load
duke
parents:
diff changeset
1072 ResourceMark rm;
a61af66fc99e Initial load
duke
parents:
diff changeset
1073 PhaseIterGVN igvn2(this,"Verify"); // Fresh and clean!
a61af66fc99e Initial load
duke
parents:
diff changeset
1074 // Fill worklist completely
a61af66fc99e Initial load
duke
parents:
diff changeset
1075 igvn2.init_worklist(C->root());
a61af66fc99e Initial load
duke
parents:
diff changeset
1076
a61af66fc99e Initial load
duke
parents:
diff changeset
1077 igvn2.set_allow_progress(false);
a61af66fc99e Initial load
duke
parents:
diff changeset
1078 igvn2.optimize();
a61af66fc99e Initial load
duke
parents:
diff changeset
1079 igvn2.set_allow_progress(true);
a61af66fc99e Initial load
duke
parents:
diff changeset
1080 }
a61af66fc99e Initial load
duke
parents:
diff changeset
1081 }
a61af66fc99e Initial load
duke
parents:
diff changeset
1082 if ( VerifyIterativeGVN && PrintOpto ) {
a61af66fc99e Initial load
duke
parents:
diff changeset
1083 if ( _verify_counter == _verify_full_passes )
a61af66fc99e Initial load
duke
parents:
diff changeset
1084 tty->print_cr("VerifyIterativeGVN: %d transforms and verify passes",
17937
78bbf4d43a14 8037816: Fix for 8036122 breaks build with Xcode5/clang
drchase
parents: 17671
diff changeset
1085 (int) _verify_full_passes);
0
a61af66fc99e Initial load
duke
parents:
diff changeset
1086 else
a61af66fc99e Initial load
duke
parents:
diff changeset
1087 tty->print_cr("VerifyIterativeGVN: %d transforms, %d full verify passes",
17937
78bbf4d43a14 8037816: Fix for 8036122 breaks build with Xcode5/clang
drchase
parents: 17671
diff changeset
1088 (int) _verify_counter, (int) _verify_full_passes);
0
a61af66fc99e Initial load
duke
parents:
diff changeset
1089 }
a61af66fc99e Initial load
duke
parents:
diff changeset
1090 #endif
a61af66fc99e Initial load
duke
parents:
diff changeset
1091 }
a61af66fc99e Initial load
duke
parents:
diff changeset
1092
a61af66fc99e Initial load
duke
parents:
diff changeset
1093
a61af66fc99e Initial load
duke
parents:
diff changeset
1094 //------------------register_new_node_with_optimizer---------------------------
a61af66fc99e Initial load
duke
parents:
diff changeset
1095 // Register a new node with the optimizer. Update the types array, the def-use
a61af66fc99e Initial load
duke
parents:
diff changeset
1096 // info. Put on worklist.
a61af66fc99e Initial load
duke
parents:
diff changeset
1097 Node* PhaseIterGVN::register_new_node_with_optimizer(Node* n, Node* orig) {
a61af66fc99e Initial load
duke
parents:
diff changeset
1098 set_type_bottom(n);
a61af66fc99e Initial load
duke
parents:
diff changeset
1099 _worklist.push(n);
a61af66fc99e Initial load
duke
parents:
diff changeset
1100 if (orig != NULL) C->copy_node_notes_to(n, orig);
a61af66fc99e Initial load
duke
parents:
diff changeset
1101 return n;
a61af66fc99e Initial load
duke
parents:
diff changeset
1102 }
a61af66fc99e Initial load
duke
parents:
diff changeset
1103
a61af66fc99e Initial load
duke
parents:
diff changeset
1104 //------------------------------transform--------------------------------------
a61af66fc99e Initial load
duke
parents:
diff changeset
1105 // Non-recursive: idealize Node 'n' with respect to its inputs and its value
a61af66fc99e Initial load
duke
parents:
diff changeset
1106 Node *PhaseIterGVN::transform( Node *n ) {
113
ba764ed4b6f2 6420645: Create a vm that uses compressed oops for up to 32gb heapsizes
coleenp
parents: 65
diff changeset
1107 if (_delay_transform) {
ba764ed4b6f2 6420645: Create a vm that uses compressed oops for up to 32gb heapsizes
coleenp
parents: 65
diff changeset
1108 // Register the node but don't optimize for now
ba764ed4b6f2 6420645: Create a vm that uses compressed oops for up to 32gb heapsizes
coleenp
parents: 65
diff changeset
1109 register_new_node_with_optimizer(n);
ba764ed4b6f2 6420645: Create a vm that uses compressed oops for up to 32gb heapsizes
coleenp
parents: 65
diff changeset
1110 return n;
ba764ed4b6f2 6420645: Create a vm that uses compressed oops for up to 32gb heapsizes
coleenp
parents: 65
diff changeset
1111 }
ba764ed4b6f2 6420645: Create a vm that uses compressed oops for up to 32gb heapsizes
coleenp
parents: 65
diff changeset
1112
0
a61af66fc99e Initial load
duke
parents:
diff changeset
1113 // If brand new node, make space in type array, and give it a type.
a61af66fc99e Initial load
duke
parents:
diff changeset
1114 ensure_type_or_null(n);
a61af66fc99e Initial load
duke
parents:
diff changeset
1115 if (type_or_null(n) == NULL) {
a61af66fc99e Initial load
duke
parents:
diff changeset
1116 set_type_bottom(n);
a61af66fc99e Initial load
duke
parents:
diff changeset
1117 }
a61af66fc99e Initial load
duke
parents:
diff changeset
1118
a61af66fc99e Initial load
duke
parents:
diff changeset
1119 return transform_old(n);
a61af66fc99e Initial load
duke
parents:
diff changeset
1120 }
a61af66fc99e Initial load
duke
parents:
diff changeset
1121
a61af66fc99e Initial load
duke
parents:
diff changeset
1122 //------------------------------transform_old----------------------------------
a61af66fc99e Initial load
duke
parents:
diff changeset
1123 Node *PhaseIterGVN::transform_old( Node *n ) {
a61af66fc99e Initial load
duke
parents:
diff changeset
1124 #ifndef PRODUCT
a61af66fc99e Initial load
duke
parents:
diff changeset
1125 debug_only(uint loop_count = 0;);
a61af66fc99e Initial load
duke
parents:
diff changeset
1126 set_transforms();
a61af66fc99e Initial load
duke
parents:
diff changeset
1127 #endif
a61af66fc99e Initial load
duke
parents:
diff changeset
1128 // Remove 'n' from hash table in case it gets modified
a61af66fc99e Initial load
duke
parents:
diff changeset
1129 _table.hash_delete(n);
a61af66fc99e Initial load
duke
parents:
diff changeset
1130 if( VerifyIterativeGVN ) {
a61af66fc99e Initial load
duke
parents:
diff changeset
1131 assert( !_table.find_index(n->_idx), "found duplicate entry in table");
a61af66fc99e Initial load
duke
parents:
diff changeset
1132 }
a61af66fc99e Initial load
duke
parents:
diff changeset
1133
a61af66fc99e Initial load
duke
parents:
diff changeset
1134 // Apply the Ideal call in a loop until it no longer applies
a61af66fc99e Initial load
duke
parents:
diff changeset
1135 Node *k = n;
a61af66fc99e Initial load
duke
parents:
diff changeset
1136 DEBUG_ONLY(dead_loop_check(k);)
305
ab075d07f1ba 6736417: Fastdebug C2 crashes in StoreBNode::Ideal
kvn
parents: 196
diff changeset
1137 DEBUG_ONLY(bool is_new = (k->outcnt() == 0);)
0
a61af66fc99e Initial load
duke
parents:
diff changeset
1138 Node *i = k->Ideal(this, /*can_reshape=*/true);
305
ab075d07f1ba 6736417: Fastdebug C2 crashes in StoreBNode::Ideal
kvn
parents: 196
diff changeset
1139 assert(i != k || is_new || i->outcnt() > 0, "don't return dead nodes");
0
a61af66fc99e Initial load
duke
parents:
diff changeset
1140 #ifndef PRODUCT
a61af66fc99e Initial load
duke
parents:
diff changeset
1141 if( VerifyIterativeGVN )
a61af66fc99e Initial load
duke
parents:
diff changeset
1142 verify_step(k);
a61af66fc99e Initial load
duke
parents:
diff changeset
1143 if( i && VerifyOpto ) {
a61af66fc99e Initial load
duke
parents:
diff changeset
1144 if( !allow_progress() ) {
a61af66fc99e Initial load
duke
parents:
diff changeset
1145 if (i->is_Add() && i->outcnt() == 1) {
a61af66fc99e Initial load
duke
parents:
diff changeset
1146 // Switched input to left side because this is the only use
a61af66fc99e Initial load
duke
parents:
diff changeset
1147 } else if( i->is_If() && (i->in(0) == NULL) ) {
a61af66fc99e Initial load
duke
parents:
diff changeset
1148 // This IF is dead because it is dominated by an equivalent IF When
a61af66fc99e Initial load
duke
parents:
diff changeset
1149 // dominating if changed, info is not propagated sparsely to 'this'
a61af66fc99e Initial load
duke
parents:
diff changeset
1150 // Propagating this info further will spuriously identify other
a61af66fc99e Initial load
duke
parents:
diff changeset
1151 // progress.
a61af66fc99e Initial load
duke
parents:
diff changeset
1152 return i;
a61af66fc99e Initial load
duke
parents:
diff changeset
1153 } else
a61af66fc99e Initial load
duke
parents:
diff changeset
1154 set_progress();
a61af66fc99e Initial load
duke
parents:
diff changeset
1155 } else
a61af66fc99e Initial load
duke
parents:
diff changeset
1156 set_progress();
a61af66fc99e Initial load
duke
parents:
diff changeset
1157 }
a61af66fc99e Initial load
duke
parents:
diff changeset
1158 #endif
a61af66fc99e Initial load
duke
parents:
diff changeset
1159
a61af66fc99e Initial load
duke
parents:
diff changeset
1160 while( i ) {
a61af66fc99e Initial load
duke
parents:
diff changeset
1161 #ifndef PRODUCT
a61af66fc99e Initial load
duke
parents:
diff changeset
1162 debug_only( if( loop_count >= K ) i->dump(4); )
a61af66fc99e Initial load
duke
parents:
diff changeset
1163 assert(loop_count < K, "infinite loop in PhaseIterGVN::transform");
a61af66fc99e Initial load
duke
parents:
diff changeset
1164 debug_only( loop_count++; )
a61af66fc99e Initial load
duke
parents:
diff changeset
1165 #endif
a61af66fc99e Initial load
duke
parents:
diff changeset
1166 assert((i->_idx >= k->_idx) || i->is_top(), "Idealize should return new nodes, use Identity to return old nodes");
a61af66fc99e Initial load
duke
parents:
diff changeset
1167 // Made a change; put users of original Node on worklist
a61af66fc99e Initial load
duke
parents:
diff changeset
1168 add_users_to_worklist( k );
a61af66fc99e Initial load
duke
parents:
diff changeset
1169 // Replacing root of transform tree?
a61af66fc99e Initial load
duke
parents:
diff changeset
1170 if( k != i ) {
a61af66fc99e Initial load
duke
parents:
diff changeset
1171 // Make users of old Node now use new.
a61af66fc99e Initial load
duke
parents:
diff changeset
1172 subsume_node( k, i );
a61af66fc99e Initial load
duke
parents:
diff changeset
1173 k = i;
a61af66fc99e Initial load
duke
parents:
diff changeset
1174 }
a61af66fc99e Initial load
duke
parents:
diff changeset
1175 DEBUG_ONLY(dead_loop_check(k);)
a61af66fc99e Initial load
duke
parents:
diff changeset
1176 // Try idealizing again
305
ab075d07f1ba 6736417: Fastdebug C2 crashes in StoreBNode::Ideal
kvn
parents: 196
diff changeset
1177 DEBUG_ONLY(is_new = (k->outcnt() == 0);)
0
a61af66fc99e Initial load
duke
parents:
diff changeset
1178 i = k->Ideal(this, /*can_reshape=*/true);
305
ab075d07f1ba 6736417: Fastdebug C2 crashes in StoreBNode::Ideal
kvn
parents: 196
diff changeset
1179 assert(i != k || is_new || i->outcnt() > 0, "don't return dead nodes");
0
a61af66fc99e Initial load
duke
parents:
diff changeset
1180 #ifndef PRODUCT
a61af66fc99e Initial load
duke
parents:
diff changeset
1181 if( VerifyIterativeGVN )
a61af66fc99e Initial load
duke
parents:
diff changeset
1182 verify_step(k);
a61af66fc99e Initial load
duke
parents:
diff changeset
1183 if( i && VerifyOpto ) set_progress();
a61af66fc99e Initial load
duke
parents:
diff changeset
1184 #endif
a61af66fc99e Initial load
duke
parents:
diff changeset
1185 }
a61af66fc99e Initial load
duke
parents:
diff changeset
1186
a61af66fc99e Initial load
duke
parents:
diff changeset
1187 // If brand new node, make space in type array.
a61af66fc99e Initial load
duke
parents:
diff changeset
1188 ensure_type_or_null(k);
a61af66fc99e Initial load
duke
parents:
diff changeset
1189
a61af66fc99e Initial load
duke
parents:
diff changeset
1190 // See what kind of values 'k' takes on at runtime
a61af66fc99e Initial load
duke
parents:
diff changeset
1191 const Type *t = k->Value(this);
a61af66fc99e Initial load
duke
parents:
diff changeset
1192 assert(t != NULL, "value sanity");
a61af66fc99e Initial load
duke
parents:
diff changeset
1193
a61af66fc99e Initial load
duke
parents:
diff changeset
1194 // Since I just called 'Value' to compute the set of run-time values
a61af66fc99e Initial load
duke
parents:
diff changeset
1195 // for this Node, and 'Value' is non-local (and therefore expensive) I'll
a61af66fc99e Initial load
duke
parents:
diff changeset
1196 // cache Value. Later requests for the local phase->type of this Node can
a61af66fc99e Initial load
duke
parents:
diff changeset
1197 // use the cached Value instead of suffering with 'bottom_type'.
a61af66fc99e Initial load
duke
parents:
diff changeset
1198 if (t != type_or_null(k)) {
a61af66fc99e Initial load
duke
parents:
diff changeset
1199 NOT_PRODUCT( set_progress(); )
a61af66fc99e Initial load
duke
parents:
diff changeset
1200 NOT_PRODUCT( inc_new_values();)
a61af66fc99e Initial load
duke
parents:
diff changeset
1201 set_type(k, t);
a61af66fc99e Initial load
duke
parents:
diff changeset
1202 // If k is a TypeNode, capture any more-precise type permanently into Node
a61af66fc99e Initial load
duke
parents:
diff changeset
1203 k->raise_bottom_type(t);
a61af66fc99e Initial load
duke
parents:
diff changeset
1204 // Move users of node to worklist
a61af66fc99e Initial load
duke
parents:
diff changeset
1205 add_users_to_worklist( k );
a61af66fc99e Initial load
duke
parents:
diff changeset
1206 }
a61af66fc99e Initial load
duke
parents:
diff changeset
1207
a61af66fc99e Initial load
duke
parents:
diff changeset
1208 // If 'k' computes a constant, replace it with a constant
a61af66fc99e Initial load
duke
parents:
diff changeset
1209 if( t->singleton() && !k->is_Con() ) {
a61af66fc99e Initial load
duke
parents:
diff changeset
1210 NOT_PRODUCT( set_progress(); )
a61af66fc99e Initial load
duke
parents:
diff changeset
1211 Node *con = makecon(t); // Make a constant
a61af66fc99e Initial load
duke
parents:
diff changeset
1212 add_users_to_worklist( k );
a61af66fc99e Initial load
duke
parents:
diff changeset
1213 subsume_node( k, con ); // Everybody using k now uses con
a61af66fc99e Initial load
duke
parents:
diff changeset
1214 return con;
a61af66fc99e Initial load
duke
parents:
diff changeset
1215 }
a61af66fc99e Initial load
duke
parents:
diff changeset
1216
a61af66fc99e Initial load
duke
parents:
diff changeset
1217 // Now check for Identities
a61af66fc99e Initial load
duke
parents:
diff changeset
1218 i = k->Identity(this); // Look for a nearby replacement
a61af66fc99e Initial load
duke
parents:
diff changeset
1219 if( i != k ) { // Found? Return replacement!
a61af66fc99e Initial load
duke
parents:
diff changeset
1220 NOT_PRODUCT( set_progress(); )
a61af66fc99e Initial load
duke
parents:
diff changeset
1221 add_users_to_worklist( k );
a61af66fc99e Initial load
duke
parents:
diff changeset
1222 subsume_node( k, i ); // Everybody using k now uses i
a61af66fc99e Initial load
duke
parents:
diff changeset
1223 return i;
a61af66fc99e Initial load
duke
parents:
diff changeset
1224 }
a61af66fc99e Initial load
duke
parents:
diff changeset
1225
a61af66fc99e Initial load
duke
parents:
diff changeset
1226 // Global Value Numbering
a61af66fc99e Initial load
duke
parents:
diff changeset
1227 i = hash_find_insert(k); // Check for pre-existing node
a61af66fc99e Initial load
duke
parents:
diff changeset
1228 if( i && (i != k) ) {
a61af66fc99e Initial load
duke
parents:
diff changeset
1229 // Return the pre-existing node if it isn't dead
a61af66fc99e Initial load
duke
parents:
diff changeset
1230 NOT_PRODUCT( set_progress(); )
a61af66fc99e Initial load
duke
parents:
diff changeset
1231 add_users_to_worklist( k );
a61af66fc99e Initial load
duke
parents:
diff changeset
1232 subsume_node( k, i ); // Everybody using k now uses i
a61af66fc99e Initial load
duke
parents:
diff changeset
1233 return i;
a61af66fc99e Initial load
duke
parents:
diff changeset
1234 }
a61af66fc99e Initial load
duke
parents:
diff changeset
1235
a61af66fc99e Initial load
duke
parents:
diff changeset
1236 // Return Idealized original
a61af66fc99e Initial load
duke
parents:
diff changeset
1237 return k;
a61af66fc99e Initial load
duke
parents:
diff changeset
1238 }
a61af66fc99e Initial load
duke
parents:
diff changeset
1239
a61af66fc99e Initial load
duke
parents:
diff changeset
1240 //---------------------------------saturate------------------------------------
a61af66fc99e Initial load
duke
parents:
diff changeset
1241 const Type* PhaseIterGVN::saturate(const Type* new_type, const Type* old_type,
a61af66fc99e Initial load
duke
parents:
diff changeset
1242 const Type* limit_type) const {
a61af66fc99e Initial load
duke
parents:
diff changeset
1243 return new_type->narrow(old_type);
a61af66fc99e Initial load
duke
parents:
diff changeset
1244 }
a61af66fc99e Initial load
duke
parents:
diff changeset
1245
a61af66fc99e Initial load
duke
parents:
diff changeset
1246 //------------------------------remove_globally_dead_node----------------------
a61af66fc99e Initial load
duke
parents:
diff changeset
1247 // Kill a globally dead Node. All uses are also globally dead and are
a61af66fc99e Initial load
duke
parents:
diff changeset
1248 // aggressively trimmed.
a61af66fc99e Initial load
duke
parents:
diff changeset
1249 void PhaseIterGVN::remove_globally_dead_node( Node *dead ) {
6244
611e8a669a2c 7147464: Java crashed while executing method with over 8k of dneg operations
dlong
parents: 4064
diff changeset
1250 enum DeleteProgress {
611e8a669a2c 7147464: Java crashed while executing method with over 8k of dneg operations
dlong
parents: 4064
diff changeset
1251 PROCESS_INPUTS,
611e8a669a2c 7147464: Java crashed while executing method with over 8k of dneg operations
dlong
parents: 4064
diff changeset
1252 PROCESS_OUTPUTS
611e8a669a2c 7147464: Java crashed while executing method with over 8k of dneg operations
dlong
parents: 4064
diff changeset
1253 };
611e8a669a2c 7147464: Java crashed while executing method with over 8k of dneg operations
dlong
parents: 4064
diff changeset
1254 assert(_stack.is_empty(), "not empty");
611e8a669a2c 7147464: Java crashed while executing method with over 8k of dneg operations
dlong
parents: 4064
diff changeset
1255 _stack.push(dead, PROCESS_INPUTS);
611e8a669a2c 7147464: Java crashed while executing method with over 8k of dneg operations
dlong
parents: 4064
diff changeset
1256
611e8a669a2c 7147464: Java crashed while executing method with over 8k of dneg operations
dlong
parents: 4064
diff changeset
1257 while (_stack.is_nonempty()) {
611e8a669a2c 7147464: Java crashed while executing method with over 8k of dneg operations
dlong
parents: 4064
diff changeset
1258 dead = _stack.node();
611e8a669a2c 7147464: Java crashed while executing method with over 8k of dneg operations
dlong
parents: 4064
diff changeset
1259 uint progress_state = _stack.index();
611e8a669a2c 7147464: Java crashed while executing method with over 8k of dneg operations
dlong
parents: 4064
diff changeset
1260 assert(dead != C->root(), "killing root, eh?");
611e8a669a2c 7147464: Java crashed while executing method with over 8k of dneg operations
dlong
parents: 4064
diff changeset
1261 assert(!dead->is_top(), "add check for top when pushing");
611e8a669a2c 7147464: Java crashed while executing method with over 8k of dneg operations
dlong
parents: 4064
diff changeset
1262 NOT_PRODUCT( set_progress(); )
611e8a669a2c 7147464: Java crashed while executing method with over 8k of dneg operations
dlong
parents: 4064
diff changeset
1263 if (progress_state == PROCESS_INPUTS) {
611e8a669a2c 7147464: Java crashed while executing method with over 8k of dneg operations
dlong
parents: 4064
diff changeset
1264 // After following inputs, continue to outputs
611e8a669a2c 7147464: Java crashed while executing method with over 8k of dneg operations
dlong
parents: 4064
diff changeset
1265 _stack.set_index(PROCESS_OUTPUTS);
611e8a669a2c 7147464: Java crashed while executing method with over 8k of dneg operations
dlong
parents: 4064
diff changeset
1266 if (!dead->is_Con()) { // Don't kill cons but uses
611e8a669a2c 7147464: Java crashed while executing method with over 8k of dneg operations
dlong
parents: 4064
diff changeset
1267 bool recurse = false;
611e8a669a2c 7147464: Java crashed while executing method with over 8k of dneg operations
dlong
parents: 4064
diff changeset
1268 // Remove from hash table
611e8a669a2c 7147464: Java crashed while executing method with over 8k of dneg operations
dlong
parents: 4064
diff changeset
1269 _table.hash_delete( dead );
611e8a669a2c 7147464: Java crashed while executing method with over 8k of dneg operations
dlong
parents: 4064
diff changeset
1270 // Smash all inputs to 'dead', isolating him completely
8868
30f42e691e70 8004640: C2 assert failure in memnode.cpp: NULL+offs not RAW address
kvn
parents: 8116
diff changeset
1271 for (uint i = 0; i < dead->req(); i++) {
6244
611e8a669a2c 7147464: Java crashed while executing method with over 8k of dneg operations
dlong
parents: 4064
diff changeset
1272 Node *in = dead->in(i);
8868
30f42e691e70 8004640: C2 assert failure in memnode.cpp: NULL+offs not RAW address
kvn
parents: 8116
diff changeset
1273 if (in != NULL && in != C->top()) { // Points to something?
30f42e691e70 8004640: C2 assert failure in memnode.cpp: NULL+offs not RAW address
kvn
parents: 8116
diff changeset
1274 int nrep = dead->replace_edge(in, NULL); // Kill edges
30f42e691e70 8004640: C2 assert failure in memnode.cpp: NULL+offs not RAW address
kvn
parents: 8116
diff changeset
1275 assert((nrep > 0), "sanity");
30f42e691e70 8004640: C2 assert failure in memnode.cpp: NULL+offs not RAW address
kvn
parents: 8116
diff changeset
1276 if (in->outcnt() == 0) { // Made input go dead?
6244
611e8a669a2c 7147464: Java crashed while executing method with over 8k of dneg operations
dlong
parents: 4064
diff changeset
1277 _stack.push(in, PROCESS_INPUTS); // Recursively remove
611e8a669a2c 7147464: Java crashed while executing method with over 8k of dneg operations
dlong
parents: 4064
diff changeset
1278 recurse = true;
611e8a669a2c 7147464: Java crashed while executing method with over 8k of dneg operations
dlong
parents: 4064
diff changeset
1279 } else if (in->outcnt() == 1 &&
611e8a669a2c 7147464: Java crashed while executing method with over 8k of dneg operations
dlong
parents: 4064
diff changeset
1280 in->has_special_unique_user()) {
611e8a669a2c 7147464: Java crashed while executing method with over 8k of dneg operations
dlong
parents: 4064
diff changeset
1281 _worklist.push(in->unique_out());
611e8a669a2c 7147464: Java crashed while executing method with over 8k of dneg operations
dlong
parents: 4064
diff changeset
1282 } else if (in->outcnt() <= 2 && dead->is_Phi()) {
8868
30f42e691e70 8004640: C2 assert failure in memnode.cpp: NULL+offs not RAW address
kvn
parents: 8116
diff changeset
1283 if (in->Opcode() == Op_Region) {
6244
611e8a669a2c 7147464: Java crashed while executing method with over 8k of dneg operations
dlong
parents: 4064
diff changeset
1284 _worklist.push(in);
8868
30f42e691e70 8004640: C2 assert failure in memnode.cpp: NULL+offs not RAW address
kvn
parents: 8116
diff changeset
1285 } else if (in->is_Store()) {
6244
611e8a669a2c 7147464: Java crashed while executing method with over 8k of dneg operations
dlong
parents: 4064
diff changeset
1286 DUIterator_Fast imax, i = in->fast_outs(imax);
611e8a669a2c 7147464: Java crashed while executing method with over 8k of dneg operations
dlong
parents: 4064
diff changeset
1287 _worklist.push(in->fast_out(i));
611e8a669a2c 7147464: Java crashed while executing method with over 8k of dneg operations
dlong
parents: 4064
diff changeset
1288 i++;
8868
30f42e691e70 8004640: C2 assert failure in memnode.cpp: NULL+offs not RAW address
kvn
parents: 8116
diff changeset
1289 if (in->outcnt() == 2) {
6244
611e8a669a2c 7147464: Java crashed while executing method with over 8k of dneg operations
dlong
parents: 4064
diff changeset
1290 _worklist.push(in->fast_out(i));
611e8a669a2c 7147464: Java crashed while executing method with over 8k of dneg operations
dlong
parents: 4064
diff changeset
1291 i++;
611e8a669a2c 7147464: Java crashed while executing method with over 8k of dneg operations
dlong
parents: 4064
diff changeset
1292 }
611e8a669a2c 7147464: Java crashed while executing method with over 8k of dneg operations
dlong
parents: 4064
diff changeset
1293 assert(!(i < imax), "sanity");
611e8a669a2c 7147464: Java crashed while executing method with over 8k of dneg operations
dlong
parents: 4064
diff changeset
1294 }
0
a61af66fc99e Initial load
duke
parents:
diff changeset
1295 }
8116
6931f425c517 8007294: ReduceFieldZeroing doesn't check for dependent load and can lead to incorrect execution
roland
parents: 8048
diff changeset
1296 if (ReduceFieldZeroing && dead->is_Load() && i == MemNode::Memory &&
6931f425c517 8007294: ReduceFieldZeroing doesn't check for dependent load and can lead to incorrect execution
roland
parents: 8048
diff changeset
1297 in->is_Proj() && in->in(0) != NULL && in->in(0)->is_Initialize()) {
6931f425c517 8007294: ReduceFieldZeroing doesn't check for dependent load and can lead to incorrect execution
roland
parents: 8048
diff changeset
1298 // A Load that directly follows an InitializeNode is
6931f425c517 8007294: ReduceFieldZeroing doesn't check for dependent load and can lead to incorrect execution
roland
parents: 8048
diff changeset
1299 // going away. The Stores that follow are candidates
6931f425c517 8007294: ReduceFieldZeroing doesn't check for dependent load and can lead to incorrect execution
roland
parents: 8048
diff changeset
1300 // again to be captured by the InitializeNode.
6931f425c517 8007294: ReduceFieldZeroing doesn't check for dependent load and can lead to incorrect execution
roland
parents: 8048
diff changeset
1301 for (DUIterator_Fast jmax, j = in->fast_outs(jmax); j < jmax; j++) {
6931f425c517 8007294: ReduceFieldZeroing doesn't check for dependent load and can lead to incorrect execution
roland
parents: 8048
diff changeset
1302 Node *n = in->fast_out(j);
6931f425c517 8007294: ReduceFieldZeroing doesn't check for dependent load and can lead to incorrect execution
roland
parents: 8048
diff changeset
1303 if (n->is_Store()) {
6931f425c517 8007294: ReduceFieldZeroing doesn't check for dependent load and can lead to incorrect execution
roland
parents: 8048
diff changeset
1304 _worklist.push(n);
6931f425c517 8007294: ReduceFieldZeroing doesn't check for dependent load and can lead to incorrect execution
roland
parents: 8048
diff changeset
1305 }
6931f425c517 8007294: ReduceFieldZeroing doesn't check for dependent load and can lead to incorrect execution
roland
parents: 8048
diff changeset
1306 }
6931f425c517 8007294: ReduceFieldZeroing doesn't check for dependent load and can lead to incorrect execution
roland
parents: 8048
diff changeset
1307 }
8868
30f42e691e70 8004640: C2 assert failure in memnode.cpp: NULL+offs not RAW address
kvn
parents: 8116
diff changeset
1308 } // if (in != NULL && in != C->top())
30f42e691e70 8004640: C2 assert failure in memnode.cpp: NULL+offs not RAW address
kvn
parents: 8116
diff changeset
1309 } // for (uint i = 0; i < dead->req(); i++)
6244
611e8a669a2c 7147464: Java crashed while executing method with over 8k of dneg operations
dlong
parents: 4064
diff changeset
1310 if (recurse) {
611e8a669a2c 7147464: Java crashed while executing method with over 8k of dneg operations
dlong
parents: 4064
diff changeset
1311 continue;
611e8a669a2c 7147464: Java crashed while executing method with over 8k of dneg operations
dlong
parents: 4064
diff changeset
1312 }
8868
30f42e691e70 8004640: C2 assert failure in memnode.cpp: NULL+offs not RAW address
kvn
parents: 8116
diff changeset
1313 } // if (!dead->is_Con())
30f42e691e70 8004640: C2 assert failure in memnode.cpp: NULL+offs not RAW address
kvn
parents: 8116
diff changeset
1314 } // if (progress_state == PROCESS_INPUTS)
0
a61af66fc99e Initial load
duke
parents:
diff changeset
1315
6244
611e8a669a2c 7147464: Java crashed while executing method with over 8k of dneg operations
dlong
parents: 4064
diff changeset
1316 // Aggressively kill globally dead uses
611e8a669a2c 7147464: Java crashed while executing method with over 8k of dneg operations
dlong
parents: 4064
diff changeset
1317 // (Rather than pushing all the outs at once, we push one at a time,
611e8a669a2c 7147464: Java crashed while executing method with over 8k of dneg operations
dlong
parents: 4064
diff changeset
1318 // plus the parent to resume later, because of the indefinite number
611e8a669a2c 7147464: Java crashed while executing method with over 8k of dneg operations
dlong
parents: 4064
diff changeset
1319 // of edge deletions per loop trip.)
611e8a669a2c 7147464: Java crashed while executing method with over 8k of dneg operations
dlong
parents: 4064
diff changeset
1320 if (dead->outcnt() > 0) {
8868
30f42e691e70 8004640: C2 assert failure in memnode.cpp: NULL+offs not RAW address
kvn
parents: 8116
diff changeset
1321 // Recursively remove output edges
6244
611e8a669a2c 7147464: Java crashed while executing method with over 8k of dneg operations
dlong
parents: 4064
diff changeset
1322 _stack.push(dead->raw_out(0), PROCESS_INPUTS);
611e8a669a2c 7147464: Java crashed while executing method with over 8k of dneg operations
dlong
parents: 4064
diff changeset
1323 } else {
8868
30f42e691e70 8004640: C2 assert failure in memnode.cpp: NULL+offs not RAW address
kvn
parents: 8116
diff changeset
1324 // Finished disconnecting all input and output edges.
6244
611e8a669a2c 7147464: Java crashed while executing method with over 8k of dneg operations
dlong
parents: 4064
diff changeset
1325 _stack.pop();
8868
30f42e691e70 8004640: C2 assert failure in memnode.cpp: NULL+offs not RAW address
kvn
parents: 8116
diff changeset
1326 // Remove dead node from iterative worklist
30f42e691e70 8004640: C2 assert failure in memnode.cpp: NULL+offs not RAW address
kvn
parents: 8116
diff changeset
1327 _worklist.remove(dead);
30f42e691e70 8004640: C2 assert failure in memnode.cpp: NULL+offs not RAW address
kvn
parents: 8116
diff changeset
1328 // Constant node that has no out-edges and has only one in-edge from
30f42e691e70 8004640: C2 assert failure in memnode.cpp: NULL+offs not RAW address
kvn
parents: 8116
diff changeset
1329 // root is usually dead. However, sometimes reshaping walk makes
30f42e691e70 8004640: C2 assert failure in memnode.cpp: NULL+offs not RAW address
kvn
parents: 8116
diff changeset
1330 // it reachable by adding use edges. So, we will NOT count Con nodes
30f42e691e70 8004640: C2 assert failure in memnode.cpp: NULL+offs not RAW address
kvn
parents: 8116
diff changeset
1331 // as dead to be conservative about the dead node count at any
30f42e691e70 8004640: C2 assert failure in memnode.cpp: NULL+offs not RAW address
kvn
parents: 8116
diff changeset
1332 // given time.
30f42e691e70 8004640: C2 assert failure in memnode.cpp: NULL+offs not RAW address
kvn
parents: 8116
diff changeset
1333 if (!dead->is_Con()) {
30f42e691e70 8004640: C2 assert failure in memnode.cpp: NULL+offs not RAW address
kvn
parents: 8116
diff changeset
1334 C->record_dead_node(dead->_idx);
30f42e691e70 8004640: C2 assert failure in memnode.cpp: NULL+offs not RAW address
kvn
parents: 8116
diff changeset
1335 }
30f42e691e70 8004640: C2 assert failure in memnode.cpp: NULL+offs not RAW address
kvn
parents: 8116
diff changeset
1336 if (dead->is_macro()) {
30f42e691e70 8004640: C2 assert failure in memnode.cpp: NULL+offs not RAW address
kvn
parents: 8116
diff changeset
1337 C->remove_macro_node(dead);
30f42e691e70 8004640: C2 assert failure in memnode.cpp: NULL+offs not RAW address
kvn
parents: 8116
diff changeset
1338 }
30f42e691e70 8004640: C2 assert failure in memnode.cpp: NULL+offs not RAW address
kvn
parents: 8116
diff changeset
1339 if (dead->is_expensive()) {
30f42e691e70 8004640: C2 assert failure in memnode.cpp: NULL+offs not RAW address
kvn
parents: 8116
diff changeset
1340 C->remove_expensive_node(dead);
30f42e691e70 8004640: C2 assert failure in memnode.cpp: NULL+offs not RAW address
kvn
parents: 8116
diff changeset
1341 }
23872
94ec11846b18 6675699: need comprehensive fix for unconstrained ConvI2L with narrowed type
thartmann
parents: 23421
diff changeset
1342 CastIINode* cast = dead->isa_CastII();
94ec11846b18 6675699: need comprehensive fix for unconstrained ConvI2L with narrowed type
thartmann
parents: 23421
diff changeset
1343 if (cast != NULL && cast->has_range_check()) {
94ec11846b18 6675699: need comprehensive fix for unconstrained ConvI2L with narrowed type
thartmann
parents: 23421
diff changeset
1344 C->remove_range_check_cast(cast);
94ec11846b18 6675699: need comprehensive fix for unconstrained ConvI2L with narrowed type
thartmann
parents: 23421
diff changeset
1345 }
0
a61af66fc99e Initial load
duke
parents:
diff changeset
1346 }
8868
30f42e691e70 8004640: C2 assert failure in memnode.cpp: NULL+offs not RAW address
kvn
parents: 8116
diff changeset
1347 } // while (_stack.is_nonempty())
0
a61af66fc99e Initial load
duke
parents:
diff changeset
1348 }
a61af66fc99e Initial load
duke
parents:
diff changeset
1349
a61af66fc99e Initial load
duke
parents:
diff changeset
1350 //------------------------------subsume_node-----------------------------------
a61af66fc99e Initial load
duke
parents:
diff changeset
1351 // Remove users from node 'old' and add them to node 'nn'.
a61af66fc99e Initial load
duke
parents:
diff changeset
1352 void PhaseIterGVN::subsume_node( Node *old, Node *nn ) {
a61af66fc99e Initial load
duke
parents:
diff changeset
1353 assert( old != hash_find(old), "should already been removed" );
a61af66fc99e Initial load
duke
parents:
diff changeset
1354 assert( old != C->top(), "cannot subsume top node");
a61af66fc99e Initial load
duke
parents:
diff changeset
1355 // Copy debug or profile information to the new version:
a61af66fc99e Initial load
duke
parents:
diff changeset
1356 C->copy_node_notes_to(nn, old);
a61af66fc99e Initial load
duke
parents:
diff changeset
1357 // Move users of node 'old' to node 'nn'
a61af66fc99e Initial load
duke
parents:
diff changeset
1358 for (DUIterator_Last imin, i = old->last_outs(imin); i >= imin; ) {
a61af66fc99e Initial load
duke
parents:
diff changeset
1359 Node* use = old->last_out(i); // for each use...
a61af66fc99e Initial load
duke
parents:
diff changeset
1360 // use might need re-hashing (but it won't if it's a new node)
a61af66fc99e Initial load
duke
parents:
diff changeset
1361 bool is_in_table = _table.hash_delete( use );
a61af66fc99e Initial load
duke
parents:
diff changeset
1362 // Update use-def info as well
a61af66fc99e Initial load
duke
parents:
diff changeset
1363 // We remove all occurrences of old within use->in,
a61af66fc99e Initial load
duke
parents:
diff changeset
1364 // so as to avoid rehashing any node more than once.
a61af66fc99e Initial load
duke
parents:
diff changeset
1365 // The hash table probe swamps any outer loop overhead.
a61af66fc99e Initial load
duke
parents:
diff changeset
1366 uint num_edges = 0;
a61af66fc99e Initial load
duke
parents:
diff changeset
1367 for (uint jmax = use->len(), j = 0; j < jmax; j++) {
a61af66fc99e Initial load
duke
parents:
diff changeset
1368 if (use->in(j) == old) {
a61af66fc99e Initial load
duke
parents:
diff changeset
1369 use->set_req(j, nn);
a61af66fc99e Initial load
duke
parents:
diff changeset
1370 ++num_edges;
a61af66fc99e Initial load
duke
parents:
diff changeset
1371 }
a61af66fc99e Initial load
duke
parents:
diff changeset
1372 }
a61af66fc99e Initial load
duke
parents:
diff changeset
1373 // Insert into GVN hash table if unique
a61af66fc99e Initial load
duke
parents:
diff changeset
1374 // If a duplicate, 'use' will be cleaned up when pulled off worklist
a61af66fc99e Initial load
duke
parents:
diff changeset
1375 if( is_in_table ) {
a61af66fc99e Initial load
duke
parents:
diff changeset
1376 hash_find_insert(use);
a61af66fc99e Initial load
duke
parents:
diff changeset
1377 }
a61af66fc99e Initial load
duke
parents:
diff changeset
1378 i -= num_edges; // we deleted 1 or more copies of this edge
a61af66fc99e Initial load
duke
parents:
diff changeset
1379 }
a61af66fc99e Initial load
duke
parents:
diff changeset
1380
a61af66fc99e Initial load
duke
parents:
diff changeset
1381 // Smash all inputs to 'old', isolating him completely
6804
e626685e9f6c 7193318: C2: remove number of inputs requirement from Node's new operator
kvn
parents: 6244
diff changeset
1382 Node *temp = new (C) Node(1);
0
a61af66fc99e Initial load
duke
parents:
diff changeset
1383 temp->init_req(0,nn); // Add a use to nn to prevent him from dying
a61af66fc99e Initial load
duke
parents:
diff changeset
1384 remove_dead_node( old );
a61af66fc99e Initial load
duke
parents:
diff changeset
1385 temp->del_req(0); // Yank bogus edge
a61af66fc99e Initial load
duke
parents:
diff changeset
1386 #ifndef PRODUCT
a61af66fc99e Initial load
duke
parents:
diff changeset
1387 if( VerifyIterativeGVN ) {
a61af66fc99e Initial load
duke
parents:
diff changeset
1388 for ( int i = 0; i < _verify_window_size; i++ ) {
a61af66fc99e Initial load
duke
parents:
diff changeset
1389 if ( _verify_window[i] == old )
a61af66fc99e Initial load
duke
parents:
diff changeset
1390 _verify_window[i] = nn;
a61af66fc99e Initial load
duke
parents:
diff changeset
1391 }
a61af66fc99e Initial load
duke
parents:
diff changeset
1392 }
a61af66fc99e Initial load
duke
parents:
diff changeset
1393 #endif
a61af66fc99e Initial load
duke
parents:
diff changeset
1394 _worklist.remove(temp); // this can be necessary
a61af66fc99e Initial load
duke
parents:
diff changeset
1395 temp->destruct(); // reuse the _idx of this little guy
a61af66fc99e Initial load
duke
parents:
diff changeset
1396 }
a61af66fc99e Initial load
duke
parents:
diff changeset
1397
a61af66fc99e Initial load
duke
parents:
diff changeset
1398 //------------------------------add_users_to_worklist--------------------------
a61af66fc99e Initial load
duke
parents:
diff changeset
1399 void PhaseIterGVN::add_users_to_worklist0( Node *n ) {
a61af66fc99e Initial load
duke
parents:
diff changeset
1400 for (DUIterator_Fast imax, i = n->fast_outs(imax); i < imax; i++) {
a61af66fc99e Initial load
duke
parents:
diff changeset
1401 _worklist.push(n->fast_out(i)); // Push on worklist
a61af66fc99e Initial load
duke
parents:
diff changeset
1402 }
a61af66fc99e Initial load
duke
parents:
diff changeset
1403 }
a61af66fc99e Initial load
duke
parents:
diff changeset
1404
a61af66fc99e Initial load
duke
parents:
diff changeset
1405 void PhaseIterGVN::add_users_to_worklist( Node *n ) {
a61af66fc99e Initial load
duke
parents:
diff changeset
1406 add_users_to_worklist0(n);
a61af66fc99e Initial load
duke
parents:
diff changeset
1407
a61af66fc99e Initial load
duke
parents:
diff changeset
1408 // Move users of node to worklist
a61af66fc99e Initial load
duke
parents:
diff changeset
1409 for (DUIterator_Fast imax, i = n->fast_outs(imax); i < imax; i++) {
a61af66fc99e Initial load
duke
parents:
diff changeset
1410 Node* use = n->fast_out(i); // Get use
a61af66fc99e Initial load
duke
parents:
diff changeset
1411
a61af66fc99e Initial load
duke
parents:
diff changeset
1412 if( use->is_Multi() || // Multi-definer? Push projs on worklist
a61af66fc99e Initial load
duke
parents:
diff changeset
1413 use->is_Store() ) // Enable store/load same address
a61af66fc99e Initial load
duke
parents:
diff changeset
1414 add_users_to_worklist0(use);
a61af66fc99e Initial load
duke
parents:
diff changeset
1415
a61af66fc99e Initial load
duke
parents:
diff changeset
1416 // If we changed the receiver type to a call, we need to revisit
a61af66fc99e Initial load
duke
parents:
diff changeset
1417 // the Catch following the call. It's looking for a non-NULL
a61af66fc99e Initial load
duke
parents:
diff changeset
1418 // receiver to know when to enable the regular fall-through path
a61af66fc99e Initial load
duke
parents:
diff changeset
1419 // in addition to the NullPtrException path.
a61af66fc99e Initial load
duke
parents:
diff changeset
1420 if (use->is_CallDynamicJava() && n == use->in(TypeFunc::Parms)) {
a61af66fc99e Initial load
duke
parents:
diff changeset
1421 Node* p = use->as_CallDynamicJava()->proj_out(TypeFunc::Control);
a61af66fc99e Initial load
duke
parents:
diff changeset
1422 if (p != NULL) {
a61af66fc99e Initial load
duke
parents:
diff changeset
1423 add_users_to_worklist0(p);
a61af66fc99e Initial load
duke
parents:
diff changeset
1424 }
a61af66fc99e Initial load
duke
parents:
diff changeset
1425 }
a61af66fc99e Initial load
duke
parents:
diff changeset
1426
20680
5b8e0f84f00f 8054478: C2: Incorrectly compiled char[] array access crashes JVM
roland
parents: 17938
diff changeset
1427 uint use_op = use->Opcode();
5b8e0f84f00f 8054478: C2: Incorrectly compiled char[] array access crashes JVM
roland
parents: 17938
diff changeset
1428 if(use->is_Cmp()) { // Enable CMP/BOOL optimization
0
a61af66fc99e Initial load
duke
parents:
diff changeset
1429 add_users_to_worklist(use); // Put Bool on worklist
a61af66fc99e Initial load
duke
parents:
diff changeset
1430 if (use->outcnt() > 0) {
a61af66fc99e Initial load
duke
parents:
diff changeset
1431 Node* bol = use->raw_out(0);
a61af66fc99e Initial load
duke
parents:
diff changeset
1432 if (bol->outcnt() > 0) {
a61af66fc99e Initial load
duke
parents:
diff changeset
1433 Node* iff = bol->raw_out(0);
20680
5b8e0f84f00f 8054478: C2: Incorrectly compiled char[] array access crashes JVM
roland
parents: 17938
diff changeset
1434 if (use_op == Op_CmpI &&
5b8e0f84f00f 8054478: C2: Incorrectly compiled char[] array access crashes JVM
roland
parents: 17938
diff changeset
1435 iff->is_CountedLoopEnd()) {
5b8e0f84f00f 8054478: C2: Incorrectly compiled char[] array access crashes JVM
roland
parents: 17938
diff changeset
1436 CountedLoopEndNode* cle = iff->as_CountedLoopEnd();
5b8e0f84f00f 8054478: C2: Incorrectly compiled char[] array access crashes JVM
roland
parents: 17938
diff changeset
1437 if (cle->limit() == n && cle->phi() != NULL) {
5b8e0f84f00f 8054478: C2: Incorrectly compiled char[] array access crashes JVM
roland
parents: 17938
diff changeset
1438 // If an opaque node feeds into the limit condition of a
5b8e0f84f00f 8054478: C2: Incorrectly compiled char[] array access crashes JVM
roland
parents: 17938
diff changeset
1439 // CountedLoop, we need to process the Phi node for the
5b8e0f84f00f 8054478: C2: Incorrectly compiled char[] array access crashes JVM
roland
parents: 17938
diff changeset
1440 // induction variable when the opaque node is removed:
5b8e0f84f00f 8054478: C2: Incorrectly compiled char[] array access crashes JVM
roland
parents: 17938
diff changeset
1441 // the range of values taken by the Phi is now known and
5b8e0f84f00f 8054478: C2: Incorrectly compiled char[] array access crashes JVM
roland
parents: 17938
diff changeset
1442 // so its type is also known.
5b8e0f84f00f 8054478: C2: Incorrectly compiled char[] array access crashes JVM
roland
parents: 17938
diff changeset
1443 _worklist.push(cle->phi());
5b8e0f84f00f 8054478: C2: Incorrectly compiled char[] array access crashes JVM
roland
parents: 17938
diff changeset
1444 }
5b8e0f84f00f 8054478: C2: Incorrectly compiled char[] array access crashes JVM
roland
parents: 17938
diff changeset
1445 } else if (iff->outcnt() == 2) {
5b8e0f84f00f 8054478: C2: Incorrectly compiled char[] array access crashes JVM
roland
parents: 17938
diff changeset
1446 // Look for the 'is_x2logic' pattern: "x ? : 0 : 1" and put the
5b8e0f84f00f 8054478: C2: Incorrectly compiled char[] array access crashes JVM
roland
parents: 17938
diff changeset
1447 // phi merging either 0 or 1 onto the worklist
0
a61af66fc99e Initial load
duke
parents:
diff changeset
1448 Node* ifproj0 = iff->raw_out(0);
a61af66fc99e Initial load
duke
parents:
diff changeset
1449 Node* ifproj1 = iff->raw_out(1);
a61af66fc99e Initial load
duke
parents:
diff changeset
1450 if (ifproj0->outcnt() > 0 && ifproj1->outcnt() > 0) {
a61af66fc99e Initial load
duke
parents:
diff changeset
1451 Node* region0 = ifproj0->raw_out(0);
a61af66fc99e Initial load
duke
parents:
diff changeset
1452 Node* region1 = ifproj1->raw_out(0);
a61af66fc99e Initial load
duke
parents:
diff changeset
1453 if( region0 == region1 )
a61af66fc99e Initial load
duke
parents:
diff changeset
1454 add_users_to_worklist0(region0);
a61af66fc99e Initial load
duke
parents:
diff changeset
1455 }
a61af66fc99e Initial load
duke
parents:
diff changeset
1456 }
a61af66fc99e Initial load
duke
parents:
diff changeset
1457 }
a61af66fc99e Initial load
duke
parents:
diff changeset
1458 }
20680
5b8e0f84f00f 8054478: C2: Incorrectly compiled char[] array access crashes JVM
roland
parents: 17938
diff changeset
1459 if (use_op == Op_CmpI) {
5b8e0f84f00f 8054478: C2: Incorrectly compiled char[] array access crashes JVM
roland
parents: 17938
diff changeset
1460 Node* in1 = use->in(1);
5b8e0f84f00f 8054478: C2: Incorrectly compiled char[] array access crashes JVM
roland
parents: 17938
diff changeset
1461 for (uint i = 0; i < in1->outcnt(); i++) {
5b8e0f84f00f 8054478: C2: Incorrectly compiled char[] array access crashes JVM
roland
parents: 17938
diff changeset
1462 if (in1->raw_out(i)->Opcode() == Op_CastII) {
5b8e0f84f00f 8054478: C2: Incorrectly compiled char[] array access crashes JVM
roland
parents: 17938
diff changeset
1463 Node* castii = in1->raw_out(i);
5b8e0f84f00f 8054478: C2: Incorrectly compiled char[] array access crashes JVM
roland
parents: 17938
diff changeset
1464 if (castii->in(0) != NULL && castii->in(0)->in(0) != NULL && castii->in(0)->in(0)->is_If()) {
5b8e0f84f00f 8054478: C2: Incorrectly compiled char[] array access crashes JVM
roland
parents: 17938
diff changeset
1465 Node* ifnode = castii->in(0)->in(0);
20681
4c228230f1d6 8066045: opto/node.hpp:355, assert(i < _max) failed: oob: i=1, _max=1
roland
parents: 20680
diff changeset
1466 if (ifnode->in(1) != NULL && ifnode->in(1)->is_Bool() && ifnode->in(1)->in(1) == use) {
20680
5b8e0f84f00f 8054478: C2: Incorrectly compiled char[] array access crashes JVM
roland
parents: 17938
diff changeset
1467 // Reprocess a CastII node that may depend on an
5b8e0f84f00f 8054478: C2: Incorrectly compiled char[] array access crashes JVM
roland
parents: 17938
diff changeset
1468 // opaque node value when the opaque node is
5b8e0f84f00f 8054478: C2: Incorrectly compiled char[] array access crashes JVM
roland
parents: 17938
diff changeset
1469 // removed. In case it carries a dependency we can do
5b8e0f84f00f 8054478: C2: Incorrectly compiled char[] array access crashes JVM
roland
parents: 17938
diff changeset
1470 // a better job of computing its type.
5b8e0f84f00f 8054478: C2: Incorrectly compiled char[] array access crashes JVM
roland
parents: 17938
diff changeset
1471 _worklist.push(castii);
5b8e0f84f00f 8054478: C2: Incorrectly compiled char[] array access crashes JVM
roland
parents: 17938
diff changeset
1472 }
5b8e0f84f00f 8054478: C2: Incorrectly compiled char[] array access crashes JVM
roland
parents: 17938
diff changeset
1473 }
5b8e0f84f00f 8054478: C2: Incorrectly compiled char[] array access crashes JVM
roland
parents: 17938
diff changeset
1474 }
5b8e0f84f00f 8054478: C2: Incorrectly compiled char[] array access crashes JVM
roland
parents: 17938
diff changeset
1475 }
5b8e0f84f00f 8054478: C2: Incorrectly compiled char[] array access crashes JVM
roland
parents: 17938
diff changeset
1476 }
0
a61af66fc99e Initial load
duke
parents:
diff changeset
1477 }
a61af66fc99e Initial load
duke
parents:
diff changeset
1478
a61af66fc99e Initial load
duke
parents:
diff changeset
1479 // If changed Cast input, check Phi users for simple cycles
65
99269dbf4ba8 6674588: (Escape Analysis) Improve Escape Analysis code
kvn
parents: 41
diff changeset
1480 if( use->is_ConstraintCast() || use->is_CheckCastPP() ) {
0
a61af66fc99e Initial load
duke
parents:
diff changeset
1481 for (DUIterator_Fast i2max, i2 = use->fast_outs(i2max); i2 < i2max; i2++) {
a61af66fc99e Initial load
duke
parents:
diff changeset
1482 Node* u = use->fast_out(i2);
a61af66fc99e Initial load
duke
parents:
diff changeset
1483 if (u->is_Phi())
a61af66fc99e Initial load
duke
parents:
diff changeset
1484 _worklist.push(u);
a61af66fc99e Initial load
duke
parents:
diff changeset
1485 }
a61af66fc99e Initial load
duke
parents:
diff changeset
1486 }
a61af66fc99e Initial load
duke
parents:
diff changeset
1487 // If changed LShift inputs, check RShift users for useless sign-ext
a61af66fc99e Initial load
duke
parents:
diff changeset
1488 if( use_op == Op_LShiftI ) {
a61af66fc99e Initial load
duke
parents:
diff changeset
1489 for (DUIterator_Fast i2max, i2 = use->fast_outs(i2max); i2 < i2max; i2++) {
a61af66fc99e Initial load
duke
parents:
diff changeset
1490 Node* u = use->fast_out(i2);
a61af66fc99e Initial load
duke
parents:
diff changeset
1491 if (u->Opcode() == Op_RShiftI)
a61af66fc99e Initial load
duke
parents:
diff changeset
1492 _worklist.push(u);
a61af66fc99e Initial load
duke
parents:
diff changeset
1493 }
a61af66fc99e Initial load
duke
parents:
diff changeset
1494 }
17936
968a17f18337 8042786: Proper fix for 8032566
kvn
parents: 17671
diff changeset
1495 // If changed AddI/SubI inputs, check CmpU for range check optimization.
968a17f18337 8042786: Proper fix for 8032566
kvn
parents: 17671
diff changeset
1496 if (use_op == Op_AddI || use_op == Op_SubI) {
968a17f18337 8042786: Proper fix for 8032566
kvn
parents: 17671
diff changeset
1497 for (DUIterator_Fast i2max, i2 = use->fast_outs(i2max); i2 < i2max; i2++) {
968a17f18337 8042786: Proper fix for 8032566
kvn
parents: 17671
diff changeset
1498 Node* u = use->fast_out(i2);
968a17f18337 8042786: Proper fix for 8032566
kvn
parents: 17671
diff changeset
1499 if (u->is_Cmp() && (u->Opcode() == Op_CmpU)) {
968a17f18337 8042786: Proper fix for 8032566
kvn
parents: 17671
diff changeset
1500 _worklist.push(u);
968a17f18337 8042786: Proper fix for 8032566
kvn
parents: 17671
diff changeset
1501 }
968a17f18337 8042786: Proper fix for 8032566
kvn
parents: 17671
diff changeset
1502 }
968a17f18337 8042786: Proper fix for 8032566
kvn
parents: 17671
diff changeset
1503 }
0
a61af66fc99e Initial load
duke
parents:
diff changeset
1504 // If changed AddP inputs, check Stores for loop invariant
a61af66fc99e Initial load
duke
parents:
diff changeset
1505 if( use_op == Op_AddP ) {
a61af66fc99e Initial load
duke
parents:
diff changeset
1506 for (DUIterator_Fast i2max, i2 = use->fast_outs(i2max); i2 < i2max; i2++) {
a61af66fc99e Initial load
duke
parents:
diff changeset
1507 Node* u = use->fast_out(i2);
a61af66fc99e Initial load
duke
parents:
diff changeset
1508 if (u->is_Mem())
a61af66fc99e Initial load
duke
parents:
diff changeset
1509 _worklist.push(u);
a61af66fc99e Initial load
duke
parents:
diff changeset
1510 }
a61af66fc99e Initial load
duke
parents:
diff changeset
1511 }
a61af66fc99e Initial load
duke
parents:
diff changeset
1512 // If changed initialization activity, check dependent Stores
a61af66fc99e Initial load
duke
parents:
diff changeset
1513 if (use_op == Op_Allocate || use_op == Op_AllocateArray) {
a61af66fc99e Initial load
duke
parents:
diff changeset
1514 InitializeNode* init = use->as_Allocate()->initialization();
a61af66fc99e Initial load
duke
parents:
diff changeset
1515 if (init != NULL) {
a61af66fc99e Initial load
duke
parents:
diff changeset
1516 Node* imem = init->proj_out(TypeFunc::Memory);
a61af66fc99e Initial load
duke
parents:
diff changeset
1517 if (imem != NULL) add_users_to_worklist0(imem);
a61af66fc99e Initial load
duke
parents:
diff changeset
1518 }
a61af66fc99e Initial load
duke
parents:
diff changeset
1519 }
a61af66fc99e Initial load
duke
parents:
diff changeset
1520 if (use_op == Op_Initialize) {
a61af66fc99e Initial load
duke
parents:
diff changeset
1521 Node* imem = use->as_Initialize()->proj_out(TypeFunc::Memory);
a61af66fc99e Initial load
duke
parents:
diff changeset
1522 if (imem != NULL) add_users_to_worklist0(imem);
a61af66fc99e Initial load
duke
parents:
diff changeset
1523 }
a61af66fc99e Initial load
duke
parents:
diff changeset
1524 }
a61af66fc99e Initial load
duke
parents:
diff changeset
1525 }
a61af66fc99e Initial load
duke
parents:
diff changeset
1526
12966
b2ee5dc63353 8024070: C2 needs some form of type speculation
roland
parents: 12167
diff changeset
1527 /**
b2ee5dc63353 8024070: C2 needs some form of type speculation
roland
parents: 12167
diff changeset
1528 * Remove the speculative part of all types that we know of
b2ee5dc63353 8024070: C2 needs some form of type speculation
roland
parents: 12167
diff changeset
1529 */
b2ee5dc63353 8024070: C2 needs some form of type speculation
roland
parents: 12167
diff changeset
1530 void PhaseIterGVN::remove_speculative_types() {
b2ee5dc63353 8024070: C2 needs some form of type speculation
roland
parents: 12167
diff changeset
1531 assert(UseTypeSpeculation, "speculation is off");
b2ee5dc63353 8024070: C2 needs some form of type speculation
roland
parents: 12167
diff changeset
1532 for (uint i = 0; i < _types.Size(); i++) {
b2ee5dc63353 8024070: C2 needs some form of type speculation
roland
parents: 12167
diff changeset
1533 const Type* t = _types.fast_lookup(i);
17671
de95063c0e34 8027422: assert(_gvn.type(obj)->higher_equal(tjp)) failed: cast_up is no longer needed
roland
parents: 17467
diff changeset
1534 if (t != NULL) {
de95063c0e34 8027422: assert(_gvn.type(obj)->higher_equal(tjp)) failed: cast_up is no longer needed
roland
parents: 17467
diff changeset
1535 _types.map(i, t->remove_speculative());
12966
b2ee5dc63353 8024070: C2 needs some form of type speculation
roland
parents: 12167
diff changeset
1536 }
b2ee5dc63353 8024070: C2 needs some form of type speculation
roland
parents: 12167
diff changeset
1537 }
17671
de95063c0e34 8027422: assert(_gvn.type(obj)->higher_equal(tjp)) failed: cast_up is no longer needed
roland
parents: 17467
diff changeset
1538 _table.check_no_speculative_types();
12966
b2ee5dc63353 8024070: C2 needs some form of type speculation
roland
parents: 12167
diff changeset
1539 }
b2ee5dc63353 8024070: C2 needs some form of type speculation
roland
parents: 12167
diff changeset
1540
0
a61af66fc99e Initial load
duke
parents:
diff changeset
1541 //=============================================================================
a61af66fc99e Initial load
duke
parents:
diff changeset
1542 #ifndef PRODUCT
a61af66fc99e Initial load
duke
parents:
diff changeset
1543 uint PhaseCCP::_total_invokes = 0;
a61af66fc99e Initial load
duke
parents:
diff changeset
1544 uint PhaseCCP::_total_constants = 0;
a61af66fc99e Initial load
duke
parents:
diff changeset
1545 #endif
a61af66fc99e Initial load
duke
parents:
diff changeset
1546 //------------------------------PhaseCCP---------------------------------------
a61af66fc99e Initial load
duke
parents:
diff changeset
1547 // Conditional Constant Propagation, ala Wegman & Zadeck
a61af66fc99e Initial load
duke
parents:
diff changeset
1548 PhaseCCP::PhaseCCP( PhaseIterGVN *igvn ) : PhaseIterGVN(igvn) {
a61af66fc99e Initial load
duke
parents:
diff changeset
1549 NOT_PRODUCT( clear_constants(); )
a61af66fc99e Initial load
duke
parents:
diff changeset
1550 assert( _worklist.size() == 0, "" );
a61af66fc99e Initial load
duke
parents:
diff changeset
1551 // Clear out _nodes from IterGVN. Must be clear to transform call.
a61af66fc99e Initial load
duke
parents:
diff changeset
1552 _nodes.clear(); // Clear out from IterGVN
a61af66fc99e Initial load
duke
parents:
diff changeset
1553 analyze();
a61af66fc99e Initial load
duke
parents:
diff changeset
1554 }
a61af66fc99e Initial load
duke
parents:
diff changeset
1555
a61af66fc99e Initial load
duke
parents:
diff changeset
1556 #ifndef PRODUCT
a61af66fc99e Initial load
duke
parents:
diff changeset
1557 //------------------------------~PhaseCCP--------------------------------------
a61af66fc99e Initial load
duke
parents:
diff changeset
1558 PhaseCCP::~PhaseCCP() {
a61af66fc99e Initial load
duke
parents:
diff changeset
1559 inc_invokes();
a61af66fc99e Initial load
duke
parents:
diff changeset
1560 _total_constants += count_constants();
a61af66fc99e Initial load
duke
parents:
diff changeset
1561 }
a61af66fc99e Initial load
duke
parents:
diff changeset
1562 #endif
a61af66fc99e Initial load
duke
parents:
diff changeset
1563
a61af66fc99e Initial load
duke
parents:
diff changeset
1564
a61af66fc99e Initial load
duke
parents:
diff changeset
1565 #ifdef ASSERT
a61af66fc99e Initial load
duke
parents:
diff changeset
1566 static bool ccp_type_widens(const Type* t, const Type* t0) {
a61af66fc99e Initial load
duke
parents:
diff changeset
1567 assert(t->meet(t0) == t, "Not monotonic");
a61af66fc99e Initial load
duke
parents:
diff changeset
1568 switch (t->base() == t0->base() ? t->base() : Type::Top) {
a61af66fc99e Initial load
duke
parents:
diff changeset
1569 case Type::Int:
a61af66fc99e Initial load
duke
parents:
diff changeset
1570 assert(t0->isa_int()->_widen <= t->isa_int()->_widen, "widen increases");
a61af66fc99e Initial load
duke
parents:
diff changeset
1571 break;
a61af66fc99e Initial load
duke
parents:
diff changeset
1572 case Type::Long:
a61af66fc99e Initial load
duke
parents:
diff changeset
1573 assert(t0->isa_long()->_widen <= t->isa_long()->_widen, "widen increases");
a61af66fc99e Initial load
duke
parents:
diff changeset
1574 break;
a61af66fc99e Initial load
duke
parents:
diff changeset
1575 }
a61af66fc99e Initial load
duke
parents:
diff changeset
1576 return true;
a61af66fc99e Initial load
duke
parents:
diff changeset
1577 }
a61af66fc99e Initial load
duke
parents:
diff changeset
1578 #endif //ASSERT
a61af66fc99e Initial load
duke
parents:
diff changeset
1579
a61af66fc99e Initial load
duke
parents:
diff changeset
1580 //------------------------------analyze----------------------------------------
a61af66fc99e Initial load
duke
parents:
diff changeset
1581 void PhaseCCP::analyze() {
a61af66fc99e Initial load
duke
parents:
diff changeset
1582 // Initialize all types to TOP, optimistic analysis
a61af66fc99e Initial load
duke
parents:
diff changeset
1583 for (int i = C->unique() - 1; i >= 0; i--) {
a61af66fc99e Initial load
duke
parents:
diff changeset
1584 _types.map(i,Type::TOP);
a61af66fc99e Initial load
duke
parents:
diff changeset
1585 }
a61af66fc99e Initial load
duke
parents:
diff changeset
1586
a61af66fc99e Initial load
duke
parents:
diff changeset
1587 // Push root onto worklist
a61af66fc99e Initial load
duke
parents:
diff changeset
1588 Unique_Node_List worklist;
a61af66fc99e Initial load
duke
parents:
diff changeset
1589 worklist.push(C->root());
a61af66fc99e Initial load
duke
parents:
diff changeset
1590
a61af66fc99e Initial load
duke
parents:
diff changeset
1591 // Pull from worklist; compute new value; push changes out.
a61af66fc99e Initial load
duke
parents:
diff changeset
1592 // This loop is the meat of CCP.
a61af66fc99e Initial load
duke
parents:
diff changeset
1593 while( worklist.size() ) {
a61af66fc99e Initial load
duke
parents:
diff changeset
1594 Node *n = worklist.pop();
a61af66fc99e Initial load
duke
parents:
diff changeset
1595 const Type *t = n->Value(this);
a61af66fc99e Initial load
duke
parents:
diff changeset
1596 if (t != type(n)) {
a61af66fc99e Initial load
duke
parents:
diff changeset
1597 assert(ccp_type_widens(t, type(n)), "ccp type must widen");
a61af66fc99e Initial load
duke
parents:
diff changeset
1598 #ifndef PRODUCT
a61af66fc99e Initial load
duke
parents:
diff changeset
1599 if( TracePhaseCCP ) {
a61af66fc99e Initial load
duke
parents:
diff changeset
1600 t->dump();
a61af66fc99e Initial load
duke
parents:
diff changeset
1601 do { tty->print("\t"); } while (tty->position() < 16);
a61af66fc99e Initial load
duke
parents:
diff changeset
1602 n->dump();
a61af66fc99e Initial load
duke
parents:
diff changeset
1603 }
a61af66fc99e Initial load
duke
parents:
diff changeset
1604 #endif
a61af66fc99e Initial load
duke
parents:
diff changeset
1605 set_type(n, t);
a61af66fc99e Initial load
duke
parents:
diff changeset
1606 for (DUIterator_Fast imax, i = n->fast_outs(imax); i < imax; i++) {
a61af66fc99e Initial load
duke
parents:
diff changeset
1607 Node* m = n->fast_out(i); // Get user
23043
03596ae35800 8060036: C2: CmpU nodes can end up with wrong type information
aeriksso
parents: 20681
diff changeset
1608 if (m->is_Region()) { // New path to Region? Must recheck Phis too
0
a61af66fc99e Initial load
duke
parents:
diff changeset
1609 for (DUIterator_Fast i2max, i2 = m->fast_outs(i2max); i2 < i2max; i2++) {
a61af66fc99e Initial load
duke
parents:
diff changeset
1610 Node* p = m->fast_out(i2); // Propagate changes to uses
23043
03596ae35800 8060036: C2: CmpU nodes can end up with wrong type information
aeriksso
parents: 20681
diff changeset
1611 if (p->bottom_type() != type(p)) { // If not already bottomed out
0
a61af66fc99e Initial load
duke
parents:
diff changeset
1612 worklist.push(p); // Propagate change to user
23043
03596ae35800 8060036: C2: CmpU nodes can end up with wrong type information
aeriksso
parents: 20681
diff changeset
1613 }
0
a61af66fc99e Initial load
duke
parents:
diff changeset
1614 }
a61af66fc99e Initial load
duke
parents:
diff changeset
1615 }
605
98cb887364d3 6810672: Comment typos
twisti
parents: 305
diff changeset
1616 // If we changed the receiver type to a call, we need to revisit
0
a61af66fc99e Initial load
duke
parents:
diff changeset
1617 // the Catch following the call. It's looking for a non-NULL
a61af66fc99e Initial load
duke
parents:
diff changeset
1618 // receiver to know when to enable the regular fall-through path
a61af66fc99e Initial load
duke
parents:
diff changeset
1619 // in addition to the NullPtrException path
a61af66fc99e Initial load
duke
parents:
diff changeset
1620 if (m->is_Call()) {
a61af66fc99e Initial load
duke
parents:
diff changeset
1621 for (DUIterator_Fast i2max, i2 = m->fast_outs(i2max); i2 < i2max; i2++) {
a61af66fc99e Initial load
duke
parents:
diff changeset
1622 Node* p = m->fast_out(i2); // Propagate changes to uses
23043
03596ae35800 8060036: C2: CmpU nodes can end up with wrong type information
aeriksso
parents: 20681
diff changeset
1623 if (p->is_Proj() && p->as_Proj()->_con == TypeFunc::Control && p->outcnt() == 1) {
0
a61af66fc99e Initial load
duke
parents:
diff changeset
1624 worklist.push(p->unique_out());
23043
03596ae35800 8060036: C2: CmpU nodes can end up with wrong type information
aeriksso
parents: 20681
diff changeset
1625 }
0
a61af66fc99e Initial load
duke
parents:
diff changeset
1626 }
a61af66fc99e Initial load
duke
parents:
diff changeset
1627 }
23043
03596ae35800 8060036: C2: CmpU nodes can end up with wrong type information
aeriksso
parents: 20681
diff changeset
1628 if (m->bottom_type() != type(m)) { // If not already bottomed out
0
a61af66fc99e Initial load
duke
parents:
diff changeset
1629 worklist.push(m); // Propagate change to user
23043
03596ae35800 8060036: C2: CmpU nodes can end up with wrong type information
aeriksso
parents: 20681
diff changeset
1630 }
03596ae35800 8060036: C2: CmpU nodes can end up with wrong type information
aeriksso
parents: 20681
diff changeset
1631
03596ae35800 8060036: C2: CmpU nodes can end up with wrong type information
aeriksso
parents: 20681
diff changeset
1632 // CmpU nodes can get their type information from two nodes up in the
03596ae35800 8060036: C2: CmpU nodes can end up with wrong type information
aeriksso
parents: 20681
diff changeset
1633 // graph (instead of from the nodes immediately above). Make sure they
03596ae35800 8060036: C2: CmpU nodes can end up with wrong type information
aeriksso
parents: 20681
diff changeset
1634 // are added to the worklist if nodes they depend on are updated, since
03596ae35800 8060036: C2: CmpU nodes can end up with wrong type information
aeriksso
parents: 20681
diff changeset
1635 // they could be missed and get wrong types otherwise.
03596ae35800 8060036: C2: CmpU nodes can end up with wrong type information
aeriksso
parents: 20681
diff changeset
1636 uint m_op = m->Opcode();
03596ae35800 8060036: C2: CmpU nodes can end up with wrong type information
aeriksso
parents: 20681
diff changeset
1637 if (m_op == Op_AddI || m_op == Op_SubI) {
03596ae35800 8060036: C2: CmpU nodes can end up with wrong type information
aeriksso
parents: 20681
diff changeset
1638 for (DUIterator_Fast i2max, i2 = m->fast_outs(i2max); i2 < i2max; i2++) {
03596ae35800 8060036: C2: CmpU nodes can end up with wrong type information
aeriksso
parents: 20681
diff changeset
1639 Node* p = m->fast_out(i2); // Propagate changes to uses
03596ae35800 8060036: C2: CmpU nodes can end up with wrong type information
aeriksso
parents: 20681
diff changeset
1640 if (p->Opcode() == Op_CmpU) {
03596ae35800 8060036: C2: CmpU nodes can end up with wrong type information
aeriksso
parents: 20681
diff changeset
1641 // Got a CmpU which might need the new type information from node n.
03596ae35800 8060036: C2: CmpU nodes can end up with wrong type information
aeriksso
parents: 20681
diff changeset
1642 if(p->bottom_type() != type(p)) { // If not already bottomed out
03596ae35800 8060036: C2: CmpU nodes can end up with wrong type information
aeriksso
parents: 20681
diff changeset
1643 worklist.push(p); // Propagate change to user
03596ae35800 8060036: C2: CmpU nodes can end up with wrong type information
aeriksso
parents: 20681
diff changeset
1644 }
03596ae35800 8060036: C2: CmpU nodes can end up with wrong type information
aeriksso
parents: 20681
diff changeset
1645 }
03596ae35800 8060036: C2: CmpU nodes can end up with wrong type information
aeriksso
parents: 20681
diff changeset
1646 }
03596ae35800 8060036: C2: CmpU nodes can end up with wrong type information
aeriksso
parents: 20681
diff changeset
1647 }
0
a61af66fc99e Initial load
duke
parents:
diff changeset
1648 }
a61af66fc99e Initial load
duke
parents:
diff changeset
1649 }
a61af66fc99e Initial load
duke
parents:
diff changeset
1650 }
a61af66fc99e Initial load
duke
parents:
diff changeset
1651 }
a61af66fc99e Initial load
duke
parents:
diff changeset
1652
a61af66fc99e Initial load
duke
parents:
diff changeset
1653 //------------------------------do_transform-----------------------------------
a61af66fc99e Initial load
duke
parents:
diff changeset
1654 // Top level driver for the recursive transformer
a61af66fc99e Initial load
duke
parents:
diff changeset
1655 void PhaseCCP::do_transform() {
a61af66fc99e Initial load
duke
parents:
diff changeset
1656 // Correct leaves of new-space Nodes; they point to old-space.
a61af66fc99e Initial load
duke
parents:
diff changeset
1657 C->set_root( transform(C->root())->as_Root() );
a61af66fc99e Initial load
duke
parents:
diff changeset
1658 assert( C->top(), "missing TOP node" );
a61af66fc99e Initial load
duke
parents:
diff changeset
1659 assert( C->root(), "missing root" );
a61af66fc99e Initial load
duke
parents:
diff changeset
1660 }
a61af66fc99e Initial load
duke
parents:
diff changeset
1661
a61af66fc99e Initial load
duke
parents:
diff changeset
1662 //------------------------------transform--------------------------------------
a61af66fc99e Initial load
duke
parents:
diff changeset
1663 // Given a Node in old-space, clone him into new-space.
a61af66fc99e Initial load
duke
parents:
diff changeset
1664 // Convert any of his old-space children into new-space children.
a61af66fc99e Initial load
duke
parents:
diff changeset
1665 Node *PhaseCCP::transform( Node *n ) {
a61af66fc99e Initial load
duke
parents:
diff changeset
1666 Node *new_node = _nodes[n->_idx]; // Check for transformed node
a61af66fc99e Initial load
duke
parents:
diff changeset
1667 if( new_node != NULL )
a61af66fc99e Initial load
duke
parents:
diff changeset
1668 return new_node; // Been there, done that, return old answer
a61af66fc99e Initial load
duke
parents:
diff changeset
1669 new_node = transform_once(n); // Check for constant
a61af66fc99e Initial load
duke
parents:
diff changeset
1670 _nodes.map( n->_idx, new_node ); // Flag as having been cloned
a61af66fc99e Initial load
duke
parents:
diff changeset
1671
a61af66fc99e Initial load
duke
parents:
diff changeset
1672 // Allocate stack of size _nodes.Size()/2 to avoid frequent realloc
23421
c1091733abe6 8011858: Use Compile::live_nodes() instead of Compile::unique() in appropriate places
zmajo
parents: 23043
diff changeset
1673 GrowableArray <Node *> trstack(C->live_nodes() >> 1);
0
a61af66fc99e Initial load
duke
parents:
diff changeset
1674
a61af66fc99e Initial load
duke
parents:
diff changeset
1675 trstack.push(new_node); // Process children of cloned node
a61af66fc99e Initial load
duke
parents:
diff changeset
1676 while ( trstack.is_nonempty() ) {
a61af66fc99e Initial load
duke
parents:
diff changeset
1677 Node *clone = trstack.pop();
a61af66fc99e Initial load
duke
parents:
diff changeset
1678 uint cnt = clone->req();
a61af66fc99e Initial load
duke
parents:
diff changeset
1679 for( uint i = 0; i < cnt; i++ ) { // For all inputs do
a61af66fc99e Initial load
duke
parents:
diff changeset
1680 Node *input = clone->in(i);
a61af66fc99e Initial load
duke
parents:
diff changeset
1681 if( input != NULL ) { // Ignore NULLs
a61af66fc99e Initial load
duke
parents:
diff changeset
1682 Node *new_input = _nodes[input->_idx]; // Check for cloned input node
a61af66fc99e Initial load
duke
parents:
diff changeset
1683 if( new_input == NULL ) {
a61af66fc99e Initial load
duke
parents:
diff changeset
1684 new_input = transform_once(input); // Check for constant
a61af66fc99e Initial load
duke
parents:
diff changeset
1685 _nodes.map( input->_idx, new_input );// Flag as having been cloned
a61af66fc99e Initial load
duke
parents:
diff changeset
1686 trstack.push(new_input);
a61af66fc99e Initial load
duke
parents:
diff changeset
1687 }
a61af66fc99e Initial load
duke
parents:
diff changeset
1688 assert( new_input == clone->in(i), "insanity check");
a61af66fc99e Initial load
duke
parents:
diff changeset
1689 }
a61af66fc99e Initial load
duke
parents:
diff changeset
1690 }
a61af66fc99e Initial load
duke
parents:
diff changeset
1691 }
a61af66fc99e Initial load
duke
parents:
diff changeset
1692 return new_node;
a61af66fc99e Initial load
duke
parents:
diff changeset
1693 }
a61af66fc99e Initial load
duke
parents:
diff changeset
1694
a61af66fc99e Initial load
duke
parents:
diff changeset
1695
a61af66fc99e Initial load
duke
parents:
diff changeset
1696 //------------------------------transform_once---------------------------------
a61af66fc99e Initial load
duke
parents:
diff changeset
1697 // For PhaseCCP, transformation is IDENTITY unless Node computed a constant.
a61af66fc99e Initial load
duke
parents:
diff changeset
1698 Node *PhaseCCP::transform_once( Node *n ) {
a61af66fc99e Initial load
duke
parents:
diff changeset
1699 const Type *t = type(n);
a61af66fc99e Initial load
duke
parents:
diff changeset
1700 // Constant? Use constant Node instead
a61af66fc99e Initial load
duke
parents:
diff changeset
1701 if( t->singleton() ) {
a61af66fc99e Initial load
duke
parents:
diff changeset
1702 Node *nn = n; // Default is to return the original constant
a61af66fc99e Initial load
duke
parents:
diff changeset
1703 if( t == Type::TOP ) {
a61af66fc99e Initial load
duke
parents:
diff changeset
1704 // cache my top node on the Compile instance
a61af66fc99e Initial load
duke
parents:
diff changeset
1705 if( C->cached_top_node() == NULL || C->cached_top_node()->in(0) == NULL ) {
a61af66fc99e Initial load
duke
parents:
diff changeset
1706 C->set_cached_top_node( ConNode::make(C, Type::TOP) );
a61af66fc99e Initial load
duke
parents:
diff changeset
1707 set_type(C->top(), Type::TOP);
a61af66fc99e Initial load
duke
parents:
diff changeset
1708 }
a61af66fc99e Initial load
duke
parents:
diff changeset
1709 nn = C->top();
a61af66fc99e Initial load
duke
parents:
diff changeset
1710 }
a61af66fc99e Initial load
duke
parents:
diff changeset
1711 if( !n->is_Con() ) {
a61af66fc99e Initial load
duke
parents:
diff changeset
1712 if( t != Type::TOP ) {
a61af66fc99e Initial load
duke
parents:
diff changeset
1713 nn = makecon(t); // ConNode::make(t);
a61af66fc99e Initial load
duke
parents:
diff changeset
1714 NOT_PRODUCT( inc_constants(); )
a61af66fc99e Initial load
duke
parents:
diff changeset
1715 } else if( n->is_Region() ) { // Unreachable region
a61af66fc99e Initial load
duke
parents:
diff changeset
1716 // Note: nn == C->top()
a61af66fc99e Initial load
duke
parents:
diff changeset
1717 n->set_req(0, NULL); // Cut selfreference
a61af66fc99e Initial load
duke
parents:
diff changeset
1718 // Eagerly remove dead phis to avoid phis copies creation.
a61af66fc99e Initial load
duke
parents:
diff changeset
1719 for (DUIterator i = n->outs(); n->has_out(i); i++) {
a61af66fc99e Initial load
duke
parents:
diff changeset
1720 Node* m = n->out(i);
a61af66fc99e Initial load
duke
parents:
diff changeset
1721 if( m->is_Phi() ) {
a61af66fc99e Initial load
duke
parents:
diff changeset
1722 assert(type(m) == Type::TOP, "Unreachable region should not have live phis.");
1621
6027dddc26c6 6677629: PhaseIterGVN::subsume_node() should call hash_delete() and add_users_to_worklist()
kvn
parents: 1552
diff changeset
1723 replace_node(m, nn);
0
a61af66fc99e Initial load
duke
parents:
diff changeset
1724 --i; // deleted this phi; rescan starting with next position
a61af66fc99e Initial load
duke
parents:
diff changeset
1725 }
a61af66fc99e Initial load
duke
parents:
diff changeset
1726 }
a61af66fc99e Initial load
duke
parents:
diff changeset
1727 }
1621
6027dddc26c6 6677629: PhaseIterGVN::subsume_node() should call hash_delete() and add_users_to_worklist()
kvn
parents: 1552
diff changeset
1728 replace_node(n,nn); // Update DefUse edges for new constant
0
a61af66fc99e Initial load
duke
parents:
diff changeset
1729 }
a61af66fc99e Initial load
duke
parents:
diff changeset
1730 return nn;
a61af66fc99e Initial load
duke
parents:
diff changeset
1731 }
a61af66fc99e Initial load
duke
parents:
diff changeset
1732
a61af66fc99e Initial load
duke
parents:
diff changeset
1733 // If x is a TypeNode, capture any more-precise type permanently into Node
a61af66fc99e Initial load
duke
parents:
diff changeset
1734 if (t != n->bottom_type()) {
a61af66fc99e Initial load
duke
parents:
diff changeset
1735 hash_delete(n); // changing bottom type may force a rehash
a61af66fc99e Initial load
duke
parents:
diff changeset
1736 n->raise_bottom_type(t);
a61af66fc99e Initial load
duke
parents:
diff changeset
1737 _worklist.push(n); // n re-enters the hash table via the worklist
a61af66fc99e Initial load
duke
parents:
diff changeset
1738 }
a61af66fc99e Initial load
duke
parents:
diff changeset
1739
a61af66fc99e Initial load
duke
parents:
diff changeset
1740 // Idealize graph using DU info. Must clone() into new-space.
a61af66fc99e Initial load
duke
parents:
diff changeset
1741 // DU info is generally used to show profitability, progress or safety
a61af66fc99e Initial load
duke
parents:
diff changeset
1742 // (but generally not needed for correctness).
a61af66fc99e Initial load
duke
parents:
diff changeset
1743 Node *nn = n->Ideal_DU_postCCP(this);
a61af66fc99e Initial load
duke
parents:
diff changeset
1744
a61af66fc99e Initial load
duke
parents:
diff changeset
1745 // TEMPORARY fix to ensure that 2nd GVN pass eliminates NULL checks
a61af66fc99e Initial load
duke
parents:
diff changeset
1746 switch( n->Opcode() ) {
a61af66fc99e Initial load
duke
parents:
diff changeset
1747 case Op_FastLock: // Revisit FastLocks for lock coarsening
a61af66fc99e Initial load
duke
parents:
diff changeset
1748 case Op_If:
a61af66fc99e Initial load
duke
parents:
diff changeset
1749 case Op_CountedLoopEnd:
a61af66fc99e Initial load
duke
parents:
diff changeset
1750 case Op_Region:
a61af66fc99e Initial load
duke
parents:
diff changeset
1751 case Op_Loop:
a61af66fc99e Initial load
duke
parents:
diff changeset
1752 case Op_CountedLoop:
a61af66fc99e Initial load
duke
parents:
diff changeset
1753 case Op_Conv2B:
a61af66fc99e Initial load
duke
parents:
diff changeset
1754 case Op_Opaque1:
a61af66fc99e Initial load
duke
parents:
diff changeset
1755 case Op_Opaque2:
a61af66fc99e Initial load
duke
parents:
diff changeset
1756 _worklist.push(n);
a61af66fc99e Initial load
duke
parents:
diff changeset
1757 break;
a61af66fc99e Initial load
duke
parents:
diff changeset
1758 default:
a61af66fc99e Initial load
duke
parents:
diff changeset
1759 break;
a61af66fc99e Initial load
duke
parents:
diff changeset
1760 }
a61af66fc99e Initial load
duke
parents:
diff changeset
1761 if( nn ) {
a61af66fc99e Initial load
duke
parents:
diff changeset
1762 _worklist.push(n);
a61af66fc99e Initial load
duke
parents:
diff changeset
1763 // Put users of 'n' onto worklist for second igvn transform
a61af66fc99e Initial load
duke
parents:
diff changeset
1764 add_users_to_worklist(n);
a61af66fc99e Initial load
duke
parents:
diff changeset
1765 return nn;
a61af66fc99e Initial load
duke
parents:
diff changeset
1766 }
a61af66fc99e Initial load
duke
parents:
diff changeset
1767
a61af66fc99e Initial load
duke
parents:
diff changeset
1768 return n;
a61af66fc99e Initial load
duke
parents:
diff changeset
1769 }
a61af66fc99e Initial load
duke
parents:
diff changeset
1770
a61af66fc99e Initial load
duke
parents:
diff changeset
1771 //---------------------------------saturate------------------------------------
a61af66fc99e Initial load
duke
parents:
diff changeset
1772 const Type* PhaseCCP::saturate(const Type* new_type, const Type* old_type,
a61af66fc99e Initial load
duke
parents:
diff changeset
1773 const Type* limit_type) const {
1009
03b336640699 6885584: A particular class structure causes large allocation spike for jit
never
parents: 927
diff changeset
1774 const Type* wide_type = new_type->widen(old_type, limit_type);
0
a61af66fc99e Initial load
duke
parents:
diff changeset
1775 if (wide_type != new_type) { // did we widen?
a61af66fc99e Initial load
duke
parents:
diff changeset
1776 // If so, we may have widened beyond the limit type. Clip it back down.
a61af66fc99e Initial load
duke
parents:
diff changeset
1777 new_type = wide_type->filter(limit_type);
a61af66fc99e Initial load
duke
parents:
diff changeset
1778 }
a61af66fc99e Initial load
duke
parents:
diff changeset
1779 return new_type;
a61af66fc99e Initial load
duke
parents:
diff changeset
1780 }
a61af66fc99e Initial load
duke
parents:
diff changeset
1781
a61af66fc99e Initial load
duke
parents:
diff changeset
1782 //------------------------------print_statistics-------------------------------
a61af66fc99e Initial load
duke
parents:
diff changeset
1783 #ifndef PRODUCT
a61af66fc99e Initial load
duke
parents:
diff changeset
1784 void PhaseCCP::print_statistics() {
a61af66fc99e Initial load
duke
parents:
diff changeset
1785 tty->print_cr("CCP: %d constants found: %d", _total_invokes, _total_constants);
a61af66fc99e Initial load
duke
parents:
diff changeset
1786 }
a61af66fc99e Initial load
duke
parents:
diff changeset
1787 #endif
a61af66fc99e Initial load
duke
parents:
diff changeset
1788
a61af66fc99e Initial load
duke
parents:
diff changeset
1789
a61af66fc99e Initial load
duke
parents:
diff changeset
1790 //=============================================================================
a61af66fc99e Initial load
duke
parents:
diff changeset
1791 #ifndef PRODUCT
a61af66fc99e Initial load
duke
parents:
diff changeset
1792 uint PhasePeephole::_total_peepholes = 0;
a61af66fc99e Initial load
duke
parents:
diff changeset
1793 #endif
a61af66fc99e Initial load
duke
parents:
diff changeset
1794 //------------------------------PhasePeephole----------------------------------
a61af66fc99e Initial load
duke
parents:
diff changeset
1795 // Conditional Constant Propagation, ala Wegman & Zadeck
a61af66fc99e Initial load
duke
parents:
diff changeset
1796 PhasePeephole::PhasePeephole( PhaseRegAlloc *regalloc, PhaseCFG &cfg )
a61af66fc99e Initial load
duke
parents:
diff changeset
1797 : PhaseTransform(Peephole), _regalloc(regalloc), _cfg(cfg) {
a61af66fc99e Initial load
duke
parents:
diff changeset
1798 NOT_PRODUCT( clear_peepholes(); )
a61af66fc99e Initial load
duke
parents:
diff changeset
1799 }
a61af66fc99e Initial load
duke
parents:
diff changeset
1800
a61af66fc99e Initial load
duke
parents:
diff changeset
1801 #ifndef PRODUCT
a61af66fc99e Initial load
duke
parents:
diff changeset
1802 //------------------------------~PhasePeephole---------------------------------
a61af66fc99e Initial load
duke
parents:
diff changeset
1803 PhasePeephole::~PhasePeephole() {
a61af66fc99e Initial load
duke
parents:
diff changeset
1804 _total_peepholes += count_peepholes();
a61af66fc99e Initial load
duke
parents:
diff changeset
1805 }
a61af66fc99e Initial load
duke
parents:
diff changeset
1806 #endif
a61af66fc99e Initial load
duke
parents:
diff changeset
1807
a61af66fc99e Initial load
duke
parents:
diff changeset
1808 //------------------------------transform--------------------------------------
a61af66fc99e Initial load
duke
parents:
diff changeset
1809 Node *PhasePeephole::transform( Node *n ) {
a61af66fc99e Initial load
duke
parents:
diff changeset
1810 ShouldNotCallThis();
a61af66fc99e Initial load
duke
parents:
diff changeset
1811 return NULL;
a61af66fc99e Initial load
duke
parents:
diff changeset
1812 }
a61af66fc99e Initial load
duke
parents:
diff changeset
1813
a61af66fc99e Initial load
duke
parents:
diff changeset
1814 //------------------------------do_transform-----------------------------------
a61af66fc99e Initial load
duke
parents:
diff changeset
1815 void PhasePeephole::do_transform() {
a61af66fc99e Initial load
duke
parents:
diff changeset
1816 bool method_name_not_printed = true;
a61af66fc99e Initial load
duke
parents:
diff changeset
1817
a61af66fc99e Initial load
duke
parents:
diff changeset
1818 // Examine each basic block
12071
adb9a7d94cb5 8023003: Cleanup the public interface to PhaseCFG
adlertz
parents: 10278
diff changeset
1819 for (uint block_number = 1; block_number < _cfg.number_of_blocks(); ++block_number) {
adb9a7d94cb5 8023003: Cleanup the public interface to PhaseCFG
adlertz
parents: 10278
diff changeset
1820 Block* block = _cfg.get_block(block_number);
0
a61af66fc99e Initial load
duke
parents:
diff changeset
1821 bool block_not_printed = true;
a61af66fc99e Initial load
duke
parents:
diff changeset
1822
a61af66fc99e Initial load
duke
parents:
diff changeset
1823 // and each instruction within a block
12167
650868c062a9 8023691: Create interface for nodes in class Block
adlertz
parents: 12071
diff changeset
1824 uint end_index = block->number_of_nodes();
0
a61af66fc99e Initial load
duke
parents:
diff changeset
1825 // block->end_idx() not valid after PhaseRegAlloc
a61af66fc99e Initial load
duke
parents:
diff changeset
1826 for( uint instruction_index = 1; instruction_index < end_index; ++instruction_index ) {
12167
650868c062a9 8023691: Create interface for nodes in class Block
adlertz
parents: 12071
diff changeset
1827 Node *n = block->get_node(instruction_index);
0
a61af66fc99e Initial load
duke
parents:
diff changeset
1828 if( n->is_Mach() ) {
a61af66fc99e Initial load
duke
parents:
diff changeset
1829 MachNode *m = n->as_Mach();
a61af66fc99e Initial load
duke
parents:
diff changeset
1830 int deleted_count = 0;
a61af66fc99e Initial load
duke
parents:
diff changeset
1831 // check for peephole opportunities
a61af66fc99e Initial load
duke
parents:
diff changeset
1832 MachNode *m2 = m->peephole( block, instruction_index, _regalloc, deleted_count, C );
a61af66fc99e Initial load
duke
parents:
diff changeset
1833 if( m2 != NULL ) {
a61af66fc99e Initial load
duke
parents:
diff changeset
1834 #ifndef PRODUCT
a61af66fc99e Initial load
duke
parents:
diff changeset
1835 if( PrintOptoPeephole ) {
a61af66fc99e Initial load
duke
parents:
diff changeset
1836 // Print method, first time only
a61af66fc99e Initial load
duke
parents:
diff changeset
1837 if( C->method() && method_name_not_printed ) {
a61af66fc99e Initial load
duke
parents:
diff changeset
1838 C->method()->print_short_name(); tty->cr();
a61af66fc99e Initial load
duke
parents:
diff changeset
1839 method_name_not_printed = false;
a61af66fc99e Initial load
duke
parents:
diff changeset
1840 }
a61af66fc99e Initial load
duke
parents:
diff changeset
1841 // Print this block
a61af66fc99e Initial load
duke
parents:
diff changeset
1842 if( Verbose && block_not_printed) {
a61af66fc99e Initial load
duke
parents:
diff changeset
1843 tty->print_cr("in block");
a61af66fc99e Initial load
duke
parents:
diff changeset
1844 block->dump();
a61af66fc99e Initial load
duke
parents:
diff changeset
1845 block_not_printed = false;
a61af66fc99e Initial load
duke
parents:
diff changeset
1846 }
a61af66fc99e Initial load
duke
parents:
diff changeset
1847 // Print instructions being deleted
a61af66fc99e Initial load
duke
parents:
diff changeset
1848 for( int i = (deleted_count - 1); i >= 0; --i ) {
12167
650868c062a9 8023691: Create interface for nodes in class Block
adlertz
parents: 12071
diff changeset
1849 block->get_node(instruction_index-i)->as_Mach()->format(_regalloc); tty->cr();
0
a61af66fc99e Initial load
duke
parents:
diff changeset
1850 }
a61af66fc99e Initial load
duke
parents:
diff changeset
1851 tty->print_cr("replaced with");
a61af66fc99e Initial load
duke
parents:
diff changeset
1852 // Print new instruction
a61af66fc99e Initial load
duke
parents:
diff changeset
1853 m2->format(_regalloc);
a61af66fc99e Initial load
duke
parents:
diff changeset
1854 tty->print("\n\n");
a61af66fc99e Initial load
duke
parents:
diff changeset
1855 }
a61af66fc99e Initial load
duke
parents:
diff changeset
1856 #endif
a61af66fc99e Initial load
duke
parents:
diff changeset
1857 // Remove old nodes from basic block and update instruction_index
a61af66fc99e Initial load
duke
parents:
diff changeset
1858 // (old nodes still exist and may have edges pointing to them
a61af66fc99e Initial load
duke
parents:
diff changeset
1859 // as register allocation info is stored in the allocator using
a61af66fc99e Initial load
duke
parents:
diff changeset
1860 // the node index to live range mappings.)
a61af66fc99e Initial load
duke
parents:
diff changeset
1861 uint safe_instruction_index = (instruction_index - deleted_count);
a61af66fc99e Initial load
duke
parents:
diff changeset
1862 for( ; (instruction_index > safe_instruction_index); --instruction_index ) {
12167
650868c062a9 8023691: Create interface for nodes in class Block
adlertz
parents: 12071
diff changeset
1863 block->remove_node( instruction_index );
0
a61af66fc99e Initial load
duke
parents:
diff changeset
1864 }
a61af66fc99e Initial load
duke
parents:
diff changeset
1865 // install new node after safe_instruction_index
12167
650868c062a9 8023691: Create interface for nodes in class Block
adlertz
parents: 12071
diff changeset
1866 block->insert_node(m2, safe_instruction_index + 1);
650868c062a9 8023691: Create interface for nodes in class Block
adlertz
parents: 12071
diff changeset
1867 end_index = block->number_of_nodes() - 1; // Recompute new block size
0
a61af66fc99e Initial load
duke
parents:
diff changeset
1868 NOT_PRODUCT( inc_peepholes(); )
a61af66fc99e Initial load
duke
parents:
diff changeset
1869 }
a61af66fc99e Initial load
duke
parents:
diff changeset
1870 }
a61af66fc99e Initial load
duke
parents:
diff changeset
1871 }
a61af66fc99e Initial load
duke
parents:
diff changeset
1872 }
a61af66fc99e Initial load
duke
parents:
diff changeset
1873 }
a61af66fc99e Initial load
duke
parents:
diff changeset
1874
a61af66fc99e Initial load
duke
parents:
diff changeset
1875 //------------------------------print_statistics-------------------------------
a61af66fc99e Initial load
duke
parents:
diff changeset
1876 #ifndef PRODUCT
a61af66fc99e Initial load
duke
parents:
diff changeset
1877 void PhasePeephole::print_statistics() {
a61af66fc99e Initial load
duke
parents:
diff changeset
1878 tty->print_cr("Peephole: peephole rules applied: %d", _total_peepholes);
a61af66fc99e Initial load
duke
parents:
diff changeset
1879 }
a61af66fc99e Initial load
duke
parents:
diff changeset
1880 #endif
a61af66fc99e Initial load
duke
parents:
diff changeset
1881
a61af66fc99e Initial load
duke
parents:
diff changeset
1882
a61af66fc99e Initial load
duke
parents:
diff changeset
1883 //=============================================================================
a61af66fc99e Initial load
duke
parents:
diff changeset
1884 //------------------------------set_req_X--------------------------------------
a61af66fc99e Initial load
duke
parents:
diff changeset
1885 void Node::set_req_X( uint i, Node *n, PhaseIterGVN *igvn ) {
a61af66fc99e Initial load
duke
parents:
diff changeset
1886 assert( is_not_dead(n), "can not use dead node");
a61af66fc99e Initial load
duke
parents:
diff changeset
1887 assert( igvn->hash_find(this) != this, "Need to remove from hash before changing edges" );
a61af66fc99e Initial load
duke
parents:
diff changeset
1888 Node *old = in(i);
a61af66fc99e Initial load
duke
parents:
diff changeset
1889 set_req(i, n);
a61af66fc99e Initial load
duke
parents:
diff changeset
1890
a61af66fc99e Initial load
duke
parents:
diff changeset
1891 // old goes dead?
a61af66fc99e Initial load
duke
parents:
diff changeset
1892 if( old ) {
a61af66fc99e Initial load
duke
parents:
diff changeset
1893 switch (old->outcnt()) {
927
662f330d7275 6866651: Regression: simple int sum crashes jvm (build 1.6.0_14-b08 and 1.7.0-ea-b59)
cfang
parents: 605
diff changeset
1894 case 0:
662f330d7275 6866651: Regression: simple int sum crashes jvm (build 1.6.0_14-b08 and 1.7.0-ea-b59)
cfang
parents: 605
diff changeset
1895 // Put into the worklist to kill later. We do not kill it now because the
662f330d7275 6866651: Regression: simple int sum crashes jvm (build 1.6.0_14-b08 and 1.7.0-ea-b59)
cfang
parents: 605
diff changeset
1896 // recursive kill will delete the current node (this) if dead-loop exists
0
a61af66fc99e Initial load
duke
parents:
diff changeset
1897 if (!old->is_top())
927
662f330d7275 6866651: Regression: simple int sum crashes jvm (build 1.6.0_14-b08 and 1.7.0-ea-b59)
cfang
parents: 605
diff changeset
1898 igvn->_worklist.push( old );
0
a61af66fc99e Initial load
duke
parents:
diff changeset
1899 break;
a61af66fc99e Initial load
duke
parents:
diff changeset
1900 case 1:
a61af66fc99e Initial load
duke
parents:
diff changeset
1901 if( old->is_Store() || old->has_special_unique_user() )
a61af66fc99e Initial load
duke
parents:
diff changeset
1902 igvn->add_users_to_worklist( old );
a61af66fc99e Initial load
duke
parents:
diff changeset
1903 break;
a61af66fc99e Initial load
duke
parents:
diff changeset
1904 case 2:
a61af66fc99e Initial load
duke
parents:
diff changeset
1905 if( old->is_Store() )
a61af66fc99e Initial load
duke
parents:
diff changeset
1906 igvn->add_users_to_worklist( old );
a61af66fc99e Initial load
duke
parents:
diff changeset
1907 if( old->Opcode() == Op_Region )
a61af66fc99e Initial load
duke
parents:
diff changeset
1908 igvn->_worklist.push(old);
a61af66fc99e Initial load
duke
parents:
diff changeset
1909 break;
a61af66fc99e Initial load
duke
parents:
diff changeset
1910 case 3:
a61af66fc99e Initial load
duke
parents:
diff changeset
1911 if( old->Opcode() == Op_Region ) {
a61af66fc99e Initial load
duke
parents:
diff changeset
1912 igvn->_worklist.push(old);
a61af66fc99e Initial load
duke
parents:
diff changeset
1913 igvn->add_users_to_worklist( old );
a61af66fc99e Initial load
duke
parents:
diff changeset
1914 }
a61af66fc99e Initial load
duke
parents:
diff changeset
1915 break;
a61af66fc99e Initial load
duke
parents:
diff changeset
1916 default:
a61af66fc99e Initial load
duke
parents:
diff changeset
1917 break;
a61af66fc99e Initial load
duke
parents:
diff changeset
1918 }
a61af66fc99e Initial load
duke
parents:
diff changeset
1919 }
a61af66fc99e Initial load
duke
parents:
diff changeset
1920
a61af66fc99e Initial load
duke
parents:
diff changeset
1921 }
a61af66fc99e Initial load
duke
parents:
diff changeset
1922
a61af66fc99e Initial load
duke
parents:
diff changeset
1923 //-------------------------------replace_by-----------------------------------
a61af66fc99e Initial load
duke
parents:
diff changeset
1924 // Using def-use info, replace one node for another. Follow the def-use info
a61af66fc99e Initial load
duke
parents:
diff changeset
1925 // to all users of the OLD node. Then make all uses point to the NEW node.
a61af66fc99e Initial load
duke
parents:
diff changeset
1926 void Node::replace_by(Node *new_node) {
a61af66fc99e Initial load
duke
parents:
diff changeset
1927 assert(!is_top(), "top node has no DU info");
a61af66fc99e Initial load
duke
parents:
diff changeset
1928 for (DUIterator_Last imin, i = last_outs(imin); i >= imin; ) {
a61af66fc99e Initial load
duke
parents:
diff changeset
1929 Node* use = last_out(i);
a61af66fc99e Initial load
duke
parents:
diff changeset
1930 uint uses_found = 0;
a61af66fc99e Initial load
duke
parents:
diff changeset
1931 for (uint j = 0; j < use->len(); j++) {
a61af66fc99e Initial load
duke
parents:
diff changeset
1932 if (use->in(j) == this) {
a61af66fc99e Initial load
duke
parents:
diff changeset
1933 if (j < use->req())
a61af66fc99e Initial load
duke
parents:
diff changeset
1934 use->set_req(j, new_node);
a61af66fc99e Initial load
duke
parents:
diff changeset
1935 else use->set_prec(j, new_node);
a61af66fc99e Initial load
duke
parents:
diff changeset
1936 uses_found++;
a61af66fc99e Initial load
duke
parents:
diff changeset
1937 }
a61af66fc99e Initial load
duke
parents:
diff changeset
1938 }
a61af66fc99e Initial load
duke
parents:
diff changeset
1939 i -= uses_found; // we deleted 1 or more copies of this edge
a61af66fc99e Initial load
duke
parents:
diff changeset
1940 }
a61af66fc99e Initial load
duke
parents:
diff changeset
1941 }
a61af66fc99e Initial load
duke
parents:
diff changeset
1942
a61af66fc99e Initial load
duke
parents:
diff changeset
1943 //=============================================================================
a61af66fc99e Initial load
duke
parents:
diff changeset
1944 //-----------------------------------------------------------------------------
a61af66fc99e Initial load
duke
parents:
diff changeset
1945 void Type_Array::grow( uint i ) {
a61af66fc99e Initial load
duke
parents:
diff changeset
1946 if( !_max ) {
a61af66fc99e Initial load
duke
parents:
diff changeset
1947 _max = 1;
a61af66fc99e Initial load
duke
parents:
diff changeset
1948 _types = (const Type**)_a->Amalloc( _max * sizeof(Type*) );
a61af66fc99e Initial load
duke
parents:
diff changeset
1949 _types[0] = NULL;
a61af66fc99e Initial load
duke
parents:
diff changeset
1950 }
a61af66fc99e Initial load
duke
parents:
diff changeset
1951 uint old = _max;
a61af66fc99e Initial load
duke
parents:
diff changeset
1952 while( i >= _max ) _max <<= 1; // Double to fit
a61af66fc99e Initial load
duke
parents:
diff changeset
1953 _types = (const Type**)_a->Arealloc( _types, old*sizeof(Type*),_max*sizeof(Type*));
a61af66fc99e Initial load
duke
parents:
diff changeset
1954 memset( &_types[old], 0, (_max-old)*sizeof(Type*) );
a61af66fc99e Initial load
duke
parents:
diff changeset
1955 }
a61af66fc99e Initial load
duke
parents:
diff changeset
1956
a61af66fc99e Initial load
duke
parents:
diff changeset
1957 //------------------------------dump-------------------------------------------
a61af66fc99e Initial load
duke
parents:
diff changeset
1958 #ifndef PRODUCT
a61af66fc99e Initial load
duke
parents:
diff changeset
1959 void Type_Array::dump() const {
a61af66fc99e Initial load
duke
parents:
diff changeset
1960 uint max = Size();
a61af66fc99e Initial load
duke
parents:
diff changeset
1961 for( uint i = 0; i < max; i++ ) {
a61af66fc99e Initial load
duke
parents:
diff changeset
1962 if( _types[i] != NULL ) {
a61af66fc99e Initial load
duke
parents:
diff changeset
1963 tty->print(" %d\t== ", i); _types[i]->dump(); tty->cr();
a61af66fc99e Initial load
duke
parents:
diff changeset
1964 }
a61af66fc99e Initial load
duke
parents:
diff changeset
1965 }
a61af66fc99e Initial load
duke
parents:
diff changeset
1966 }
a61af66fc99e Initial load
duke
parents:
diff changeset
1967 #endif