annotate src/share/vm/opto/loopTransform.cpp @ 3333:ae93231c7a1f

7039652: Performance regression after 7004547 changes Summary: Use unrolled_count() to limit unrolling and use the stride check only for initial stride value. Reviewed-by: never
author kvn
date Thu, 28 Apr 2011 16:40:23 -0700
parents 6c97c830fb6f
children bad7ecd0b6ed
Ignore whitespace changes - Everywhere: Within whitespace: At end of lines:
rev   line source
0
a61af66fc99e Initial load
duke
parents:
diff changeset
1 /*
2426
1d1603768966 7010070: Update all 2010 Oracle-changed OpenJDK files to have the proper copyright dates - second pass
trims
parents: 2417
diff changeset
2 * Copyright (c) 2000, 2011, Oracle and/or its affiliates. All rights reserved.
0
a61af66fc99e Initial load
duke
parents:
diff changeset
3 * DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS FILE HEADER.
a61af66fc99e Initial load
duke
parents:
diff changeset
4 *
a61af66fc99e Initial load
duke
parents:
diff changeset
5 * This code is free software; you can redistribute it and/or modify it
a61af66fc99e Initial load
duke
parents:
diff changeset
6 * under the terms of the GNU General Public License version 2 only, as
a61af66fc99e Initial load
duke
parents:
diff changeset
7 * published by the Free Software Foundation.
a61af66fc99e Initial load
duke
parents:
diff changeset
8 *
a61af66fc99e Initial load
duke
parents:
diff changeset
9 * This code is distributed in the hope that it will be useful, but WITHOUT
a61af66fc99e Initial load
duke
parents:
diff changeset
10 * ANY WARRANTY; without even the implied warranty of MERCHANTABILITY or
a61af66fc99e Initial load
duke
parents:
diff changeset
11 * FITNESS FOR A PARTICULAR PURPOSE. See the GNU General Public License
a61af66fc99e Initial load
duke
parents:
diff changeset
12 * version 2 for more details (a copy is included in the LICENSE file that
a61af66fc99e Initial load
duke
parents:
diff changeset
13 * accompanied this code).
a61af66fc99e Initial load
duke
parents:
diff changeset
14 *
a61af66fc99e Initial load
duke
parents:
diff changeset
15 * You should have received a copy of the GNU General Public License version
a61af66fc99e Initial load
duke
parents:
diff changeset
16 * 2 along with this work; if not, write to the Free Software Foundation,
a61af66fc99e Initial load
duke
parents:
diff changeset
17 * Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301 USA.
a61af66fc99e Initial load
duke
parents:
diff changeset
18 *
1552
c18cbe5936b8 6941466: Oracle rebranding changes for Hotspot repositories
trims
parents: 1303
diff changeset
19 * Please contact Oracle, 500 Oracle Parkway, Redwood Shores, CA 94065 USA
c18cbe5936b8 6941466: Oracle rebranding changes for Hotspot repositories
trims
parents: 1303
diff changeset
20 * or visit www.oracle.com if you need additional information or have any
c18cbe5936b8 6941466: Oracle rebranding changes for Hotspot repositories
trims
parents: 1303
diff changeset
21 * questions.
0
a61af66fc99e Initial load
duke
parents:
diff changeset
22 *
a61af66fc99e Initial load
duke
parents:
diff changeset
23 */
a61af66fc99e Initial load
duke
parents:
diff changeset
24
1972
f95d63e2154a 6989984: Use standard include model for Hospot
stefank
parents: 1844
diff changeset
25 #include "precompiled.hpp"
f95d63e2154a 6989984: Use standard include model for Hospot
stefank
parents: 1844
diff changeset
26 #include "compiler/compileLog.hpp"
f95d63e2154a 6989984: Use standard include model for Hospot
stefank
parents: 1844
diff changeset
27 #include "memory/allocation.inline.hpp"
f95d63e2154a 6989984: Use standard include model for Hospot
stefank
parents: 1844
diff changeset
28 #include "opto/addnode.hpp"
f95d63e2154a 6989984: Use standard include model for Hospot
stefank
parents: 1844
diff changeset
29 #include "opto/callnode.hpp"
f95d63e2154a 6989984: Use standard include model for Hospot
stefank
parents: 1844
diff changeset
30 #include "opto/connode.hpp"
f95d63e2154a 6989984: Use standard include model for Hospot
stefank
parents: 1844
diff changeset
31 #include "opto/divnode.hpp"
f95d63e2154a 6989984: Use standard include model for Hospot
stefank
parents: 1844
diff changeset
32 #include "opto/loopnode.hpp"
f95d63e2154a 6989984: Use standard include model for Hospot
stefank
parents: 1844
diff changeset
33 #include "opto/mulnode.hpp"
f95d63e2154a 6989984: Use standard include model for Hospot
stefank
parents: 1844
diff changeset
34 #include "opto/rootnode.hpp"
f95d63e2154a 6989984: Use standard include model for Hospot
stefank
parents: 1844
diff changeset
35 #include "opto/runtime.hpp"
f95d63e2154a 6989984: Use standard include model for Hospot
stefank
parents: 1844
diff changeset
36 #include "opto/subnode.hpp"
0
a61af66fc99e Initial load
duke
parents:
diff changeset
37
a61af66fc99e Initial load
duke
parents:
diff changeset
38 //------------------------------is_loop_exit-----------------------------------
a61af66fc99e Initial load
duke
parents:
diff changeset
39 // Given an IfNode, return the loop-exiting projection or NULL if both
a61af66fc99e Initial load
duke
parents:
diff changeset
40 // arms remain in the loop.
a61af66fc99e Initial load
duke
parents:
diff changeset
41 Node *IdealLoopTree::is_loop_exit(Node *iff) const {
a61af66fc99e Initial load
duke
parents:
diff changeset
42 if( iff->outcnt() != 2 ) return NULL; // Ignore partially dead tests
a61af66fc99e Initial load
duke
parents:
diff changeset
43 PhaseIdealLoop *phase = _phase;
a61af66fc99e Initial load
duke
parents:
diff changeset
44 // Test is an IfNode, has 2 projections. If BOTH are in the loop
a61af66fc99e Initial load
duke
parents:
diff changeset
45 // we need loop unswitching instead of peeling.
a61af66fc99e Initial load
duke
parents:
diff changeset
46 if( !is_member(phase->get_loop( iff->raw_out(0) )) )
a61af66fc99e Initial load
duke
parents:
diff changeset
47 return iff->raw_out(0);
a61af66fc99e Initial load
duke
parents:
diff changeset
48 if( !is_member(phase->get_loop( iff->raw_out(1) )) )
a61af66fc99e Initial load
duke
parents:
diff changeset
49 return iff->raw_out(1);
a61af66fc99e Initial load
duke
parents:
diff changeset
50 return NULL;
a61af66fc99e Initial load
duke
parents:
diff changeset
51 }
a61af66fc99e Initial load
duke
parents:
diff changeset
52
a61af66fc99e Initial load
duke
parents:
diff changeset
53
a61af66fc99e Initial load
duke
parents:
diff changeset
54 //=============================================================================
a61af66fc99e Initial load
duke
parents:
diff changeset
55
a61af66fc99e Initial load
duke
parents:
diff changeset
56
a61af66fc99e Initial load
duke
parents:
diff changeset
57 //------------------------------record_for_igvn----------------------------
a61af66fc99e Initial load
duke
parents:
diff changeset
58 // Put loop body on igvn work list
a61af66fc99e Initial load
duke
parents:
diff changeset
59 void IdealLoopTree::record_for_igvn() {
a61af66fc99e Initial load
duke
parents:
diff changeset
60 for( uint i = 0; i < _body.size(); i++ ) {
a61af66fc99e Initial load
duke
parents:
diff changeset
61 Node *n = _body.at(i);
a61af66fc99e Initial load
duke
parents:
diff changeset
62 _phase->_igvn._worklist.push(n);
a61af66fc99e Initial load
duke
parents:
diff changeset
63 }
a61af66fc99e Initial load
duke
parents:
diff changeset
64 }
a61af66fc99e Initial load
duke
parents:
diff changeset
65
2465
3af54845df98 7004555: Add new policy for one iteration loops
kvn
parents: 2453
diff changeset
66 //------------------------------compute_exact_trip_count-----------------------
3af54845df98 7004555: Add new policy for one iteration loops
kvn
parents: 2453
diff changeset
67 // Compute loop exact trip count if possible. Do not recalculate trip count for
3af54845df98 7004555: Add new policy for one iteration loops
kvn
parents: 2453
diff changeset
68 // split loops (pre-main-post) which have their limits and inits behind Opaque node.
3af54845df98 7004555: Add new policy for one iteration loops
kvn
parents: 2453
diff changeset
69 void IdealLoopTree::compute_exact_trip_count( PhaseIdealLoop *phase ) {
3af54845df98 7004555: Add new policy for one iteration loops
kvn
parents: 2453
diff changeset
70 if (!_head->as_Loop()->is_valid_counted_loop()) {
3af54845df98 7004555: Add new policy for one iteration loops
kvn
parents: 2453
diff changeset
71 return;
3af54845df98 7004555: Add new policy for one iteration loops
kvn
parents: 2453
diff changeset
72 }
3af54845df98 7004555: Add new policy for one iteration loops
kvn
parents: 2453
diff changeset
73 CountedLoopNode* cl = _head->as_CountedLoop();
3af54845df98 7004555: Add new policy for one iteration loops
kvn
parents: 2453
diff changeset
74 // Trip count may become nonexact for iteration split loops since
3af54845df98 7004555: Add new policy for one iteration loops
kvn
parents: 2453
diff changeset
75 // RCE modifies limits. Note, _trip_count value is not reset since
3af54845df98 7004555: Add new policy for one iteration loops
kvn
parents: 2453
diff changeset
76 // it is used to limit unrolling of main loop.
3af54845df98 7004555: Add new policy for one iteration loops
kvn
parents: 2453
diff changeset
77 cl->set_nonexact_trip_count();
3af54845df98 7004555: Add new policy for one iteration loops
kvn
parents: 2453
diff changeset
78
3af54845df98 7004555: Add new policy for one iteration loops
kvn
parents: 2453
diff changeset
79 // Loop's test should be part of loop.
3af54845df98 7004555: Add new policy for one iteration loops
kvn
parents: 2453
diff changeset
80 if (!phase->is_member(this, phase->get_ctrl(cl->loopexit()->in(CountedLoopEndNode::TestValue))))
3af54845df98 7004555: Add new policy for one iteration loops
kvn
parents: 2453
diff changeset
81 return; // Infinite loop
3af54845df98 7004555: Add new policy for one iteration loops
kvn
parents: 2453
diff changeset
82
3af54845df98 7004555: Add new policy for one iteration loops
kvn
parents: 2453
diff changeset
83 #ifdef ASSERT
3af54845df98 7004555: Add new policy for one iteration loops
kvn
parents: 2453
diff changeset
84 BoolTest::mask bt = cl->loopexit()->test_trip();
3af54845df98 7004555: Add new policy for one iteration loops
kvn
parents: 2453
diff changeset
85 assert(bt == BoolTest::lt || bt == BoolTest::gt ||
3af54845df98 7004555: Add new policy for one iteration loops
kvn
parents: 2453
diff changeset
86 bt == BoolTest::ne, "canonical test is expected");
3af54845df98 7004555: Add new policy for one iteration loops
kvn
parents: 2453
diff changeset
87 #endif
3af54845df98 7004555: Add new policy for one iteration loops
kvn
parents: 2453
diff changeset
88
3af54845df98 7004555: Add new policy for one iteration loops
kvn
parents: 2453
diff changeset
89 Node* init_n = cl->init_trip();
3af54845df98 7004555: Add new policy for one iteration loops
kvn
parents: 2453
diff changeset
90 Node* limit_n = cl->limit();
3af54845df98 7004555: Add new policy for one iteration loops
kvn
parents: 2453
diff changeset
91 if (init_n != NULL && init_n->is_Con() &&
3af54845df98 7004555: Add new policy for one iteration loops
kvn
parents: 2453
diff changeset
92 limit_n != NULL && limit_n->is_Con()) {
3af54845df98 7004555: Add new policy for one iteration loops
kvn
parents: 2453
diff changeset
93 // Use longs to avoid integer overflow.
3af54845df98 7004555: Add new policy for one iteration loops
kvn
parents: 2453
diff changeset
94 int stride_con = cl->stride_con();
3af54845df98 7004555: Add new policy for one iteration loops
kvn
parents: 2453
diff changeset
95 long init_con = cl->init_trip()->get_int();
3af54845df98 7004555: Add new policy for one iteration loops
kvn
parents: 2453
diff changeset
96 long limit_con = cl->limit()->get_int();
3af54845df98 7004555: Add new policy for one iteration loops
kvn
parents: 2453
diff changeset
97 int stride_m = stride_con - (stride_con > 0 ? 1 : -1);
3af54845df98 7004555: Add new policy for one iteration loops
kvn
parents: 2453
diff changeset
98 long trip_count = (limit_con - init_con + stride_m)/stride_con;
3af54845df98 7004555: Add new policy for one iteration loops
kvn
parents: 2453
diff changeset
99 if (trip_count > 0 && (julong)trip_count < (julong)max_juint) {
3af54845df98 7004555: Add new policy for one iteration loops
kvn
parents: 2453
diff changeset
100 // Set exact trip count.
3af54845df98 7004555: Add new policy for one iteration loops
kvn
parents: 2453
diff changeset
101 cl->set_exact_trip_count((uint)trip_count);
3af54845df98 7004555: Add new policy for one iteration loops
kvn
parents: 2453
diff changeset
102 }
3af54845df98 7004555: Add new policy for one iteration loops
kvn
parents: 2453
diff changeset
103 }
3af54845df98 7004555: Add new policy for one iteration loops
kvn
parents: 2453
diff changeset
104 }
3af54845df98 7004555: Add new policy for one iteration loops
kvn
parents: 2453
diff changeset
105
0
a61af66fc99e Initial load
duke
parents:
diff changeset
106 //------------------------------compute_profile_trip_cnt----------------------------
a61af66fc99e Initial load
duke
parents:
diff changeset
107 // Compute loop trip count from profile data as
a61af66fc99e Initial load
duke
parents:
diff changeset
108 // (backedge_count + loop_exit_count) / loop_exit_count
a61af66fc99e Initial load
duke
parents:
diff changeset
109 void IdealLoopTree::compute_profile_trip_cnt( PhaseIdealLoop *phase ) {
a61af66fc99e Initial load
duke
parents:
diff changeset
110 if (!_head->is_CountedLoop()) {
a61af66fc99e Initial load
duke
parents:
diff changeset
111 return;
a61af66fc99e Initial load
duke
parents:
diff changeset
112 }
a61af66fc99e Initial load
duke
parents:
diff changeset
113 CountedLoopNode* head = _head->as_CountedLoop();
a61af66fc99e Initial load
duke
parents:
diff changeset
114 if (head->profile_trip_cnt() != COUNT_UNKNOWN) {
a61af66fc99e Initial load
duke
parents:
diff changeset
115 return; // Already computed
a61af66fc99e Initial load
duke
parents:
diff changeset
116 }
a61af66fc99e Initial load
duke
parents:
diff changeset
117 float trip_cnt = (float)max_jint; // default is big
a61af66fc99e Initial load
duke
parents:
diff changeset
118
a61af66fc99e Initial load
duke
parents:
diff changeset
119 Node* back = head->in(LoopNode::LoopBackControl);
a61af66fc99e Initial load
duke
parents:
diff changeset
120 while (back != head) {
a61af66fc99e Initial load
duke
parents:
diff changeset
121 if ((back->Opcode() == Op_IfTrue || back->Opcode() == Op_IfFalse) &&
a61af66fc99e Initial load
duke
parents:
diff changeset
122 back->in(0) &&
a61af66fc99e Initial load
duke
parents:
diff changeset
123 back->in(0)->is_If() &&
a61af66fc99e Initial load
duke
parents:
diff changeset
124 back->in(0)->as_If()->_fcnt != COUNT_UNKNOWN &&
a61af66fc99e Initial load
duke
parents:
diff changeset
125 back->in(0)->as_If()->_prob != PROB_UNKNOWN) {
a61af66fc99e Initial load
duke
parents:
diff changeset
126 break;
a61af66fc99e Initial load
duke
parents:
diff changeset
127 }
a61af66fc99e Initial load
duke
parents:
diff changeset
128 back = phase->idom(back);
a61af66fc99e Initial load
duke
parents:
diff changeset
129 }
a61af66fc99e Initial load
duke
parents:
diff changeset
130 if (back != head) {
a61af66fc99e Initial load
duke
parents:
diff changeset
131 assert((back->Opcode() == Op_IfTrue || back->Opcode() == Op_IfFalse) &&
a61af66fc99e Initial load
duke
parents:
diff changeset
132 back->in(0), "if-projection exists");
a61af66fc99e Initial load
duke
parents:
diff changeset
133 IfNode* back_if = back->in(0)->as_If();
a61af66fc99e Initial load
duke
parents:
diff changeset
134 float loop_back_cnt = back_if->_fcnt * back_if->_prob;
a61af66fc99e Initial load
duke
parents:
diff changeset
135
a61af66fc99e Initial load
duke
parents:
diff changeset
136 // Now compute a loop exit count
a61af66fc99e Initial load
duke
parents:
diff changeset
137 float loop_exit_cnt = 0.0f;
a61af66fc99e Initial load
duke
parents:
diff changeset
138 for( uint i = 0; i < _body.size(); i++ ) {
a61af66fc99e Initial load
duke
parents:
diff changeset
139 Node *n = _body[i];
a61af66fc99e Initial load
duke
parents:
diff changeset
140 if( n->is_If() ) {
a61af66fc99e Initial load
duke
parents:
diff changeset
141 IfNode *iff = n->as_If();
a61af66fc99e Initial load
duke
parents:
diff changeset
142 if( iff->_fcnt != COUNT_UNKNOWN && iff->_prob != PROB_UNKNOWN ) {
a61af66fc99e Initial load
duke
parents:
diff changeset
143 Node *exit = is_loop_exit(iff);
a61af66fc99e Initial load
duke
parents:
diff changeset
144 if( exit ) {
a61af66fc99e Initial load
duke
parents:
diff changeset
145 float exit_prob = iff->_prob;
a61af66fc99e Initial load
duke
parents:
diff changeset
146 if (exit->Opcode() == Op_IfFalse) exit_prob = 1.0 - exit_prob;
a61af66fc99e Initial load
duke
parents:
diff changeset
147 if (exit_prob > PROB_MIN) {
a61af66fc99e Initial load
duke
parents:
diff changeset
148 float exit_cnt = iff->_fcnt * exit_prob;
a61af66fc99e Initial load
duke
parents:
diff changeset
149 loop_exit_cnt += exit_cnt;
a61af66fc99e Initial load
duke
parents:
diff changeset
150 }
a61af66fc99e Initial load
duke
parents:
diff changeset
151 }
a61af66fc99e Initial load
duke
parents:
diff changeset
152 }
a61af66fc99e Initial load
duke
parents:
diff changeset
153 }
a61af66fc99e Initial load
duke
parents:
diff changeset
154 }
a61af66fc99e Initial load
duke
parents:
diff changeset
155 if (loop_exit_cnt > 0.0f) {
a61af66fc99e Initial load
duke
parents:
diff changeset
156 trip_cnt = (loop_back_cnt + loop_exit_cnt) / loop_exit_cnt;
a61af66fc99e Initial load
duke
parents:
diff changeset
157 } else {
a61af66fc99e Initial load
duke
parents:
diff changeset
158 // No exit count so use
a61af66fc99e Initial load
duke
parents:
diff changeset
159 trip_cnt = loop_back_cnt;
a61af66fc99e Initial load
duke
parents:
diff changeset
160 }
a61af66fc99e Initial load
duke
parents:
diff changeset
161 }
a61af66fc99e Initial load
duke
parents:
diff changeset
162 #ifndef PRODUCT
a61af66fc99e Initial load
duke
parents:
diff changeset
163 if (TraceProfileTripCount) {
a61af66fc99e Initial load
duke
parents:
diff changeset
164 tty->print_cr("compute_profile_trip_cnt lp: %d cnt: %f\n", head->_idx, trip_cnt);
a61af66fc99e Initial load
duke
parents:
diff changeset
165 }
a61af66fc99e Initial load
duke
parents:
diff changeset
166 #endif
a61af66fc99e Initial load
duke
parents:
diff changeset
167 head->set_profile_trip_cnt(trip_cnt);
a61af66fc99e Initial load
duke
parents:
diff changeset
168 }
a61af66fc99e Initial load
duke
parents:
diff changeset
169
a61af66fc99e Initial load
duke
parents:
diff changeset
170 //---------------------is_invariant_addition-----------------------------
a61af66fc99e Initial load
duke
parents:
diff changeset
171 // Return nonzero index of invariant operand for an Add or Sub
605
98cb887364d3 6810672: Comment typos
twisti
parents: 420
diff changeset
172 // of (nonconstant) invariant and variant values. Helper for reassociate_invariants.
0
a61af66fc99e Initial load
duke
parents:
diff changeset
173 int IdealLoopTree::is_invariant_addition(Node* n, PhaseIdealLoop *phase) {
a61af66fc99e Initial load
duke
parents:
diff changeset
174 int op = n->Opcode();
a61af66fc99e Initial load
duke
parents:
diff changeset
175 if (op == Op_AddI || op == Op_SubI) {
a61af66fc99e Initial load
duke
parents:
diff changeset
176 bool in1_invar = this->is_invariant(n->in(1));
a61af66fc99e Initial load
duke
parents:
diff changeset
177 bool in2_invar = this->is_invariant(n->in(2));
a61af66fc99e Initial load
duke
parents:
diff changeset
178 if (in1_invar && !in2_invar) return 1;
a61af66fc99e Initial load
duke
parents:
diff changeset
179 if (!in1_invar && in2_invar) return 2;
a61af66fc99e Initial load
duke
parents:
diff changeset
180 }
a61af66fc99e Initial load
duke
parents:
diff changeset
181 return 0;
a61af66fc99e Initial load
duke
parents:
diff changeset
182 }
a61af66fc99e Initial load
duke
parents:
diff changeset
183
a61af66fc99e Initial load
duke
parents:
diff changeset
184 //---------------------reassociate_add_sub-----------------------------
a61af66fc99e Initial load
duke
parents:
diff changeset
185 // Reassociate invariant add and subtract expressions:
a61af66fc99e Initial load
duke
parents:
diff changeset
186 //
a61af66fc99e Initial load
duke
parents:
diff changeset
187 // inv1 + (x + inv2) => ( inv1 + inv2) + x
a61af66fc99e Initial load
duke
parents:
diff changeset
188 // (x + inv2) + inv1 => ( inv1 + inv2) + x
a61af66fc99e Initial load
duke
parents:
diff changeset
189 // inv1 + (x - inv2) => ( inv1 - inv2) + x
a61af66fc99e Initial load
duke
parents:
diff changeset
190 // inv1 - (inv2 - x) => ( inv1 - inv2) + x
a61af66fc99e Initial load
duke
parents:
diff changeset
191 // (x + inv2) - inv1 => (-inv1 + inv2) + x
a61af66fc99e Initial load
duke
parents:
diff changeset
192 // (x - inv2) + inv1 => ( inv1 - inv2) + x
a61af66fc99e Initial load
duke
parents:
diff changeset
193 // (x - inv2) - inv1 => (-inv1 - inv2) + x
a61af66fc99e Initial load
duke
parents:
diff changeset
194 // inv1 + (inv2 - x) => ( inv1 + inv2) - x
a61af66fc99e Initial load
duke
parents:
diff changeset
195 // inv1 - (x - inv2) => ( inv1 + inv2) - x
a61af66fc99e Initial load
duke
parents:
diff changeset
196 // (inv2 - x) + inv1 => ( inv1 + inv2) - x
a61af66fc99e Initial load
duke
parents:
diff changeset
197 // (inv2 - x) - inv1 => (-inv1 + inv2) - x
a61af66fc99e Initial load
duke
parents:
diff changeset
198 // inv1 - (x + inv2) => ( inv1 - inv2) - x
a61af66fc99e Initial load
duke
parents:
diff changeset
199 //
a61af66fc99e Initial load
duke
parents:
diff changeset
200 Node* IdealLoopTree::reassociate_add_sub(Node* n1, PhaseIdealLoop *phase) {
a61af66fc99e Initial load
duke
parents:
diff changeset
201 if (!n1->is_Add() && !n1->is_Sub() || n1->outcnt() == 0) return NULL;
a61af66fc99e Initial load
duke
parents:
diff changeset
202 if (is_invariant(n1)) return NULL;
a61af66fc99e Initial load
duke
parents:
diff changeset
203 int inv1_idx = is_invariant_addition(n1, phase);
a61af66fc99e Initial load
duke
parents:
diff changeset
204 if (!inv1_idx) return NULL;
a61af66fc99e Initial load
duke
parents:
diff changeset
205 // Don't mess with add of constant (igvn moves them to expression tree root.)
a61af66fc99e Initial load
duke
parents:
diff changeset
206 if (n1->is_Add() && n1->in(2)->is_Con()) return NULL;
a61af66fc99e Initial load
duke
parents:
diff changeset
207 Node* inv1 = n1->in(inv1_idx);
a61af66fc99e Initial load
duke
parents:
diff changeset
208 Node* n2 = n1->in(3 - inv1_idx);
a61af66fc99e Initial load
duke
parents:
diff changeset
209 int inv2_idx = is_invariant_addition(n2, phase);
a61af66fc99e Initial load
duke
parents:
diff changeset
210 if (!inv2_idx) return NULL;
a61af66fc99e Initial load
duke
parents:
diff changeset
211 Node* x = n2->in(3 - inv2_idx);
a61af66fc99e Initial load
duke
parents:
diff changeset
212 Node* inv2 = n2->in(inv2_idx);
a61af66fc99e Initial load
duke
parents:
diff changeset
213
a61af66fc99e Initial load
duke
parents:
diff changeset
214 bool neg_x = n2->is_Sub() && inv2_idx == 1;
a61af66fc99e Initial load
duke
parents:
diff changeset
215 bool neg_inv2 = n2->is_Sub() && inv2_idx == 2;
a61af66fc99e Initial load
duke
parents:
diff changeset
216 bool neg_inv1 = n1->is_Sub() && inv1_idx == 2;
a61af66fc99e Initial load
duke
parents:
diff changeset
217 if (n1->is_Sub() && inv1_idx == 1) {
a61af66fc99e Initial load
duke
parents:
diff changeset
218 neg_x = !neg_x;
a61af66fc99e Initial load
duke
parents:
diff changeset
219 neg_inv2 = !neg_inv2;
a61af66fc99e Initial load
duke
parents:
diff changeset
220 }
a61af66fc99e Initial load
duke
parents:
diff changeset
221 Node* inv1_c = phase->get_ctrl(inv1);
a61af66fc99e Initial load
duke
parents:
diff changeset
222 Node* inv2_c = phase->get_ctrl(inv2);
a61af66fc99e Initial load
duke
parents:
diff changeset
223 Node* n_inv1;
a61af66fc99e Initial load
duke
parents:
diff changeset
224 if (neg_inv1) {
a61af66fc99e Initial load
duke
parents:
diff changeset
225 Node *zero = phase->_igvn.intcon(0);
a61af66fc99e Initial load
duke
parents:
diff changeset
226 phase->set_ctrl(zero, phase->C->root());
a61af66fc99e Initial load
duke
parents:
diff changeset
227 n_inv1 = new (phase->C, 3) SubINode(zero, inv1);
a61af66fc99e Initial load
duke
parents:
diff changeset
228 phase->register_new_node(n_inv1, inv1_c);
a61af66fc99e Initial load
duke
parents:
diff changeset
229 } else {
a61af66fc99e Initial load
duke
parents:
diff changeset
230 n_inv1 = inv1;
a61af66fc99e Initial load
duke
parents:
diff changeset
231 }
a61af66fc99e Initial load
duke
parents:
diff changeset
232 Node* inv;
a61af66fc99e Initial load
duke
parents:
diff changeset
233 if (neg_inv2) {
a61af66fc99e Initial load
duke
parents:
diff changeset
234 inv = new (phase->C, 3) SubINode(n_inv1, inv2);
a61af66fc99e Initial load
duke
parents:
diff changeset
235 } else {
a61af66fc99e Initial load
duke
parents:
diff changeset
236 inv = new (phase->C, 3) AddINode(n_inv1, inv2);
a61af66fc99e Initial load
duke
parents:
diff changeset
237 }
a61af66fc99e Initial load
duke
parents:
diff changeset
238 phase->register_new_node(inv, phase->get_early_ctrl(inv));
a61af66fc99e Initial load
duke
parents:
diff changeset
239
a61af66fc99e Initial load
duke
parents:
diff changeset
240 Node* addx;
a61af66fc99e Initial load
duke
parents:
diff changeset
241 if (neg_x) {
a61af66fc99e Initial load
duke
parents:
diff changeset
242 addx = new (phase->C, 3) SubINode(inv, x);
a61af66fc99e Initial load
duke
parents:
diff changeset
243 } else {
a61af66fc99e Initial load
duke
parents:
diff changeset
244 addx = new (phase->C, 3) AddINode(x, inv);
a61af66fc99e Initial load
duke
parents:
diff changeset
245 }
a61af66fc99e Initial load
duke
parents:
diff changeset
246 phase->register_new_node(addx, phase->get_ctrl(x));
1621
6027dddc26c6 6677629: PhaseIterGVN::subsume_node() should call hash_delete() and add_users_to_worklist()
kvn
parents: 1552
diff changeset
247 phase->_igvn.replace_node(n1, addx);
2383
9dc311b8473e 7008866: Missing loop predicate for loop with multiple entries
kvn
parents: 1972
diff changeset
248 assert(phase->get_loop(phase->get_ctrl(n1)) == this, "");
9dc311b8473e 7008866: Missing loop predicate for loop with multiple entries
kvn
parents: 1972
diff changeset
249 _body.yank(n1);
0
a61af66fc99e Initial load
duke
parents:
diff changeset
250 return addx;
a61af66fc99e Initial load
duke
parents:
diff changeset
251 }
a61af66fc99e Initial load
duke
parents:
diff changeset
252
a61af66fc99e Initial load
duke
parents:
diff changeset
253 //---------------------reassociate_invariants-----------------------------
a61af66fc99e Initial load
duke
parents:
diff changeset
254 // Reassociate invariant expressions:
a61af66fc99e Initial load
duke
parents:
diff changeset
255 void IdealLoopTree::reassociate_invariants(PhaseIdealLoop *phase) {
a61af66fc99e Initial load
duke
parents:
diff changeset
256 for (int i = _body.size() - 1; i >= 0; i--) {
a61af66fc99e Initial load
duke
parents:
diff changeset
257 Node *n = _body.at(i);
a61af66fc99e Initial load
duke
parents:
diff changeset
258 for (int j = 0; j < 5; j++) {
a61af66fc99e Initial load
duke
parents:
diff changeset
259 Node* nn = reassociate_add_sub(n, phase);
a61af66fc99e Initial load
duke
parents:
diff changeset
260 if (nn == NULL) break;
a61af66fc99e Initial load
duke
parents:
diff changeset
261 n = nn; // again
a61af66fc99e Initial load
duke
parents:
diff changeset
262 };
a61af66fc99e Initial load
duke
parents:
diff changeset
263 }
a61af66fc99e Initial load
duke
parents:
diff changeset
264 }
a61af66fc99e Initial load
duke
parents:
diff changeset
265
a61af66fc99e Initial load
duke
parents:
diff changeset
266 //------------------------------policy_peeling---------------------------------
a61af66fc99e Initial load
duke
parents:
diff changeset
267 // Return TRUE or FALSE if the loop should be peeled or not. Peel if we can
a61af66fc99e Initial load
duke
parents:
diff changeset
268 // make some loop-invariant test (usually a null-check) happen before the loop.
a61af66fc99e Initial load
duke
parents:
diff changeset
269 bool IdealLoopTree::policy_peeling( PhaseIdealLoop *phase ) const {
a61af66fc99e Initial load
duke
parents:
diff changeset
270 Node *test = ((IdealLoopTree*)this)->tail();
a61af66fc99e Initial load
duke
parents:
diff changeset
271 int body_size = ((IdealLoopTree*)this)->_body.size();
a61af66fc99e Initial load
duke
parents:
diff changeset
272 int uniq = phase->C->unique();
a61af66fc99e Initial load
duke
parents:
diff changeset
273 // Peeling does loop cloning which can result in O(N^2) node construction
a61af66fc99e Initial load
duke
parents:
diff changeset
274 if( body_size > 255 /* Prevent overflow for large body_size */
a61af66fc99e Initial load
duke
parents:
diff changeset
275 || (body_size * body_size + uniq > MaxNodeLimit) ) {
a61af66fc99e Initial load
duke
parents:
diff changeset
276 return false; // too large to safely clone
a61af66fc99e Initial load
duke
parents:
diff changeset
277 }
a61af66fc99e Initial load
duke
parents:
diff changeset
278 while( test != _head ) { // Scan till run off top of loop
a61af66fc99e Initial load
duke
parents:
diff changeset
279 if( test->is_If() ) { // Test?
a61af66fc99e Initial load
duke
parents:
diff changeset
280 Node *ctrl = phase->get_ctrl(test->in(1));
a61af66fc99e Initial load
duke
parents:
diff changeset
281 if (ctrl->is_top())
a61af66fc99e Initial load
duke
parents:
diff changeset
282 return false; // Found dead test on live IF? No peeling!
a61af66fc99e Initial load
duke
parents:
diff changeset
283 // Standard IF only has one input value to check for loop invariance
a61af66fc99e Initial load
duke
parents:
diff changeset
284 assert( test->Opcode() == Op_If || test->Opcode() == Op_CountedLoopEnd, "Check this code when new subtype is added");
a61af66fc99e Initial load
duke
parents:
diff changeset
285 // Condition is not a member of this loop?
a61af66fc99e Initial load
duke
parents:
diff changeset
286 if( !is_member(phase->get_loop(ctrl)) &&
a61af66fc99e Initial load
duke
parents:
diff changeset
287 is_loop_exit(test) )
a61af66fc99e Initial load
duke
parents:
diff changeset
288 return true; // Found reason to peel!
a61af66fc99e Initial load
duke
parents:
diff changeset
289 }
a61af66fc99e Initial load
duke
parents:
diff changeset
290 // Walk up dominators to loop _head looking for test which is
a61af66fc99e Initial load
duke
parents:
diff changeset
291 // executed on every path thru loop.
a61af66fc99e Initial load
duke
parents:
diff changeset
292 test = phase->idom(test);
a61af66fc99e Initial load
duke
parents:
diff changeset
293 }
a61af66fc99e Initial load
duke
parents:
diff changeset
294 return false;
a61af66fc99e Initial load
duke
parents:
diff changeset
295 }
a61af66fc99e Initial load
duke
parents:
diff changeset
296
a61af66fc99e Initial load
duke
parents:
diff changeset
297 //------------------------------peeled_dom_test_elim---------------------------
a61af66fc99e Initial load
duke
parents:
diff changeset
298 // If we got the effect of peeling, either by actually peeling or by making
a61af66fc99e Initial load
duke
parents:
diff changeset
299 // a pre-loop which must execute at least once, we can remove all
a61af66fc99e Initial load
duke
parents:
diff changeset
300 // loop-invariant dominated tests in the main body.
a61af66fc99e Initial load
duke
parents:
diff changeset
301 void PhaseIdealLoop::peeled_dom_test_elim( IdealLoopTree *loop, Node_List &old_new ) {
a61af66fc99e Initial load
duke
parents:
diff changeset
302 bool progress = true;
a61af66fc99e Initial load
duke
parents:
diff changeset
303 while( progress ) {
a61af66fc99e Initial load
duke
parents:
diff changeset
304 progress = false; // Reset for next iteration
a61af66fc99e Initial load
duke
parents:
diff changeset
305 Node *prev = loop->_head->in(LoopNode::LoopBackControl);//loop->tail();
a61af66fc99e Initial load
duke
parents:
diff changeset
306 Node *test = prev->in(0);
a61af66fc99e Initial load
duke
parents:
diff changeset
307 while( test != loop->_head ) { // Scan till run off top of loop
a61af66fc99e Initial load
duke
parents:
diff changeset
308
a61af66fc99e Initial load
duke
parents:
diff changeset
309 int p_op = prev->Opcode();
a61af66fc99e Initial load
duke
parents:
diff changeset
310 if( (p_op == Op_IfFalse || p_op == Op_IfTrue) &&
a61af66fc99e Initial load
duke
parents:
diff changeset
311 test->is_If() && // Test?
a61af66fc99e Initial load
duke
parents:
diff changeset
312 !test->in(1)->is_Con() && // And not already obvious?
a61af66fc99e Initial load
duke
parents:
diff changeset
313 // Condition is not a member of this loop?
a61af66fc99e Initial load
duke
parents:
diff changeset
314 !loop->is_member(get_loop(get_ctrl(test->in(1))))){
a61af66fc99e Initial load
duke
parents:
diff changeset
315 // Walk loop body looking for instances of this test
a61af66fc99e Initial load
duke
parents:
diff changeset
316 for( uint i = 0; i < loop->_body.size(); i++ ) {
a61af66fc99e Initial load
duke
parents:
diff changeset
317 Node *n = loop->_body.at(i);
a61af66fc99e Initial load
duke
parents:
diff changeset
318 if( n->is_If() && n->in(1) == test->in(1) /*&& n != loop->tail()->in(0)*/ ) {
a61af66fc99e Initial load
duke
parents:
diff changeset
319 // IfNode was dominated by version in peeled loop body
a61af66fc99e Initial load
duke
parents:
diff changeset
320 progress = true;
a61af66fc99e Initial load
duke
parents:
diff changeset
321 dominated_by( old_new[prev->_idx], n );
a61af66fc99e Initial load
duke
parents:
diff changeset
322 }
a61af66fc99e Initial load
duke
parents:
diff changeset
323 }
a61af66fc99e Initial load
duke
parents:
diff changeset
324 }
a61af66fc99e Initial load
duke
parents:
diff changeset
325 prev = test;
a61af66fc99e Initial load
duke
parents:
diff changeset
326 test = idom(test);
a61af66fc99e Initial load
duke
parents:
diff changeset
327 } // End of scan tests in loop
a61af66fc99e Initial load
duke
parents:
diff changeset
328
a61af66fc99e Initial load
duke
parents:
diff changeset
329 } // End of while( progress )
a61af66fc99e Initial load
duke
parents:
diff changeset
330 }
a61af66fc99e Initial load
duke
parents:
diff changeset
331
a61af66fc99e Initial load
duke
parents:
diff changeset
332 //------------------------------do_peeling-------------------------------------
a61af66fc99e Initial load
duke
parents:
diff changeset
333 // Peel the first iteration of the given loop.
a61af66fc99e Initial load
duke
parents:
diff changeset
334 // Step 1: Clone the loop body. The clone becomes the peeled iteration.
a61af66fc99e Initial load
duke
parents:
diff changeset
335 // The pre-loop illegally has 2 control users (old & new loops).
a61af66fc99e Initial load
duke
parents:
diff changeset
336 // Step 2: Make the old-loop fall-in edges point to the peeled iteration.
a61af66fc99e Initial load
duke
parents:
diff changeset
337 // Do this by making the old-loop fall-in edges act as if they came
a61af66fc99e Initial load
duke
parents:
diff changeset
338 // around the loopback from the prior iteration (follow the old-loop
a61af66fc99e Initial load
duke
parents:
diff changeset
339 // backedges) and then map to the new peeled iteration. This leaves
a61af66fc99e Initial load
duke
parents:
diff changeset
340 // the pre-loop with only 1 user (the new peeled iteration), but the
a61af66fc99e Initial load
duke
parents:
diff changeset
341 // peeled-loop backedge has 2 users.
a61af66fc99e Initial load
duke
parents:
diff changeset
342 // Step 3: Cut the backedge on the clone (so its not a loop) and remove the
a61af66fc99e Initial load
duke
parents:
diff changeset
343 // extra backedge user.
2445
08eb13460b3a 7004535: Clone loop predicate during loop unswitch
kvn
parents: 2417
diff changeset
344 //
08eb13460b3a 7004535: Clone loop predicate during loop unswitch
kvn
parents: 2417
diff changeset
345 // orig
08eb13460b3a 7004535: Clone loop predicate during loop unswitch
kvn
parents: 2417
diff changeset
346 //
08eb13460b3a 7004535: Clone loop predicate during loop unswitch
kvn
parents: 2417
diff changeset
347 // stmt1
08eb13460b3a 7004535: Clone loop predicate during loop unswitch
kvn
parents: 2417
diff changeset
348 // |
08eb13460b3a 7004535: Clone loop predicate during loop unswitch
kvn
parents: 2417
diff changeset
349 // v
08eb13460b3a 7004535: Clone loop predicate during loop unswitch
kvn
parents: 2417
diff changeset
350 // loop predicate
08eb13460b3a 7004535: Clone loop predicate during loop unswitch
kvn
parents: 2417
diff changeset
351 // |
08eb13460b3a 7004535: Clone loop predicate during loop unswitch
kvn
parents: 2417
diff changeset
352 // v
08eb13460b3a 7004535: Clone loop predicate during loop unswitch
kvn
parents: 2417
diff changeset
353 // loop<----+
08eb13460b3a 7004535: Clone loop predicate during loop unswitch
kvn
parents: 2417
diff changeset
354 // | |
08eb13460b3a 7004535: Clone loop predicate during loop unswitch
kvn
parents: 2417
diff changeset
355 // stmt2 |
08eb13460b3a 7004535: Clone loop predicate during loop unswitch
kvn
parents: 2417
diff changeset
356 // | |
08eb13460b3a 7004535: Clone loop predicate during loop unswitch
kvn
parents: 2417
diff changeset
357 // v |
08eb13460b3a 7004535: Clone loop predicate during loop unswitch
kvn
parents: 2417
diff changeset
358 // if ^
08eb13460b3a 7004535: Clone loop predicate during loop unswitch
kvn
parents: 2417
diff changeset
359 // / \ |
08eb13460b3a 7004535: Clone loop predicate during loop unswitch
kvn
parents: 2417
diff changeset
360 // / \ |
08eb13460b3a 7004535: Clone loop predicate during loop unswitch
kvn
parents: 2417
diff changeset
361 // v v |
08eb13460b3a 7004535: Clone loop predicate during loop unswitch
kvn
parents: 2417
diff changeset
362 // false true |
08eb13460b3a 7004535: Clone loop predicate during loop unswitch
kvn
parents: 2417
diff changeset
363 // / \ |
08eb13460b3a 7004535: Clone loop predicate during loop unswitch
kvn
parents: 2417
diff changeset
364 // / ----+
08eb13460b3a 7004535: Clone loop predicate during loop unswitch
kvn
parents: 2417
diff changeset
365 // |
08eb13460b3a 7004535: Clone loop predicate during loop unswitch
kvn
parents: 2417
diff changeset
366 // v
08eb13460b3a 7004535: Clone loop predicate during loop unswitch
kvn
parents: 2417
diff changeset
367 // exit
08eb13460b3a 7004535: Clone loop predicate during loop unswitch
kvn
parents: 2417
diff changeset
368 //
08eb13460b3a 7004535: Clone loop predicate during loop unswitch
kvn
parents: 2417
diff changeset
369 //
08eb13460b3a 7004535: Clone loop predicate during loop unswitch
kvn
parents: 2417
diff changeset
370 // after clone loop
08eb13460b3a 7004535: Clone loop predicate during loop unswitch
kvn
parents: 2417
diff changeset
371 //
08eb13460b3a 7004535: Clone loop predicate during loop unswitch
kvn
parents: 2417
diff changeset
372 // stmt1
08eb13460b3a 7004535: Clone loop predicate during loop unswitch
kvn
parents: 2417
diff changeset
373 // |
08eb13460b3a 7004535: Clone loop predicate during loop unswitch
kvn
parents: 2417
diff changeset
374 // v
08eb13460b3a 7004535: Clone loop predicate during loop unswitch
kvn
parents: 2417
diff changeset
375 // loop predicate
08eb13460b3a 7004535: Clone loop predicate during loop unswitch
kvn
parents: 2417
diff changeset
376 // / \
08eb13460b3a 7004535: Clone loop predicate during loop unswitch
kvn
parents: 2417
diff changeset
377 // clone / \ orig
08eb13460b3a 7004535: Clone loop predicate during loop unswitch
kvn
parents: 2417
diff changeset
378 // / \
08eb13460b3a 7004535: Clone loop predicate during loop unswitch
kvn
parents: 2417
diff changeset
379 // / \
08eb13460b3a 7004535: Clone loop predicate during loop unswitch
kvn
parents: 2417
diff changeset
380 // v v
08eb13460b3a 7004535: Clone loop predicate during loop unswitch
kvn
parents: 2417
diff changeset
381 // +---->loop clone loop<----+
08eb13460b3a 7004535: Clone loop predicate during loop unswitch
kvn
parents: 2417
diff changeset
382 // | | | |
08eb13460b3a 7004535: Clone loop predicate during loop unswitch
kvn
parents: 2417
diff changeset
383 // | stmt2 clone stmt2 |
08eb13460b3a 7004535: Clone loop predicate during loop unswitch
kvn
parents: 2417
diff changeset
384 // | | | |
08eb13460b3a 7004535: Clone loop predicate during loop unswitch
kvn
parents: 2417
diff changeset
385 // | v v |
08eb13460b3a 7004535: Clone loop predicate during loop unswitch
kvn
parents: 2417
diff changeset
386 // ^ if clone If ^
08eb13460b3a 7004535: Clone loop predicate during loop unswitch
kvn
parents: 2417
diff changeset
387 // | / \ / \ |
08eb13460b3a 7004535: Clone loop predicate during loop unswitch
kvn
parents: 2417
diff changeset
388 // | / \ / \ |
08eb13460b3a 7004535: Clone loop predicate during loop unswitch
kvn
parents: 2417
diff changeset
389 // | v v v v |
08eb13460b3a 7004535: Clone loop predicate during loop unswitch
kvn
parents: 2417
diff changeset
390 // | true false false true |
08eb13460b3a 7004535: Clone loop predicate during loop unswitch
kvn
parents: 2417
diff changeset
391 // | / \ / \ |
08eb13460b3a 7004535: Clone loop predicate during loop unswitch
kvn
parents: 2417
diff changeset
392 // +---- \ / ----+
08eb13460b3a 7004535: Clone loop predicate during loop unswitch
kvn
parents: 2417
diff changeset
393 // \ /
08eb13460b3a 7004535: Clone loop predicate during loop unswitch
kvn
parents: 2417
diff changeset
394 // 1v v2
08eb13460b3a 7004535: Clone loop predicate during loop unswitch
kvn
parents: 2417
diff changeset
395 // region
08eb13460b3a 7004535: Clone loop predicate during loop unswitch
kvn
parents: 2417
diff changeset
396 // |
08eb13460b3a 7004535: Clone loop predicate during loop unswitch
kvn
parents: 2417
diff changeset
397 // v
08eb13460b3a 7004535: Clone loop predicate during loop unswitch
kvn
parents: 2417
diff changeset
398 // exit
08eb13460b3a 7004535: Clone loop predicate during loop unswitch
kvn
parents: 2417
diff changeset
399 //
08eb13460b3a 7004535: Clone loop predicate during loop unswitch
kvn
parents: 2417
diff changeset
400 //
08eb13460b3a 7004535: Clone loop predicate during loop unswitch
kvn
parents: 2417
diff changeset
401 // after peel and predicate move
08eb13460b3a 7004535: Clone loop predicate during loop unswitch
kvn
parents: 2417
diff changeset
402 //
08eb13460b3a 7004535: Clone loop predicate during loop unswitch
kvn
parents: 2417
diff changeset
403 // stmt1
08eb13460b3a 7004535: Clone loop predicate during loop unswitch
kvn
parents: 2417
diff changeset
404 // /
08eb13460b3a 7004535: Clone loop predicate during loop unswitch
kvn
parents: 2417
diff changeset
405 // /
08eb13460b3a 7004535: Clone loop predicate during loop unswitch
kvn
parents: 2417
diff changeset
406 // clone / orig
08eb13460b3a 7004535: Clone loop predicate during loop unswitch
kvn
parents: 2417
diff changeset
407 // /
08eb13460b3a 7004535: Clone loop predicate during loop unswitch
kvn
parents: 2417
diff changeset
408 // / +----------+
08eb13460b3a 7004535: Clone loop predicate during loop unswitch
kvn
parents: 2417
diff changeset
409 // / | |
08eb13460b3a 7004535: Clone loop predicate during loop unswitch
kvn
parents: 2417
diff changeset
410 // / loop predicate |
08eb13460b3a 7004535: Clone loop predicate during loop unswitch
kvn
parents: 2417
diff changeset
411 // / | |
08eb13460b3a 7004535: Clone loop predicate during loop unswitch
kvn
parents: 2417
diff changeset
412 // v v |
08eb13460b3a 7004535: Clone loop predicate during loop unswitch
kvn
parents: 2417
diff changeset
413 // TOP-->loop clone loop<----+ |
08eb13460b3a 7004535: Clone loop predicate during loop unswitch
kvn
parents: 2417
diff changeset
414 // | | | |
08eb13460b3a 7004535: Clone loop predicate during loop unswitch
kvn
parents: 2417
diff changeset
415 // stmt2 clone stmt2 | |
08eb13460b3a 7004535: Clone loop predicate during loop unswitch
kvn
parents: 2417
diff changeset
416 // | | | ^
08eb13460b3a 7004535: Clone loop predicate during loop unswitch
kvn
parents: 2417
diff changeset
417 // v v | |
08eb13460b3a 7004535: Clone loop predicate during loop unswitch
kvn
parents: 2417
diff changeset
418 // if clone If ^ |
08eb13460b3a 7004535: Clone loop predicate during loop unswitch
kvn
parents: 2417
diff changeset
419 // / \ / \ | |
08eb13460b3a 7004535: Clone loop predicate during loop unswitch
kvn
parents: 2417
diff changeset
420 // / \ / \ | |
08eb13460b3a 7004535: Clone loop predicate during loop unswitch
kvn
parents: 2417
diff changeset
421 // v v v v | |
08eb13460b3a 7004535: Clone loop predicate during loop unswitch
kvn
parents: 2417
diff changeset
422 // true false false true | |
08eb13460b3a 7004535: Clone loop predicate during loop unswitch
kvn
parents: 2417
diff changeset
423 // | \ / \ | |
08eb13460b3a 7004535: Clone loop predicate during loop unswitch
kvn
parents: 2417
diff changeset
424 // | \ / ----+ ^
08eb13460b3a 7004535: Clone loop predicate during loop unswitch
kvn
parents: 2417
diff changeset
425 // | \ / |
08eb13460b3a 7004535: Clone loop predicate during loop unswitch
kvn
parents: 2417
diff changeset
426 // | 1v v2 |
08eb13460b3a 7004535: Clone loop predicate during loop unswitch
kvn
parents: 2417
diff changeset
427 // v region |
08eb13460b3a 7004535: Clone loop predicate during loop unswitch
kvn
parents: 2417
diff changeset
428 // | | |
08eb13460b3a 7004535: Clone loop predicate during loop unswitch
kvn
parents: 2417
diff changeset
429 // | v |
08eb13460b3a 7004535: Clone loop predicate during loop unswitch
kvn
parents: 2417
diff changeset
430 // | exit |
08eb13460b3a 7004535: Clone loop predicate during loop unswitch
kvn
parents: 2417
diff changeset
431 // | |
08eb13460b3a 7004535: Clone loop predicate during loop unswitch
kvn
parents: 2417
diff changeset
432 // +--------------->-----------------+
08eb13460b3a 7004535: Clone loop predicate during loop unswitch
kvn
parents: 2417
diff changeset
433 //
08eb13460b3a 7004535: Clone loop predicate during loop unswitch
kvn
parents: 2417
diff changeset
434 //
08eb13460b3a 7004535: Clone loop predicate during loop unswitch
kvn
parents: 2417
diff changeset
435 // final graph
08eb13460b3a 7004535: Clone loop predicate during loop unswitch
kvn
parents: 2417
diff changeset
436 //
08eb13460b3a 7004535: Clone loop predicate during loop unswitch
kvn
parents: 2417
diff changeset
437 // stmt1
08eb13460b3a 7004535: Clone loop predicate during loop unswitch
kvn
parents: 2417
diff changeset
438 // |
08eb13460b3a 7004535: Clone loop predicate during loop unswitch
kvn
parents: 2417
diff changeset
439 // v
08eb13460b3a 7004535: Clone loop predicate during loop unswitch
kvn
parents: 2417
diff changeset
440 // stmt2 clone
08eb13460b3a 7004535: Clone loop predicate during loop unswitch
kvn
parents: 2417
diff changeset
441 // |
08eb13460b3a 7004535: Clone loop predicate during loop unswitch
kvn
parents: 2417
diff changeset
442 // v
08eb13460b3a 7004535: Clone loop predicate during loop unswitch
kvn
parents: 2417
diff changeset
443 // if clone
08eb13460b3a 7004535: Clone loop predicate during loop unswitch
kvn
parents: 2417
diff changeset
444 // / |
08eb13460b3a 7004535: Clone loop predicate during loop unswitch
kvn
parents: 2417
diff changeset
445 // / |
08eb13460b3a 7004535: Clone loop predicate during loop unswitch
kvn
parents: 2417
diff changeset
446 // v v
08eb13460b3a 7004535: Clone loop predicate during loop unswitch
kvn
parents: 2417
diff changeset
447 // false true
08eb13460b3a 7004535: Clone loop predicate during loop unswitch
kvn
parents: 2417
diff changeset
448 // | |
08eb13460b3a 7004535: Clone loop predicate during loop unswitch
kvn
parents: 2417
diff changeset
449 // | v
08eb13460b3a 7004535: Clone loop predicate during loop unswitch
kvn
parents: 2417
diff changeset
450 // | loop predicate
08eb13460b3a 7004535: Clone loop predicate during loop unswitch
kvn
parents: 2417
diff changeset
451 // | |
08eb13460b3a 7004535: Clone loop predicate during loop unswitch
kvn
parents: 2417
diff changeset
452 // | v
08eb13460b3a 7004535: Clone loop predicate during loop unswitch
kvn
parents: 2417
diff changeset
453 // | loop<----+
08eb13460b3a 7004535: Clone loop predicate during loop unswitch
kvn
parents: 2417
diff changeset
454 // | | |
08eb13460b3a 7004535: Clone loop predicate during loop unswitch
kvn
parents: 2417
diff changeset
455 // | stmt2 |
08eb13460b3a 7004535: Clone loop predicate during loop unswitch
kvn
parents: 2417
diff changeset
456 // | | |
08eb13460b3a 7004535: Clone loop predicate during loop unswitch
kvn
parents: 2417
diff changeset
457 // | v |
08eb13460b3a 7004535: Clone loop predicate during loop unswitch
kvn
parents: 2417
diff changeset
458 // v if ^
08eb13460b3a 7004535: Clone loop predicate during loop unswitch
kvn
parents: 2417
diff changeset
459 // | / \ |
08eb13460b3a 7004535: Clone loop predicate during loop unswitch
kvn
parents: 2417
diff changeset
460 // | / \ |
08eb13460b3a 7004535: Clone loop predicate during loop unswitch
kvn
parents: 2417
diff changeset
461 // | v v |
08eb13460b3a 7004535: Clone loop predicate during loop unswitch
kvn
parents: 2417
diff changeset
462 // | false true |
08eb13460b3a 7004535: Clone loop predicate during loop unswitch
kvn
parents: 2417
diff changeset
463 // | | \ |
08eb13460b3a 7004535: Clone loop predicate during loop unswitch
kvn
parents: 2417
diff changeset
464 // v v --+
08eb13460b3a 7004535: Clone loop predicate during loop unswitch
kvn
parents: 2417
diff changeset
465 // region
08eb13460b3a 7004535: Clone loop predicate during loop unswitch
kvn
parents: 2417
diff changeset
466 // |
08eb13460b3a 7004535: Clone loop predicate during loop unswitch
kvn
parents: 2417
diff changeset
467 // v
08eb13460b3a 7004535: Clone loop predicate during loop unswitch
kvn
parents: 2417
diff changeset
468 // exit
08eb13460b3a 7004535: Clone loop predicate during loop unswitch
kvn
parents: 2417
diff changeset
469 //
0
a61af66fc99e Initial load
duke
parents:
diff changeset
470 void PhaseIdealLoop::do_peeling( IdealLoopTree *loop, Node_List &old_new ) {
a61af66fc99e Initial load
duke
parents:
diff changeset
471
a61af66fc99e Initial load
duke
parents:
diff changeset
472 C->set_major_progress();
a61af66fc99e Initial load
duke
parents:
diff changeset
473 // Peeling a 'main' loop in a pre/main/post situation obfuscates the
a61af66fc99e Initial load
duke
parents:
diff changeset
474 // 'pre' loop from the main and the 'pre' can no longer have it's
a61af66fc99e Initial load
duke
parents:
diff changeset
475 // iterations adjusted. Therefore, we need to declare this loop as
a61af66fc99e Initial load
duke
parents:
diff changeset
476 // no longer a 'main' loop; it will need new pre and post loops before
a61af66fc99e Initial load
duke
parents:
diff changeset
477 // we can do further RCE.
2383
9dc311b8473e 7008866: Missing loop predicate for loop with multiple entries
kvn
parents: 1972
diff changeset
478 #ifndef PRODUCT
9dc311b8473e 7008866: Missing loop predicate for loop with multiple entries
kvn
parents: 1972
diff changeset
479 if (TraceLoopOpts) {
9dc311b8473e 7008866: Missing loop predicate for loop with multiple entries
kvn
parents: 1972
diff changeset
480 tty->print("Peel ");
9dc311b8473e 7008866: Missing loop predicate for loop with multiple entries
kvn
parents: 1972
diff changeset
481 loop->dump_head();
9dc311b8473e 7008866: Missing loop predicate for loop with multiple entries
kvn
parents: 1972
diff changeset
482 }
9dc311b8473e 7008866: Missing loop predicate for loop with multiple entries
kvn
parents: 1972
diff changeset
483 #endif
2445
08eb13460b3a 7004535: Clone loop predicate during loop unswitch
kvn
parents: 2417
diff changeset
484 Node* head = loop->_head;
08eb13460b3a 7004535: Clone loop predicate during loop unswitch
kvn
parents: 2417
diff changeset
485 bool counted_loop = head->is_CountedLoop();
08eb13460b3a 7004535: Clone loop predicate during loop unswitch
kvn
parents: 2417
diff changeset
486 if (counted_loop) {
08eb13460b3a 7004535: Clone loop predicate during loop unswitch
kvn
parents: 2417
diff changeset
487 CountedLoopNode *cl = head->as_CountedLoop();
0
a61af66fc99e Initial load
duke
parents:
diff changeset
488 assert(cl->trip_count() > 0, "peeling a fully unrolled loop");
a61af66fc99e Initial load
duke
parents:
diff changeset
489 cl->set_trip_count(cl->trip_count() - 1);
2383
9dc311b8473e 7008866: Missing loop predicate for loop with multiple entries
kvn
parents: 1972
diff changeset
490 if (cl->is_main_loop()) {
0
a61af66fc99e Initial load
duke
parents:
diff changeset
491 cl->set_normal_loop();
a61af66fc99e Initial load
duke
parents:
diff changeset
492 #ifndef PRODUCT
2383
9dc311b8473e 7008866: Missing loop predicate for loop with multiple entries
kvn
parents: 1972
diff changeset
493 if (PrintOpto && VerifyLoopOptimizations) {
0
a61af66fc99e Initial load
duke
parents:
diff changeset
494 tty->print("Peeling a 'main' loop; resetting to 'normal' ");
a61af66fc99e Initial load
duke
parents:
diff changeset
495 loop->dump_head();
a61af66fc99e Initial load
duke
parents:
diff changeset
496 }
a61af66fc99e Initial load
duke
parents:
diff changeset
497 #endif
a61af66fc99e Initial load
duke
parents:
diff changeset
498 }
a61af66fc99e Initial load
duke
parents:
diff changeset
499 }
2445
08eb13460b3a 7004535: Clone loop predicate during loop unswitch
kvn
parents: 2417
diff changeset
500 Node* entry = head->in(LoopNode::EntryControl);
0
a61af66fc99e Initial load
duke
parents:
diff changeset
501
a61af66fc99e Initial load
duke
parents:
diff changeset
502 // Step 1: Clone the loop body. The clone becomes the peeled iteration.
a61af66fc99e Initial load
duke
parents:
diff changeset
503 // The pre-loop illegally has 2 control users (old & new loops).
2445
08eb13460b3a 7004535: Clone loop predicate during loop unswitch
kvn
parents: 2417
diff changeset
504 clone_loop( loop, old_new, dom_depth(head) );
0
a61af66fc99e Initial load
duke
parents:
diff changeset
505
a61af66fc99e Initial load
duke
parents:
diff changeset
506 // Step 2: Make the old-loop fall-in edges point to the peeled iteration.
a61af66fc99e Initial load
duke
parents:
diff changeset
507 // Do this by making the old-loop fall-in edges act as if they came
a61af66fc99e Initial load
duke
parents:
diff changeset
508 // around the loopback from the prior iteration (follow the old-loop
a61af66fc99e Initial load
duke
parents:
diff changeset
509 // backedges) and then map to the new peeled iteration. This leaves
a61af66fc99e Initial load
duke
parents:
diff changeset
510 // the pre-loop with only 1 user (the new peeled iteration), but the
a61af66fc99e Initial load
duke
parents:
diff changeset
511 // peeled-loop backedge has 2 users.
2445
08eb13460b3a 7004535: Clone loop predicate during loop unswitch
kvn
parents: 2417
diff changeset
512 Node* new_exit_value = old_new[head->in(LoopNode::LoopBackControl)->_idx];
08eb13460b3a 7004535: Clone loop predicate during loop unswitch
kvn
parents: 2417
diff changeset
513 new_exit_value = move_loop_predicates(entry, new_exit_value);
08eb13460b3a 7004535: Clone loop predicate during loop unswitch
kvn
parents: 2417
diff changeset
514 _igvn.hash_delete(head);
08eb13460b3a 7004535: Clone loop predicate during loop unswitch
kvn
parents: 2417
diff changeset
515 head->set_req(LoopNode::EntryControl, new_exit_value);
08eb13460b3a 7004535: Clone loop predicate during loop unswitch
kvn
parents: 2417
diff changeset
516 for (DUIterator_Fast jmax, j = head->fast_outs(jmax); j < jmax; j++) {
08eb13460b3a 7004535: Clone loop predicate during loop unswitch
kvn
parents: 2417
diff changeset
517 Node* old = head->fast_out(j);
08eb13460b3a 7004535: Clone loop predicate during loop unswitch
kvn
parents: 2417
diff changeset
518 if (old->in(0) == loop->_head && old->req() == 3 && old->is_Phi()) {
08eb13460b3a 7004535: Clone loop predicate during loop unswitch
kvn
parents: 2417
diff changeset
519 new_exit_value = old_new[old->in(LoopNode::LoopBackControl)->_idx];
08eb13460b3a 7004535: Clone loop predicate during loop unswitch
kvn
parents: 2417
diff changeset
520 if (!new_exit_value ) // Backedge value is ALSO loop invariant?
0
a61af66fc99e Initial load
duke
parents:
diff changeset
521 // Then loop body backedge value remains the same.
a61af66fc99e Initial load
duke
parents:
diff changeset
522 new_exit_value = old->in(LoopNode::LoopBackControl);
a61af66fc99e Initial load
duke
parents:
diff changeset
523 _igvn.hash_delete(old);
a61af66fc99e Initial load
duke
parents:
diff changeset
524 old->set_req(LoopNode::EntryControl, new_exit_value);
a61af66fc99e Initial load
duke
parents:
diff changeset
525 }
a61af66fc99e Initial load
duke
parents:
diff changeset
526 }
a61af66fc99e Initial load
duke
parents:
diff changeset
527
a61af66fc99e Initial load
duke
parents:
diff changeset
528
a61af66fc99e Initial load
duke
parents:
diff changeset
529 // Step 3: Cut the backedge on the clone (so its not a loop) and remove the
a61af66fc99e Initial load
duke
parents:
diff changeset
530 // extra backedge user.
2445
08eb13460b3a 7004535: Clone loop predicate during loop unswitch
kvn
parents: 2417
diff changeset
531 Node* new_head = old_new[head->_idx];
08eb13460b3a 7004535: Clone loop predicate during loop unswitch
kvn
parents: 2417
diff changeset
532 _igvn.hash_delete(new_head);
08eb13460b3a 7004535: Clone loop predicate during loop unswitch
kvn
parents: 2417
diff changeset
533 new_head->set_req(LoopNode::LoopBackControl, C->top());
08eb13460b3a 7004535: Clone loop predicate during loop unswitch
kvn
parents: 2417
diff changeset
534 for (DUIterator_Fast j2max, j2 = new_head->fast_outs(j2max); j2 < j2max; j2++) {
08eb13460b3a 7004535: Clone loop predicate during loop unswitch
kvn
parents: 2417
diff changeset
535 Node* use = new_head->fast_out(j2);
08eb13460b3a 7004535: Clone loop predicate during loop unswitch
kvn
parents: 2417
diff changeset
536 if (use->in(0) == new_head && use->req() == 3 && use->is_Phi()) {
0
a61af66fc99e Initial load
duke
parents:
diff changeset
537 _igvn.hash_delete(use);
a61af66fc99e Initial load
duke
parents:
diff changeset
538 use->set_req(LoopNode::LoopBackControl, C->top());
a61af66fc99e Initial load
duke
parents:
diff changeset
539 }
a61af66fc99e Initial load
duke
parents:
diff changeset
540 }
a61af66fc99e Initial load
duke
parents:
diff changeset
541
a61af66fc99e Initial load
duke
parents:
diff changeset
542
a61af66fc99e Initial load
duke
parents:
diff changeset
543 // Step 4: Correct dom-depth info. Set to loop-head depth.
2445
08eb13460b3a 7004535: Clone loop predicate during loop unswitch
kvn
parents: 2417
diff changeset
544 int dd = dom_depth(head);
08eb13460b3a 7004535: Clone loop predicate during loop unswitch
kvn
parents: 2417
diff changeset
545 set_idom(head, head->in(1), dd);
0
a61af66fc99e Initial load
duke
parents:
diff changeset
546 for (uint j3 = 0; j3 < loop->_body.size(); j3++) {
a61af66fc99e Initial load
duke
parents:
diff changeset
547 Node *old = loop->_body.at(j3);
a61af66fc99e Initial load
duke
parents:
diff changeset
548 Node *nnn = old_new[old->_idx];
a61af66fc99e Initial load
duke
parents:
diff changeset
549 if (!has_ctrl(nnn))
a61af66fc99e Initial load
duke
parents:
diff changeset
550 set_idom(nnn, idom(nnn), dd-1);
a61af66fc99e Initial load
duke
parents:
diff changeset
551 // While we're at it, remove any SafePoints from the peeled code
2445
08eb13460b3a 7004535: Clone loop predicate during loop unswitch
kvn
parents: 2417
diff changeset
552 if (old->Opcode() == Op_SafePoint) {
0
a61af66fc99e Initial load
duke
parents:
diff changeset
553 Node *nnn = old_new[old->_idx];
a61af66fc99e Initial load
duke
parents:
diff changeset
554 lazy_replace(nnn,nnn->in(TypeFunc::Control));
a61af66fc99e Initial load
duke
parents:
diff changeset
555 }
a61af66fc99e Initial load
duke
parents:
diff changeset
556 }
a61af66fc99e Initial load
duke
parents:
diff changeset
557
a61af66fc99e Initial load
duke
parents:
diff changeset
558 // Now force out all loop-invariant dominating tests. The optimizer
a61af66fc99e Initial load
duke
parents:
diff changeset
559 // finds some, but we _know_ they are all useless.
a61af66fc99e Initial load
duke
parents:
diff changeset
560 peeled_dom_test_elim(loop,old_new);
a61af66fc99e Initial load
duke
parents:
diff changeset
561
a61af66fc99e Initial load
duke
parents:
diff changeset
562 loop->record_for_igvn();
a61af66fc99e Initial load
duke
parents:
diff changeset
563 }
a61af66fc99e Initial load
duke
parents:
diff changeset
564
2453
d7a3fed1c1c9 7004547: regular loop unroll should not unroll more than max unrolling
kvn
parents: 2448
diff changeset
565 #define EMPTY_LOOP_SIZE 7 // number of nodes in an empty loop
d7a3fed1c1c9 7004547: regular loop unroll should not unroll more than max unrolling
kvn
parents: 2448
diff changeset
566
0
a61af66fc99e Initial load
duke
parents:
diff changeset
567 //------------------------------policy_maximally_unroll------------------------
2453
d7a3fed1c1c9 7004547: regular loop unroll should not unroll more than max unrolling
kvn
parents: 2448
diff changeset
568 // Calculate exact loop trip count and return true if loop can be maximally
d7a3fed1c1c9 7004547: regular loop unroll should not unroll more than max unrolling
kvn
parents: 2448
diff changeset
569 // unrolled.
0
a61af66fc99e Initial load
duke
parents:
diff changeset
570 bool IdealLoopTree::policy_maximally_unroll( PhaseIdealLoop *phase ) const {
a61af66fc99e Initial load
duke
parents:
diff changeset
571 CountedLoopNode *cl = _head->as_CountedLoop();
2412
f9424955eb18 7029152: Ideal nodes for String intrinsics miss memory edge optimization
kvn
parents: 2403
diff changeset
572 assert(cl->is_normal_loop(), "");
2453
d7a3fed1c1c9 7004547: regular loop unroll should not unroll more than max unrolling
kvn
parents: 2448
diff changeset
573 if (!cl->is_valid_counted_loop())
d7a3fed1c1c9 7004547: regular loop unroll should not unroll more than max unrolling
kvn
parents: 2448
diff changeset
574 return false; // Malformed counted loop
0
a61af66fc99e Initial load
duke
parents:
diff changeset
575
2465
3af54845df98 7004555: Add new policy for one iteration loops
kvn
parents: 2453
diff changeset
576 if (!cl->has_exact_trip_count()) {
3af54845df98 7004555: Add new policy for one iteration loops
kvn
parents: 2453
diff changeset
577 // Trip count is not exact.
0
a61af66fc99e Initial load
duke
parents:
diff changeset
578 return false;
a61af66fc99e Initial load
duke
parents:
diff changeset
579 }
a61af66fc99e Initial load
duke
parents:
diff changeset
580
2465
3af54845df98 7004555: Add new policy for one iteration loops
kvn
parents: 2453
diff changeset
581 uint trip_count = cl->trip_count();
3af54845df98 7004555: Add new policy for one iteration loops
kvn
parents: 2453
diff changeset
582 // Note, max_juint is used to indicate unknown trip count.
3af54845df98 7004555: Add new policy for one iteration loops
kvn
parents: 2453
diff changeset
583 assert(trip_count > 1, "one iteration loop should be optimized out already");
3af54845df98 7004555: Add new policy for one iteration loops
kvn
parents: 2453
diff changeset
584 assert(trip_count < max_juint, "exact trip_count should be less than max_uint.");
0
a61af66fc99e Initial load
duke
parents:
diff changeset
585
a61af66fc99e Initial load
duke
parents:
diff changeset
586 // Real policy: if we maximally unroll, does it get too big?
a61af66fc99e Initial load
duke
parents:
diff changeset
587 // Allow the unrolled mess to get larger than standard loop
a61af66fc99e Initial load
duke
parents:
diff changeset
588 // size. After all, it will no longer be a loop.
a61af66fc99e Initial load
duke
parents:
diff changeset
589 uint body_size = _body.size();
a61af66fc99e Initial load
duke
parents:
diff changeset
590 uint unroll_limit = (uint)LoopUnrollLimit * 4;
a61af66fc99e Initial load
duke
parents:
diff changeset
591 assert( (intx)unroll_limit == LoopUnrollLimit * 4, "LoopUnrollLimit must fit in 32bits");
2412
f9424955eb18 7029152: Ideal nodes for String intrinsics miss memory edge optimization
kvn
parents: 2403
diff changeset
592 if (trip_count > unroll_limit || body_size > unroll_limit) {
f9424955eb18 7029152: Ideal nodes for String intrinsics miss memory edge optimization
kvn
parents: 2403
diff changeset
593 return false;
f9424955eb18 7029152: Ideal nodes for String intrinsics miss memory edge optimization
kvn
parents: 2403
diff changeset
594 }
f9424955eb18 7029152: Ideal nodes for String intrinsics miss memory edge optimization
kvn
parents: 2403
diff changeset
595
2453
d7a3fed1c1c9 7004547: regular loop unroll should not unroll more than max unrolling
kvn
parents: 2448
diff changeset
596 // Take into account that after unroll conjoined heads and tails will fold,
d7a3fed1c1c9 7004547: regular loop unroll should not unroll more than max unrolling
kvn
parents: 2448
diff changeset
597 // otherwise policy_unroll() may allow more unrolling than max unrolling.
d7a3fed1c1c9 7004547: regular loop unroll should not unroll more than max unrolling
kvn
parents: 2448
diff changeset
598 uint new_body_size = EMPTY_LOOP_SIZE + (body_size - EMPTY_LOOP_SIZE) * trip_count;
d7a3fed1c1c9 7004547: regular loop unroll should not unroll more than max unrolling
kvn
parents: 2448
diff changeset
599 uint tst_body_size = (new_body_size - EMPTY_LOOP_SIZE) / trip_count + EMPTY_LOOP_SIZE;
d7a3fed1c1c9 7004547: regular loop unroll should not unroll more than max unrolling
kvn
parents: 2448
diff changeset
600 if (body_size != tst_body_size) // Check for int overflow
d7a3fed1c1c9 7004547: regular loop unroll should not unroll more than max unrolling
kvn
parents: 2448
diff changeset
601 return false;
d7a3fed1c1c9 7004547: regular loop unroll should not unroll more than max unrolling
kvn
parents: 2448
diff changeset
602 if (new_body_size > unroll_limit ||
d7a3fed1c1c9 7004547: regular loop unroll should not unroll more than max unrolling
kvn
parents: 2448
diff changeset
603 // Unrolling can result in a large amount of node construction
d7a3fed1c1c9 7004547: regular loop unroll should not unroll more than max unrolling
kvn
parents: 2448
diff changeset
604 new_body_size >= MaxNodeLimit - phase->C->unique()) {
d7a3fed1c1c9 7004547: regular loop unroll should not unroll more than max unrolling
kvn
parents: 2448
diff changeset
605 return false;
d7a3fed1c1c9 7004547: regular loop unroll should not unroll more than max unrolling
kvn
parents: 2448
diff changeset
606 }
d7a3fed1c1c9 7004547: regular loop unroll should not unroll more than max unrolling
kvn
parents: 2448
diff changeset
607
2417
cb162b348743 7032696: Fix for 7029152 broke VM
kvn
parents: 2412
diff changeset
608 // Currently we don't have policy to optimize one iteration loops.
cb162b348743 7032696: Fix for 7029152 broke VM
kvn
parents: 2412
diff changeset
609 // Maximally unrolling transformation is used for that:
cb162b348743 7032696: Fix for 7029152 broke VM
kvn
parents: 2412
diff changeset
610 // it is peeled and the original loop become non reachable (dead).
2453
d7a3fed1c1c9 7004547: regular loop unroll should not unroll more than max unrolling
kvn
parents: 2448
diff changeset
611 // Also fully unroll a loop with few iterations regardless next
d7a3fed1c1c9 7004547: regular loop unroll should not unroll more than max unrolling
kvn
parents: 2448
diff changeset
612 // conditions since following loop optimizations will split
d7a3fed1c1c9 7004547: regular loop unroll should not unroll more than max unrolling
kvn
parents: 2448
diff changeset
613 // such loop anyway (pre-main-post).
d7a3fed1c1c9 7004547: regular loop unroll should not unroll more than max unrolling
kvn
parents: 2448
diff changeset
614 if (trip_count <= 3)
2417
cb162b348743 7032696: Fix for 7029152 broke VM
kvn
parents: 2412
diff changeset
615 return true;
cb162b348743 7032696: Fix for 7029152 broke VM
kvn
parents: 2412
diff changeset
616
2412
f9424955eb18 7029152: Ideal nodes for String intrinsics miss memory edge optimization
kvn
parents: 2403
diff changeset
617 // Do not unroll a loop with String intrinsics code.
f9424955eb18 7029152: Ideal nodes for String intrinsics miss memory edge optimization
kvn
parents: 2403
diff changeset
618 // String intrinsics are large and have loops.
f9424955eb18 7029152: Ideal nodes for String intrinsics miss memory edge optimization
kvn
parents: 2403
diff changeset
619 for (uint k = 0; k < _body.size(); k++) {
f9424955eb18 7029152: Ideal nodes for String intrinsics miss memory edge optimization
kvn
parents: 2403
diff changeset
620 Node* n = _body.at(k);
f9424955eb18 7029152: Ideal nodes for String intrinsics miss memory edge optimization
kvn
parents: 2403
diff changeset
621 switch (n->Opcode()) {
f9424955eb18 7029152: Ideal nodes for String intrinsics miss memory edge optimization
kvn
parents: 2403
diff changeset
622 case Op_StrComp:
f9424955eb18 7029152: Ideal nodes for String intrinsics miss memory edge optimization
kvn
parents: 2403
diff changeset
623 case Op_StrEquals:
f9424955eb18 7029152: Ideal nodes for String intrinsics miss memory edge optimization
kvn
parents: 2403
diff changeset
624 case Op_StrIndexOf:
f9424955eb18 7029152: Ideal nodes for String intrinsics miss memory edge optimization
kvn
parents: 2403
diff changeset
625 case Op_AryEq: {
f9424955eb18 7029152: Ideal nodes for String intrinsics miss memory edge optimization
kvn
parents: 2403
diff changeset
626 return false;
f9424955eb18 7029152: Ideal nodes for String intrinsics miss memory edge optimization
kvn
parents: 2403
diff changeset
627 }
f9424955eb18 7029152: Ideal nodes for String intrinsics miss memory edge optimization
kvn
parents: 2403
diff changeset
628 } // switch
f9424955eb18 7029152: Ideal nodes for String intrinsics miss memory edge optimization
kvn
parents: 2403
diff changeset
629 }
f9424955eb18 7029152: Ideal nodes for String intrinsics miss memory edge optimization
kvn
parents: 2403
diff changeset
630
2453
d7a3fed1c1c9 7004547: regular loop unroll should not unroll more than max unrolling
kvn
parents: 2448
diff changeset
631 return true; // Do maximally unroll
0
a61af66fc99e Initial load
duke
parents:
diff changeset
632 }
a61af66fc99e Initial load
duke
parents:
diff changeset
633
a61af66fc99e Initial load
duke
parents:
diff changeset
634
3333
ae93231c7a1f 7039652: Performance regression after 7004547 changes
kvn
parents: 2468
diff changeset
635 #define MAX_UNROLL 16 // maximum number of unrolls for main loop
ae93231c7a1f 7039652: Performance regression after 7004547 changes
kvn
parents: 2468
diff changeset
636
0
a61af66fc99e Initial load
duke
parents:
diff changeset
637 //------------------------------policy_unroll----------------------------------
a61af66fc99e Initial load
duke
parents:
diff changeset
638 // Return TRUE or FALSE if the loop should be unrolled or not. Unroll if
a61af66fc99e Initial load
duke
parents:
diff changeset
639 // the loop is a CountedLoop and the body is small enough.
a61af66fc99e Initial load
duke
parents:
diff changeset
640 bool IdealLoopTree::policy_unroll( PhaseIdealLoop *phase ) const {
a61af66fc99e Initial load
duke
parents:
diff changeset
641
a61af66fc99e Initial load
duke
parents:
diff changeset
642 CountedLoopNode *cl = _head->as_CountedLoop();
2412
f9424955eb18 7029152: Ideal nodes for String intrinsics miss memory edge optimization
kvn
parents: 2403
diff changeset
643 assert(cl->is_normal_loop() || cl->is_main_loop(), "");
0
a61af66fc99e Initial load
duke
parents:
diff changeset
644
2453
d7a3fed1c1c9 7004547: regular loop unroll should not unroll more than max unrolling
kvn
parents: 2448
diff changeset
645 if (!cl->is_valid_counted_loop())
d7a3fed1c1c9 7004547: regular loop unroll should not unroll more than max unrolling
kvn
parents: 2448
diff changeset
646 return false; // Malformed counted loop
0
a61af66fc99e Initial load
duke
parents:
diff changeset
647
a61af66fc99e Initial load
duke
parents:
diff changeset
648 // protect against over-unrolling
2412
f9424955eb18 7029152: Ideal nodes for String intrinsics miss memory edge optimization
kvn
parents: 2403
diff changeset
649 if (cl->trip_count() <= 1) return false;
0
a61af66fc99e Initial load
duke
parents:
diff changeset
650
3333
ae93231c7a1f 7039652: Performance regression after 7004547 changes
kvn
parents: 2468
diff changeset
651 int future_unroll_ct = cl->unrolled_count() * 2;
ae93231c7a1f 7039652: Performance regression after 7004547 changes
kvn
parents: 2468
diff changeset
652 if (future_unroll_ct > MAX_UNROLL) return false;
2453
d7a3fed1c1c9 7004547: regular loop unroll should not unroll more than max unrolling
kvn
parents: 2448
diff changeset
653
3333
ae93231c7a1f 7039652: Performance regression after 7004547 changes
kvn
parents: 2468
diff changeset
654 // Check for initial stride being a small enough constant
ae93231c7a1f 7039652: Performance regression after 7004547 changes
kvn
parents: 2468
diff changeset
655 if (abs(cl->stride_con()) > (1<<2)*future_unroll_ct) return false;
0
a61af66fc99e Initial load
duke
parents:
diff changeset
656
a61af66fc99e Initial load
duke
parents:
diff changeset
657 // Don't unroll if the next round of unrolling would push us
a61af66fc99e Initial load
duke
parents:
diff changeset
658 // over the expected trip count of the loop. One is subtracted
a61af66fc99e Initial load
duke
parents:
diff changeset
659 // from the expected trip count because the pre-loop normally
a61af66fc99e Initial load
duke
parents:
diff changeset
660 // executes 1 iteration.
a61af66fc99e Initial load
duke
parents:
diff changeset
661 if (UnrollLimitForProfileCheck > 0 &&
a61af66fc99e Initial load
duke
parents:
diff changeset
662 cl->profile_trip_cnt() != COUNT_UNKNOWN &&
a61af66fc99e Initial load
duke
parents:
diff changeset
663 future_unroll_ct > UnrollLimitForProfileCheck &&
a61af66fc99e Initial load
duke
parents:
diff changeset
664 (float)future_unroll_ct > cl->profile_trip_cnt() - 1.0) {
a61af66fc99e Initial load
duke
parents:
diff changeset
665 return false;
a61af66fc99e Initial load
duke
parents:
diff changeset
666 }
a61af66fc99e Initial load
duke
parents:
diff changeset
667
a61af66fc99e Initial load
duke
parents:
diff changeset
668 // When unroll count is greater than LoopUnrollMin, don't unroll if:
a61af66fc99e Initial load
duke
parents:
diff changeset
669 // the residual iterations are more than 10% of the trip count
a61af66fc99e Initial load
duke
parents:
diff changeset
670 // and rounds of "unroll,optimize" are not making significant progress
a61af66fc99e Initial load
duke
parents:
diff changeset
671 // Progress defined as current size less than 20% larger than previous size.
a61af66fc99e Initial load
duke
parents:
diff changeset
672 if (UseSuperWord && cl->node_count_before_unroll() > 0 &&
a61af66fc99e Initial load
duke
parents:
diff changeset
673 future_unroll_ct > LoopUnrollMin &&
a61af66fc99e Initial load
duke
parents:
diff changeset
674 (future_unroll_ct - 1) * 10.0 > cl->profile_trip_cnt() &&
a61af66fc99e Initial load
duke
parents:
diff changeset
675 1.2 * cl->node_count_before_unroll() < (double)_body.size()) {
a61af66fc99e Initial load
duke
parents:
diff changeset
676 return false;
a61af66fc99e Initial load
duke
parents:
diff changeset
677 }
a61af66fc99e Initial load
duke
parents:
diff changeset
678
a61af66fc99e Initial load
duke
parents:
diff changeset
679 Node *init_n = cl->init_trip();
a61af66fc99e Initial load
duke
parents:
diff changeset
680 Node *limit_n = cl->limit();
a61af66fc99e Initial load
duke
parents:
diff changeset
681 // Non-constant bounds.
a61af66fc99e Initial load
duke
parents:
diff changeset
682 // Protect against over-unrolling when init or/and limit are not constant
a61af66fc99e Initial load
duke
parents:
diff changeset
683 // (so that trip_count's init value is maxint) but iv range is known.
2412
f9424955eb18 7029152: Ideal nodes for String intrinsics miss memory edge optimization
kvn
parents: 2403
diff changeset
684 if (init_n == NULL || !init_n->is_Con() ||
f9424955eb18 7029152: Ideal nodes for String intrinsics miss memory edge optimization
kvn
parents: 2403
diff changeset
685 limit_n == NULL || !limit_n->is_Con()) {
0
a61af66fc99e Initial load
duke
parents:
diff changeset
686 Node* phi = cl->phi();
2412
f9424955eb18 7029152: Ideal nodes for String intrinsics miss memory edge optimization
kvn
parents: 2403
diff changeset
687 if (phi != NULL) {
0
a61af66fc99e Initial load
duke
parents:
diff changeset
688 assert(phi->is_Phi() && phi->in(0) == _head, "Counted loop should have iv phi.");
a61af66fc99e Initial load
duke
parents:
diff changeset
689 const TypeInt* iv_type = phase->_igvn.type(phi)->is_int();
a61af66fc99e Initial load
duke
parents:
diff changeset
690 int next_stride = cl->stride_con() * 2; // stride after this unroll
2412
f9424955eb18 7029152: Ideal nodes for String intrinsics miss memory edge optimization
kvn
parents: 2403
diff changeset
691 if (next_stride > 0) {
f9424955eb18 7029152: Ideal nodes for String intrinsics miss memory edge optimization
kvn
parents: 2403
diff changeset
692 if (iv_type->_lo + next_stride <= iv_type->_lo || // overflow
f9424955eb18 7029152: Ideal nodes for String intrinsics miss memory edge optimization
kvn
parents: 2403
diff changeset
693 iv_type->_lo + next_stride > iv_type->_hi) {
0
a61af66fc99e Initial load
duke
parents:
diff changeset
694 return false; // over-unrolling
a61af66fc99e Initial load
duke
parents:
diff changeset
695 }
2412
f9424955eb18 7029152: Ideal nodes for String intrinsics miss memory edge optimization
kvn
parents: 2403
diff changeset
696 } else if (next_stride < 0) {
f9424955eb18 7029152: Ideal nodes for String intrinsics miss memory edge optimization
kvn
parents: 2403
diff changeset
697 if (iv_type->_hi + next_stride >= iv_type->_hi || // overflow
f9424955eb18 7029152: Ideal nodes for String intrinsics miss memory edge optimization
kvn
parents: 2403
diff changeset
698 iv_type->_hi + next_stride < iv_type->_lo) {
0
a61af66fc99e Initial load
duke
parents:
diff changeset
699 return false; // over-unrolling
a61af66fc99e Initial load
duke
parents:
diff changeset
700 }
a61af66fc99e Initial load
duke
parents:
diff changeset
701 }
a61af66fc99e Initial load
duke
parents:
diff changeset
702 }
a61af66fc99e Initial load
duke
parents:
diff changeset
703 }
a61af66fc99e Initial load
duke
parents:
diff changeset
704
a61af66fc99e Initial load
duke
parents:
diff changeset
705 // Adjust body_size to determine if we unroll or not
a61af66fc99e Initial load
duke
parents:
diff changeset
706 uint body_size = _body.size();
a61af66fc99e Initial load
duke
parents:
diff changeset
707 // Key test to unroll CaffeineMark's Logic test
a61af66fc99e Initial load
duke
parents:
diff changeset
708 int xors_in_loop = 0;
a61af66fc99e Initial load
duke
parents:
diff changeset
709 // Also count ModL, DivL and MulL which expand mightly
2412
f9424955eb18 7029152: Ideal nodes for String intrinsics miss memory edge optimization
kvn
parents: 2403
diff changeset
710 for (uint k = 0; k < _body.size(); k++) {
f9424955eb18 7029152: Ideal nodes for String intrinsics miss memory edge optimization
kvn
parents: 2403
diff changeset
711 Node* n = _body.at(k);
f9424955eb18 7029152: Ideal nodes for String intrinsics miss memory edge optimization
kvn
parents: 2403
diff changeset
712 switch (n->Opcode()) {
f9424955eb18 7029152: Ideal nodes for String intrinsics miss memory edge optimization
kvn
parents: 2403
diff changeset
713 case Op_XorI: xors_in_loop++; break; // CaffeineMark's Logic test
f9424955eb18 7029152: Ideal nodes for String intrinsics miss memory edge optimization
kvn
parents: 2403
diff changeset
714 case Op_ModL: body_size += 30; break;
f9424955eb18 7029152: Ideal nodes for String intrinsics miss memory edge optimization
kvn
parents: 2403
diff changeset
715 case Op_DivL: body_size += 30; break;
f9424955eb18 7029152: Ideal nodes for String intrinsics miss memory edge optimization
kvn
parents: 2403
diff changeset
716 case Op_MulL: body_size += 10; break;
f9424955eb18 7029152: Ideal nodes for String intrinsics miss memory edge optimization
kvn
parents: 2403
diff changeset
717 case Op_StrComp:
f9424955eb18 7029152: Ideal nodes for String intrinsics miss memory edge optimization
kvn
parents: 2403
diff changeset
718 case Op_StrEquals:
f9424955eb18 7029152: Ideal nodes for String intrinsics miss memory edge optimization
kvn
parents: 2403
diff changeset
719 case Op_StrIndexOf:
f9424955eb18 7029152: Ideal nodes for String intrinsics miss memory edge optimization
kvn
parents: 2403
diff changeset
720 case Op_AryEq: {
f9424955eb18 7029152: Ideal nodes for String intrinsics miss memory edge optimization
kvn
parents: 2403
diff changeset
721 // Do not unroll a loop with String intrinsics code.
f9424955eb18 7029152: Ideal nodes for String intrinsics miss memory edge optimization
kvn
parents: 2403
diff changeset
722 // String intrinsics are large and have loops.
f9424955eb18 7029152: Ideal nodes for String intrinsics miss memory edge optimization
kvn
parents: 2403
diff changeset
723 return false;
f9424955eb18 7029152: Ideal nodes for String intrinsics miss memory edge optimization
kvn
parents: 2403
diff changeset
724 }
f9424955eb18 7029152: Ideal nodes for String intrinsics miss memory edge optimization
kvn
parents: 2403
diff changeset
725 } // switch
0
a61af66fc99e Initial load
duke
parents:
diff changeset
726 }
a61af66fc99e Initial load
duke
parents:
diff changeset
727
a61af66fc99e Initial load
duke
parents:
diff changeset
728 // Check for being too big
2412
f9424955eb18 7029152: Ideal nodes for String intrinsics miss memory edge optimization
kvn
parents: 2403
diff changeset
729 if (body_size > (uint)LoopUnrollLimit) {
f9424955eb18 7029152: Ideal nodes for String intrinsics miss memory edge optimization
kvn
parents: 2403
diff changeset
730 if (xors_in_loop >= 4 && body_size < (uint)LoopUnrollLimit*4) return true;
0
a61af66fc99e Initial load
duke
parents:
diff changeset
731 // Normal case: loop too big
a61af66fc99e Initial load
duke
parents:
diff changeset
732 return false;
a61af66fc99e Initial load
duke
parents:
diff changeset
733 }
a61af66fc99e Initial load
duke
parents:
diff changeset
734
a61af66fc99e Initial load
duke
parents:
diff changeset
735 // Unroll once! (Each trip will soon do double iterations)
a61af66fc99e Initial load
duke
parents:
diff changeset
736 return true;
a61af66fc99e Initial load
duke
parents:
diff changeset
737 }
a61af66fc99e Initial load
duke
parents:
diff changeset
738
a61af66fc99e Initial load
duke
parents:
diff changeset
739 //------------------------------policy_align-----------------------------------
a61af66fc99e Initial load
duke
parents:
diff changeset
740 // Return TRUE or FALSE if the loop should be cache-line aligned. Gather the
a61af66fc99e Initial load
duke
parents:
diff changeset
741 // expression that does the alignment. Note that only one array base can be
605
98cb887364d3 6810672: Comment typos
twisti
parents: 420
diff changeset
742 // aligned in a loop (unless the VM guarantees mutual alignment). Note that
0
a61af66fc99e Initial load
duke
parents:
diff changeset
743 // if we vectorize short memory ops into longer memory ops, we may want to
a61af66fc99e Initial load
duke
parents:
diff changeset
744 // increase alignment.
a61af66fc99e Initial load
duke
parents:
diff changeset
745 bool IdealLoopTree::policy_align( PhaseIdealLoop *phase ) const {
a61af66fc99e Initial load
duke
parents:
diff changeset
746 return false;
a61af66fc99e Initial load
duke
parents:
diff changeset
747 }
a61af66fc99e Initial load
duke
parents:
diff changeset
748
a61af66fc99e Initial load
duke
parents:
diff changeset
749 //------------------------------policy_range_check-----------------------------
a61af66fc99e Initial load
duke
parents:
diff changeset
750 // Return TRUE or FALSE if the loop should be range-check-eliminated.
a61af66fc99e Initial load
duke
parents:
diff changeset
751 // Actually we do iteration-splitting, a more powerful form of RCE.
a61af66fc99e Initial load
duke
parents:
diff changeset
752 bool IdealLoopTree::policy_range_check( PhaseIdealLoop *phase ) const {
a61af66fc99e Initial load
duke
parents:
diff changeset
753 if( !RangeCheckElimination ) return false;
a61af66fc99e Initial load
duke
parents:
diff changeset
754
a61af66fc99e Initial load
duke
parents:
diff changeset
755 CountedLoopNode *cl = _head->as_CountedLoop();
a61af66fc99e Initial load
duke
parents:
diff changeset
756 // If we unrolled with no intention of doing RCE and we later
a61af66fc99e Initial load
duke
parents:
diff changeset
757 // changed our minds, we got no pre-loop. Either we need to
a61af66fc99e Initial load
duke
parents:
diff changeset
758 // make a new pre-loop, or we gotta disallow RCE.
a61af66fc99e Initial load
duke
parents:
diff changeset
759 if( cl->is_main_no_pre_loop() ) return false; // Disallowed for now.
a61af66fc99e Initial load
duke
parents:
diff changeset
760 Node *trip_counter = cl->phi();
a61af66fc99e Initial load
duke
parents:
diff changeset
761
a61af66fc99e Initial load
duke
parents:
diff changeset
762 // Check loop body for tests of trip-counter plus loop-invariant vs
a61af66fc99e Initial load
duke
parents:
diff changeset
763 // loop-invariant.
a61af66fc99e Initial load
duke
parents:
diff changeset
764 for( uint i = 0; i < _body.size(); i++ ) {
a61af66fc99e Initial load
duke
parents:
diff changeset
765 Node *iff = _body[i];
a61af66fc99e Initial load
duke
parents:
diff changeset
766 if( iff->Opcode() == Op_If ) { // Test?
a61af66fc99e Initial load
duke
parents:
diff changeset
767
a61af66fc99e Initial load
duke
parents:
diff changeset
768 // Comparing trip+off vs limit
a61af66fc99e Initial load
duke
parents:
diff changeset
769 Node *bol = iff->in(1);
a61af66fc99e Initial load
duke
parents:
diff changeset
770 if( bol->req() != 2 ) continue; // dead constant test
1172
b2b6a9bf6238 6894779: Loop Predication for Loop Optimizer in C2
cfang
parents: 844
diff changeset
771 if (!bol->is_Bool()) {
b2b6a9bf6238 6894779: Loop Predication for Loop Optimizer in C2
cfang
parents: 844
diff changeset
772 assert(UseLoopPredicate && bol->Opcode() == Op_Conv2B, "predicate check only");
b2b6a9bf6238 6894779: Loop Predication for Loop Optimizer in C2
cfang
parents: 844
diff changeset
773 continue;
b2b6a9bf6238 6894779: Loop Predication for Loop Optimizer in C2
cfang
parents: 844
diff changeset
774 }
0
a61af66fc99e Initial load
duke
parents:
diff changeset
775 Node *cmp = bol->in(1);
a61af66fc99e Initial load
duke
parents:
diff changeset
776
a61af66fc99e Initial load
duke
parents:
diff changeset
777 Node *rc_exp = cmp->in(1);
a61af66fc99e Initial load
duke
parents:
diff changeset
778 Node *limit = cmp->in(2);
a61af66fc99e Initial load
duke
parents:
diff changeset
779
a61af66fc99e Initial load
duke
parents:
diff changeset
780 Node *limit_c = phase->get_ctrl(limit);
a61af66fc99e Initial load
duke
parents:
diff changeset
781 if( limit_c == phase->C->top() )
a61af66fc99e Initial load
duke
parents:
diff changeset
782 return false; // Found dead test on live IF? No RCE!
a61af66fc99e Initial load
duke
parents:
diff changeset
783 if( is_member(phase->get_loop(limit_c) ) ) {
a61af66fc99e Initial load
duke
parents:
diff changeset
784 // Compare might have operands swapped; commute them
a61af66fc99e Initial load
duke
parents:
diff changeset
785 rc_exp = cmp->in(2);
a61af66fc99e Initial load
duke
parents:
diff changeset
786 limit = cmp->in(1);
a61af66fc99e Initial load
duke
parents:
diff changeset
787 limit_c = phase->get_ctrl(limit);
a61af66fc99e Initial load
duke
parents:
diff changeset
788 if( is_member(phase->get_loop(limit_c) ) )
a61af66fc99e Initial load
duke
parents:
diff changeset
789 continue; // Both inputs are loop varying; cannot RCE
a61af66fc99e Initial load
duke
parents:
diff changeset
790 }
a61af66fc99e Initial load
duke
parents:
diff changeset
791
a61af66fc99e Initial load
duke
parents:
diff changeset
792 if (!phase->is_scaled_iv_plus_offset(rc_exp, trip_counter, NULL, NULL)) {
a61af66fc99e Initial load
duke
parents:
diff changeset
793 continue;
a61af66fc99e Initial load
duke
parents:
diff changeset
794 }
a61af66fc99e Initial load
duke
parents:
diff changeset
795 // Yeah! Found a test like 'trip+off vs limit'
a61af66fc99e Initial load
duke
parents:
diff changeset
796 // Test is an IfNode, has 2 projections. If BOTH are in the loop
a61af66fc99e Initial load
duke
parents:
diff changeset
797 // we need loop unswitching instead of iteration splitting.
a61af66fc99e Initial load
duke
parents:
diff changeset
798 if( is_loop_exit(iff) )
a61af66fc99e Initial load
duke
parents:
diff changeset
799 return true; // Found reason to split iterations
a61af66fc99e Initial load
duke
parents:
diff changeset
800 } // End of is IF
a61af66fc99e Initial load
duke
parents:
diff changeset
801 }
a61af66fc99e Initial load
duke
parents:
diff changeset
802
a61af66fc99e Initial load
duke
parents:
diff changeset
803 return false;
a61af66fc99e Initial load
duke
parents:
diff changeset
804 }
a61af66fc99e Initial load
duke
parents:
diff changeset
805
a61af66fc99e Initial load
duke
parents:
diff changeset
806 //------------------------------policy_peel_only-------------------------------
a61af66fc99e Initial load
duke
parents:
diff changeset
807 // Return TRUE or FALSE if the loop should NEVER be RCE'd or aligned. Useful
a61af66fc99e Initial load
duke
parents:
diff changeset
808 // for unrolling loops with NO array accesses.
a61af66fc99e Initial load
duke
parents:
diff changeset
809 bool IdealLoopTree::policy_peel_only( PhaseIdealLoop *phase ) const {
a61af66fc99e Initial load
duke
parents:
diff changeset
810
a61af66fc99e Initial load
duke
parents:
diff changeset
811 for( uint i = 0; i < _body.size(); i++ )
a61af66fc99e Initial load
duke
parents:
diff changeset
812 if( _body[i]->is_Mem() )
a61af66fc99e Initial load
duke
parents:
diff changeset
813 return false;
a61af66fc99e Initial load
duke
parents:
diff changeset
814
a61af66fc99e Initial load
duke
parents:
diff changeset
815 // No memory accesses at all!
a61af66fc99e Initial load
duke
parents:
diff changeset
816 return true;
a61af66fc99e Initial load
duke
parents:
diff changeset
817 }
a61af66fc99e Initial load
duke
parents:
diff changeset
818
a61af66fc99e Initial load
duke
parents:
diff changeset
819 //------------------------------clone_up_backedge_goo--------------------------
a61af66fc99e Initial load
duke
parents:
diff changeset
820 // If Node n lives in the back_ctrl block and cannot float, we clone a private
a61af66fc99e Initial load
duke
parents:
diff changeset
821 // version of n in preheader_ctrl block and return that, otherwise return n.
a61af66fc99e Initial load
duke
parents:
diff changeset
822 Node *PhaseIdealLoop::clone_up_backedge_goo( Node *back_ctrl, Node *preheader_ctrl, Node *n ) {
a61af66fc99e Initial load
duke
parents:
diff changeset
823 if( get_ctrl(n) != back_ctrl ) return n;
a61af66fc99e Initial load
duke
parents:
diff changeset
824
a61af66fc99e Initial load
duke
parents:
diff changeset
825 Node *x = NULL; // If required, a clone of 'n'
a61af66fc99e Initial load
duke
parents:
diff changeset
826 // Check for 'n' being pinned in the backedge.
a61af66fc99e Initial load
duke
parents:
diff changeset
827 if( n->in(0) && n->in(0) == back_ctrl ) {
a61af66fc99e Initial load
duke
parents:
diff changeset
828 x = n->clone(); // Clone a copy of 'n' to preheader
a61af66fc99e Initial load
duke
parents:
diff changeset
829 x->set_req( 0, preheader_ctrl ); // Fix x's control input to preheader
a61af66fc99e Initial load
duke
parents:
diff changeset
830 }
a61af66fc99e Initial load
duke
parents:
diff changeset
831
a61af66fc99e Initial load
duke
parents:
diff changeset
832 // Recursive fixup any other input edges into x.
a61af66fc99e Initial load
duke
parents:
diff changeset
833 // If there are no changes we can just return 'n', otherwise
a61af66fc99e Initial load
duke
parents:
diff changeset
834 // we need to clone a private copy and change it.
a61af66fc99e Initial load
duke
parents:
diff changeset
835 for( uint i = 1; i < n->req(); i++ ) {
a61af66fc99e Initial load
duke
parents:
diff changeset
836 Node *g = clone_up_backedge_goo( back_ctrl, preheader_ctrl, n->in(i) );
a61af66fc99e Initial load
duke
parents:
diff changeset
837 if( g != n->in(i) ) {
a61af66fc99e Initial load
duke
parents:
diff changeset
838 if( !x )
a61af66fc99e Initial load
duke
parents:
diff changeset
839 x = n->clone();
a61af66fc99e Initial load
duke
parents:
diff changeset
840 x->set_req(i, g);
a61af66fc99e Initial load
duke
parents:
diff changeset
841 }
a61af66fc99e Initial load
duke
parents:
diff changeset
842 }
a61af66fc99e Initial load
duke
parents:
diff changeset
843 if( x ) { // x can legally float to pre-header location
a61af66fc99e Initial load
duke
parents:
diff changeset
844 register_new_node( x, preheader_ctrl );
a61af66fc99e Initial load
duke
parents:
diff changeset
845 return x;
a61af66fc99e Initial load
duke
parents:
diff changeset
846 } else { // raise n to cover LCA of uses
a61af66fc99e Initial load
duke
parents:
diff changeset
847 set_ctrl( n, find_non_split_ctrl(back_ctrl->in(0)) );
a61af66fc99e Initial load
duke
parents:
diff changeset
848 }
a61af66fc99e Initial load
duke
parents:
diff changeset
849 return n;
a61af66fc99e Initial load
duke
parents:
diff changeset
850 }
a61af66fc99e Initial load
duke
parents:
diff changeset
851
a61af66fc99e Initial load
duke
parents:
diff changeset
852 //------------------------------insert_pre_post_loops--------------------------
a61af66fc99e Initial load
duke
parents:
diff changeset
853 // Insert pre and post loops. If peel_only is set, the pre-loop can not have
a61af66fc99e Initial load
duke
parents:
diff changeset
854 // more iterations added. It acts as a 'peel' only, no lower-bound RCE, no
a61af66fc99e Initial load
duke
parents:
diff changeset
855 // alignment. Useful to unroll loops that do no array accesses.
a61af66fc99e Initial load
duke
parents:
diff changeset
856 void PhaseIdealLoop::insert_pre_post_loops( IdealLoopTree *loop, Node_List &old_new, bool peel_only ) {
a61af66fc99e Initial load
duke
parents:
diff changeset
857
2383
9dc311b8473e 7008866: Missing loop predicate for loop with multiple entries
kvn
parents: 1972
diff changeset
858 #ifndef PRODUCT
9dc311b8473e 7008866: Missing loop predicate for loop with multiple entries
kvn
parents: 1972
diff changeset
859 if (TraceLoopOpts) {
9dc311b8473e 7008866: Missing loop predicate for loop with multiple entries
kvn
parents: 1972
diff changeset
860 if (peel_only)
9dc311b8473e 7008866: Missing loop predicate for loop with multiple entries
kvn
parents: 1972
diff changeset
861 tty->print("PeelMainPost ");
9dc311b8473e 7008866: Missing loop predicate for loop with multiple entries
kvn
parents: 1972
diff changeset
862 else
9dc311b8473e 7008866: Missing loop predicate for loop with multiple entries
kvn
parents: 1972
diff changeset
863 tty->print("PreMainPost ");
9dc311b8473e 7008866: Missing loop predicate for loop with multiple entries
kvn
parents: 1972
diff changeset
864 loop->dump_head();
9dc311b8473e 7008866: Missing loop predicate for loop with multiple entries
kvn
parents: 1972
diff changeset
865 }
9dc311b8473e 7008866: Missing loop predicate for loop with multiple entries
kvn
parents: 1972
diff changeset
866 #endif
0
a61af66fc99e Initial load
duke
parents:
diff changeset
867 C->set_major_progress();
a61af66fc99e Initial load
duke
parents:
diff changeset
868
a61af66fc99e Initial load
duke
parents:
diff changeset
869 // Find common pieces of the loop being guarded with pre & post loops
a61af66fc99e Initial load
duke
parents:
diff changeset
870 CountedLoopNode *main_head = loop->_head->as_CountedLoop();
a61af66fc99e Initial load
duke
parents:
diff changeset
871 assert( main_head->is_normal_loop(), "" );
a61af66fc99e Initial load
duke
parents:
diff changeset
872 CountedLoopEndNode *main_end = main_head->loopexit();
a61af66fc99e Initial load
duke
parents:
diff changeset
873 assert( main_end->outcnt() == 2, "1 true, 1 false path only" );
a61af66fc99e Initial load
duke
parents:
diff changeset
874 uint dd_main_head = dom_depth(main_head);
a61af66fc99e Initial load
duke
parents:
diff changeset
875 uint max = main_head->outcnt();
a61af66fc99e Initial load
duke
parents:
diff changeset
876
a61af66fc99e Initial load
duke
parents:
diff changeset
877 Node *pre_header= main_head->in(LoopNode::EntryControl);
a61af66fc99e Initial load
duke
parents:
diff changeset
878 Node *init = main_head->init_trip();
a61af66fc99e Initial load
duke
parents:
diff changeset
879 Node *incr = main_end ->incr();
a61af66fc99e Initial load
duke
parents:
diff changeset
880 Node *limit = main_end ->limit();
a61af66fc99e Initial load
duke
parents:
diff changeset
881 Node *stride = main_end ->stride();
a61af66fc99e Initial load
duke
parents:
diff changeset
882 Node *cmp = main_end ->cmp_node();
a61af66fc99e Initial load
duke
parents:
diff changeset
883 BoolTest::mask b_test = main_end->test_trip();
a61af66fc99e Initial load
duke
parents:
diff changeset
884
a61af66fc99e Initial load
duke
parents:
diff changeset
885 // Need only 1 user of 'bol' because I will be hacking the loop bounds.
a61af66fc99e Initial load
duke
parents:
diff changeset
886 Node *bol = main_end->in(CountedLoopEndNode::TestValue);
a61af66fc99e Initial load
duke
parents:
diff changeset
887 if( bol->outcnt() != 1 ) {
a61af66fc99e Initial load
duke
parents:
diff changeset
888 bol = bol->clone();
a61af66fc99e Initial load
duke
parents:
diff changeset
889 register_new_node(bol,main_end->in(CountedLoopEndNode::TestControl));
a61af66fc99e Initial load
duke
parents:
diff changeset
890 _igvn.hash_delete(main_end);
a61af66fc99e Initial load
duke
parents:
diff changeset
891 main_end->set_req(CountedLoopEndNode::TestValue, bol);
a61af66fc99e Initial load
duke
parents:
diff changeset
892 }
a61af66fc99e Initial load
duke
parents:
diff changeset
893 // Need only 1 user of 'cmp' because I will be hacking the loop bounds.
a61af66fc99e Initial load
duke
parents:
diff changeset
894 if( cmp->outcnt() != 1 ) {
a61af66fc99e Initial load
duke
parents:
diff changeset
895 cmp = cmp->clone();
a61af66fc99e Initial load
duke
parents:
diff changeset
896 register_new_node(cmp,main_end->in(CountedLoopEndNode::TestControl));
a61af66fc99e Initial load
duke
parents:
diff changeset
897 _igvn.hash_delete(bol);
a61af66fc99e Initial load
duke
parents:
diff changeset
898 bol->set_req(1, cmp);
a61af66fc99e Initial load
duke
parents:
diff changeset
899 }
a61af66fc99e Initial load
duke
parents:
diff changeset
900
a61af66fc99e Initial load
duke
parents:
diff changeset
901 //------------------------------
a61af66fc99e Initial load
duke
parents:
diff changeset
902 // Step A: Create Post-Loop.
a61af66fc99e Initial load
duke
parents:
diff changeset
903 Node* main_exit = main_end->proj_out(false);
a61af66fc99e Initial load
duke
parents:
diff changeset
904 assert( main_exit->Opcode() == Op_IfFalse, "" );
a61af66fc99e Initial load
duke
parents:
diff changeset
905 int dd_main_exit = dom_depth(main_exit);
a61af66fc99e Initial load
duke
parents:
diff changeset
906
a61af66fc99e Initial load
duke
parents:
diff changeset
907 // Step A1: Clone the loop body. The clone becomes the post-loop. The main
a61af66fc99e Initial load
duke
parents:
diff changeset
908 // loop pre-header illegally has 2 control users (old & new loops).
a61af66fc99e Initial load
duke
parents:
diff changeset
909 clone_loop( loop, old_new, dd_main_exit );
a61af66fc99e Initial load
duke
parents:
diff changeset
910 assert( old_new[main_end ->_idx]->Opcode() == Op_CountedLoopEnd, "" );
a61af66fc99e Initial load
duke
parents:
diff changeset
911 CountedLoopNode *post_head = old_new[main_head->_idx]->as_CountedLoop();
a61af66fc99e Initial load
duke
parents:
diff changeset
912 post_head->set_post_loop(main_head);
a61af66fc99e Initial load
duke
parents:
diff changeset
913
400
cc80376deb0c 6667595: Set probability FAIR for pre-, post- loops and ALWAYS for main loop
kvn
parents: 367
diff changeset
914 // Reduce the post-loop trip count.
cc80376deb0c 6667595: Set probability FAIR for pre-, post- loops and ALWAYS for main loop
kvn
parents: 367
diff changeset
915 CountedLoopEndNode* post_end = old_new[main_end ->_idx]->as_CountedLoopEnd();
cc80376deb0c 6667595: Set probability FAIR for pre-, post- loops and ALWAYS for main loop
kvn
parents: 367
diff changeset
916 post_end->_prob = PROB_FAIR;
cc80376deb0c 6667595: Set probability FAIR for pre-, post- loops and ALWAYS for main loop
kvn
parents: 367
diff changeset
917
0
a61af66fc99e Initial load
duke
parents:
diff changeset
918 // Build the main-loop normal exit.
a61af66fc99e Initial load
duke
parents:
diff changeset
919 IfFalseNode *new_main_exit = new (C, 1) IfFalseNode(main_end);
a61af66fc99e Initial load
duke
parents:
diff changeset
920 _igvn.register_new_node_with_optimizer( new_main_exit );
a61af66fc99e Initial load
duke
parents:
diff changeset
921 set_idom(new_main_exit, main_end, dd_main_exit );
a61af66fc99e Initial load
duke
parents:
diff changeset
922 set_loop(new_main_exit, loop->_parent);
a61af66fc99e Initial load
duke
parents:
diff changeset
923
a61af66fc99e Initial load
duke
parents:
diff changeset
924 // Step A2: Build a zero-trip guard for the post-loop. After leaving the
a61af66fc99e Initial load
duke
parents:
diff changeset
925 // main-loop, the post-loop may not execute at all. We 'opaque' the incr
a61af66fc99e Initial load
duke
parents:
diff changeset
926 // (the main-loop trip-counter exit value) because we will be changing
a61af66fc99e Initial load
duke
parents:
diff changeset
927 // the exit value (via unrolling) so we cannot constant-fold away the zero
a61af66fc99e Initial load
duke
parents:
diff changeset
928 // trip guard until all unrolling is done.
216
8d191a7697e2 6715633: when matching a memory node the adr_type should not change
kvn
parents: 113
diff changeset
929 Node *zer_opaq = new (C, 2) Opaque1Node(C, incr);
0
a61af66fc99e Initial load
duke
parents:
diff changeset
930 Node *zer_cmp = new (C, 3) CmpINode( zer_opaq, limit );
a61af66fc99e Initial load
duke
parents:
diff changeset
931 Node *zer_bol = new (C, 2) BoolNode( zer_cmp, b_test );
a61af66fc99e Initial load
duke
parents:
diff changeset
932 register_new_node( zer_opaq, new_main_exit );
a61af66fc99e Initial load
duke
parents:
diff changeset
933 register_new_node( zer_cmp , new_main_exit );
a61af66fc99e Initial load
duke
parents:
diff changeset
934 register_new_node( zer_bol , new_main_exit );
a61af66fc99e Initial load
duke
parents:
diff changeset
935
a61af66fc99e Initial load
duke
parents:
diff changeset
936 // Build the IfNode
a61af66fc99e Initial load
duke
parents:
diff changeset
937 IfNode *zer_iff = new (C, 2) IfNode( new_main_exit, zer_bol, PROB_FAIR, COUNT_UNKNOWN );
a61af66fc99e Initial load
duke
parents:
diff changeset
938 _igvn.register_new_node_with_optimizer( zer_iff );
a61af66fc99e Initial load
duke
parents:
diff changeset
939 set_idom(zer_iff, new_main_exit, dd_main_exit);
a61af66fc99e Initial load
duke
parents:
diff changeset
940 set_loop(zer_iff, loop->_parent);
a61af66fc99e Initial load
duke
parents:
diff changeset
941
a61af66fc99e Initial load
duke
parents:
diff changeset
942 // Plug in the false-path, taken if we need to skip post-loop
a61af66fc99e Initial load
duke
parents:
diff changeset
943 _igvn.hash_delete( main_exit );
a61af66fc99e Initial load
duke
parents:
diff changeset
944 main_exit->set_req(0, zer_iff);
a61af66fc99e Initial load
duke
parents:
diff changeset
945 _igvn._worklist.push(main_exit);
a61af66fc99e Initial load
duke
parents:
diff changeset
946 set_idom(main_exit, zer_iff, dd_main_exit);
a61af66fc99e Initial load
duke
parents:
diff changeset
947 set_idom(main_exit->unique_out(), zer_iff, dd_main_exit);
a61af66fc99e Initial load
duke
parents:
diff changeset
948 // Make the true-path, must enter the post loop
a61af66fc99e Initial load
duke
parents:
diff changeset
949 Node *zer_taken = new (C, 1) IfTrueNode( zer_iff );
a61af66fc99e Initial load
duke
parents:
diff changeset
950 _igvn.register_new_node_with_optimizer( zer_taken );
a61af66fc99e Initial load
duke
parents:
diff changeset
951 set_idom(zer_taken, zer_iff, dd_main_exit);
a61af66fc99e Initial load
duke
parents:
diff changeset
952 set_loop(zer_taken, loop->_parent);
a61af66fc99e Initial load
duke
parents:
diff changeset
953 // Plug in the true path
a61af66fc99e Initial load
duke
parents:
diff changeset
954 _igvn.hash_delete( post_head );
a61af66fc99e Initial load
duke
parents:
diff changeset
955 post_head->set_req(LoopNode::EntryControl, zer_taken);
a61af66fc99e Initial load
duke
parents:
diff changeset
956 set_idom(post_head, zer_taken, dd_main_exit);
a61af66fc99e Initial load
duke
parents:
diff changeset
957
a61af66fc99e Initial load
duke
parents:
diff changeset
958 // Step A3: Make the fall-in values to the post-loop come from the
a61af66fc99e Initial load
duke
parents:
diff changeset
959 // fall-out values of the main-loop.
a61af66fc99e Initial load
duke
parents:
diff changeset
960 for (DUIterator_Fast imax, i = main_head->fast_outs(imax); i < imax; i++) {
a61af66fc99e Initial load
duke
parents:
diff changeset
961 Node* main_phi = main_head->fast_out(i);
a61af66fc99e Initial load
duke
parents:
diff changeset
962 if( main_phi->is_Phi() && main_phi->in(0) == main_head && main_phi->outcnt() >0 ) {
a61af66fc99e Initial load
duke
parents:
diff changeset
963 Node *post_phi = old_new[main_phi->_idx];
a61af66fc99e Initial load
duke
parents:
diff changeset
964 Node *fallmain = clone_up_backedge_goo(main_head->back_control(),
a61af66fc99e Initial load
duke
parents:
diff changeset
965 post_head->init_control(),
a61af66fc99e Initial load
duke
parents:
diff changeset
966 main_phi->in(LoopNode::LoopBackControl));
a61af66fc99e Initial load
duke
parents:
diff changeset
967 _igvn.hash_delete(post_phi);
a61af66fc99e Initial load
duke
parents:
diff changeset
968 post_phi->set_req( LoopNode::EntryControl, fallmain );
a61af66fc99e Initial load
duke
parents:
diff changeset
969 }
a61af66fc99e Initial load
duke
parents:
diff changeset
970 }
a61af66fc99e Initial load
duke
parents:
diff changeset
971
a61af66fc99e Initial load
duke
parents:
diff changeset
972 // Update local caches for next stanza
a61af66fc99e Initial load
duke
parents:
diff changeset
973 main_exit = new_main_exit;
a61af66fc99e Initial load
duke
parents:
diff changeset
974
a61af66fc99e Initial load
duke
parents:
diff changeset
975
a61af66fc99e Initial load
duke
parents:
diff changeset
976 //------------------------------
a61af66fc99e Initial load
duke
parents:
diff changeset
977 // Step B: Create Pre-Loop.
a61af66fc99e Initial load
duke
parents:
diff changeset
978
a61af66fc99e Initial load
duke
parents:
diff changeset
979 // Step B1: Clone the loop body. The clone becomes the pre-loop. The main
a61af66fc99e Initial load
duke
parents:
diff changeset
980 // loop pre-header illegally has 2 control users (old & new loops).
a61af66fc99e Initial load
duke
parents:
diff changeset
981 clone_loop( loop, old_new, dd_main_head );
a61af66fc99e Initial load
duke
parents:
diff changeset
982 CountedLoopNode* pre_head = old_new[main_head->_idx]->as_CountedLoop();
a61af66fc99e Initial load
duke
parents:
diff changeset
983 CountedLoopEndNode* pre_end = old_new[main_end ->_idx]->as_CountedLoopEnd();
a61af66fc99e Initial load
duke
parents:
diff changeset
984 pre_head->set_pre_loop(main_head);
a61af66fc99e Initial load
duke
parents:
diff changeset
985 Node *pre_incr = old_new[incr->_idx];
a61af66fc99e Initial load
duke
parents:
diff changeset
986
400
cc80376deb0c 6667595: Set probability FAIR for pre-, post- loops and ALWAYS for main loop
kvn
parents: 367
diff changeset
987 // Reduce the pre-loop trip count.
cc80376deb0c 6667595: Set probability FAIR for pre-, post- loops and ALWAYS for main loop
kvn
parents: 367
diff changeset
988 pre_end->_prob = PROB_FAIR;
cc80376deb0c 6667595: Set probability FAIR for pre-, post- loops and ALWAYS for main loop
kvn
parents: 367
diff changeset
989
0
a61af66fc99e Initial load
duke
parents:
diff changeset
990 // Find the pre-loop normal exit.
a61af66fc99e Initial load
duke
parents:
diff changeset
991 Node* pre_exit = pre_end->proj_out(false);
a61af66fc99e Initial load
duke
parents:
diff changeset
992 assert( pre_exit->Opcode() == Op_IfFalse, "" );
a61af66fc99e Initial load
duke
parents:
diff changeset
993 IfFalseNode *new_pre_exit = new (C, 1) IfFalseNode(pre_end);
a61af66fc99e Initial load
duke
parents:
diff changeset
994 _igvn.register_new_node_with_optimizer( new_pre_exit );
a61af66fc99e Initial load
duke
parents:
diff changeset
995 set_idom(new_pre_exit, pre_end, dd_main_head);
a61af66fc99e Initial load
duke
parents:
diff changeset
996 set_loop(new_pre_exit, loop->_parent);
a61af66fc99e Initial load
duke
parents:
diff changeset
997
a61af66fc99e Initial load
duke
parents:
diff changeset
998 // Step B2: Build a zero-trip guard for the main-loop. After leaving the
a61af66fc99e Initial load
duke
parents:
diff changeset
999 // pre-loop, the main-loop may not execute at all. Later in life this
a61af66fc99e Initial load
duke
parents:
diff changeset
1000 // zero-trip guard will become the minimum-trip guard when we unroll
a61af66fc99e Initial load
duke
parents:
diff changeset
1001 // the main-loop.
216
8d191a7697e2 6715633: when matching a memory node the adr_type should not change
kvn
parents: 113
diff changeset
1002 Node *min_opaq = new (C, 2) Opaque1Node(C, limit);
0
a61af66fc99e Initial load
duke
parents:
diff changeset
1003 Node *min_cmp = new (C, 3) CmpINode( pre_incr, min_opaq );
a61af66fc99e Initial load
duke
parents:
diff changeset
1004 Node *min_bol = new (C, 2) BoolNode( min_cmp, b_test );
a61af66fc99e Initial load
duke
parents:
diff changeset
1005 register_new_node( min_opaq, new_pre_exit );
a61af66fc99e Initial load
duke
parents:
diff changeset
1006 register_new_node( min_cmp , new_pre_exit );
a61af66fc99e Initial load
duke
parents:
diff changeset
1007 register_new_node( min_bol , new_pre_exit );
a61af66fc99e Initial load
duke
parents:
diff changeset
1008
400
cc80376deb0c 6667595: Set probability FAIR for pre-, post- loops and ALWAYS for main loop
kvn
parents: 367
diff changeset
1009 // Build the IfNode (assume the main-loop is executed always).
cc80376deb0c 6667595: Set probability FAIR for pre-, post- loops and ALWAYS for main loop
kvn
parents: 367
diff changeset
1010 IfNode *min_iff = new (C, 2) IfNode( new_pre_exit, min_bol, PROB_ALWAYS, COUNT_UNKNOWN );
0
a61af66fc99e Initial load
duke
parents:
diff changeset
1011 _igvn.register_new_node_with_optimizer( min_iff );
a61af66fc99e Initial load
duke
parents:
diff changeset
1012 set_idom(min_iff, new_pre_exit, dd_main_head);
a61af66fc99e Initial load
duke
parents:
diff changeset
1013 set_loop(min_iff, loop->_parent);
a61af66fc99e Initial load
duke
parents:
diff changeset
1014
a61af66fc99e Initial load
duke
parents:
diff changeset
1015 // Plug in the false-path, taken if we need to skip main-loop
a61af66fc99e Initial load
duke
parents:
diff changeset
1016 _igvn.hash_delete( pre_exit );
a61af66fc99e Initial load
duke
parents:
diff changeset
1017 pre_exit->set_req(0, min_iff);
a61af66fc99e Initial load
duke
parents:
diff changeset
1018 set_idom(pre_exit, min_iff, dd_main_head);
a61af66fc99e Initial load
duke
parents:
diff changeset
1019 set_idom(pre_exit->unique_out(), min_iff, dd_main_head);
a61af66fc99e Initial load
duke
parents:
diff changeset
1020 // Make the true-path, must enter the main loop
a61af66fc99e Initial load
duke
parents:
diff changeset
1021 Node *min_taken = new (C, 1) IfTrueNode( min_iff );
a61af66fc99e Initial load
duke
parents:
diff changeset
1022 _igvn.register_new_node_with_optimizer( min_taken );
a61af66fc99e Initial load
duke
parents:
diff changeset
1023 set_idom(min_taken, min_iff, dd_main_head);
a61af66fc99e Initial load
duke
parents:
diff changeset
1024 set_loop(min_taken, loop->_parent);
a61af66fc99e Initial load
duke
parents:
diff changeset
1025 // Plug in the true path
a61af66fc99e Initial load
duke
parents:
diff changeset
1026 _igvn.hash_delete( main_head );
a61af66fc99e Initial load
duke
parents:
diff changeset
1027 main_head->set_req(LoopNode::EntryControl, min_taken);
a61af66fc99e Initial load
duke
parents:
diff changeset
1028 set_idom(main_head, min_taken, dd_main_head);
a61af66fc99e Initial load
duke
parents:
diff changeset
1029
a61af66fc99e Initial load
duke
parents:
diff changeset
1030 // Step B3: Make the fall-in values to the main-loop come from the
a61af66fc99e Initial load
duke
parents:
diff changeset
1031 // fall-out values of the pre-loop.
a61af66fc99e Initial load
duke
parents:
diff changeset
1032 for (DUIterator_Fast i2max, i2 = main_head->fast_outs(i2max); i2 < i2max; i2++) {
a61af66fc99e Initial load
duke
parents:
diff changeset
1033 Node* main_phi = main_head->fast_out(i2);
a61af66fc99e Initial load
duke
parents:
diff changeset
1034 if( main_phi->is_Phi() && main_phi->in(0) == main_head && main_phi->outcnt() > 0 ) {
a61af66fc99e Initial load
duke
parents:
diff changeset
1035 Node *pre_phi = old_new[main_phi->_idx];
a61af66fc99e Initial load
duke
parents:
diff changeset
1036 Node *fallpre = clone_up_backedge_goo(pre_head->back_control(),
a61af66fc99e Initial load
duke
parents:
diff changeset
1037 main_head->init_control(),
a61af66fc99e Initial load
duke
parents:
diff changeset
1038 pre_phi->in(LoopNode::LoopBackControl));
a61af66fc99e Initial load
duke
parents:
diff changeset
1039 _igvn.hash_delete(main_phi);
a61af66fc99e Initial load
duke
parents:
diff changeset
1040 main_phi->set_req( LoopNode::EntryControl, fallpre );
a61af66fc99e Initial load
duke
parents:
diff changeset
1041 }
a61af66fc99e Initial load
duke
parents:
diff changeset
1042 }
a61af66fc99e Initial load
duke
parents:
diff changeset
1043
a61af66fc99e Initial load
duke
parents:
diff changeset
1044 // Step B4: Shorten the pre-loop to run only 1 iteration (for now).
a61af66fc99e Initial load
duke
parents:
diff changeset
1045 // RCE and alignment may change this later.
a61af66fc99e Initial load
duke
parents:
diff changeset
1046 Node *cmp_end = pre_end->cmp_node();
a61af66fc99e Initial load
duke
parents:
diff changeset
1047 assert( cmp_end->in(2) == limit, "" );
a61af66fc99e Initial load
duke
parents:
diff changeset
1048 Node *pre_limit = new (C, 3) AddINode( init, stride );
a61af66fc99e Initial load
duke
parents:
diff changeset
1049
a61af66fc99e Initial load
duke
parents:
diff changeset
1050 // Save the original loop limit in this Opaque1 node for
a61af66fc99e Initial load
duke
parents:
diff changeset
1051 // use by range check elimination.
216
8d191a7697e2 6715633: when matching a memory node the adr_type should not change
kvn
parents: 113
diff changeset
1052 Node *pre_opaq = new (C, 3) Opaque1Node(C, pre_limit, limit);
0
a61af66fc99e Initial load
duke
parents:
diff changeset
1053
a61af66fc99e Initial load
duke
parents:
diff changeset
1054 register_new_node( pre_limit, pre_head->in(0) );
a61af66fc99e Initial load
duke
parents:
diff changeset
1055 register_new_node( pre_opaq , pre_head->in(0) );
a61af66fc99e Initial load
duke
parents:
diff changeset
1056
a61af66fc99e Initial load
duke
parents:
diff changeset
1057 // Since no other users of pre-loop compare, I can hack limit directly
a61af66fc99e Initial load
duke
parents:
diff changeset
1058 assert( cmp_end->outcnt() == 1, "no other users" );
a61af66fc99e Initial load
duke
parents:
diff changeset
1059 _igvn.hash_delete(cmp_end);
a61af66fc99e Initial load
duke
parents:
diff changeset
1060 cmp_end->set_req(2, peel_only ? pre_limit : pre_opaq);
a61af66fc99e Initial load
duke
parents:
diff changeset
1061
a61af66fc99e Initial load
duke
parents:
diff changeset
1062 // Special case for not-equal loop bounds:
a61af66fc99e Initial load
duke
parents:
diff changeset
1063 // Change pre loop test, main loop test, and the
a61af66fc99e Initial load
duke
parents:
diff changeset
1064 // main loop guard test to use lt or gt depending on stride
a61af66fc99e Initial load
duke
parents:
diff changeset
1065 // direction:
a61af66fc99e Initial load
duke
parents:
diff changeset
1066 // positive stride use <
a61af66fc99e Initial load
duke
parents:
diff changeset
1067 // negative stride use >
a61af66fc99e Initial load
duke
parents:
diff changeset
1068
a61af66fc99e Initial load
duke
parents:
diff changeset
1069 if (pre_end->in(CountedLoopEndNode::TestValue)->as_Bool()->_test._test == BoolTest::ne) {
a61af66fc99e Initial load
duke
parents:
diff changeset
1070
a61af66fc99e Initial load
duke
parents:
diff changeset
1071 BoolTest::mask new_test = (main_end->stride_con() > 0) ? BoolTest::lt : BoolTest::gt;
a61af66fc99e Initial load
duke
parents:
diff changeset
1072 // Modify pre loop end condition
a61af66fc99e Initial load
duke
parents:
diff changeset
1073 Node* pre_bol = pre_end->in(CountedLoopEndNode::TestValue)->as_Bool();
a61af66fc99e Initial load
duke
parents:
diff changeset
1074 BoolNode* new_bol0 = new (C, 2) BoolNode(pre_bol->in(1), new_test);
a61af66fc99e Initial load
duke
parents:
diff changeset
1075 register_new_node( new_bol0, pre_head->in(0) );
a61af66fc99e Initial load
duke
parents:
diff changeset
1076 _igvn.hash_delete(pre_end);
a61af66fc99e Initial load
duke
parents:
diff changeset
1077 pre_end->set_req(CountedLoopEndNode::TestValue, new_bol0);
a61af66fc99e Initial load
duke
parents:
diff changeset
1078 // Modify main loop guard condition
a61af66fc99e Initial load
duke
parents:
diff changeset
1079 assert(min_iff->in(CountedLoopEndNode::TestValue) == min_bol, "guard okay");
a61af66fc99e Initial load
duke
parents:
diff changeset
1080 BoolNode* new_bol1 = new (C, 2) BoolNode(min_bol->in(1), new_test);
a61af66fc99e Initial load
duke
parents:
diff changeset
1081 register_new_node( new_bol1, new_pre_exit );
a61af66fc99e Initial load
duke
parents:
diff changeset
1082 _igvn.hash_delete(min_iff);
a61af66fc99e Initial load
duke
parents:
diff changeset
1083 min_iff->set_req(CountedLoopEndNode::TestValue, new_bol1);
a61af66fc99e Initial load
duke
parents:
diff changeset
1084 // Modify main loop end condition
a61af66fc99e Initial load
duke
parents:
diff changeset
1085 BoolNode* main_bol = main_end->in(CountedLoopEndNode::TestValue)->as_Bool();
a61af66fc99e Initial load
duke
parents:
diff changeset
1086 BoolNode* new_bol2 = new (C, 2) BoolNode(main_bol->in(1), new_test);
a61af66fc99e Initial load
duke
parents:
diff changeset
1087 register_new_node( new_bol2, main_end->in(CountedLoopEndNode::TestControl) );
a61af66fc99e Initial load
duke
parents:
diff changeset
1088 _igvn.hash_delete(main_end);
a61af66fc99e Initial load
duke
parents:
diff changeset
1089 main_end->set_req(CountedLoopEndNode::TestValue, new_bol2);
a61af66fc99e Initial load
duke
parents:
diff changeset
1090 }
a61af66fc99e Initial load
duke
parents:
diff changeset
1091
a61af66fc99e Initial load
duke
parents:
diff changeset
1092 // Flag main loop
a61af66fc99e Initial load
duke
parents:
diff changeset
1093 main_head->set_main_loop();
a61af66fc99e Initial load
duke
parents:
diff changeset
1094 if( peel_only ) main_head->set_main_no_pre_loop();
a61af66fc99e Initial load
duke
parents:
diff changeset
1095
a61af66fc99e Initial load
duke
parents:
diff changeset
1096 // It's difficult to be precise about the trip-counts
a61af66fc99e Initial load
duke
parents:
diff changeset
1097 // for the pre/post loops. They are usually very short,
a61af66fc99e Initial load
duke
parents:
diff changeset
1098 // so guess that 4 trips is a reasonable value.
a61af66fc99e Initial load
duke
parents:
diff changeset
1099 post_head->set_profile_trip_cnt(4.0);
a61af66fc99e Initial load
duke
parents:
diff changeset
1100 pre_head->set_profile_trip_cnt(4.0);
a61af66fc99e Initial load
duke
parents:
diff changeset
1101
a61af66fc99e Initial load
duke
parents:
diff changeset
1102 // Now force out all loop-invariant dominating tests. The optimizer
a61af66fc99e Initial load
duke
parents:
diff changeset
1103 // finds some, but we _know_ they are all useless.
a61af66fc99e Initial load
duke
parents:
diff changeset
1104 peeled_dom_test_elim(loop,old_new);
a61af66fc99e Initial load
duke
parents:
diff changeset
1105 }
a61af66fc99e Initial load
duke
parents:
diff changeset
1106
a61af66fc99e Initial load
duke
parents:
diff changeset
1107 //------------------------------is_invariant-----------------------------
a61af66fc99e Initial load
duke
parents:
diff changeset
1108 // Return true if n is invariant
a61af66fc99e Initial load
duke
parents:
diff changeset
1109 bool IdealLoopTree::is_invariant(Node* n) const {
1172
b2b6a9bf6238 6894779: Loop Predication for Loop Optimizer in C2
cfang
parents: 844
diff changeset
1110 Node *n_c = _phase->has_ctrl(n) ? _phase->get_ctrl(n) : n;
0
a61af66fc99e Initial load
duke
parents:
diff changeset
1111 if (n_c->is_top()) return false;
a61af66fc99e Initial load
duke
parents:
diff changeset
1112 return !is_member(_phase->get_loop(n_c));
a61af66fc99e Initial load
duke
parents:
diff changeset
1113 }
a61af66fc99e Initial load
duke
parents:
diff changeset
1114
a61af66fc99e Initial load
duke
parents:
diff changeset
1115
a61af66fc99e Initial load
duke
parents:
diff changeset
1116 //------------------------------do_unroll--------------------------------------
a61af66fc99e Initial load
duke
parents:
diff changeset
1117 // Unroll the loop body one step - make each trip do 2 iterations.
a61af66fc99e Initial load
duke
parents:
diff changeset
1118 void PhaseIdealLoop::do_unroll( IdealLoopTree *loop, Node_List &old_new, bool adjust_min_trip ) {
2383
9dc311b8473e 7008866: Missing loop predicate for loop with multiple entries
kvn
parents: 1972
diff changeset
1119 assert(LoopUnrollLimit, "");
9dc311b8473e 7008866: Missing loop predicate for loop with multiple entries
kvn
parents: 1972
diff changeset
1120 CountedLoopNode *loop_head = loop->_head->as_CountedLoop();
9dc311b8473e 7008866: Missing loop predicate for loop with multiple entries
kvn
parents: 1972
diff changeset
1121 CountedLoopEndNode *loop_end = loop_head->loopexit();
9dc311b8473e 7008866: Missing loop predicate for loop with multiple entries
kvn
parents: 1972
diff changeset
1122 assert(loop_end, "");
0
a61af66fc99e Initial load
duke
parents:
diff changeset
1123 #ifndef PRODUCT
2383
9dc311b8473e 7008866: Missing loop predicate for loop with multiple entries
kvn
parents: 1972
diff changeset
1124 if (PrintOpto && VerifyLoopOptimizations) {
0
a61af66fc99e Initial load
duke
parents:
diff changeset
1125 tty->print("Unrolling ");
a61af66fc99e Initial load
duke
parents:
diff changeset
1126 loop->dump_head();
2383
9dc311b8473e 7008866: Missing loop predicate for loop with multiple entries
kvn
parents: 1972
diff changeset
1127 } else if (TraceLoopOpts) {
2465
3af54845df98 7004555: Add new policy for one iteration loops
kvn
parents: 2453
diff changeset
1128 if (loop_head->trip_count() < (uint)LoopUnrollLimit) {
2453
d7a3fed1c1c9 7004547: regular loop unroll should not unroll more than max unrolling
kvn
parents: 2448
diff changeset
1129 tty->print("Unroll %d(%2d) ", loop_head->unrolled_count()*2, loop_head->trip_count());
d7a3fed1c1c9 7004547: regular loop unroll should not unroll more than max unrolling
kvn
parents: 2448
diff changeset
1130 } else {
d7a3fed1c1c9 7004547: regular loop unroll should not unroll more than max unrolling
kvn
parents: 2448
diff changeset
1131 tty->print("Unroll %d ", loop_head->unrolled_count()*2);
d7a3fed1c1c9 7004547: regular loop unroll should not unroll more than max unrolling
kvn
parents: 2448
diff changeset
1132 }
2383
9dc311b8473e 7008866: Missing loop predicate for loop with multiple entries
kvn
parents: 1972
diff changeset
1133 loop->dump_head();
0
a61af66fc99e Initial load
duke
parents:
diff changeset
1134 }
a61af66fc99e Initial load
duke
parents:
diff changeset
1135 #endif
a61af66fc99e Initial load
duke
parents:
diff changeset
1136
a61af66fc99e Initial load
duke
parents:
diff changeset
1137 // Remember loop node count before unrolling to detect
a61af66fc99e Initial load
duke
parents:
diff changeset
1138 // if rounds of unroll,optimize are making progress
a61af66fc99e Initial load
duke
parents:
diff changeset
1139 loop_head->set_node_count_before_unroll(loop->_body.size());
a61af66fc99e Initial load
duke
parents:
diff changeset
1140
a61af66fc99e Initial load
duke
parents:
diff changeset
1141 Node *ctrl = loop_head->in(LoopNode::EntryControl);
a61af66fc99e Initial load
duke
parents:
diff changeset
1142 Node *limit = loop_head->limit();
a61af66fc99e Initial load
duke
parents:
diff changeset
1143 Node *init = loop_head->init_trip();
2383
9dc311b8473e 7008866: Missing loop predicate for loop with multiple entries
kvn
parents: 1972
diff changeset
1144 Node *stride = loop_head->stride();
0
a61af66fc99e Initial load
duke
parents:
diff changeset
1145
a61af66fc99e Initial load
duke
parents:
diff changeset
1146 Node *opaq = NULL;
a61af66fc99e Initial load
duke
parents:
diff changeset
1147 if( adjust_min_trip ) { // If not maximally unrolling, need adjustment
a61af66fc99e Initial load
duke
parents:
diff changeset
1148 assert( loop_head->is_main_loop(), "" );
a61af66fc99e Initial load
duke
parents:
diff changeset
1149 assert( ctrl->Opcode() == Op_IfTrue || ctrl->Opcode() == Op_IfFalse, "" );
a61af66fc99e Initial load
duke
parents:
diff changeset
1150 Node *iff = ctrl->in(0);
a61af66fc99e Initial load
duke
parents:
diff changeset
1151 assert( iff->Opcode() == Op_If, "" );
a61af66fc99e Initial load
duke
parents:
diff changeset
1152 Node *bol = iff->in(1);
a61af66fc99e Initial load
duke
parents:
diff changeset
1153 assert( bol->Opcode() == Op_Bool, "" );
a61af66fc99e Initial load
duke
parents:
diff changeset
1154 Node *cmp = bol->in(1);
a61af66fc99e Initial load
duke
parents:
diff changeset
1155 assert( cmp->Opcode() == Op_CmpI, "" );
a61af66fc99e Initial load
duke
parents:
diff changeset
1156 opaq = cmp->in(2);
a61af66fc99e Initial load
duke
parents:
diff changeset
1157 // Occasionally it's possible for a pre-loop Opaque1 node to be
a61af66fc99e Initial load
duke
parents:
diff changeset
1158 // optimized away and then another round of loop opts attempted.
a61af66fc99e Initial load
duke
parents:
diff changeset
1159 // We can not optimize this particular loop in that case.
a61af66fc99e Initial load
duke
parents:
diff changeset
1160 if( opaq->Opcode() != Op_Opaque1 )
a61af66fc99e Initial load
duke
parents:
diff changeset
1161 return; // Cannot find pre-loop! Bail out!
a61af66fc99e Initial load
duke
parents:
diff changeset
1162 }
a61af66fc99e Initial load
duke
parents:
diff changeset
1163
a61af66fc99e Initial load
duke
parents:
diff changeset
1164 C->set_major_progress();
a61af66fc99e Initial load
duke
parents:
diff changeset
1165
a61af66fc99e Initial load
duke
parents:
diff changeset
1166 // Adjust max trip count. The trip count is intentionally rounded
a61af66fc99e Initial load
duke
parents:
diff changeset
1167 // down here (e.g. 15-> 7-> 3-> 1) because if we unwittingly over-unroll,
a61af66fc99e Initial load
duke
parents:
diff changeset
1168 // the main, unrolled, part of the loop will never execute as it is protected
a61af66fc99e Initial load
duke
parents:
diff changeset
1169 // by the min-trip test. See bug 4834191 for a case where we over-unrolled
a61af66fc99e Initial load
duke
parents:
diff changeset
1170 // and later determined that part of the unrolled loop was dead.
a61af66fc99e Initial load
duke
parents:
diff changeset
1171 loop_head->set_trip_count(loop_head->trip_count() / 2);
a61af66fc99e Initial load
duke
parents:
diff changeset
1172
a61af66fc99e Initial load
duke
parents:
diff changeset
1173 // Double the count of original iterations in the unrolled loop body.
a61af66fc99e Initial load
duke
parents:
diff changeset
1174 loop_head->double_unrolled_count();
a61af66fc99e Initial load
duke
parents:
diff changeset
1175
a61af66fc99e Initial load
duke
parents:
diff changeset
1176 // -----------
a61af66fc99e Initial load
duke
parents:
diff changeset
1177 // Step 2: Cut back the trip counter for an unroll amount of 2.
a61af66fc99e Initial load
duke
parents:
diff changeset
1178 // Loop will normally trip (limit - init)/stride_con. Since it's a
a61af66fc99e Initial load
duke
parents:
diff changeset
1179 // CountedLoop this is exact (stride divides limit-init exactly).
a61af66fc99e Initial load
duke
parents:
diff changeset
1180 // We are going to double the loop body, so we want to knock off any
a61af66fc99e Initial load
duke
parents:
diff changeset
1181 // odd iteration: (trip_cnt & ~1). Then back compute a new limit.
a61af66fc99e Initial load
duke
parents:
diff changeset
1182 Node *span = new (C, 3) SubINode( limit, init );
a61af66fc99e Initial load
duke
parents:
diff changeset
1183 register_new_node( span, ctrl );
2383
9dc311b8473e 7008866: Missing loop predicate for loop with multiple entries
kvn
parents: 1972
diff changeset
1184 Node *trip = new (C, 3) DivINode( 0, span, stride );
0
a61af66fc99e Initial load
duke
parents:
diff changeset
1185 register_new_node( trip, ctrl );
a61af66fc99e Initial load
duke
parents:
diff changeset
1186 Node *mtwo = _igvn.intcon(-2);
a61af66fc99e Initial load
duke
parents:
diff changeset
1187 set_ctrl(mtwo, C->root());
a61af66fc99e Initial load
duke
parents:
diff changeset
1188 Node *rond = new (C, 3) AndINode( trip, mtwo );
a61af66fc99e Initial load
duke
parents:
diff changeset
1189 register_new_node( rond, ctrl );
2383
9dc311b8473e 7008866: Missing loop predicate for loop with multiple entries
kvn
parents: 1972
diff changeset
1190 Node *spn2 = new (C, 3) MulINode( rond, stride );
0
a61af66fc99e Initial load
duke
parents:
diff changeset
1191 register_new_node( spn2, ctrl );
a61af66fc99e Initial load
duke
parents:
diff changeset
1192 Node *lim2 = new (C, 3) AddINode( spn2, init );
a61af66fc99e Initial load
duke
parents:
diff changeset
1193 register_new_node( lim2, ctrl );
a61af66fc99e Initial load
duke
parents:
diff changeset
1194
a61af66fc99e Initial load
duke
parents:
diff changeset
1195 // Hammer in the new limit
a61af66fc99e Initial load
duke
parents:
diff changeset
1196 Node *ctrl2 = loop_end->in(0);
a61af66fc99e Initial load
duke
parents:
diff changeset
1197 Node *cmp2 = new (C, 3) CmpINode( loop_head->incr(), lim2 );
a61af66fc99e Initial load
duke
parents:
diff changeset
1198 register_new_node( cmp2, ctrl2 );
a61af66fc99e Initial load
duke
parents:
diff changeset
1199 Node *bol2 = new (C, 2) BoolNode( cmp2, loop_end->test_trip() );
a61af66fc99e Initial load
duke
parents:
diff changeset
1200 register_new_node( bol2, ctrl2 );
a61af66fc99e Initial load
duke
parents:
diff changeset
1201 _igvn.hash_delete(loop_end);
a61af66fc99e Initial load
duke
parents:
diff changeset
1202 loop_end->set_req(CountedLoopEndNode::TestValue, bol2);
a61af66fc99e Initial load
duke
parents:
diff changeset
1203
a61af66fc99e Initial load
duke
parents:
diff changeset
1204 // Step 3: Find the min-trip test guaranteed before a 'main' loop.
a61af66fc99e Initial load
duke
parents:
diff changeset
1205 // Make it a 1-trip test (means at least 2 trips).
a61af66fc99e Initial load
duke
parents:
diff changeset
1206 if( adjust_min_trip ) {
a61af66fc99e Initial load
duke
parents:
diff changeset
1207 // Guard test uses an 'opaque' node which is not shared. Hence I
a61af66fc99e Initial load
duke
parents:
diff changeset
1208 // can edit it's inputs directly. Hammer in the new limit for the
a61af66fc99e Initial load
duke
parents:
diff changeset
1209 // minimum-trip guard.
a61af66fc99e Initial load
duke
parents:
diff changeset
1210 assert( opaq->outcnt() == 1, "" );
a61af66fc99e Initial load
duke
parents:
diff changeset
1211 _igvn.hash_delete(opaq);
a61af66fc99e Initial load
duke
parents:
diff changeset
1212 opaq->set_req(1, lim2);
a61af66fc99e Initial load
duke
parents:
diff changeset
1213 }
a61af66fc99e Initial load
duke
parents:
diff changeset
1214
a61af66fc99e Initial load
duke
parents:
diff changeset
1215 // ---------
a61af66fc99e Initial load
duke
parents:
diff changeset
1216 // Step 4: Clone the loop body. Move it inside the loop. This loop body
a61af66fc99e Initial load
duke
parents:
diff changeset
1217 // represents the odd iterations; since the loop trips an even number of
a61af66fc99e Initial load
duke
parents:
diff changeset
1218 // times its backedge is never taken. Kill the backedge.
a61af66fc99e Initial load
duke
parents:
diff changeset
1219 uint dd = dom_depth(loop_head);
a61af66fc99e Initial load
duke
parents:
diff changeset
1220 clone_loop( loop, old_new, dd );
a61af66fc99e Initial load
duke
parents:
diff changeset
1221
a61af66fc99e Initial load
duke
parents:
diff changeset
1222 // Make backedges of the clone equal to backedges of the original.
a61af66fc99e Initial load
duke
parents:
diff changeset
1223 // Make the fall-in from the original come from the fall-out of the clone.
a61af66fc99e Initial load
duke
parents:
diff changeset
1224 for (DUIterator_Fast jmax, j = loop_head->fast_outs(jmax); j < jmax; j++) {
a61af66fc99e Initial load
duke
parents:
diff changeset
1225 Node* phi = loop_head->fast_out(j);
a61af66fc99e Initial load
duke
parents:
diff changeset
1226 if( phi->is_Phi() && phi->in(0) == loop_head && phi->outcnt() > 0 ) {
a61af66fc99e Initial load
duke
parents:
diff changeset
1227 Node *newphi = old_new[phi->_idx];
a61af66fc99e Initial load
duke
parents:
diff changeset
1228 _igvn.hash_delete( phi );
a61af66fc99e Initial load
duke
parents:
diff changeset
1229 _igvn.hash_delete( newphi );
a61af66fc99e Initial load
duke
parents:
diff changeset
1230
a61af66fc99e Initial load
duke
parents:
diff changeset
1231 phi ->set_req(LoopNode:: EntryControl, newphi->in(LoopNode::LoopBackControl));
a61af66fc99e Initial load
duke
parents:
diff changeset
1232 newphi->set_req(LoopNode::LoopBackControl, phi ->in(LoopNode::LoopBackControl));
a61af66fc99e Initial load
duke
parents:
diff changeset
1233 phi ->set_req(LoopNode::LoopBackControl, C->top());
a61af66fc99e Initial load
duke
parents:
diff changeset
1234 }
a61af66fc99e Initial load
duke
parents:
diff changeset
1235 }
a61af66fc99e Initial load
duke
parents:
diff changeset
1236 Node *clone_head = old_new[loop_head->_idx];
a61af66fc99e Initial load
duke
parents:
diff changeset
1237 _igvn.hash_delete( clone_head );
a61af66fc99e Initial load
duke
parents:
diff changeset
1238 loop_head ->set_req(LoopNode:: EntryControl, clone_head->in(LoopNode::LoopBackControl));
a61af66fc99e Initial load
duke
parents:
diff changeset
1239 clone_head->set_req(LoopNode::LoopBackControl, loop_head ->in(LoopNode::LoopBackControl));
a61af66fc99e Initial load
duke
parents:
diff changeset
1240 loop_head ->set_req(LoopNode::LoopBackControl, C->top());
a61af66fc99e Initial load
duke
parents:
diff changeset
1241 loop->_head = clone_head; // New loop header
a61af66fc99e Initial load
duke
parents:
diff changeset
1242
a61af66fc99e Initial load
duke
parents:
diff changeset
1243 set_idom(loop_head, loop_head ->in(LoopNode::EntryControl), dd);
a61af66fc99e Initial load
duke
parents:
diff changeset
1244 set_idom(clone_head, clone_head->in(LoopNode::EntryControl), dd);
a61af66fc99e Initial load
duke
parents:
diff changeset
1245
a61af66fc99e Initial load
duke
parents:
diff changeset
1246 // Kill the clone's backedge
a61af66fc99e Initial load
duke
parents:
diff changeset
1247 Node *newcle = old_new[loop_end->_idx];
a61af66fc99e Initial load
duke
parents:
diff changeset
1248 _igvn.hash_delete( newcle );
a61af66fc99e Initial load
duke
parents:
diff changeset
1249 Node *one = _igvn.intcon(1);
a61af66fc99e Initial load
duke
parents:
diff changeset
1250 set_ctrl(one, C->root());
a61af66fc99e Initial load
duke
parents:
diff changeset
1251 newcle->set_req(1, one);
a61af66fc99e Initial load
duke
parents:
diff changeset
1252 // Force clone into same loop body
a61af66fc99e Initial load
duke
parents:
diff changeset
1253 uint max = loop->_body.size();
a61af66fc99e Initial load
duke
parents:
diff changeset
1254 for( uint k = 0; k < max; k++ ) {
a61af66fc99e Initial load
duke
parents:
diff changeset
1255 Node *old = loop->_body.at(k);
a61af66fc99e Initial load
duke
parents:
diff changeset
1256 Node *nnn = old_new[old->_idx];
a61af66fc99e Initial load
duke
parents:
diff changeset
1257 loop->_body.push(nnn);
a61af66fc99e Initial load
duke
parents:
diff changeset
1258 if (!has_ctrl(old))
a61af66fc99e Initial load
duke
parents:
diff changeset
1259 set_loop(nnn, loop);
a61af66fc99e Initial load
duke
parents:
diff changeset
1260 }
367
194b8e3a2fc4 6384206: Phis which are later unneeded are impairing our ability to inline based on static types
never
parents: 235
diff changeset
1261
194b8e3a2fc4 6384206: Phis which are later unneeded are impairing our ability to inline based on static types
never
parents: 235
diff changeset
1262 loop->record_for_igvn();
0
a61af66fc99e Initial load
duke
parents:
diff changeset
1263 }
a61af66fc99e Initial load
duke
parents:
diff changeset
1264
a61af66fc99e Initial load
duke
parents:
diff changeset
1265 //------------------------------do_maximally_unroll----------------------------
a61af66fc99e Initial load
duke
parents:
diff changeset
1266
a61af66fc99e Initial load
duke
parents:
diff changeset
1267 void PhaseIdealLoop::do_maximally_unroll( IdealLoopTree *loop, Node_List &old_new ) {
a61af66fc99e Initial load
duke
parents:
diff changeset
1268 CountedLoopNode *cl = loop->_head->as_CountedLoop();
2383
9dc311b8473e 7008866: Missing loop predicate for loop with multiple entries
kvn
parents: 1972
diff changeset
1269 assert(cl->trip_count() > 0, "");
9dc311b8473e 7008866: Missing loop predicate for loop with multiple entries
kvn
parents: 1972
diff changeset
1270 #ifndef PRODUCT
9dc311b8473e 7008866: Missing loop predicate for loop with multiple entries
kvn
parents: 1972
diff changeset
1271 if (TraceLoopOpts) {
9dc311b8473e 7008866: Missing loop predicate for loop with multiple entries
kvn
parents: 1972
diff changeset
1272 tty->print("MaxUnroll %d ", cl->trip_count());
9dc311b8473e 7008866: Missing loop predicate for loop with multiple entries
kvn
parents: 1972
diff changeset
1273 loop->dump_head();
9dc311b8473e 7008866: Missing loop predicate for loop with multiple entries
kvn
parents: 1972
diff changeset
1274 }
9dc311b8473e 7008866: Missing loop predicate for loop with multiple entries
kvn
parents: 1972
diff changeset
1275 #endif
0
a61af66fc99e Initial load
duke
parents:
diff changeset
1276
a61af66fc99e Initial load
duke
parents:
diff changeset
1277 // If loop is tripping an odd number of times, peel odd iteration
2383
9dc311b8473e 7008866: Missing loop predicate for loop with multiple entries
kvn
parents: 1972
diff changeset
1278 if ((cl->trip_count() & 1) == 1) {
9dc311b8473e 7008866: Missing loop predicate for loop with multiple entries
kvn
parents: 1972
diff changeset
1279 do_peeling(loop, old_new);
0
a61af66fc99e Initial load
duke
parents:
diff changeset
1280 }
a61af66fc99e Initial load
duke
parents:
diff changeset
1281
a61af66fc99e Initial load
duke
parents:
diff changeset
1282 // Now its tripping an even number of times remaining. Double loop body.
a61af66fc99e Initial load
duke
parents:
diff changeset
1283 // Do not adjust pre-guards; they are not needed and do not exist.
2383
9dc311b8473e 7008866: Missing loop predicate for loop with multiple entries
kvn
parents: 1972
diff changeset
1284 if (cl->trip_count() > 0) {
9dc311b8473e 7008866: Missing loop predicate for loop with multiple entries
kvn
parents: 1972
diff changeset
1285 do_unroll(loop, old_new, false);
0
a61af66fc99e Initial load
duke
parents:
diff changeset
1286 }
a61af66fc99e Initial load
duke
parents:
diff changeset
1287 }
a61af66fc99e Initial load
duke
parents:
diff changeset
1288
a61af66fc99e Initial load
duke
parents:
diff changeset
1289 //------------------------------dominates_backedge---------------------------------
a61af66fc99e Initial load
duke
parents:
diff changeset
1290 // Returns true if ctrl is executed on every complete iteration
a61af66fc99e Initial load
duke
parents:
diff changeset
1291 bool IdealLoopTree::dominates_backedge(Node* ctrl) {
a61af66fc99e Initial load
duke
parents:
diff changeset
1292 assert(ctrl->is_CFG(), "must be control");
a61af66fc99e Initial load
duke
parents:
diff changeset
1293 Node* backedge = _head->as_Loop()->in(LoopNode::LoopBackControl);
a61af66fc99e Initial load
duke
parents:
diff changeset
1294 return _phase->dom_lca_internal(ctrl, backedge) == ctrl;
a61af66fc99e Initial load
duke
parents:
diff changeset
1295 }
a61af66fc99e Initial load
duke
parents:
diff changeset
1296
a61af66fc99e Initial load
duke
parents:
diff changeset
1297 //------------------------------add_constraint---------------------------------
a61af66fc99e Initial load
duke
parents:
diff changeset
1298 // Constrain the main loop iterations so the condition:
a61af66fc99e Initial load
duke
parents:
diff changeset
1299 // scale_con * I + offset < limit
a61af66fc99e Initial load
duke
parents:
diff changeset
1300 // always holds true. That is, either increase the number of iterations in
a61af66fc99e Initial load
duke
parents:
diff changeset
1301 // the pre-loop or the post-loop until the condition holds true in the main
a61af66fc99e Initial load
duke
parents:
diff changeset
1302 // loop. Stride, scale, offset and limit are all loop invariant. Further,
a61af66fc99e Initial load
duke
parents:
diff changeset
1303 // stride and scale are constants (offset and limit often are).
a61af66fc99e Initial load
duke
parents:
diff changeset
1304 void PhaseIdealLoop::add_constraint( int stride_con, int scale_con, Node *offset, Node *limit, Node *pre_ctrl, Node **pre_limit, Node **main_limit ) {
a61af66fc99e Initial load
duke
parents:
diff changeset
1305
a61af66fc99e Initial load
duke
parents:
diff changeset
1306 // Compute "I :: (limit-offset)/scale_con"
a61af66fc99e Initial load
duke
parents:
diff changeset
1307 Node *con = new (C, 3) SubINode( limit, offset );
a61af66fc99e Initial load
duke
parents:
diff changeset
1308 register_new_node( con, pre_ctrl );
a61af66fc99e Initial load
duke
parents:
diff changeset
1309 Node *scale = _igvn.intcon(scale_con);
a61af66fc99e Initial load
duke
parents:
diff changeset
1310 set_ctrl(scale, C->root());
a61af66fc99e Initial load
duke
parents:
diff changeset
1311 Node *X = new (C, 3) DivINode( 0, con, scale );
a61af66fc99e Initial load
duke
parents:
diff changeset
1312 register_new_node( X, pre_ctrl );
a61af66fc99e Initial load
duke
parents:
diff changeset
1313
a61af66fc99e Initial load
duke
parents:
diff changeset
1314 // For positive stride, the pre-loop limit always uses a MAX function
a61af66fc99e Initial load
duke
parents:
diff changeset
1315 // and the main loop a MIN function. For negative stride these are
a61af66fc99e Initial load
duke
parents:
diff changeset
1316 // reversed.
a61af66fc99e Initial load
duke
parents:
diff changeset
1317
a61af66fc99e Initial load
duke
parents:
diff changeset
1318 // Also for positive stride*scale the affine function is increasing, so the
a61af66fc99e Initial load
duke
parents:
diff changeset
1319 // pre-loop must check for underflow and the post-loop for overflow.
a61af66fc99e Initial load
duke
parents:
diff changeset
1320 // Negative stride*scale reverses this; pre-loop checks for overflow and
a61af66fc99e Initial load
duke
parents:
diff changeset
1321 // post-loop for underflow.
a61af66fc99e Initial load
duke
parents:
diff changeset
1322 if( stride_con*scale_con > 0 ) {
a61af66fc99e Initial load
duke
parents:
diff changeset
1323 // Compute I < (limit-offset)/scale_con
a61af66fc99e Initial load
duke
parents:
diff changeset
1324 // Adjust main-loop last iteration to be MIN/MAX(main_loop,X)
a61af66fc99e Initial load
duke
parents:
diff changeset
1325 *main_limit = (stride_con > 0)
a61af66fc99e Initial load
duke
parents:
diff changeset
1326 ? (Node*)(new (C, 3) MinINode( *main_limit, X ))
a61af66fc99e Initial load
duke
parents:
diff changeset
1327 : (Node*)(new (C, 3) MaxINode( *main_limit, X ));
a61af66fc99e Initial load
duke
parents:
diff changeset
1328 register_new_node( *main_limit, pre_ctrl );
a61af66fc99e Initial load
duke
parents:
diff changeset
1329
a61af66fc99e Initial load
duke
parents:
diff changeset
1330 } else {
a61af66fc99e Initial load
duke
parents:
diff changeset
1331 // Compute (limit-offset)/scale_con + SGN(-scale_con) <= I
a61af66fc99e Initial load
duke
parents:
diff changeset
1332 // Add the negation of the main-loop constraint to the pre-loop.
a61af66fc99e Initial load
duke
parents:
diff changeset
1333 // See footnote [++] below for a derivation of the limit expression.
a61af66fc99e Initial load
duke
parents:
diff changeset
1334 Node *incr = _igvn.intcon(scale_con > 0 ? -1 : 1);
a61af66fc99e Initial load
duke
parents:
diff changeset
1335 set_ctrl(incr, C->root());
a61af66fc99e Initial load
duke
parents:
diff changeset
1336 Node *adj = new (C, 3) AddINode( X, incr );
a61af66fc99e Initial load
duke
parents:
diff changeset
1337 register_new_node( adj, pre_ctrl );
a61af66fc99e Initial load
duke
parents:
diff changeset
1338 *pre_limit = (scale_con > 0)
a61af66fc99e Initial load
duke
parents:
diff changeset
1339 ? (Node*)new (C, 3) MinINode( *pre_limit, adj )
a61af66fc99e Initial load
duke
parents:
diff changeset
1340 : (Node*)new (C, 3) MaxINode( *pre_limit, adj );
a61af66fc99e Initial load
duke
parents:
diff changeset
1341 register_new_node( *pre_limit, pre_ctrl );
a61af66fc99e Initial load
duke
parents:
diff changeset
1342
a61af66fc99e Initial load
duke
parents:
diff changeset
1343 // [++] Here's the algebra that justifies the pre-loop limit expression:
a61af66fc99e Initial load
duke
parents:
diff changeset
1344 //
a61af66fc99e Initial load
duke
parents:
diff changeset
1345 // NOT( scale_con * I + offset < limit )
a61af66fc99e Initial load
duke
parents:
diff changeset
1346 // ==
a61af66fc99e Initial load
duke
parents:
diff changeset
1347 // scale_con * I + offset >= limit
a61af66fc99e Initial load
duke
parents:
diff changeset
1348 // ==
a61af66fc99e Initial load
duke
parents:
diff changeset
1349 // SGN(scale_con) * I >= (limit-offset)/|scale_con|
a61af66fc99e Initial load
duke
parents:
diff changeset
1350 // ==
a61af66fc99e Initial load
duke
parents:
diff changeset
1351 // (limit-offset)/|scale_con| <= I * SGN(scale_con)
a61af66fc99e Initial load
duke
parents:
diff changeset
1352 // ==
a61af66fc99e Initial load
duke
parents:
diff changeset
1353 // (limit-offset)/|scale_con|-1 < I * SGN(scale_con)
a61af66fc99e Initial load
duke
parents:
diff changeset
1354 // ==
a61af66fc99e Initial load
duke
parents:
diff changeset
1355 // ( if (scale_con > 0) /*common case*/
a61af66fc99e Initial load
duke
parents:
diff changeset
1356 // (limit-offset)/scale_con - 1 < I
a61af66fc99e Initial load
duke
parents:
diff changeset
1357 // else
a61af66fc99e Initial load
duke
parents:
diff changeset
1358 // (limit-offset)/scale_con + 1 > I
a61af66fc99e Initial load
duke
parents:
diff changeset
1359 // )
a61af66fc99e Initial load
duke
parents:
diff changeset
1360 // ( if (scale_con > 0) /*common case*/
a61af66fc99e Initial load
duke
parents:
diff changeset
1361 // (limit-offset)/scale_con + SGN(-scale_con) < I
a61af66fc99e Initial load
duke
parents:
diff changeset
1362 // else
a61af66fc99e Initial load
duke
parents:
diff changeset
1363 // (limit-offset)/scale_con + SGN(-scale_con) > I
a61af66fc99e Initial load
duke
parents:
diff changeset
1364 }
a61af66fc99e Initial load
duke
parents:
diff changeset
1365 }
a61af66fc99e Initial load
duke
parents:
diff changeset
1366
a61af66fc99e Initial load
duke
parents:
diff changeset
1367
a61af66fc99e Initial load
duke
parents:
diff changeset
1368 //------------------------------is_scaled_iv---------------------------------
a61af66fc99e Initial load
duke
parents:
diff changeset
1369 // Return true if exp is a constant times an induction var
a61af66fc99e Initial load
duke
parents:
diff changeset
1370 bool PhaseIdealLoop::is_scaled_iv(Node* exp, Node* iv, int* p_scale) {
a61af66fc99e Initial load
duke
parents:
diff changeset
1371 if (exp == iv) {
a61af66fc99e Initial load
duke
parents:
diff changeset
1372 if (p_scale != NULL) {
a61af66fc99e Initial load
duke
parents:
diff changeset
1373 *p_scale = 1;
a61af66fc99e Initial load
duke
parents:
diff changeset
1374 }
a61af66fc99e Initial load
duke
parents:
diff changeset
1375 return true;
a61af66fc99e Initial load
duke
parents:
diff changeset
1376 }
a61af66fc99e Initial load
duke
parents:
diff changeset
1377 int opc = exp->Opcode();
a61af66fc99e Initial load
duke
parents:
diff changeset
1378 if (opc == Op_MulI) {
a61af66fc99e Initial load
duke
parents:
diff changeset
1379 if (exp->in(1) == iv && exp->in(2)->is_Con()) {
a61af66fc99e Initial load
duke
parents:
diff changeset
1380 if (p_scale != NULL) {
a61af66fc99e Initial load
duke
parents:
diff changeset
1381 *p_scale = exp->in(2)->get_int();
a61af66fc99e Initial load
duke
parents:
diff changeset
1382 }
a61af66fc99e Initial load
duke
parents:
diff changeset
1383 return true;
a61af66fc99e Initial load
duke
parents:
diff changeset
1384 }
a61af66fc99e Initial load
duke
parents:
diff changeset
1385 if (exp->in(2) == iv && exp->in(1)->is_Con()) {
a61af66fc99e Initial load
duke
parents:
diff changeset
1386 if (p_scale != NULL) {
a61af66fc99e Initial load
duke
parents:
diff changeset
1387 *p_scale = exp->in(1)->get_int();
a61af66fc99e Initial load
duke
parents:
diff changeset
1388 }
a61af66fc99e Initial load
duke
parents:
diff changeset
1389 return true;
a61af66fc99e Initial load
duke
parents:
diff changeset
1390 }
a61af66fc99e Initial load
duke
parents:
diff changeset
1391 } else if (opc == Op_LShiftI) {
a61af66fc99e Initial load
duke
parents:
diff changeset
1392 if (exp->in(1) == iv && exp->in(2)->is_Con()) {
a61af66fc99e Initial load
duke
parents:
diff changeset
1393 if (p_scale != NULL) {
a61af66fc99e Initial load
duke
parents:
diff changeset
1394 *p_scale = 1 << exp->in(2)->get_int();
a61af66fc99e Initial load
duke
parents:
diff changeset
1395 }
a61af66fc99e Initial load
duke
parents:
diff changeset
1396 return true;
a61af66fc99e Initial load
duke
parents:
diff changeset
1397 }
a61af66fc99e Initial load
duke
parents:
diff changeset
1398 }
a61af66fc99e Initial load
duke
parents:
diff changeset
1399 return false;
a61af66fc99e Initial load
duke
parents:
diff changeset
1400 }
a61af66fc99e Initial load
duke
parents:
diff changeset
1401
a61af66fc99e Initial load
duke
parents:
diff changeset
1402 //-----------------------------is_scaled_iv_plus_offset------------------------------
a61af66fc99e Initial load
duke
parents:
diff changeset
1403 // Return true if exp is a simple induction variable expression: k1*iv + (invar + k2)
a61af66fc99e Initial load
duke
parents:
diff changeset
1404 bool PhaseIdealLoop::is_scaled_iv_plus_offset(Node* exp, Node* iv, int* p_scale, Node** p_offset, int depth) {
a61af66fc99e Initial load
duke
parents:
diff changeset
1405 if (is_scaled_iv(exp, iv, p_scale)) {
a61af66fc99e Initial load
duke
parents:
diff changeset
1406 if (p_offset != NULL) {
a61af66fc99e Initial load
duke
parents:
diff changeset
1407 Node *zero = _igvn.intcon(0);
a61af66fc99e Initial load
duke
parents:
diff changeset
1408 set_ctrl(zero, C->root());
a61af66fc99e Initial load
duke
parents:
diff changeset
1409 *p_offset = zero;
a61af66fc99e Initial load
duke
parents:
diff changeset
1410 }
a61af66fc99e Initial load
duke
parents:
diff changeset
1411 return true;
a61af66fc99e Initial load
duke
parents:
diff changeset
1412 }
a61af66fc99e Initial load
duke
parents:
diff changeset
1413 int opc = exp->Opcode();
a61af66fc99e Initial load
duke
parents:
diff changeset
1414 if (opc == Op_AddI) {
a61af66fc99e Initial load
duke
parents:
diff changeset
1415 if (is_scaled_iv(exp->in(1), iv, p_scale)) {
a61af66fc99e Initial load
duke
parents:
diff changeset
1416 if (p_offset != NULL) {
a61af66fc99e Initial load
duke
parents:
diff changeset
1417 *p_offset = exp->in(2);
a61af66fc99e Initial load
duke
parents:
diff changeset
1418 }
a61af66fc99e Initial load
duke
parents:
diff changeset
1419 return true;
a61af66fc99e Initial load
duke
parents:
diff changeset
1420 }
a61af66fc99e Initial load
duke
parents:
diff changeset
1421 if (exp->in(2)->is_Con()) {
a61af66fc99e Initial load
duke
parents:
diff changeset
1422 Node* offset2 = NULL;
a61af66fc99e Initial load
duke
parents:
diff changeset
1423 if (depth < 2 &&
a61af66fc99e Initial load
duke
parents:
diff changeset
1424 is_scaled_iv_plus_offset(exp->in(1), iv, p_scale,
a61af66fc99e Initial load
duke
parents:
diff changeset
1425 p_offset != NULL ? &offset2 : NULL, depth+1)) {
a61af66fc99e Initial load
duke
parents:
diff changeset
1426 if (p_offset != NULL) {
a61af66fc99e Initial load
duke
parents:
diff changeset
1427 Node *ctrl_off2 = get_ctrl(offset2);
a61af66fc99e Initial load
duke
parents:
diff changeset
1428 Node* offset = new (C, 3) AddINode(offset2, exp->in(2));
a61af66fc99e Initial load
duke
parents:
diff changeset
1429 register_new_node(offset, ctrl_off2);
a61af66fc99e Initial load
duke
parents:
diff changeset
1430 *p_offset = offset;
a61af66fc99e Initial load
duke
parents:
diff changeset
1431 }
a61af66fc99e Initial load
duke
parents:
diff changeset
1432 return true;
a61af66fc99e Initial load
duke
parents:
diff changeset
1433 }
a61af66fc99e Initial load
duke
parents:
diff changeset
1434 }
a61af66fc99e Initial load
duke
parents:
diff changeset
1435 } else if (opc == Op_SubI) {
a61af66fc99e Initial load
duke
parents:
diff changeset
1436 if (is_scaled_iv(exp->in(1), iv, p_scale)) {
a61af66fc99e Initial load
duke
parents:
diff changeset
1437 if (p_offset != NULL) {
a61af66fc99e Initial load
duke
parents:
diff changeset
1438 Node *zero = _igvn.intcon(0);
a61af66fc99e Initial load
duke
parents:
diff changeset
1439 set_ctrl(zero, C->root());
a61af66fc99e Initial load
duke
parents:
diff changeset
1440 Node *ctrl_off = get_ctrl(exp->in(2));
a61af66fc99e Initial load
duke
parents:
diff changeset
1441 Node* offset = new (C, 3) SubINode(zero, exp->in(2));
a61af66fc99e Initial load
duke
parents:
diff changeset
1442 register_new_node(offset, ctrl_off);
a61af66fc99e Initial load
duke
parents:
diff changeset
1443 *p_offset = offset;
a61af66fc99e Initial load
duke
parents:
diff changeset
1444 }
a61af66fc99e Initial load
duke
parents:
diff changeset
1445 return true;
a61af66fc99e Initial load
duke
parents:
diff changeset
1446 }
a61af66fc99e Initial load
duke
parents:
diff changeset
1447 if (is_scaled_iv(exp->in(2), iv, p_scale)) {
a61af66fc99e Initial load
duke
parents:
diff changeset
1448 if (p_offset != NULL) {
a61af66fc99e Initial load
duke
parents:
diff changeset
1449 *p_scale *= -1;
a61af66fc99e Initial load
duke
parents:
diff changeset
1450 *p_offset = exp->in(1);
a61af66fc99e Initial load
duke
parents:
diff changeset
1451 }
a61af66fc99e Initial load
duke
parents:
diff changeset
1452 return true;
a61af66fc99e Initial load
duke
parents:
diff changeset
1453 }
a61af66fc99e Initial load
duke
parents:
diff changeset
1454 }
a61af66fc99e Initial load
duke
parents:
diff changeset
1455 return false;
a61af66fc99e Initial load
duke
parents:
diff changeset
1456 }
a61af66fc99e Initial load
duke
parents:
diff changeset
1457
a61af66fc99e Initial load
duke
parents:
diff changeset
1458 //------------------------------do_range_check---------------------------------
a61af66fc99e Initial load
duke
parents:
diff changeset
1459 // Eliminate range-checks and other trip-counter vs loop-invariant tests.
a61af66fc99e Initial load
duke
parents:
diff changeset
1460 void PhaseIdealLoop::do_range_check( IdealLoopTree *loop, Node_List &old_new ) {
a61af66fc99e Initial load
duke
parents:
diff changeset
1461 #ifndef PRODUCT
2383
9dc311b8473e 7008866: Missing loop predicate for loop with multiple entries
kvn
parents: 1972
diff changeset
1462 if (PrintOpto && VerifyLoopOptimizations) {
0
a61af66fc99e Initial load
duke
parents:
diff changeset
1463 tty->print("Range Check Elimination ");
a61af66fc99e Initial load
duke
parents:
diff changeset
1464 loop->dump_head();
2383
9dc311b8473e 7008866: Missing loop predicate for loop with multiple entries
kvn
parents: 1972
diff changeset
1465 } else if (TraceLoopOpts) {
9dc311b8473e 7008866: Missing loop predicate for loop with multiple entries
kvn
parents: 1972
diff changeset
1466 tty->print("RangeCheck ");
9dc311b8473e 7008866: Missing loop predicate for loop with multiple entries
kvn
parents: 1972
diff changeset
1467 loop->dump_head();
0
a61af66fc99e Initial load
duke
parents:
diff changeset
1468 }
a61af66fc99e Initial load
duke
parents:
diff changeset
1469 #endif
2383
9dc311b8473e 7008866: Missing loop predicate for loop with multiple entries
kvn
parents: 1972
diff changeset
1470 assert(RangeCheckElimination, "");
0
a61af66fc99e Initial load
duke
parents:
diff changeset
1471 CountedLoopNode *cl = loop->_head->as_CountedLoop();
2383
9dc311b8473e 7008866: Missing loop predicate for loop with multiple entries
kvn
parents: 1972
diff changeset
1472 assert(cl->is_main_loop(), "");
9dc311b8473e 7008866: Missing loop predicate for loop with multiple entries
kvn
parents: 1972
diff changeset
1473
9dc311b8473e 7008866: Missing loop predicate for loop with multiple entries
kvn
parents: 1972
diff changeset
1474 // protect against stride not being a constant
9dc311b8473e 7008866: Missing loop predicate for loop with multiple entries
kvn
parents: 1972
diff changeset
1475 if (!cl->stride_is_con())
9dc311b8473e 7008866: Missing loop predicate for loop with multiple entries
kvn
parents: 1972
diff changeset
1476 return;
0
a61af66fc99e Initial load
duke
parents:
diff changeset
1477
a61af66fc99e Initial load
duke
parents:
diff changeset
1478 // Find the trip counter; we are iteration splitting based on it
a61af66fc99e Initial load
duke
parents:
diff changeset
1479 Node *trip_counter = cl->phi();
a61af66fc99e Initial load
duke
parents:
diff changeset
1480 // Find the main loop limit; we will trim it's iterations
a61af66fc99e Initial load
duke
parents:
diff changeset
1481 // to not ever trip end tests
a61af66fc99e Initial load
duke
parents:
diff changeset
1482 Node *main_limit = cl->limit();
2383
9dc311b8473e 7008866: Missing loop predicate for loop with multiple entries
kvn
parents: 1972
diff changeset
1483
9dc311b8473e 7008866: Missing loop predicate for loop with multiple entries
kvn
parents: 1972
diff changeset
1484 // Need to find the main-loop zero-trip guard
9dc311b8473e 7008866: Missing loop predicate for loop with multiple entries
kvn
parents: 1972
diff changeset
1485 Node *ctrl = cl->in(LoopNode::EntryControl);
9dc311b8473e 7008866: Missing loop predicate for loop with multiple entries
kvn
parents: 1972
diff changeset
1486 assert(ctrl->Opcode() == Op_IfTrue || ctrl->Opcode() == Op_IfFalse, "");
9dc311b8473e 7008866: Missing loop predicate for loop with multiple entries
kvn
parents: 1972
diff changeset
1487 Node *iffm = ctrl->in(0);
9dc311b8473e 7008866: Missing loop predicate for loop with multiple entries
kvn
parents: 1972
diff changeset
1488 assert(iffm->Opcode() == Op_If, "");
9dc311b8473e 7008866: Missing loop predicate for loop with multiple entries
kvn
parents: 1972
diff changeset
1489 Node *bolzm = iffm->in(1);
9dc311b8473e 7008866: Missing loop predicate for loop with multiple entries
kvn
parents: 1972
diff changeset
1490 assert(bolzm->Opcode() == Op_Bool, "");
9dc311b8473e 7008866: Missing loop predicate for loop with multiple entries
kvn
parents: 1972
diff changeset
1491 Node *cmpzm = bolzm->in(1);
9dc311b8473e 7008866: Missing loop predicate for loop with multiple entries
kvn
parents: 1972
diff changeset
1492 assert(cmpzm->is_Cmp(), "");
9dc311b8473e 7008866: Missing loop predicate for loop with multiple entries
kvn
parents: 1972
diff changeset
1493 Node *opqzm = cmpzm->in(2);
9dc311b8473e 7008866: Missing loop predicate for loop with multiple entries
kvn
parents: 1972
diff changeset
1494 // Can not optimize a loop if pre-loop Opaque1 node is optimized
9dc311b8473e 7008866: Missing loop predicate for loop with multiple entries
kvn
parents: 1972
diff changeset
1495 // away and then another round of loop opts attempted.
9dc311b8473e 7008866: Missing loop predicate for loop with multiple entries
kvn
parents: 1972
diff changeset
1496 if (opqzm->Opcode() != Op_Opaque1)
9dc311b8473e 7008866: Missing loop predicate for loop with multiple entries
kvn
parents: 1972
diff changeset
1497 return;
9dc311b8473e 7008866: Missing loop predicate for loop with multiple entries
kvn
parents: 1972
diff changeset
1498 assert(opqzm->in(1) == main_limit, "do not understand situation");
9dc311b8473e 7008866: Missing loop predicate for loop with multiple entries
kvn
parents: 1972
diff changeset
1499
0
a61af66fc99e Initial load
duke
parents:
diff changeset
1500 // Find the pre-loop limit; we will expand it's iterations to
a61af66fc99e Initial load
duke
parents:
diff changeset
1501 // not ever trip low tests.
a61af66fc99e Initial load
duke
parents:
diff changeset
1502 Node *p_f = iffm->in(0);
2383
9dc311b8473e 7008866: Missing loop predicate for loop with multiple entries
kvn
parents: 1972
diff changeset
1503 assert(p_f->Opcode() == Op_IfFalse, "");
0
a61af66fc99e Initial load
duke
parents:
diff changeset
1504 CountedLoopEndNode *pre_end = p_f->in(0)->as_CountedLoopEnd();
2383
9dc311b8473e 7008866: Missing loop predicate for loop with multiple entries
kvn
parents: 1972
diff changeset
1505 assert(pre_end->loopnode()->is_pre_loop(), "");
0
a61af66fc99e Initial load
duke
parents:
diff changeset
1506 Node *pre_opaq1 = pre_end->limit();
a61af66fc99e Initial load
duke
parents:
diff changeset
1507 // Occasionally it's possible for a pre-loop Opaque1 node to be
a61af66fc99e Initial load
duke
parents:
diff changeset
1508 // optimized away and then another round of loop opts attempted.
a61af66fc99e Initial load
duke
parents:
diff changeset
1509 // We can not optimize this particular loop in that case.
2383
9dc311b8473e 7008866: Missing loop predicate for loop with multiple entries
kvn
parents: 1972
diff changeset
1510 if (pre_opaq1->Opcode() != Op_Opaque1)
0
a61af66fc99e Initial load
duke
parents:
diff changeset
1511 return;
a61af66fc99e Initial load
duke
parents:
diff changeset
1512 Opaque1Node *pre_opaq = (Opaque1Node*)pre_opaq1;
a61af66fc99e Initial load
duke
parents:
diff changeset
1513 Node *pre_limit = pre_opaq->in(1);
a61af66fc99e Initial load
duke
parents:
diff changeset
1514
a61af66fc99e Initial load
duke
parents:
diff changeset
1515 // Where do we put new limit calculations
a61af66fc99e Initial load
duke
parents:
diff changeset
1516 Node *pre_ctrl = pre_end->loopnode()->in(LoopNode::EntryControl);
a61af66fc99e Initial load
duke
parents:
diff changeset
1517
a61af66fc99e Initial load
duke
parents:
diff changeset
1518 // Ensure the original loop limit is available from the
a61af66fc99e Initial load
duke
parents:
diff changeset
1519 // pre-loop Opaque1 node.
a61af66fc99e Initial load
duke
parents:
diff changeset
1520 Node *orig_limit = pre_opaq->original_loop_limit();
2383
9dc311b8473e 7008866: Missing loop predicate for loop with multiple entries
kvn
parents: 1972
diff changeset
1521 if (orig_limit == NULL || _igvn.type(orig_limit) == Type::TOP)
0
a61af66fc99e Initial load
duke
parents:
diff changeset
1522 return;
a61af66fc99e Initial load
duke
parents:
diff changeset
1523
a61af66fc99e Initial load
duke
parents:
diff changeset
1524 // Must know if its a count-up or count-down loop
a61af66fc99e Initial load
duke
parents:
diff changeset
1525
a61af66fc99e Initial load
duke
parents:
diff changeset
1526 int stride_con = cl->stride_con();
a61af66fc99e Initial load
duke
parents:
diff changeset
1527 Node *zero = _igvn.intcon(0);
a61af66fc99e Initial load
duke
parents:
diff changeset
1528 Node *one = _igvn.intcon(1);
a61af66fc99e Initial load
duke
parents:
diff changeset
1529 set_ctrl(zero, C->root());
a61af66fc99e Initial load
duke
parents:
diff changeset
1530 set_ctrl(one, C->root());
a61af66fc99e Initial load
duke
parents:
diff changeset
1531
a61af66fc99e Initial load
duke
parents:
diff changeset
1532 // Range checks that do not dominate the loop backedge (ie.
a61af66fc99e Initial load
duke
parents:
diff changeset
1533 // conditionally executed) can lengthen the pre loop limit beyond
a61af66fc99e Initial load
duke
parents:
diff changeset
1534 // the original loop limit. To prevent this, the pre limit is
a61af66fc99e Initial load
duke
parents:
diff changeset
1535 // (for stride > 0) MINed with the original loop limit (MAXed
a61af66fc99e Initial load
duke
parents:
diff changeset
1536 // stride < 0) when some range_check (rc) is conditionally
a61af66fc99e Initial load
duke
parents:
diff changeset
1537 // executed.
a61af66fc99e Initial load
duke
parents:
diff changeset
1538 bool conditional_rc = false;
a61af66fc99e Initial load
duke
parents:
diff changeset
1539
a61af66fc99e Initial load
duke
parents:
diff changeset
1540 // Check loop body for tests of trip-counter plus loop-invariant vs
a61af66fc99e Initial load
duke
parents:
diff changeset
1541 // loop-invariant.
a61af66fc99e Initial load
duke
parents:
diff changeset
1542 for( uint i = 0; i < loop->_body.size(); i++ ) {
a61af66fc99e Initial load
duke
parents:
diff changeset
1543 Node *iff = loop->_body[i];
a61af66fc99e Initial load
duke
parents:
diff changeset
1544 if( iff->Opcode() == Op_If ) { // Test?
a61af66fc99e Initial load
duke
parents:
diff changeset
1545
a61af66fc99e Initial load
duke
parents:
diff changeset
1546 // Test is an IfNode, has 2 projections. If BOTH are in the loop
a61af66fc99e Initial load
duke
parents:
diff changeset
1547 // we need loop unswitching instead of iteration splitting.
a61af66fc99e Initial load
duke
parents:
diff changeset
1548 Node *exit = loop->is_loop_exit(iff);
a61af66fc99e Initial load
duke
parents:
diff changeset
1549 if( !exit ) continue;
a61af66fc99e Initial load
duke
parents:
diff changeset
1550 int flip = (exit->Opcode() == Op_IfTrue) ? 1 : 0;
a61af66fc99e Initial load
duke
parents:
diff changeset
1551
a61af66fc99e Initial load
duke
parents:
diff changeset
1552 // Get boolean condition to test
a61af66fc99e Initial load
duke
parents:
diff changeset
1553 Node *i1 = iff->in(1);
a61af66fc99e Initial load
duke
parents:
diff changeset
1554 if( !i1->is_Bool() ) continue;
a61af66fc99e Initial load
duke
parents:
diff changeset
1555 BoolNode *bol = i1->as_Bool();
a61af66fc99e Initial load
duke
parents:
diff changeset
1556 BoolTest b_test = bol->_test;
a61af66fc99e Initial load
duke
parents:
diff changeset
1557 // Flip sense of test if exit condition is flipped
a61af66fc99e Initial load
duke
parents:
diff changeset
1558 if( flip )
a61af66fc99e Initial load
duke
parents:
diff changeset
1559 b_test = b_test.negate();
a61af66fc99e Initial load
duke
parents:
diff changeset
1560
a61af66fc99e Initial load
duke
parents:
diff changeset
1561 // Get compare
a61af66fc99e Initial load
duke
parents:
diff changeset
1562 Node *cmp = bol->in(1);
a61af66fc99e Initial load
duke
parents:
diff changeset
1563
a61af66fc99e Initial load
duke
parents:
diff changeset
1564 // Look for trip_counter + offset vs limit
a61af66fc99e Initial load
duke
parents:
diff changeset
1565 Node *rc_exp = cmp->in(1);
a61af66fc99e Initial load
duke
parents:
diff changeset
1566 Node *limit = cmp->in(2);
a61af66fc99e Initial load
duke
parents:
diff changeset
1567 jint scale_con= 1; // Assume trip counter not scaled
a61af66fc99e Initial load
duke
parents:
diff changeset
1568
a61af66fc99e Initial load
duke
parents:
diff changeset
1569 Node *limit_c = get_ctrl(limit);
a61af66fc99e Initial load
duke
parents:
diff changeset
1570 if( loop->is_member(get_loop(limit_c) ) ) {
a61af66fc99e Initial load
duke
parents:
diff changeset
1571 // Compare might have operands swapped; commute them
a61af66fc99e Initial load
duke
parents:
diff changeset
1572 b_test = b_test.commute();
a61af66fc99e Initial load
duke
parents:
diff changeset
1573 rc_exp = cmp->in(2);
a61af66fc99e Initial load
duke
parents:
diff changeset
1574 limit = cmp->in(1);
a61af66fc99e Initial load
duke
parents:
diff changeset
1575 limit_c = get_ctrl(limit);
a61af66fc99e Initial load
duke
parents:
diff changeset
1576 if( loop->is_member(get_loop(limit_c) ) )
a61af66fc99e Initial load
duke
parents:
diff changeset
1577 continue; // Both inputs are loop varying; cannot RCE
a61af66fc99e Initial load
duke
parents:
diff changeset
1578 }
a61af66fc99e Initial load
duke
parents:
diff changeset
1579 // Here we know 'limit' is loop invariant
a61af66fc99e Initial load
duke
parents:
diff changeset
1580
a61af66fc99e Initial load
duke
parents:
diff changeset
1581 // 'limit' maybe pinned below the zero trip test (probably from a
a61af66fc99e Initial load
duke
parents:
diff changeset
1582 // previous round of rce), in which case, it can't be used in the
a61af66fc99e Initial load
duke
parents:
diff changeset
1583 // zero trip test expression which must occur before the zero test's if.
a61af66fc99e Initial load
duke
parents:
diff changeset
1584 if( limit_c == ctrl ) {
a61af66fc99e Initial load
duke
parents:
diff changeset
1585 continue; // Don't rce this check but continue looking for other candidates.
a61af66fc99e Initial load
duke
parents:
diff changeset
1586 }
a61af66fc99e Initial load
duke
parents:
diff changeset
1587
a61af66fc99e Initial load
duke
parents:
diff changeset
1588 // Check for scaled induction variable plus an offset
a61af66fc99e Initial load
duke
parents:
diff changeset
1589 Node *offset = NULL;
a61af66fc99e Initial load
duke
parents:
diff changeset
1590
a61af66fc99e Initial load
duke
parents:
diff changeset
1591 if (!is_scaled_iv_plus_offset(rc_exp, trip_counter, &scale_con, &offset)) {
a61af66fc99e Initial load
duke
parents:
diff changeset
1592 continue;
a61af66fc99e Initial load
duke
parents:
diff changeset
1593 }
a61af66fc99e Initial load
duke
parents:
diff changeset
1594
a61af66fc99e Initial load
duke
parents:
diff changeset
1595 Node *offset_c = get_ctrl(offset);
a61af66fc99e Initial load
duke
parents:
diff changeset
1596 if( loop->is_member( get_loop(offset_c) ) )
a61af66fc99e Initial load
duke
parents:
diff changeset
1597 continue; // Offset is not really loop invariant
a61af66fc99e Initial load
duke
parents:
diff changeset
1598 // Here we know 'offset' is loop invariant.
a61af66fc99e Initial load
duke
parents:
diff changeset
1599
a61af66fc99e Initial load
duke
parents:
diff changeset
1600 // As above for the 'limit', the 'offset' maybe pinned below the
a61af66fc99e Initial load
duke
parents:
diff changeset
1601 // zero trip test.
a61af66fc99e Initial load
duke
parents:
diff changeset
1602 if( offset_c == ctrl ) {
a61af66fc99e Initial load
duke
parents:
diff changeset
1603 continue; // Don't rce this check but continue looking for other candidates.
a61af66fc99e Initial load
duke
parents:
diff changeset
1604 }
a61af66fc99e Initial load
duke
parents:
diff changeset
1605
a61af66fc99e Initial load
duke
parents:
diff changeset
1606 // At this point we have the expression as:
a61af66fc99e Initial load
duke
parents:
diff changeset
1607 // scale_con * trip_counter + offset :: limit
a61af66fc99e Initial load
duke
parents:
diff changeset
1608 // where scale_con, offset and limit are loop invariant. Trip_counter
a61af66fc99e Initial load
duke
parents:
diff changeset
1609 // monotonically increases by stride_con, a constant. Both (or either)
a61af66fc99e Initial load
duke
parents:
diff changeset
1610 // stride_con and scale_con can be negative which will flip about the
a61af66fc99e Initial load
duke
parents:
diff changeset
1611 // sense of the test.
a61af66fc99e Initial load
duke
parents:
diff changeset
1612
a61af66fc99e Initial load
duke
parents:
diff changeset
1613 // Adjust pre and main loop limits to guard the correct iteration set
a61af66fc99e Initial load
duke
parents:
diff changeset
1614 if( cmp->Opcode() == Op_CmpU ) {// Unsigned compare is really 2 tests
a61af66fc99e Initial load
duke
parents:
diff changeset
1615 if( b_test._test == BoolTest::lt ) { // Range checks always use lt
a61af66fc99e Initial load
duke
parents:
diff changeset
1616 // The overflow limit: scale*I+offset < limit
a61af66fc99e Initial load
duke
parents:
diff changeset
1617 add_constraint( stride_con, scale_con, offset, limit, pre_ctrl, &pre_limit, &main_limit );
a61af66fc99e Initial load
duke
parents:
diff changeset
1618 // The underflow limit: 0 <= scale*I+offset.
a61af66fc99e Initial load
duke
parents:
diff changeset
1619 // Some math yields: -scale*I-(offset+1) < 0
a61af66fc99e Initial load
duke
parents:
diff changeset
1620 Node *plus_one = new (C, 3) AddINode( offset, one );
a61af66fc99e Initial load
duke
parents:
diff changeset
1621 register_new_node( plus_one, pre_ctrl );
a61af66fc99e Initial load
duke
parents:
diff changeset
1622 Node *neg_offset = new (C, 3) SubINode( zero, plus_one );
a61af66fc99e Initial load
duke
parents:
diff changeset
1623 register_new_node( neg_offset, pre_ctrl );
a61af66fc99e Initial load
duke
parents:
diff changeset
1624 add_constraint( stride_con, -scale_con, neg_offset, zero, pre_ctrl, &pre_limit, &main_limit );
a61af66fc99e Initial load
duke
parents:
diff changeset
1625 if (!conditional_rc) {
a61af66fc99e Initial load
duke
parents:
diff changeset
1626 conditional_rc = !loop->dominates_backedge(iff);
a61af66fc99e Initial load
duke
parents:
diff changeset
1627 }
a61af66fc99e Initial load
duke
parents:
diff changeset
1628 } else {
a61af66fc99e Initial load
duke
parents:
diff changeset
1629 #ifndef PRODUCT
a61af66fc99e Initial load
duke
parents:
diff changeset
1630 if( PrintOpto )
a61af66fc99e Initial load
duke
parents:
diff changeset
1631 tty->print_cr("missed RCE opportunity");
a61af66fc99e Initial load
duke
parents:
diff changeset
1632 #endif
a61af66fc99e Initial load
duke
parents:
diff changeset
1633 continue; // In release mode, ignore it
a61af66fc99e Initial load
duke
parents:
diff changeset
1634 }
a61af66fc99e Initial load
duke
parents:
diff changeset
1635 } else { // Otherwise work on normal compares
a61af66fc99e Initial load
duke
parents:
diff changeset
1636 switch( b_test._test ) {
a61af66fc99e Initial load
duke
parents:
diff changeset
1637 case BoolTest::ge: // Convert X >= Y to -X <= -Y
a61af66fc99e Initial load
duke
parents:
diff changeset
1638 scale_con = -scale_con;
a61af66fc99e Initial load
duke
parents:
diff changeset
1639 offset = new (C, 3) SubINode( zero, offset );
a61af66fc99e Initial load
duke
parents:
diff changeset
1640 register_new_node( offset, pre_ctrl );
a61af66fc99e Initial load
duke
parents:
diff changeset
1641 limit = new (C, 3) SubINode( zero, limit );
a61af66fc99e Initial load
duke
parents:
diff changeset
1642 register_new_node( limit, pre_ctrl );
a61af66fc99e Initial load
duke
parents:
diff changeset
1643 // Fall into LE case
a61af66fc99e Initial load
duke
parents:
diff changeset
1644 case BoolTest::le: // Convert X <= Y to X < Y+1
a61af66fc99e Initial load
duke
parents:
diff changeset
1645 limit = new (C, 3) AddINode( limit, one );
a61af66fc99e Initial load
duke
parents:
diff changeset
1646 register_new_node( limit, pre_ctrl );
a61af66fc99e Initial load
duke
parents:
diff changeset
1647 // Fall into LT case
a61af66fc99e Initial load
duke
parents:
diff changeset
1648 case BoolTest::lt:
a61af66fc99e Initial load
duke
parents:
diff changeset
1649 add_constraint( stride_con, scale_con, offset, limit, pre_ctrl, &pre_limit, &main_limit );
a61af66fc99e Initial load
duke
parents:
diff changeset
1650 if (!conditional_rc) {
a61af66fc99e Initial load
duke
parents:
diff changeset
1651 conditional_rc = !loop->dominates_backedge(iff);
a61af66fc99e Initial load
duke
parents:
diff changeset
1652 }
a61af66fc99e Initial load
duke
parents:
diff changeset
1653 break;
a61af66fc99e Initial load
duke
parents:
diff changeset
1654 default:
a61af66fc99e Initial load
duke
parents:
diff changeset
1655 #ifndef PRODUCT
a61af66fc99e Initial load
duke
parents:
diff changeset
1656 if( PrintOpto )
a61af66fc99e Initial load
duke
parents:
diff changeset
1657 tty->print_cr("missed RCE opportunity");
a61af66fc99e Initial load
duke
parents:
diff changeset
1658 #endif
a61af66fc99e Initial load
duke
parents:
diff changeset
1659 continue; // Unhandled case
a61af66fc99e Initial load
duke
parents:
diff changeset
1660 }
a61af66fc99e Initial load
duke
parents:
diff changeset
1661 }
a61af66fc99e Initial load
duke
parents:
diff changeset
1662
a61af66fc99e Initial load
duke
parents:
diff changeset
1663 // Kill the eliminated test
a61af66fc99e Initial load
duke
parents:
diff changeset
1664 C->set_major_progress();
a61af66fc99e Initial load
duke
parents:
diff changeset
1665 Node *kill_con = _igvn.intcon( 1-flip );
a61af66fc99e Initial load
duke
parents:
diff changeset
1666 set_ctrl(kill_con, C->root());
a61af66fc99e Initial load
duke
parents:
diff changeset
1667 _igvn.hash_delete(iff);
a61af66fc99e Initial load
duke
parents:
diff changeset
1668 iff->set_req(1, kill_con);
a61af66fc99e Initial load
duke
parents:
diff changeset
1669 _igvn._worklist.push(iff);
a61af66fc99e Initial load
duke
parents:
diff changeset
1670 // Find surviving projection
a61af66fc99e Initial load
duke
parents:
diff changeset
1671 assert(iff->is_If(), "");
a61af66fc99e Initial load
duke
parents:
diff changeset
1672 ProjNode* dp = ((IfNode*)iff)->proj_out(1-flip);
a61af66fc99e Initial load
duke
parents:
diff changeset
1673 // Find loads off the surviving projection; remove their control edge
a61af66fc99e Initial load
duke
parents:
diff changeset
1674 for (DUIterator_Fast imax, i = dp->fast_outs(imax); i < imax; i++) {
a61af66fc99e Initial load
duke
parents:
diff changeset
1675 Node* cd = dp->fast_out(i); // Control-dependent node
a61af66fc99e Initial load
duke
parents:
diff changeset
1676 if( cd->is_Load() ) { // Loads can now float around in the loop
a61af66fc99e Initial load
duke
parents:
diff changeset
1677 _igvn.hash_delete(cd);
a61af66fc99e Initial load
duke
parents:
diff changeset
1678 // Allow the load to float around in the loop, or before it
a61af66fc99e Initial load
duke
parents:
diff changeset
1679 // but NOT before the pre-loop.
a61af66fc99e Initial load
duke
parents:
diff changeset
1680 cd->set_req(0, ctrl); // ctrl, not NULL
a61af66fc99e Initial load
duke
parents:
diff changeset
1681 _igvn._worklist.push(cd);
a61af66fc99e Initial load
duke
parents:
diff changeset
1682 --i;
a61af66fc99e Initial load
duke
parents:
diff changeset
1683 --imax;
a61af66fc99e Initial load
duke
parents:
diff changeset
1684 }
a61af66fc99e Initial load
duke
parents:
diff changeset
1685 }
a61af66fc99e Initial load
duke
parents:
diff changeset
1686
a61af66fc99e Initial load
duke
parents:
diff changeset
1687 } // End of is IF
a61af66fc99e Initial load
duke
parents:
diff changeset
1688
a61af66fc99e Initial load
duke
parents:
diff changeset
1689 }
a61af66fc99e Initial load
duke
parents:
diff changeset
1690
a61af66fc99e Initial load
duke
parents:
diff changeset
1691 // Update loop limits
a61af66fc99e Initial load
duke
parents:
diff changeset
1692 if (conditional_rc) {
a61af66fc99e Initial load
duke
parents:
diff changeset
1693 pre_limit = (stride_con > 0) ? (Node*)new (C,3) MinINode(pre_limit, orig_limit)
a61af66fc99e Initial load
duke
parents:
diff changeset
1694 : (Node*)new (C,3) MaxINode(pre_limit, orig_limit);
a61af66fc99e Initial load
duke
parents:
diff changeset
1695 register_new_node(pre_limit, pre_ctrl);
a61af66fc99e Initial load
duke
parents:
diff changeset
1696 }
a61af66fc99e Initial load
duke
parents:
diff changeset
1697 _igvn.hash_delete(pre_opaq);
a61af66fc99e Initial load
duke
parents:
diff changeset
1698 pre_opaq->set_req(1, pre_limit);
a61af66fc99e Initial load
duke
parents:
diff changeset
1699
a61af66fc99e Initial load
duke
parents:
diff changeset
1700 // Note:: we are making the main loop limit no longer precise;
a61af66fc99e Initial load
duke
parents:
diff changeset
1701 // need to round up based on stride.
a61af66fc99e Initial load
duke
parents:
diff changeset
1702 if( stride_con != 1 && stride_con != -1 ) { // Cutout for common case
a61af66fc99e Initial load
duke
parents:
diff changeset
1703 // "Standard" round-up logic: ([main_limit-init+(y-1)]/y)*y+init
a61af66fc99e Initial load
duke
parents:
diff changeset
1704 // Hopefully, compiler will optimize for powers of 2.
a61af66fc99e Initial load
duke
parents:
diff changeset
1705 Node *ctrl = get_ctrl(main_limit);
a61af66fc99e Initial load
duke
parents:
diff changeset
1706 Node *stride = cl->stride();
a61af66fc99e Initial load
duke
parents:
diff changeset
1707 Node *init = cl->init_trip();
a61af66fc99e Initial load
duke
parents:
diff changeset
1708 Node *span = new (C, 3) SubINode(main_limit,init);
a61af66fc99e Initial load
duke
parents:
diff changeset
1709 register_new_node(span,ctrl);
a61af66fc99e Initial load
duke
parents:
diff changeset
1710 Node *rndup = _igvn.intcon(stride_con + ((stride_con>0)?-1:1));
a61af66fc99e Initial load
duke
parents:
diff changeset
1711 Node *add = new (C, 3) AddINode(span,rndup);
a61af66fc99e Initial load
duke
parents:
diff changeset
1712 register_new_node(add,ctrl);
a61af66fc99e Initial load
duke
parents:
diff changeset
1713 Node *div = new (C, 3) DivINode(0,add,stride);
a61af66fc99e Initial load
duke
parents:
diff changeset
1714 register_new_node(div,ctrl);
a61af66fc99e Initial load
duke
parents:
diff changeset
1715 Node *mul = new (C, 3) MulINode(div,stride);
a61af66fc99e Initial load
duke
parents:
diff changeset
1716 register_new_node(mul,ctrl);
a61af66fc99e Initial load
duke
parents:
diff changeset
1717 Node *newlim = new (C, 3) AddINode(mul,init);
a61af66fc99e Initial load
duke
parents:
diff changeset
1718 register_new_node(newlim,ctrl);
a61af66fc99e Initial load
duke
parents:
diff changeset
1719 main_limit = newlim;
a61af66fc99e Initial load
duke
parents:
diff changeset
1720 }
a61af66fc99e Initial load
duke
parents:
diff changeset
1721
a61af66fc99e Initial load
duke
parents:
diff changeset
1722 Node *main_cle = cl->loopexit();
a61af66fc99e Initial load
duke
parents:
diff changeset
1723 Node *main_bol = main_cle->in(1);
a61af66fc99e Initial load
duke
parents:
diff changeset
1724 // Hacking loop bounds; need private copies of exit test
a61af66fc99e Initial load
duke
parents:
diff changeset
1725 if( main_bol->outcnt() > 1 ) {// BoolNode shared?
a61af66fc99e Initial load
duke
parents:
diff changeset
1726 _igvn.hash_delete(main_cle);
a61af66fc99e Initial load
duke
parents:
diff changeset
1727 main_bol = main_bol->clone();// Clone a private BoolNode
a61af66fc99e Initial load
duke
parents:
diff changeset
1728 register_new_node( main_bol, main_cle->in(0) );
a61af66fc99e Initial load
duke
parents:
diff changeset
1729 main_cle->set_req(1,main_bol);
a61af66fc99e Initial load
duke
parents:
diff changeset
1730 }
a61af66fc99e Initial load
duke
parents:
diff changeset
1731 Node *main_cmp = main_bol->in(1);
a61af66fc99e Initial load
duke
parents:
diff changeset
1732 if( main_cmp->outcnt() > 1 ) { // CmpNode shared?
a61af66fc99e Initial load
duke
parents:
diff changeset
1733 _igvn.hash_delete(main_bol);
a61af66fc99e Initial load
duke
parents:
diff changeset
1734 main_cmp = main_cmp->clone();// Clone a private CmpNode
a61af66fc99e Initial load
duke
parents:
diff changeset
1735 register_new_node( main_cmp, main_cle->in(0) );
a61af66fc99e Initial load
duke
parents:
diff changeset
1736 main_bol->set_req(1,main_cmp);
a61af66fc99e Initial load
duke
parents:
diff changeset
1737 }
a61af66fc99e Initial load
duke
parents:
diff changeset
1738 // Hack the now-private loop bounds
a61af66fc99e Initial load
duke
parents:
diff changeset
1739 _igvn.hash_delete(main_cmp);
a61af66fc99e Initial load
duke
parents:
diff changeset
1740 main_cmp->set_req(2, main_limit);
a61af66fc99e Initial load
duke
parents:
diff changeset
1741 _igvn._worklist.push(main_cmp);
a61af66fc99e Initial load
duke
parents:
diff changeset
1742 // The OpaqueNode is unshared by design
a61af66fc99e Initial load
duke
parents:
diff changeset
1743 _igvn.hash_delete(opqzm);
a61af66fc99e Initial load
duke
parents:
diff changeset
1744 assert( opqzm->outcnt() == 1, "cannot hack shared node" );
a61af66fc99e Initial load
duke
parents:
diff changeset
1745 opqzm->set_req(1,main_limit);
a61af66fc99e Initial load
duke
parents:
diff changeset
1746 _igvn._worklist.push(opqzm);
a61af66fc99e Initial load
duke
parents:
diff changeset
1747 }
a61af66fc99e Initial load
duke
parents:
diff changeset
1748
a61af66fc99e Initial load
duke
parents:
diff changeset
1749 //------------------------------DCE_loop_body----------------------------------
a61af66fc99e Initial load
duke
parents:
diff changeset
1750 // Remove simplistic dead code from loop body
a61af66fc99e Initial load
duke
parents:
diff changeset
1751 void IdealLoopTree::DCE_loop_body() {
a61af66fc99e Initial load
duke
parents:
diff changeset
1752 for( uint i = 0; i < _body.size(); i++ )
a61af66fc99e Initial load
duke
parents:
diff changeset
1753 if( _body.at(i)->outcnt() == 0 )
a61af66fc99e Initial load
duke
parents:
diff changeset
1754 _body.map( i--, _body.pop() );
a61af66fc99e Initial load
duke
parents:
diff changeset
1755 }
a61af66fc99e Initial load
duke
parents:
diff changeset
1756
a61af66fc99e Initial load
duke
parents:
diff changeset
1757
a61af66fc99e Initial load
duke
parents:
diff changeset
1758 //------------------------------adjust_loop_exit_prob--------------------------
a61af66fc99e Initial load
duke
parents:
diff changeset
1759 // Look for loop-exit tests with the 50/50 (or worse) guesses from the parsing stage.
a61af66fc99e Initial load
duke
parents:
diff changeset
1760 // Replace with a 1-in-10 exit guess.
a61af66fc99e Initial load
duke
parents:
diff changeset
1761 void IdealLoopTree::adjust_loop_exit_prob( PhaseIdealLoop *phase ) {
a61af66fc99e Initial load
duke
parents:
diff changeset
1762 Node *test = tail();
a61af66fc99e Initial load
duke
parents:
diff changeset
1763 while( test != _head ) {
a61af66fc99e Initial load
duke
parents:
diff changeset
1764 uint top = test->Opcode();
a61af66fc99e Initial load
duke
parents:
diff changeset
1765 if( top == Op_IfTrue || top == Op_IfFalse ) {
a61af66fc99e Initial load
duke
parents:
diff changeset
1766 int test_con = ((ProjNode*)test)->_con;
a61af66fc99e Initial load
duke
parents:
diff changeset
1767 assert(top == (uint)(test_con? Op_IfTrue: Op_IfFalse), "sanity");
a61af66fc99e Initial load
duke
parents:
diff changeset
1768 IfNode *iff = test->in(0)->as_If();
a61af66fc99e Initial load
duke
parents:
diff changeset
1769 if( iff->outcnt() == 2 ) { // Ignore dead tests
a61af66fc99e Initial load
duke
parents:
diff changeset
1770 Node *bol = iff->in(1);
a61af66fc99e Initial load
duke
parents:
diff changeset
1771 if( bol && bol->req() > 1 && bol->in(1) &&
a61af66fc99e Initial load
duke
parents:
diff changeset
1772 ((bol->in(1)->Opcode() == Op_StorePConditional ) ||
420
a1980da045cc 6462850: generate biased locking code in C2 ideal graph
kvn
parents: 401
diff changeset
1773 (bol->in(1)->Opcode() == Op_StoreIConditional ) ||
0
a61af66fc99e Initial load
duke
parents:
diff changeset
1774 (bol->in(1)->Opcode() == Op_StoreLConditional ) ||
a61af66fc99e Initial load
duke
parents:
diff changeset
1775 (bol->in(1)->Opcode() == Op_CompareAndSwapI ) ||
a61af66fc99e Initial load
duke
parents:
diff changeset
1776 (bol->in(1)->Opcode() == Op_CompareAndSwapL ) ||
113
ba764ed4b6f2 6420645: Create a vm that uses compressed oops for up to 32gb heapsizes
coleenp
parents: 39
diff changeset
1777 (bol->in(1)->Opcode() == Op_CompareAndSwapP ) ||
ba764ed4b6f2 6420645: Create a vm that uses compressed oops for up to 32gb heapsizes
coleenp
parents: 39
diff changeset
1778 (bol->in(1)->Opcode() == Op_CompareAndSwapN )))
0
a61af66fc99e Initial load
duke
parents:
diff changeset
1779 return; // Allocation loops RARELY take backedge
a61af66fc99e Initial load
duke
parents:
diff changeset
1780 // Find the OTHER exit path from the IF
a61af66fc99e Initial load
duke
parents:
diff changeset
1781 Node* ex = iff->proj_out(1-test_con);
a61af66fc99e Initial load
duke
parents:
diff changeset
1782 float p = iff->_prob;
a61af66fc99e Initial load
duke
parents:
diff changeset
1783 if( !phase->is_member( this, ex ) && iff->_fcnt == COUNT_UNKNOWN ) {
a61af66fc99e Initial load
duke
parents:
diff changeset
1784 if( top == Op_IfTrue ) {
a61af66fc99e Initial load
duke
parents:
diff changeset
1785 if( p < (PROB_FAIR + PROB_UNLIKELY_MAG(3))) {
a61af66fc99e Initial load
duke
parents:
diff changeset
1786 iff->_prob = PROB_STATIC_FREQUENT;
a61af66fc99e Initial load
duke
parents:
diff changeset
1787 }
a61af66fc99e Initial load
duke
parents:
diff changeset
1788 } else {
a61af66fc99e Initial load
duke
parents:
diff changeset
1789 if( p > (PROB_FAIR - PROB_UNLIKELY_MAG(3))) {
a61af66fc99e Initial load
duke
parents:
diff changeset
1790 iff->_prob = PROB_STATIC_INFREQUENT;
a61af66fc99e Initial load
duke
parents:
diff changeset
1791 }
a61af66fc99e Initial load
duke
parents:
diff changeset
1792 }
a61af66fc99e Initial load
duke
parents:
diff changeset
1793 }
a61af66fc99e Initial load
duke
parents:
diff changeset
1794 }
a61af66fc99e Initial load
duke
parents:
diff changeset
1795 }
a61af66fc99e Initial load
duke
parents:
diff changeset
1796 test = phase->idom(test);
a61af66fc99e Initial load
duke
parents:
diff changeset
1797 }
a61af66fc99e Initial load
duke
parents:
diff changeset
1798 }
a61af66fc99e Initial load
duke
parents:
diff changeset
1799
a61af66fc99e Initial load
duke
parents:
diff changeset
1800
a61af66fc99e Initial load
duke
parents:
diff changeset
1801 //------------------------------policy_do_remove_empty_loop--------------------
a61af66fc99e Initial load
duke
parents:
diff changeset
1802 // Micro-benchmark spamming. Policy is to always remove empty loops.
a61af66fc99e Initial load
duke
parents:
diff changeset
1803 // The 'DO' part is to replace the trip counter with the value it will
a61af66fc99e Initial load
duke
parents:
diff changeset
1804 // have on the last iteration. This will break the loop.
a61af66fc99e Initial load
duke
parents:
diff changeset
1805 bool IdealLoopTree::policy_do_remove_empty_loop( PhaseIdealLoop *phase ) {
a61af66fc99e Initial load
duke
parents:
diff changeset
1806 // Minimum size must be empty loop
2453
d7a3fed1c1c9 7004547: regular loop unroll should not unroll more than max unrolling
kvn
parents: 2448
diff changeset
1807 if (_body.size() > EMPTY_LOOP_SIZE)
2383
9dc311b8473e 7008866: Missing loop predicate for loop with multiple entries
kvn
parents: 1972
diff changeset
1808 return false;
0
a61af66fc99e Initial load
duke
parents:
diff changeset
1809
2383
9dc311b8473e 7008866: Missing loop predicate for loop with multiple entries
kvn
parents: 1972
diff changeset
1810 if (!_head->is_CountedLoop())
9dc311b8473e 7008866: Missing loop predicate for loop with multiple entries
kvn
parents: 1972
diff changeset
1811 return false; // Dead loop
0
a61af66fc99e Initial load
duke
parents:
diff changeset
1812 CountedLoopNode *cl = _head->as_CountedLoop();
2383
9dc311b8473e 7008866: Missing loop predicate for loop with multiple entries
kvn
parents: 1972
diff changeset
1813 if (!cl->loopexit())
9dc311b8473e 7008866: Missing loop predicate for loop with multiple entries
kvn
parents: 1972
diff changeset
1814 return false; // Malformed loop
9dc311b8473e 7008866: Missing loop predicate for loop with multiple entries
kvn
parents: 1972
diff changeset
1815 if (!phase->is_member(this, phase->get_ctrl(cl->loopexit()->in(CountedLoopEndNode::TestValue))))
0
a61af66fc99e Initial load
duke
parents:
diff changeset
1816 return false; // Infinite loop
2403
1927db75dd85 7024475: loop doesn't terminate when compiled
never
parents: 2383
diff changeset
1817
0
a61af66fc99e Initial load
duke
parents:
diff changeset
1818 #ifdef ASSERT
a61af66fc99e Initial load
duke
parents:
diff changeset
1819 // Ensure only one phi which is the iv.
a61af66fc99e Initial load
duke
parents:
diff changeset
1820 Node* iv = NULL;
a61af66fc99e Initial load
duke
parents:
diff changeset
1821 for (DUIterator_Fast imax, i = cl->fast_outs(imax); i < imax; i++) {
a61af66fc99e Initial load
duke
parents:
diff changeset
1822 Node* n = cl->fast_out(i);
a61af66fc99e Initial load
duke
parents:
diff changeset
1823 if (n->Opcode() == Op_Phi) {
a61af66fc99e Initial load
duke
parents:
diff changeset
1824 assert(iv == NULL, "Too many phis" );
a61af66fc99e Initial load
duke
parents:
diff changeset
1825 iv = n;
a61af66fc99e Initial load
duke
parents:
diff changeset
1826 }
a61af66fc99e Initial load
duke
parents:
diff changeset
1827 }
a61af66fc99e Initial load
duke
parents:
diff changeset
1828 assert(iv == cl->phi(), "Wrong phi" );
a61af66fc99e Initial load
duke
parents:
diff changeset
1829 #endif
2403
1927db75dd85 7024475: loop doesn't terminate when compiled
never
parents: 2383
diff changeset
1830
1927db75dd85 7024475: loop doesn't terminate when compiled
never
parents: 2383
diff changeset
1831 // main and post loops have explicitly created zero trip guard
1927db75dd85 7024475: loop doesn't terminate when compiled
never
parents: 2383
diff changeset
1832 bool needs_guard = !cl->is_main_loop() && !cl->is_post_loop();
1927db75dd85 7024475: loop doesn't terminate when compiled
never
parents: 2383
diff changeset
1833 if (needs_guard) {
2465
3af54845df98 7004555: Add new policy for one iteration loops
kvn
parents: 2453
diff changeset
1834 // Skip guard if values not overlap.
3af54845df98 7004555: Add new policy for one iteration loops
kvn
parents: 2453
diff changeset
1835 const TypeInt* init_t = phase->_igvn.type(cl->init_trip())->is_int();
3af54845df98 7004555: Add new policy for one iteration loops
kvn
parents: 2453
diff changeset
1836 const TypeInt* limit_t = phase->_igvn.type(cl->limit())->is_int();
3af54845df98 7004555: Add new policy for one iteration loops
kvn
parents: 2453
diff changeset
1837 int stride_con = cl->stride_con();
3af54845df98 7004555: Add new policy for one iteration loops
kvn
parents: 2453
diff changeset
1838 if (stride_con > 0) {
3af54845df98 7004555: Add new policy for one iteration loops
kvn
parents: 2453
diff changeset
1839 needs_guard = (init_t->_hi >= limit_t->_lo);
3af54845df98 7004555: Add new policy for one iteration loops
kvn
parents: 2453
diff changeset
1840 } else {
3af54845df98 7004555: Add new policy for one iteration loops
kvn
parents: 2453
diff changeset
1841 needs_guard = (init_t->_lo <= limit_t->_hi);
3af54845df98 7004555: Add new policy for one iteration loops
kvn
parents: 2453
diff changeset
1842 }
3af54845df98 7004555: Add new policy for one iteration loops
kvn
parents: 2453
diff changeset
1843 }
3af54845df98 7004555: Add new policy for one iteration loops
kvn
parents: 2453
diff changeset
1844 if (needs_guard) {
2403
1927db75dd85 7024475: loop doesn't terminate when compiled
never
parents: 2383
diff changeset
1845 // Check for an obvious zero trip guard.
2445
08eb13460b3a 7004535: Clone loop predicate during loop unswitch
kvn
parents: 2417
diff changeset
1846 Node* inctrl = PhaseIdealLoop::skip_loop_predicates(cl->in(LoopNode::EntryControl));
2403
1927db75dd85 7024475: loop doesn't terminate when compiled
never
parents: 2383
diff changeset
1847 if (inctrl->Opcode() == Op_IfTrue) {
1927db75dd85 7024475: loop doesn't terminate when compiled
never
parents: 2383
diff changeset
1848 // The test should look like just the backedge of a CountedLoop
1927db75dd85 7024475: loop doesn't terminate when compiled
never
parents: 2383
diff changeset
1849 Node* iff = inctrl->in(0);
1927db75dd85 7024475: loop doesn't terminate when compiled
never
parents: 2383
diff changeset
1850 if (iff->is_If()) {
1927db75dd85 7024475: loop doesn't terminate when compiled
never
parents: 2383
diff changeset
1851 Node* bol = iff->in(1);
1927db75dd85 7024475: loop doesn't terminate when compiled
never
parents: 2383
diff changeset
1852 if (bol->is_Bool() && bol->as_Bool()->_test._test == cl->loopexit()->test_trip()) {
1927db75dd85 7024475: loop doesn't terminate when compiled
never
parents: 2383
diff changeset
1853 Node* cmp = bol->in(1);
1927db75dd85 7024475: loop doesn't terminate when compiled
never
parents: 2383
diff changeset
1854 if (cmp->is_Cmp() && cmp->in(1) == cl->init_trip() && cmp->in(2) == cl->limit()) {
1927db75dd85 7024475: loop doesn't terminate when compiled
never
parents: 2383
diff changeset
1855 needs_guard = false;
1927db75dd85 7024475: loop doesn't terminate when compiled
never
parents: 2383
diff changeset
1856 }
1927db75dd85 7024475: loop doesn't terminate when compiled
never
parents: 2383
diff changeset
1857 }
1927db75dd85 7024475: loop doesn't terminate when compiled
never
parents: 2383
diff changeset
1858 }
1927db75dd85 7024475: loop doesn't terminate when compiled
never
parents: 2383
diff changeset
1859 }
1927db75dd85 7024475: loop doesn't terminate when compiled
never
parents: 2383
diff changeset
1860 }
1927db75dd85 7024475: loop doesn't terminate when compiled
never
parents: 2383
diff changeset
1861
1927db75dd85 7024475: loop doesn't terminate when compiled
never
parents: 2383
diff changeset
1862 #ifndef PRODUCT
1927db75dd85 7024475: loop doesn't terminate when compiled
never
parents: 2383
diff changeset
1863 if (PrintOpto) {
1927db75dd85 7024475: loop doesn't terminate when compiled
never
parents: 2383
diff changeset
1864 tty->print("Removing empty loop with%s zero trip guard", needs_guard ? "out" : "");
1927db75dd85 7024475: loop doesn't terminate when compiled
never
parents: 2383
diff changeset
1865 this->dump_head();
1927db75dd85 7024475: loop doesn't terminate when compiled
never
parents: 2383
diff changeset
1866 } else if (TraceLoopOpts) {
1927db75dd85 7024475: loop doesn't terminate when compiled
never
parents: 2383
diff changeset
1867 tty->print("Empty with%s zero trip guard ", needs_guard ? "out" : "");
1927db75dd85 7024475: loop doesn't terminate when compiled
never
parents: 2383
diff changeset
1868 this->dump_head();
1927db75dd85 7024475: loop doesn't terminate when compiled
never
parents: 2383
diff changeset
1869 }
1927db75dd85 7024475: loop doesn't terminate when compiled
never
parents: 2383
diff changeset
1870 #endif
1927db75dd85 7024475: loop doesn't terminate when compiled
never
parents: 2383
diff changeset
1871
1927db75dd85 7024475: loop doesn't terminate when compiled
never
parents: 2383
diff changeset
1872 if (needs_guard) {
1927db75dd85 7024475: loop doesn't terminate when compiled
never
parents: 2383
diff changeset
1873 // Peel the loop to ensure there's a zero trip guard
1927db75dd85 7024475: loop doesn't terminate when compiled
never
parents: 2383
diff changeset
1874 Node_List old_new;
1927db75dd85 7024475: loop doesn't terminate when compiled
never
parents: 2383
diff changeset
1875 phase->do_peeling(this, old_new);
1927db75dd85 7024475: loop doesn't terminate when compiled
never
parents: 2383
diff changeset
1876 }
1927db75dd85 7024475: loop doesn't terminate when compiled
never
parents: 2383
diff changeset
1877
0
a61af66fc99e Initial load
duke
parents:
diff changeset
1878 // Replace the phi at loop head with the final value of the last
a61af66fc99e Initial load
duke
parents:
diff changeset
1879 // iteration. Then the CountedLoopEnd will collapse (backedge never
a61af66fc99e Initial load
duke
parents:
diff changeset
1880 // taken) and all loop-invariant uses of the exit values will be correct.
a61af66fc99e Initial load
duke
parents:
diff changeset
1881 Node *phi = cl->phi();
a61af66fc99e Initial load
duke
parents:
diff changeset
1882 Node *final = new (phase->C, 3) SubINode( cl->limit(), cl->stride() );
a61af66fc99e Initial load
duke
parents:
diff changeset
1883 phase->register_new_node(final,cl->in(LoopNode::EntryControl));
1621
6027dddc26c6 6677629: PhaseIterGVN::subsume_node() should call hash_delete() and add_users_to_worklist()
kvn
parents: 1552
diff changeset
1884 phase->_igvn.replace_node(phi,final);
0
a61af66fc99e Initial load
duke
parents:
diff changeset
1885 phase->C->set_major_progress();
a61af66fc99e Initial load
duke
parents:
diff changeset
1886 return true;
a61af66fc99e Initial load
duke
parents:
diff changeset
1887 }
a61af66fc99e Initial load
duke
parents:
diff changeset
1888
2465
3af54845df98 7004555: Add new policy for one iteration loops
kvn
parents: 2453
diff changeset
1889 //------------------------------policy_do_one_iteration_loop-------------------
3af54845df98 7004555: Add new policy for one iteration loops
kvn
parents: 2453
diff changeset
1890 // Convert one iteration loop into normal code.
3af54845df98 7004555: Add new policy for one iteration loops
kvn
parents: 2453
diff changeset
1891 bool IdealLoopTree::policy_do_one_iteration_loop( PhaseIdealLoop *phase ) {
3af54845df98 7004555: Add new policy for one iteration loops
kvn
parents: 2453
diff changeset
1892 if (!_head->as_Loop()->is_valid_counted_loop())
3af54845df98 7004555: Add new policy for one iteration loops
kvn
parents: 2453
diff changeset
1893 return false; // Only for counted loop
3af54845df98 7004555: Add new policy for one iteration loops
kvn
parents: 2453
diff changeset
1894
3af54845df98 7004555: Add new policy for one iteration loops
kvn
parents: 2453
diff changeset
1895 CountedLoopNode *cl = _head->as_CountedLoop();
3af54845df98 7004555: Add new policy for one iteration loops
kvn
parents: 2453
diff changeset
1896 if (!cl->has_exact_trip_count() || cl->trip_count() != 1) {
3af54845df98 7004555: Add new policy for one iteration loops
kvn
parents: 2453
diff changeset
1897 return false;
3af54845df98 7004555: Add new policy for one iteration loops
kvn
parents: 2453
diff changeset
1898 }
3af54845df98 7004555: Add new policy for one iteration loops
kvn
parents: 2453
diff changeset
1899
3af54845df98 7004555: Add new policy for one iteration loops
kvn
parents: 2453
diff changeset
1900 #ifndef PRODUCT
3af54845df98 7004555: Add new policy for one iteration loops
kvn
parents: 2453
diff changeset
1901 if(TraceLoopOpts) {
3af54845df98 7004555: Add new policy for one iteration loops
kvn
parents: 2453
diff changeset
1902 tty->print("OneIteration ");
3af54845df98 7004555: Add new policy for one iteration loops
kvn
parents: 2453
diff changeset
1903 this->dump_head();
3af54845df98 7004555: Add new policy for one iteration loops
kvn
parents: 2453
diff changeset
1904 }
3af54845df98 7004555: Add new policy for one iteration loops
kvn
parents: 2453
diff changeset
1905 #endif
3af54845df98 7004555: Add new policy for one iteration loops
kvn
parents: 2453
diff changeset
1906
3af54845df98 7004555: Add new policy for one iteration loops
kvn
parents: 2453
diff changeset
1907 Node *init_n = cl->init_trip();
3af54845df98 7004555: Add new policy for one iteration loops
kvn
parents: 2453
diff changeset
1908 #ifdef ASSERT
3af54845df98 7004555: Add new policy for one iteration loops
kvn
parents: 2453
diff changeset
1909 // Loop boundaries should be constant since trip count is exact.
3af54845df98 7004555: Add new policy for one iteration loops
kvn
parents: 2453
diff changeset
1910 assert(init_n->get_int() + cl->stride_con() >= cl->limit()->get_int(), "should be one iteration");
3af54845df98 7004555: Add new policy for one iteration loops
kvn
parents: 2453
diff changeset
1911 #endif
3af54845df98 7004555: Add new policy for one iteration loops
kvn
parents: 2453
diff changeset
1912 // Replace the phi at loop head with the value of the init_trip.
3af54845df98 7004555: Add new policy for one iteration loops
kvn
parents: 2453
diff changeset
1913 // Then the CountedLoopEnd will collapse (backedge will not be taken)
3af54845df98 7004555: Add new policy for one iteration loops
kvn
parents: 2453
diff changeset
1914 // and all loop-invariant uses of the exit values will be correct.
3af54845df98 7004555: Add new policy for one iteration loops
kvn
parents: 2453
diff changeset
1915 phase->_igvn.replace_node(cl->phi(), cl->init_trip());
3af54845df98 7004555: Add new policy for one iteration loops
kvn
parents: 2453
diff changeset
1916 phase->C->set_major_progress();
3af54845df98 7004555: Add new policy for one iteration loops
kvn
parents: 2453
diff changeset
1917 return true;
3af54845df98 7004555: Add new policy for one iteration loops
kvn
parents: 2453
diff changeset
1918 }
0
a61af66fc99e Initial load
duke
parents:
diff changeset
1919
a61af66fc99e Initial load
duke
parents:
diff changeset
1920 //=============================================================================
a61af66fc99e Initial load
duke
parents:
diff changeset
1921 //------------------------------iteration_split_impl---------------------------
401
ee8f06bfb27c 6743188: incomplete fix for 6700047 C2 failed in idom_no_update
never
parents: 400
diff changeset
1922 bool IdealLoopTree::iteration_split_impl( PhaseIdealLoop *phase, Node_List &old_new ) {
2465
3af54845df98 7004555: Add new policy for one iteration loops
kvn
parents: 2453
diff changeset
1923 // Compute exact loop trip count if possible.
3af54845df98 7004555: Add new policy for one iteration loops
kvn
parents: 2453
diff changeset
1924 compute_exact_trip_count(phase);
3af54845df98 7004555: Add new policy for one iteration loops
kvn
parents: 2453
diff changeset
1925
3af54845df98 7004555: Add new policy for one iteration loops
kvn
parents: 2453
diff changeset
1926 // Convert one iteration loop into normal code.
3af54845df98 7004555: Add new policy for one iteration loops
kvn
parents: 2453
diff changeset
1927 if (policy_do_one_iteration_loop(phase))
3af54845df98 7004555: Add new policy for one iteration loops
kvn
parents: 2453
diff changeset
1928 return true;
3af54845df98 7004555: Add new policy for one iteration loops
kvn
parents: 2453
diff changeset
1929
0
a61af66fc99e Initial load
duke
parents:
diff changeset
1930 // Check and remove empty loops (spam micro-benchmarks)
2465
3af54845df98 7004555: Add new policy for one iteration loops
kvn
parents: 2453
diff changeset
1931 if (policy_do_remove_empty_loop(phase))
1172
b2b6a9bf6238 6894779: Loop Predication for Loop Optimizer in C2
cfang
parents: 844
diff changeset
1932 return true; // Here we removed an empty loop
0
a61af66fc99e Initial load
duke
parents:
diff changeset
1933
a61af66fc99e Initial load
duke
parents:
diff changeset
1934 bool should_peel = policy_peeling(phase); // Should we peel?
a61af66fc99e Initial load
duke
parents:
diff changeset
1935
a61af66fc99e Initial load
duke
parents:
diff changeset
1936 bool should_unswitch = policy_unswitching(phase);
a61af66fc99e Initial load
duke
parents:
diff changeset
1937
a61af66fc99e Initial load
duke
parents:
diff changeset
1938 // Non-counted loops may be peeled; exactly 1 iteration is peeled.
a61af66fc99e Initial load
duke
parents:
diff changeset
1939 // This removes loop-invariant tests (usually null checks).
2465
3af54845df98 7004555: Add new policy for one iteration loops
kvn
parents: 2453
diff changeset
1940 if (!_head->is_CountedLoop()) { // Non-counted loop
0
a61af66fc99e Initial load
duke
parents:
diff changeset
1941 if (PartialPeelLoop && phase->partial_peel(this, old_new)) {
401
ee8f06bfb27c 6743188: incomplete fix for 6700047 C2 failed in idom_no_update
never
parents: 400
diff changeset
1942 // Partial peel succeeded so terminate this round of loop opts
ee8f06bfb27c 6743188: incomplete fix for 6700047 C2 failed in idom_no_update
never
parents: 400
diff changeset
1943 return false;
0
a61af66fc99e Initial load
duke
parents:
diff changeset
1944 }
2465
3af54845df98 7004555: Add new policy for one iteration loops
kvn
parents: 2453
diff changeset
1945 if (should_peel) { // Should we peel?
0
a61af66fc99e Initial load
duke
parents:
diff changeset
1946 #ifndef PRODUCT
a61af66fc99e Initial load
duke
parents:
diff changeset
1947 if (PrintOpto) tty->print_cr("should_peel");
a61af66fc99e Initial load
duke
parents:
diff changeset
1948 #endif
a61af66fc99e Initial load
duke
parents:
diff changeset
1949 phase->do_peeling(this,old_new);
2465
3af54845df98 7004555: Add new policy for one iteration loops
kvn
parents: 2453
diff changeset
1950 } else if (should_unswitch) {
0
a61af66fc99e Initial load
duke
parents:
diff changeset
1951 phase->do_unswitching(this, old_new);
a61af66fc99e Initial load
duke
parents:
diff changeset
1952 }
401
ee8f06bfb27c 6743188: incomplete fix for 6700047 C2 failed in idom_no_update
never
parents: 400
diff changeset
1953 return true;
0
a61af66fc99e Initial load
duke
parents:
diff changeset
1954 }
a61af66fc99e Initial load
duke
parents:
diff changeset
1955 CountedLoopNode *cl = _head->as_CountedLoop();
a61af66fc99e Initial load
duke
parents:
diff changeset
1956
2465
3af54845df98 7004555: Add new policy for one iteration loops
kvn
parents: 2453
diff changeset
1957 if (!cl->loopexit()) return true; // Ignore various kinds of broken loops
0
a61af66fc99e Initial load
duke
parents:
diff changeset
1958
a61af66fc99e Initial load
duke
parents:
diff changeset
1959 // Do nothing special to pre- and post- loops
2465
3af54845df98 7004555: Add new policy for one iteration loops
kvn
parents: 2453
diff changeset
1960 if (cl->is_pre_loop() || cl->is_post_loop()) return true;
0
a61af66fc99e Initial load
duke
parents:
diff changeset
1961
a61af66fc99e Initial load
duke
parents:
diff changeset
1962 // Compute loop trip count from profile data
a61af66fc99e Initial load
duke
parents:
diff changeset
1963 compute_profile_trip_cnt(phase);
a61af66fc99e Initial load
duke
parents:
diff changeset
1964
a61af66fc99e Initial load
duke
parents:
diff changeset
1965 // Before attempting fancy unrolling, RCE or alignment, see if we want
a61af66fc99e Initial load
duke
parents:
diff changeset
1966 // to completely unroll this loop or do loop unswitching.
2465
3af54845df98 7004555: Add new policy for one iteration loops
kvn
parents: 2453
diff changeset
1967 if (cl->is_normal_loop()) {
789
273b2358ef1a 6837146: Should perform unswitch before maximally unroll in loop transformation
cfang
parents: 605
diff changeset
1968 if (should_unswitch) {
273b2358ef1a 6837146: Should perform unswitch before maximally unroll in loop transformation
cfang
parents: 605
diff changeset
1969 phase->do_unswitching(this, old_new);
273b2358ef1a 6837146: Should perform unswitch before maximally unroll in loop transformation
cfang
parents: 605
diff changeset
1970 return true;
273b2358ef1a 6837146: Should perform unswitch before maximally unroll in loop transformation
cfang
parents: 605
diff changeset
1971 }
0
a61af66fc99e Initial load
duke
parents:
diff changeset
1972 bool should_maximally_unroll = policy_maximally_unroll(phase);
2465
3af54845df98 7004555: Add new policy for one iteration loops
kvn
parents: 2453
diff changeset
1973 if (should_maximally_unroll) {
0
a61af66fc99e Initial load
duke
parents:
diff changeset
1974 // Here we did some unrolling and peeling. Eventually we will
a61af66fc99e Initial load
duke
parents:
diff changeset
1975 // completely unroll this loop and it will no longer be a loop.
a61af66fc99e Initial load
duke
parents:
diff changeset
1976 phase->do_maximally_unroll(this,old_new);
401
ee8f06bfb27c 6743188: incomplete fix for 6700047 C2 failed in idom_no_update
never
parents: 400
diff changeset
1977 return true;
0
a61af66fc99e Initial load
duke
parents:
diff changeset
1978 }
a61af66fc99e Initial load
duke
parents:
diff changeset
1979 }
a61af66fc99e Initial load
duke
parents:
diff changeset
1980
2453
d7a3fed1c1c9 7004547: regular loop unroll should not unroll more than max unrolling
kvn
parents: 2448
diff changeset
1981 // Skip next optimizations if running low on nodes. Note that
d7a3fed1c1c9 7004547: regular loop unroll should not unroll more than max unrolling
kvn
parents: 2448
diff changeset
1982 // policy_unswitching and policy_maximally_unroll have this check.
d7a3fed1c1c9 7004547: regular loop unroll should not unroll more than max unrolling
kvn
parents: 2448
diff changeset
1983 uint nodes_left = MaxNodeLimit - phase->C->unique();
d7a3fed1c1c9 7004547: regular loop unroll should not unroll more than max unrolling
kvn
parents: 2448
diff changeset
1984 if ((2 * _body.size()) > nodes_left) {
d7a3fed1c1c9 7004547: regular loop unroll should not unroll more than max unrolling
kvn
parents: 2448
diff changeset
1985 return true;
d7a3fed1c1c9 7004547: regular loop unroll should not unroll more than max unrolling
kvn
parents: 2448
diff changeset
1986 }
0
a61af66fc99e Initial load
duke
parents:
diff changeset
1987
a61af66fc99e Initial load
duke
parents:
diff changeset
1988 // Counted loops may be peeled, may need some iterations run up
a61af66fc99e Initial load
duke
parents:
diff changeset
1989 // front for RCE, and may want to align loop refs to a cache
a61af66fc99e Initial load
duke
parents:
diff changeset
1990 // line. Thus we clone a full loop up front whose trip count is
a61af66fc99e Initial load
duke
parents:
diff changeset
1991 // at least 1 (if peeling), but may be several more.
a61af66fc99e Initial load
duke
parents:
diff changeset
1992
a61af66fc99e Initial load
duke
parents:
diff changeset
1993 // The main loop will start cache-line aligned with at least 1
a61af66fc99e Initial load
duke
parents:
diff changeset
1994 // iteration of the unrolled body (zero-trip test required) and
a61af66fc99e Initial load
duke
parents:
diff changeset
1995 // will have some range checks removed.
a61af66fc99e Initial load
duke
parents:
diff changeset
1996
a61af66fc99e Initial load
duke
parents:
diff changeset
1997 // A post-loop will finish any odd iterations (leftover after
a61af66fc99e Initial load
duke
parents:
diff changeset
1998 // unrolling), plus any needed for RCE purposes.
a61af66fc99e Initial load
duke
parents:
diff changeset
1999
a61af66fc99e Initial load
duke
parents:
diff changeset
2000 bool should_unroll = policy_unroll(phase);
a61af66fc99e Initial load
duke
parents:
diff changeset
2001
a61af66fc99e Initial load
duke
parents:
diff changeset
2002 bool should_rce = policy_range_check(phase);
a61af66fc99e Initial load
duke
parents:
diff changeset
2003
a61af66fc99e Initial load
duke
parents:
diff changeset
2004 bool should_align = policy_align(phase);
a61af66fc99e Initial load
duke
parents:
diff changeset
2005
a61af66fc99e Initial load
duke
parents:
diff changeset
2006 // If not RCE'ing (iteration splitting) or Aligning, then we do not
a61af66fc99e Initial load
duke
parents:
diff changeset
2007 // need a pre-loop. We may still need to peel an initial iteration but
a61af66fc99e Initial load
duke
parents:
diff changeset
2008 // we will not be needing an unknown number of pre-iterations.
a61af66fc99e Initial load
duke
parents:
diff changeset
2009 //
a61af66fc99e Initial load
duke
parents:
diff changeset
2010 // Basically, if may_rce_align reports FALSE first time through,
a61af66fc99e Initial load
duke
parents:
diff changeset
2011 // we will not be able to later do RCE or Aligning on this loop.
a61af66fc99e Initial load
duke
parents:
diff changeset
2012 bool may_rce_align = !policy_peel_only(phase) || should_rce || should_align;
a61af66fc99e Initial load
duke
parents:
diff changeset
2013
a61af66fc99e Initial load
duke
parents:
diff changeset
2014 // If we have any of these conditions (RCE, alignment, unrolling) met, then
a61af66fc99e Initial load
duke
parents:
diff changeset
2015 // we switch to the pre-/main-/post-loop model. This model also covers
a61af66fc99e Initial load
duke
parents:
diff changeset
2016 // peeling.
2465
3af54845df98 7004555: Add new policy for one iteration loops
kvn
parents: 2453
diff changeset
2017 if (should_rce || should_align || should_unroll) {
3af54845df98 7004555: Add new policy for one iteration loops
kvn
parents: 2453
diff changeset
2018 if (cl->is_normal_loop()) // Convert to 'pre/main/post' loops
0
a61af66fc99e Initial load
duke
parents:
diff changeset
2019 phase->insert_pre_post_loops(this,old_new, !may_rce_align);
a61af66fc99e Initial load
duke
parents:
diff changeset
2020
a61af66fc99e Initial load
duke
parents:
diff changeset
2021 // Adjust the pre- and main-loop limits to let the pre and post loops run
a61af66fc99e Initial load
duke
parents:
diff changeset
2022 // with full checks, but the main-loop with no checks. Remove said
a61af66fc99e Initial load
duke
parents:
diff changeset
2023 // checks from the main body.
2465
3af54845df98 7004555: Add new policy for one iteration loops
kvn
parents: 2453
diff changeset
2024 if (should_rce)
0
a61af66fc99e Initial load
duke
parents:
diff changeset
2025 phase->do_range_check(this,old_new);
a61af66fc99e Initial load
duke
parents:
diff changeset
2026
a61af66fc99e Initial load
duke
parents:
diff changeset
2027 // Double loop body for unrolling. Adjust the minimum-trip test (will do
a61af66fc99e Initial load
duke
parents:
diff changeset
2028 // twice as many iterations as before) and the main body limit (only do
a61af66fc99e Initial load
duke
parents:
diff changeset
2029 // an even number of trips). If we are peeling, we might enable some RCE
a61af66fc99e Initial load
duke
parents:
diff changeset
2030 // and we'd rather unroll the post-RCE'd loop SO... do not unroll if
a61af66fc99e Initial load
duke
parents:
diff changeset
2031 // peeling.
2465
3af54845df98 7004555: Add new policy for one iteration loops
kvn
parents: 2453
diff changeset
2032 if (should_unroll && !should_peel)
3af54845df98 7004555: Add new policy for one iteration loops
kvn
parents: 2453
diff changeset
2033 phase->do_unroll(this,old_new, true);
0
a61af66fc99e Initial load
duke
parents:
diff changeset
2034
a61af66fc99e Initial load
duke
parents:
diff changeset
2035 // Adjust the pre-loop limits to align the main body
a61af66fc99e Initial load
duke
parents:
diff changeset
2036 // iterations.
2465
3af54845df98 7004555: Add new policy for one iteration loops
kvn
parents: 2453
diff changeset
2037 if (should_align)
0
a61af66fc99e Initial load
duke
parents:
diff changeset
2038 Unimplemented();
a61af66fc99e Initial load
duke
parents:
diff changeset
2039
a61af66fc99e Initial load
duke
parents:
diff changeset
2040 } else { // Else we have an unchanged counted loop
2465
3af54845df98 7004555: Add new policy for one iteration loops
kvn
parents: 2453
diff changeset
2041 if (should_peel) // Might want to peel but do nothing else
0
a61af66fc99e Initial load
duke
parents:
diff changeset
2042 phase->do_peeling(this,old_new);
a61af66fc99e Initial load
duke
parents:
diff changeset
2043 }
401
ee8f06bfb27c 6743188: incomplete fix for 6700047 C2 failed in idom_no_update
never
parents: 400
diff changeset
2044 return true;
0
a61af66fc99e Initial load
duke
parents:
diff changeset
2045 }
a61af66fc99e Initial load
duke
parents:
diff changeset
2046
a61af66fc99e Initial load
duke
parents:
diff changeset
2047
a61af66fc99e Initial load
duke
parents:
diff changeset
2048 //=============================================================================
a61af66fc99e Initial load
duke
parents:
diff changeset
2049 //------------------------------iteration_split--------------------------------
401
ee8f06bfb27c 6743188: incomplete fix for 6700047 C2 failed in idom_no_update
never
parents: 400
diff changeset
2050 bool IdealLoopTree::iteration_split( PhaseIdealLoop *phase, Node_List &old_new ) {
0
a61af66fc99e Initial load
duke
parents:
diff changeset
2051 // Recursively iteration split nested loops
2383
9dc311b8473e 7008866: Missing loop predicate for loop with multiple entries
kvn
parents: 1972
diff changeset
2052 if (_child && !_child->iteration_split(phase, old_new))
401
ee8f06bfb27c 6743188: incomplete fix for 6700047 C2 failed in idom_no_update
never
parents: 400
diff changeset
2053 return false;
0
a61af66fc99e Initial load
duke
parents:
diff changeset
2054
a61af66fc99e Initial load
duke
parents:
diff changeset
2055 // Clean out prior deadwood
a61af66fc99e Initial load
duke
parents:
diff changeset
2056 DCE_loop_body();
a61af66fc99e Initial load
duke
parents:
diff changeset
2057
a61af66fc99e Initial load
duke
parents:
diff changeset
2058
a61af66fc99e Initial load
duke
parents:
diff changeset
2059 // Look for loop-exit tests with my 50/50 guesses from the Parsing stage.
a61af66fc99e Initial load
duke
parents:
diff changeset
2060 // Replace with a 1-in-10 exit guess.
2383
9dc311b8473e 7008866: Missing loop predicate for loop with multiple entries
kvn
parents: 1972
diff changeset
2061 if (_parent /*not the root loop*/ &&
0
a61af66fc99e Initial load
duke
parents:
diff changeset
2062 !_irreducible &&
a61af66fc99e Initial load
duke
parents:
diff changeset
2063 // Also ignore the occasional dead backedge
2383
9dc311b8473e 7008866: Missing loop predicate for loop with multiple entries
kvn
parents: 1972
diff changeset
2064 !tail()->is_top()) {
0
a61af66fc99e Initial load
duke
parents:
diff changeset
2065 adjust_loop_exit_prob(phase);
a61af66fc99e Initial load
duke
parents:
diff changeset
2066 }
a61af66fc99e Initial load
duke
parents:
diff changeset
2067
a61af66fc99e Initial load
duke
parents:
diff changeset
2068 // Gate unrolling, RCE and peeling efforts.
2383
9dc311b8473e 7008866: Missing loop predicate for loop with multiple entries
kvn
parents: 1972
diff changeset
2069 if (!_child && // If not an inner loop, do not split
0
a61af66fc99e Initial load
duke
parents:
diff changeset
2070 !_irreducible &&
39
76256d272075 6667612: (Escape Analysis) disable loop cloning if it has a scalar replaceable allocation
kvn
parents: 0
diff changeset
2071 _allow_optimizations &&
2383
9dc311b8473e 7008866: Missing loop predicate for loop with multiple entries
kvn
parents: 1972
diff changeset
2072 !tail()->is_top()) { // Also ignore the occasional dead backedge
0
a61af66fc99e Initial load
duke
parents:
diff changeset
2073 if (!_has_call) {
2383
9dc311b8473e 7008866: Missing loop predicate for loop with multiple entries
kvn
parents: 1972
diff changeset
2074 if (!iteration_split_impl(phase, old_new)) {
1172
b2b6a9bf6238 6894779: Loop Predication for Loop Optimizer in C2
cfang
parents: 844
diff changeset
2075 return false;
b2b6a9bf6238 6894779: Loop Predication for Loop Optimizer in C2
cfang
parents: 844
diff changeset
2076 }
0
a61af66fc99e Initial load
duke
parents:
diff changeset
2077 } else if (policy_unswitching(phase)) {
a61af66fc99e Initial load
duke
parents:
diff changeset
2078 phase->do_unswitching(this, old_new);
a61af66fc99e Initial load
duke
parents:
diff changeset
2079 }
a61af66fc99e Initial load
duke
parents:
diff changeset
2080 }
a61af66fc99e Initial load
duke
parents:
diff changeset
2081
a61af66fc99e Initial load
duke
parents:
diff changeset
2082 // Minor offset re-organization to remove loop-fallout uses of
2383
9dc311b8473e 7008866: Missing loop predicate for loop with multiple entries
kvn
parents: 1972
diff changeset
2083 // trip counter when there was no major reshaping.
9dc311b8473e 7008866: Missing loop predicate for loop with multiple entries
kvn
parents: 1972
diff changeset
2084 phase->reorg_offsets(this);
9dc311b8473e 7008866: Missing loop predicate for loop with multiple entries
kvn
parents: 1972
diff changeset
2085
9dc311b8473e 7008866: Missing loop predicate for loop with multiple entries
kvn
parents: 1972
diff changeset
2086 if (_next && !_next->iteration_split(phase, old_new))
401
ee8f06bfb27c 6743188: incomplete fix for 6700047 C2 failed in idom_no_update
never
parents: 400
diff changeset
2087 return false;
ee8f06bfb27c 6743188: incomplete fix for 6700047 C2 failed in idom_no_update
never
parents: 400
diff changeset
2088 return true;
0
a61af66fc99e Initial load
duke
parents:
diff changeset
2089 }
1172
b2b6a9bf6238 6894779: Loop Predication for Loop Optimizer in C2
cfang
parents: 844
diff changeset
2090
b2b6a9bf6238 6894779: Loop Predication for Loop Optimizer in C2
cfang
parents: 844
diff changeset
2091
2445
08eb13460b3a 7004535: Clone loop predicate during loop unswitch
kvn
parents: 2417
diff changeset
2092 //=============================================================================
1763
d6f45b55c972 4809552: Optimize Arrays.fill(...)
never
parents: 1621
diff changeset
2093 // Process all the loops in the loop tree and replace any fill
d6f45b55c972 4809552: Optimize Arrays.fill(...)
never
parents: 1621
diff changeset
2094 // patterns with an intrisc version.
d6f45b55c972 4809552: Optimize Arrays.fill(...)
never
parents: 1621
diff changeset
2095 bool PhaseIdealLoop::do_intrinsify_fill() {
d6f45b55c972 4809552: Optimize Arrays.fill(...)
never
parents: 1621
diff changeset
2096 bool changed = false;
d6f45b55c972 4809552: Optimize Arrays.fill(...)
never
parents: 1621
diff changeset
2097 for (LoopTreeIterator iter(_ltree_root); !iter.done(); iter.next()) {
d6f45b55c972 4809552: Optimize Arrays.fill(...)
never
parents: 1621
diff changeset
2098 IdealLoopTree* lpt = iter.current();
d6f45b55c972 4809552: Optimize Arrays.fill(...)
never
parents: 1621
diff changeset
2099 changed |= intrinsify_fill(lpt);
d6f45b55c972 4809552: Optimize Arrays.fill(...)
never
parents: 1621
diff changeset
2100 }
d6f45b55c972 4809552: Optimize Arrays.fill(...)
never
parents: 1621
diff changeset
2101 return changed;
d6f45b55c972 4809552: Optimize Arrays.fill(...)
never
parents: 1621
diff changeset
2102 }
d6f45b55c972 4809552: Optimize Arrays.fill(...)
never
parents: 1621
diff changeset
2103
d6f45b55c972 4809552: Optimize Arrays.fill(...)
never
parents: 1621
diff changeset
2104
d6f45b55c972 4809552: Optimize Arrays.fill(...)
never
parents: 1621
diff changeset
2105 // Examine an inner loop looking for a a single store of an invariant
d6f45b55c972 4809552: Optimize Arrays.fill(...)
never
parents: 1621
diff changeset
2106 // value in a unit stride loop,
d6f45b55c972 4809552: Optimize Arrays.fill(...)
never
parents: 1621
diff changeset
2107 bool PhaseIdealLoop::match_fill_loop(IdealLoopTree* lpt, Node*& store, Node*& store_value,
d6f45b55c972 4809552: Optimize Arrays.fill(...)
never
parents: 1621
diff changeset
2108 Node*& shift, Node*& con) {
d6f45b55c972 4809552: Optimize Arrays.fill(...)
never
parents: 1621
diff changeset
2109 const char* msg = NULL;
d6f45b55c972 4809552: Optimize Arrays.fill(...)
never
parents: 1621
diff changeset
2110 Node* msg_node = NULL;
d6f45b55c972 4809552: Optimize Arrays.fill(...)
never
parents: 1621
diff changeset
2111
d6f45b55c972 4809552: Optimize Arrays.fill(...)
never
parents: 1621
diff changeset
2112 store_value = NULL;
d6f45b55c972 4809552: Optimize Arrays.fill(...)
never
parents: 1621
diff changeset
2113 con = NULL;
d6f45b55c972 4809552: Optimize Arrays.fill(...)
never
parents: 1621
diff changeset
2114 shift = NULL;
d6f45b55c972 4809552: Optimize Arrays.fill(...)
never
parents: 1621
diff changeset
2115
d6f45b55c972 4809552: Optimize Arrays.fill(...)
never
parents: 1621
diff changeset
2116 // Process the loop looking for stores. If there are multiple
d6f45b55c972 4809552: Optimize Arrays.fill(...)
never
parents: 1621
diff changeset
2117 // stores or extra control flow give at this point.
d6f45b55c972 4809552: Optimize Arrays.fill(...)
never
parents: 1621
diff changeset
2118 CountedLoopNode* head = lpt->_head->as_CountedLoop();
d6f45b55c972 4809552: Optimize Arrays.fill(...)
never
parents: 1621
diff changeset
2119 for (uint i = 0; msg == NULL && i < lpt->_body.size(); i++) {
d6f45b55c972 4809552: Optimize Arrays.fill(...)
never
parents: 1621
diff changeset
2120 Node* n = lpt->_body.at(i);
d6f45b55c972 4809552: Optimize Arrays.fill(...)
never
parents: 1621
diff changeset
2121 if (n->outcnt() == 0) continue; // Ignore dead
d6f45b55c972 4809552: Optimize Arrays.fill(...)
never
parents: 1621
diff changeset
2122 if (n->is_Store()) {
d6f45b55c972 4809552: Optimize Arrays.fill(...)
never
parents: 1621
diff changeset
2123 if (store != NULL) {
d6f45b55c972 4809552: Optimize Arrays.fill(...)
never
parents: 1621
diff changeset
2124 msg = "multiple stores";
d6f45b55c972 4809552: Optimize Arrays.fill(...)
never
parents: 1621
diff changeset
2125 break;
d6f45b55c972 4809552: Optimize Arrays.fill(...)
never
parents: 1621
diff changeset
2126 }
d6f45b55c972 4809552: Optimize Arrays.fill(...)
never
parents: 1621
diff changeset
2127 int opc = n->Opcode();
d6f45b55c972 4809552: Optimize Arrays.fill(...)
never
parents: 1621
diff changeset
2128 if (opc == Op_StoreP || opc == Op_StoreN || opc == Op_StoreCM) {
d6f45b55c972 4809552: Optimize Arrays.fill(...)
never
parents: 1621
diff changeset
2129 msg = "oop fills not handled";
d6f45b55c972 4809552: Optimize Arrays.fill(...)
never
parents: 1621
diff changeset
2130 break;
d6f45b55c972 4809552: Optimize Arrays.fill(...)
never
parents: 1621
diff changeset
2131 }
d6f45b55c972 4809552: Optimize Arrays.fill(...)
never
parents: 1621
diff changeset
2132 Node* value = n->in(MemNode::ValueIn);
d6f45b55c972 4809552: Optimize Arrays.fill(...)
never
parents: 1621
diff changeset
2133 if (!lpt->is_invariant(value)) {
d6f45b55c972 4809552: Optimize Arrays.fill(...)
never
parents: 1621
diff changeset
2134 msg = "variant store value";
1785
5e4f03302987 6982533: Crash in ~StubRoutines::jbyte_fill with AggressiveOpts enabled
never
parents: 1763
diff changeset
2135 } else if (!_igvn.type(n->in(MemNode::Address))->isa_aryptr()) {
5e4f03302987 6982533: Crash in ~StubRoutines::jbyte_fill with AggressiveOpts enabled
never
parents: 1763
diff changeset
2136 msg = "not array address";
1763
d6f45b55c972 4809552: Optimize Arrays.fill(...)
never
parents: 1621
diff changeset
2137 }
d6f45b55c972 4809552: Optimize Arrays.fill(...)
never
parents: 1621
diff changeset
2138 store = n;
d6f45b55c972 4809552: Optimize Arrays.fill(...)
never
parents: 1621
diff changeset
2139 store_value = value;
d6f45b55c972 4809552: Optimize Arrays.fill(...)
never
parents: 1621
diff changeset
2140 } else if (n->is_If() && n != head->loopexit()) {
d6f45b55c972 4809552: Optimize Arrays.fill(...)
never
parents: 1621
diff changeset
2141 msg = "extra control flow";
d6f45b55c972 4809552: Optimize Arrays.fill(...)
never
parents: 1621
diff changeset
2142 msg_node = n;
d6f45b55c972 4809552: Optimize Arrays.fill(...)
never
parents: 1621
diff changeset
2143 }
d6f45b55c972 4809552: Optimize Arrays.fill(...)
never
parents: 1621
diff changeset
2144 }
d6f45b55c972 4809552: Optimize Arrays.fill(...)
never
parents: 1621
diff changeset
2145
d6f45b55c972 4809552: Optimize Arrays.fill(...)
never
parents: 1621
diff changeset
2146 if (store == NULL) {
d6f45b55c972 4809552: Optimize Arrays.fill(...)
never
parents: 1621
diff changeset
2147 // No store in loop
d6f45b55c972 4809552: Optimize Arrays.fill(...)
never
parents: 1621
diff changeset
2148 return false;
d6f45b55c972 4809552: Optimize Arrays.fill(...)
never
parents: 1621
diff changeset
2149 }
d6f45b55c972 4809552: Optimize Arrays.fill(...)
never
parents: 1621
diff changeset
2150
d6f45b55c972 4809552: Optimize Arrays.fill(...)
never
parents: 1621
diff changeset
2151 if (msg == NULL && head->stride_con() != 1) {
d6f45b55c972 4809552: Optimize Arrays.fill(...)
never
parents: 1621
diff changeset
2152 // could handle negative strides too
d6f45b55c972 4809552: Optimize Arrays.fill(...)
never
parents: 1621
diff changeset
2153 if (head->stride_con() < 0) {
d6f45b55c972 4809552: Optimize Arrays.fill(...)
never
parents: 1621
diff changeset
2154 msg = "negative stride";
d6f45b55c972 4809552: Optimize Arrays.fill(...)
never
parents: 1621
diff changeset
2155 } else {
d6f45b55c972 4809552: Optimize Arrays.fill(...)
never
parents: 1621
diff changeset
2156 msg = "non-unit stride";
d6f45b55c972 4809552: Optimize Arrays.fill(...)
never
parents: 1621
diff changeset
2157 }
d6f45b55c972 4809552: Optimize Arrays.fill(...)
never
parents: 1621
diff changeset
2158 }
d6f45b55c972 4809552: Optimize Arrays.fill(...)
never
parents: 1621
diff changeset
2159
d6f45b55c972 4809552: Optimize Arrays.fill(...)
never
parents: 1621
diff changeset
2160 if (msg == NULL && !store->in(MemNode::Address)->is_AddP()) {
d6f45b55c972 4809552: Optimize Arrays.fill(...)
never
parents: 1621
diff changeset
2161 msg = "can't handle store address";
d6f45b55c972 4809552: Optimize Arrays.fill(...)
never
parents: 1621
diff changeset
2162 msg_node = store->in(MemNode::Address);
d6f45b55c972 4809552: Optimize Arrays.fill(...)
never
parents: 1621
diff changeset
2163 }
d6f45b55c972 4809552: Optimize Arrays.fill(...)
never
parents: 1621
diff changeset
2164
1813
c77e8f982901 6984979: OptimizeFill misses some cases with an odd memory graph
never
parents: 1785
diff changeset
2165 if (msg == NULL &&
c77e8f982901 6984979: OptimizeFill misses some cases with an odd memory graph
never
parents: 1785
diff changeset
2166 (!store->in(MemNode::Memory)->is_Phi() ||
c77e8f982901 6984979: OptimizeFill misses some cases with an odd memory graph
never
parents: 1785
diff changeset
2167 store->in(MemNode::Memory)->in(LoopNode::LoopBackControl) != store)) {
c77e8f982901 6984979: OptimizeFill misses some cases with an odd memory graph
never
parents: 1785
diff changeset
2168 msg = "store memory isn't proper phi";
c77e8f982901 6984979: OptimizeFill misses some cases with an odd memory graph
never
parents: 1785
diff changeset
2169 msg_node = store->in(MemNode::Memory);
c77e8f982901 6984979: OptimizeFill misses some cases with an odd memory graph
never
parents: 1785
diff changeset
2170 }
c77e8f982901 6984979: OptimizeFill misses some cases with an odd memory graph
never
parents: 1785
diff changeset
2171
1763
d6f45b55c972 4809552: Optimize Arrays.fill(...)
never
parents: 1621
diff changeset
2172 // Make sure there is an appropriate fill routine
d6f45b55c972 4809552: Optimize Arrays.fill(...)
never
parents: 1621
diff changeset
2173 BasicType t = store->as_Mem()->memory_type();
d6f45b55c972 4809552: Optimize Arrays.fill(...)
never
parents: 1621
diff changeset
2174 const char* fill_name;
d6f45b55c972 4809552: Optimize Arrays.fill(...)
never
parents: 1621
diff changeset
2175 if (msg == NULL &&
d6f45b55c972 4809552: Optimize Arrays.fill(...)
never
parents: 1621
diff changeset
2176 StubRoutines::select_fill_function(t, false, fill_name) == NULL) {
d6f45b55c972 4809552: Optimize Arrays.fill(...)
never
parents: 1621
diff changeset
2177 msg = "unsupported store";
d6f45b55c972 4809552: Optimize Arrays.fill(...)
never
parents: 1621
diff changeset
2178 msg_node = store;
d6f45b55c972 4809552: Optimize Arrays.fill(...)
never
parents: 1621
diff changeset
2179 }
d6f45b55c972 4809552: Optimize Arrays.fill(...)
never
parents: 1621
diff changeset
2180
d6f45b55c972 4809552: Optimize Arrays.fill(...)
never
parents: 1621
diff changeset
2181 if (msg != NULL) {
d6f45b55c972 4809552: Optimize Arrays.fill(...)
never
parents: 1621
diff changeset
2182 #ifndef PRODUCT
d6f45b55c972 4809552: Optimize Arrays.fill(...)
never
parents: 1621
diff changeset
2183 if (TraceOptimizeFill) {
d6f45b55c972 4809552: Optimize Arrays.fill(...)
never
parents: 1621
diff changeset
2184 tty->print_cr("not fill intrinsic candidate: %s", msg);
d6f45b55c972 4809552: Optimize Arrays.fill(...)
never
parents: 1621
diff changeset
2185 if (msg_node != NULL) msg_node->dump();
d6f45b55c972 4809552: Optimize Arrays.fill(...)
never
parents: 1621
diff changeset
2186 }
d6f45b55c972 4809552: Optimize Arrays.fill(...)
never
parents: 1621
diff changeset
2187 #endif
d6f45b55c972 4809552: Optimize Arrays.fill(...)
never
parents: 1621
diff changeset
2188 return false;
d6f45b55c972 4809552: Optimize Arrays.fill(...)
never
parents: 1621
diff changeset
2189 }
d6f45b55c972 4809552: Optimize Arrays.fill(...)
never
parents: 1621
diff changeset
2190
d6f45b55c972 4809552: Optimize Arrays.fill(...)
never
parents: 1621
diff changeset
2191 // Make sure the address expression can be handled. It should be
d6f45b55c972 4809552: Optimize Arrays.fill(...)
never
parents: 1621
diff changeset
2192 // head->phi * elsize + con. head->phi might have a ConvI2L.
d6f45b55c972 4809552: Optimize Arrays.fill(...)
never
parents: 1621
diff changeset
2193 Node* elements[4];
d6f45b55c972 4809552: Optimize Arrays.fill(...)
never
parents: 1621
diff changeset
2194 Node* conv = NULL;
1785
5e4f03302987 6982533: Crash in ~StubRoutines::jbyte_fill with AggressiveOpts enabled
never
parents: 1763
diff changeset
2195 bool found_index = false;
1763
d6f45b55c972 4809552: Optimize Arrays.fill(...)
never
parents: 1621
diff changeset
2196 int count = store->in(MemNode::Address)->as_AddP()->unpack_offsets(elements, ARRAY_SIZE(elements));
d6f45b55c972 4809552: Optimize Arrays.fill(...)
never
parents: 1621
diff changeset
2197 for (int e = 0; e < count; e++) {
d6f45b55c972 4809552: Optimize Arrays.fill(...)
never
parents: 1621
diff changeset
2198 Node* n = elements[e];
d6f45b55c972 4809552: Optimize Arrays.fill(...)
never
parents: 1621
diff changeset
2199 if (n->is_Con() && con == NULL) {
d6f45b55c972 4809552: Optimize Arrays.fill(...)
never
parents: 1621
diff changeset
2200 con = n;
d6f45b55c972 4809552: Optimize Arrays.fill(...)
never
parents: 1621
diff changeset
2201 } else if (n->Opcode() == Op_LShiftX && shift == NULL) {
d6f45b55c972 4809552: Optimize Arrays.fill(...)
never
parents: 1621
diff changeset
2202 Node* value = n->in(1);
d6f45b55c972 4809552: Optimize Arrays.fill(...)
never
parents: 1621
diff changeset
2203 #ifdef _LP64
d6f45b55c972 4809552: Optimize Arrays.fill(...)
never
parents: 1621
diff changeset
2204 if (value->Opcode() == Op_ConvI2L) {
d6f45b55c972 4809552: Optimize Arrays.fill(...)
never
parents: 1621
diff changeset
2205 conv = value;
d6f45b55c972 4809552: Optimize Arrays.fill(...)
never
parents: 1621
diff changeset
2206 value = value->in(1);
d6f45b55c972 4809552: Optimize Arrays.fill(...)
never
parents: 1621
diff changeset
2207 }
d6f45b55c972 4809552: Optimize Arrays.fill(...)
never
parents: 1621
diff changeset
2208 #endif
d6f45b55c972 4809552: Optimize Arrays.fill(...)
never
parents: 1621
diff changeset
2209 if (value != head->phi()) {
d6f45b55c972 4809552: Optimize Arrays.fill(...)
never
parents: 1621
diff changeset
2210 msg = "unhandled shift in address";
d6f45b55c972 4809552: Optimize Arrays.fill(...)
never
parents: 1621
diff changeset
2211 } else {
2448
8b2317d732ec 7026957: assert(type2aelembytes(store->as_Mem()->memory_type(), true) == 1 << shift->in(2)->get_int()) failed
never
parents: 2445
diff changeset
2212 if (type2aelembytes(store->as_Mem()->memory_type(), true) != (1 << n->in(2)->get_int())) {
8b2317d732ec 7026957: assert(type2aelembytes(store->as_Mem()->memory_type(), true) == 1 << shift->in(2)->get_int()) failed
never
parents: 2445
diff changeset
2213 msg = "scale doesn't match";
8b2317d732ec 7026957: assert(type2aelembytes(store->as_Mem()->memory_type(), true) == 1 << shift->in(2)->get_int()) failed
never
parents: 2445
diff changeset
2214 } else {
8b2317d732ec 7026957: assert(type2aelembytes(store->as_Mem()->memory_type(), true) == 1 << shift->in(2)->get_int()) failed
never
parents: 2445
diff changeset
2215 found_index = true;
8b2317d732ec 7026957: assert(type2aelembytes(store->as_Mem()->memory_type(), true) == 1 << shift->in(2)->get_int()) failed
never
parents: 2445
diff changeset
2216 shift = n;
8b2317d732ec 7026957: assert(type2aelembytes(store->as_Mem()->memory_type(), true) == 1 << shift->in(2)->get_int()) failed
never
parents: 2445
diff changeset
2217 }
1763
d6f45b55c972 4809552: Optimize Arrays.fill(...)
never
parents: 1621
diff changeset
2218 }
d6f45b55c972 4809552: Optimize Arrays.fill(...)
never
parents: 1621
diff changeset
2219 } else if (n->Opcode() == Op_ConvI2L && conv == NULL) {
d6f45b55c972 4809552: Optimize Arrays.fill(...)
never
parents: 1621
diff changeset
2220 if (n->in(1) == head->phi()) {
1785
5e4f03302987 6982533: Crash in ~StubRoutines::jbyte_fill with AggressiveOpts enabled
never
parents: 1763
diff changeset
2221 found_index = true;
1763
d6f45b55c972 4809552: Optimize Arrays.fill(...)
never
parents: 1621
diff changeset
2222 conv = n;
d6f45b55c972 4809552: Optimize Arrays.fill(...)
never
parents: 1621
diff changeset
2223 } else {
d6f45b55c972 4809552: Optimize Arrays.fill(...)
never
parents: 1621
diff changeset
2224 msg = "unhandled input to ConvI2L";
d6f45b55c972 4809552: Optimize Arrays.fill(...)
never
parents: 1621
diff changeset
2225 }
d6f45b55c972 4809552: Optimize Arrays.fill(...)
never
parents: 1621
diff changeset
2226 } else if (n == head->phi()) {
d6f45b55c972 4809552: Optimize Arrays.fill(...)
never
parents: 1621
diff changeset
2227 // no shift, check below for allowed cases
1785
5e4f03302987 6982533: Crash in ~StubRoutines::jbyte_fill with AggressiveOpts enabled
never
parents: 1763
diff changeset
2228 found_index = true;
1763
d6f45b55c972 4809552: Optimize Arrays.fill(...)
never
parents: 1621
diff changeset
2229 } else {
d6f45b55c972 4809552: Optimize Arrays.fill(...)
never
parents: 1621
diff changeset
2230 msg = "unhandled node in address";
d6f45b55c972 4809552: Optimize Arrays.fill(...)
never
parents: 1621
diff changeset
2231 msg_node = n;
d6f45b55c972 4809552: Optimize Arrays.fill(...)
never
parents: 1621
diff changeset
2232 }
d6f45b55c972 4809552: Optimize Arrays.fill(...)
never
parents: 1621
diff changeset
2233 }
d6f45b55c972 4809552: Optimize Arrays.fill(...)
never
parents: 1621
diff changeset
2234
d6f45b55c972 4809552: Optimize Arrays.fill(...)
never
parents: 1621
diff changeset
2235 if (count == -1) {
d6f45b55c972 4809552: Optimize Arrays.fill(...)
never
parents: 1621
diff changeset
2236 msg = "malformed address expression";
d6f45b55c972 4809552: Optimize Arrays.fill(...)
never
parents: 1621
diff changeset
2237 msg_node = store;
d6f45b55c972 4809552: Optimize Arrays.fill(...)
never
parents: 1621
diff changeset
2238 }
d6f45b55c972 4809552: Optimize Arrays.fill(...)
never
parents: 1621
diff changeset
2239
1785
5e4f03302987 6982533: Crash in ~StubRoutines::jbyte_fill with AggressiveOpts enabled
never
parents: 1763
diff changeset
2240 if (!found_index) {
5e4f03302987 6982533: Crash in ~StubRoutines::jbyte_fill with AggressiveOpts enabled
never
parents: 1763
diff changeset
2241 msg = "missing use of index";
5e4f03302987 6982533: Crash in ~StubRoutines::jbyte_fill with AggressiveOpts enabled
never
parents: 1763
diff changeset
2242 }
5e4f03302987 6982533: Crash in ~StubRoutines::jbyte_fill with AggressiveOpts enabled
never
parents: 1763
diff changeset
2243
1763
d6f45b55c972 4809552: Optimize Arrays.fill(...)
never
parents: 1621
diff changeset
2244 // byte sized items won't have a shift
d6f45b55c972 4809552: Optimize Arrays.fill(...)
never
parents: 1621
diff changeset
2245 if (msg == NULL && shift == NULL && t != T_BYTE && t != T_BOOLEAN) {
d6f45b55c972 4809552: Optimize Arrays.fill(...)
never
parents: 1621
diff changeset
2246 msg = "can't find shift";
d6f45b55c972 4809552: Optimize Arrays.fill(...)
never
parents: 1621
diff changeset
2247 msg_node = store;
d6f45b55c972 4809552: Optimize Arrays.fill(...)
never
parents: 1621
diff changeset
2248 }
d6f45b55c972 4809552: Optimize Arrays.fill(...)
never
parents: 1621
diff changeset
2249
d6f45b55c972 4809552: Optimize Arrays.fill(...)
never
parents: 1621
diff changeset
2250 if (msg != NULL) {
d6f45b55c972 4809552: Optimize Arrays.fill(...)
never
parents: 1621
diff changeset
2251 #ifndef PRODUCT
d6f45b55c972 4809552: Optimize Arrays.fill(...)
never
parents: 1621
diff changeset
2252 if (TraceOptimizeFill) {
d6f45b55c972 4809552: Optimize Arrays.fill(...)
never
parents: 1621
diff changeset
2253 tty->print_cr("not fill intrinsic: %s", msg);
d6f45b55c972 4809552: Optimize Arrays.fill(...)
never
parents: 1621
diff changeset
2254 if (msg_node != NULL) msg_node->dump();
d6f45b55c972 4809552: Optimize Arrays.fill(...)
never
parents: 1621
diff changeset
2255 }
d6f45b55c972 4809552: Optimize Arrays.fill(...)
never
parents: 1621
diff changeset
2256 #endif
d6f45b55c972 4809552: Optimize Arrays.fill(...)
never
parents: 1621
diff changeset
2257 return false;
d6f45b55c972 4809552: Optimize Arrays.fill(...)
never
parents: 1621
diff changeset
2258 }
d6f45b55c972 4809552: Optimize Arrays.fill(...)
never
parents: 1621
diff changeset
2259
d6f45b55c972 4809552: Optimize Arrays.fill(...)
never
parents: 1621
diff changeset
2260 // No make sure all the other nodes in the loop can be handled
d6f45b55c972 4809552: Optimize Arrays.fill(...)
never
parents: 1621
diff changeset
2261 VectorSet ok(Thread::current()->resource_area());
d6f45b55c972 4809552: Optimize Arrays.fill(...)
never
parents: 1621
diff changeset
2262
d6f45b55c972 4809552: Optimize Arrays.fill(...)
never
parents: 1621
diff changeset
2263 // store related values are ok
d6f45b55c972 4809552: Optimize Arrays.fill(...)
never
parents: 1621
diff changeset
2264 ok.set(store->_idx);
d6f45b55c972 4809552: Optimize Arrays.fill(...)
never
parents: 1621
diff changeset
2265 ok.set(store->in(MemNode::Memory)->_idx);
d6f45b55c972 4809552: Optimize Arrays.fill(...)
never
parents: 1621
diff changeset
2266
d6f45b55c972 4809552: Optimize Arrays.fill(...)
never
parents: 1621
diff changeset
2267 // Loop structure is ok
d6f45b55c972 4809552: Optimize Arrays.fill(...)
never
parents: 1621
diff changeset
2268 ok.set(head->_idx);
d6f45b55c972 4809552: Optimize Arrays.fill(...)
never
parents: 1621
diff changeset
2269 ok.set(head->loopexit()->_idx);
d6f45b55c972 4809552: Optimize Arrays.fill(...)
never
parents: 1621
diff changeset
2270 ok.set(head->phi()->_idx);
d6f45b55c972 4809552: Optimize Arrays.fill(...)
never
parents: 1621
diff changeset
2271 ok.set(head->incr()->_idx);
d6f45b55c972 4809552: Optimize Arrays.fill(...)
never
parents: 1621
diff changeset
2272 ok.set(head->loopexit()->cmp_node()->_idx);
d6f45b55c972 4809552: Optimize Arrays.fill(...)
never
parents: 1621
diff changeset
2273 ok.set(head->loopexit()->in(1)->_idx);
d6f45b55c972 4809552: Optimize Arrays.fill(...)
never
parents: 1621
diff changeset
2274
d6f45b55c972 4809552: Optimize Arrays.fill(...)
never
parents: 1621
diff changeset
2275 // Address elements are ok
d6f45b55c972 4809552: Optimize Arrays.fill(...)
never
parents: 1621
diff changeset
2276 if (con) ok.set(con->_idx);
d6f45b55c972 4809552: Optimize Arrays.fill(...)
never
parents: 1621
diff changeset
2277 if (shift) ok.set(shift->_idx);
d6f45b55c972 4809552: Optimize Arrays.fill(...)
never
parents: 1621
diff changeset
2278 if (conv) ok.set(conv->_idx);
d6f45b55c972 4809552: Optimize Arrays.fill(...)
never
parents: 1621
diff changeset
2279
d6f45b55c972 4809552: Optimize Arrays.fill(...)
never
parents: 1621
diff changeset
2280 for (uint i = 0; msg == NULL && i < lpt->_body.size(); i++) {
d6f45b55c972 4809552: Optimize Arrays.fill(...)
never
parents: 1621
diff changeset
2281 Node* n = lpt->_body.at(i);
d6f45b55c972 4809552: Optimize Arrays.fill(...)
never
parents: 1621
diff changeset
2282 if (n->outcnt() == 0) continue; // Ignore dead
d6f45b55c972 4809552: Optimize Arrays.fill(...)
never
parents: 1621
diff changeset
2283 if (ok.test(n->_idx)) continue;
d6f45b55c972 4809552: Optimize Arrays.fill(...)
never
parents: 1621
diff changeset
2284 // Backedge projection is ok
d6f45b55c972 4809552: Optimize Arrays.fill(...)
never
parents: 1621
diff changeset
2285 if (n->is_IfTrue() && n->in(0) == head->loopexit()) continue;
d6f45b55c972 4809552: Optimize Arrays.fill(...)
never
parents: 1621
diff changeset
2286 if (!n->is_AddP()) {
d6f45b55c972 4809552: Optimize Arrays.fill(...)
never
parents: 1621
diff changeset
2287 msg = "unhandled node";
d6f45b55c972 4809552: Optimize Arrays.fill(...)
never
parents: 1621
diff changeset
2288 msg_node = n;
d6f45b55c972 4809552: Optimize Arrays.fill(...)
never
parents: 1621
diff changeset
2289 break;
d6f45b55c972 4809552: Optimize Arrays.fill(...)
never
parents: 1621
diff changeset
2290 }
d6f45b55c972 4809552: Optimize Arrays.fill(...)
never
parents: 1621
diff changeset
2291 }
d6f45b55c972 4809552: Optimize Arrays.fill(...)
never
parents: 1621
diff changeset
2292
d6f45b55c972 4809552: Optimize Arrays.fill(...)
never
parents: 1621
diff changeset
2293 // Make sure no unexpected values are used outside the loop
d6f45b55c972 4809552: Optimize Arrays.fill(...)
never
parents: 1621
diff changeset
2294 for (uint i = 0; msg == NULL && i < lpt->_body.size(); i++) {
d6f45b55c972 4809552: Optimize Arrays.fill(...)
never
parents: 1621
diff changeset
2295 Node* n = lpt->_body.at(i);
d6f45b55c972 4809552: Optimize Arrays.fill(...)
never
parents: 1621
diff changeset
2296 // These values can be replaced with other nodes if they are used
d6f45b55c972 4809552: Optimize Arrays.fill(...)
never
parents: 1621
diff changeset
2297 // outside the loop.
1813
c77e8f982901 6984979: OptimizeFill misses some cases with an odd memory graph
never
parents: 1785
diff changeset
2298 if (n == store || n == head->loopexit() || n == head->incr() || n == store->in(MemNode::Memory)) continue;
1763
d6f45b55c972 4809552: Optimize Arrays.fill(...)
never
parents: 1621
diff changeset
2299 for (SimpleDUIterator iter(n); iter.has_next(); iter.next()) {
d6f45b55c972 4809552: Optimize Arrays.fill(...)
never
parents: 1621
diff changeset
2300 Node* use = iter.get();
d6f45b55c972 4809552: Optimize Arrays.fill(...)
never
parents: 1621
diff changeset
2301 if (!lpt->_body.contains(use)) {
d6f45b55c972 4809552: Optimize Arrays.fill(...)
never
parents: 1621
diff changeset
2302 msg = "node is used outside loop";
d6f45b55c972 4809552: Optimize Arrays.fill(...)
never
parents: 1621
diff changeset
2303 // lpt->_body.dump();
d6f45b55c972 4809552: Optimize Arrays.fill(...)
never
parents: 1621
diff changeset
2304 msg_node = n;
d6f45b55c972 4809552: Optimize Arrays.fill(...)
never
parents: 1621
diff changeset
2305 break;
d6f45b55c972 4809552: Optimize Arrays.fill(...)
never
parents: 1621
diff changeset
2306 }
d6f45b55c972 4809552: Optimize Arrays.fill(...)
never
parents: 1621
diff changeset
2307 }
d6f45b55c972 4809552: Optimize Arrays.fill(...)
never
parents: 1621
diff changeset
2308 }
d6f45b55c972 4809552: Optimize Arrays.fill(...)
never
parents: 1621
diff changeset
2309
d6f45b55c972 4809552: Optimize Arrays.fill(...)
never
parents: 1621
diff changeset
2310 #ifdef ASSERT
d6f45b55c972 4809552: Optimize Arrays.fill(...)
never
parents: 1621
diff changeset
2311 if (TraceOptimizeFill) {
d6f45b55c972 4809552: Optimize Arrays.fill(...)
never
parents: 1621
diff changeset
2312 if (msg != NULL) {
d6f45b55c972 4809552: Optimize Arrays.fill(...)
never
parents: 1621
diff changeset
2313 tty->print_cr("no fill intrinsic: %s", msg);
d6f45b55c972 4809552: Optimize Arrays.fill(...)
never
parents: 1621
diff changeset
2314 if (msg_node != NULL) msg_node->dump();
d6f45b55c972 4809552: Optimize Arrays.fill(...)
never
parents: 1621
diff changeset
2315 } else {
d6f45b55c972 4809552: Optimize Arrays.fill(...)
never
parents: 1621
diff changeset
2316 tty->print_cr("fill intrinsic for:");
d6f45b55c972 4809552: Optimize Arrays.fill(...)
never
parents: 1621
diff changeset
2317 }
d6f45b55c972 4809552: Optimize Arrays.fill(...)
never
parents: 1621
diff changeset
2318 store->dump();
d6f45b55c972 4809552: Optimize Arrays.fill(...)
never
parents: 1621
diff changeset
2319 if (Verbose) {
d6f45b55c972 4809552: Optimize Arrays.fill(...)
never
parents: 1621
diff changeset
2320 lpt->_body.dump();
d6f45b55c972 4809552: Optimize Arrays.fill(...)
never
parents: 1621
diff changeset
2321 }
d6f45b55c972 4809552: Optimize Arrays.fill(...)
never
parents: 1621
diff changeset
2322 }
d6f45b55c972 4809552: Optimize Arrays.fill(...)
never
parents: 1621
diff changeset
2323 #endif
d6f45b55c972 4809552: Optimize Arrays.fill(...)
never
parents: 1621
diff changeset
2324
d6f45b55c972 4809552: Optimize Arrays.fill(...)
never
parents: 1621
diff changeset
2325 return msg == NULL;
d6f45b55c972 4809552: Optimize Arrays.fill(...)
never
parents: 1621
diff changeset
2326 }
d6f45b55c972 4809552: Optimize Arrays.fill(...)
never
parents: 1621
diff changeset
2327
d6f45b55c972 4809552: Optimize Arrays.fill(...)
never
parents: 1621
diff changeset
2328
d6f45b55c972 4809552: Optimize Arrays.fill(...)
never
parents: 1621
diff changeset
2329
d6f45b55c972 4809552: Optimize Arrays.fill(...)
never
parents: 1621
diff changeset
2330 bool PhaseIdealLoop::intrinsify_fill(IdealLoopTree* lpt) {
d6f45b55c972 4809552: Optimize Arrays.fill(...)
never
parents: 1621
diff changeset
2331 // Only for counted inner loops
d6f45b55c972 4809552: Optimize Arrays.fill(...)
never
parents: 1621
diff changeset
2332 if (!lpt->is_counted() || !lpt->is_inner()) {
d6f45b55c972 4809552: Optimize Arrays.fill(...)
never
parents: 1621
diff changeset
2333 return false;
d6f45b55c972 4809552: Optimize Arrays.fill(...)
never
parents: 1621
diff changeset
2334 }
d6f45b55c972 4809552: Optimize Arrays.fill(...)
never
parents: 1621
diff changeset
2335
d6f45b55c972 4809552: Optimize Arrays.fill(...)
never
parents: 1621
diff changeset
2336 // Must have constant stride
d6f45b55c972 4809552: Optimize Arrays.fill(...)
never
parents: 1621
diff changeset
2337 CountedLoopNode* head = lpt->_head->as_CountedLoop();
d6f45b55c972 4809552: Optimize Arrays.fill(...)
never
parents: 1621
diff changeset
2338 if (!head->stride_is_con() || !head->is_normal_loop()) {
d6f45b55c972 4809552: Optimize Arrays.fill(...)
never
parents: 1621
diff changeset
2339 return false;
d6f45b55c972 4809552: Optimize Arrays.fill(...)
never
parents: 1621
diff changeset
2340 }
d6f45b55c972 4809552: Optimize Arrays.fill(...)
never
parents: 1621
diff changeset
2341
d6f45b55c972 4809552: Optimize Arrays.fill(...)
never
parents: 1621
diff changeset
2342 // Check that the body only contains a store of a loop invariant
d6f45b55c972 4809552: Optimize Arrays.fill(...)
never
parents: 1621
diff changeset
2343 // value that is indexed by the loop phi.
d6f45b55c972 4809552: Optimize Arrays.fill(...)
never
parents: 1621
diff changeset
2344 Node* store = NULL;
d6f45b55c972 4809552: Optimize Arrays.fill(...)
never
parents: 1621
diff changeset
2345 Node* store_value = NULL;
d6f45b55c972 4809552: Optimize Arrays.fill(...)
never
parents: 1621
diff changeset
2346 Node* shift = NULL;
d6f45b55c972 4809552: Optimize Arrays.fill(...)
never
parents: 1621
diff changeset
2347 Node* offset = NULL;
d6f45b55c972 4809552: Optimize Arrays.fill(...)
never
parents: 1621
diff changeset
2348 if (!match_fill_loop(lpt, store, store_value, shift, offset)) {
d6f45b55c972 4809552: Optimize Arrays.fill(...)
never
parents: 1621
diff changeset
2349 return false;
d6f45b55c972 4809552: Optimize Arrays.fill(...)
never
parents: 1621
diff changeset
2350 }
d6f45b55c972 4809552: Optimize Arrays.fill(...)
never
parents: 1621
diff changeset
2351
2445
08eb13460b3a 7004535: Clone loop predicate during loop unswitch
kvn
parents: 2417
diff changeset
2352 #ifndef PRODUCT
08eb13460b3a 7004535: Clone loop predicate during loop unswitch
kvn
parents: 2417
diff changeset
2353 if (TraceLoopOpts) {
08eb13460b3a 7004535: Clone loop predicate during loop unswitch
kvn
parents: 2417
diff changeset
2354 tty->print("ArrayFill ");
08eb13460b3a 7004535: Clone loop predicate during loop unswitch
kvn
parents: 2417
diff changeset
2355 lpt->dump_head();
08eb13460b3a 7004535: Clone loop predicate during loop unswitch
kvn
parents: 2417
diff changeset
2356 }
08eb13460b3a 7004535: Clone loop predicate during loop unswitch
kvn
parents: 2417
diff changeset
2357 #endif
08eb13460b3a 7004535: Clone loop predicate during loop unswitch
kvn
parents: 2417
diff changeset
2358
1763
d6f45b55c972 4809552: Optimize Arrays.fill(...)
never
parents: 1621
diff changeset
2359 // Now replace the whole loop body by a call to a fill routine that
d6f45b55c972 4809552: Optimize Arrays.fill(...)
never
parents: 1621
diff changeset
2360 // covers the same region as the loop.
d6f45b55c972 4809552: Optimize Arrays.fill(...)
never
parents: 1621
diff changeset
2361 Node* base = store->in(MemNode::Address)->as_AddP()->in(AddPNode::Base);
d6f45b55c972 4809552: Optimize Arrays.fill(...)
never
parents: 1621
diff changeset
2362
d6f45b55c972 4809552: Optimize Arrays.fill(...)
never
parents: 1621
diff changeset
2363 // Build an expression for the beginning of the copy region
d6f45b55c972 4809552: Optimize Arrays.fill(...)
never
parents: 1621
diff changeset
2364 Node* index = head->init_trip();
d6f45b55c972 4809552: Optimize Arrays.fill(...)
never
parents: 1621
diff changeset
2365 #ifdef _LP64
d6f45b55c972 4809552: Optimize Arrays.fill(...)
never
parents: 1621
diff changeset
2366 index = new (C, 2) ConvI2LNode(index);
d6f45b55c972 4809552: Optimize Arrays.fill(...)
never
parents: 1621
diff changeset
2367 _igvn.register_new_node_with_optimizer(index);
d6f45b55c972 4809552: Optimize Arrays.fill(...)
never
parents: 1621
diff changeset
2368 #endif
d6f45b55c972 4809552: Optimize Arrays.fill(...)
never
parents: 1621
diff changeset
2369 if (shift != NULL) {
d6f45b55c972 4809552: Optimize Arrays.fill(...)
never
parents: 1621
diff changeset
2370 // byte arrays don't require a shift but others do.
d6f45b55c972 4809552: Optimize Arrays.fill(...)
never
parents: 1621
diff changeset
2371 index = new (C, 3) LShiftXNode(index, shift->in(2));
d6f45b55c972 4809552: Optimize Arrays.fill(...)
never
parents: 1621
diff changeset
2372 _igvn.register_new_node_with_optimizer(index);
d6f45b55c972 4809552: Optimize Arrays.fill(...)
never
parents: 1621
diff changeset
2373 }
d6f45b55c972 4809552: Optimize Arrays.fill(...)
never
parents: 1621
diff changeset
2374 index = new (C, 4) AddPNode(base, base, index);
d6f45b55c972 4809552: Optimize Arrays.fill(...)
never
parents: 1621
diff changeset
2375 _igvn.register_new_node_with_optimizer(index);
d6f45b55c972 4809552: Optimize Arrays.fill(...)
never
parents: 1621
diff changeset
2376 Node* from = new (C, 4) AddPNode(base, index, offset);
d6f45b55c972 4809552: Optimize Arrays.fill(...)
never
parents: 1621
diff changeset
2377 _igvn.register_new_node_with_optimizer(from);
d6f45b55c972 4809552: Optimize Arrays.fill(...)
never
parents: 1621
diff changeset
2378 // Compute the number of elements to copy
d6f45b55c972 4809552: Optimize Arrays.fill(...)
never
parents: 1621
diff changeset
2379 Node* len = new (C, 3) SubINode(head->limit(), head->init_trip());
d6f45b55c972 4809552: Optimize Arrays.fill(...)
never
parents: 1621
diff changeset
2380 _igvn.register_new_node_with_optimizer(len);
d6f45b55c972 4809552: Optimize Arrays.fill(...)
never
parents: 1621
diff changeset
2381
d6f45b55c972 4809552: Optimize Arrays.fill(...)
never
parents: 1621
diff changeset
2382 BasicType t = store->as_Mem()->memory_type();
d6f45b55c972 4809552: Optimize Arrays.fill(...)
never
parents: 1621
diff changeset
2383 bool aligned = false;
d6f45b55c972 4809552: Optimize Arrays.fill(...)
never
parents: 1621
diff changeset
2384 if (offset != NULL && head->init_trip()->is_Con()) {
d6f45b55c972 4809552: Optimize Arrays.fill(...)
never
parents: 1621
diff changeset
2385 int element_size = type2aelembytes(t);
d6f45b55c972 4809552: Optimize Arrays.fill(...)
never
parents: 1621
diff changeset
2386 aligned = (offset->find_intptr_t_type()->get_con() + head->init_trip()->get_int() * element_size) % HeapWordSize == 0;
d6f45b55c972 4809552: Optimize Arrays.fill(...)
never
parents: 1621
diff changeset
2387 }
d6f45b55c972 4809552: Optimize Arrays.fill(...)
never
parents: 1621
diff changeset
2388
d6f45b55c972 4809552: Optimize Arrays.fill(...)
never
parents: 1621
diff changeset
2389 // Build a call to the fill routine
d6f45b55c972 4809552: Optimize Arrays.fill(...)
never
parents: 1621
diff changeset
2390 const char* fill_name;
d6f45b55c972 4809552: Optimize Arrays.fill(...)
never
parents: 1621
diff changeset
2391 address fill = StubRoutines::select_fill_function(t, aligned, fill_name);
d6f45b55c972 4809552: Optimize Arrays.fill(...)
never
parents: 1621
diff changeset
2392 assert(fill != NULL, "what?");
d6f45b55c972 4809552: Optimize Arrays.fill(...)
never
parents: 1621
diff changeset
2393
d6f45b55c972 4809552: Optimize Arrays.fill(...)
never
parents: 1621
diff changeset
2394 // Convert float/double to int/long for fill routines
d6f45b55c972 4809552: Optimize Arrays.fill(...)
never
parents: 1621
diff changeset
2395 if (t == T_FLOAT) {
d6f45b55c972 4809552: Optimize Arrays.fill(...)
never
parents: 1621
diff changeset
2396 store_value = new (C, 2) MoveF2INode(store_value);
d6f45b55c972 4809552: Optimize Arrays.fill(...)
never
parents: 1621
diff changeset
2397 _igvn.register_new_node_with_optimizer(store_value);
d6f45b55c972 4809552: Optimize Arrays.fill(...)
never
parents: 1621
diff changeset
2398 } else if (t == T_DOUBLE) {
d6f45b55c972 4809552: Optimize Arrays.fill(...)
never
parents: 1621
diff changeset
2399 store_value = new (C, 2) MoveD2LNode(store_value);
d6f45b55c972 4809552: Optimize Arrays.fill(...)
never
parents: 1621
diff changeset
2400 _igvn.register_new_node_with_optimizer(store_value);
d6f45b55c972 4809552: Optimize Arrays.fill(...)
never
parents: 1621
diff changeset
2401 }
d6f45b55c972 4809552: Optimize Arrays.fill(...)
never
parents: 1621
diff changeset
2402
d6f45b55c972 4809552: Optimize Arrays.fill(...)
never
parents: 1621
diff changeset
2403 Node* mem_phi = store->in(MemNode::Memory);
d6f45b55c972 4809552: Optimize Arrays.fill(...)
never
parents: 1621
diff changeset
2404 Node* result_ctrl;
d6f45b55c972 4809552: Optimize Arrays.fill(...)
never
parents: 1621
diff changeset
2405 Node* result_mem;
d6f45b55c972 4809552: Optimize Arrays.fill(...)
never
parents: 1621
diff changeset
2406 const TypeFunc* call_type = OptoRuntime::array_fill_Type();
d6f45b55c972 4809552: Optimize Arrays.fill(...)
never
parents: 1621
diff changeset
2407 int size = call_type->domain()->cnt();
d6f45b55c972 4809552: Optimize Arrays.fill(...)
never
parents: 1621
diff changeset
2408 CallLeafNode *call = new (C, size) CallLeafNoFPNode(call_type, fill,
d6f45b55c972 4809552: Optimize Arrays.fill(...)
never
parents: 1621
diff changeset
2409 fill_name, TypeAryPtr::get_array_body_type(t));
d6f45b55c972 4809552: Optimize Arrays.fill(...)
never
parents: 1621
diff changeset
2410 call->init_req(TypeFunc::Parms+0, from);
d6f45b55c972 4809552: Optimize Arrays.fill(...)
never
parents: 1621
diff changeset
2411 call->init_req(TypeFunc::Parms+1, store_value);
1844
75588558f1bf 6980792: Crash "exception happened outside interpreter, nmethods and vtable stubs (1)"
never
parents: 1813
diff changeset
2412 #ifdef _LP64
75588558f1bf 6980792: Crash "exception happened outside interpreter, nmethods and vtable stubs (1)"
never
parents: 1813
diff changeset
2413 len = new (C, 2) ConvI2LNode(len);
75588558f1bf 6980792: Crash "exception happened outside interpreter, nmethods and vtable stubs (1)"
never
parents: 1813
diff changeset
2414 _igvn.register_new_node_with_optimizer(len);
75588558f1bf 6980792: Crash "exception happened outside interpreter, nmethods and vtable stubs (1)"
never
parents: 1813
diff changeset
2415 #endif
1763
d6f45b55c972 4809552: Optimize Arrays.fill(...)
never
parents: 1621
diff changeset
2416 call->init_req(TypeFunc::Parms+2, len);
1844
75588558f1bf 6980792: Crash "exception happened outside interpreter, nmethods and vtable stubs (1)"
never
parents: 1813
diff changeset
2417 #ifdef _LP64
75588558f1bf 6980792: Crash "exception happened outside interpreter, nmethods and vtable stubs (1)"
never
parents: 1813
diff changeset
2418 call->init_req(TypeFunc::Parms+3, C->top());
75588558f1bf 6980792: Crash "exception happened outside interpreter, nmethods and vtable stubs (1)"
never
parents: 1813
diff changeset
2419 #endif
1763
d6f45b55c972 4809552: Optimize Arrays.fill(...)
never
parents: 1621
diff changeset
2420 call->init_req( TypeFunc::Control, head->init_control());
d6f45b55c972 4809552: Optimize Arrays.fill(...)
never
parents: 1621
diff changeset
2421 call->init_req( TypeFunc::I_O , C->top() ) ; // does no i/o
d6f45b55c972 4809552: Optimize Arrays.fill(...)
never
parents: 1621
diff changeset
2422 call->init_req( TypeFunc::Memory , mem_phi->in(LoopNode::EntryControl) );
d6f45b55c972 4809552: Optimize Arrays.fill(...)
never
parents: 1621
diff changeset
2423 call->init_req( TypeFunc::ReturnAdr, C->start()->proj_out(TypeFunc::ReturnAdr) );
d6f45b55c972 4809552: Optimize Arrays.fill(...)
never
parents: 1621
diff changeset
2424 call->init_req( TypeFunc::FramePtr, C->start()->proj_out(TypeFunc::FramePtr) );
d6f45b55c972 4809552: Optimize Arrays.fill(...)
never
parents: 1621
diff changeset
2425 _igvn.register_new_node_with_optimizer(call);
d6f45b55c972 4809552: Optimize Arrays.fill(...)
never
parents: 1621
diff changeset
2426 result_ctrl = new (C, 1) ProjNode(call,TypeFunc::Control);
d6f45b55c972 4809552: Optimize Arrays.fill(...)
never
parents: 1621
diff changeset
2427 _igvn.register_new_node_with_optimizer(result_ctrl);
d6f45b55c972 4809552: Optimize Arrays.fill(...)
never
parents: 1621
diff changeset
2428 result_mem = new (C, 1) ProjNode(call,TypeFunc::Memory);
d6f45b55c972 4809552: Optimize Arrays.fill(...)
never
parents: 1621
diff changeset
2429 _igvn.register_new_node_with_optimizer(result_mem);
d6f45b55c972 4809552: Optimize Arrays.fill(...)
never
parents: 1621
diff changeset
2430
d6f45b55c972 4809552: Optimize Arrays.fill(...)
never
parents: 1621
diff changeset
2431 // If this fill is tightly coupled to an allocation and overwrites
d6f45b55c972 4809552: Optimize Arrays.fill(...)
never
parents: 1621
diff changeset
2432 // the whole body, allow it to take over the zeroing.
d6f45b55c972 4809552: Optimize Arrays.fill(...)
never
parents: 1621
diff changeset
2433 AllocateNode* alloc = AllocateNode::Ideal_allocation(base, this);
d6f45b55c972 4809552: Optimize Arrays.fill(...)
never
parents: 1621
diff changeset
2434 if (alloc != NULL && alloc->is_AllocateArray()) {
d6f45b55c972 4809552: Optimize Arrays.fill(...)
never
parents: 1621
diff changeset
2435 Node* length = alloc->as_AllocateArray()->Ideal_length();
d6f45b55c972 4809552: Optimize Arrays.fill(...)
never
parents: 1621
diff changeset
2436 if (head->limit() == length &&
d6f45b55c972 4809552: Optimize Arrays.fill(...)
never
parents: 1621
diff changeset
2437 head->init_trip() == _igvn.intcon(0)) {
d6f45b55c972 4809552: Optimize Arrays.fill(...)
never
parents: 1621
diff changeset
2438 if (TraceOptimizeFill) {
d6f45b55c972 4809552: Optimize Arrays.fill(...)
never
parents: 1621
diff changeset
2439 tty->print_cr("Eliminated zeroing in allocation");
d6f45b55c972 4809552: Optimize Arrays.fill(...)
never
parents: 1621
diff changeset
2440 }
d6f45b55c972 4809552: Optimize Arrays.fill(...)
never
parents: 1621
diff changeset
2441 alloc->maybe_set_complete(&_igvn);
d6f45b55c972 4809552: Optimize Arrays.fill(...)
never
parents: 1621
diff changeset
2442 } else {
d6f45b55c972 4809552: Optimize Arrays.fill(...)
never
parents: 1621
diff changeset
2443 #ifdef ASSERT
d6f45b55c972 4809552: Optimize Arrays.fill(...)
never
parents: 1621
diff changeset
2444 if (TraceOptimizeFill) {
d6f45b55c972 4809552: Optimize Arrays.fill(...)
never
parents: 1621
diff changeset
2445 tty->print_cr("filling array but bounds don't match");
d6f45b55c972 4809552: Optimize Arrays.fill(...)
never
parents: 1621
diff changeset
2446 alloc->dump();
d6f45b55c972 4809552: Optimize Arrays.fill(...)
never
parents: 1621
diff changeset
2447 head->init_trip()->dump();
d6f45b55c972 4809552: Optimize Arrays.fill(...)
never
parents: 1621
diff changeset
2448 head->limit()->dump();
d6f45b55c972 4809552: Optimize Arrays.fill(...)
never
parents: 1621
diff changeset
2449 length->dump();
d6f45b55c972 4809552: Optimize Arrays.fill(...)
never
parents: 1621
diff changeset
2450 }
d6f45b55c972 4809552: Optimize Arrays.fill(...)
never
parents: 1621
diff changeset
2451 #endif
d6f45b55c972 4809552: Optimize Arrays.fill(...)
never
parents: 1621
diff changeset
2452 }
d6f45b55c972 4809552: Optimize Arrays.fill(...)
never
parents: 1621
diff changeset
2453 }
d6f45b55c972 4809552: Optimize Arrays.fill(...)
never
parents: 1621
diff changeset
2454
d6f45b55c972 4809552: Optimize Arrays.fill(...)
never
parents: 1621
diff changeset
2455 // Redirect the old control and memory edges that are outside the loop.
d6f45b55c972 4809552: Optimize Arrays.fill(...)
never
parents: 1621
diff changeset
2456 Node* exit = head->loopexit()->proj_out(0);
1813
c77e8f982901 6984979: OptimizeFill misses some cases with an odd memory graph
never
parents: 1785
diff changeset
2457 // Sometimes the memory phi of the head is used as the outgoing
c77e8f982901 6984979: OptimizeFill misses some cases with an odd memory graph
never
parents: 1785
diff changeset
2458 // state of the loop. It's safe in this case to replace it with the
c77e8f982901 6984979: OptimizeFill misses some cases with an odd memory graph
never
parents: 1785
diff changeset
2459 // result_mem.
c77e8f982901 6984979: OptimizeFill misses some cases with an odd memory graph
never
parents: 1785
diff changeset
2460 _igvn.replace_node(store->in(MemNode::Memory), result_mem);
1763
d6f45b55c972 4809552: Optimize Arrays.fill(...)
never
parents: 1621
diff changeset
2461 _igvn.replace_node(exit, result_ctrl);
d6f45b55c972 4809552: Optimize Arrays.fill(...)
never
parents: 1621
diff changeset
2462 _igvn.replace_node(store, result_mem);
d6f45b55c972 4809552: Optimize Arrays.fill(...)
never
parents: 1621
diff changeset
2463 // Any uses the increment outside of the loop become the loop limit.
d6f45b55c972 4809552: Optimize Arrays.fill(...)
never
parents: 1621
diff changeset
2464 _igvn.replace_node(head->incr(), head->limit());
d6f45b55c972 4809552: Optimize Arrays.fill(...)
never
parents: 1621
diff changeset
2465
d6f45b55c972 4809552: Optimize Arrays.fill(...)
never
parents: 1621
diff changeset
2466 // Disconnect the head from the loop.
d6f45b55c972 4809552: Optimize Arrays.fill(...)
never
parents: 1621
diff changeset
2467 for (uint i = 0; i < lpt->_body.size(); i++) {
d6f45b55c972 4809552: Optimize Arrays.fill(...)
never
parents: 1621
diff changeset
2468 Node* n = lpt->_body.at(i);
d6f45b55c972 4809552: Optimize Arrays.fill(...)
never
parents: 1621
diff changeset
2469 _igvn.replace_node(n, C->top());
d6f45b55c972 4809552: Optimize Arrays.fill(...)
never
parents: 1621
diff changeset
2470 }
d6f45b55c972 4809552: Optimize Arrays.fill(...)
never
parents: 1621
diff changeset
2471
d6f45b55c972 4809552: Optimize Arrays.fill(...)
never
parents: 1621
diff changeset
2472 return true;
d6f45b55c972 4809552: Optimize Arrays.fill(...)
never
parents: 1621
diff changeset
2473 }