author | redestad |
Thu, 14 Nov 2019 15:24:35 +0100 | |
changeset 59081 | 95a99e617f28 |
parent 58351 | d322bf161e31 |
child 58679 | 9c3209ff7550 |
permissions | -rw-r--r-- |
9101 | 1 |
/* |
51078 | 2 |
* Copyright (c) 2011, 2018, Oracle and/or its affiliates. All rights reserved. |
9101 | 3 |
* DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS FILE HEADER. |
4 |
* |
|
5 |
* This code is free software; you can redistribute it and/or modify it |
|
6 |
* under the terms of the GNU General Public License version 2 only, as |
|
7 |
* published by the Free Software Foundation. |
|
8 |
* |
|
9 |
* This code is distributed in the hope that it will be useful, but WITHOUT |
|
10 |
* ANY WARRANTY; without even the implied warranty of MERCHANTABILITY or |
|
11 |
* FITNESS FOR A PARTICULAR PURPOSE. See the GNU General Public License |
|
12 |
* version 2 for more details (a copy is included in the LICENSE file that |
|
13 |
* accompanied this code). |
|
14 |
* |
|
15 |
* You should have received a copy of the GNU General Public License version |
|
16 |
* 2 along with this work; if not, write to the Free Software Foundation, |
|
17 |
* Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301 USA. |
|
18 |
* |
|
19 |
* Please contact Oracle, 500 Oracle Parkway, Redwood Shores, CA 94065 USA |
|
20 |
* or visit www.oracle.com if you need additional information or have any |
|
21 |
* questions. |
|
22 |
* |
|
23 |
*/ |
|
24 |
||
25 |
#include "precompiled.hpp" |
|
26 |
#include "opto/loopnode.hpp" |
|
27 |
#include "opto/addnode.hpp" |
|
28 |
#include "opto/callnode.hpp" |
|
29 |
#include "opto/connode.hpp" |
|
23528 | 30 |
#include "opto/convertnode.hpp" |
9101 | 31 |
#include "opto/loopnode.hpp" |
45965 | 32 |
#include "opto/matcher.hpp" |
9101 | 33 |
#include "opto/mulnode.hpp" |
23528 | 34 |
#include "opto/opaquenode.hpp" |
9101 | 35 |
#include "opto/rootnode.hpp" |
36 |
#include "opto/subnode.hpp" |
|
50623
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
37 |
#include <fenv.h> |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
38 |
#include <math.h> |
9101 | 39 |
|
40 |
/* |
|
41 |
* The general idea of Loop Predication is to insert a predicate on the entry |
|
42 |
* path to a loop, and raise a uncommon trap if the check of the condition fails. |
|
43 |
* The condition checks are promoted from inside the loop body, and thus |
|
44 |
* the checks inside the loop could be eliminated. Currently, loop predication |
|
45 |
* optimization has been applied to remove array range check and loop invariant |
|
46 |
* checks (such as null checks). |
|
47 |
*/ |
|
48 |
||
49 |
//-------------------------------register_control------------------------- |
|
50 |
void PhaseIdealLoop::register_control(Node* n, IdealLoopTree *loop, Node* pred) { |
|
51 |
assert(n->is_CFG(), "must be control node"); |
|
52 |
_igvn.register_new_node_with_optimizer(n); |
|
53 |
loop->_body.push(n); |
|
54 |
set_loop(n, loop); |
|
55 |
// When called from beautify_loops() idom is not constructed yet. |
|
56 |
if (_idom != NULL) { |
|
57 |
set_idom(n, pred, dom_depth(pred)); |
|
58 |
} |
|
59 |
} |
|
60 |
||
61 |
//------------------------------create_new_if_for_predicate------------------------ |
|
62 |
// create a new if above the uct_if_pattern for the predicate to be promoted. |
|
63 |
// |
|
64 |
// before after |
|
65 |
// ---------- ---------- |
|
66 |
// ctrl ctrl |
|
67 |
// | | |
|
68 |
// | | |
|
69 |
// v v |
|
70 |
// iff new_iff |
|
71 |
// / \ / \ |
|
72 |
// / \ / \ |
|
73 |
// v v v v |
|
74 |
// uncommon_proj cont_proj if_uct if_cont |
|
75 |
// \ | | | | |
|
76 |
// \ | | | | |
|
77 |
// v v v | v |
|
78 |
// rgn loop | iff |
|
79 |
// | | / \ |
|
80 |
// | | / \ |
|
81 |
// v | v v |
|
82 |
// uncommon_trap | uncommon_proj cont_proj |
|
83 |
// \ \ | | |
|
84 |
// \ \ | | |
|
85 |
// v v v v |
|
86 |
// rgn loop |
|
87 |
// | |
|
88 |
// | |
|
89 |
// v |
|
90 |
// uncommon_trap |
|
91 |
// |
|
92 |
// |
|
93 |
// We will create a region to guard the uct call if there is no one there. |
|
50632
fd430e352427
8205033: [REDO] Induction variable of over-unrolled loop conflicts with range checks
thartmann
parents:
50623
diff
changeset
|
94 |
// The true projection (if_cont) of the new_iff is returned. |
30309
da3efc8ed2cb
8078426: mb/jvm/compiler/InterfaceCalls/testAC2 - assert(predicate_proj == 0L) failed: only one predicate entry expected
roland
parents:
26166
diff
changeset
|
95 |
// This code is also used to clone predicates to cloned loops. |
9101 | 96 |
ProjNode* PhaseIdealLoop::create_new_if_for_predicate(ProjNode* cont_proj, Node* new_entry, |
34164
a9e6034d7707
8137168: Replace IfNode with a new RangeCheckNode for range checks
roland
parents:
32733
diff
changeset
|
97 |
Deoptimization::DeoptReason reason, |
a9e6034d7707
8137168: Replace IfNode with a new RangeCheckNode for range checks
roland
parents:
32733
diff
changeset
|
98 |
int opcode) { |
21089
e1986ff6fe2e
8024069: replace_in_map() should operate on parent maps
roland
parents:
17383
diff
changeset
|
99 |
assert(cont_proj->is_uncommon_trap_if_pattern(reason), "must be a uct if pattern!"); |
9101 | 100 |
IfNode* iff = cont_proj->in(0)->as_If(); |
101 |
||
102 |
ProjNode *uncommon_proj = iff->proj_out(1 - cont_proj->_con); |
|
103 |
Node *rgn = uncommon_proj->unique_ctrl_out(); |
|
104 |
assert(rgn->is_Region() || rgn->is_Call(), "must be a region or call uct"); |
|
105 |
||
106 |
uint proj_index = 1; // region's edge corresponding to uncommon_proj |
|
107 |
if (!rgn->is_Region()) { // create a region to guard the call |
|
108 |
assert(rgn->is_Call(), "must be call uct"); |
|
109 |
CallNode* call = rgn->as_Call(); |
|
110 |
IdealLoopTree* loop = get_loop(call); |
|
24923
9631f7d691dc
8034812: remove IDX_INIT macro hack in Node class
thartmann
parents:
24424
diff
changeset
|
111 |
rgn = new RegionNode(1); |
9101 | 112 |
rgn->add_req(uncommon_proj); |
113 |
register_control(rgn, loop, uncommon_proj); |
|
25913
81dbc151e91c
8040213: C2 does not put all modified nodes on IGVN worklist
thartmann
parents:
24923
diff
changeset
|
114 |
_igvn.replace_input_of(call, 0, rgn); |
9101 | 115 |
// When called from beautify_loops() idom is not constructed yet. |
116 |
if (_idom != NULL) { |
|
117 |
set_idom(call, rgn, dom_depth(rgn)); |
|
118 |
} |
|
32733 | 119 |
for (DUIterator_Fast imax, i = uncommon_proj->fast_outs(imax); i < imax; i++) { |
120 |
Node* n = uncommon_proj->fast_out(i); |
|
121 |
if (n->is_Load() || n->is_Store()) { |
|
122 |
_igvn.replace_input_of(n, 0, rgn); |
|
123 |
--i; --imax; |
|
124 |
} |
|
125 |
} |
|
9101 | 126 |
} else { |
127 |
// Find region's edge corresponding to uncommon_proj |
|
128 |
for (; proj_index < rgn->req(); proj_index++) |
|
129 |
if (rgn->in(proj_index) == uncommon_proj) break; |
|
130 |
assert(proj_index < rgn->req(), "sanity"); |
|
131 |
} |
|
132 |
||
133 |
Node* entry = iff->in(0); |
|
134 |
if (new_entry != NULL) { |
|
135 |
// Clonning the predicate to new location. |
|
136 |
entry = new_entry; |
|
137 |
} |
|
138 |
// Create new_iff |
|
139 |
IdealLoopTree* lp = get_loop(entry); |
|
34164
a9e6034d7707
8137168: Replace IfNode with a new RangeCheckNode for range checks
roland
parents:
32733
diff
changeset
|
140 |
IfNode* new_iff = NULL; |
a9e6034d7707
8137168: Replace IfNode with a new RangeCheckNode for range checks
roland
parents:
32733
diff
changeset
|
141 |
if (opcode == Op_If) { |
a9e6034d7707
8137168: Replace IfNode with a new RangeCheckNode for range checks
roland
parents:
32733
diff
changeset
|
142 |
new_iff = new IfNode(entry, iff->in(1), iff->_prob, iff->_fcnt); |
a9e6034d7707
8137168: Replace IfNode with a new RangeCheckNode for range checks
roland
parents:
32733
diff
changeset
|
143 |
} else { |
a9e6034d7707
8137168: Replace IfNode with a new RangeCheckNode for range checks
roland
parents:
32733
diff
changeset
|
144 |
assert(opcode == Op_RangeCheck, "no other if variant here"); |
a9e6034d7707
8137168: Replace IfNode with a new RangeCheckNode for range checks
roland
parents:
32733
diff
changeset
|
145 |
new_iff = new RangeCheckNode(entry, iff->in(1), iff->_prob, iff->_fcnt); |
a9e6034d7707
8137168: Replace IfNode with a new RangeCheckNode for range checks
roland
parents:
32733
diff
changeset
|
146 |
} |
9101 | 147 |
register_control(new_iff, lp, entry); |
24923
9631f7d691dc
8034812: remove IDX_INIT macro hack in Node class
thartmann
parents:
24424
diff
changeset
|
148 |
Node *if_cont = new IfTrueNode(new_iff); |
9631f7d691dc
8034812: remove IDX_INIT macro hack in Node class
thartmann
parents:
24424
diff
changeset
|
149 |
Node *if_uct = new IfFalseNode(new_iff); |
9101 | 150 |
if (cont_proj->is_IfFalse()) { |
151 |
// Swap |
|
152 |
Node* tmp = if_uct; if_uct = if_cont; if_cont = tmp; |
|
153 |
} |
|
154 |
register_control(if_cont, lp, new_iff); |
|
155 |
register_control(if_uct, get_loop(rgn), new_iff); |
|
156 |
||
157 |
// if_uct to rgn |
|
158 |
_igvn.hash_delete(rgn); |
|
159 |
rgn->add_req(if_uct); |
|
160 |
// When called from beautify_loops() idom is not constructed yet. |
|
161 |
if (_idom != NULL) { |
|
162 |
Node* ridom = idom(rgn); |
|
53357
c52a37f40324
8215757: C2: PhaseIdealLoop::create_new_if_for_predicate() computes wrong IDOM
vlivanov
parents:
53306
diff
changeset
|
163 |
Node* nrdom = dom_lca_internal(ridom, new_iff); |
9101 | 164 |
set_idom(rgn, nrdom, dom_depth(rgn)); |
165 |
} |
|
166 |
||
167 |
// If rgn has phis add new edges which has the same |
|
168 |
// value as on original uncommon_proj pass. |
|
169 |
assert(rgn->in(rgn->req() -1) == if_uct, "new edge should be last"); |
|
170 |
bool has_phi = false; |
|
171 |
for (DUIterator_Fast imax, i = rgn->fast_outs(imax); i < imax; i++) { |
|
172 |
Node* use = rgn->fast_out(i); |
|
173 |
if (use->is_Phi() && use->outcnt() > 0) { |
|
174 |
assert(use->in(0) == rgn, ""); |
|
12958
009b6c9586d8
7173340: C2: code cleanup: use PhaseIterGVN::replace_edge(Node*, int, Node*) where applicable
kvn
parents:
10263
diff
changeset
|
175 |
_igvn.rehash_node_delayed(use); |
9101 | 176 |
use->add_req(use->in(proj_index)); |
177 |
has_phi = true; |
|
178 |
} |
|
179 |
} |
|
180 |
assert(!has_phi || rgn->req() > 3, "no phis when region is created"); |
|
181 |
||
182 |
if (new_entry == NULL) { |
|
183 |
// Attach if_cont to iff |
|
25913
81dbc151e91c
8040213: C2 does not put all modified nodes on IGVN worklist
thartmann
parents:
24923
diff
changeset
|
184 |
_igvn.replace_input_of(iff, 0, if_cont); |
9101 | 185 |
if (_idom != NULL) { |
186 |
set_idom(iff, if_cont, dom_depth(iff)); |
|
187 |
} |
|
188 |
} |
|
189 |
return if_cont->as_Proj(); |
|
190 |
} |
|
191 |
||
192 |
//------------------------------create_new_if_for_predicate------------------------ |
|
193 |
// Create a new if below new_entry for the predicate to be cloned (IGVN optimization) |
|
194 |
ProjNode* PhaseIterGVN::create_new_if_for_predicate(ProjNode* cont_proj, Node* new_entry, |
|
34164
a9e6034d7707
8137168: Replace IfNode with a new RangeCheckNode for range checks
roland
parents:
32733
diff
changeset
|
195 |
Deoptimization::DeoptReason reason, |
a9e6034d7707
8137168: Replace IfNode with a new RangeCheckNode for range checks
roland
parents:
32733
diff
changeset
|
196 |
int opcode) { |
9101 | 197 |
assert(new_entry != 0, "only used for clone predicate"); |
21089
e1986ff6fe2e
8024069: replace_in_map() should operate on parent maps
roland
parents:
17383
diff
changeset
|
198 |
assert(cont_proj->is_uncommon_trap_if_pattern(reason), "must be a uct if pattern!"); |
9101 | 199 |
IfNode* iff = cont_proj->in(0)->as_If(); |
200 |
||
201 |
ProjNode *uncommon_proj = iff->proj_out(1 - cont_proj->_con); |
|
202 |
Node *rgn = uncommon_proj->unique_ctrl_out(); |
|
203 |
assert(rgn->is_Region() || rgn->is_Call(), "must be a region or call uct"); |
|
204 |
||
205 |
uint proj_index = 1; // region's edge corresponding to uncommon_proj |
|
206 |
if (!rgn->is_Region()) { // create a region to guard the call |
|
207 |
assert(rgn->is_Call(), "must be call uct"); |
|
208 |
CallNode* call = rgn->as_Call(); |
|
24923
9631f7d691dc
8034812: remove IDX_INIT macro hack in Node class
thartmann
parents:
24424
diff
changeset
|
209 |
rgn = new RegionNode(1); |
9101 | 210 |
register_new_node_with_optimizer(rgn); |
211 |
rgn->add_req(uncommon_proj); |
|
25913
81dbc151e91c
8040213: C2 does not put all modified nodes on IGVN worklist
thartmann
parents:
24923
diff
changeset
|
212 |
replace_input_of(call, 0, rgn); |
9101 | 213 |
} else { |
214 |
// Find region's edge corresponding to uncommon_proj |
|
215 |
for (; proj_index < rgn->req(); proj_index++) |
|
216 |
if (rgn->in(proj_index) == uncommon_proj) break; |
|
217 |
assert(proj_index < rgn->req(), "sanity"); |
|
218 |
} |
|
219 |
||
220 |
// Create new_iff in new location. |
|
34164
a9e6034d7707
8137168: Replace IfNode with a new RangeCheckNode for range checks
roland
parents:
32733
diff
changeset
|
221 |
IfNode* new_iff = NULL; |
a9e6034d7707
8137168: Replace IfNode with a new RangeCheckNode for range checks
roland
parents:
32733
diff
changeset
|
222 |
if (opcode == Op_If) { |
a9e6034d7707
8137168: Replace IfNode with a new RangeCheckNode for range checks
roland
parents:
32733
diff
changeset
|
223 |
new_iff = new IfNode(new_entry, iff->in(1), iff->_prob, iff->_fcnt); |
a9e6034d7707
8137168: Replace IfNode with a new RangeCheckNode for range checks
roland
parents:
32733
diff
changeset
|
224 |
} else { |
a9e6034d7707
8137168: Replace IfNode with a new RangeCheckNode for range checks
roland
parents:
32733
diff
changeset
|
225 |
assert(opcode == Op_RangeCheck, "no other if variant here"); |
a9e6034d7707
8137168: Replace IfNode with a new RangeCheckNode for range checks
roland
parents:
32733
diff
changeset
|
226 |
new_iff = new RangeCheckNode(new_entry, iff->in(1), iff->_prob, iff->_fcnt); |
a9e6034d7707
8137168: Replace IfNode with a new RangeCheckNode for range checks
roland
parents:
32733
diff
changeset
|
227 |
} |
9101 | 228 |
|
229 |
register_new_node_with_optimizer(new_iff); |
|
24923
9631f7d691dc
8034812: remove IDX_INIT macro hack in Node class
thartmann
parents:
24424
diff
changeset
|
230 |
Node *if_cont = new IfTrueNode(new_iff); |
9631f7d691dc
8034812: remove IDX_INIT macro hack in Node class
thartmann
parents:
24424
diff
changeset
|
231 |
Node *if_uct = new IfFalseNode(new_iff); |
9101 | 232 |
if (cont_proj->is_IfFalse()) { |
233 |
// Swap |
|
234 |
Node* tmp = if_uct; if_uct = if_cont; if_cont = tmp; |
|
235 |
} |
|
236 |
register_new_node_with_optimizer(if_cont); |
|
237 |
register_new_node_with_optimizer(if_uct); |
|
238 |
||
239 |
// if_uct to rgn |
|
240 |
hash_delete(rgn); |
|
241 |
rgn->add_req(if_uct); |
|
242 |
||
243 |
// If rgn has phis add corresponding new edges which has the same |
|
244 |
// value as on original uncommon_proj pass. |
|
245 |
assert(rgn->in(rgn->req() -1) == if_uct, "new edge should be last"); |
|
246 |
bool has_phi = false; |
|
247 |
for (DUIterator_Fast imax, i = rgn->fast_outs(imax); i < imax; i++) { |
|
248 |
Node* use = rgn->fast_out(i); |
|
249 |
if (use->is_Phi() && use->outcnt() > 0) { |
|
12958
009b6c9586d8
7173340: C2: code cleanup: use PhaseIterGVN::replace_edge(Node*, int, Node*) where applicable
kvn
parents:
10263
diff
changeset
|
250 |
rehash_node_delayed(use); |
9101 | 251 |
use->add_req(use->in(proj_index)); |
252 |
has_phi = true; |
|
253 |
} |
|
254 |
} |
|
255 |
assert(!has_phi || rgn->req() > 3, "no phis when region is created"); |
|
256 |
||
257 |
return if_cont->as_Proj(); |
|
258 |
} |
|
259 |
||
260 |
//--------------------------clone_predicate----------------------- |
|
261 |
ProjNode* PhaseIdealLoop::clone_predicate(ProjNode* predicate_proj, Node* new_entry, |
|
262 |
Deoptimization::DeoptReason reason, |
|
263 |
PhaseIdealLoop* loop_phase, |
|
264 |
PhaseIterGVN* igvn) { |
|
265 |
ProjNode* new_predicate_proj; |
|
266 |
if (loop_phase != NULL) { |
|
34164
a9e6034d7707
8137168: Replace IfNode with a new RangeCheckNode for range checks
roland
parents:
32733
diff
changeset
|
267 |
new_predicate_proj = loop_phase->create_new_if_for_predicate(predicate_proj, new_entry, reason, Op_If); |
9101 | 268 |
} else { |
34164
a9e6034d7707
8137168: Replace IfNode with a new RangeCheckNode for range checks
roland
parents:
32733
diff
changeset
|
269 |
new_predicate_proj = igvn->create_new_if_for_predicate(predicate_proj, new_entry, reason, Op_If); |
9101 | 270 |
} |
271 |
IfNode* iff = new_predicate_proj->in(0)->as_If(); |
|
272 |
Node* ctrl = iff->in(0); |
|
273 |
||
274 |
// Match original condition since predicate's projections could be swapped. |
|
275 |
assert(predicate_proj->in(0)->in(1)->in(1)->Opcode()==Op_Opaque1, "must be"); |
|
24923
9631f7d691dc
8034812: remove IDX_INIT macro hack in Node class
thartmann
parents:
24424
diff
changeset
|
276 |
Node* opq = new Opaque1Node(igvn->C, predicate_proj->in(0)->in(1)->in(1)->in(1)); |
9101 | 277 |
igvn->C->add_predicate_opaq(opq); |
278 |
||
24923
9631f7d691dc
8034812: remove IDX_INIT macro hack in Node class
thartmann
parents:
24424
diff
changeset
|
279 |
Node* bol = new Conv2BNode(opq); |
9101 | 280 |
if (loop_phase != NULL) { |
281 |
loop_phase->register_new_node(opq, ctrl); |
|
282 |
loop_phase->register_new_node(bol, ctrl); |
|
283 |
} else { |
|
284 |
igvn->register_new_node_with_optimizer(opq); |
|
285 |
igvn->register_new_node_with_optimizer(bol); |
|
286 |
} |
|
287 |
igvn->hash_delete(iff); |
|
288 |
iff->set_req(1, bol); |
|
289 |
return new_predicate_proj; |
|
290 |
} |
|
291 |
||
292 |
||
293 |
//--------------------------clone_loop_predicates----------------------- |
|
294 |
// Interface from IGVN |
|
9446 | 295 |
Node* PhaseIterGVN::clone_loop_predicates(Node* old_entry, Node* new_entry, bool clone_limit_check) { |
10258
10c77b8c8d3e
7068051: SIGSEGV in PhaseIdealLoop::build_loop_late_post
kvn
parents:
10253
diff
changeset
|
296 |
return PhaseIdealLoop::clone_loop_predicates(old_entry, new_entry, clone_limit_check, NULL, this); |
9101 | 297 |
} |
298 |
||
299 |
// Interface from PhaseIdealLoop |
|
9446 | 300 |
Node* PhaseIdealLoop::clone_loop_predicates(Node* old_entry, Node* new_entry, bool clone_limit_check) { |
10258
10c77b8c8d3e
7068051: SIGSEGV in PhaseIdealLoop::build_loop_late_post
kvn
parents:
10253
diff
changeset
|
301 |
return clone_loop_predicates(old_entry, new_entry, clone_limit_check, this, &this->_igvn); |
9101 | 302 |
} |
303 |
||
304 |
// Clone loop predicates to cloned loops (peeled, unswitched, split_if). |
|
305 |
Node* PhaseIdealLoop::clone_loop_predicates(Node* old_entry, Node* new_entry, |
|
52608
61241fc2217a
8211451: ~2.5% regression on compression benchmark starting with 12-b11
roland
parents:
52490
diff
changeset
|
306 |
bool clone_limit_check, |
61241fc2217a
8211451: ~2.5% regression on compression benchmark starting with 12-b11
roland
parents:
52490
diff
changeset
|
307 |
PhaseIdealLoop* loop_phase, |
61241fc2217a
8211451: ~2.5% regression on compression benchmark starting with 12-b11
roland
parents:
52490
diff
changeset
|
308 |
PhaseIterGVN* igvn) { |
9101 | 309 |
#ifdef ASSERT |
310 |
if (new_entry == NULL || !(new_entry->is_Proj() || new_entry->is_Region() || new_entry->is_SafePoint())) { |
|
311 |
if (new_entry != NULL) |
|
312 |
new_entry->dump(); |
|
313 |
assert(false, "not IfTrue, IfFalse, Region or SafePoint"); |
|
314 |
} |
|
315 |
#endif |
|
316 |
// Search original predicates |
|
317 |
Node* entry = old_entry; |
|
9446 | 318 |
ProjNode* limit_check_proj = NULL; |
37287
c2660335bf81
8072422: Cleanup: Remove some unused flags/code in loop optimizations
zmajo
parents:
35558
diff
changeset
|
319 |
limit_check_proj = find_predicate_insertion_point(entry, Deoptimization::Reason_loop_limit_check); |
c2660335bf81
8072422: Cleanup: Remove some unused flags/code in loop optimizations
zmajo
parents:
35558
diff
changeset
|
320 |
if (limit_check_proj != NULL) { |
52608
61241fc2217a
8211451: ~2.5% regression on compression benchmark starting with 12-b11
roland
parents:
52490
diff
changeset
|
321 |
entry = skip_loop_predicates(entry); |
9446 | 322 |
} |
50623
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
323 |
ProjNode* profile_predicate_proj = NULL; |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
324 |
ProjNode* predicate_proj = NULL; |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
325 |
if (UseProfiledLoopPredicate) { |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
326 |
profile_predicate_proj = find_predicate_insertion_point(entry, Deoptimization::Reason_profile_predicate); |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
327 |
if (profile_predicate_proj != NULL) { |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
328 |
entry = skip_loop_predicates(entry); |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
329 |
} |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
330 |
} |
9101 | 331 |
if (UseLoopPredicate) { |
50623
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
332 |
predicate_proj = find_predicate_insertion_point(entry, Deoptimization::Reason_predicate); |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
333 |
} |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
334 |
if (predicate_proj != NULL) { // right pattern that can be used by loop predication |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
335 |
// clone predicate |
58106
79186d82463e
8230470: Shenandoah doesn't need change from JDK-8212610 anymore
roland
parents:
54423
diff
changeset
|
336 |
new_entry = clone_predicate(predicate_proj, new_entry, |
79186d82463e
8230470: Shenandoah doesn't need change from JDK-8212610 anymore
roland
parents:
54423
diff
changeset
|
337 |
Deoptimization::Reason_predicate, |
79186d82463e
8230470: Shenandoah doesn't need change from JDK-8212610 anymore
roland
parents:
54423
diff
changeset
|
338 |
loop_phase, igvn); |
79186d82463e
8230470: Shenandoah doesn't need change from JDK-8212610 anymore
roland
parents:
54423
diff
changeset
|
339 |
assert(new_entry != NULL && new_entry->is_Proj(), "IfTrue or IfFalse after clone predicate"); |
50623
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
340 |
if (TraceLoopPredicate) { |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
341 |
tty->print("Loop Predicate cloned: "); |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
342 |
debug_only( new_entry->in(0)->dump(); ); |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
343 |
} |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
344 |
} |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
345 |
if (profile_predicate_proj != NULL) { // right pattern that can be used by loop predication |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
346 |
// clone predicate |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
347 |
new_entry = clone_predicate(profile_predicate_proj, new_entry, |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
348 |
Deoptimization::Reason_profile_predicate, |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
349 |
loop_phase, igvn); |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
350 |
assert(new_entry != NULL && new_entry->is_Proj(), "IfTrue or IfFalse after clone predicate"); |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
351 |
if (TraceLoopPredicate) { |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
352 |
tty->print("Loop Predicate cloned: "); |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
353 |
debug_only( new_entry->in(0)->dump(); ); |
9101 | 354 |
} |
355 |
} |
|
9446 | 356 |
if (limit_check_proj != NULL && clone_limit_check) { |
357 |
// Clone loop limit check last to insert it before loop. |
|
358 |
// Don't clone a limit check which was already finalized |
|
359 |
// for this counted loop (only one limit check is needed). |
|
10258
10c77b8c8d3e
7068051: SIGSEGV in PhaseIdealLoop::build_loop_late_post
kvn
parents:
10253
diff
changeset
|
360 |
new_entry = clone_predicate(limit_check_proj, new_entry, |
10c77b8c8d3e
7068051: SIGSEGV in PhaseIdealLoop::build_loop_late_post
kvn
parents:
10253
diff
changeset
|
361 |
Deoptimization::Reason_loop_limit_check, |
10c77b8c8d3e
7068051: SIGSEGV in PhaseIdealLoop::build_loop_late_post
kvn
parents:
10253
diff
changeset
|
362 |
loop_phase, igvn); |
10c77b8c8d3e
7068051: SIGSEGV in PhaseIdealLoop::build_loop_late_post
kvn
parents:
10253
diff
changeset
|
363 |
assert(new_entry != NULL && new_entry->is_Proj(), "IfTrue or IfFalse after clone limit check"); |
9446 | 364 |
if (TraceLoopLimitCheck) { |
10258
10c77b8c8d3e
7068051: SIGSEGV in PhaseIdealLoop::build_loop_late_post
kvn
parents:
10253
diff
changeset
|
365 |
tty->print("Loop Limit Check cloned: "); |
9446 | 366 |
debug_only( new_entry->in(0)->dump(); ) |
367 |
} |
|
368 |
} |
|
9101 | 369 |
return new_entry; |
370 |
} |
|
371 |
||
372 |
//--------------------------skip_loop_predicates------------------------------ |
|
373 |
// Skip related predicates. |
|
374 |
Node* PhaseIdealLoop::skip_loop_predicates(Node* entry) { |
|
50623
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
375 |
IfNode* iff = entry->in(0)->as_If(); |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
376 |
ProjNode* uncommon_proj = iff->proj_out(1 - entry->as_Proj()->_con); |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
377 |
Node* rgn = uncommon_proj->unique_ctrl_out(); |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
378 |
assert(rgn->is_Region() || rgn->is_Call(), "must be a region or call uct"); |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
379 |
entry = entry->in(0)->in(0); |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
380 |
while (entry != NULL && entry->is_Proj() && entry->in(0)->is_If()) { |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
381 |
uncommon_proj = entry->in(0)->as_If()->proj_out(1 - entry->as_Proj()->_con); |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
382 |
if (uncommon_proj->unique_ctrl_out() != rgn) |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
383 |
break; |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
384 |
entry = entry->in(0)->in(0); |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
385 |
} |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
386 |
return entry; |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
387 |
} |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
388 |
|
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
389 |
Node* PhaseIdealLoop::skip_all_loop_predicates(Node* entry) { |
9101 | 390 |
Node* predicate = NULL; |
37287
c2660335bf81
8072422: Cleanup: Remove some unused flags/code in loop optimizations
zmajo
parents:
35558
diff
changeset
|
391 |
predicate = find_predicate_insertion_point(entry, Deoptimization::Reason_loop_limit_check); |
c2660335bf81
8072422: Cleanup: Remove some unused flags/code in loop optimizations
zmajo
parents:
35558
diff
changeset
|
392 |
if (predicate != NULL) { |
52608
61241fc2217a
8211451: ~2.5% regression on compression benchmark starting with 12-b11
roland
parents:
52490
diff
changeset
|
393 |
entry = skip_loop_predicates(entry); |
9446 | 394 |
} |
50623
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
395 |
if (UseProfiledLoopPredicate) { |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
396 |
predicate = find_predicate_insertion_point(entry, Deoptimization::Reason_profile_predicate); |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
397 |
if (predicate != NULL) { // right pattern that can be used by loop predication |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
398 |
entry = skip_loop_predicates(entry); |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
399 |
} |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
400 |
} |
9101 | 401 |
if (UseLoopPredicate) { |
402 |
predicate = find_predicate_insertion_point(entry, Deoptimization::Reason_predicate); |
|
403 |
if (predicate != NULL) { // right pattern that can be used by loop predication |
|
50623
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
404 |
entry = skip_loop_predicates(entry); |
9101 | 405 |
} |
406 |
} |
|
407 |
return entry; |
|
408 |
} |
|
409 |
||
410 |
//--------------------------find_predicate_insertion_point------------------- |
|
411 |
// Find a good location to insert a predicate |
|
412 |
ProjNode* PhaseIdealLoop::find_predicate_insertion_point(Node* start_c, Deoptimization::DeoptReason reason) { |
|
413 |
if (start_c == NULL || !start_c->is_Proj()) |
|
414 |
return NULL; |
|
21089
e1986ff6fe2e
8024069: replace_in_map() should operate on parent maps
roland
parents:
17383
diff
changeset
|
415 |
if (start_c->as_Proj()->is_uncommon_trap_if_pattern(reason)) { |
9101 | 416 |
return start_c->as_Proj(); |
417 |
} |
|
418 |
return NULL; |
|
419 |
} |
|
420 |
||
421 |
//--------------------------find_predicate------------------------------------ |
|
422 |
// Find a predicate |
|
423 |
Node* PhaseIdealLoop::find_predicate(Node* entry) { |
|
424 |
Node* predicate = NULL; |
|
37287
c2660335bf81
8072422: Cleanup: Remove some unused flags/code in loop optimizations
zmajo
parents:
35558
diff
changeset
|
425 |
predicate = find_predicate_insertion_point(entry, Deoptimization::Reason_loop_limit_check); |
c2660335bf81
8072422: Cleanup: Remove some unused flags/code in loop optimizations
zmajo
parents:
35558
diff
changeset
|
426 |
if (predicate != NULL) { // right pattern that can be used by loop predication |
c2660335bf81
8072422: Cleanup: Remove some unused flags/code in loop optimizations
zmajo
parents:
35558
diff
changeset
|
427 |
return entry; |
9446 | 428 |
} |
9101 | 429 |
if (UseLoopPredicate) { |
430 |
predicate = find_predicate_insertion_point(entry, Deoptimization::Reason_predicate); |
|
431 |
if (predicate != NULL) { // right pattern that can be used by loop predication |
|
432 |
return entry; |
|
433 |
} |
|
434 |
} |
|
50623
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
435 |
if (UseProfiledLoopPredicate) { |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
436 |
predicate = find_predicate_insertion_point(entry, Deoptimization::Reason_profile_predicate); |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
437 |
if (predicate != NULL) { // right pattern that can be used by loop predication |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
438 |
return entry; |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
439 |
} |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
440 |
} |
9101 | 441 |
return NULL; |
442 |
} |
|
443 |
||
444 |
//------------------------------Invariance----------------------------------- |
|
445 |
// Helper class for loop_predication_impl to compute invariance on the fly and |
|
446 |
// clone invariants. |
|
447 |
class Invariance : public StackObj { |
|
448 |
VectorSet _visited, _invariant; |
|
449 |
Node_Stack _stack; |
|
450 |
VectorSet _clone_visited; |
|
451 |
Node_List _old_new; // map of old to new (clone) |
|
452 |
IdealLoopTree* _lpt; |
|
453 |
PhaseIdealLoop* _phase; |
|
454 |
||
455 |
// Helper function to set up the invariance for invariance computation |
|
456 |
// If n is a known invariant, set up directly. Otherwise, look up the |
|
457 |
// the possibility to push n onto the stack for further processing. |
|
458 |
void visit(Node* use, Node* n) { |
|
459 |
if (_lpt->is_invariant(n)) { // known invariant |
|
460 |
_invariant.set(n->_idx); |
|
461 |
} else if (!n->is_CFG()) { |
|
462 |
Node *n_ctrl = _phase->ctrl_or_self(n); |
|
463 |
Node *u_ctrl = _phase->ctrl_or_self(use); // self if use is a CFG |
|
464 |
if (_phase->is_dominator(n_ctrl, u_ctrl)) { |
|
465 |
_stack.push(n, n->in(0) == NULL ? 1 : 0); |
|
466 |
} |
|
467 |
} |
|
468 |
} |
|
469 |
||
470 |
// Compute invariance for "the_node" and (possibly) all its inputs recursively |
|
471 |
// on the fly |
|
472 |
void compute_invariance(Node* n) { |
|
473 |
assert(_visited.test(n->_idx), "must be"); |
|
474 |
visit(n, n); |
|
475 |
while (_stack.is_nonempty()) { |
|
476 |
Node* n = _stack.node(); |
|
477 |
uint idx = _stack.index(); |
|
478 |
if (idx == n->req()) { // all inputs are processed |
|
479 |
_stack.pop(); |
|
480 |
// n is invariant if it's inputs are all invariant |
|
481 |
bool all_inputs_invariant = true; |
|
482 |
for (uint i = 0; i < n->req(); i++) { |
|
483 |
Node* in = n->in(i); |
|
484 |
if (in == NULL) continue; |
|
485 |
assert(_visited.test(in->_idx), "must have visited input"); |
|
486 |
if (!_invariant.test(in->_idx)) { // bad guy |
|
487 |
all_inputs_invariant = false; |
|
488 |
break; |
|
489 |
} |
|
490 |
} |
|
491 |
if (all_inputs_invariant) { |
|
31035
0f0743952c41
8077504: Unsafe load can loose control dependency and cause crash
roland
parents:
30309
diff
changeset
|
492 |
// If n's control is a predicate that was moved out of the |
0f0743952c41
8077504: Unsafe load can loose control dependency and cause crash
roland
parents:
30309
diff
changeset
|
493 |
// loop, it was marked invariant but n is only invariant if |
0f0743952c41
8077504: Unsafe load can loose control dependency and cause crash
roland
parents:
30309
diff
changeset
|
494 |
// it depends only on that test. Otherwise, unless that test |
0f0743952c41
8077504: Unsafe load can loose control dependency and cause crash
roland
parents:
30309
diff
changeset
|
495 |
// is out of the loop, it's not invariant. |
0f0743952c41
8077504: Unsafe load can loose control dependency and cause crash
roland
parents:
30309
diff
changeset
|
496 |
if (n->is_CFG() || n->depends_only_on_test() || n->in(0) == NULL || !_phase->is_member(_lpt, n->in(0))) { |
0f0743952c41
8077504: Unsafe load can loose control dependency and cause crash
roland
parents:
30309
diff
changeset
|
497 |
_invariant.set(n->_idx); // I am a invariant too |
0f0743952c41
8077504: Unsafe load can loose control dependency and cause crash
roland
parents:
30309
diff
changeset
|
498 |
} |
9101 | 499 |
} |
500 |
} else { // process next input |
|
501 |
_stack.set_index(idx + 1); |
|
502 |
Node* m = n->in(idx); |
|
503 |
if (m != NULL && !_visited.test_set(m->_idx)) { |
|
504 |
visit(n, m); |
|
505 |
} |
|
506 |
} |
|
507 |
} |
|
508 |
} |
|
509 |
||
510 |
// Helper function to set up _old_new map for clone_nodes. |
|
511 |
// If n is a known invariant, set up directly ("clone" of n == n). |
|
512 |
// Otherwise, push n onto the stack for real cloning. |
|
513 |
void clone_visit(Node* n) { |
|
514 |
assert(_invariant.test(n->_idx), "must be invariant"); |
|
515 |
if (_lpt->is_invariant(n)) { // known invariant |
|
516 |
_old_new.map(n->_idx, n); |
|
517 |
} else { // to be cloned |
|
518 |
assert(!n->is_CFG(), "should not see CFG here"); |
|
519 |
_stack.push(n, n->in(0) == NULL ? 1 : 0); |
|
520 |
} |
|
521 |
} |
|
522 |
||
523 |
// Clone "n" and (possibly) all its inputs recursively |
|
524 |
void clone_nodes(Node* n, Node* ctrl) { |
|
525 |
clone_visit(n); |
|
526 |
while (_stack.is_nonempty()) { |
|
527 |
Node* n = _stack.node(); |
|
528 |
uint idx = _stack.index(); |
|
529 |
if (idx == n->req()) { // all inputs processed, clone n! |
|
530 |
_stack.pop(); |
|
531 |
// clone invariant node |
|
532 |
Node* n_cl = n->clone(); |
|
533 |
_old_new.map(n->_idx, n_cl); |
|
534 |
_phase->register_new_node(n_cl, ctrl); |
|
535 |
for (uint i = 0; i < n->req(); i++) { |
|
536 |
Node* in = n_cl->in(i); |
|
537 |
if (in == NULL) continue; |
|
538 |
n_cl->set_req(i, _old_new[in->_idx]); |
|
539 |
} |
|
540 |
} else { // process next input |
|
541 |
_stack.set_index(idx + 1); |
|
542 |
Node* m = n->in(idx); |
|
543 |
if (m != NULL && !_clone_visited.test_set(m->_idx)) { |
|
544 |
clone_visit(m); // visit the input |
|
545 |
} |
|
546 |
} |
|
547 |
} |
|
548 |
} |
|
549 |
||
550 |
public: |
|
551 |
Invariance(Arena* area, IdealLoopTree* lpt) : |
|
51333
f6641fcf7b7e
8208670: Compiler changes to allow enabling -Wreorder
tschatzl
parents:
51078
diff
changeset
|
552 |
_visited(area), _invariant(area), |
f6641fcf7b7e
8208670: Compiler changes to allow enabling -Wreorder
tschatzl
parents:
51078
diff
changeset
|
553 |
_stack(area, 10 /* guess */), |
f6641fcf7b7e
8208670: Compiler changes to allow enabling -Wreorder
tschatzl
parents:
51078
diff
changeset
|
554 |
_clone_visited(area), _old_new(area), |
f6641fcf7b7e
8208670: Compiler changes to allow enabling -Wreorder
tschatzl
parents:
51078
diff
changeset
|
555 |
_lpt(lpt), _phase(lpt->_phase) |
35558
9b3b2740c3ec
8146792: Predicate moved after partial peel may lead to broken graph
roland
parents:
34180
diff
changeset
|
556 |
{ |
48145 | 557 |
LoopNode* head = _lpt->_head->as_Loop(); |
558 |
Node* entry = head->skip_strip_mined()->in(LoopNode::EntryControl); |
|
35558
9b3b2740c3ec
8146792: Predicate moved after partial peel may lead to broken graph
roland
parents:
34180
diff
changeset
|
559 |
if (entry->outcnt() != 1) { |
9b3b2740c3ec
8146792: Predicate moved after partial peel may lead to broken graph
roland
parents:
34180
diff
changeset
|
560 |
// If a node is pinned between the predicates and the loop |
9b3b2740c3ec
8146792: Predicate moved after partial peel may lead to broken graph
roland
parents:
34180
diff
changeset
|
561 |
// entry, we won't be able to move any node in the loop that |
9b3b2740c3ec
8146792: Predicate moved after partial peel may lead to broken graph
roland
parents:
34180
diff
changeset
|
562 |
// depends on it above it in a predicate. Mark all those nodes |
9b3b2740c3ec
8146792: Predicate moved after partial peel may lead to broken graph
roland
parents:
34180
diff
changeset
|
563 |
// as non loop invariatnt. |
9b3b2740c3ec
8146792: Predicate moved after partial peel may lead to broken graph
roland
parents:
34180
diff
changeset
|
564 |
Unique_Node_List wq; |
9b3b2740c3ec
8146792: Predicate moved after partial peel may lead to broken graph
roland
parents:
34180
diff
changeset
|
565 |
wq.push(entry); |
9b3b2740c3ec
8146792: Predicate moved after partial peel may lead to broken graph
roland
parents:
34180
diff
changeset
|
566 |
for (uint next = 0; next < wq.size(); ++next) { |
9b3b2740c3ec
8146792: Predicate moved after partial peel may lead to broken graph
roland
parents:
34180
diff
changeset
|
567 |
Node *n = wq.at(next); |
9b3b2740c3ec
8146792: Predicate moved after partial peel may lead to broken graph
roland
parents:
34180
diff
changeset
|
568 |
for (DUIterator_Fast imax, i = n->fast_outs(imax); i < imax; i++) { |
9b3b2740c3ec
8146792: Predicate moved after partial peel may lead to broken graph
roland
parents:
34180
diff
changeset
|
569 |
Node* u = n->fast_out(i); |
9b3b2740c3ec
8146792: Predicate moved after partial peel may lead to broken graph
roland
parents:
34180
diff
changeset
|
570 |
if (!u->is_CFG()) { |
9b3b2740c3ec
8146792: Predicate moved after partial peel may lead to broken graph
roland
parents:
34180
diff
changeset
|
571 |
Node* c = _phase->get_ctrl(u); |
9b3b2740c3ec
8146792: Predicate moved after partial peel may lead to broken graph
roland
parents:
34180
diff
changeset
|
572 |
if (_lpt->is_member(_phase->get_loop(c)) || _phase->is_dominator(c, head)) { |
9b3b2740c3ec
8146792: Predicate moved after partial peel may lead to broken graph
roland
parents:
34180
diff
changeset
|
573 |
_visited.set(u->_idx); |
9b3b2740c3ec
8146792: Predicate moved after partial peel may lead to broken graph
roland
parents:
34180
diff
changeset
|
574 |
wq.push(u); |
9b3b2740c3ec
8146792: Predicate moved after partial peel may lead to broken graph
roland
parents:
34180
diff
changeset
|
575 |
} |
9b3b2740c3ec
8146792: Predicate moved after partial peel may lead to broken graph
roland
parents:
34180
diff
changeset
|
576 |
} |
9b3b2740c3ec
8146792: Predicate moved after partial peel may lead to broken graph
roland
parents:
34180
diff
changeset
|
577 |
} |
9b3b2740c3ec
8146792: Predicate moved after partial peel may lead to broken graph
roland
parents:
34180
diff
changeset
|
578 |
} |
9b3b2740c3ec
8146792: Predicate moved after partial peel may lead to broken graph
roland
parents:
34180
diff
changeset
|
579 |
} |
9b3b2740c3ec
8146792: Predicate moved after partial peel may lead to broken graph
roland
parents:
34180
diff
changeset
|
580 |
} |
9101 | 581 |
|
582 |
// Map old to n for invariance computation and clone |
|
583 |
void map_ctrl(Node* old, Node* n) { |
|
584 |
assert(old->is_CFG() && n->is_CFG(), "must be"); |
|
585 |
_old_new.map(old->_idx, n); // "clone" of old is n |
|
586 |
_invariant.set(old->_idx); // old is invariant |
|
587 |
_clone_visited.set(old->_idx); |
|
588 |
} |
|
589 |
||
590 |
// Driver function to compute invariance |
|
591 |
bool is_invariant(Node* n) { |
|
592 |
if (!_visited.test_set(n->_idx)) |
|
593 |
compute_invariance(n); |
|
594 |
return (_invariant.test(n->_idx) != 0); |
|
595 |
} |
|
596 |
||
597 |
// Driver function to clone invariant |
|
598 |
Node* clone(Node* n, Node* ctrl) { |
|
599 |
assert(ctrl->is_CFG(), "must be"); |
|
600 |
assert(_invariant.test(n->_idx), "must be an invariant"); |
|
601 |
if (!_clone_visited.test(n->_idx)) |
|
602 |
clone_nodes(n, ctrl); |
|
603 |
return _old_new[n->_idx]; |
|
604 |
} |
|
605 |
}; |
|
606 |
||
607 |
//------------------------------is_range_check_if ----------------------------------- |
|
608 |
// Returns true if the predicate of iff is in "scale*iv + offset u< load_range(ptr)" format |
|
609 |
// Note: this function is particularly designed for loop predication. We require load_range |
|
610 |
// and offset to be loop invariant computed on the fly by "invar" |
|
611 |
bool IdealLoopTree::is_range_check_if(IfNode *iff, PhaseIdealLoop *phase, Invariance& invar) const { |
|
612 |
if (!is_loop_exit(iff)) { |
|
613 |
return false; |
|
614 |
} |
|
615 |
if (!iff->in(1)->is_Bool()) { |
|
616 |
return false; |
|
617 |
} |
|
618 |
const BoolNode *bol = iff->in(1)->as_Bool(); |
|
619 |
if (bol->_test._test != BoolTest::lt) { |
|
620 |
return false; |
|
621 |
} |
|
622 |
if (!bol->in(1)->is_Cmp()) { |
|
623 |
return false; |
|
624 |
} |
|
625 |
const CmpNode *cmp = bol->in(1)->as_Cmp(); |
|
626 |
if (cmp->Opcode() != Op_CmpU) { |
|
627 |
return false; |
|
628 |
} |
|
629 |
Node* range = cmp->in(2); |
|
34180
f0ec91019db2
8042997: Make intrinsic some or all check index/range methods
roland
parents:
34164
diff
changeset
|
630 |
if (range->Opcode() != Op_LoadRange && !iff->is_RangeCheck()) { |
9101 | 631 |
const TypeInt* tint = phase->_igvn.type(range)->isa_int(); |
9446 | 632 |
if (tint == NULL || tint->empty() || tint->_lo < 0) { |
9101 | 633 |
// Allow predication on positive values that aren't LoadRanges. |
634 |
// This allows optimization of loops where the length of the |
|
635 |
// array is a known value and doesn't need to be loaded back |
|
636 |
// from the array. |
|
637 |
return false; |
|
638 |
} |
|
639 |
} |
|
640 |
if (!invar.is_invariant(range)) { |
|
641 |
return false; |
|
642 |
} |
|
643 |
Node *iv = _head->as_CountedLoop()->phi(); |
|
644 |
int scale = 0; |
|
645 |
Node *offset = NULL; |
|
646 |
if (!phase->is_scaled_iv_plus_offset(cmp->in(1), iv, &scale, &offset)) { |
|
647 |
return false; |
|
648 |
} |
|
649 |
if (offset && !invar.is_invariant(offset)) { // offset must be invariant |
|
650 |
return false; |
|
651 |
} |
|
652 |
return true; |
|
653 |
} |
|
654 |
||
655 |
//------------------------------rc_predicate----------------------------------- |
|
656 |
// Create a range check predicate |
|
657 |
// |
|
658 |
// for (i = init; i < limit; i += stride) { |
|
659 |
// a[scale*i+offset] |
|
660 |
// } |
|
661 |
// |
|
662 |
// Compute max(scale*i + offset) for init <= i < limit and build the predicate |
|
663 |
// as "max(scale*i + offset) u< a.length". |
|
664 |
// |
|
665 |
// There are two cases for max(scale*i + offset): |
|
666 |
// (1) stride*scale > 0 |
|
667 |
// max(scale*i + offset) = scale*(limit-stride) + offset |
|
668 |
// (2) stride*scale < 0 |
|
669 |
// max(scale*i + offset) = scale*init + offset |
|
9446 | 670 |
BoolNode* PhaseIdealLoop::rc_predicate(IdealLoopTree *loop, Node* ctrl, |
9101 | 671 |
int scale, Node* offset, |
45965 | 672 |
Node* init, Node* limit, jint stride, |
673 |
Node* range, bool upper, bool &overflow) { |
|
49487
bde392011cd8
8193130: Bad graph when unrolled loop bounds conflicts with range checks
roland
parents:
48145
diff
changeset
|
674 |
jint con_limit = (limit != NULL && limit->is_Con()) ? limit->get_int() : 0; |
45965 | 675 |
jint con_init = init->is_Con() ? init->get_int() : 0; |
676 |
jint con_offset = offset->is_Con() ? offset->get_int() : 0; |
|
677 |
||
9437
9981851b4b8c
7009361: JSR 292 Invalid value on stack on solaris-sparc with -Xcomp
never
parents:
9101
diff
changeset
|
678 |
stringStream* predString = NULL; |
9981851b4b8c
7009361: JSR 292 Invalid value on stack on solaris-sparc with -Xcomp
never
parents:
9101
diff
changeset
|
679 |
if (TraceLoopPredicate) { |
9981851b4b8c
7009361: JSR 292 Invalid value on stack on solaris-sparc with -Xcomp
never
parents:
9101
diff
changeset
|
680 |
predString = new stringStream(); |
9981851b4b8c
7009361: JSR 292 Invalid value on stack on solaris-sparc with -Xcomp
never
parents:
9101
diff
changeset
|
681 |
predString->print("rc_predicate "); |
9981851b4b8c
7009361: JSR 292 Invalid value on stack on solaris-sparc with -Xcomp
never
parents:
9101
diff
changeset
|
682 |
} |
9101 | 683 |
|
45965 | 684 |
overflow = false; |
685 |
Node* max_idx_expr = NULL; |
|
686 |
const TypeInt* idx_type = TypeInt::INT; |
|
687 |
if ((stride > 0) == (scale > 0) == upper) { |
|
51078 | 688 |
guarantee(limit != NULL, "sanity"); |
45965 | 689 |
if (TraceLoopPredicate) { |
46735 | 690 |
if (limit->is_Con()) { |
691 |
predString->print("(%d ", con_limit); |
|
692 |
} else { |
|
693 |
predString->print("(limit "); |
|
694 |
} |
|
45965 | 695 |
predString->print("- %d) ", stride); |
696 |
} |
|
697 |
// Check if (limit - stride) may overflow |
|
698 |
const TypeInt* limit_type = _igvn.type(limit)->isa_int(); |
|
699 |
jint limit_lo = limit_type->_lo; |
|
700 |
jint limit_hi = limit_type->_hi; |
|
701 |
if ((stride > 0 && (java_subtract(limit_lo, stride) < limit_lo)) || |
|
702 |
(stride < 0 && (java_subtract(limit_hi, stride) > limit_hi))) { |
|
703 |
// No overflow possible |
|
704 |
ConINode* con_stride = _igvn.intcon(stride); |
|
705 |
set_ctrl(con_stride, C->root()); |
|
706 |
max_idx_expr = new SubINode(limit, con_stride); |
|
707 |
idx_type = TypeInt::make(limit_lo - stride, limit_hi - stride, limit_type->_widen); |
|
708 |
} else { |
|
709 |
// May overflow |
|
710 |
overflow = true; |
|
711 |
limit = new ConvI2LNode(limit); |
|
712 |
register_new_node(limit, ctrl); |
|
713 |
ConLNode* con_stride = _igvn.longcon(stride); |
|
714 |
set_ctrl(con_stride, C->root()); |
|
715 |
max_idx_expr = new SubLNode(limit, con_stride); |
|
716 |
} |
|
37287
c2660335bf81
8072422: Cleanup: Remove some unused flags/code in loop optimizations
zmajo
parents:
35558
diff
changeset
|
717 |
register_new_node(max_idx_expr, ctrl); |
9101 | 718 |
} else { |
45965 | 719 |
if (TraceLoopPredicate) { |
46735 | 720 |
if (init->is_Con()) { |
721 |
predString->print("%d ", con_init); |
|
722 |
} else { |
|
723 |
predString->print("init "); |
|
724 |
} |
|
45965 | 725 |
} |
726 |
idx_type = _igvn.type(init)->isa_int(); |
|
727 |
max_idx_expr = init; |
|
9101 | 728 |
} |
729 |
||
730 |
if (scale != 1) { |
|
731 |
ConNode* con_scale = _igvn.intcon(scale); |
|
35558
9b3b2740c3ec
8146792: Predicate moved after partial peel may lead to broken graph
roland
parents:
34180
diff
changeset
|
732 |
set_ctrl(con_scale, C->root()); |
45965 | 733 |
if (TraceLoopPredicate) { |
734 |
predString->print("* %d ", scale); |
|
735 |
} |
|
736 |
// Check if (scale * max_idx_expr) may overflow |
|
737 |
const TypeInt* scale_type = TypeInt::make(scale); |
|
738 |
MulINode* mul = new MulINode(max_idx_expr, con_scale); |
|
739 |
idx_type = (TypeInt*)mul->mul_ring(idx_type, scale_type); |
|
740 |
if (overflow || TypeInt::INT->higher_equal(idx_type)) { |
|
741 |
// May overflow |
|
742 |
mul->destruct(); |
|
743 |
if (!overflow) { |
|
744 |
max_idx_expr = new ConvI2LNode(max_idx_expr); |
|
745 |
register_new_node(max_idx_expr, ctrl); |
|
746 |
} |
|
747 |
overflow = true; |
|
748 |
con_scale = _igvn.longcon(scale); |
|
749 |
set_ctrl(con_scale, C->root()); |
|
750 |
max_idx_expr = new MulLNode(max_idx_expr, con_scale); |
|
751 |
} else { |
|
752 |
// No overflow possible |
|
753 |
max_idx_expr = mul; |
|
754 |
} |
|
9101 | 755 |
register_new_node(max_idx_expr, ctrl); |
756 |
} |
|
757 |
||
45965 | 758 |
if (offset && (!offset->is_Con() || con_offset != 0)){ |
759 |
if (TraceLoopPredicate) { |
|
46630
75aa3e39d02c
8182299: Enable disabled clang warnings, build on OSX 10 + Xcode 8
jwilhelm
parents:
37287
diff
changeset
|
760 |
if (offset->is_Con()) { |
46735 | 761 |
predString->print("+ %d ", con_offset); |
46630
75aa3e39d02c
8182299: Enable disabled clang warnings, build on OSX 10 + Xcode 8
jwilhelm
parents:
37287
diff
changeset
|
762 |
} else { |
46735 | 763 |
predString->print("+ offset"); |
46630
75aa3e39d02c
8182299: Enable disabled clang warnings, build on OSX 10 + Xcode 8
jwilhelm
parents:
37287
diff
changeset
|
764 |
} |
45965 | 765 |
} |
766 |
// Check if (max_idx_expr + offset) may overflow |
|
767 |
const TypeInt* offset_type = _igvn.type(offset)->isa_int(); |
|
768 |
jint lo = java_add(idx_type->_lo, offset_type->_lo); |
|
769 |
jint hi = java_add(idx_type->_hi, offset_type->_hi); |
|
770 |
if (overflow || (lo > hi) || |
|
771 |
((idx_type->_lo & offset_type->_lo) < 0 && lo >= 0) || |
|
772 |
((~(idx_type->_hi | offset_type->_hi)) < 0 && hi < 0)) { |
|
773 |
// May overflow |
|
774 |
if (!overflow) { |
|
775 |
max_idx_expr = new ConvI2LNode(max_idx_expr); |
|
776 |
register_new_node(max_idx_expr, ctrl); |
|
777 |
} |
|
778 |
overflow = true; |
|
779 |
offset = new ConvI2LNode(offset); |
|
780 |
register_new_node(offset, ctrl); |
|
781 |
max_idx_expr = new AddLNode(max_idx_expr, offset); |
|
782 |
} else { |
|
783 |
// No overflow possible |
|
784 |
max_idx_expr = new AddINode(max_idx_expr, offset); |
|
785 |
} |
|
9101 | 786 |
register_new_node(max_idx_expr, ctrl); |
787 |
} |
|
788 |
||
45965 | 789 |
CmpNode* cmp = NULL; |
790 |
if (overflow) { |
|
791 |
// Integer expressions may overflow, do long comparison |
|
792 |
range = new ConvI2LNode(range); |
|
793 |
register_new_node(range, ctrl); |
|
49487
bde392011cd8
8193130: Bad graph when unrolled loop bounds conflicts with range checks
roland
parents:
48145
diff
changeset
|
794 |
cmp = new CmpULNode(max_idx_expr, range); |
45965 | 795 |
} else { |
796 |
cmp = new CmpUNode(max_idx_expr, range); |
|
797 |
} |
|
9101 | 798 |
register_new_node(cmp, ctrl); |
24923
9631f7d691dc
8034812: remove IDX_INIT macro hack in Node class
thartmann
parents:
24424
diff
changeset
|
799 |
BoolNode* bol = new BoolNode(cmp, BoolTest::lt); |
9101 | 800 |
register_new_node(bol, ctrl); |
801 |
||
9437
9981851b4b8c
7009361: JSR 292 Invalid value on stack on solaris-sparc with -Xcomp
never
parents:
9101
diff
changeset
|
802 |
if (TraceLoopPredicate) { |
9981851b4b8c
7009361: JSR 292 Invalid value on stack on solaris-sparc with -Xcomp
never
parents:
9101
diff
changeset
|
803 |
predString->print_cr("<u range"); |
24424
2658d7834c6e
8037816: Fix for 8036122 breaks build with Xcode5/clang
drchase
parents:
23528
diff
changeset
|
804 |
tty->print("%s", predString->as_string()); |
9437
9981851b4b8c
7009361: JSR 292 Invalid value on stack on solaris-sparc with -Xcomp
never
parents:
9101
diff
changeset
|
805 |
} |
9101 | 806 |
return bol; |
807 |
} |
|
808 |
||
50623
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
809 |
// Should loop predication look not only in the path from tail to head |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
810 |
// but also in branches of the loop body? |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
811 |
bool PhaseIdealLoop::loop_predication_should_follow_branches(IdealLoopTree *loop, ProjNode *predicate_proj, float& loop_trip_cnt) { |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
812 |
if (!UseProfiledLoopPredicate) { |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
813 |
return false; |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
814 |
} |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
815 |
|
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
816 |
if (predicate_proj == NULL) { |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
817 |
return false; |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
818 |
} |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
819 |
|
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
820 |
LoopNode* head = loop->_head->as_Loop(); |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
821 |
bool follow_branches = true; |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
822 |
IdealLoopTree* l = loop->_child; |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
823 |
// For leaf loops and loops with a single inner loop |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
824 |
while (l != NULL && follow_branches) { |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
825 |
IdealLoopTree* child = l; |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
826 |
if (child->_child != NULL && |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
827 |
child->_head->is_OuterStripMinedLoop()) { |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
828 |
assert(child->_child->_next == NULL, "only one inner loop for strip mined loop"); |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
829 |
assert(child->_child->_head->is_CountedLoop() && child->_child->_head->as_CountedLoop()->is_strip_mined(), "inner loop should be strip mined"); |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
830 |
child = child->_child; |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
831 |
} |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
832 |
if (child->_child != NULL || child->_irreducible) { |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
833 |
follow_branches = false; |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
834 |
} |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
835 |
l = l->_next; |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
836 |
} |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
837 |
if (follow_branches) { |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
838 |
loop->compute_profile_trip_cnt(this); |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
839 |
if (head->is_profile_trip_failed()) { |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
840 |
follow_branches = false; |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
841 |
} else { |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
842 |
loop_trip_cnt = head->profile_trip_cnt(); |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
843 |
if (head->is_CountedLoop()) { |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
844 |
CountedLoopNode* cl = head->as_CountedLoop(); |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
845 |
if (cl->phi() != NULL) { |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
846 |
const TypeInt* t = _igvn.type(cl->phi())->is_int(); |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
847 |
float worst_case_trip_cnt = ((float)t->_hi - t->_lo) / ABS(cl->stride_con()); |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
848 |
if (worst_case_trip_cnt < loop_trip_cnt) { |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
849 |
loop_trip_cnt = worst_case_trip_cnt; |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
850 |
} |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
851 |
} |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
852 |
} |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
853 |
} |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
854 |
} |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
855 |
return follow_branches; |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
856 |
} |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
857 |
|
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
858 |
// Compute probability of reaching some CFG node from a fixed |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
859 |
// dominating CFG node |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
860 |
class PathFrequency { |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
861 |
private: |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
862 |
Node* _dom; // frequencies are computed relative to this node |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
863 |
Node_Stack _stack; |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
864 |
GrowableArray<float> _freqs_stack; // keep track of intermediate result at regions |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
865 |
GrowableArray<float> _freqs; // cache frequencies |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
866 |
PhaseIdealLoop* _phase; |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
867 |
|
52490
61915e1458bc
8205574: Loop predication "assert(f <= 1 && f >= 0) failed Incorrect frequency"
roland
parents:
52414
diff
changeset
|
868 |
void set_rounding(int mode) { |
61915e1458bc
8205574: Loop predication "assert(f <= 1 && f >= 0) failed Incorrect frequency"
roland
parents:
52414
diff
changeset
|
869 |
// fesetround is broken on windows |
61915e1458bc
8205574: Loop predication "assert(f <= 1 && f >= 0) failed Incorrect frequency"
roland
parents:
52414
diff
changeset
|
870 |
NOT_WINDOWS(fesetround(mode);) |
61915e1458bc
8205574: Loop predication "assert(f <= 1 && f >= 0) failed Incorrect frequency"
roland
parents:
52414
diff
changeset
|
871 |
} |
61915e1458bc
8205574: Loop predication "assert(f <= 1 && f >= 0) failed Incorrect frequency"
roland
parents:
52414
diff
changeset
|
872 |
|
61915e1458bc
8205574: Loop predication "assert(f <= 1 && f >= 0) failed Incorrect frequency"
roland
parents:
52414
diff
changeset
|
873 |
void check_frequency(float f) { |
61915e1458bc
8205574: Loop predication "assert(f <= 1 && f >= 0) failed Incorrect frequency"
roland
parents:
52414
diff
changeset
|
874 |
NOT_WINDOWS(assert(f <= 1 && f >= 0, "Incorrect frequency");) |
61915e1458bc
8205574: Loop predication "assert(f <= 1 && f >= 0) failed Incorrect frequency"
roland
parents:
52414
diff
changeset
|
875 |
} |
61915e1458bc
8205574: Loop predication "assert(f <= 1 && f >= 0) failed Incorrect frequency"
roland
parents:
52414
diff
changeset
|
876 |
|
50623
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
877 |
public: |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
878 |
PathFrequency(Node* dom, PhaseIdealLoop* phase) |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
879 |
: _dom(dom), _stack(0), _phase(phase) { |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
880 |
} |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
881 |
|
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
882 |
float to(Node* n) { |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
883 |
// post order walk on the CFG graph from n to _dom |
52490
61915e1458bc
8205574: Loop predication "assert(f <= 1 && f >= 0) failed Incorrect frequency"
roland
parents:
52414
diff
changeset
|
884 |
set_rounding(FE_TOWARDZERO); // make sure rounding doesn't push frequency above 1 |
50623
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
885 |
IdealLoopTree* loop = _phase->get_loop(_dom); |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
886 |
Node* c = n; |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
887 |
for (;;) { |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
888 |
assert(_phase->get_loop(c) == loop, "have to be in the same loop"); |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
889 |
if (c == _dom || _freqs.at_grow(c->_idx, -1) >= 0) { |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
890 |
float f = c == _dom ? 1 : _freqs.at(c->_idx); |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
891 |
Node* prev = c; |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
892 |
while (_stack.size() > 0 && prev == c) { |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
893 |
Node* n = _stack.node(); |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
894 |
if (!n->is_Region()) { |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
895 |
if (_phase->get_loop(n) != _phase->get_loop(n->in(0))) { |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
896 |
// Found an inner loop: compute frequency of reaching this |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
897 |
// exit from the loop head by looking at the number of |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
898 |
// times each loop exit was taken |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
899 |
IdealLoopTree* inner_loop = _phase->get_loop(n->in(0)); |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
900 |
LoopNode* inner_head = inner_loop->_head->as_Loop(); |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
901 |
assert(_phase->get_loop(n) == loop, "only 1 inner loop"); |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
902 |
if (inner_head->is_OuterStripMinedLoop()) { |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
903 |
inner_head->verify_strip_mined(1); |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
904 |
if (n->in(0) == inner_head->in(LoopNode::LoopBackControl)->in(0)) { |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
905 |
n = n->in(0)->in(0)->in(0); |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
906 |
} |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
907 |
inner_loop = inner_loop->_child; |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
908 |
inner_head = inner_loop->_head->as_Loop(); |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
909 |
inner_head->verify_strip_mined(1); |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
910 |
} |
52490
61915e1458bc
8205574: Loop predication "assert(f <= 1 && f >= 0) failed Incorrect frequency"
roland
parents:
52414
diff
changeset
|
911 |
set_rounding(FE_UPWARD); // make sure rounding doesn't push frequency above 1 |
50623
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
912 |
float loop_exit_cnt = 0.0f; |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
913 |
for (uint i = 0; i < inner_loop->_body.size(); i++) { |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
914 |
Node *n = inner_loop->_body[i]; |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
915 |
float c = inner_loop->compute_profile_trip_cnt_helper(n); |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
916 |
loop_exit_cnt += c; |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
917 |
} |
52490
61915e1458bc
8205574: Loop predication "assert(f <= 1 && f >= 0) failed Incorrect frequency"
roland
parents:
52414
diff
changeset
|
918 |
set_rounding(FE_TOWARDZERO); |
50623
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
919 |
float cnt = -1; |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
920 |
if (n->in(0)->is_If()) { |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
921 |
IfNode* iff = n->in(0)->as_If(); |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
922 |
float p = n->in(0)->as_If()->_prob; |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
923 |
if (n->Opcode() == Op_IfFalse) { |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
924 |
p = 1 - p; |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
925 |
} |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
926 |
if (p > PROB_MIN) { |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
927 |
cnt = p * iff->_fcnt; |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
928 |
} else { |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
929 |
cnt = 0; |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
930 |
} |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
931 |
} else { |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
932 |
assert(n->in(0)->is_Jump(), "unsupported node kind"); |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
933 |
JumpNode* jmp = n->in(0)->as_Jump(); |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
934 |
float p = n->in(0)->as_Jump()->_probs[n->as_JumpProj()->_con]; |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
935 |
cnt = p * jmp->_fcnt; |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
936 |
} |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
937 |
float this_exit_f = cnt > 0 ? cnt / loop_exit_cnt : 0; |
52490
61915e1458bc
8205574: Loop predication "assert(f <= 1 && f >= 0) failed Incorrect frequency"
roland
parents:
52414
diff
changeset
|
938 |
check_frequency(this_exit_f); |
50623
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
939 |
f = f * this_exit_f; |
52490
61915e1458bc
8205574: Loop predication "assert(f <= 1 && f >= 0) failed Incorrect frequency"
roland
parents:
52414
diff
changeset
|
940 |
check_frequency(f); |
50623
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
941 |
} else { |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
942 |
float p = -1; |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
943 |
if (n->in(0)->is_If()) { |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
944 |
p = n->in(0)->as_If()->_prob; |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
945 |
if (n->Opcode() == Op_IfFalse) { |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
946 |
p = 1 - p; |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
947 |
} |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
948 |
} else { |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
949 |
assert(n->in(0)->is_Jump(), "unsupported node kind"); |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
950 |
p = n->in(0)->as_Jump()->_probs[n->as_JumpProj()->_con]; |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
951 |
} |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
952 |
f = f * p; |
52490
61915e1458bc
8205574: Loop predication "assert(f <= 1 && f >= 0) failed Incorrect frequency"
roland
parents:
52414
diff
changeset
|
953 |
check_frequency(f); |
50623
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
954 |
} |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
955 |
_freqs.at_put_grow(n->_idx, (float)f, -1); |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
956 |
_stack.pop(); |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
957 |
} else { |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
958 |
float prev_f = _freqs_stack.pop(); |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
959 |
float new_f = f; |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
960 |
f = new_f + prev_f; |
52490
61915e1458bc
8205574: Loop predication "assert(f <= 1 && f >= 0) failed Incorrect frequency"
roland
parents:
52414
diff
changeset
|
961 |
check_frequency(f); |
50623
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
962 |
uint i = _stack.index(); |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
963 |
if (i < n->req()) { |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
964 |
c = n->in(i); |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
965 |
_stack.set_index(i+1); |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
966 |
_freqs_stack.push(f); |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
967 |
} else { |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
968 |
_freqs.at_put_grow(n->_idx, f, -1); |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
969 |
_stack.pop(); |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
970 |
} |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
971 |
} |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
972 |
} |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
973 |
if (_stack.size() == 0) { |
52490
61915e1458bc
8205574: Loop predication "assert(f <= 1 && f >= 0) failed Incorrect frequency"
roland
parents:
52414
diff
changeset
|
974 |
set_rounding(FE_TONEAREST); |
61915e1458bc
8205574: Loop predication "assert(f <= 1 && f >= 0) failed Incorrect frequency"
roland
parents:
52414
diff
changeset
|
975 |
check_frequency(f); |
50623
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
976 |
return f; |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
977 |
} |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
978 |
} else if (c->is_Loop()) { |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
979 |
ShouldNotReachHere(); |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
980 |
c = c->in(LoopNode::EntryControl); |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
981 |
} else if (c->is_Region()) { |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
982 |
_freqs_stack.push(0); |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
983 |
_stack.push(c, 2); |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
984 |
c = c->in(1); |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
985 |
} else { |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
986 |
if (c->is_IfProj()) { |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
987 |
IfNode* iff = c->in(0)->as_If(); |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
988 |
if (iff->_prob == PROB_UNKNOWN) { |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
989 |
// assume never taken |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
990 |
_freqs.at_put_grow(c->_idx, 0, -1); |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
991 |
} else if (_phase->get_loop(c) != _phase->get_loop(iff)) { |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
992 |
if (iff->_fcnt == COUNT_UNKNOWN) { |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
993 |
// assume never taken |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
994 |
_freqs.at_put_grow(c->_idx, 0, -1); |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
995 |
} else { |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
996 |
// skip over loop |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
997 |
_stack.push(c, 1); |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
998 |
c = _phase->get_loop(c->in(0))->_head->as_Loop()->skip_strip_mined()->in(LoopNode::EntryControl); |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
999 |
} |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1000 |
} else { |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1001 |
_stack.push(c, 1); |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1002 |
c = iff; |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1003 |
} |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1004 |
} else if (c->is_JumpProj()) { |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1005 |
JumpNode* jmp = c->in(0)->as_Jump(); |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1006 |
if (_phase->get_loop(c) != _phase->get_loop(jmp)) { |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1007 |
if (jmp->_fcnt == COUNT_UNKNOWN) { |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1008 |
// assume never taken |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1009 |
_freqs.at_put_grow(c->_idx, 0, -1); |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1010 |
} else { |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1011 |
// skip over loop |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1012 |
_stack.push(c, 1); |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1013 |
c = _phase->get_loop(c->in(0))->_head->as_Loop()->skip_strip_mined()->in(LoopNode::EntryControl); |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1014 |
} |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1015 |
} else { |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1016 |
_stack.push(c, 1); |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1017 |
c = jmp; |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1018 |
} |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1019 |
} else if (c->Opcode() == Op_CatchProj && |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1020 |
c->in(0)->Opcode() == Op_Catch && |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1021 |
c->in(0)->in(0)->is_Proj() && |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1022 |
c->in(0)->in(0)->in(0)->is_Call()) { |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1023 |
// assume exceptions are never thrown |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1024 |
uint con = c->as_Proj()->_con; |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1025 |
if (con == CatchProjNode::fall_through_index) { |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1026 |
Node* call = c->in(0)->in(0)->in(0)->in(0); |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1027 |
if (_phase->get_loop(call) != _phase->get_loop(c)) { |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1028 |
_freqs.at_put_grow(c->_idx, 0, -1); |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1029 |
} else { |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1030 |
c = call; |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1031 |
} |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1032 |
} else { |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1033 |
assert(con >= CatchProjNode::catch_all_index, "what else?"); |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1034 |
_freqs.at_put_grow(c->_idx, 0, -1); |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1035 |
} |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1036 |
} else if (c->unique_ctrl_out() == NULL && !c->is_If() && !c->is_Jump()) { |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1037 |
ShouldNotReachHere(); |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1038 |
} else { |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1039 |
c = c->in(0); |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1040 |
} |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1041 |
} |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1042 |
} |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1043 |
ShouldNotReachHere(); |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1044 |
return -1; |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1045 |
} |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1046 |
}; |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1047 |
|
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1048 |
void PhaseIdealLoop::loop_predication_follow_branches(Node *n, IdealLoopTree *loop, float loop_trip_cnt, |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1049 |
PathFrequency& pf, Node_Stack& stack, VectorSet& seen, |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1050 |
Node_List& if_proj_list) { |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1051 |
assert(n->is_Region(), "start from a region"); |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1052 |
Node* tail = loop->tail(); |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1053 |
stack.push(n, 1); |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1054 |
do { |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1055 |
Node* c = stack.node(); |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1056 |
assert(c->is_Region() || c->is_IfProj(), "only region here"); |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1057 |
uint i = stack.index(); |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1058 |
|
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1059 |
if (i < c->req()) { |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1060 |
stack.set_index(i+1); |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1061 |
Node* in = c->in(i); |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1062 |
while (!is_dominator(in, tail) && !seen.test_set(in->_idx)) { |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1063 |
IdealLoopTree* in_loop = get_loop(in); |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1064 |
if (in_loop != loop) { |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1065 |
in = in_loop->_head->in(LoopNode::EntryControl); |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1066 |
} else if (in->is_Region()) { |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1067 |
stack.push(in, 1); |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1068 |
break; |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1069 |
} else if (in->is_IfProj() && |
50923
c98bf5aa35c5
8205515: assert(opcode == Op_RangeCheck) failed: no other if variant here
roland
parents:
50632
diff
changeset
|
1070 |
in->as_Proj()->is_uncommon_trap_if_pattern(Deoptimization::Reason_none) && |
c98bf5aa35c5
8205515: assert(opcode == Op_RangeCheck) failed: no other if variant here
roland
parents:
50632
diff
changeset
|
1071 |
(in->in(0)->Opcode() == Op_If || |
c98bf5aa35c5
8205515: assert(opcode == Op_RangeCheck) failed: no other if variant here
roland
parents:
50632
diff
changeset
|
1072 |
in->in(0)->Opcode() == Op_RangeCheck)) { |
50623
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1073 |
if (pf.to(in) * loop_trip_cnt >= 1) { |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1074 |
stack.push(in, 1); |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1075 |
} |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1076 |
in = in->in(0); |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1077 |
} else { |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1078 |
in = in->in(0); |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1079 |
} |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1080 |
} |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1081 |
} else { |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1082 |
if (c->is_IfProj()) { |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1083 |
if_proj_list.push(c); |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1084 |
} |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1085 |
stack.pop(); |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1086 |
} |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1087 |
|
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1088 |
} while (stack.size() > 0); |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1089 |
} |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1090 |
|
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1091 |
|
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1092 |
bool PhaseIdealLoop::loop_predication_impl_helper(IdealLoopTree *loop, ProjNode* proj, ProjNode *predicate_proj, |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1093 |
CountedLoopNode *cl, ConNode* zero, Invariance& invar, |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1094 |
Deoptimization::DeoptReason reason) { |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1095 |
// Following are changed to nonnull when a predicate can be hoisted |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1096 |
ProjNode* new_predicate_proj = NULL; |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1097 |
IfNode* iff = proj->in(0)->as_If(); |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1098 |
Node* test = iff->in(1); |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1099 |
if (!test->is_Bool()){ //Conv2B, ... |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1100 |
return false; |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1101 |
} |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1102 |
BoolNode* bol = test->as_Bool(); |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1103 |
if (invar.is_invariant(bol)) { |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1104 |
// Invariant test |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1105 |
new_predicate_proj = create_new_if_for_predicate(predicate_proj, NULL, |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1106 |
reason, |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1107 |
iff->Opcode()); |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1108 |
Node* ctrl = new_predicate_proj->in(0)->as_If()->in(0); |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1109 |
BoolNode* new_predicate_bol = invar.clone(bol, ctrl)->as_Bool(); |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1110 |
|
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1111 |
// Negate test if necessary |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1112 |
bool negated = false; |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1113 |
if (proj->_con != predicate_proj->_con) { |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1114 |
new_predicate_bol = new BoolNode(new_predicate_bol->in(1), new_predicate_bol->_test.negate()); |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1115 |
register_new_node(new_predicate_bol, ctrl); |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1116 |
negated = true; |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1117 |
} |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1118 |
IfNode* new_predicate_iff = new_predicate_proj->in(0)->as_If(); |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1119 |
_igvn.hash_delete(new_predicate_iff); |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1120 |
new_predicate_iff->set_req(1, new_predicate_bol); |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1121 |
#ifndef PRODUCT |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1122 |
if (TraceLoopPredicate) { |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1123 |
tty->print("Predicate invariant if%s: %d ", negated ? " negated" : "", new_predicate_iff->_idx); |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1124 |
loop->dump_head(); |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1125 |
} else if (TraceLoopOpts) { |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1126 |
tty->print("Predicate IC "); |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1127 |
loop->dump_head(); |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1128 |
} |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1129 |
#endif |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1130 |
} else if (cl != NULL && loop->is_range_check_if(iff, this, invar)) { |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1131 |
// Range check for counted loops |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1132 |
const Node* cmp = bol->in(1)->as_Cmp(); |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1133 |
Node* idx = cmp->in(1); |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1134 |
assert(!invar.is_invariant(idx), "index is variant"); |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1135 |
Node* rng = cmp->in(2); |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1136 |
assert(rng->Opcode() == Op_LoadRange || iff->is_RangeCheck() || _igvn.type(rng)->is_int()->_lo >= 0, "must be"); |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1137 |
assert(invar.is_invariant(rng), "range must be invariant"); |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1138 |
int scale = 1; |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1139 |
Node* offset = zero; |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1140 |
bool ok = is_scaled_iv_plus_offset(idx, cl->phi(), &scale, &offset); |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1141 |
assert(ok, "must be index expression"); |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1142 |
|
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1143 |
Node* init = cl->init_trip(); |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1144 |
// Limit is not exact. |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1145 |
// Calculate exact limit here. |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1146 |
// Note, counted loop's test is '<' or '>'. |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1147 |
Node* limit = exact_limit(loop); |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1148 |
int stride = cl->stride()->get_int(); |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1149 |
|
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1150 |
// Build if's for the upper and lower bound tests. The |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1151 |
// lower_bound test will dominate the upper bound test and all |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1152 |
// cloned or created nodes will use the lower bound test as |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1153 |
// their declared control. |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1154 |
|
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1155 |
// Perform cloning to keep Invariance state correct since the |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1156 |
// late schedule will place invariant things in the loop. |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1157 |
Node *ctrl = predicate_proj->in(0)->as_If()->in(0); |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1158 |
rng = invar.clone(rng, ctrl); |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1159 |
if (offset && offset != zero) { |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1160 |
assert(invar.is_invariant(offset), "offset must be loop invariant"); |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1161 |
offset = invar.clone(offset, ctrl); |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1162 |
} |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1163 |
// If predicate expressions may overflow in the integer range, longs are used. |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1164 |
bool overflow = false; |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1165 |
|
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1166 |
// Test the lower bound |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1167 |
BoolNode* lower_bound_bol = rc_predicate(loop, ctrl, scale, offset, init, limit, stride, rng, false, overflow); |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1168 |
// Negate test if necessary |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1169 |
bool negated = false; |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1170 |
if (proj->_con != predicate_proj->_con) { |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1171 |
lower_bound_bol = new BoolNode(lower_bound_bol->in(1), lower_bound_bol->_test.negate()); |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1172 |
register_new_node(lower_bound_bol, ctrl); |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1173 |
negated = true; |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1174 |
} |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1175 |
ProjNode* lower_bound_proj = create_new_if_for_predicate(predicate_proj, NULL, reason, overflow ? Op_If : iff->Opcode()); |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1176 |
IfNode* lower_bound_iff = lower_bound_proj->in(0)->as_If(); |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1177 |
_igvn.hash_delete(lower_bound_iff); |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1178 |
lower_bound_iff->set_req(1, lower_bound_bol); |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1179 |
if (TraceLoopPredicate) tty->print_cr("lower bound check if: %s %d ", negated ? " negated" : "", lower_bound_iff->_idx); |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1180 |
|
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1181 |
// Test the upper bound |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1182 |
BoolNode* upper_bound_bol = rc_predicate(loop, lower_bound_proj, scale, offset, init, limit, stride, rng, true, overflow); |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1183 |
negated = false; |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1184 |
if (proj->_con != predicate_proj->_con) { |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1185 |
upper_bound_bol = new BoolNode(upper_bound_bol->in(1), upper_bound_bol->_test.negate()); |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1186 |
register_new_node(upper_bound_bol, ctrl); |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1187 |
negated = true; |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1188 |
} |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1189 |
ProjNode* upper_bound_proj = create_new_if_for_predicate(predicate_proj, NULL, reason, overflow ? Op_If : iff->Opcode()); |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1190 |
assert(upper_bound_proj->in(0)->as_If()->in(0) == lower_bound_proj, "should dominate"); |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1191 |
IfNode* upper_bound_iff = upper_bound_proj->in(0)->as_If(); |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1192 |
_igvn.hash_delete(upper_bound_iff); |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1193 |
upper_bound_iff->set_req(1, upper_bound_bol); |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1194 |
if (TraceLoopPredicate) tty->print_cr("upper bound check if: %s %d ", negated ? " negated" : "", lower_bound_iff->_idx); |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1195 |
|
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1196 |
// Fall through into rest of the clean up code which will move |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1197 |
// any dependent nodes onto the upper bound test. |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1198 |
new_predicate_proj = upper_bound_proj; |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1199 |
|
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1200 |
if (iff->is_RangeCheck()) { |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1201 |
new_predicate_proj = insert_skeleton_predicate(iff, loop, proj, predicate_proj, upper_bound_proj, scale, offset, init, limit, stride, rng, overflow, reason); |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1202 |
} |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1203 |
|
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1204 |
#ifndef PRODUCT |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1205 |
if (TraceLoopOpts && !TraceLoopPredicate) { |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1206 |
tty->print("Predicate RC "); |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1207 |
loop->dump_head(); |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1208 |
} |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1209 |
#endif |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1210 |
} else { |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1211 |
// Loop variant check (for example, range check in non-counted loop) |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1212 |
// with uncommon trap. |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1213 |
return false; |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1214 |
} |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1215 |
assert(new_predicate_proj != NULL, "sanity"); |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1216 |
// Success - attach condition (new_predicate_bol) to predicate if |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1217 |
invar.map_ctrl(proj, new_predicate_proj); // so that invariance test can be appropriate |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1218 |
|
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1219 |
// Eliminate the old If in the loop body |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1220 |
dominated_by( new_predicate_proj, iff, proj->_con != new_predicate_proj->_con ); |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1221 |
|
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1222 |
C->set_major_progress(); |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1223 |
return true; |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1224 |
} |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1225 |
|
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1226 |
|
49487
bde392011cd8
8193130: Bad graph when unrolled loop bounds conflicts with range checks
roland
parents:
48145
diff
changeset
|
1227 |
// After pre/main/post loops are created, we'll put a copy of some |
50632
fd430e352427
8205033: [REDO] Induction variable of over-unrolled loop conflicts with range checks
thartmann
parents:
50623
diff
changeset
|
1228 |
// range checks between the pre and main loop to validate the value |
fd430e352427
8205033: [REDO] Induction variable of over-unrolled loop conflicts with range checks
thartmann
parents:
50623
diff
changeset
|
1229 |
// of the main loop induction variable. Make a copy of the predicates |
fd430e352427
8205033: [REDO] Induction variable of over-unrolled loop conflicts with range checks
thartmann
parents:
50623
diff
changeset
|
1230 |
// here with an opaque node as a place holder for the value (will be |
53306
8e260023fc53
8216135: C2 assert(!had_error) failed: bad dominance
roland
parents:
52925
diff
changeset
|
1231 |
// updated by PhaseIdealLoop::clone_skeleton_predicate()). |
49487
bde392011cd8
8193130: Bad graph when unrolled loop bounds conflicts with range checks
roland
parents:
48145
diff
changeset
|
1232 |
ProjNode* PhaseIdealLoop::insert_skeleton_predicate(IfNode* iff, IdealLoopTree *loop, |
bde392011cd8
8193130: Bad graph when unrolled loop bounds conflicts with range checks
roland
parents:
48145
diff
changeset
|
1233 |
ProjNode* proj, ProjNode *predicate_proj, |
bde392011cd8
8193130: Bad graph when unrolled loop bounds conflicts with range checks
roland
parents:
48145
diff
changeset
|
1234 |
ProjNode* upper_bound_proj, |
bde392011cd8
8193130: Bad graph when unrolled loop bounds conflicts with range checks
roland
parents:
48145
diff
changeset
|
1235 |
int scale, Node* offset, |
bde392011cd8
8193130: Bad graph when unrolled loop bounds conflicts with range checks
roland
parents:
48145
diff
changeset
|
1236 |
Node* init, Node* limit, jint stride, |
50623
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1237 |
Node* rng, bool &overflow, |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1238 |
Deoptimization::DeoptReason reason) { |
49487
bde392011cd8
8193130: Bad graph when unrolled loop bounds conflicts with range checks
roland
parents:
48145
diff
changeset
|
1239 |
assert(proj->_con && predicate_proj->_con, "not a range check?"); |
bde392011cd8
8193130: Bad graph when unrolled loop bounds conflicts with range checks
roland
parents:
48145
diff
changeset
|
1240 |
Node* opaque_init = new Opaque1Node(C, init); |
bde392011cd8
8193130: Bad graph when unrolled loop bounds conflicts with range checks
roland
parents:
48145
diff
changeset
|
1241 |
register_new_node(opaque_init, upper_bound_proj); |
bde392011cd8
8193130: Bad graph when unrolled loop bounds conflicts with range checks
roland
parents:
48145
diff
changeset
|
1242 |
BoolNode* bol = rc_predicate(loop, upper_bound_proj, scale, offset, opaque_init, limit, stride, rng, (stride > 0) != (scale > 0), overflow); |
bde392011cd8
8193130: Bad graph when unrolled loop bounds conflicts with range checks
roland
parents:
48145
diff
changeset
|
1243 |
Node* opaque_bol = new Opaque4Node(C, bol, _igvn.intcon(1)); // This will go away once loop opts are over |
bde392011cd8
8193130: Bad graph when unrolled loop bounds conflicts with range checks
roland
parents:
48145
diff
changeset
|
1244 |
register_new_node(opaque_bol, upper_bound_proj); |
50623
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1245 |
ProjNode* new_proj = create_new_if_for_predicate(predicate_proj, NULL, reason, overflow ? Op_If : iff->Opcode()); |
49487
bde392011cd8
8193130: Bad graph when unrolled loop bounds conflicts with range checks
roland
parents:
48145
diff
changeset
|
1246 |
_igvn.replace_input_of(new_proj->in(0), 1, opaque_bol); |
bde392011cd8
8193130: Bad graph when unrolled loop bounds conflicts with range checks
roland
parents:
48145
diff
changeset
|
1247 |
assert(opaque_init->outcnt() > 0, "should be used"); |
bde392011cd8
8193130: Bad graph when unrolled loop bounds conflicts with range checks
roland
parents:
48145
diff
changeset
|
1248 |
return new_proj; |
bde392011cd8
8193130: Bad graph when unrolled loop bounds conflicts with range checks
roland
parents:
48145
diff
changeset
|
1249 |
} |
bde392011cd8
8193130: Bad graph when unrolled loop bounds conflicts with range checks
roland
parents:
48145
diff
changeset
|
1250 |
|
9101 | 1251 |
//------------------------------ loop_predication_impl-------------------------- |
1252 |
// Insert loop predicates for null checks and range checks |
|
1253 |
bool PhaseIdealLoop::loop_predication_impl(IdealLoopTree *loop) { |
|
1254 |
if (!UseLoopPredicate) return false; |
|
1255 |
||
1256 |
if (!loop->_head->is_Loop()) { |
|
1257 |
// Could be a simple region when irreducible loops are present. |
|
1258 |
return false; |
|
1259 |
} |
|
9446 | 1260 |
LoopNode* head = loop->_head->as_Loop(); |
9101 | 1261 |
|
9446 | 1262 |
if (head->unique_ctrl_out()->Opcode() == Op_NeverBranch) { |
9101 | 1263 |
// do nothing for infinite loops |
1264 |
return false; |
|
1265 |
} |
|
1266 |
||
48145 | 1267 |
if (head->is_OuterStripMinedLoop()) { |
1268 |
return false; |
|
1269 |
} |
|
1270 |
||
9101 | 1271 |
CountedLoopNode *cl = NULL; |
10263
fa58671dde31
7077439: Possible reference through NULL in loopPredicate.cpp:726
kvn
parents:
10258
diff
changeset
|
1272 |
if (head->is_valid_counted_loop()) { |
9446 | 1273 |
cl = head->as_CountedLoop(); |
9101 | 1274 |
// do nothing for iteration-splitted loops |
1275 |
if (!cl->is_normal_loop()) return false; |
|
10253
35b975b1e8f3
7070134: Hotspot crashes with sigsegv from PorterStemmer
kvn
parents:
9446
diff
changeset
|
1276 |
// Avoid RCE if Counted loop's test is '!='. |
35b975b1e8f3
7070134: Hotspot crashes with sigsegv from PorterStemmer
kvn
parents:
9446
diff
changeset
|
1277 |
BoolTest::mask bt = cl->loopexit()->test_trip(); |
35b975b1e8f3
7070134: Hotspot crashes with sigsegv from PorterStemmer
kvn
parents:
9446
diff
changeset
|
1278 |
if (bt != BoolTest::lt && bt != BoolTest::gt) |
35b975b1e8f3
7070134: Hotspot crashes with sigsegv from PorterStemmer
kvn
parents:
9446
diff
changeset
|
1279 |
cl = NULL; |
9101 | 1280 |
} |
1281 |
||
48145 | 1282 |
Node* entry = head->skip_strip_mined()->in(LoopNode::EntryControl); |
50623
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1283 |
ProjNode *loop_limit_proj = NULL; |
9446 | 1284 |
ProjNode *predicate_proj = NULL; |
50623
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1285 |
ProjNode *profile_predicate_proj = NULL; |
9446 | 1286 |
// Loop limit check predicate should be near the loop. |
50623
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1287 |
loop_limit_proj = find_predicate_insertion_point(entry, Deoptimization::Reason_loop_limit_check); |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1288 |
if (loop_limit_proj != NULL) { |
52608
61241fc2217a
8211451: ~2.5% regression on compression benchmark starting with 12-b11
roland
parents:
52490
diff
changeset
|
1289 |
entry = skip_loop_predicates(loop_limit_proj); |
50623
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1290 |
} |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1291 |
bool has_profile_predicates = false; |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1292 |
profile_predicate_proj = find_predicate_insertion_point(entry, Deoptimization::Reason_profile_predicate); |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1293 |
if (profile_predicate_proj != NULL) { |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1294 |
Node* n = skip_loop_predicates(entry); |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1295 |
// Check if predicates were already added to the profile predicate |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1296 |
// block |
50923
c98bf5aa35c5
8205515: assert(opcode == Op_RangeCheck) failed: no other if variant here
roland
parents:
50632
diff
changeset
|
1297 |
if (n != entry->in(0)->in(0) || n->outcnt() != 1) { |
50623
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1298 |
has_profile_predicates = true; |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1299 |
} |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1300 |
entry = n; |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1301 |
} |
9446 | 1302 |
predicate_proj = find_predicate_insertion_point(entry, Deoptimization::Reason_predicate); |
50623
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1303 |
|
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1304 |
float loop_trip_cnt = -1; |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1305 |
bool follow_branches = loop_predication_should_follow_branches(loop, profile_predicate_proj, loop_trip_cnt); |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1306 |
assert(!follow_branches || loop_trip_cnt >= 0, "negative trip count?"); |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1307 |
|
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1308 |
if (predicate_proj == NULL && !follow_branches) { |
9101 | 1309 |
#ifndef PRODUCT |
1310 |
if (TraceLoopPredicate) { |
|
1311 |
tty->print("missing predicate:"); |
|
1312 |
loop->dump_head(); |
|
9446 | 1313 |
head->dump(1); |
9101 | 1314 |
} |
1315 |
#endif |
|
1316 |
return false; |
|
1317 |
} |
|
1318 |
ConNode* zero = _igvn.intcon(0); |
|
1319 |
set_ctrl(zero, C->root()); |
|
1320 |
||
1321 |
ResourceArea *area = Thread::current()->resource_area(); |
|
1322 |
Invariance invar(area, loop); |
|
1323 |
||
1324 |
// Create list of if-projs such that a newer proj dominates all older |
|
1325 |
// projs in the list, and they all dominate loop->tail() |
|
1326 |
Node_List if_proj_list(area); |
|
50623
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1327 |
Node_List regions(area); |
9101 | 1328 |
Node *current_proj = loop->tail(); //start from tail |
50623
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1329 |
|
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1330 |
|
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1331 |
Node_List controls(area); |
9101 | 1332 |
while (current_proj != head) { |
1333 |
if (loop == get_loop(current_proj) && // still in the loop ? |
|
1334 |
current_proj->is_Proj() && // is a projection ? |
|
34164
a9e6034d7707
8137168: Replace IfNode with a new RangeCheckNode for range checks
roland
parents:
32733
diff
changeset
|
1335 |
(current_proj->in(0)->Opcode() == Op_If || |
a9e6034d7707
8137168: Replace IfNode with a new RangeCheckNode for range checks
roland
parents:
32733
diff
changeset
|
1336 |
current_proj->in(0)->Opcode() == Op_RangeCheck)) { // is a if projection ? |
9101 | 1337 |
if_proj_list.push(current_proj); |
1338 |
} |
|
50623
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1339 |
if (follow_branches && |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1340 |
current_proj->Opcode() == Op_Region && |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1341 |
loop == get_loop(current_proj)) { |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1342 |
regions.push(current_proj); |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1343 |
} |
9101 | 1344 |
current_proj = idom(current_proj); |
1345 |
} |
|
1346 |
||
1347 |
bool hoisted = false; // true if at least one proj is promoted |
|
1348 |
||
50623
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1349 |
if (!has_profile_predicates) { |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1350 |
while (if_proj_list.size() > 0) { |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1351 |
Node* n = if_proj_list.pop(); |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1352 |
|
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1353 |
ProjNode* proj = n->as_Proj(); |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1354 |
IfNode* iff = proj->in(0)->as_If(); |
9101 | 1355 |
|
50623
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1356 |
CallStaticJavaNode* call = proj->is_uncommon_trap_if_pattern(Deoptimization::Reason_none); |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1357 |
if (call == NULL) { |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1358 |
if (loop->is_loop_exit(iff)) { |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1359 |
// stop processing the remaining projs in the list because the execution of them |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1360 |
// depends on the condition of "iff" (iff->in(1)). |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1361 |
break; |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1362 |
} else { |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1363 |
// Both arms are inside the loop. There are two cases: |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1364 |
// (1) there is one backward branch. In this case, any remaining proj |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1365 |
// in the if_proj list post-dominates "iff". So, the condition of "iff" |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1366 |
// does not determine the execution the remining projs directly, and we |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1367 |
// can safely continue. |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1368 |
// (2) both arms are forwarded, i.e. a diamond shape. In this case, "proj" |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1369 |
// does not dominate loop->tail(), so it can not be in the if_proj list. |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1370 |
continue; |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1371 |
} |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1372 |
} |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1373 |
Deoptimization::DeoptReason reason = Deoptimization::trap_request_reason(call->uncommon_trap_request()); |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1374 |
if (reason == Deoptimization::Reason_predicate) { |
9101 | 1375 |
break; |
50623
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1376 |
} |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1377 |
|
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1378 |
if (predicate_proj != NULL) { |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1379 |
hoisted = loop_predication_impl_helper(loop, proj, predicate_proj, cl, zero, invar, Deoptimization::Reason_predicate) | hoisted; |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1380 |
} |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1381 |
} // end while |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1382 |
} |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1383 |
|
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1384 |
if (follow_branches) { |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1385 |
PathFrequency pf(loop->_head, this); |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1386 |
|
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1387 |
// Some projections were skipped by regular predicates because of |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1388 |
// an early loop exit. Try them with profile data. |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1389 |
while (if_proj_list.size() > 0) { |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1390 |
Node* proj = if_proj_list.pop(); |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1391 |
float f = pf.to(proj); |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1392 |
if (proj->as_Proj()->is_uncommon_trap_if_pattern(Deoptimization::Reason_none) && |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1393 |
f * loop_trip_cnt >= 1) { |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1394 |
hoisted = loop_predication_impl_helper(loop, proj->as_Proj(), profile_predicate_proj, cl, zero, invar, Deoptimization::Reason_profile_predicate) | hoisted; |
9101 | 1395 |
} |
1396 |
} |
|
1397 |
||
50623
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1398 |
// And look into all branches |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1399 |
Node_Stack stack(0); |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1400 |
VectorSet seen(Thread::current()->resource_area()); |
58351
d322bf161e31
8231223: C2's conditional move optimization fails with assert(bol->Opcode() == Op_Bool) failed
thartmann
parents:
58106
diff
changeset
|
1401 |
Node_List if_proj_list_freq(area); |
50623
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1402 |
while (regions.size() > 0) { |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1403 |
Node* c = regions.pop(); |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1404 |
loop_predication_follow_branches(c, loop, loop_trip_cnt, pf, stack, seen, if_proj_list_freq); |
9101 | 1405 |
} |
1406 |
||
50623
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1407 |
for (uint i = 0; i < if_proj_list_freq.size(); i++) { |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1408 |
ProjNode* proj = if_proj_list_freq.at(i)->as_Proj(); |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1409 |
hoisted = loop_predication_impl_helper(loop, proj, profile_predicate_proj, cl, zero, invar, Deoptimization::Reason_profile_predicate) | hoisted; |
9101 | 1410 |
} |
50623
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1411 |
} |
9101 | 1412 |
|
1413 |
#ifndef PRODUCT |
|
1414 |
// report that the loop predication has been actually performed |
|
1415 |
// for this loop |
|
1416 |
if (TraceLoopPredicate && hoisted) { |
|
1417 |
tty->print("Loop Predication Performed:"); |
|
1418 |
loop->dump_head(); |
|
1419 |
} |
|
1420 |
#endif |
|
1421 |
||
48145 | 1422 |
head->verify_strip_mined(1); |
1423 |
||
9101 | 1424 |
return hoisted; |
1425 |
} |
|
1426 |
||
1427 |
//------------------------------loop_predication-------------------------------- |
|
1428 |
// driver routine for loop predication optimization |
|
1429 |
bool IdealLoopTree::loop_predication( PhaseIdealLoop *phase) { |
|
1430 |
bool hoisted = false; |
|
1431 |
// Recursively promote predicates |
|
1432 |
if (_child) { |
|
1433 |
hoisted = _child->loop_predication( phase); |
|
1434 |
} |
|
1435 |
||
1436 |
// self |
|
1437 |
if (!_irreducible && !tail()->is_top()) { |
|
1438 |
hoisted |= phase->loop_predication_impl(this); |
|
1439 |
} |
|
1440 |
||
1441 |
if (_next) { //sibling |
|
1442 |
hoisted |= _next->loop_predication( phase); |
|
1443 |
} |
|
1444 |
||
1445 |
return hoisted; |
|
1446 |
} |