author | thartmann |
Tue, 22 Jan 2019 18:25:56 +0100 | |
changeset 53429 | 1b292ae4eb50 |
parent 53357 | c52a37f40324 |
child 54423 | 6c0ab8bd8da5 |
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 |
||
52414
6d42c07ba238
8212610: Fix handling of memory in PhaseIdealLoop::clone_loop_predicates()
roland
parents:
51333
diff
changeset
|
304 |
void PhaseIdealLoop::clone_loop_predicates_fix_mem(ProjNode* dom_proj , ProjNode* proj, |
6d42c07ba238
8212610: Fix handling of memory in PhaseIdealLoop::clone_loop_predicates()
roland
parents:
51333
diff
changeset
|
305 |
PhaseIdealLoop* loop_phase, |
6d42c07ba238
8212610: Fix handling of memory in PhaseIdealLoop::clone_loop_predicates()
roland
parents:
51333
diff
changeset
|
306 |
PhaseIterGVN* igvn) { |
6d42c07ba238
8212610: Fix handling of memory in PhaseIdealLoop::clone_loop_predicates()
roland
parents:
51333
diff
changeset
|
307 |
Compile* C = NULL; |
6d42c07ba238
8212610: Fix handling of memory in PhaseIdealLoop::clone_loop_predicates()
roland
parents:
51333
diff
changeset
|
308 |
if (loop_phase != NULL) { |
6d42c07ba238
8212610: Fix handling of memory in PhaseIdealLoop::clone_loop_predicates()
roland
parents:
51333
diff
changeset
|
309 |
igvn = &loop_phase->igvn(); |
6d42c07ba238
8212610: Fix handling of memory in PhaseIdealLoop::clone_loop_predicates()
roland
parents:
51333
diff
changeset
|
310 |
} |
6d42c07ba238
8212610: Fix handling of memory in PhaseIdealLoop::clone_loop_predicates()
roland
parents:
51333
diff
changeset
|
311 |
C = igvn->C; |
6d42c07ba238
8212610: Fix handling of memory in PhaseIdealLoop::clone_loop_predicates()
roland
parents:
51333
diff
changeset
|
312 |
ProjNode* other_dom_proj = dom_proj->in(0)->as_Multi()->proj_out(1-dom_proj->_con); |
6d42c07ba238
8212610: Fix handling of memory in PhaseIdealLoop::clone_loop_predicates()
roland
parents:
51333
diff
changeset
|
313 |
Node* dom_r = other_dom_proj->unique_ctrl_out(); |
6d42c07ba238
8212610: Fix handling of memory in PhaseIdealLoop::clone_loop_predicates()
roland
parents:
51333
diff
changeset
|
314 |
if (dom_r->is_Region()) { |
6d42c07ba238
8212610: Fix handling of memory in PhaseIdealLoop::clone_loop_predicates()
roland
parents:
51333
diff
changeset
|
315 |
assert(dom_r->unique_ctrl_out()->is_Call(), "unc expected"); |
6d42c07ba238
8212610: Fix handling of memory in PhaseIdealLoop::clone_loop_predicates()
roland
parents:
51333
diff
changeset
|
316 |
ProjNode* other_proj = proj->in(0)->as_Multi()->proj_out(1-proj->_con); |
6d42c07ba238
8212610: Fix handling of memory in PhaseIdealLoop::clone_loop_predicates()
roland
parents:
51333
diff
changeset
|
317 |
Node* r = other_proj->unique_ctrl_out(); |
6d42c07ba238
8212610: Fix handling of memory in PhaseIdealLoop::clone_loop_predicates()
roland
parents:
51333
diff
changeset
|
318 |
assert(r->is_Region() && r->unique_ctrl_out()->is_Call(), "cloned predicate should have caused region to be added"); |
6d42c07ba238
8212610: Fix handling of memory in PhaseIdealLoop::clone_loop_predicates()
roland
parents:
51333
diff
changeset
|
319 |
for (DUIterator_Fast imax, i = dom_r->fast_outs(imax); i < imax; i++) { |
6d42c07ba238
8212610: Fix handling of memory in PhaseIdealLoop::clone_loop_predicates()
roland
parents:
51333
diff
changeset
|
320 |
Node* dom_use = dom_r->fast_out(i); |
6d42c07ba238
8212610: Fix handling of memory in PhaseIdealLoop::clone_loop_predicates()
roland
parents:
51333
diff
changeset
|
321 |
if (dom_use->is_Phi() && dom_use->bottom_type() == Type::MEMORY) { |
6d42c07ba238
8212610: Fix handling of memory in PhaseIdealLoop::clone_loop_predicates()
roland
parents:
51333
diff
changeset
|
322 |
assert(dom_use->in(0) == dom_r, ""); |
6d42c07ba238
8212610: Fix handling of memory in PhaseIdealLoop::clone_loop_predicates()
roland
parents:
51333
diff
changeset
|
323 |
Node* phi = NULL; |
6d42c07ba238
8212610: Fix handling of memory in PhaseIdealLoop::clone_loop_predicates()
roland
parents:
51333
diff
changeset
|
324 |
for (DUIterator_Fast jmax, j = r->fast_outs(jmax); j < jmax; j++) { |
6d42c07ba238
8212610: Fix handling of memory in PhaseIdealLoop::clone_loop_predicates()
roland
parents:
51333
diff
changeset
|
325 |
Node* use = r->fast_out(j); |
6d42c07ba238
8212610: Fix handling of memory in PhaseIdealLoop::clone_loop_predicates()
roland
parents:
51333
diff
changeset
|
326 |
if (use->is_Phi() && use->bottom_type() == Type::MEMORY && |
6d42c07ba238
8212610: Fix handling of memory in PhaseIdealLoop::clone_loop_predicates()
roland
parents:
51333
diff
changeset
|
327 |
use->adr_type() == dom_use->adr_type()) { |
6d42c07ba238
8212610: Fix handling of memory in PhaseIdealLoop::clone_loop_predicates()
roland
parents:
51333
diff
changeset
|
328 |
assert(use->in(0) == r, ""); |
6d42c07ba238
8212610: Fix handling of memory in PhaseIdealLoop::clone_loop_predicates()
roland
parents:
51333
diff
changeset
|
329 |
assert(phi == NULL, "only one phi"); |
6d42c07ba238
8212610: Fix handling of memory in PhaseIdealLoop::clone_loop_predicates()
roland
parents:
51333
diff
changeset
|
330 |
phi = use; |
6d42c07ba238
8212610: Fix handling of memory in PhaseIdealLoop::clone_loop_predicates()
roland
parents:
51333
diff
changeset
|
331 |
} |
6d42c07ba238
8212610: Fix handling of memory in PhaseIdealLoop::clone_loop_predicates()
roland
parents:
51333
diff
changeset
|
332 |
} |
6d42c07ba238
8212610: Fix handling of memory in PhaseIdealLoop::clone_loop_predicates()
roland
parents:
51333
diff
changeset
|
333 |
if (phi == NULL) { |
6d42c07ba238
8212610: Fix handling of memory in PhaseIdealLoop::clone_loop_predicates()
roland
parents:
51333
diff
changeset
|
334 |
const TypePtr* adr_type = dom_use->adr_type(); |
6d42c07ba238
8212610: Fix handling of memory in PhaseIdealLoop::clone_loop_predicates()
roland
parents:
51333
diff
changeset
|
335 |
int alias = C->get_alias_index(adr_type); |
6d42c07ba238
8212610: Fix handling of memory in PhaseIdealLoop::clone_loop_predicates()
roland
parents:
51333
diff
changeset
|
336 |
Node* call = r->unique_ctrl_out(); |
6d42c07ba238
8212610: Fix handling of memory in PhaseIdealLoop::clone_loop_predicates()
roland
parents:
51333
diff
changeset
|
337 |
Node* mem = call->in(TypeFunc::Memory); |
6d42c07ba238
8212610: Fix handling of memory in PhaseIdealLoop::clone_loop_predicates()
roland
parents:
51333
diff
changeset
|
338 |
MergeMemNode* mm = NULL; |
6d42c07ba238
8212610: Fix handling of memory in PhaseIdealLoop::clone_loop_predicates()
roland
parents:
51333
diff
changeset
|
339 |
if (mem->is_MergeMem()) { |
6d42c07ba238
8212610: Fix handling of memory in PhaseIdealLoop::clone_loop_predicates()
roland
parents:
51333
diff
changeset
|
340 |
mm = mem->clone()->as_MergeMem(); |
6d42c07ba238
8212610: Fix handling of memory in PhaseIdealLoop::clone_loop_predicates()
roland
parents:
51333
diff
changeset
|
341 |
if (adr_type == TypePtr::BOTTOM) { |
6d42c07ba238
8212610: Fix handling of memory in PhaseIdealLoop::clone_loop_predicates()
roland
parents:
51333
diff
changeset
|
342 |
mem = mem->as_MergeMem()->base_memory(); |
6d42c07ba238
8212610: Fix handling of memory in PhaseIdealLoop::clone_loop_predicates()
roland
parents:
51333
diff
changeset
|
343 |
} else { |
6d42c07ba238
8212610: Fix handling of memory in PhaseIdealLoop::clone_loop_predicates()
roland
parents:
51333
diff
changeset
|
344 |
mem = mem->as_MergeMem()->memory_at(alias); |
6d42c07ba238
8212610: Fix handling of memory in PhaseIdealLoop::clone_loop_predicates()
roland
parents:
51333
diff
changeset
|
345 |
} |
6d42c07ba238
8212610: Fix handling of memory in PhaseIdealLoop::clone_loop_predicates()
roland
parents:
51333
diff
changeset
|
346 |
} else { |
6d42c07ba238
8212610: Fix handling of memory in PhaseIdealLoop::clone_loop_predicates()
roland
parents:
51333
diff
changeset
|
347 |
mm = MergeMemNode::make(mem); |
6d42c07ba238
8212610: Fix handling of memory in PhaseIdealLoop::clone_loop_predicates()
roland
parents:
51333
diff
changeset
|
348 |
} |
6d42c07ba238
8212610: Fix handling of memory in PhaseIdealLoop::clone_loop_predicates()
roland
parents:
51333
diff
changeset
|
349 |
phi = PhiNode::make(r, mem, Type::MEMORY, adr_type); |
6d42c07ba238
8212610: Fix handling of memory in PhaseIdealLoop::clone_loop_predicates()
roland
parents:
51333
diff
changeset
|
350 |
if (adr_type == TypePtr::BOTTOM) { |
6d42c07ba238
8212610: Fix handling of memory in PhaseIdealLoop::clone_loop_predicates()
roland
parents:
51333
diff
changeset
|
351 |
mm->set_base_memory(phi); |
6d42c07ba238
8212610: Fix handling of memory in PhaseIdealLoop::clone_loop_predicates()
roland
parents:
51333
diff
changeset
|
352 |
} else { |
6d42c07ba238
8212610: Fix handling of memory in PhaseIdealLoop::clone_loop_predicates()
roland
parents:
51333
diff
changeset
|
353 |
mm->set_memory_at(alias, phi); |
6d42c07ba238
8212610: Fix handling of memory in PhaseIdealLoop::clone_loop_predicates()
roland
parents:
51333
diff
changeset
|
354 |
} |
6d42c07ba238
8212610: Fix handling of memory in PhaseIdealLoop::clone_loop_predicates()
roland
parents:
51333
diff
changeset
|
355 |
if (loop_phase != NULL) { |
6d42c07ba238
8212610: Fix handling of memory in PhaseIdealLoop::clone_loop_predicates()
roland
parents:
51333
diff
changeset
|
356 |
loop_phase->register_new_node(mm, r); |
6d42c07ba238
8212610: Fix handling of memory in PhaseIdealLoop::clone_loop_predicates()
roland
parents:
51333
diff
changeset
|
357 |
loop_phase->register_new_node(phi, r); |
6d42c07ba238
8212610: Fix handling of memory in PhaseIdealLoop::clone_loop_predicates()
roland
parents:
51333
diff
changeset
|
358 |
} else { |
6d42c07ba238
8212610: Fix handling of memory in PhaseIdealLoop::clone_loop_predicates()
roland
parents:
51333
diff
changeset
|
359 |
igvn->register_new_node_with_optimizer(mm); |
6d42c07ba238
8212610: Fix handling of memory in PhaseIdealLoop::clone_loop_predicates()
roland
parents:
51333
diff
changeset
|
360 |
igvn->register_new_node_with_optimizer(phi); |
6d42c07ba238
8212610: Fix handling of memory in PhaseIdealLoop::clone_loop_predicates()
roland
parents:
51333
diff
changeset
|
361 |
} |
6d42c07ba238
8212610: Fix handling of memory in PhaseIdealLoop::clone_loop_predicates()
roland
parents:
51333
diff
changeset
|
362 |
igvn->replace_input_of(call, TypeFunc::Memory, mm); |
6d42c07ba238
8212610: Fix handling of memory in PhaseIdealLoop::clone_loop_predicates()
roland
parents:
51333
diff
changeset
|
363 |
} |
6d42c07ba238
8212610: Fix handling of memory in PhaseIdealLoop::clone_loop_predicates()
roland
parents:
51333
diff
changeset
|
364 |
igvn->replace_input_of(phi, r->find_edge(other_proj), dom_use->in(dom_r->find_edge(other_dom_proj))); |
6d42c07ba238
8212610: Fix handling of memory in PhaseIdealLoop::clone_loop_predicates()
roland
parents:
51333
diff
changeset
|
365 |
} |
6d42c07ba238
8212610: Fix handling of memory in PhaseIdealLoop::clone_loop_predicates()
roland
parents:
51333
diff
changeset
|
366 |
} |
6d42c07ba238
8212610: Fix handling of memory in PhaseIdealLoop::clone_loop_predicates()
roland
parents:
51333
diff
changeset
|
367 |
} |
6d42c07ba238
8212610: Fix handling of memory in PhaseIdealLoop::clone_loop_predicates()
roland
parents:
51333
diff
changeset
|
368 |
} |
6d42c07ba238
8212610: Fix handling of memory in PhaseIdealLoop::clone_loop_predicates()
roland
parents:
51333
diff
changeset
|
369 |
|
6d42c07ba238
8212610: Fix handling of memory in PhaseIdealLoop::clone_loop_predicates()
roland
parents:
51333
diff
changeset
|
370 |
|
9101 | 371 |
// Clone loop predicates to cloned loops (peeled, unswitched, split_if). |
372 |
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
|
373 |
bool clone_limit_check, |
61241fc2217a
8211451: ~2.5% regression on compression benchmark starting with 12-b11
roland
parents:
52490
diff
changeset
|
374 |
PhaseIdealLoop* loop_phase, |
61241fc2217a
8211451: ~2.5% regression on compression benchmark starting with 12-b11
roland
parents:
52490
diff
changeset
|
375 |
PhaseIterGVN* igvn) { |
9101 | 376 |
#ifdef ASSERT |
377 |
if (new_entry == NULL || !(new_entry->is_Proj() || new_entry->is_Region() || new_entry->is_SafePoint())) { |
|
378 |
if (new_entry != NULL) |
|
379 |
new_entry->dump(); |
|
380 |
assert(false, "not IfTrue, IfFalse, Region or SafePoint"); |
|
381 |
} |
|
382 |
#endif |
|
383 |
// Search original predicates |
|
384 |
Node* entry = old_entry; |
|
9446 | 385 |
ProjNode* limit_check_proj = NULL; |
37287
c2660335bf81
8072422: Cleanup: Remove some unused flags/code in loop optimizations
zmajo
parents:
35558
diff
changeset
|
386 |
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
|
387 |
if (limit_check_proj != NULL) { |
52608
61241fc2217a
8211451: ~2.5% regression on compression benchmark starting with 12-b11
roland
parents:
52490
diff
changeset
|
388 |
entry = skip_loop_predicates(entry); |
9446 | 389 |
} |
50623
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
390 |
ProjNode* profile_predicate_proj = NULL; |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
391 |
ProjNode* predicate_proj = NULL; |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
392 |
if (UseProfiledLoopPredicate) { |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
393 |
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
|
394 |
if (profile_predicate_proj != NULL) { |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
395 |
entry = skip_loop_predicates(entry); |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
396 |
} |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
397 |
} |
9101 | 398 |
if (UseLoopPredicate) { |
50623
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
399 |
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
|
400 |
} |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
401 |
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
|
402 |
// clone predicate |
52414
6d42c07ba238
8212610: Fix handling of memory in PhaseIdealLoop::clone_loop_predicates()
roland
parents:
51333
diff
changeset
|
403 |
ProjNode* proj = clone_predicate(predicate_proj, new_entry, |
6d42c07ba238
8212610: Fix handling of memory in PhaseIdealLoop::clone_loop_predicates()
roland
parents:
51333
diff
changeset
|
404 |
Deoptimization::Reason_predicate, |
6d42c07ba238
8212610: Fix handling of memory in PhaseIdealLoop::clone_loop_predicates()
roland
parents:
51333
diff
changeset
|
405 |
loop_phase, igvn); |
6d42c07ba238
8212610: Fix handling of memory in PhaseIdealLoop::clone_loop_predicates()
roland
parents:
51333
diff
changeset
|
406 |
assert(proj != NULL, "IfTrue or IfFalse after clone predicate"); |
6d42c07ba238
8212610: Fix handling of memory in PhaseIdealLoop::clone_loop_predicates()
roland
parents:
51333
diff
changeset
|
407 |
new_entry = proj; |
50623
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
408 |
if (TraceLoopPredicate) { |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
409 |
tty->print("Loop Predicate cloned: "); |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
410 |
debug_only( new_entry->in(0)->dump(); ); |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
411 |
} |
52414
6d42c07ba238
8212610: Fix handling of memory in PhaseIdealLoop::clone_loop_predicates()
roland
parents:
51333
diff
changeset
|
412 |
if (profile_predicate_proj != NULL) { |
6d42c07ba238
8212610: Fix handling of memory in PhaseIdealLoop::clone_loop_predicates()
roland
parents:
51333
diff
changeset
|
413 |
// A node that produces memory may be out of loop and depend on |
6d42c07ba238
8212610: Fix handling of memory in PhaseIdealLoop::clone_loop_predicates()
roland
parents:
51333
diff
changeset
|
414 |
// a profiled predicates. In that case the memory state at the |
6d42c07ba238
8212610: Fix handling of memory in PhaseIdealLoop::clone_loop_predicates()
roland
parents:
51333
diff
changeset
|
415 |
// end of profiled predicates and at the end of predicates are |
6d42c07ba238
8212610: Fix handling of memory in PhaseIdealLoop::clone_loop_predicates()
roland
parents:
51333
diff
changeset
|
416 |
// not the same. The cloned predicates are dominated by the |
6d42c07ba238
8212610: Fix handling of memory in PhaseIdealLoop::clone_loop_predicates()
roland
parents:
51333
diff
changeset
|
417 |
// profiled predicates but may have the wrong memory |
6d42c07ba238
8212610: Fix handling of memory in PhaseIdealLoop::clone_loop_predicates()
roland
parents:
51333
diff
changeset
|
418 |
// state. Update it. |
6d42c07ba238
8212610: Fix handling of memory in PhaseIdealLoop::clone_loop_predicates()
roland
parents:
51333
diff
changeset
|
419 |
clone_loop_predicates_fix_mem(profile_predicate_proj, proj, loop_phase, igvn); |
6d42c07ba238
8212610: Fix handling of memory in PhaseIdealLoop::clone_loop_predicates()
roland
parents:
51333
diff
changeset
|
420 |
} |
50623
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
421 |
} |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
422 |
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
|
423 |
// clone predicate |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
424 |
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
|
425 |
Deoptimization::Reason_profile_predicate, |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
426 |
loop_phase, igvn); |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
427 |
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
|
428 |
if (TraceLoopPredicate) { |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
429 |
tty->print("Loop Predicate cloned: "); |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
430 |
debug_only( new_entry->in(0)->dump(); ); |
9101 | 431 |
} |
432 |
} |
|
9446 | 433 |
if (limit_check_proj != NULL && clone_limit_check) { |
434 |
// Clone loop limit check last to insert it before loop. |
|
435 |
// Don't clone a limit check which was already finalized |
|
436 |
// 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
|
437 |
new_entry = clone_predicate(limit_check_proj, new_entry, |
10c77b8c8d3e
7068051: SIGSEGV in PhaseIdealLoop::build_loop_late_post
kvn
parents:
10253
diff
changeset
|
438 |
Deoptimization::Reason_loop_limit_check, |
10c77b8c8d3e
7068051: SIGSEGV in PhaseIdealLoop::build_loop_late_post
kvn
parents:
10253
diff
changeset
|
439 |
loop_phase, igvn); |
10c77b8c8d3e
7068051: SIGSEGV in PhaseIdealLoop::build_loop_late_post
kvn
parents:
10253
diff
changeset
|
440 |
assert(new_entry != NULL && new_entry->is_Proj(), "IfTrue or IfFalse after clone limit check"); |
9446 | 441 |
if (TraceLoopLimitCheck) { |
10258
10c77b8c8d3e
7068051: SIGSEGV in PhaseIdealLoop::build_loop_late_post
kvn
parents:
10253
diff
changeset
|
442 |
tty->print("Loop Limit Check cloned: "); |
9446 | 443 |
debug_only( new_entry->in(0)->dump(); ) |
444 |
} |
|
445 |
} |
|
9101 | 446 |
return new_entry; |
447 |
} |
|
448 |
||
449 |
//--------------------------skip_loop_predicates------------------------------ |
|
450 |
// Skip related predicates. |
|
451 |
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
|
452 |
IfNode* iff = entry->in(0)->as_If(); |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
453 |
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
|
454 |
Node* rgn = uncommon_proj->unique_ctrl_out(); |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
455 |
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
|
456 |
entry = entry->in(0)->in(0); |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
457 |
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
|
458 |
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
|
459 |
if (uncommon_proj->unique_ctrl_out() != rgn) |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
460 |
break; |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
461 |
entry = entry->in(0)->in(0); |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
462 |
} |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
463 |
return entry; |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
464 |
} |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
465 |
|
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
466 |
Node* PhaseIdealLoop::skip_all_loop_predicates(Node* entry) { |
9101 | 467 |
Node* predicate = NULL; |
37287
c2660335bf81
8072422: Cleanup: Remove some unused flags/code in loop optimizations
zmajo
parents:
35558
diff
changeset
|
468 |
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
|
469 |
if (predicate != NULL) { |
52608
61241fc2217a
8211451: ~2.5% regression on compression benchmark starting with 12-b11
roland
parents:
52490
diff
changeset
|
470 |
entry = skip_loop_predicates(entry); |
9446 | 471 |
} |
50623
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
472 |
if (UseProfiledLoopPredicate) { |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
473 |
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
|
474 |
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
|
475 |
entry = skip_loop_predicates(entry); |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
476 |
} |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
477 |
} |
9101 | 478 |
if (UseLoopPredicate) { |
479 |
predicate = find_predicate_insertion_point(entry, Deoptimization::Reason_predicate); |
|
480 |
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
|
481 |
entry = skip_loop_predicates(entry); |
9101 | 482 |
} |
483 |
} |
|
484 |
return entry; |
|
485 |
} |
|
486 |
||
487 |
//--------------------------find_predicate_insertion_point------------------- |
|
488 |
// Find a good location to insert a predicate |
|
489 |
ProjNode* PhaseIdealLoop::find_predicate_insertion_point(Node* start_c, Deoptimization::DeoptReason reason) { |
|
490 |
if (start_c == NULL || !start_c->is_Proj()) |
|
491 |
return NULL; |
|
21089
e1986ff6fe2e
8024069: replace_in_map() should operate on parent maps
roland
parents:
17383
diff
changeset
|
492 |
if (start_c->as_Proj()->is_uncommon_trap_if_pattern(reason)) { |
9101 | 493 |
return start_c->as_Proj(); |
494 |
} |
|
495 |
return NULL; |
|
496 |
} |
|
497 |
||
498 |
//--------------------------find_predicate------------------------------------ |
|
499 |
// Find a predicate |
|
500 |
Node* PhaseIdealLoop::find_predicate(Node* entry) { |
|
501 |
Node* predicate = NULL; |
|
37287
c2660335bf81
8072422: Cleanup: Remove some unused flags/code in loop optimizations
zmajo
parents:
35558
diff
changeset
|
502 |
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
|
503 |
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
|
504 |
return entry; |
9446 | 505 |
} |
9101 | 506 |
if (UseLoopPredicate) { |
507 |
predicate = find_predicate_insertion_point(entry, Deoptimization::Reason_predicate); |
|
508 |
if (predicate != NULL) { // right pattern that can be used by loop predication |
|
509 |
return entry; |
|
510 |
} |
|
511 |
} |
|
50623
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
512 |
if (UseProfiledLoopPredicate) { |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
513 |
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
|
514 |
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
|
515 |
return entry; |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
516 |
} |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
517 |
} |
9101 | 518 |
return NULL; |
519 |
} |
|
520 |
||
521 |
//------------------------------Invariance----------------------------------- |
|
522 |
// Helper class for loop_predication_impl to compute invariance on the fly and |
|
523 |
// clone invariants. |
|
524 |
class Invariance : public StackObj { |
|
525 |
VectorSet _visited, _invariant; |
|
526 |
Node_Stack _stack; |
|
527 |
VectorSet _clone_visited; |
|
528 |
Node_List _old_new; // map of old to new (clone) |
|
529 |
IdealLoopTree* _lpt; |
|
530 |
PhaseIdealLoop* _phase; |
|
531 |
||
532 |
// Helper function to set up the invariance for invariance computation |
|
533 |
// If n is a known invariant, set up directly. Otherwise, look up the |
|
534 |
// the possibility to push n onto the stack for further processing. |
|
535 |
void visit(Node* use, Node* n) { |
|
536 |
if (_lpt->is_invariant(n)) { // known invariant |
|
537 |
_invariant.set(n->_idx); |
|
538 |
} else if (!n->is_CFG()) { |
|
52925
9c18c9d839d3
8214259: Implementation: JEP 189: Shenandoah: A Low-Pause-Time Garbage Collector (Experimental)
rkennke
parents:
52608
diff
changeset
|
539 |
if (n->Opcode() == Op_ShenandoahWriteBarrier) { |
9c18c9d839d3
8214259: Implementation: JEP 189: Shenandoah: A Low-Pause-Time Garbage Collector (Experimental)
rkennke
parents:
52608
diff
changeset
|
540 |
return; |
9c18c9d839d3
8214259: Implementation: JEP 189: Shenandoah: A Low-Pause-Time Garbage Collector (Experimental)
rkennke
parents:
52608
diff
changeset
|
541 |
} |
9101 | 542 |
Node *n_ctrl = _phase->ctrl_or_self(n); |
543 |
Node *u_ctrl = _phase->ctrl_or_self(use); // self if use is a CFG |
|
544 |
if (_phase->is_dominator(n_ctrl, u_ctrl)) { |
|
545 |
_stack.push(n, n->in(0) == NULL ? 1 : 0); |
|
546 |
} |
|
547 |
} |
|
548 |
} |
|
549 |
||
550 |
// Compute invariance for "the_node" and (possibly) all its inputs recursively |
|
551 |
// on the fly |
|
552 |
void compute_invariance(Node* n) { |
|
553 |
assert(_visited.test(n->_idx), "must be"); |
|
554 |
visit(n, n); |
|
555 |
while (_stack.is_nonempty()) { |
|
556 |
Node* n = _stack.node(); |
|
557 |
uint idx = _stack.index(); |
|
558 |
if (idx == n->req()) { // all inputs are processed |
|
559 |
_stack.pop(); |
|
560 |
// n is invariant if it's inputs are all invariant |
|
561 |
bool all_inputs_invariant = true; |
|
562 |
for (uint i = 0; i < n->req(); i++) { |
|
563 |
Node* in = n->in(i); |
|
564 |
if (in == NULL) continue; |
|
565 |
assert(_visited.test(in->_idx), "must have visited input"); |
|
566 |
if (!_invariant.test(in->_idx)) { // bad guy |
|
567 |
all_inputs_invariant = false; |
|
568 |
break; |
|
569 |
} |
|
570 |
} |
|
571 |
if (all_inputs_invariant) { |
|
31035
0f0743952c41
8077504: Unsafe load can loose control dependency and cause crash
roland
parents:
30309
diff
changeset
|
572 |
// 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
|
573 |
// 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
|
574 |
// 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
|
575 |
// 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
|
576 |
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
|
577 |
_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
|
578 |
} |
9101 | 579 |
} |
580 |
} else { // process next input |
|
581 |
_stack.set_index(idx + 1); |
|
582 |
Node* m = n->in(idx); |
|
583 |
if (m != NULL && !_visited.test_set(m->_idx)) { |
|
584 |
visit(n, m); |
|
585 |
} |
|
586 |
} |
|
587 |
} |
|
588 |
} |
|
589 |
||
590 |
// Helper function to set up _old_new map for clone_nodes. |
|
591 |
// If n is a known invariant, set up directly ("clone" of n == n). |
|
592 |
// Otherwise, push n onto the stack for real cloning. |
|
593 |
void clone_visit(Node* n) { |
|
594 |
assert(_invariant.test(n->_idx), "must be invariant"); |
|
595 |
if (_lpt->is_invariant(n)) { // known invariant |
|
596 |
_old_new.map(n->_idx, n); |
|
597 |
} else { // to be cloned |
|
598 |
assert(!n->is_CFG(), "should not see CFG here"); |
|
599 |
_stack.push(n, n->in(0) == NULL ? 1 : 0); |
|
600 |
} |
|
601 |
} |
|
602 |
||
603 |
// Clone "n" and (possibly) all its inputs recursively |
|
604 |
void clone_nodes(Node* n, Node* ctrl) { |
|
605 |
clone_visit(n); |
|
606 |
while (_stack.is_nonempty()) { |
|
607 |
Node* n = _stack.node(); |
|
608 |
uint idx = _stack.index(); |
|
609 |
if (idx == n->req()) { // all inputs processed, clone n! |
|
610 |
_stack.pop(); |
|
611 |
// clone invariant node |
|
612 |
Node* n_cl = n->clone(); |
|
613 |
_old_new.map(n->_idx, n_cl); |
|
614 |
_phase->register_new_node(n_cl, ctrl); |
|
615 |
for (uint i = 0; i < n->req(); i++) { |
|
616 |
Node* in = n_cl->in(i); |
|
617 |
if (in == NULL) continue; |
|
618 |
n_cl->set_req(i, _old_new[in->_idx]); |
|
619 |
} |
|
620 |
} else { // process next input |
|
621 |
_stack.set_index(idx + 1); |
|
622 |
Node* m = n->in(idx); |
|
623 |
if (m != NULL && !_clone_visited.test_set(m->_idx)) { |
|
624 |
clone_visit(m); // visit the input |
|
625 |
} |
|
626 |
} |
|
627 |
} |
|
628 |
} |
|
629 |
||
630 |
public: |
|
631 |
Invariance(Arena* area, IdealLoopTree* lpt) : |
|
51333
f6641fcf7b7e
8208670: Compiler changes to allow enabling -Wreorder
tschatzl
parents:
51078
diff
changeset
|
632 |
_visited(area), _invariant(area), |
f6641fcf7b7e
8208670: Compiler changes to allow enabling -Wreorder
tschatzl
parents:
51078
diff
changeset
|
633 |
_stack(area, 10 /* guess */), |
f6641fcf7b7e
8208670: Compiler changes to allow enabling -Wreorder
tschatzl
parents:
51078
diff
changeset
|
634 |
_clone_visited(area), _old_new(area), |
f6641fcf7b7e
8208670: Compiler changes to allow enabling -Wreorder
tschatzl
parents:
51078
diff
changeset
|
635 |
_lpt(lpt), _phase(lpt->_phase) |
35558
9b3b2740c3ec
8146792: Predicate moved after partial peel may lead to broken graph
roland
parents:
34180
diff
changeset
|
636 |
{ |
48145 | 637 |
LoopNode* head = _lpt->_head->as_Loop(); |
638 |
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
|
639 |
if (entry->outcnt() != 1) { |
9b3b2740c3ec
8146792: Predicate moved after partial peel may lead to broken graph
roland
parents:
34180
diff
changeset
|
640 |
// 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
|
641 |
// 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
|
642 |
// 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
|
643 |
// as non loop invariatnt. |
9b3b2740c3ec
8146792: Predicate moved after partial peel may lead to broken graph
roland
parents:
34180
diff
changeset
|
644 |
Unique_Node_List wq; |
9b3b2740c3ec
8146792: Predicate moved after partial peel may lead to broken graph
roland
parents:
34180
diff
changeset
|
645 |
wq.push(entry); |
9b3b2740c3ec
8146792: Predicate moved after partial peel may lead to broken graph
roland
parents:
34180
diff
changeset
|
646 |
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
|
647 |
Node *n = wq.at(next); |
9b3b2740c3ec
8146792: Predicate moved after partial peel may lead to broken graph
roland
parents:
34180
diff
changeset
|
648 |
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
|
649 |
Node* u = n->fast_out(i); |
9b3b2740c3ec
8146792: Predicate moved after partial peel may lead to broken graph
roland
parents:
34180
diff
changeset
|
650 |
if (!u->is_CFG()) { |
9b3b2740c3ec
8146792: Predicate moved after partial peel may lead to broken graph
roland
parents:
34180
diff
changeset
|
651 |
Node* c = _phase->get_ctrl(u); |
9b3b2740c3ec
8146792: Predicate moved after partial peel may lead to broken graph
roland
parents:
34180
diff
changeset
|
652 |
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
|
653 |
_visited.set(u->_idx); |
9b3b2740c3ec
8146792: Predicate moved after partial peel may lead to broken graph
roland
parents:
34180
diff
changeset
|
654 |
wq.push(u); |
9b3b2740c3ec
8146792: Predicate moved after partial peel may lead to broken graph
roland
parents:
34180
diff
changeset
|
655 |
} |
9b3b2740c3ec
8146792: Predicate moved after partial peel may lead to broken graph
roland
parents:
34180
diff
changeset
|
656 |
} |
9b3b2740c3ec
8146792: Predicate moved after partial peel may lead to broken graph
roland
parents:
34180
diff
changeset
|
657 |
} |
9b3b2740c3ec
8146792: Predicate moved after partial peel may lead to broken graph
roland
parents:
34180
diff
changeset
|
658 |
} |
9b3b2740c3ec
8146792: Predicate moved after partial peel may lead to broken graph
roland
parents:
34180
diff
changeset
|
659 |
} |
9b3b2740c3ec
8146792: Predicate moved after partial peel may lead to broken graph
roland
parents:
34180
diff
changeset
|
660 |
} |
9101 | 661 |
|
662 |
// Map old to n for invariance computation and clone |
|
663 |
void map_ctrl(Node* old, Node* n) { |
|
664 |
assert(old->is_CFG() && n->is_CFG(), "must be"); |
|
665 |
_old_new.map(old->_idx, n); // "clone" of old is n |
|
666 |
_invariant.set(old->_idx); // old is invariant |
|
667 |
_clone_visited.set(old->_idx); |
|
668 |
} |
|
669 |
||
670 |
// Driver function to compute invariance |
|
671 |
bool is_invariant(Node* n) { |
|
672 |
if (!_visited.test_set(n->_idx)) |
|
673 |
compute_invariance(n); |
|
674 |
return (_invariant.test(n->_idx) != 0); |
|
675 |
} |
|
676 |
||
677 |
// Driver function to clone invariant |
|
678 |
Node* clone(Node* n, Node* ctrl) { |
|
679 |
assert(ctrl->is_CFG(), "must be"); |
|
680 |
assert(_invariant.test(n->_idx), "must be an invariant"); |
|
681 |
if (!_clone_visited.test(n->_idx)) |
|
682 |
clone_nodes(n, ctrl); |
|
683 |
return _old_new[n->_idx]; |
|
684 |
} |
|
685 |
}; |
|
686 |
||
687 |
//------------------------------is_range_check_if ----------------------------------- |
|
688 |
// Returns true if the predicate of iff is in "scale*iv + offset u< load_range(ptr)" format |
|
689 |
// Note: this function is particularly designed for loop predication. We require load_range |
|
690 |
// and offset to be loop invariant computed on the fly by "invar" |
|
691 |
bool IdealLoopTree::is_range_check_if(IfNode *iff, PhaseIdealLoop *phase, Invariance& invar) const { |
|
692 |
if (!is_loop_exit(iff)) { |
|
693 |
return false; |
|
694 |
} |
|
695 |
if (!iff->in(1)->is_Bool()) { |
|
696 |
return false; |
|
697 |
} |
|
698 |
const BoolNode *bol = iff->in(1)->as_Bool(); |
|
699 |
if (bol->_test._test != BoolTest::lt) { |
|
700 |
return false; |
|
701 |
} |
|
702 |
if (!bol->in(1)->is_Cmp()) { |
|
703 |
return false; |
|
704 |
} |
|
705 |
const CmpNode *cmp = bol->in(1)->as_Cmp(); |
|
706 |
if (cmp->Opcode() != Op_CmpU) { |
|
707 |
return false; |
|
708 |
} |
|
709 |
Node* range = cmp->in(2); |
|
34180
f0ec91019db2
8042997: Make intrinsic some or all check index/range methods
roland
parents:
34164
diff
changeset
|
710 |
if (range->Opcode() != Op_LoadRange && !iff->is_RangeCheck()) { |
9101 | 711 |
const TypeInt* tint = phase->_igvn.type(range)->isa_int(); |
9446 | 712 |
if (tint == NULL || tint->empty() || tint->_lo < 0) { |
9101 | 713 |
// Allow predication on positive values that aren't LoadRanges. |
714 |
// This allows optimization of loops where the length of the |
|
715 |
// array is a known value and doesn't need to be loaded back |
|
716 |
// from the array. |
|
717 |
return false; |
|
718 |
} |
|
719 |
} |
|
720 |
if (!invar.is_invariant(range)) { |
|
721 |
return false; |
|
722 |
} |
|
723 |
Node *iv = _head->as_CountedLoop()->phi(); |
|
724 |
int scale = 0; |
|
725 |
Node *offset = NULL; |
|
726 |
if (!phase->is_scaled_iv_plus_offset(cmp->in(1), iv, &scale, &offset)) { |
|
727 |
return false; |
|
728 |
} |
|
729 |
if (offset && !invar.is_invariant(offset)) { // offset must be invariant |
|
730 |
return false; |
|
731 |
} |
|
732 |
return true; |
|
733 |
} |
|
734 |
||
735 |
//------------------------------rc_predicate----------------------------------- |
|
736 |
// Create a range check predicate |
|
737 |
// |
|
738 |
// for (i = init; i < limit; i += stride) { |
|
739 |
// a[scale*i+offset] |
|
740 |
// } |
|
741 |
// |
|
742 |
// Compute max(scale*i + offset) for init <= i < limit and build the predicate |
|
743 |
// as "max(scale*i + offset) u< a.length". |
|
744 |
// |
|
745 |
// There are two cases for max(scale*i + offset): |
|
746 |
// (1) stride*scale > 0 |
|
747 |
// max(scale*i + offset) = scale*(limit-stride) + offset |
|
748 |
// (2) stride*scale < 0 |
|
749 |
// max(scale*i + offset) = scale*init + offset |
|
9446 | 750 |
BoolNode* PhaseIdealLoop::rc_predicate(IdealLoopTree *loop, Node* ctrl, |
9101 | 751 |
int scale, Node* offset, |
45965 | 752 |
Node* init, Node* limit, jint stride, |
753 |
Node* range, bool upper, bool &overflow) { |
|
49487
bde392011cd8
8193130: Bad graph when unrolled loop bounds conflicts with range checks
roland
parents:
48145
diff
changeset
|
754 |
jint con_limit = (limit != NULL && limit->is_Con()) ? limit->get_int() : 0; |
45965 | 755 |
jint con_init = init->is_Con() ? init->get_int() : 0; |
756 |
jint con_offset = offset->is_Con() ? offset->get_int() : 0; |
|
757 |
||
9437
9981851b4b8c
7009361: JSR 292 Invalid value on stack on solaris-sparc with -Xcomp
never
parents:
9101
diff
changeset
|
758 |
stringStream* predString = NULL; |
9981851b4b8c
7009361: JSR 292 Invalid value on stack on solaris-sparc with -Xcomp
never
parents:
9101
diff
changeset
|
759 |
if (TraceLoopPredicate) { |
9981851b4b8c
7009361: JSR 292 Invalid value on stack on solaris-sparc with -Xcomp
never
parents:
9101
diff
changeset
|
760 |
predString = new stringStream(); |
9981851b4b8c
7009361: JSR 292 Invalid value on stack on solaris-sparc with -Xcomp
never
parents:
9101
diff
changeset
|
761 |
predString->print("rc_predicate "); |
9981851b4b8c
7009361: JSR 292 Invalid value on stack on solaris-sparc with -Xcomp
never
parents:
9101
diff
changeset
|
762 |
} |
9101 | 763 |
|
45965 | 764 |
overflow = false; |
765 |
Node* max_idx_expr = NULL; |
|
766 |
const TypeInt* idx_type = TypeInt::INT; |
|
767 |
if ((stride > 0) == (scale > 0) == upper) { |
|
51078 | 768 |
guarantee(limit != NULL, "sanity"); |
45965 | 769 |
if (TraceLoopPredicate) { |
46735 | 770 |
if (limit->is_Con()) { |
771 |
predString->print("(%d ", con_limit); |
|
772 |
} else { |
|
773 |
predString->print("(limit "); |
|
774 |
} |
|
45965 | 775 |
predString->print("- %d) ", stride); |
776 |
} |
|
777 |
// Check if (limit - stride) may overflow |
|
778 |
const TypeInt* limit_type = _igvn.type(limit)->isa_int(); |
|
779 |
jint limit_lo = limit_type->_lo; |
|
780 |
jint limit_hi = limit_type->_hi; |
|
781 |
if ((stride > 0 && (java_subtract(limit_lo, stride) < limit_lo)) || |
|
782 |
(stride < 0 && (java_subtract(limit_hi, stride) > limit_hi))) { |
|
783 |
// No overflow possible |
|
784 |
ConINode* con_stride = _igvn.intcon(stride); |
|
785 |
set_ctrl(con_stride, C->root()); |
|
786 |
max_idx_expr = new SubINode(limit, con_stride); |
|
787 |
idx_type = TypeInt::make(limit_lo - stride, limit_hi - stride, limit_type->_widen); |
|
788 |
} else { |
|
789 |
// May overflow |
|
790 |
overflow = true; |
|
791 |
limit = new ConvI2LNode(limit); |
|
792 |
register_new_node(limit, ctrl); |
|
793 |
ConLNode* con_stride = _igvn.longcon(stride); |
|
794 |
set_ctrl(con_stride, C->root()); |
|
795 |
max_idx_expr = new SubLNode(limit, con_stride); |
|
796 |
} |
|
37287
c2660335bf81
8072422: Cleanup: Remove some unused flags/code in loop optimizations
zmajo
parents:
35558
diff
changeset
|
797 |
register_new_node(max_idx_expr, ctrl); |
9101 | 798 |
} else { |
45965 | 799 |
if (TraceLoopPredicate) { |
46735 | 800 |
if (init->is_Con()) { |
801 |
predString->print("%d ", con_init); |
|
802 |
} else { |
|
803 |
predString->print("init "); |
|
804 |
} |
|
45965 | 805 |
} |
806 |
idx_type = _igvn.type(init)->isa_int(); |
|
807 |
max_idx_expr = init; |
|
9101 | 808 |
} |
809 |
||
810 |
if (scale != 1) { |
|
811 |
ConNode* con_scale = _igvn.intcon(scale); |
|
35558
9b3b2740c3ec
8146792: Predicate moved after partial peel may lead to broken graph
roland
parents:
34180
diff
changeset
|
812 |
set_ctrl(con_scale, C->root()); |
45965 | 813 |
if (TraceLoopPredicate) { |
814 |
predString->print("* %d ", scale); |
|
815 |
} |
|
816 |
// Check if (scale * max_idx_expr) may overflow |
|
817 |
const TypeInt* scale_type = TypeInt::make(scale); |
|
818 |
MulINode* mul = new MulINode(max_idx_expr, con_scale); |
|
819 |
idx_type = (TypeInt*)mul->mul_ring(idx_type, scale_type); |
|
820 |
if (overflow || TypeInt::INT->higher_equal(idx_type)) { |
|
821 |
// May overflow |
|
822 |
mul->destruct(); |
|
823 |
if (!overflow) { |
|
824 |
max_idx_expr = new ConvI2LNode(max_idx_expr); |
|
825 |
register_new_node(max_idx_expr, ctrl); |
|
826 |
} |
|
827 |
overflow = true; |
|
828 |
con_scale = _igvn.longcon(scale); |
|
829 |
set_ctrl(con_scale, C->root()); |
|
830 |
max_idx_expr = new MulLNode(max_idx_expr, con_scale); |
|
831 |
} else { |
|
832 |
// No overflow possible |
|
833 |
max_idx_expr = mul; |
|
834 |
} |
|
9101 | 835 |
register_new_node(max_idx_expr, ctrl); |
836 |
} |
|
837 |
||
45965 | 838 |
if (offset && (!offset->is_Con() || con_offset != 0)){ |
839 |
if (TraceLoopPredicate) { |
|
46630
75aa3e39d02c
8182299: Enable disabled clang warnings, build on OSX 10 + Xcode 8
jwilhelm
parents:
37287
diff
changeset
|
840 |
if (offset->is_Con()) { |
46735 | 841 |
predString->print("+ %d ", con_offset); |
46630
75aa3e39d02c
8182299: Enable disabled clang warnings, build on OSX 10 + Xcode 8
jwilhelm
parents:
37287
diff
changeset
|
842 |
} else { |
46735 | 843 |
predString->print("+ offset"); |
46630
75aa3e39d02c
8182299: Enable disabled clang warnings, build on OSX 10 + Xcode 8
jwilhelm
parents:
37287
diff
changeset
|
844 |
} |
45965 | 845 |
} |
846 |
// Check if (max_idx_expr + offset) may overflow |
|
847 |
const TypeInt* offset_type = _igvn.type(offset)->isa_int(); |
|
848 |
jint lo = java_add(idx_type->_lo, offset_type->_lo); |
|
849 |
jint hi = java_add(idx_type->_hi, offset_type->_hi); |
|
850 |
if (overflow || (lo > hi) || |
|
851 |
((idx_type->_lo & offset_type->_lo) < 0 && lo >= 0) || |
|
852 |
((~(idx_type->_hi | offset_type->_hi)) < 0 && hi < 0)) { |
|
853 |
// May overflow |
|
854 |
if (!overflow) { |
|
855 |
max_idx_expr = new ConvI2LNode(max_idx_expr); |
|
856 |
register_new_node(max_idx_expr, ctrl); |
|
857 |
} |
|
858 |
overflow = true; |
|
859 |
offset = new ConvI2LNode(offset); |
|
860 |
register_new_node(offset, ctrl); |
|
861 |
max_idx_expr = new AddLNode(max_idx_expr, offset); |
|
862 |
} else { |
|
863 |
// No overflow possible |
|
864 |
max_idx_expr = new AddINode(max_idx_expr, offset); |
|
865 |
} |
|
9101 | 866 |
register_new_node(max_idx_expr, ctrl); |
867 |
} |
|
868 |
||
45965 | 869 |
CmpNode* cmp = NULL; |
870 |
if (overflow) { |
|
871 |
// Integer expressions may overflow, do long comparison |
|
872 |
range = new ConvI2LNode(range); |
|
873 |
register_new_node(range, ctrl); |
|
49487
bde392011cd8
8193130: Bad graph when unrolled loop bounds conflicts with range checks
roland
parents:
48145
diff
changeset
|
874 |
cmp = new CmpULNode(max_idx_expr, range); |
45965 | 875 |
} else { |
876 |
cmp = new CmpUNode(max_idx_expr, range); |
|
877 |
} |
|
9101 | 878 |
register_new_node(cmp, ctrl); |
24923
9631f7d691dc
8034812: remove IDX_INIT macro hack in Node class
thartmann
parents:
24424
diff
changeset
|
879 |
BoolNode* bol = new BoolNode(cmp, BoolTest::lt); |
9101 | 880 |
register_new_node(bol, ctrl); |
881 |
||
9437
9981851b4b8c
7009361: JSR 292 Invalid value on stack on solaris-sparc with -Xcomp
never
parents:
9101
diff
changeset
|
882 |
if (TraceLoopPredicate) { |
9981851b4b8c
7009361: JSR 292 Invalid value on stack on solaris-sparc with -Xcomp
never
parents:
9101
diff
changeset
|
883 |
predString->print_cr("<u range"); |
24424
2658d7834c6e
8037816: Fix for 8036122 breaks build with Xcode5/clang
drchase
parents:
23528
diff
changeset
|
884 |
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
|
885 |
} |
9101 | 886 |
return bol; |
887 |
} |
|
888 |
||
50623
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
889 |
// 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
|
890 |
// 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
|
891 |
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
|
892 |
if (!UseProfiledLoopPredicate) { |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
893 |
return false; |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
894 |
} |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
895 |
|
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
896 |
if (predicate_proj == NULL) { |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
897 |
return false; |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
898 |
} |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
899 |
|
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
900 |
LoopNode* head = loop->_head->as_Loop(); |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
901 |
bool follow_branches = true; |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
902 |
IdealLoopTree* l = loop->_child; |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
903 |
// 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
|
904 |
while (l != NULL && follow_branches) { |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
905 |
IdealLoopTree* child = l; |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
906 |
if (child->_child != NULL && |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
907 |
child->_head->is_OuterStripMinedLoop()) { |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
908 |
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
|
909 |
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
|
910 |
child = child->_child; |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
911 |
} |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
912 |
if (child->_child != NULL || child->_irreducible) { |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
913 |
follow_branches = false; |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
914 |
} |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
915 |
l = l->_next; |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
916 |
} |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
917 |
if (follow_branches) { |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
918 |
loop->compute_profile_trip_cnt(this); |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
919 |
if (head->is_profile_trip_failed()) { |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
920 |
follow_branches = false; |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
921 |
} else { |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
922 |
loop_trip_cnt = head->profile_trip_cnt(); |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
923 |
if (head->is_CountedLoop()) { |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
924 |
CountedLoopNode* cl = head->as_CountedLoop(); |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
925 |
if (cl->phi() != NULL) { |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
926 |
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
|
927 |
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
|
928 |
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
|
929 |
loop_trip_cnt = worst_case_trip_cnt; |
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 |
} |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
932 |
} |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
933 |
} |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
934 |
} |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
935 |
return follow_branches; |
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 |
|
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
938 |
// 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
|
939 |
// dominating CFG node |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
940 |
class PathFrequency { |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
941 |
private: |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
942 |
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
|
943 |
Node_Stack _stack; |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
944 |
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
|
945 |
GrowableArray<float> _freqs; // cache frequencies |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
946 |
PhaseIdealLoop* _phase; |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
947 |
|
52490
61915e1458bc
8205574: Loop predication "assert(f <= 1 && f >= 0) failed Incorrect frequency"
roland
parents:
52414
diff
changeset
|
948 |
void set_rounding(int mode) { |
61915e1458bc
8205574: Loop predication "assert(f <= 1 && f >= 0) failed Incorrect frequency"
roland
parents:
52414
diff
changeset
|
949 |
// fesetround is broken on windows |
61915e1458bc
8205574: Loop predication "assert(f <= 1 && f >= 0) failed Incorrect frequency"
roland
parents:
52414
diff
changeset
|
950 |
NOT_WINDOWS(fesetround(mode);) |
61915e1458bc
8205574: Loop predication "assert(f <= 1 && f >= 0) failed Incorrect frequency"
roland
parents:
52414
diff
changeset
|
951 |
} |
61915e1458bc
8205574: Loop predication "assert(f <= 1 && f >= 0) failed Incorrect frequency"
roland
parents:
52414
diff
changeset
|
952 |
|
61915e1458bc
8205574: Loop predication "assert(f <= 1 && f >= 0) failed Incorrect frequency"
roland
parents:
52414
diff
changeset
|
953 |
void check_frequency(float f) { |
61915e1458bc
8205574: Loop predication "assert(f <= 1 && f >= 0) failed Incorrect frequency"
roland
parents:
52414
diff
changeset
|
954 |
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
|
955 |
} |
61915e1458bc
8205574: Loop predication "assert(f <= 1 && f >= 0) failed Incorrect frequency"
roland
parents:
52414
diff
changeset
|
956 |
|
50623
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
957 |
public: |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
958 |
PathFrequency(Node* dom, PhaseIdealLoop* phase) |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
959 |
: _dom(dom), _stack(0), _phase(phase) { |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
960 |
} |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
961 |
|
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
962 |
float to(Node* n) { |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
963 |
// 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
|
964 |
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
|
965 |
IdealLoopTree* loop = _phase->get_loop(_dom); |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
966 |
Node* c = n; |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
967 |
for (;;) { |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
968 |
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
|
969 |
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
|
970 |
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
|
971 |
Node* prev = c; |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
972 |
while (_stack.size() > 0 && prev == c) { |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
973 |
Node* n = _stack.node(); |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
974 |
if (!n->is_Region()) { |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
975 |
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
|
976 |
// 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
|
977 |
// 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
|
978 |
// times each loop exit was taken |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
979 |
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
|
980 |
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
|
981 |
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
|
982 |
if (inner_head->is_OuterStripMinedLoop()) { |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
983 |
inner_head->verify_strip_mined(1); |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
984 |
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
|
985 |
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
|
986 |
} |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
987 |
inner_loop = inner_loop->_child; |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
988 |
inner_head = inner_loop->_head->as_Loop(); |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
989 |
inner_head->verify_strip_mined(1); |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
990 |
} |
52490
61915e1458bc
8205574: Loop predication "assert(f <= 1 && f >= 0) failed Incorrect frequency"
roland
parents:
52414
diff
changeset
|
991 |
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
|
992 |
float loop_exit_cnt = 0.0f; |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
993 |
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
|
994 |
Node *n = inner_loop->_body[i]; |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
995 |
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
|
996 |
loop_exit_cnt += c; |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
997 |
} |
52490
61915e1458bc
8205574: Loop predication "assert(f <= 1 && f >= 0) failed Incorrect frequency"
roland
parents:
52414
diff
changeset
|
998 |
set_rounding(FE_TOWARDZERO); |
50623
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
999 |
float cnt = -1; |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1000 |
if (n->in(0)->is_If()) { |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1001 |
IfNode* iff = n->in(0)->as_If(); |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1002 |
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
|
1003 |
if (n->Opcode() == Op_IfFalse) { |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1004 |
p = 1 - p; |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1005 |
} |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1006 |
if (p > PROB_MIN) { |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1007 |
cnt = p * iff->_fcnt; |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1008 |
} else { |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1009 |
cnt = 0; |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1010 |
} |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1011 |
} else { |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1012 |
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
|
1013 |
JumpNode* jmp = n->in(0)->as_Jump(); |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1014 |
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
|
1015 |
cnt = p * jmp->_fcnt; |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1016 |
} |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1017 |
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
|
1018 |
check_frequency(this_exit_f); |
50623
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1019 |
f = f * this_exit_f; |
52490
61915e1458bc
8205574: Loop predication "assert(f <= 1 && f >= 0) failed Incorrect frequency"
roland
parents:
52414
diff
changeset
|
1020 |
check_frequency(f); |
50623
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1021 |
} else { |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1022 |
float p = -1; |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1023 |
if (n->in(0)->is_If()) { |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1024 |
p = n->in(0)->as_If()->_prob; |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1025 |
if (n->Opcode() == Op_IfFalse) { |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1026 |
p = 1 - p; |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1027 |
} |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1028 |
} else { |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1029 |
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
|
1030 |
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
|
1031 |
} |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1032 |
f = f * p; |
52490
61915e1458bc
8205574: Loop predication "assert(f <= 1 && f >= 0) failed Incorrect frequency"
roland
parents:
52414
diff
changeset
|
1033 |
check_frequency(f); |
50623
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1034 |
} |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1035 |
_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
|
1036 |
_stack.pop(); |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1037 |
} else { |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1038 |
float prev_f = _freqs_stack.pop(); |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1039 |
float new_f = f; |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1040 |
f = new_f + prev_f; |
52490
61915e1458bc
8205574: Loop predication "assert(f <= 1 && f >= 0) failed Incorrect frequency"
roland
parents:
52414
diff
changeset
|
1041 |
check_frequency(f); |
50623
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1042 |
uint i = _stack.index(); |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1043 |
if (i < n->req()) { |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1044 |
c = n->in(i); |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1045 |
_stack.set_index(i+1); |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1046 |
_freqs_stack.push(f); |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1047 |
} else { |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1048 |
_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
|
1049 |
_stack.pop(); |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1050 |
} |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1051 |
} |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1052 |
} |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1053 |
if (_stack.size() == 0) { |
52490
61915e1458bc
8205574: Loop predication "assert(f <= 1 && f >= 0) failed Incorrect frequency"
roland
parents:
52414
diff
changeset
|
1054 |
set_rounding(FE_TONEAREST); |
61915e1458bc
8205574: Loop predication "assert(f <= 1 && f >= 0) failed Incorrect frequency"
roland
parents:
52414
diff
changeset
|
1055 |
check_frequency(f); |
50623
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1056 |
return f; |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1057 |
} |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1058 |
} else if (c->is_Loop()) { |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1059 |
ShouldNotReachHere(); |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1060 |
c = c->in(LoopNode::EntryControl); |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1061 |
} else if (c->is_Region()) { |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1062 |
_freqs_stack.push(0); |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1063 |
_stack.push(c, 2); |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1064 |
c = c->in(1); |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1065 |
} else { |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1066 |
if (c->is_IfProj()) { |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1067 |
IfNode* iff = c->in(0)->as_If(); |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1068 |
if (iff->_prob == PROB_UNKNOWN) { |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1069 |
// assume never taken |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1070 |
_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
|
1071 |
} 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
|
1072 |
if (iff->_fcnt == COUNT_UNKNOWN) { |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1073 |
// assume never taken |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1074 |
_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
|
1075 |
} else { |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1076 |
// skip over loop |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1077 |
_stack.push(c, 1); |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1078 |
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
|
1079 |
} |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1080 |
} else { |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1081 |
_stack.push(c, 1); |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1082 |
c = iff; |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1083 |
} |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1084 |
} else if (c->is_JumpProj()) { |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1085 |
JumpNode* jmp = c->in(0)->as_Jump(); |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1086 |
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
|
1087 |
if (jmp->_fcnt == COUNT_UNKNOWN) { |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1088 |
// assume never taken |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1089 |
_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
|
1090 |
} else { |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1091 |
// skip over loop |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1092 |
_stack.push(c, 1); |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1093 |
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
|
1094 |
} |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1095 |
} else { |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1096 |
_stack.push(c, 1); |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1097 |
c = jmp; |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1098 |
} |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1099 |
} else if (c->Opcode() == Op_CatchProj && |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1100 |
c->in(0)->Opcode() == Op_Catch && |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1101 |
c->in(0)->in(0)->is_Proj() && |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1102 |
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
|
1103 |
// assume exceptions are never thrown |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1104 |
uint con = c->as_Proj()->_con; |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1105 |
if (con == CatchProjNode::fall_through_index) { |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1106 |
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
|
1107 |
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
|
1108 |
_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
|
1109 |
} else { |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1110 |
c = call; |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1111 |
} |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1112 |
} else { |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1113 |
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
|
1114 |
_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
|
1115 |
} |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1116 |
} 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
|
1117 |
ShouldNotReachHere(); |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1118 |
} else { |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1119 |
c = c->in(0); |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1120 |
} |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1121 |
} |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1122 |
} |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1123 |
ShouldNotReachHere(); |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1124 |
return -1; |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1125 |
} |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1126 |
}; |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1127 |
|
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1128 |
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
|
1129 |
PathFrequency& pf, Node_Stack& stack, VectorSet& seen, |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1130 |
Node_List& if_proj_list) { |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1131 |
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
|
1132 |
Node* tail = loop->tail(); |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1133 |
stack.push(n, 1); |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1134 |
do { |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1135 |
Node* c = stack.node(); |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1136 |
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
|
1137 |
uint i = stack.index(); |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1138 |
|
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1139 |
if (i < c->req()) { |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1140 |
stack.set_index(i+1); |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1141 |
Node* in = c->in(i); |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1142 |
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
|
1143 |
IdealLoopTree* in_loop = get_loop(in); |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1144 |
if (in_loop != loop) { |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1145 |
in = in_loop->_head->in(LoopNode::EntryControl); |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1146 |
} else if (in->is_Region()) { |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1147 |
stack.push(in, 1); |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1148 |
break; |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1149 |
} else if (in->is_IfProj() && |
50923
c98bf5aa35c5
8205515: assert(opcode == Op_RangeCheck) failed: no other if variant here
roland
parents:
50632
diff
changeset
|
1150 |
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
|
1151 |
(in->in(0)->Opcode() == Op_If || |
c98bf5aa35c5
8205515: assert(opcode == Op_RangeCheck) failed: no other if variant here
roland
parents:
50632
diff
changeset
|
1152 |
in->in(0)->Opcode() == Op_RangeCheck)) { |
50623
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1153 |
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
|
1154 |
stack.push(in, 1); |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1155 |
} |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1156 |
in = in->in(0); |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1157 |
} else { |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1158 |
in = in->in(0); |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1159 |
} |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1160 |
} |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1161 |
} else { |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1162 |
if (c->is_IfProj()) { |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1163 |
if_proj_list.push(c); |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1164 |
} |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1165 |
stack.pop(); |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1166 |
} |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1167 |
|
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1168 |
} while (stack.size() > 0); |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1169 |
} |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1170 |
|
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1171 |
|
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1172 |
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
|
1173 |
CountedLoopNode *cl, ConNode* zero, Invariance& invar, |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1174 |
Deoptimization::DeoptReason reason) { |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1175 |
// 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
|
1176 |
ProjNode* new_predicate_proj = NULL; |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1177 |
IfNode* iff = proj->in(0)->as_If(); |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1178 |
Node* test = iff->in(1); |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1179 |
if (!test->is_Bool()){ //Conv2B, ... |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1180 |
return false; |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1181 |
} |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1182 |
BoolNode* bol = test->as_Bool(); |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1183 |
if (invar.is_invariant(bol)) { |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1184 |
// Invariant test |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1185 |
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
|
1186 |
reason, |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1187 |
iff->Opcode()); |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1188 |
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
|
1189 |
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
|
1190 |
|
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1191 |
// Negate test if necessary |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1192 |
bool negated = false; |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1193 |
if (proj->_con != predicate_proj->_con) { |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1194 |
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
|
1195 |
register_new_node(new_predicate_bol, ctrl); |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1196 |
negated = true; |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1197 |
} |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1198 |
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
|
1199 |
_igvn.hash_delete(new_predicate_iff); |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1200 |
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
|
1201 |
#ifndef PRODUCT |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1202 |
if (TraceLoopPredicate) { |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1203 |
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
|
1204 |
loop->dump_head(); |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1205 |
} else if (TraceLoopOpts) { |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1206 |
tty->print("Predicate IC "); |
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 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
|
1211 |
// Range check for counted loops |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1212 |
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
|
1213 |
Node* idx = cmp->in(1); |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1214 |
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
|
1215 |
Node* rng = cmp->in(2); |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1216 |
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
|
1217 |
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
|
1218 |
int scale = 1; |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1219 |
Node* offset = zero; |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1220 |
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
|
1221 |
assert(ok, "must be index expression"); |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1222 |
|
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1223 |
Node* init = cl->init_trip(); |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1224 |
// Limit is not exact. |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1225 |
// Calculate exact limit here. |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1226 |
// Note, counted loop's test is '<' or '>'. |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1227 |
Node* limit = exact_limit(loop); |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1228 |
int stride = cl->stride()->get_int(); |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1229 |
|
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1230 |
// 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
|
1231 |
// 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
|
1232 |
// 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
|
1233 |
// their declared control. |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1234 |
|
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1235 |
// 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
|
1236 |
// 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
|
1237 |
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
|
1238 |
rng = invar.clone(rng, ctrl); |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1239 |
if (offset && offset != zero) { |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1240 |
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
|
1241 |
offset = invar.clone(offset, ctrl); |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1242 |
} |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1243 |
// 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
|
1244 |
bool overflow = false; |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1245 |
|
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1246 |
// Test the lower bound |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1247 |
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
|
1248 |
// Negate test if necessary |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1249 |
bool negated = false; |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1250 |
if (proj->_con != predicate_proj->_con) { |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1251 |
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
|
1252 |
register_new_node(lower_bound_bol, ctrl); |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1253 |
negated = true; |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1254 |
} |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1255 |
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
|
1256 |
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
|
1257 |
_igvn.hash_delete(lower_bound_iff); |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1258 |
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
|
1259 |
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
|
1260 |
|
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1261 |
// Test the upper bound |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1262 |
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
|
1263 |
negated = false; |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1264 |
if (proj->_con != predicate_proj->_con) { |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1265 |
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
|
1266 |
register_new_node(upper_bound_bol, ctrl); |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1267 |
negated = true; |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1268 |
} |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1269 |
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
|
1270 |
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
|
1271 |
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
|
1272 |
_igvn.hash_delete(upper_bound_iff); |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1273 |
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
|
1274 |
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
|
1275 |
|
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1276 |
// 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
|
1277 |
// 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
|
1278 |
new_predicate_proj = upper_bound_proj; |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1279 |
|
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1280 |
if (iff->is_RangeCheck()) { |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1281 |
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
|
1282 |
} |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1283 |
|
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1284 |
#ifndef PRODUCT |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1285 |
if (TraceLoopOpts && !TraceLoopPredicate) { |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1286 |
tty->print("Predicate RC "); |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1287 |
loop->dump_head(); |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1288 |
} |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1289 |
#endif |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1290 |
} else { |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1291 |
// 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
|
1292 |
// with uncommon trap. |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1293 |
return false; |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1294 |
} |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1295 |
assert(new_predicate_proj != NULL, "sanity"); |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1296 |
// 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
|
1297 |
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
|
1298 |
|
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1299 |
// 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
|
1300 |
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
|
1301 |
|
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1302 |
C->set_major_progress(); |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1303 |
return true; |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1304 |
} |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1305 |
|
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1306 |
|
49487
bde392011cd8
8193130: Bad graph when unrolled loop bounds conflicts with range checks
roland
parents:
48145
diff
changeset
|
1307 |
// 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
|
1308 |
// 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
|
1309 |
// 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
|
1310 |
// 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
|
1311 |
// updated by PhaseIdealLoop::clone_skeleton_predicate()). |
49487
bde392011cd8
8193130: Bad graph when unrolled loop bounds conflicts with range checks
roland
parents:
48145
diff
changeset
|
1312 |
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
|
1313 |
ProjNode* proj, ProjNode *predicate_proj, |
bde392011cd8
8193130: Bad graph when unrolled loop bounds conflicts with range checks
roland
parents:
48145
diff
changeset
|
1314 |
ProjNode* upper_bound_proj, |
bde392011cd8
8193130: Bad graph when unrolled loop bounds conflicts with range checks
roland
parents:
48145
diff
changeset
|
1315 |
int scale, Node* offset, |
bde392011cd8
8193130: Bad graph when unrolled loop bounds conflicts with range checks
roland
parents:
48145
diff
changeset
|
1316 |
Node* init, Node* limit, jint stride, |
50623
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1317 |
Node* rng, bool &overflow, |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1318 |
Deoptimization::DeoptReason reason) { |
49487
bde392011cd8
8193130: Bad graph when unrolled loop bounds conflicts with range checks
roland
parents:
48145
diff
changeset
|
1319 |
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
|
1320 |
Node* opaque_init = new Opaque1Node(C, init); |
bde392011cd8
8193130: Bad graph when unrolled loop bounds conflicts with range checks
roland
parents:
48145
diff
changeset
|
1321 |
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
|
1322 |
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
|
1323 |
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
|
1324 |
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
|
1325 |
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
|
1326 |
_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
|
1327 |
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
|
1328 |
return new_proj; |
bde392011cd8
8193130: Bad graph when unrolled loop bounds conflicts with range checks
roland
parents:
48145
diff
changeset
|
1329 |
} |
bde392011cd8
8193130: Bad graph when unrolled loop bounds conflicts with range checks
roland
parents:
48145
diff
changeset
|
1330 |
|
9101 | 1331 |
//------------------------------ loop_predication_impl-------------------------- |
1332 |
// Insert loop predicates for null checks and range checks |
|
1333 |
bool PhaseIdealLoop::loop_predication_impl(IdealLoopTree *loop) { |
|
1334 |
if (!UseLoopPredicate) return false; |
|
1335 |
||
1336 |
if (!loop->_head->is_Loop()) { |
|
1337 |
// Could be a simple region when irreducible loops are present. |
|
1338 |
return false; |
|
1339 |
} |
|
9446 | 1340 |
LoopNode* head = loop->_head->as_Loop(); |
9101 | 1341 |
|
9446 | 1342 |
if (head->unique_ctrl_out()->Opcode() == Op_NeverBranch) { |
9101 | 1343 |
// do nothing for infinite loops |
1344 |
return false; |
|
1345 |
} |
|
1346 |
||
48145 | 1347 |
if (head->is_OuterStripMinedLoop()) { |
1348 |
return false; |
|
1349 |
} |
|
1350 |
||
9101 | 1351 |
CountedLoopNode *cl = NULL; |
10263
fa58671dde31
7077439: Possible reference through NULL in loopPredicate.cpp:726
kvn
parents:
10258
diff
changeset
|
1352 |
if (head->is_valid_counted_loop()) { |
9446 | 1353 |
cl = head->as_CountedLoop(); |
9101 | 1354 |
// do nothing for iteration-splitted loops |
1355 |
if (!cl->is_normal_loop()) return false; |
|
10253
35b975b1e8f3
7070134: Hotspot crashes with sigsegv from PorterStemmer
kvn
parents:
9446
diff
changeset
|
1356 |
// Avoid RCE if Counted loop's test is '!='. |
35b975b1e8f3
7070134: Hotspot crashes with sigsegv from PorterStemmer
kvn
parents:
9446
diff
changeset
|
1357 |
BoolTest::mask bt = cl->loopexit()->test_trip(); |
35b975b1e8f3
7070134: Hotspot crashes with sigsegv from PorterStemmer
kvn
parents:
9446
diff
changeset
|
1358 |
if (bt != BoolTest::lt && bt != BoolTest::gt) |
35b975b1e8f3
7070134: Hotspot crashes with sigsegv from PorterStemmer
kvn
parents:
9446
diff
changeset
|
1359 |
cl = NULL; |
9101 | 1360 |
} |
1361 |
||
48145 | 1362 |
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
|
1363 |
ProjNode *loop_limit_proj = NULL; |
9446 | 1364 |
ProjNode *predicate_proj = NULL; |
50623
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1365 |
ProjNode *profile_predicate_proj = NULL; |
9446 | 1366 |
// 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
|
1367 |
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
|
1368 |
if (loop_limit_proj != NULL) { |
52608
61241fc2217a
8211451: ~2.5% regression on compression benchmark starting with 12-b11
roland
parents:
52490
diff
changeset
|
1369 |
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
|
1370 |
} |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1371 |
bool has_profile_predicates = false; |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1372 |
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
|
1373 |
if (profile_predicate_proj != NULL) { |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1374 |
Node* n = skip_loop_predicates(entry); |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1375 |
// 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
|
1376 |
// block |
50923
c98bf5aa35c5
8205515: assert(opcode == Op_RangeCheck) failed: no other if variant here
roland
parents:
50632
diff
changeset
|
1377 |
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
|
1378 |
has_profile_predicates = true; |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1379 |
} |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1380 |
entry = n; |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1381 |
} |
9446 | 1382 |
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
|
1383 |
|
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1384 |
float loop_trip_cnt = -1; |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1385 |
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
|
1386 |
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
|
1387 |
|
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1388 |
if (predicate_proj == NULL && !follow_branches) { |
9101 | 1389 |
#ifndef PRODUCT |
1390 |
if (TraceLoopPredicate) { |
|
1391 |
tty->print("missing predicate:"); |
|
1392 |
loop->dump_head(); |
|
9446 | 1393 |
head->dump(1); |
9101 | 1394 |
} |
1395 |
#endif |
|
1396 |
return false; |
|
1397 |
} |
|
1398 |
ConNode* zero = _igvn.intcon(0); |
|
1399 |
set_ctrl(zero, C->root()); |
|
1400 |
||
1401 |
ResourceArea *area = Thread::current()->resource_area(); |
|
1402 |
Invariance invar(area, loop); |
|
1403 |
||
1404 |
// Create list of if-projs such that a newer proj dominates all older |
|
1405 |
// projs in the list, and they all dominate loop->tail() |
|
1406 |
Node_List if_proj_list(area); |
|
50623
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1407 |
Node_List regions(area); |
9101 | 1408 |
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
|
1409 |
|
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1410 |
|
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1411 |
Node_List controls(area); |
9101 | 1412 |
while (current_proj != head) { |
1413 |
if (loop == get_loop(current_proj) && // still in the loop ? |
|
1414 |
current_proj->is_Proj() && // is a projection ? |
|
34164
a9e6034d7707
8137168: Replace IfNode with a new RangeCheckNode for range checks
roland
parents:
32733
diff
changeset
|
1415 |
(current_proj->in(0)->Opcode() == Op_If || |
a9e6034d7707
8137168: Replace IfNode with a new RangeCheckNode for range checks
roland
parents:
32733
diff
changeset
|
1416 |
current_proj->in(0)->Opcode() == Op_RangeCheck)) { // is a if projection ? |
9101 | 1417 |
if_proj_list.push(current_proj); |
1418 |
} |
|
50623
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1419 |
if (follow_branches && |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1420 |
current_proj->Opcode() == Op_Region && |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1421 |
loop == get_loop(current_proj)) { |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1422 |
regions.push(current_proj); |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1423 |
} |
9101 | 1424 |
current_proj = idom(current_proj); |
1425 |
} |
|
1426 |
||
1427 |
bool hoisted = false; // true if at least one proj is promoted |
|
1428 |
||
50623
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1429 |
if (!has_profile_predicates) { |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1430 |
while (if_proj_list.size() > 0) { |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1431 |
Node* n = if_proj_list.pop(); |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1432 |
|
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1433 |
ProjNode* proj = n->as_Proj(); |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1434 |
IfNode* iff = proj->in(0)->as_If(); |
9101 | 1435 |
|
50623
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1436 |
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
|
1437 |
if (call == NULL) { |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1438 |
if (loop->is_loop_exit(iff)) { |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1439 |
// 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
|
1440 |
// 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
|
1441 |
break; |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1442 |
} else { |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1443 |
// 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
|
1444 |
// (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
|
1445 |
// 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
|
1446 |
// 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
|
1447 |
// can safely continue. |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1448 |
// (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
|
1449 |
// 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
|
1450 |
continue; |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1451 |
} |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1452 |
} |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1453 |
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
|
1454 |
if (reason == Deoptimization::Reason_predicate) { |
9101 | 1455 |
break; |
50623
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1456 |
} |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1457 |
|
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1458 |
if (predicate_proj != NULL) { |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1459 |
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
|
1460 |
} |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1461 |
} // end while |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1462 |
} |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1463 |
|
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1464 |
Node_List if_proj_list_freq(area); |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1465 |
if (follow_branches) { |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1466 |
PathFrequency pf(loop->_head, this); |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1467 |
|
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1468 |
// 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
|
1469 |
// 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
|
1470 |
while (if_proj_list.size() > 0) { |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1471 |
Node* proj = if_proj_list.pop(); |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1472 |
float f = pf.to(proj); |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1473 |
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
|
1474 |
f * loop_trip_cnt >= 1) { |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1475 |
hoisted = loop_predication_impl_helper(loop, proj->as_Proj(), profile_predicate_proj, cl, zero, invar, Deoptimization::Reason_profile_predicate) | hoisted; |
9101 | 1476 |
} |
1477 |
} |
|
1478 |
||
50623
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1479 |
// And look into all branches |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1480 |
Node_Stack stack(0); |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1481 |
VectorSet seen(Thread::current()->resource_area()); |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1482 |
while (regions.size() > 0) { |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1483 |
Node* c = regions.pop(); |
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1484 |
loop_predication_follow_branches(c, loop, loop_trip_cnt, pf, stack, seen, if_proj_list_freq); |
9101 | 1485 |
} |
1486 |
||
50623
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1487 |
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
|
1488 |
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
|
1489 |
hoisted = loop_predication_impl_helper(loop, proj, profile_predicate_proj, cl, zero, invar, Deoptimization::Reason_profile_predicate) | hoisted; |
9101 | 1490 |
} |
50623
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50561
diff
changeset
|
1491 |
} |
9101 | 1492 |
|
1493 |
#ifndef PRODUCT |
|
1494 |
// report that the loop predication has been actually performed |
|
1495 |
// for this loop |
|
1496 |
if (TraceLoopPredicate && hoisted) { |
|
1497 |
tty->print("Loop Predication Performed:"); |
|
1498 |
loop->dump_head(); |
|
1499 |
} |
|
1500 |
#endif |
|
1501 |
||
48145 | 1502 |
head->verify_strip_mined(1); |
1503 |
||
9101 | 1504 |
return hoisted; |
1505 |
} |
|
1506 |
||
1507 |
//------------------------------loop_predication-------------------------------- |
|
1508 |
// driver routine for loop predication optimization |
|
1509 |
bool IdealLoopTree::loop_predication( PhaseIdealLoop *phase) { |
|
1510 |
bool hoisted = false; |
|
1511 |
// Recursively promote predicates |
|
1512 |
if (_child) { |
|
1513 |
hoisted = _child->loop_predication( phase); |
|
1514 |
} |
|
1515 |
||
1516 |
// self |
|
1517 |
if (!_irreducible && !tail()->is_top()) { |
|
1518 |
hoisted |= phase->loop_predication_impl(this); |
|
1519 |
} |
|
1520 |
||
1521 |
if (_next) { //sibling |
|
1522 |
hoisted |= _next->loop_predication( phase); |
|
1523 |
} |
|
1524 |
||
1525 |
return hoisted; |
|
1526 |
} |