annotate src/share/vm/opto/loopopts.cpp @ 3840:4e761e7e6e12

7070134: Hotspot crashes with sigsegv from PorterStemmer Summary: Do not move data nodes which are attached to a predicate test to a dominating test. Reviewed-by: never
author kvn
date Tue, 26 Jul 2011 19:35:23 -0700
parents 273b56978029
children d8cb48376797
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: 2383
diff changeset
2 * Copyright (c) 1999, 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: 1254
diff changeset
19 * Please contact Oracle, 500 Oracle Parkway, Redwood Shores, CA 94065 USA
c18cbe5936b8 6941466: Oracle rebranding changes for Hotspot repositories
trims
parents: 1254
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: 1254
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: 1621
diff changeset
25 #include "precompiled.hpp"
f95d63e2154a 6989984: Use standard include model for Hospot
stefank
parents: 1621
diff changeset
26 #include "memory/allocation.inline.hpp"
f95d63e2154a 6989984: Use standard include model for Hospot
stefank
parents: 1621
diff changeset
27 #include "opto/addnode.hpp"
f95d63e2154a 6989984: Use standard include model for Hospot
stefank
parents: 1621
diff changeset
28 #include "opto/connode.hpp"
f95d63e2154a 6989984: Use standard include model for Hospot
stefank
parents: 1621
diff changeset
29 #include "opto/divnode.hpp"
f95d63e2154a 6989984: Use standard include model for Hospot
stefank
parents: 1621
diff changeset
30 #include "opto/loopnode.hpp"
f95d63e2154a 6989984: Use standard include model for Hospot
stefank
parents: 1621
diff changeset
31 #include "opto/mulnode.hpp"
f95d63e2154a 6989984: Use standard include model for Hospot
stefank
parents: 1621
diff changeset
32 #include "opto/rootnode.hpp"
f95d63e2154a 6989984: Use standard include model for Hospot
stefank
parents: 1621
diff changeset
33 #include "opto/subnode.hpp"
0
a61af66fc99e Initial load
duke
parents:
diff changeset
34
a61af66fc99e Initial load
duke
parents:
diff changeset
35 //=============================================================================
a61af66fc99e Initial load
duke
parents:
diff changeset
36 //------------------------------split_thru_phi---------------------------------
a61af66fc99e Initial load
duke
parents:
diff changeset
37 // Split Node 'n' through merge point if there is enough win.
a61af66fc99e Initial load
duke
parents:
diff changeset
38 Node *PhaseIdealLoop::split_thru_phi( Node *n, Node *region, int policy ) {
69
8bb88f9877e5 6659207: access violation in CompilerThread0
never
parents: 64
diff changeset
39 if (n->Opcode() == Op_ConvI2L && n->bottom_type() != TypeLong::LONG) {
8bb88f9877e5 6659207: access violation in CompilerThread0
never
parents: 64
diff changeset
40 // ConvI2L may have type information on it which is unsafe to push up
8bb88f9877e5 6659207: access violation in CompilerThread0
never
parents: 64
diff changeset
41 // so disable this for now
8bb88f9877e5 6659207: access violation in CompilerThread0
never
parents: 64
diff changeset
42 return NULL;
8bb88f9877e5 6659207: access violation in CompilerThread0
never
parents: 64
diff changeset
43 }
0
a61af66fc99e Initial load
duke
parents:
diff changeset
44 int wins = 0;
2383
9dc311b8473e 7008866: Missing loop predicate for loop with multiple entries
kvn
parents: 1972
diff changeset
45 assert(!n->is_CFG(), "");
9dc311b8473e 7008866: Missing loop predicate for loop with multiple entries
kvn
parents: 1972
diff changeset
46 assert(region->is_Region(), "");
64
b8f5ba577b02 6673473: (Escape Analysis) Add the instance's field information to PhiNode
kvn
parents: 35
diff changeset
47
b8f5ba577b02 6673473: (Escape Analysis) Add the instance's field information to PhiNode
kvn
parents: 35
diff changeset
48 const Type* type = n->bottom_type();
b8f5ba577b02 6673473: (Escape Analysis) Add the instance's field information to PhiNode
kvn
parents: 35
diff changeset
49 const TypeOopPtr *t_oop = _igvn.type(n)->isa_oopptr();
b8f5ba577b02 6673473: (Escape Analysis) Add the instance's field information to PhiNode
kvn
parents: 35
diff changeset
50 Node *phi;
2383
9dc311b8473e 7008866: Missing loop predicate for loop with multiple entries
kvn
parents: 1972
diff changeset
51 if (t_oop != NULL && t_oop->is_known_instance_field()) {
64
b8f5ba577b02 6673473: (Escape Analysis) Add the instance's field information to PhiNode
kvn
parents: 35
diff changeset
52 int iid = t_oop->instance_id();
b8f5ba577b02 6673473: (Escape Analysis) Add the instance's field information to PhiNode
kvn
parents: 35
diff changeset
53 int index = C->get_alias_index(t_oop);
b8f5ba577b02 6673473: (Escape Analysis) Add the instance's field information to PhiNode
kvn
parents: 35
diff changeset
54 int offset = t_oop->offset();
b8f5ba577b02 6673473: (Escape Analysis) Add the instance's field information to PhiNode
kvn
parents: 35
diff changeset
55 phi = new (C,region->req()) PhiNode(region, type, NULL, iid, index, offset);
b8f5ba577b02 6673473: (Escape Analysis) Add the instance's field information to PhiNode
kvn
parents: 35
diff changeset
56 } else {
1254
4ee1c645110e 6924097: assert((_type == Type::MEMORY) == (_adr_type != 0),"adr_type for memory phis only")
kvn
parents: 866
diff changeset
57 phi = PhiNode::make_blank(region, n);
64
b8f5ba577b02 6673473: (Escape Analysis) Add the instance's field information to PhiNode
kvn
parents: 35
diff changeset
58 }
0
a61af66fc99e Initial load
duke
parents:
diff changeset
59 uint old_unique = C->unique();
2383
9dc311b8473e 7008866: Missing loop predicate for loop with multiple entries
kvn
parents: 1972
diff changeset
60 for (uint i = 1; i < region->req(); i++) {
0
a61af66fc99e Initial load
duke
parents:
diff changeset
61 Node *x;
a61af66fc99e Initial load
duke
parents:
diff changeset
62 Node* the_clone = NULL;
2383
9dc311b8473e 7008866: Missing loop predicate for loop with multiple entries
kvn
parents: 1972
diff changeset
63 if (region->in(i) == C->top()) {
0
a61af66fc99e Initial load
duke
parents:
diff changeset
64 x = C->top(); // Dead path? Use a dead data op
a61af66fc99e Initial load
duke
parents:
diff changeset
65 } else {
a61af66fc99e Initial load
duke
parents:
diff changeset
66 x = n->clone(); // Else clone up the data op
a61af66fc99e Initial load
duke
parents:
diff changeset
67 the_clone = x; // Remember for possible deletion.
a61af66fc99e Initial load
duke
parents:
diff changeset
68 // Alter data node to use pre-phi inputs
2383
9dc311b8473e 7008866: Missing loop predicate for loop with multiple entries
kvn
parents: 1972
diff changeset
69 if (n->in(0) == region)
0
a61af66fc99e Initial load
duke
parents:
diff changeset
70 x->set_req( 0, region->in(i) );
2383
9dc311b8473e 7008866: Missing loop predicate for loop with multiple entries
kvn
parents: 1972
diff changeset
71 for (uint j = 1; j < n->req(); j++) {
0
a61af66fc99e Initial load
duke
parents:
diff changeset
72 Node *in = n->in(j);
2383
9dc311b8473e 7008866: Missing loop predicate for loop with multiple entries
kvn
parents: 1972
diff changeset
73 if (in->is_Phi() && in->in(0) == region)
0
a61af66fc99e Initial load
duke
parents:
diff changeset
74 x->set_req( j, in->in(i) ); // Use pre-Phi input for the clone
a61af66fc99e Initial load
duke
parents:
diff changeset
75 }
a61af66fc99e Initial load
duke
parents:
diff changeset
76 }
a61af66fc99e Initial load
duke
parents:
diff changeset
77 // Check for a 'win' on some paths
a61af66fc99e Initial load
duke
parents:
diff changeset
78 const Type *t = x->Value(&_igvn);
a61af66fc99e Initial load
duke
parents:
diff changeset
79
a61af66fc99e Initial load
duke
parents:
diff changeset
80 bool singleton = t->singleton();
a61af66fc99e Initial load
duke
parents:
diff changeset
81
a61af66fc99e Initial load
duke
parents:
diff changeset
82 // A TOP singleton indicates that there are no possible values incoming
a61af66fc99e Initial load
duke
parents:
diff changeset
83 // along a particular edge. In most cases, this is OK, and the Phi will
a61af66fc99e Initial load
duke
parents:
diff changeset
84 // be eliminated later in an Ideal call. However, we can't allow this to
a61af66fc99e Initial load
duke
parents:
diff changeset
85 // happen if the singleton occurs on loop entry, as the elimination of
a61af66fc99e Initial load
duke
parents:
diff changeset
86 // the PhiNode may cause the resulting node to migrate back to a previous
a61af66fc99e Initial load
duke
parents:
diff changeset
87 // loop iteration.
2383
9dc311b8473e 7008866: Missing loop predicate for loop with multiple entries
kvn
parents: 1972
diff changeset
88 if (singleton && t == Type::TOP) {
0
a61af66fc99e Initial load
duke
parents:
diff changeset
89 // Is_Loop() == false does not confirm the absence of a loop (e.g., an
a61af66fc99e Initial load
duke
parents:
diff changeset
90 // irreducible loop may not be indicated by an affirmative is_Loop());
a61af66fc99e Initial load
duke
parents:
diff changeset
91 // therefore, the only top we can split thru a phi is on a backedge of
a61af66fc99e Initial load
duke
parents:
diff changeset
92 // a loop.
a61af66fc99e Initial load
duke
parents:
diff changeset
93 singleton &= region->is_Loop() && (i != LoopNode::EntryControl);
a61af66fc99e Initial load
duke
parents:
diff changeset
94 }
a61af66fc99e Initial load
duke
parents:
diff changeset
95
2383
9dc311b8473e 7008866: Missing loop predicate for loop with multiple entries
kvn
parents: 1972
diff changeset
96 if (singleton) {
0
a61af66fc99e Initial load
duke
parents:
diff changeset
97 wins++;
a61af66fc99e Initial load
duke
parents:
diff changeset
98 x = ((PhaseGVN&)_igvn).makecon(t);
a61af66fc99e Initial load
duke
parents:
diff changeset
99 } else {
a61af66fc99e Initial load
duke
parents:
diff changeset
100 // We now call Identity to try to simplify the cloned node.
a61af66fc99e Initial load
duke
parents:
diff changeset
101 // Note that some Identity methods call phase->type(this).
a61af66fc99e Initial load
duke
parents:
diff changeset
102 // Make sure that the type array is big enough for
a61af66fc99e Initial load
duke
parents:
diff changeset
103 // our new node, even though we may throw the node away.
a61af66fc99e Initial load
duke
parents:
diff changeset
104 // (Note: This tweaking with igvn only works because x is a new node.)
a61af66fc99e Initial load
duke
parents:
diff changeset
105 _igvn.set_type(x, t);
293
c3e045194476 6731641: assert(m->adr_type() == mach->adr_type(),"matcher should not change adr type")
kvn
parents: 253
diff changeset
106 // If x is a TypeNode, capture any more-precise type permanently into Node
605
98cb887364d3 6810672: Comment typos
twisti
parents: 401
diff changeset
107 // otherwise it will be not updated during igvn->transform since
293
c3e045194476 6731641: assert(m->adr_type() == mach->adr_type(),"matcher should not change adr type")
kvn
parents: 253
diff changeset
108 // igvn->type(x) is set to x->Value() already.
c3e045194476 6731641: assert(m->adr_type() == mach->adr_type(),"matcher should not change adr type")
kvn
parents: 253
diff changeset
109 x->raise_bottom_type(t);
0
a61af66fc99e Initial load
duke
parents:
diff changeset
110 Node *y = x->Identity(&_igvn);
2383
9dc311b8473e 7008866: Missing loop predicate for loop with multiple entries
kvn
parents: 1972
diff changeset
111 if (y != x) {
0
a61af66fc99e Initial load
duke
parents:
diff changeset
112 wins++;
a61af66fc99e Initial load
duke
parents:
diff changeset
113 x = y;
a61af66fc99e Initial load
duke
parents:
diff changeset
114 } else {
a61af66fc99e Initial load
duke
parents:
diff changeset
115 y = _igvn.hash_find(x);
2383
9dc311b8473e 7008866: Missing loop predicate for loop with multiple entries
kvn
parents: 1972
diff changeset
116 if (y) {
0
a61af66fc99e Initial load
duke
parents:
diff changeset
117 wins++;
a61af66fc99e Initial load
duke
parents:
diff changeset
118 x = y;
a61af66fc99e Initial load
duke
parents:
diff changeset
119 } else {
a61af66fc99e Initial load
duke
parents:
diff changeset
120 // Else x is a new node we are keeping
a61af66fc99e Initial load
duke
parents:
diff changeset
121 // We do not need register_new_node_with_optimizer
a61af66fc99e Initial load
duke
parents:
diff changeset
122 // because set_type has already been called.
a61af66fc99e Initial load
duke
parents:
diff changeset
123 _igvn._worklist.push(x);
a61af66fc99e Initial load
duke
parents:
diff changeset
124 }
a61af66fc99e Initial load
duke
parents:
diff changeset
125 }
a61af66fc99e Initial load
duke
parents:
diff changeset
126 }
a61af66fc99e Initial load
duke
parents:
diff changeset
127 if (x != the_clone && the_clone != NULL)
a61af66fc99e Initial load
duke
parents:
diff changeset
128 _igvn.remove_dead_node(the_clone);
a61af66fc99e Initial load
duke
parents:
diff changeset
129 phi->set_req( i, x );
a61af66fc99e Initial load
duke
parents:
diff changeset
130 }
a61af66fc99e Initial load
duke
parents:
diff changeset
131 // Too few wins?
2383
9dc311b8473e 7008866: Missing loop predicate for loop with multiple entries
kvn
parents: 1972
diff changeset
132 if (wins <= policy) {
0
a61af66fc99e Initial load
duke
parents:
diff changeset
133 _igvn.remove_dead_node(phi);
a61af66fc99e Initial load
duke
parents:
diff changeset
134 return NULL;
a61af66fc99e Initial load
duke
parents:
diff changeset
135 }
a61af66fc99e Initial load
duke
parents:
diff changeset
136
a61af66fc99e Initial load
duke
parents:
diff changeset
137 // Record Phi
a61af66fc99e Initial load
duke
parents:
diff changeset
138 register_new_node( phi, region );
a61af66fc99e Initial load
duke
parents:
diff changeset
139
2383
9dc311b8473e 7008866: Missing loop predicate for loop with multiple entries
kvn
parents: 1972
diff changeset
140 for (uint i2 = 1; i2 < phi->req(); i2++) {
0
a61af66fc99e Initial load
duke
parents:
diff changeset
141 Node *x = phi->in(i2);
a61af66fc99e Initial load
duke
parents:
diff changeset
142 // If we commoned up the cloned 'x' with another existing Node,
a61af66fc99e Initial load
duke
parents:
diff changeset
143 // the existing Node picks up a new use. We need to make the
a61af66fc99e Initial load
duke
parents:
diff changeset
144 // existing Node occur higher up so it dominates its uses.
a61af66fc99e Initial load
duke
parents:
diff changeset
145 Node *old_ctrl;
a61af66fc99e Initial load
duke
parents:
diff changeset
146 IdealLoopTree *old_loop;
a61af66fc99e Initial load
duke
parents:
diff changeset
147
2383
9dc311b8473e 7008866: Missing loop predicate for loop with multiple entries
kvn
parents: 1972
diff changeset
148 if (x->is_Con()) {
9dc311b8473e 7008866: Missing loop predicate for loop with multiple entries
kvn
parents: 1972
diff changeset
149 // Constant's control is always root.
9dc311b8473e 7008866: Missing loop predicate for loop with multiple entries
kvn
parents: 1972
diff changeset
150 set_ctrl(x, C->root());
9dc311b8473e 7008866: Missing loop predicate for loop with multiple entries
kvn
parents: 1972
diff changeset
151 continue;
9dc311b8473e 7008866: Missing loop predicate for loop with multiple entries
kvn
parents: 1972
diff changeset
152 }
0
a61af66fc99e Initial load
duke
parents:
diff changeset
153 // The occasional new node
2383
9dc311b8473e 7008866: Missing loop predicate for loop with multiple entries
kvn
parents: 1972
diff changeset
154 if (x->_idx >= old_unique) { // Found a new, unplaced node?
9dc311b8473e 7008866: Missing loop predicate for loop with multiple entries
kvn
parents: 1972
diff changeset
155 old_ctrl = NULL;
9dc311b8473e 7008866: Missing loop predicate for loop with multiple entries
kvn
parents: 1972
diff changeset
156 old_loop = NULL; // Not in any prior loop
0
a61af66fc99e Initial load
duke
parents:
diff changeset
157 } else {
2383
9dc311b8473e 7008866: Missing loop predicate for loop with multiple entries
kvn
parents: 1972
diff changeset
158 old_ctrl = get_ctrl(x);
0
a61af66fc99e Initial load
duke
parents:
diff changeset
159 old_loop = get_loop(old_ctrl); // Get prior loop
a61af66fc99e Initial load
duke
parents:
diff changeset
160 }
a61af66fc99e Initial load
duke
parents:
diff changeset
161 // New late point must dominate new use
2383
9dc311b8473e 7008866: Missing loop predicate for loop with multiple entries
kvn
parents: 1972
diff changeset
162 Node *new_ctrl = dom_lca(old_ctrl, region->in(i2));
9dc311b8473e 7008866: Missing loop predicate for loop with multiple entries
kvn
parents: 1972
diff changeset
163 if (new_ctrl == old_ctrl) // Nothing is changed
9dc311b8473e 7008866: Missing loop predicate for loop with multiple entries
kvn
parents: 1972
diff changeset
164 continue;
9dc311b8473e 7008866: Missing loop predicate for loop with multiple entries
kvn
parents: 1972
diff changeset
165
9dc311b8473e 7008866: Missing loop predicate for loop with multiple entries
kvn
parents: 1972
diff changeset
166 IdealLoopTree *new_loop = get_loop(new_ctrl);
9dc311b8473e 7008866: Missing loop predicate for loop with multiple entries
kvn
parents: 1972
diff changeset
167
9dc311b8473e 7008866: Missing loop predicate for loop with multiple entries
kvn
parents: 1972
diff changeset
168 // Don't move x into a loop if its uses are
9dc311b8473e 7008866: Missing loop predicate for loop with multiple entries
kvn
parents: 1972
diff changeset
169 // outside of loop. Otherwise x will be cloned
9dc311b8473e 7008866: Missing loop predicate for loop with multiple entries
kvn
parents: 1972
diff changeset
170 // for each use outside of this loop.
9dc311b8473e 7008866: Missing loop predicate for loop with multiple entries
kvn
parents: 1972
diff changeset
171 IdealLoopTree *use_loop = get_loop(region);
9dc311b8473e 7008866: Missing loop predicate for loop with multiple entries
kvn
parents: 1972
diff changeset
172 if (!new_loop->is_member(use_loop) &&
9dc311b8473e 7008866: Missing loop predicate for loop with multiple entries
kvn
parents: 1972
diff changeset
173 (old_loop == NULL || !new_loop->is_member(old_loop))) {
9dc311b8473e 7008866: Missing loop predicate for loop with multiple entries
kvn
parents: 1972
diff changeset
174 // Take early control, later control will be recalculated
9dc311b8473e 7008866: Missing loop predicate for loop with multiple entries
kvn
parents: 1972
diff changeset
175 // during next iteration of loop optimizations.
9dc311b8473e 7008866: Missing loop predicate for loop with multiple entries
kvn
parents: 1972
diff changeset
176 new_ctrl = get_early_ctrl(x);
9dc311b8473e 7008866: Missing loop predicate for loop with multiple entries
kvn
parents: 1972
diff changeset
177 new_loop = get_loop(new_ctrl);
9dc311b8473e 7008866: Missing loop predicate for loop with multiple entries
kvn
parents: 1972
diff changeset
178 }
0
a61af66fc99e Initial load
duke
parents:
diff changeset
179 // Set new location
a61af66fc99e Initial load
duke
parents:
diff changeset
180 set_ctrl(x, new_ctrl);
a61af66fc99e Initial load
duke
parents:
diff changeset
181 // If changing loop bodies, see if we need to collect into new body
2383
9dc311b8473e 7008866: Missing loop predicate for loop with multiple entries
kvn
parents: 1972
diff changeset
182 if (old_loop != new_loop) {
9dc311b8473e 7008866: Missing loop predicate for loop with multiple entries
kvn
parents: 1972
diff changeset
183 if (old_loop && !old_loop->_child)
0
a61af66fc99e Initial load
duke
parents:
diff changeset
184 old_loop->_body.yank(x);
2383
9dc311b8473e 7008866: Missing loop predicate for loop with multiple entries
kvn
parents: 1972
diff changeset
185 if (!new_loop->_child)
0
a61af66fc99e Initial load
duke
parents:
diff changeset
186 new_loop->_body.push(x); // Collect body info
a61af66fc99e Initial load
duke
parents:
diff changeset
187 }
a61af66fc99e Initial load
duke
parents:
diff changeset
188 }
a61af66fc99e Initial load
duke
parents:
diff changeset
189
a61af66fc99e Initial load
duke
parents:
diff changeset
190 return phi;
a61af66fc99e Initial load
duke
parents:
diff changeset
191 }
a61af66fc99e Initial load
duke
parents:
diff changeset
192
a61af66fc99e Initial load
duke
parents:
diff changeset
193 //------------------------------dominated_by------------------------------------
a61af66fc99e Initial load
duke
parents:
diff changeset
194 // Replace the dominated test with an obvious true or false. Place it on the
a61af66fc99e Initial load
duke
parents:
diff changeset
195 // IGVN worklist for later cleanup. Move control-dependent data Nodes on the
a61af66fc99e Initial load
duke
parents:
diff changeset
196 // live path up to the dominating control.
3840
4e761e7e6e12 7070134: Hotspot crashes with sigsegv from PorterStemmer
kvn
parents: 3281
diff changeset
197 void PhaseIdealLoop::dominated_by( Node *prevdom, Node *iff, bool flip, bool exclude_loop_predicate ) {
0
a61af66fc99e Initial load
duke
parents:
diff changeset
198 #ifndef PRODUCT
2383
9dc311b8473e 7008866: Missing loop predicate for loop with multiple entries
kvn
parents: 1972
diff changeset
199 if (VerifyLoopOptimizations && PrintOpto) tty->print_cr("dominating test");
0
a61af66fc99e Initial load
duke
parents:
diff changeset
200 #endif
a61af66fc99e Initial load
duke
parents:
diff changeset
201
a61af66fc99e Initial load
duke
parents:
diff changeset
202
a61af66fc99e Initial load
duke
parents:
diff changeset
203 // prevdom is the dominating projection of the dominating test.
a61af66fc99e Initial load
duke
parents:
diff changeset
204 assert( iff->is_If(), "" );
a61af66fc99e Initial load
duke
parents:
diff changeset
205 assert( iff->Opcode() == Op_If || iff->Opcode() == Op_CountedLoopEnd, "Check this code when new subtype is added");
a61af66fc99e Initial load
duke
parents:
diff changeset
206 int pop = prevdom->Opcode();
a61af66fc99e Initial load
duke
parents:
diff changeset
207 assert( pop == Op_IfFalse || pop == Op_IfTrue, "" );
2383
9dc311b8473e 7008866: Missing loop predicate for loop with multiple entries
kvn
parents: 1972
diff changeset
208 if (flip) {
9dc311b8473e 7008866: Missing loop predicate for loop with multiple entries
kvn
parents: 1972
diff changeset
209 if (pop == Op_IfTrue)
9dc311b8473e 7008866: Missing loop predicate for loop with multiple entries
kvn
parents: 1972
diff changeset
210 pop = Op_IfFalse;
9dc311b8473e 7008866: Missing loop predicate for loop with multiple entries
kvn
parents: 1972
diff changeset
211 else
9dc311b8473e 7008866: Missing loop predicate for loop with multiple entries
kvn
parents: 1972
diff changeset
212 pop = Op_IfTrue;
9dc311b8473e 7008866: Missing loop predicate for loop with multiple entries
kvn
parents: 1972
diff changeset
213 }
0
a61af66fc99e Initial load
duke
parents:
diff changeset
214 // 'con' is set to true or false to kill the dominated test.
a61af66fc99e Initial load
duke
parents:
diff changeset
215 Node *con = _igvn.makecon(pop == Op_IfTrue ? TypeInt::ONE : TypeInt::ZERO);
a61af66fc99e Initial load
duke
parents:
diff changeset
216 set_ctrl(con, C->root()); // Constant gets a new use
a61af66fc99e Initial load
duke
parents:
diff changeset
217 // Hack the dominated test
a61af66fc99e Initial load
duke
parents:
diff changeset
218 _igvn.hash_delete(iff);
a61af66fc99e Initial load
duke
parents:
diff changeset
219 iff->set_req(1, con);
a61af66fc99e Initial load
duke
parents:
diff changeset
220 _igvn._worklist.push(iff);
a61af66fc99e Initial load
duke
parents:
diff changeset
221
a61af66fc99e Initial load
duke
parents:
diff changeset
222 // If I dont have a reachable TRUE and FALSE path following the IfNode then
a61af66fc99e Initial load
duke
parents:
diff changeset
223 // I can assume this path reaches an infinite loop. In this case it's not
a61af66fc99e Initial load
duke
parents:
diff changeset
224 // important to optimize the data Nodes - either the whole compilation will
a61af66fc99e Initial load
duke
parents:
diff changeset
225 // be tossed or this path (and all data Nodes) will go dead.
2383
9dc311b8473e 7008866: Missing loop predicate for loop with multiple entries
kvn
parents: 1972
diff changeset
226 if (iff->outcnt() != 2) return;
0
a61af66fc99e Initial load
duke
parents:
diff changeset
227
a61af66fc99e Initial load
duke
parents:
diff changeset
228 // Make control-dependent data Nodes on the live path (path that will remain
a61af66fc99e Initial load
duke
parents:
diff changeset
229 // once the dominated IF is removed) become control-dependent on the
a61af66fc99e Initial load
duke
parents:
diff changeset
230 // dominating projection.
3840
4e761e7e6e12 7070134: Hotspot crashes with sigsegv from PorterStemmer
kvn
parents: 3281
diff changeset
231 Node* dp = iff->as_If()->proj_out(pop == Op_IfTrue);
4e761e7e6e12 7070134: Hotspot crashes with sigsegv from PorterStemmer
kvn
parents: 3281
diff changeset
232
4e761e7e6e12 7070134: Hotspot crashes with sigsegv from PorterStemmer
kvn
parents: 3281
diff changeset
233 // Loop predicates may have depending checks which should not
4e761e7e6e12 7070134: Hotspot crashes with sigsegv from PorterStemmer
kvn
parents: 3281
diff changeset
234 // be skipped. For example, range check predicate has two checks
4e761e7e6e12 7070134: Hotspot crashes with sigsegv from PorterStemmer
kvn
parents: 3281
diff changeset
235 // for lower and upper bounds.
4e761e7e6e12 7070134: Hotspot crashes with sigsegv from PorterStemmer
kvn
parents: 3281
diff changeset
236 ProjNode* unc_proj = iff->as_If()->proj_out(1 - dp->as_Proj()->_con)->as_Proj();
4e761e7e6e12 7070134: Hotspot crashes with sigsegv from PorterStemmer
kvn
parents: 3281
diff changeset
237 if (exclude_loop_predicate &&
4e761e7e6e12 7070134: Hotspot crashes with sigsegv from PorterStemmer
kvn
parents: 3281
diff changeset
238 is_uncommon_trap_proj(unc_proj, Deoptimization::Reason_predicate))
4e761e7e6e12 7070134: Hotspot crashes with sigsegv from PorterStemmer
kvn
parents: 3281
diff changeset
239 return; // Let IGVN transformation change control dependence.
4e761e7e6e12 7070134: Hotspot crashes with sigsegv from PorterStemmer
kvn
parents: 3281
diff changeset
240
0
a61af66fc99e Initial load
duke
parents:
diff changeset
241 IdealLoopTree *old_loop = get_loop(dp);
a61af66fc99e Initial load
duke
parents:
diff changeset
242
a61af66fc99e Initial load
duke
parents:
diff changeset
243 for (DUIterator_Fast imax, i = dp->fast_outs(imax); i < imax; i++) {
a61af66fc99e Initial load
duke
parents:
diff changeset
244 Node* cd = dp->fast_out(i); // Control-dependent node
2383
9dc311b8473e 7008866: Missing loop predicate for loop with multiple entries
kvn
parents: 1972
diff changeset
245 if (cd->depends_only_on_test()) {
9dc311b8473e 7008866: Missing loop predicate for loop with multiple entries
kvn
parents: 1972
diff changeset
246 assert(cd->in(0) == dp, "");
9dc311b8473e 7008866: Missing loop predicate for loop with multiple entries
kvn
parents: 1972
diff changeset
247 _igvn.hash_delete(cd);
0
a61af66fc99e Initial load
duke
parents:
diff changeset
248 cd->set_req(0, prevdom);
2383
9dc311b8473e 7008866: Missing loop predicate for loop with multiple entries
kvn
parents: 1972
diff changeset
249 set_early_ctrl(cd);
0
a61af66fc99e Initial load
duke
parents:
diff changeset
250 _igvn._worklist.push(cd);
a61af66fc99e Initial load
duke
parents:
diff changeset
251 IdealLoopTree *new_loop = get_loop(get_ctrl(cd));
2383
9dc311b8473e 7008866: Missing loop predicate for loop with multiple entries
kvn
parents: 1972
diff changeset
252 if (old_loop != new_loop) {
9dc311b8473e 7008866: Missing loop predicate for loop with multiple entries
kvn
parents: 1972
diff changeset
253 if (!old_loop->_child) old_loop->_body.yank(cd);
9dc311b8473e 7008866: Missing loop predicate for loop with multiple entries
kvn
parents: 1972
diff changeset
254 if (!new_loop->_child) new_loop->_body.push(cd);
0
a61af66fc99e Initial load
duke
parents:
diff changeset
255 }
a61af66fc99e Initial load
duke
parents:
diff changeset
256 --i;
a61af66fc99e Initial load
duke
parents:
diff changeset
257 --imax;
a61af66fc99e Initial load
duke
parents:
diff changeset
258 }
a61af66fc99e Initial load
duke
parents:
diff changeset
259 }
a61af66fc99e Initial load
duke
parents:
diff changeset
260 }
a61af66fc99e Initial load
duke
parents:
diff changeset
261
a61af66fc99e Initial load
duke
parents:
diff changeset
262 //------------------------------has_local_phi_input----------------------------
a61af66fc99e Initial load
duke
parents:
diff changeset
263 // Return TRUE if 'n' has Phi inputs from its local block and no other
a61af66fc99e Initial load
duke
parents:
diff changeset
264 // block-local inputs (all non-local-phi inputs come from earlier blocks)
a61af66fc99e Initial load
duke
parents:
diff changeset
265 Node *PhaseIdealLoop::has_local_phi_input( Node *n ) {
a61af66fc99e Initial load
duke
parents:
diff changeset
266 Node *n_ctrl = get_ctrl(n);
a61af66fc99e Initial load
duke
parents:
diff changeset
267 // See if some inputs come from a Phi in this block, or from before
a61af66fc99e Initial load
duke
parents:
diff changeset
268 // this block.
a61af66fc99e Initial load
duke
parents:
diff changeset
269 uint i;
a61af66fc99e Initial load
duke
parents:
diff changeset
270 for( i = 1; i < n->req(); i++ ) {
a61af66fc99e Initial load
duke
parents:
diff changeset
271 Node *phi = n->in(i);
a61af66fc99e Initial load
duke
parents:
diff changeset
272 if( phi->is_Phi() && phi->in(0) == n_ctrl )
a61af66fc99e Initial load
duke
parents:
diff changeset
273 break;
a61af66fc99e Initial load
duke
parents:
diff changeset
274 }
a61af66fc99e Initial load
duke
parents:
diff changeset
275 if( i >= n->req() )
a61af66fc99e Initial load
duke
parents:
diff changeset
276 return NULL; // No Phi inputs; nowhere to clone thru
a61af66fc99e Initial load
duke
parents:
diff changeset
277
a61af66fc99e Initial load
duke
parents:
diff changeset
278 // Check for inputs created between 'n' and the Phi input. These
a61af66fc99e Initial load
duke
parents:
diff changeset
279 // must split as well; they have already been given the chance
a61af66fc99e Initial load
duke
parents:
diff changeset
280 // (courtesy of a post-order visit) and since they did not we must
a61af66fc99e Initial load
duke
parents:
diff changeset
281 // recover the 'cost' of splitting them by being very profitable
a61af66fc99e Initial load
duke
parents:
diff changeset
282 // when splitting 'n'. Since this is unlikely we simply give up.
a61af66fc99e Initial load
duke
parents:
diff changeset
283 for( i = 1; i < n->req(); i++ ) {
a61af66fc99e Initial load
duke
parents:
diff changeset
284 Node *m = n->in(i);
a61af66fc99e Initial load
duke
parents:
diff changeset
285 if( get_ctrl(m) == n_ctrl && !m->is_Phi() ) {
a61af66fc99e Initial load
duke
parents:
diff changeset
286 // We allow the special case of AddP's with no local inputs.
a61af66fc99e Initial load
duke
parents:
diff changeset
287 // This allows us to split-up address expressions.
a61af66fc99e Initial load
duke
parents:
diff changeset
288 if (m->is_AddP() &&
a61af66fc99e Initial load
duke
parents:
diff changeset
289 get_ctrl(m->in(2)) != n_ctrl &&
a61af66fc99e Initial load
duke
parents:
diff changeset
290 get_ctrl(m->in(3)) != n_ctrl) {
a61af66fc99e Initial load
duke
parents:
diff changeset
291 // Move the AddP up to dominating point
a61af66fc99e Initial load
duke
parents:
diff changeset
292 set_ctrl_and_loop(m, find_non_split_ctrl(idom(n_ctrl)));
a61af66fc99e Initial load
duke
parents:
diff changeset
293 continue;
a61af66fc99e Initial load
duke
parents:
diff changeset
294 }
a61af66fc99e Initial load
duke
parents:
diff changeset
295 return NULL;
a61af66fc99e Initial load
duke
parents:
diff changeset
296 }
a61af66fc99e Initial load
duke
parents:
diff changeset
297 }
a61af66fc99e Initial load
duke
parents:
diff changeset
298
a61af66fc99e Initial load
duke
parents:
diff changeset
299 return n_ctrl;
a61af66fc99e Initial load
duke
parents:
diff changeset
300 }
a61af66fc99e Initial load
duke
parents:
diff changeset
301
a61af66fc99e Initial load
duke
parents:
diff changeset
302 //------------------------------remix_address_expressions----------------------
a61af66fc99e Initial load
duke
parents:
diff changeset
303 // Rework addressing expressions to get the most loop-invariant stuff
a61af66fc99e Initial load
duke
parents:
diff changeset
304 // moved out. We'd like to do all associative operators, but it's especially
a61af66fc99e Initial load
duke
parents:
diff changeset
305 // important (common) to do address expressions.
a61af66fc99e Initial load
duke
parents:
diff changeset
306 Node *PhaseIdealLoop::remix_address_expressions( Node *n ) {
a61af66fc99e Initial load
duke
parents:
diff changeset
307 if (!has_ctrl(n)) return NULL;
a61af66fc99e Initial load
duke
parents:
diff changeset
308 Node *n_ctrl = get_ctrl(n);
a61af66fc99e Initial load
duke
parents:
diff changeset
309 IdealLoopTree *n_loop = get_loop(n_ctrl);
a61af66fc99e Initial load
duke
parents:
diff changeset
310
a61af66fc99e Initial load
duke
parents:
diff changeset
311 // See if 'n' mixes loop-varying and loop-invariant inputs and
a61af66fc99e Initial load
duke
parents:
diff changeset
312 // itself is loop-varying.
a61af66fc99e Initial load
duke
parents:
diff changeset
313
a61af66fc99e Initial load
duke
parents:
diff changeset
314 // Only interested in binary ops (and AddP)
a61af66fc99e Initial load
duke
parents:
diff changeset
315 if( n->req() < 3 || n->req() > 4 ) return NULL;
a61af66fc99e Initial load
duke
parents:
diff changeset
316
a61af66fc99e Initial load
duke
parents:
diff changeset
317 Node *n1_ctrl = get_ctrl(n->in( 1));
a61af66fc99e Initial load
duke
parents:
diff changeset
318 Node *n2_ctrl = get_ctrl(n->in( 2));
a61af66fc99e Initial load
duke
parents:
diff changeset
319 Node *n3_ctrl = get_ctrl(n->in(n->req() == 3 ? 2 : 3));
a61af66fc99e Initial load
duke
parents:
diff changeset
320 IdealLoopTree *n1_loop = get_loop( n1_ctrl );
a61af66fc99e Initial load
duke
parents:
diff changeset
321 IdealLoopTree *n2_loop = get_loop( n2_ctrl );
a61af66fc99e Initial load
duke
parents:
diff changeset
322 IdealLoopTree *n3_loop = get_loop( n3_ctrl );
a61af66fc99e Initial load
duke
parents:
diff changeset
323
a61af66fc99e Initial load
duke
parents:
diff changeset
324 // Does one of my inputs spin in a tighter loop than self?
a61af66fc99e Initial load
duke
parents:
diff changeset
325 if( (n_loop->is_member( n1_loop ) && n_loop != n1_loop) ||
a61af66fc99e Initial load
duke
parents:
diff changeset
326 (n_loop->is_member( n2_loop ) && n_loop != n2_loop) ||
a61af66fc99e Initial load
duke
parents:
diff changeset
327 (n_loop->is_member( n3_loop ) && n_loop != n3_loop) )
a61af66fc99e Initial load
duke
parents:
diff changeset
328 return NULL; // Leave well enough alone
a61af66fc99e Initial load
duke
parents:
diff changeset
329
a61af66fc99e Initial load
duke
parents:
diff changeset
330 // Is at least one of my inputs loop-invariant?
a61af66fc99e Initial load
duke
parents:
diff changeset
331 if( n1_loop == n_loop &&
a61af66fc99e Initial load
duke
parents:
diff changeset
332 n2_loop == n_loop &&
a61af66fc99e Initial load
duke
parents:
diff changeset
333 n3_loop == n_loop )
a61af66fc99e Initial load
duke
parents:
diff changeset
334 return NULL; // No loop-invariant inputs
a61af66fc99e Initial load
duke
parents:
diff changeset
335
a61af66fc99e Initial load
duke
parents:
diff changeset
336
a61af66fc99e Initial load
duke
parents:
diff changeset
337 int n_op = n->Opcode();
a61af66fc99e Initial load
duke
parents:
diff changeset
338
a61af66fc99e Initial load
duke
parents:
diff changeset
339 // Replace expressions like ((V+I) << 2) with (V<<2 + I<<2).
a61af66fc99e Initial load
duke
parents:
diff changeset
340 if( n_op == Op_LShiftI ) {
a61af66fc99e Initial load
duke
parents:
diff changeset
341 // Scale is loop invariant
a61af66fc99e Initial load
duke
parents:
diff changeset
342 Node *scale = n->in(2);
a61af66fc99e Initial load
duke
parents:
diff changeset
343 Node *scale_ctrl = get_ctrl(scale);
a61af66fc99e Initial load
duke
parents:
diff changeset
344 IdealLoopTree *scale_loop = get_loop(scale_ctrl );
a61af66fc99e Initial load
duke
parents:
diff changeset
345 if( n_loop == scale_loop || !scale_loop->is_member( n_loop ) )
a61af66fc99e Initial load
duke
parents:
diff changeset
346 return NULL;
a61af66fc99e Initial load
duke
parents:
diff changeset
347 const TypeInt *scale_t = scale->bottom_type()->isa_int();
a61af66fc99e Initial load
duke
parents:
diff changeset
348 if( scale_t && scale_t->is_con() && scale_t->get_con() >= 16 )
a61af66fc99e Initial load
duke
parents:
diff changeset
349 return NULL; // Dont bother with byte/short masking
a61af66fc99e Initial load
duke
parents:
diff changeset
350 // Add must vary with loop (else shift would be loop-invariant)
a61af66fc99e Initial load
duke
parents:
diff changeset
351 Node *add = n->in(1);
a61af66fc99e Initial load
duke
parents:
diff changeset
352 Node *add_ctrl = get_ctrl(add);
a61af66fc99e Initial load
duke
parents:
diff changeset
353 IdealLoopTree *add_loop = get_loop(add_ctrl);
a61af66fc99e Initial load
duke
parents:
diff changeset
354 //assert( n_loop == add_loop, "" );
a61af66fc99e Initial load
duke
parents:
diff changeset
355 if( n_loop != add_loop ) return NULL; // happens w/ evil ZKM loops
a61af66fc99e Initial load
duke
parents:
diff changeset
356
a61af66fc99e Initial load
duke
parents:
diff changeset
357 // Convert I-V into I+ (0-V); same for V-I
a61af66fc99e Initial load
duke
parents:
diff changeset
358 if( add->Opcode() == Op_SubI &&
a61af66fc99e Initial load
duke
parents:
diff changeset
359 _igvn.type( add->in(1) ) != TypeInt::ZERO ) {
a61af66fc99e Initial load
duke
parents:
diff changeset
360 Node *zero = _igvn.intcon(0);
a61af66fc99e Initial load
duke
parents:
diff changeset
361 set_ctrl(zero, C->root());
a61af66fc99e Initial load
duke
parents:
diff changeset
362 Node *neg = new (C, 3) SubINode( _igvn.intcon(0), add->in(2) );
a61af66fc99e Initial load
duke
parents:
diff changeset
363 register_new_node( neg, get_ctrl(add->in(2) ) );
a61af66fc99e Initial load
duke
parents:
diff changeset
364 add = new (C, 3) AddINode( add->in(1), neg );
a61af66fc99e Initial load
duke
parents:
diff changeset
365 register_new_node( add, add_ctrl );
a61af66fc99e Initial load
duke
parents:
diff changeset
366 }
a61af66fc99e Initial load
duke
parents:
diff changeset
367 if( add->Opcode() != Op_AddI ) return NULL;
a61af66fc99e Initial load
duke
parents:
diff changeset
368 // See if one add input is loop invariant
a61af66fc99e Initial load
duke
parents:
diff changeset
369 Node *add_var = add->in(1);
a61af66fc99e Initial load
duke
parents:
diff changeset
370 Node *add_var_ctrl = get_ctrl(add_var);
a61af66fc99e Initial load
duke
parents:
diff changeset
371 IdealLoopTree *add_var_loop = get_loop(add_var_ctrl );
a61af66fc99e Initial load
duke
parents:
diff changeset
372 Node *add_invar = add->in(2);
a61af66fc99e Initial load
duke
parents:
diff changeset
373 Node *add_invar_ctrl = get_ctrl(add_invar);
a61af66fc99e Initial load
duke
parents:
diff changeset
374 IdealLoopTree *add_invar_loop = get_loop(add_invar_ctrl );
a61af66fc99e Initial load
duke
parents:
diff changeset
375 if( add_var_loop == n_loop ) {
a61af66fc99e Initial load
duke
parents:
diff changeset
376 } else if( add_invar_loop == n_loop ) {
a61af66fc99e Initial load
duke
parents:
diff changeset
377 // Swap to find the invariant part
a61af66fc99e Initial load
duke
parents:
diff changeset
378 add_invar = add_var;
a61af66fc99e Initial load
duke
parents:
diff changeset
379 add_invar_ctrl = add_var_ctrl;
a61af66fc99e Initial load
duke
parents:
diff changeset
380 add_invar_loop = add_var_loop;
a61af66fc99e Initial load
duke
parents:
diff changeset
381 add_var = add->in(2);
a61af66fc99e Initial load
duke
parents:
diff changeset
382 Node *add_var_ctrl = get_ctrl(add_var);
a61af66fc99e Initial load
duke
parents:
diff changeset
383 IdealLoopTree *add_var_loop = get_loop(add_var_ctrl );
a61af66fc99e Initial load
duke
parents:
diff changeset
384 } else // Else neither input is loop invariant
a61af66fc99e Initial load
duke
parents:
diff changeset
385 return NULL;
a61af66fc99e Initial load
duke
parents:
diff changeset
386 if( n_loop == add_invar_loop || !add_invar_loop->is_member( n_loop ) )
a61af66fc99e Initial load
duke
parents:
diff changeset
387 return NULL; // No invariant part of the add?
a61af66fc99e Initial load
duke
parents:
diff changeset
388
a61af66fc99e Initial load
duke
parents:
diff changeset
389 // Yes! Reshape address expression!
a61af66fc99e Initial load
duke
parents:
diff changeset
390 Node *inv_scale = new (C, 3) LShiftINode( add_invar, scale );
850
fd50a67f97d1 6860469: remix_address_expressions sets incorrect control causing crash in split_if_with_block_post
never
parents: 834
diff changeset
391 Node *inv_scale_ctrl =
fd50a67f97d1 6860469: remix_address_expressions sets incorrect control causing crash in split_if_with_block_post
never
parents: 834
diff changeset
392 dom_depth(add_invar_ctrl) > dom_depth(scale_ctrl) ?
fd50a67f97d1 6860469: remix_address_expressions sets incorrect control causing crash in split_if_with_block_post
never
parents: 834
diff changeset
393 add_invar_ctrl : scale_ctrl;
fd50a67f97d1 6860469: remix_address_expressions sets incorrect control causing crash in split_if_with_block_post
never
parents: 834
diff changeset
394 register_new_node( inv_scale, inv_scale_ctrl );
0
a61af66fc99e Initial load
duke
parents:
diff changeset
395 Node *var_scale = new (C, 3) LShiftINode( add_var, scale );
a61af66fc99e Initial load
duke
parents:
diff changeset
396 register_new_node( var_scale, n_ctrl );
a61af66fc99e Initial load
duke
parents:
diff changeset
397 Node *var_add = new (C, 3) AddINode( var_scale, inv_scale );
a61af66fc99e Initial load
duke
parents:
diff changeset
398 register_new_node( var_add, n_ctrl );
1621
6027dddc26c6 6677629: PhaseIterGVN::subsume_node() should call hash_delete() and add_users_to_worklist()
kvn
parents: 1552
diff changeset
399 _igvn.replace_node( n, var_add );
0
a61af66fc99e Initial load
duke
parents:
diff changeset
400 return var_add;
a61af66fc99e Initial load
duke
parents:
diff changeset
401 }
a61af66fc99e Initial load
duke
parents:
diff changeset
402
a61af66fc99e Initial load
duke
parents:
diff changeset
403 // Replace (I+V) with (V+I)
a61af66fc99e Initial load
duke
parents:
diff changeset
404 if( n_op == Op_AddI ||
a61af66fc99e Initial load
duke
parents:
diff changeset
405 n_op == Op_AddL ||
a61af66fc99e Initial load
duke
parents:
diff changeset
406 n_op == Op_AddF ||
a61af66fc99e Initial load
duke
parents:
diff changeset
407 n_op == Op_AddD ||
a61af66fc99e Initial load
duke
parents:
diff changeset
408 n_op == Op_MulI ||
a61af66fc99e Initial load
duke
parents:
diff changeset
409 n_op == Op_MulL ||
a61af66fc99e Initial load
duke
parents:
diff changeset
410 n_op == Op_MulF ||
a61af66fc99e Initial load
duke
parents:
diff changeset
411 n_op == Op_MulD ) {
a61af66fc99e Initial load
duke
parents:
diff changeset
412 if( n2_loop == n_loop ) {
a61af66fc99e Initial load
duke
parents:
diff changeset
413 assert( n1_loop != n_loop, "" );
a61af66fc99e Initial load
duke
parents:
diff changeset
414 n->swap_edges(1, 2);
a61af66fc99e Initial load
duke
parents:
diff changeset
415 }
a61af66fc99e Initial load
duke
parents:
diff changeset
416 }
a61af66fc99e Initial load
duke
parents:
diff changeset
417
a61af66fc99e Initial load
duke
parents:
diff changeset
418 // Replace ((I1 +p V) +p I2) with ((I1 +p I2) +p V),
a61af66fc99e Initial load
duke
parents:
diff changeset
419 // but not if I2 is a constant.
a61af66fc99e Initial load
duke
parents:
diff changeset
420 if( n_op == Op_AddP ) {
a61af66fc99e Initial load
duke
parents:
diff changeset
421 if( n2_loop == n_loop && n3_loop != n_loop ) {
a61af66fc99e Initial load
duke
parents:
diff changeset
422 if( n->in(2)->Opcode() == Op_AddP && !n->in(3)->is_Con() ) {
a61af66fc99e Initial load
duke
parents:
diff changeset
423 Node *n22_ctrl = get_ctrl(n->in(2)->in(2));
a61af66fc99e Initial load
duke
parents:
diff changeset
424 Node *n23_ctrl = get_ctrl(n->in(2)->in(3));
a61af66fc99e Initial load
duke
parents:
diff changeset
425 IdealLoopTree *n22loop = get_loop( n22_ctrl );
a61af66fc99e Initial load
duke
parents:
diff changeset
426 IdealLoopTree *n23_loop = get_loop( n23_ctrl );
a61af66fc99e Initial load
duke
parents:
diff changeset
427 if( n22loop != n_loop && n22loop->is_member(n_loop) &&
a61af66fc99e Initial load
duke
parents:
diff changeset
428 n23_loop == n_loop ) {
a61af66fc99e Initial load
duke
parents:
diff changeset
429 Node *add1 = new (C, 4) AddPNode( n->in(1), n->in(2)->in(2), n->in(3) );
a61af66fc99e Initial load
duke
parents:
diff changeset
430 // Stuff new AddP in the loop preheader
a61af66fc99e Initial load
duke
parents:
diff changeset
431 register_new_node( add1, n_loop->_head->in(LoopNode::EntryControl) );
a61af66fc99e Initial load
duke
parents:
diff changeset
432 Node *add2 = new (C, 4) AddPNode( n->in(1), add1, n->in(2)->in(3) );
a61af66fc99e Initial load
duke
parents:
diff changeset
433 register_new_node( add2, n_ctrl );
1621
6027dddc26c6 6677629: PhaseIterGVN::subsume_node() should call hash_delete() and add_users_to_worklist()
kvn
parents: 1552
diff changeset
434 _igvn.replace_node( n, add2 );
0
a61af66fc99e Initial load
duke
parents:
diff changeset
435 return add2;
a61af66fc99e Initial load
duke
parents:
diff changeset
436 }
a61af66fc99e Initial load
duke
parents:
diff changeset
437 }
a61af66fc99e Initial load
duke
parents:
diff changeset
438 }
a61af66fc99e Initial load
duke
parents:
diff changeset
439
a61af66fc99e Initial load
duke
parents:
diff changeset
440 // Replace (I1 +p (I2 + V)) with ((I1 +p I2) +p V)
a61af66fc99e Initial load
duke
parents:
diff changeset
441 if( n2_loop != n_loop && n3_loop == n_loop ) {
a61af66fc99e Initial load
duke
parents:
diff changeset
442 if( n->in(3)->Opcode() == Op_AddI ) {
a61af66fc99e Initial load
duke
parents:
diff changeset
443 Node *V = n->in(3)->in(1);
a61af66fc99e Initial load
duke
parents:
diff changeset
444 Node *I = n->in(3)->in(2);
a61af66fc99e Initial load
duke
parents:
diff changeset
445 if( is_member(n_loop,get_ctrl(V)) ) {
a61af66fc99e Initial load
duke
parents:
diff changeset
446 } else {
a61af66fc99e Initial load
duke
parents:
diff changeset
447 Node *tmp = V; V = I; I = tmp;
a61af66fc99e Initial load
duke
parents:
diff changeset
448 }
a61af66fc99e Initial load
duke
parents:
diff changeset
449 if( !is_member(n_loop,get_ctrl(I)) ) {
a61af66fc99e Initial load
duke
parents:
diff changeset
450 Node *add1 = new (C, 4) AddPNode( n->in(1), n->in(2), I );
a61af66fc99e Initial load
duke
parents:
diff changeset
451 // Stuff new AddP in the loop preheader
a61af66fc99e Initial load
duke
parents:
diff changeset
452 register_new_node( add1, n_loop->_head->in(LoopNode::EntryControl) );
a61af66fc99e Initial load
duke
parents:
diff changeset
453 Node *add2 = new (C, 4) AddPNode( n->in(1), add1, V );
a61af66fc99e Initial load
duke
parents:
diff changeset
454 register_new_node( add2, n_ctrl );
1621
6027dddc26c6 6677629: PhaseIterGVN::subsume_node() should call hash_delete() and add_users_to_worklist()
kvn
parents: 1552
diff changeset
455 _igvn.replace_node( n, add2 );
0
a61af66fc99e Initial load
duke
parents:
diff changeset
456 return add2;
a61af66fc99e Initial load
duke
parents:
diff changeset
457 }
a61af66fc99e Initial load
duke
parents:
diff changeset
458 }
a61af66fc99e Initial load
duke
parents:
diff changeset
459 }
a61af66fc99e Initial load
duke
parents:
diff changeset
460 }
a61af66fc99e Initial load
duke
parents:
diff changeset
461
a61af66fc99e Initial load
duke
parents:
diff changeset
462 return NULL;
a61af66fc99e Initial load
duke
parents:
diff changeset
463 }
a61af66fc99e Initial load
duke
parents:
diff changeset
464
a61af66fc99e Initial load
duke
parents:
diff changeset
465 //------------------------------conditional_move-------------------------------
a61af66fc99e Initial load
duke
parents:
diff changeset
466 // Attempt to replace a Phi with a conditional move. We have some pretty
a61af66fc99e Initial load
duke
parents:
diff changeset
467 // strict profitability requirements. All Phis at the merge point must
a61af66fc99e Initial load
duke
parents:
diff changeset
468 // be converted, so we can remove the control flow. We need to limit the
a61af66fc99e Initial load
duke
parents:
diff changeset
469 // number of c-moves to a small handful. All code that was in the side-arms
a61af66fc99e Initial load
duke
parents:
diff changeset
470 // of the CFG diamond is now speculatively executed. This code has to be
a61af66fc99e Initial load
duke
parents:
diff changeset
471 // "cheap enough". We are pretty much limited to CFG diamonds that merge
a61af66fc99e Initial load
duke
parents:
diff changeset
472 // 1 or 2 items with a total of 1 or 2 ops executed speculatively.
a61af66fc99e Initial load
duke
parents:
diff changeset
473 Node *PhaseIdealLoop::conditional_move( Node *region ) {
a61af66fc99e Initial load
duke
parents:
diff changeset
474
a61af66fc99e Initial load
duke
parents:
diff changeset
475 assert( region->is_Region(), "sanity check" );
a61af66fc99e Initial load
duke
parents:
diff changeset
476 if( region->req() != 3 ) return NULL;
a61af66fc99e Initial load
duke
parents:
diff changeset
477
a61af66fc99e Initial load
duke
parents:
diff changeset
478 // Check for CFG diamond
a61af66fc99e Initial load
duke
parents:
diff changeset
479 Node *lp = region->in(1);
a61af66fc99e Initial load
duke
parents:
diff changeset
480 Node *rp = region->in(2);
a61af66fc99e Initial load
duke
parents:
diff changeset
481 if( !lp || !rp ) return NULL;
a61af66fc99e Initial load
duke
parents:
diff changeset
482 Node *lp_c = lp->in(0);
a61af66fc99e Initial load
duke
parents:
diff changeset
483 if( lp_c == NULL || lp_c != rp->in(0) || !lp_c->is_If() ) return NULL;
a61af66fc99e Initial load
duke
parents:
diff changeset
484 IfNode *iff = lp_c->as_If();
a61af66fc99e Initial load
duke
parents:
diff changeset
485
a61af66fc99e Initial load
duke
parents:
diff changeset
486 // Check for highly predictable branch. No point in CMOV'ing if
a61af66fc99e Initial load
duke
parents:
diff changeset
487 // we are going to predict accurately all the time.
a61af66fc99e Initial load
duke
parents:
diff changeset
488 // %%% This hides patterns produced by utility methods like Math.min.
a61af66fc99e Initial load
duke
parents:
diff changeset
489 if( iff->_prob < PROB_UNLIKELY_MAG(3) ||
a61af66fc99e Initial load
duke
parents:
diff changeset
490 iff->_prob > PROB_LIKELY_MAG(3) )
a61af66fc99e Initial load
duke
parents:
diff changeset
491 return NULL;
a61af66fc99e Initial load
duke
parents:
diff changeset
492
a61af66fc99e Initial load
duke
parents:
diff changeset
493 // Check for ops pinned in an arm of the diamond.
a61af66fc99e Initial load
duke
parents:
diff changeset
494 // Can't remove the control flow in this case
a61af66fc99e Initial load
duke
parents:
diff changeset
495 if( lp->outcnt() > 1 ) return NULL;
a61af66fc99e Initial load
duke
parents:
diff changeset
496 if( rp->outcnt() > 1 ) return NULL;
a61af66fc99e Initial load
duke
parents:
diff changeset
497
a61af66fc99e Initial load
duke
parents:
diff changeset
498 // Check profitability
a61af66fc99e Initial load
duke
parents:
diff changeset
499 int cost = 0;
35
e2ae28d2ce91 6667588: Don't generate duplicated CMP for float/double values
kvn
parents: 0
diff changeset
500 int phis = 0;
0
a61af66fc99e Initial load
duke
parents:
diff changeset
501 for (DUIterator_Fast imax, i = region->fast_outs(imax); i < imax; i++) {
a61af66fc99e Initial load
duke
parents:
diff changeset
502 Node *out = region->fast_out(i);
a61af66fc99e Initial load
duke
parents:
diff changeset
503 if( !out->is_Phi() ) continue; // Ignore other control edges, etc
35
e2ae28d2ce91 6667588: Don't generate duplicated CMP for float/double values
kvn
parents: 0
diff changeset
504 phis++;
0
a61af66fc99e Initial load
duke
parents:
diff changeset
505 PhiNode* phi = out->as_Phi();
a61af66fc99e Initial load
duke
parents:
diff changeset
506 switch (phi->type()->basic_type()) {
a61af66fc99e Initial load
duke
parents:
diff changeset
507 case T_LONG:
a61af66fc99e Initial load
duke
parents:
diff changeset
508 cost++; // Probably encodes as 2 CMOV's
a61af66fc99e Initial load
duke
parents:
diff changeset
509 case T_INT: // These all CMOV fine
a61af66fc99e Initial load
duke
parents:
diff changeset
510 case T_FLOAT:
a61af66fc99e Initial load
duke
parents:
diff changeset
511 case T_DOUBLE:
a61af66fc99e Initial load
duke
parents:
diff changeset
512 case T_ADDRESS: // (RawPtr)
a61af66fc99e Initial load
duke
parents:
diff changeset
513 cost++;
a61af66fc99e Initial load
duke
parents:
diff changeset
514 break;
293
c3e045194476 6731641: assert(m->adr_type() == mach->adr_type(),"matcher should not change adr type")
kvn
parents: 253
diff changeset
515 case T_NARROWOOP: // Fall through
0
a61af66fc99e Initial load
duke
parents:
diff changeset
516 case T_OBJECT: { // Base oops are OK, but not derived oops
293
c3e045194476 6731641: assert(m->adr_type() == mach->adr_type(),"matcher should not change adr type")
kvn
parents: 253
diff changeset
517 const TypeOopPtr *tp = phi->type()->make_ptr()->isa_oopptr();
0
a61af66fc99e Initial load
duke
parents:
diff changeset
518 // Derived pointers are Bad (tm): what's the Base (for GC purposes) of a
a61af66fc99e Initial load
duke
parents:
diff changeset
519 // CMOVE'd derived pointer? It's a CMOVE'd derived base. Thus
a61af66fc99e Initial load
duke
parents:
diff changeset
520 // CMOVE'ing a derived pointer requires we also CMOVE the base. If we
a61af66fc99e Initial load
duke
parents:
diff changeset
521 // have a Phi for the base here that we convert to a CMOVE all is well
a61af66fc99e Initial load
duke
parents:
diff changeset
522 // and good. But if the base is dead, we'll not make a CMOVE. Later
a61af66fc99e Initial load
duke
parents:
diff changeset
523 // the allocator will have to produce a base by creating a CMOVE of the
a61af66fc99e Initial load
duke
parents:
diff changeset
524 // relevant bases. This puts the allocator in the business of
a61af66fc99e Initial load
duke
parents:
diff changeset
525 // manufacturing expensive instructions, generally a bad plan.
a61af66fc99e Initial load
duke
parents:
diff changeset
526 // Just Say No to Conditionally-Moved Derived Pointers.
a61af66fc99e Initial load
duke
parents:
diff changeset
527 if( tp && tp->offset() != 0 )
a61af66fc99e Initial load
duke
parents:
diff changeset
528 return NULL;
a61af66fc99e Initial load
duke
parents:
diff changeset
529 cost++;
a61af66fc99e Initial load
duke
parents:
diff changeset
530 break;
a61af66fc99e Initial load
duke
parents:
diff changeset
531 }
a61af66fc99e Initial load
duke
parents:
diff changeset
532 default:
a61af66fc99e Initial load
duke
parents:
diff changeset
533 return NULL; // In particular, can't do memory or I/O
a61af66fc99e Initial load
duke
parents:
diff changeset
534 }
a61af66fc99e Initial load
duke
parents:
diff changeset
535 // Add in cost any speculative ops
a61af66fc99e Initial load
duke
parents:
diff changeset
536 for( uint j = 1; j < region->req(); j++ ) {
a61af66fc99e Initial load
duke
parents:
diff changeset
537 Node *proj = region->in(j);
a61af66fc99e Initial load
duke
parents:
diff changeset
538 Node *inp = phi->in(j);
a61af66fc99e Initial load
duke
parents:
diff changeset
539 if (get_ctrl(inp) == proj) { // Found local op
a61af66fc99e Initial load
duke
parents:
diff changeset
540 cost++;
a61af66fc99e Initial load
duke
parents:
diff changeset
541 // Check for a chain of dependent ops; these will all become
a61af66fc99e Initial load
duke
parents:
diff changeset
542 // speculative in a CMOV.
a61af66fc99e Initial load
duke
parents:
diff changeset
543 for( uint k = 1; k < inp->req(); k++ )
a61af66fc99e Initial load
duke
parents:
diff changeset
544 if (get_ctrl(inp->in(k)) == proj)
a61af66fc99e Initial load
duke
parents:
diff changeset
545 return NULL; // Too much speculative goo
a61af66fc99e Initial load
duke
parents:
diff changeset
546 }
a61af66fc99e Initial load
duke
parents:
diff changeset
547 }
293
c3e045194476 6731641: assert(m->adr_type() == mach->adr_type(),"matcher should not change adr type")
kvn
parents: 253
diff changeset
548 // See if the Phi is used by a Cmp or Narrow oop Decode/Encode.
c3e045194476 6731641: assert(m->adr_type() == mach->adr_type(),"matcher should not change adr type")
kvn
parents: 253
diff changeset
549 // This will likely Split-If, a higher-payoff operation.
0
a61af66fc99e Initial load
duke
parents:
diff changeset
550 for (DUIterator_Fast kmax, k = phi->fast_outs(kmax); k < kmax; k++) {
a61af66fc99e Initial load
duke
parents:
diff changeset
551 Node* use = phi->fast_out(k);
293
c3e045194476 6731641: assert(m->adr_type() == mach->adr_type(),"matcher should not change adr type")
kvn
parents: 253
diff changeset
552 if( use->is_Cmp() || use->is_DecodeN() || use->is_EncodeP() )
0
a61af66fc99e Initial load
duke
parents:
diff changeset
553 return NULL;
a61af66fc99e Initial load
duke
parents:
diff changeset
554 }
a61af66fc99e Initial load
duke
parents:
diff changeset
555 }
a61af66fc99e Initial load
duke
parents:
diff changeset
556 if( cost >= ConditionalMoveLimit ) return NULL; // Too much goo
35
e2ae28d2ce91 6667588: Don't generate duplicated CMP for float/double values
kvn
parents: 0
diff changeset
557 Node* bol = iff->in(1);
e2ae28d2ce91 6667588: Don't generate duplicated CMP for float/double values
kvn
parents: 0
diff changeset
558 assert( bol->Opcode() == Op_Bool, "" );
e2ae28d2ce91 6667588: Don't generate duplicated CMP for float/double values
kvn
parents: 0
diff changeset
559 int cmp_op = bol->in(1)->Opcode();
e2ae28d2ce91 6667588: Don't generate duplicated CMP for float/double values
kvn
parents: 0
diff changeset
560 // It is expensive to generate flags from a float compare.
e2ae28d2ce91 6667588: Don't generate duplicated CMP for float/double values
kvn
parents: 0
diff changeset
561 // Avoid duplicated float compare.
e2ae28d2ce91 6667588: Don't generate duplicated CMP for float/double values
kvn
parents: 0
diff changeset
562 if( phis > 1 && (cmp_op == Op_CmpF || cmp_op == Op_CmpD)) return NULL;
0
a61af66fc99e Initial load
duke
parents:
diff changeset
563
a61af66fc99e Initial load
duke
parents:
diff changeset
564 // --------------
a61af66fc99e Initial load
duke
parents:
diff changeset
565 // Now replace all Phis with CMOV's
a61af66fc99e Initial load
duke
parents:
diff changeset
566 Node *cmov_ctrl = iff->in(0);
a61af66fc99e Initial load
duke
parents:
diff changeset
567 uint flip = (lp->Opcode() == Op_IfTrue);
a61af66fc99e Initial load
duke
parents:
diff changeset
568 while( 1 ) {
a61af66fc99e Initial load
duke
parents:
diff changeset
569 PhiNode* phi = NULL;
a61af66fc99e Initial load
duke
parents:
diff changeset
570 for (DUIterator_Fast imax, i = region->fast_outs(imax); i < imax; i++) {
a61af66fc99e Initial load
duke
parents:
diff changeset
571 Node *out = region->fast_out(i);
a61af66fc99e Initial load
duke
parents:
diff changeset
572 if (out->is_Phi()) {
a61af66fc99e Initial load
duke
parents:
diff changeset
573 phi = out->as_Phi();
a61af66fc99e Initial load
duke
parents:
diff changeset
574 break;
a61af66fc99e Initial load
duke
parents:
diff changeset
575 }
a61af66fc99e Initial load
duke
parents:
diff changeset
576 }
a61af66fc99e Initial load
duke
parents:
diff changeset
577 if (phi == NULL) break;
a61af66fc99e Initial load
duke
parents:
diff changeset
578 #ifndef PRODUCT
a61af66fc99e Initial load
duke
parents:
diff changeset
579 if( PrintOpto && VerifyLoopOptimizations ) tty->print_cr("CMOV");
a61af66fc99e Initial load
duke
parents:
diff changeset
580 #endif
a61af66fc99e Initial load
duke
parents:
diff changeset
581 // Move speculative ops
a61af66fc99e Initial load
duke
parents:
diff changeset
582 for( uint j = 1; j < region->req(); j++ ) {
a61af66fc99e Initial load
duke
parents:
diff changeset
583 Node *proj = region->in(j);
a61af66fc99e Initial load
duke
parents:
diff changeset
584 Node *inp = phi->in(j);
a61af66fc99e Initial load
duke
parents:
diff changeset
585 if (get_ctrl(inp) == proj) { // Found local op
a61af66fc99e Initial load
duke
parents:
diff changeset
586 #ifndef PRODUCT
a61af66fc99e Initial load
duke
parents:
diff changeset
587 if( PrintOpto && VerifyLoopOptimizations ) {
a61af66fc99e Initial load
duke
parents:
diff changeset
588 tty->print(" speculate: ");
a61af66fc99e Initial load
duke
parents:
diff changeset
589 inp->dump();
a61af66fc99e Initial load
duke
parents:
diff changeset
590 }
a61af66fc99e Initial load
duke
parents:
diff changeset
591 #endif
a61af66fc99e Initial load
duke
parents:
diff changeset
592 set_ctrl(inp, cmov_ctrl);
a61af66fc99e Initial load
duke
parents:
diff changeset
593 }
a61af66fc99e Initial load
duke
parents:
diff changeset
594 }
a61af66fc99e Initial load
duke
parents:
diff changeset
595 Node *cmov = CMoveNode::make( C, cmov_ctrl, iff->in(1), phi->in(1+flip), phi->in(2-flip), _igvn.type(phi) );
a61af66fc99e Initial load
duke
parents:
diff changeset
596 register_new_node( cmov, cmov_ctrl );
1621
6027dddc26c6 6677629: PhaseIterGVN::subsume_node() should call hash_delete() and add_users_to_worklist()
kvn
parents: 1552
diff changeset
597 _igvn.replace_node( phi, cmov );
0
a61af66fc99e Initial load
duke
parents:
diff changeset
598 #ifndef PRODUCT
a61af66fc99e Initial load
duke
parents:
diff changeset
599 if( VerifyLoopOptimizations ) verify();
a61af66fc99e Initial load
duke
parents:
diff changeset
600 #endif
a61af66fc99e Initial load
duke
parents:
diff changeset
601 }
a61af66fc99e Initial load
duke
parents:
diff changeset
602
a61af66fc99e Initial load
duke
parents:
diff changeset
603 // The useless CFG diamond will fold up later; see the optimization in
a61af66fc99e Initial load
duke
parents:
diff changeset
604 // RegionNode::Ideal.
a61af66fc99e Initial load
duke
parents:
diff changeset
605 _igvn._worklist.push(region);
a61af66fc99e Initial load
duke
parents:
diff changeset
606
a61af66fc99e Initial load
duke
parents:
diff changeset
607 return iff->in(1);
a61af66fc99e Initial load
duke
parents:
diff changeset
608 }
a61af66fc99e Initial load
duke
parents:
diff changeset
609
a61af66fc99e Initial load
duke
parents:
diff changeset
610 //------------------------------split_if_with_blocks_pre-----------------------
a61af66fc99e Initial load
duke
parents:
diff changeset
611 // Do the real work in a non-recursive function. Data nodes want to be
a61af66fc99e Initial load
duke
parents:
diff changeset
612 // cloned in the pre-order so they can feed each other nicely.
a61af66fc99e Initial load
duke
parents:
diff changeset
613 Node *PhaseIdealLoop::split_if_with_blocks_pre( Node *n ) {
a61af66fc99e Initial load
duke
parents:
diff changeset
614 // Cloning these guys is unlikely to win
a61af66fc99e Initial load
duke
parents:
diff changeset
615 int n_op = n->Opcode();
a61af66fc99e Initial load
duke
parents:
diff changeset
616 if( n_op == Op_MergeMem ) return n;
a61af66fc99e Initial load
duke
parents:
diff changeset
617 if( n->is_Proj() ) return n;
a61af66fc99e Initial load
duke
parents:
diff changeset
618 // Do not clone-up CmpFXXX variations, as these are always
a61af66fc99e Initial load
duke
parents:
diff changeset
619 // followed by a CmpI
a61af66fc99e Initial load
duke
parents:
diff changeset
620 if( n->is_Cmp() ) return n;
a61af66fc99e Initial load
duke
parents:
diff changeset
621 // Attempt to use a conditional move instead of a phi/branch
a61af66fc99e Initial load
duke
parents:
diff changeset
622 if( ConditionalMoveLimit > 0 && n_op == Op_Region ) {
a61af66fc99e Initial load
duke
parents:
diff changeset
623 Node *cmov = conditional_move( n );
a61af66fc99e Initial load
duke
parents:
diff changeset
624 if( cmov ) return cmov;
a61af66fc99e Initial load
duke
parents:
diff changeset
625 }
253
b0fe4deeb9fb 6726999: nsk/stress/jck12a/jck12a010 assert(n != null,"Bad immediate dominator info.")
kvn
parents: 251
diff changeset
626 if( n->is_CFG() || n->is_LoadStore() )
b0fe4deeb9fb 6726999: nsk/stress/jck12a/jck12a010 assert(n != null,"Bad immediate dominator info.")
kvn
parents: 251
diff changeset
627 return n;
0
a61af66fc99e Initial load
duke
parents:
diff changeset
628 if( n_op == Op_Opaque1 || // Opaque nodes cannot be mod'd
a61af66fc99e Initial load
duke
parents:
diff changeset
629 n_op == Op_Opaque2 ) {
a61af66fc99e Initial load
duke
parents:
diff changeset
630 if( !C->major_progress() ) // If chance of no more loop opts...
a61af66fc99e Initial load
duke
parents:
diff changeset
631 _igvn._worklist.push(n); // maybe we'll remove them
a61af66fc99e Initial load
duke
parents:
diff changeset
632 return n;
a61af66fc99e Initial load
duke
parents:
diff changeset
633 }
a61af66fc99e Initial load
duke
parents:
diff changeset
634
a61af66fc99e Initial load
duke
parents:
diff changeset
635 if( n->is_Con() ) return n; // No cloning for Con nodes
a61af66fc99e Initial load
duke
parents:
diff changeset
636
a61af66fc99e Initial load
duke
parents:
diff changeset
637 Node *n_ctrl = get_ctrl(n);
a61af66fc99e Initial load
duke
parents:
diff changeset
638 if( !n_ctrl ) return n; // Dead node
a61af66fc99e Initial load
duke
parents:
diff changeset
639
a61af66fc99e Initial load
duke
parents:
diff changeset
640 // Attempt to remix address expressions for loop invariants
a61af66fc99e Initial load
duke
parents:
diff changeset
641 Node *m = remix_address_expressions( n );
a61af66fc99e Initial load
duke
parents:
diff changeset
642 if( m ) return m;
a61af66fc99e Initial load
duke
parents:
diff changeset
643
a61af66fc99e Initial load
duke
parents:
diff changeset
644 // Determine if the Node has inputs from some local Phi.
a61af66fc99e Initial load
duke
parents:
diff changeset
645 // Returns the block to clone thru.
a61af66fc99e Initial load
duke
parents:
diff changeset
646 Node *n_blk = has_local_phi_input( n );
a61af66fc99e Initial load
duke
parents:
diff changeset
647 if( !n_blk ) return n;
a61af66fc99e Initial load
duke
parents:
diff changeset
648 // Do not clone the trip counter through on a CountedLoop
a61af66fc99e Initial load
duke
parents:
diff changeset
649 // (messes up the canonical shape).
a61af66fc99e Initial load
duke
parents:
diff changeset
650 if( n_blk->is_CountedLoop() && n->Opcode() == Op_AddI ) return n;
a61af66fc99e Initial load
duke
parents:
diff changeset
651
a61af66fc99e Initial load
duke
parents:
diff changeset
652 // Check for having no control input; not pinned. Allow
a61af66fc99e Initial load
duke
parents:
diff changeset
653 // dominating control.
a61af66fc99e Initial load
duke
parents:
diff changeset
654 if( n->in(0) ) {
a61af66fc99e Initial load
duke
parents:
diff changeset
655 Node *dom = idom(n_blk);
a61af66fc99e Initial load
duke
parents:
diff changeset
656 if( dom_lca( n->in(0), dom ) != n->in(0) )
a61af66fc99e Initial load
duke
parents:
diff changeset
657 return n;
a61af66fc99e Initial load
duke
parents:
diff changeset
658 }
a61af66fc99e Initial load
duke
parents:
diff changeset
659 // Policy: when is it profitable. You must get more wins than
a61af66fc99e Initial load
duke
parents:
diff changeset
660 // policy before it is considered profitable. Policy is usually 0,
a61af66fc99e Initial load
duke
parents:
diff changeset
661 // so 1 win is considered profitable. Big merges will require big
a61af66fc99e Initial load
duke
parents:
diff changeset
662 // cloning, so get a larger policy.
a61af66fc99e Initial load
duke
parents:
diff changeset
663 int policy = n_blk->req() >> 2;
a61af66fc99e Initial load
duke
parents:
diff changeset
664
a61af66fc99e Initial load
duke
parents:
diff changeset
665 // If the loop is a candidate for range check elimination,
a61af66fc99e Initial load
duke
parents:
diff changeset
666 // delay splitting through it's phi until a later loop optimization
a61af66fc99e Initial load
duke
parents:
diff changeset
667 if (n_blk->is_CountedLoop()) {
a61af66fc99e Initial load
duke
parents:
diff changeset
668 IdealLoopTree *lp = get_loop(n_blk);
a61af66fc99e Initial load
duke
parents:
diff changeset
669 if (lp && lp->_rce_candidate) {
a61af66fc99e Initial load
duke
parents:
diff changeset
670 return n;
a61af66fc99e Initial load
duke
parents:
diff changeset
671 }
a61af66fc99e Initial load
duke
parents:
diff changeset
672 }
a61af66fc99e Initial load
duke
parents:
diff changeset
673
a61af66fc99e Initial load
duke
parents:
diff changeset
674 // Use same limit as split_if_with_blocks_post
a61af66fc99e Initial load
duke
parents:
diff changeset
675 if( C->unique() > 35000 ) return n; // Method too big
a61af66fc99e Initial load
duke
parents:
diff changeset
676
a61af66fc99e Initial load
duke
parents:
diff changeset
677 // Split 'n' through the merge point if it is profitable
a61af66fc99e Initial load
duke
parents:
diff changeset
678 Node *phi = split_thru_phi( n, n_blk, policy );
a61af66fc99e Initial load
duke
parents:
diff changeset
679 if( !phi ) return n;
a61af66fc99e Initial load
duke
parents:
diff changeset
680
a61af66fc99e Initial load
duke
parents:
diff changeset
681 // Found a Phi to split thru!
a61af66fc99e Initial load
duke
parents:
diff changeset
682 // Replace 'n' with the new phi
1621
6027dddc26c6 6677629: PhaseIterGVN::subsume_node() should call hash_delete() and add_users_to_worklist()
kvn
parents: 1552
diff changeset
683 _igvn.replace_node( n, phi );
0
a61af66fc99e Initial load
duke
parents:
diff changeset
684 // Moved a load around the loop, 'en-registering' something.
a61af66fc99e Initial load
duke
parents:
diff changeset
685 if( n_blk->Opcode() == Op_Loop && n->is_Load() &&
a61af66fc99e Initial load
duke
parents:
diff changeset
686 !phi->in(LoopNode::LoopBackControl)->is_Load() )
a61af66fc99e Initial load
duke
parents:
diff changeset
687 C->set_major_progress();
a61af66fc99e Initial load
duke
parents:
diff changeset
688
a61af66fc99e Initial load
duke
parents:
diff changeset
689 return phi;
a61af66fc99e Initial load
duke
parents:
diff changeset
690 }
a61af66fc99e Initial load
duke
parents:
diff changeset
691
a61af66fc99e Initial load
duke
parents:
diff changeset
692 static bool merge_point_too_heavy(Compile* C, Node* region) {
a61af66fc99e Initial load
duke
parents:
diff changeset
693 // Bail out if the region and its phis have too many users.
a61af66fc99e Initial load
duke
parents:
diff changeset
694 int weight = 0;
a61af66fc99e Initial load
duke
parents:
diff changeset
695 for (DUIterator_Fast imax, i = region->fast_outs(imax); i < imax; i++) {
a61af66fc99e Initial load
duke
parents:
diff changeset
696 weight += region->fast_out(i)->outcnt();
a61af66fc99e Initial load
duke
parents:
diff changeset
697 }
a61af66fc99e Initial load
duke
parents:
diff changeset
698 int nodes_left = MaxNodeLimit - C->unique();
a61af66fc99e Initial load
duke
parents:
diff changeset
699 if (weight * 8 > nodes_left) {
a61af66fc99e Initial load
duke
parents:
diff changeset
700 #ifndef PRODUCT
a61af66fc99e Initial load
duke
parents:
diff changeset
701 if (PrintOpto)
a61af66fc99e Initial load
duke
parents:
diff changeset
702 tty->print_cr("*** Split-if bails out: %d nodes, region weight %d", C->unique(), weight);
a61af66fc99e Initial load
duke
parents:
diff changeset
703 #endif
a61af66fc99e Initial load
duke
parents:
diff changeset
704 return true;
a61af66fc99e Initial load
duke
parents:
diff changeset
705 } else {
a61af66fc99e Initial load
duke
parents:
diff changeset
706 return false;
a61af66fc99e Initial load
duke
parents:
diff changeset
707 }
a61af66fc99e Initial load
duke
parents:
diff changeset
708 }
a61af66fc99e Initial load
duke
parents:
diff changeset
709
a61af66fc99e Initial load
duke
parents:
diff changeset
710 static bool merge_point_safe(Node* region) {
a61af66fc99e Initial load
duke
parents:
diff changeset
711 // 4799512: Stop split_if_with_blocks from splitting a block with a ConvI2LNode
a61af66fc99e Initial load
duke
parents:
diff changeset
712 // having a PhiNode input. This sidesteps the dangerous case where the split
a61af66fc99e Initial load
duke
parents:
diff changeset
713 // ConvI2LNode may become TOP if the input Value() does not
a61af66fc99e Initial load
duke
parents:
diff changeset
714 // overlap the ConvI2L range, leaving a node which may not dominate its
a61af66fc99e Initial load
duke
parents:
diff changeset
715 // uses.
a61af66fc99e Initial load
duke
parents:
diff changeset
716 // A better fix for this problem can be found in the BugTraq entry, but
a61af66fc99e Initial load
duke
parents:
diff changeset
717 // expediency for Mantis demands this hack.
834
0f2d888530e7 6855164: SIGSEGV during compilation of method involving loop over CharSequence.
cfang
parents: 605
diff changeset
718 // 6855164: If the merge point has a FastLockNode with a PhiNode input, we stop
0f2d888530e7 6855164: SIGSEGV during compilation of method involving loop over CharSequence.
cfang
parents: 605
diff changeset
719 // split_if_with_blocks from splitting a block because we could not move around
0f2d888530e7 6855164: SIGSEGV during compilation of method involving loop over CharSequence.
cfang
parents: 605
diff changeset
720 // the FastLockNode.
0
a61af66fc99e Initial load
duke
parents:
diff changeset
721 for (DUIterator_Fast imax, i = region->fast_outs(imax); i < imax; i++) {
a61af66fc99e Initial load
duke
parents:
diff changeset
722 Node* n = region->fast_out(i);
a61af66fc99e Initial load
duke
parents:
diff changeset
723 if (n->is_Phi()) {
a61af66fc99e Initial load
duke
parents:
diff changeset
724 for (DUIterator_Fast jmax, j = n->fast_outs(jmax); j < jmax; j++) {
a61af66fc99e Initial load
duke
parents:
diff changeset
725 Node* m = n->fast_out(j);
834
0f2d888530e7 6855164: SIGSEGV during compilation of method involving loop over CharSequence.
cfang
parents: 605
diff changeset
726 if (m->is_FastLock())
0
a61af66fc99e Initial load
duke
parents:
diff changeset
727 return false;
834
0f2d888530e7 6855164: SIGSEGV during compilation of method involving loop over CharSequence.
cfang
parents: 605
diff changeset
728 #ifdef _LP64
0f2d888530e7 6855164: SIGSEGV during compilation of method involving loop over CharSequence.
cfang
parents: 605
diff changeset
729 if (m->Opcode() == Op_ConvI2L)
0f2d888530e7 6855164: SIGSEGV during compilation of method involving loop over CharSequence.
cfang
parents: 605
diff changeset
730 return false;
0f2d888530e7 6855164: SIGSEGV during compilation of method involving loop over CharSequence.
cfang
parents: 605
diff changeset
731 #endif
0
a61af66fc99e Initial load
duke
parents:
diff changeset
732 }
a61af66fc99e Initial load
duke
parents:
diff changeset
733 }
a61af66fc99e Initial load
duke
parents:
diff changeset
734 }
a61af66fc99e Initial load
duke
parents:
diff changeset
735 return true;
a61af66fc99e Initial load
duke
parents:
diff changeset
736 }
a61af66fc99e Initial load
duke
parents:
diff changeset
737
a61af66fc99e Initial load
duke
parents:
diff changeset
738
a61af66fc99e Initial load
duke
parents:
diff changeset
739 //------------------------------place_near_use---------------------------------
a61af66fc99e Initial load
duke
parents:
diff changeset
740 // Place some computation next to use but not inside inner loops.
a61af66fc99e Initial load
duke
parents:
diff changeset
741 // For inner loop uses move it to the preheader area.
a61af66fc99e Initial load
duke
parents:
diff changeset
742 Node *PhaseIdealLoop::place_near_use( Node *useblock ) const {
a61af66fc99e Initial load
duke
parents:
diff changeset
743 IdealLoopTree *u_loop = get_loop( useblock );
a61af66fc99e Initial load
duke
parents:
diff changeset
744 return (u_loop->_irreducible || u_loop->_child)
a61af66fc99e Initial load
duke
parents:
diff changeset
745 ? useblock
a61af66fc99e Initial load
duke
parents:
diff changeset
746 : u_loop->_head->in(LoopNode::EntryControl);
a61af66fc99e Initial load
duke
parents:
diff changeset
747 }
a61af66fc99e Initial load
duke
parents:
diff changeset
748
a61af66fc99e Initial load
duke
parents:
diff changeset
749
a61af66fc99e Initial load
duke
parents:
diff changeset
750 //------------------------------split_if_with_blocks_post----------------------
a61af66fc99e Initial load
duke
parents:
diff changeset
751 // Do the real work in a non-recursive function. CFG hackery wants to be
a61af66fc99e Initial load
duke
parents:
diff changeset
752 // in the post-order, so it can dirty the I-DOM info and not use the dirtied
a61af66fc99e Initial load
duke
parents:
diff changeset
753 // info.
a61af66fc99e Initial load
duke
parents:
diff changeset
754 void PhaseIdealLoop::split_if_with_blocks_post( Node *n ) {
a61af66fc99e Initial load
duke
parents:
diff changeset
755
a61af66fc99e Initial load
duke
parents:
diff changeset
756 // Cloning Cmp through Phi's involves the split-if transform.
a61af66fc99e Initial load
duke
parents:
diff changeset
757 // FastLock is not used by an If
a61af66fc99e Initial load
duke
parents:
diff changeset
758 if( n->is_Cmp() && !n->is_FastLock() ) {
a61af66fc99e Initial load
duke
parents:
diff changeset
759 if( C->unique() > 35000 ) return; // Method too big
a61af66fc99e Initial load
duke
parents:
diff changeset
760
a61af66fc99e Initial load
duke
parents:
diff changeset
761 // Do not do 'split-if' if irreducible loops are present.
a61af66fc99e Initial load
duke
parents:
diff changeset
762 if( _has_irreducible_loops )
a61af66fc99e Initial load
duke
parents:
diff changeset
763 return;
a61af66fc99e Initial load
duke
parents:
diff changeset
764
a61af66fc99e Initial load
duke
parents:
diff changeset
765 Node *n_ctrl = get_ctrl(n);
a61af66fc99e Initial load
duke
parents:
diff changeset
766 // Determine if the Node has inputs from some local Phi.
a61af66fc99e Initial load
duke
parents:
diff changeset
767 // Returns the block to clone thru.
a61af66fc99e Initial load
duke
parents:
diff changeset
768 Node *n_blk = has_local_phi_input( n );
a61af66fc99e Initial load
duke
parents:
diff changeset
769 if( n_blk != n_ctrl ) return;
a61af66fc99e Initial load
duke
parents:
diff changeset
770
a61af66fc99e Initial load
duke
parents:
diff changeset
771 if( merge_point_too_heavy(C, n_ctrl) )
a61af66fc99e Initial load
duke
parents:
diff changeset
772 return;
a61af66fc99e Initial load
duke
parents:
diff changeset
773
a61af66fc99e Initial load
duke
parents:
diff changeset
774 if( n->outcnt() != 1 ) return; // Multiple bool's from 1 compare?
a61af66fc99e Initial load
duke
parents:
diff changeset
775 Node *bol = n->unique_out();
a61af66fc99e Initial load
duke
parents:
diff changeset
776 assert( bol->is_Bool(), "expect a bool here" );
a61af66fc99e Initial load
duke
parents:
diff changeset
777 if( bol->outcnt() != 1 ) return;// Multiple branches from 1 compare?
a61af66fc99e Initial load
duke
parents:
diff changeset
778 Node *iff = bol->unique_out();
a61af66fc99e Initial load
duke
parents:
diff changeset
779
a61af66fc99e Initial load
duke
parents:
diff changeset
780 // Check some safety conditions
a61af66fc99e Initial load
duke
parents:
diff changeset
781 if( iff->is_If() ) { // Classic split-if?
a61af66fc99e Initial load
duke
parents:
diff changeset
782 if( iff->in(0) != n_ctrl ) return; // Compare must be in same blk as if
a61af66fc99e Initial load
duke
parents:
diff changeset
783 } else if (iff->is_CMove()) { // Trying to split-up a CMOVE
a61af66fc99e Initial load
duke
parents:
diff changeset
784 if( get_ctrl(iff->in(2)) == n_ctrl ||
a61af66fc99e Initial load
duke
parents:
diff changeset
785 get_ctrl(iff->in(3)) == n_ctrl )
a61af66fc99e Initial load
duke
parents:
diff changeset
786 return; // Inputs not yet split-up
a61af66fc99e Initial load
duke
parents:
diff changeset
787 if ( get_loop(n_ctrl) != get_loop(get_ctrl(iff)) ) {
a61af66fc99e Initial load
duke
parents:
diff changeset
788 return; // Loop-invar test gates loop-varying CMOVE
a61af66fc99e Initial load
duke
parents:
diff changeset
789 }
a61af66fc99e Initial load
duke
parents:
diff changeset
790 } else {
a61af66fc99e Initial load
duke
parents:
diff changeset
791 return; // some other kind of node, such as an Allocate
a61af66fc99e Initial load
duke
parents:
diff changeset
792 }
a61af66fc99e Initial load
duke
parents:
diff changeset
793
a61af66fc99e Initial load
duke
parents:
diff changeset
794 // Do not do 'split-if' if some paths are dead. First do dead code
a61af66fc99e Initial load
duke
parents:
diff changeset
795 // elimination and then see if its still profitable.
a61af66fc99e Initial load
duke
parents:
diff changeset
796 for( uint i = 1; i < n_ctrl->req(); i++ )
a61af66fc99e Initial load
duke
parents:
diff changeset
797 if( n_ctrl->in(i) == C->top() )
a61af66fc99e Initial load
duke
parents:
diff changeset
798 return;
a61af66fc99e Initial load
duke
parents:
diff changeset
799
a61af66fc99e Initial load
duke
parents:
diff changeset
800 // When is split-if profitable? Every 'win' on means some control flow
a61af66fc99e Initial load
duke
parents:
diff changeset
801 // goes dead, so it's almost always a win.
a61af66fc99e Initial load
duke
parents:
diff changeset
802 int policy = 0;
a61af66fc99e Initial load
duke
parents:
diff changeset
803 // If trying to do a 'Split-If' at the loop head, it is only
a61af66fc99e Initial load
duke
parents:
diff changeset
804 // profitable if the cmp folds up on BOTH paths. Otherwise we
a61af66fc99e Initial load
duke
parents:
diff changeset
805 // risk peeling a loop forever.
a61af66fc99e Initial load
duke
parents:
diff changeset
806
a61af66fc99e Initial load
duke
parents:
diff changeset
807 // CNC - Disabled for now. Requires careful handling of loop
a61af66fc99e Initial load
duke
parents:
diff changeset
808 // body selection for the cloned code. Also, make sure we check
a61af66fc99e Initial load
duke
parents:
diff changeset
809 // for any input path not being in the same loop as n_ctrl. For
a61af66fc99e Initial load
duke
parents:
diff changeset
810 // irreducible loops we cannot check for 'n_ctrl->is_Loop()'
a61af66fc99e Initial load
duke
parents:
diff changeset
811 // because the alternative loop entry points won't be converted
a61af66fc99e Initial load
duke
parents:
diff changeset
812 // into LoopNodes.
a61af66fc99e Initial load
duke
parents:
diff changeset
813 IdealLoopTree *n_loop = get_loop(n_ctrl);
a61af66fc99e Initial load
duke
parents:
diff changeset
814 for( uint j = 1; j < n_ctrl->req(); j++ )
a61af66fc99e Initial load
duke
parents:
diff changeset
815 if( get_loop(n_ctrl->in(j)) != n_loop )
a61af66fc99e Initial load
duke
parents:
diff changeset
816 return;
a61af66fc99e Initial load
duke
parents:
diff changeset
817
a61af66fc99e Initial load
duke
parents:
diff changeset
818 // Check for safety of the merge point.
a61af66fc99e Initial load
duke
parents:
diff changeset
819 if( !merge_point_safe(n_ctrl) ) {
a61af66fc99e Initial load
duke
parents:
diff changeset
820 return;
a61af66fc99e Initial load
duke
parents:
diff changeset
821 }
a61af66fc99e Initial load
duke
parents:
diff changeset
822
a61af66fc99e Initial load
duke
parents:
diff changeset
823 // Split compare 'n' through the merge point if it is profitable
a61af66fc99e Initial load
duke
parents:
diff changeset
824 Node *phi = split_thru_phi( n, n_ctrl, policy );
a61af66fc99e Initial load
duke
parents:
diff changeset
825 if( !phi ) return;
a61af66fc99e Initial load
duke
parents:
diff changeset
826
a61af66fc99e Initial load
duke
parents:
diff changeset
827 // Found a Phi to split thru!
a61af66fc99e Initial load
duke
parents:
diff changeset
828 // Replace 'n' with the new phi
1621
6027dddc26c6 6677629: PhaseIterGVN::subsume_node() should call hash_delete() and add_users_to_worklist()
kvn
parents: 1552
diff changeset
829 _igvn.replace_node( n, phi );
0
a61af66fc99e Initial load
duke
parents:
diff changeset
830
a61af66fc99e Initial load
duke
parents:
diff changeset
831 // Now split the bool up thru the phi
a61af66fc99e Initial load
duke
parents:
diff changeset
832 Node *bolphi = split_thru_phi( bol, n_ctrl, -1 );
1621
6027dddc26c6 6677629: PhaseIterGVN::subsume_node() should call hash_delete() and add_users_to_worklist()
kvn
parents: 1552
diff changeset
833 _igvn.replace_node( bol, bolphi );
0
a61af66fc99e Initial load
duke
parents:
diff changeset
834 assert( iff->in(1) == bolphi, "" );
a61af66fc99e Initial load
duke
parents:
diff changeset
835 if( bolphi->Value(&_igvn)->singleton() )
a61af66fc99e Initial load
duke
parents:
diff changeset
836 return;
a61af66fc99e Initial load
duke
parents:
diff changeset
837
a61af66fc99e Initial load
duke
parents:
diff changeset
838 // Conditional-move? Must split up now
a61af66fc99e Initial load
duke
parents:
diff changeset
839 if( !iff->is_If() ) {
a61af66fc99e Initial load
duke
parents:
diff changeset
840 Node *cmovphi = split_thru_phi( iff, n_ctrl, -1 );
1621
6027dddc26c6 6677629: PhaseIterGVN::subsume_node() should call hash_delete() and add_users_to_worklist()
kvn
parents: 1552
diff changeset
841 _igvn.replace_node( iff, cmovphi );
0
a61af66fc99e Initial load
duke
parents:
diff changeset
842 return;
a61af66fc99e Initial load
duke
parents:
diff changeset
843 }
a61af66fc99e Initial load
duke
parents:
diff changeset
844
a61af66fc99e Initial load
duke
parents:
diff changeset
845 // Now split the IF
a61af66fc99e Initial load
duke
parents:
diff changeset
846 do_split_if( iff );
a61af66fc99e Initial load
duke
parents:
diff changeset
847 return;
a61af66fc99e Initial load
duke
parents:
diff changeset
848 }
a61af66fc99e Initial load
duke
parents:
diff changeset
849
a61af66fc99e Initial load
duke
parents:
diff changeset
850 // Check for an IF ready to split; one that has its
a61af66fc99e Initial load
duke
parents:
diff changeset
851 // condition codes input coming from a Phi at the block start.
a61af66fc99e Initial load
duke
parents:
diff changeset
852 int n_op = n->Opcode();
a61af66fc99e Initial load
duke
parents:
diff changeset
853
a61af66fc99e Initial load
duke
parents:
diff changeset
854 // Check for an IF being dominated by another IF same test
a61af66fc99e Initial load
duke
parents:
diff changeset
855 if( n_op == Op_If ) {
a61af66fc99e Initial load
duke
parents:
diff changeset
856 Node *bol = n->in(1);
a61af66fc99e Initial load
duke
parents:
diff changeset
857 uint max = bol->outcnt();
a61af66fc99e Initial load
duke
parents:
diff changeset
858 // Check for same test used more than once?
a61af66fc99e Initial load
duke
parents:
diff changeset
859 if( n_op == Op_If && max > 1 && bol->is_Bool() ) {
a61af66fc99e Initial load
duke
parents:
diff changeset
860 // Search up IDOMs to see if this IF is dominated.
a61af66fc99e Initial load
duke
parents:
diff changeset
861 Node *cutoff = get_ctrl(bol);
a61af66fc99e Initial load
duke
parents:
diff changeset
862
a61af66fc99e Initial load
duke
parents:
diff changeset
863 // Now search up IDOMs till cutoff, looking for a dominating test
a61af66fc99e Initial load
duke
parents:
diff changeset
864 Node *prevdom = n;
a61af66fc99e Initial load
duke
parents:
diff changeset
865 Node *dom = idom(prevdom);
a61af66fc99e Initial load
duke
parents:
diff changeset
866 while( dom != cutoff ) {
a61af66fc99e Initial load
duke
parents:
diff changeset
867 if( dom->req() > 1 && dom->in(1) == bol && prevdom->in(0) == dom ) {
a61af66fc99e Initial load
duke
parents:
diff changeset
868 // Replace the dominated test with an obvious true or false.
a61af66fc99e Initial load
duke
parents:
diff changeset
869 // Place it on the IGVN worklist for later cleanup.
a61af66fc99e Initial load
duke
parents:
diff changeset
870 C->set_major_progress();
3840
4e761e7e6e12 7070134: Hotspot crashes with sigsegv from PorterStemmer
kvn
parents: 3281
diff changeset
871 dominated_by( prevdom, n, false, true );
0
a61af66fc99e Initial load
duke
parents:
diff changeset
872 #ifndef PRODUCT
a61af66fc99e Initial load
duke
parents:
diff changeset
873 if( VerifyLoopOptimizations ) verify();
a61af66fc99e Initial load
duke
parents:
diff changeset
874 #endif
a61af66fc99e Initial load
duke
parents:
diff changeset
875 return;
a61af66fc99e Initial load
duke
parents:
diff changeset
876 }
a61af66fc99e Initial load
duke
parents:
diff changeset
877 prevdom = dom;
a61af66fc99e Initial load
duke
parents:
diff changeset
878 dom = idom(prevdom);
a61af66fc99e Initial load
duke
parents:
diff changeset
879 }
a61af66fc99e Initial load
duke
parents:
diff changeset
880 }
a61af66fc99e Initial load
duke
parents:
diff changeset
881 }
a61af66fc99e Initial load
duke
parents:
diff changeset
882
a61af66fc99e Initial load
duke
parents:
diff changeset
883 // See if a shared loop-varying computation has no loop-varying uses.
a61af66fc99e Initial load
duke
parents:
diff changeset
884 // Happens if something is only used for JVM state in uncommon trap exits,
a61af66fc99e Initial load
duke
parents:
diff changeset
885 // like various versions of induction variable+offset. Clone the
a61af66fc99e Initial load
duke
parents:
diff changeset
886 // computation per usage to allow it to sink out of the loop.
a61af66fc99e Initial load
duke
parents:
diff changeset
887 if (has_ctrl(n) && !n->in(0)) {// n not dead and has no control edge (can float about)
a61af66fc99e Initial load
duke
parents:
diff changeset
888 Node *n_ctrl = get_ctrl(n);
a61af66fc99e Initial load
duke
parents:
diff changeset
889 IdealLoopTree *n_loop = get_loop(n_ctrl);
a61af66fc99e Initial load
duke
parents:
diff changeset
890 if( n_loop != _ltree_root ) {
a61af66fc99e Initial load
duke
parents:
diff changeset
891 DUIterator_Fast imax, i = n->fast_outs(imax);
a61af66fc99e Initial load
duke
parents:
diff changeset
892 for (; i < imax; i++) {
a61af66fc99e Initial load
duke
parents:
diff changeset
893 Node* u = n->fast_out(i);
a61af66fc99e Initial load
duke
parents:
diff changeset
894 if( !has_ctrl(u) ) break; // Found control user
a61af66fc99e Initial load
duke
parents:
diff changeset
895 IdealLoopTree *u_loop = get_loop(get_ctrl(u));
a61af66fc99e Initial load
duke
parents:
diff changeset
896 if( u_loop == n_loop ) break; // Found loop-varying use
a61af66fc99e Initial load
duke
parents:
diff changeset
897 if( n_loop->is_member( u_loop ) ) break; // Found use in inner loop
a61af66fc99e Initial load
duke
parents:
diff changeset
898 if( u->Opcode() == Op_Opaque1 ) break; // Found loop limit, bugfix for 4677003
a61af66fc99e Initial load
duke
parents:
diff changeset
899 }
a61af66fc99e Initial load
duke
parents:
diff changeset
900 bool did_break = (i < imax); // Did we break out of the previous loop?
a61af66fc99e Initial load
duke
parents:
diff changeset
901 if (!did_break && n->outcnt() > 1) { // All uses in outer loops!
a61af66fc99e Initial load
duke
parents:
diff changeset
902 Node *late_load_ctrl;
a61af66fc99e Initial load
duke
parents:
diff changeset
903 if (n->is_Load()) {
a61af66fc99e Initial load
duke
parents:
diff changeset
904 // If n is a load, get and save the result from get_late_ctrl(),
a61af66fc99e Initial load
duke
parents:
diff changeset
905 // to be later used in calculating the control for n's clones.
a61af66fc99e Initial load
duke
parents:
diff changeset
906 clear_dom_lca_tags();
a61af66fc99e Initial load
duke
parents:
diff changeset
907 late_load_ctrl = get_late_ctrl(n, n_ctrl);
a61af66fc99e Initial load
duke
parents:
diff changeset
908 }
a61af66fc99e Initial load
duke
parents:
diff changeset
909 // If n is a load, and the late control is the same as the current
a61af66fc99e Initial load
duke
parents:
diff changeset
910 // control, then the cloning of n is a pointless exercise, because
a61af66fc99e Initial load
duke
parents:
diff changeset
911 // GVN will ensure that we end up where we started.
a61af66fc99e Initial load
duke
parents:
diff changeset
912 if (!n->is_Load() || late_load_ctrl != n_ctrl) {
a61af66fc99e Initial load
duke
parents:
diff changeset
913 for (DUIterator_Last jmin, j = n->last_outs(jmin); j >= jmin; ) {
a61af66fc99e Initial load
duke
parents:
diff changeset
914 Node *u = n->last_out(j); // Clone private computation per use
a61af66fc99e Initial load
duke
parents:
diff changeset
915 _igvn.hash_delete(u);
a61af66fc99e Initial load
duke
parents:
diff changeset
916 _igvn._worklist.push(u);
a61af66fc99e Initial load
duke
parents:
diff changeset
917 Node *x = n->clone(); // Clone computation
a61af66fc99e Initial load
duke
parents:
diff changeset
918 Node *x_ctrl = NULL;
a61af66fc99e Initial load
duke
parents:
diff changeset
919 if( u->is_Phi() ) {
a61af66fc99e Initial load
duke
parents:
diff changeset
920 // Replace all uses of normal nodes. Replace Phi uses
605
98cb887364d3 6810672: Comment typos
twisti
parents: 401
diff changeset
921 // individually, so the separate Nodes can sink down
0
a61af66fc99e Initial load
duke
parents:
diff changeset
922 // different paths.
a61af66fc99e Initial load
duke
parents:
diff changeset
923 uint k = 1;
a61af66fc99e Initial load
duke
parents:
diff changeset
924 while( u->in(k) != n ) k++;
a61af66fc99e Initial load
duke
parents:
diff changeset
925 u->set_req( k, x );
a61af66fc99e Initial load
duke
parents:
diff changeset
926 // x goes next to Phi input path
a61af66fc99e Initial load
duke
parents:
diff changeset
927 x_ctrl = u->in(0)->in(k);
a61af66fc99e Initial load
duke
parents:
diff changeset
928 --j;
a61af66fc99e Initial load
duke
parents:
diff changeset
929 } else { // Normal use
a61af66fc99e Initial load
duke
parents:
diff changeset
930 // Replace all uses
a61af66fc99e Initial load
duke
parents:
diff changeset
931 for( uint k = 0; k < u->req(); k++ ) {
a61af66fc99e Initial load
duke
parents:
diff changeset
932 if( u->in(k) == n ) {
a61af66fc99e Initial load
duke
parents:
diff changeset
933 u->set_req( k, x );
a61af66fc99e Initial load
duke
parents:
diff changeset
934 --j;
a61af66fc99e Initial load
duke
parents:
diff changeset
935 }
a61af66fc99e Initial load
duke
parents:
diff changeset
936 }
a61af66fc99e Initial load
duke
parents:
diff changeset
937 x_ctrl = get_ctrl(u);
a61af66fc99e Initial load
duke
parents:
diff changeset
938 }
a61af66fc99e Initial load
duke
parents:
diff changeset
939
a61af66fc99e Initial load
duke
parents:
diff changeset
940 // Find control for 'x' next to use but not inside inner loops.
a61af66fc99e Initial load
duke
parents:
diff changeset
941 // For inner loop uses get the preheader area.
a61af66fc99e Initial load
duke
parents:
diff changeset
942 x_ctrl = place_near_use(x_ctrl);
a61af66fc99e Initial load
duke
parents:
diff changeset
943
a61af66fc99e Initial load
duke
parents:
diff changeset
944 if (n->is_Load()) {
a61af66fc99e Initial load
duke
parents:
diff changeset
945 // For loads, add a control edge to a CFG node outside of the loop
a61af66fc99e Initial load
duke
parents:
diff changeset
946 // to force them to not combine and return back inside the loop
a61af66fc99e Initial load
duke
parents:
diff changeset
947 // during GVN optimization (4641526).
a61af66fc99e Initial load
duke
parents:
diff changeset
948 //
a61af66fc99e Initial load
duke
parents:
diff changeset
949 // Because we are setting the actual control input, factor in
a61af66fc99e Initial load
duke
parents:
diff changeset
950 // the result from get_late_ctrl() so we respect any
a61af66fc99e Initial load
duke
parents:
diff changeset
951 // anti-dependences. (6233005).
a61af66fc99e Initial load
duke
parents:
diff changeset
952 x_ctrl = dom_lca(late_load_ctrl, x_ctrl);
a61af66fc99e Initial load
duke
parents:
diff changeset
953
a61af66fc99e Initial load
duke
parents:
diff changeset
954 // Don't allow the control input to be a CFG splitting node.
a61af66fc99e Initial load
duke
parents:
diff changeset
955 // Such nodes should only have ProjNodes as outs, e.g. IfNode
a61af66fc99e Initial load
duke
parents:
diff changeset
956 // should only have IfTrueNode and IfFalseNode (4985384).
a61af66fc99e Initial load
duke
parents:
diff changeset
957 x_ctrl = find_non_split_ctrl(x_ctrl);
a61af66fc99e Initial load
duke
parents:
diff changeset
958 assert(dom_depth(n_ctrl) <= dom_depth(x_ctrl), "n is later than its clone");
a61af66fc99e Initial load
duke
parents:
diff changeset
959
a61af66fc99e Initial load
duke
parents:
diff changeset
960 x->set_req(0, x_ctrl);
a61af66fc99e Initial load
duke
parents:
diff changeset
961 }
a61af66fc99e Initial load
duke
parents:
diff changeset
962 register_new_node(x, x_ctrl);
a61af66fc99e Initial load
duke
parents:
diff changeset
963
a61af66fc99e Initial load
duke
parents:
diff changeset
964 // Some institutional knowledge is needed here: 'x' is
a61af66fc99e Initial load
duke
parents:
diff changeset
965 // yanked because if the optimizer runs GVN on it all the
a61af66fc99e Initial load
duke
parents:
diff changeset
966 // cloned x's will common up and undo this optimization and
a61af66fc99e Initial load
duke
parents:
diff changeset
967 // be forced back in the loop. This is annoying because it
a61af66fc99e Initial load
duke
parents:
diff changeset
968 // makes +VerifyOpto report false-positives on progress. I
a61af66fc99e Initial load
duke
parents:
diff changeset
969 // tried setting control edges on the x's to force them to
a61af66fc99e Initial load
duke
parents:
diff changeset
970 // not combine, but the matching gets worried when it tries
a61af66fc99e Initial load
duke
parents:
diff changeset
971 // to fold a StoreP and an AddP together (as part of an
a61af66fc99e Initial load
duke
parents:
diff changeset
972 // address expression) and the AddP and StoreP have
a61af66fc99e Initial load
duke
parents:
diff changeset
973 // different controls.
318
60bc5071073f 6738933: assert with base pointers must match with compressed oops enabled
never
parents: 293
diff changeset
974 if( !x->is_Load() && !x->is_DecodeN() ) _igvn._worklist.yank(x);
0
a61af66fc99e Initial load
duke
parents:
diff changeset
975 }
a61af66fc99e Initial load
duke
parents:
diff changeset
976 _igvn.remove_dead_node(n);
a61af66fc99e Initial load
duke
parents:
diff changeset
977 }
a61af66fc99e Initial load
duke
parents:
diff changeset
978 }
a61af66fc99e Initial load
duke
parents:
diff changeset
979 }
a61af66fc99e Initial load
duke
parents:
diff changeset
980 }
a61af66fc99e Initial load
duke
parents:
diff changeset
981
a61af66fc99e Initial load
duke
parents:
diff changeset
982 // Check for Opaque2's who's loop has disappeared - who's input is in the
a61af66fc99e Initial load
duke
parents:
diff changeset
983 // same loop nest as their output. Remove 'em, they are no longer useful.
a61af66fc99e Initial load
duke
parents:
diff changeset
984 if( n_op == Op_Opaque2 &&
a61af66fc99e Initial load
duke
parents:
diff changeset
985 n->in(1) != NULL &&
a61af66fc99e Initial load
duke
parents:
diff changeset
986 get_loop(get_ctrl(n)) == get_loop(get_ctrl(n->in(1))) ) {
1621
6027dddc26c6 6677629: PhaseIterGVN::subsume_node() should call hash_delete() and add_users_to_worklist()
kvn
parents: 1552
diff changeset
987 _igvn.replace_node( n, n->in(1) );
0
a61af66fc99e Initial load
duke
parents:
diff changeset
988 }
a61af66fc99e Initial load
duke
parents:
diff changeset
989 }
a61af66fc99e Initial load
duke
parents:
diff changeset
990
a61af66fc99e Initial load
duke
parents:
diff changeset
991 //------------------------------split_if_with_blocks---------------------------
a61af66fc99e Initial load
duke
parents:
diff changeset
992 // Check for aggressive application of 'split-if' optimization,
a61af66fc99e Initial load
duke
parents:
diff changeset
993 // using basic block level info.
a61af66fc99e Initial load
duke
parents:
diff changeset
994 void PhaseIdealLoop::split_if_with_blocks( VectorSet &visited, Node_Stack &nstack ) {
a61af66fc99e Initial load
duke
parents:
diff changeset
995 Node *n = C->root();
a61af66fc99e Initial load
duke
parents:
diff changeset
996 visited.set(n->_idx); // first, mark node as visited
a61af66fc99e Initial load
duke
parents:
diff changeset
997 // Do pre-visit work for root
a61af66fc99e Initial load
duke
parents:
diff changeset
998 n = split_if_with_blocks_pre( n );
a61af66fc99e Initial load
duke
parents:
diff changeset
999 uint cnt = n->outcnt();
a61af66fc99e Initial load
duke
parents:
diff changeset
1000 uint i = 0;
a61af66fc99e Initial load
duke
parents:
diff changeset
1001 while (true) {
a61af66fc99e Initial load
duke
parents:
diff changeset
1002 // Visit all children
a61af66fc99e Initial load
duke
parents:
diff changeset
1003 if (i < cnt) {
a61af66fc99e Initial load
duke
parents:
diff changeset
1004 Node* use = n->raw_out(i);
a61af66fc99e Initial load
duke
parents:
diff changeset
1005 ++i;
a61af66fc99e Initial load
duke
parents:
diff changeset
1006 if (use->outcnt() != 0 && !visited.test_set(use->_idx)) {
a61af66fc99e Initial load
duke
parents:
diff changeset
1007 // Now do pre-visit work for this use
a61af66fc99e Initial load
duke
parents:
diff changeset
1008 use = split_if_with_blocks_pre( use );
a61af66fc99e Initial load
duke
parents:
diff changeset
1009 nstack.push(n, i); // Save parent and next use's index.
a61af66fc99e Initial load
duke
parents:
diff changeset
1010 n = use; // Process all children of current use.
a61af66fc99e Initial load
duke
parents:
diff changeset
1011 cnt = use->outcnt();
a61af66fc99e Initial load
duke
parents:
diff changeset
1012 i = 0;
a61af66fc99e Initial load
duke
parents:
diff changeset
1013 }
a61af66fc99e Initial load
duke
parents:
diff changeset
1014 }
a61af66fc99e Initial load
duke
parents:
diff changeset
1015 else {
a61af66fc99e Initial load
duke
parents:
diff changeset
1016 // All of n's children have been processed, complete post-processing.
a61af66fc99e Initial load
duke
parents:
diff changeset
1017 if (cnt != 0 && !n->is_Con()) {
a61af66fc99e Initial load
duke
parents:
diff changeset
1018 assert(has_node(n), "no dead nodes");
a61af66fc99e Initial load
duke
parents:
diff changeset
1019 split_if_with_blocks_post( n );
a61af66fc99e Initial load
duke
parents:
diff changeset
1020 }
a61af66fc99e Initial load
duke
parents:
diff changeset
1021 if (nstack.is_empty()) {
a61af66fc99e Initial load
duke
parents:
diff changeset
1022 // Finished all nodes on stack.
a61af66fc99e Initial load
duke
parents:
diff changeset
1023 break;
a61af66fc99e Initial load
duke
parents:
diff changeset
1024 }
a61af66fc99e Initial load
duke
parents:
diff changeset
1025 // Get saved parent node and next use's index. Visit the rest of uses.
a61af66fc99e Initial load
duke
parents:
diff changeset
1026 n = nstack.node();
a61af66fc99e Initial load
duke
parents:
diff changeset
1027 cnt = n->outcnt();
a61af66fc99e Initial load
duke
parents:
diff changeset
1028 i = nstack.index();
a61af66fc99e Initial load
duke
parents:
diff changeset
1029 nstack.pop();
a61af66fc99e Initial load
duke
parents:
diff changeset
1030 }
a61af66fc99e Initial load
duke
parents:
diff changeset
1031 }
a61af66fc99e Initial load
duke
parents:
diff changeset
1032 }
a61af66fc99e Initial load
duke
parents:
diff changeset
1033
a61af66fc99e Initial load
duke
parents:
diff changeset
1034
a61af66fc99e Initial load
duke
parents:
diff changeset
1035 //=============================================================================
a61af66fc99e Initial load
duke
parents:
diff changeset
1036 //
a61af66fc99e Initial load
duke
parents:
diff changeset
1037 // C L O N E A L O O P B O D Y
a61af66fc99e Initial load
duke
parents:
diff changeset
1038 //
a61af66fc99e Initial load
duke
parents:
diff changeset
1039
a61af66fc99e Initial load
duke
parents:
diff changeset
1040 //------------------------------clone_iff--------------------------------------
a61af66fc99e Initial load
duke
parents:
diff changeset
1041 // Passed in a Phi merging (recursively) some nearly equivalent Bool/Cmps.
a61af66fc99e Initial load
duke
parents:
diff changeset
1042 // "Nearly" because all Nodes have been cloned from the original in the loop,
a61af66fc99e Initial load
duke
parents:
diff changeset
1043 // but the fall-in edges to the Cmp are different. Clone bool/Cmp pairs
a61af66fc99e Initial load
duke
parents:
diff changeset
1044 // through the Phi recursively, and return a Bool.
a61af66fc99e Initial load
duke
parents:
diff changeset
1045 BoolNode *PhaseIdealLoop::clone_iff( PhiNode *phi, IdealLoopTree *loop ) {
a61af66fc99e Initial load
duke
parents:
diff changeset
1046
a61af66fc99e Initial load
duke
parents:
diff changeset
1047 // Convert this Phi into a Phi merging Bools
a61af66fc99e Initial load
duke
parents:
diff changeset
1048 uint i;
a61af66fc99e Initial load
duke
parents:
diff changeset
1049 for( i = 1; i < phi->req(); i++ ) {
a61af66fc99e Initial load
duke
parents:
diff changeset
1050 Node *b = phi->in(i);
a61af66fc99e Initial load
duke
parents:
diff changeset
1051 if( b->is_Phi() ) {
a61af66fc99e Initial load
duke
parents:
diff changeset
1052 _igvn.hash_delete(phi);
a61af66fc99e Initial load
duke
parents:
diff changeset
1053 _igvn._worklist.push(phi);
a61af66fc99e Initial load
duke
parents:
diff changeset
1054 phi->set_req(i, clone_iff( b->as_Phi(), loop ));
a61af66fc99e Initial load
duke
parents:
diff changeset
1055 } else {
a61af66fc99e Initial load
duke
parents:
diff changeset
1056 assert( b->is_Bool(), "" );
a61af66fc99e Initial load
duke
parents:
diff changeset
1057 }
a61af66fc99e Initial load
duke
parents:
diff changeset
1058 }
a61af66fc99e Initial load
duke
parents:
diff changeset
1059
a61af66fc99e Initial load
duke
parents:
diff changeset
1060 Node *sample_bool = phi->in(1);
a61af66fc99e Initial load
duke
parents:
diff changeset
1061 Node *sample_cmp = sample_bool->in(1);
a61af66fc99e Initial load
duke
parents:
diff changeset
1062
a61af66fc99e Initial load
duke
parents:
diff changeset
1063 // Make Phis to merge the Cmp's inputs.
a61af66fc99e Initial load
duke
parents:
diff changeset
1064 int size = phi->in(0)->req();
a61af66fc99e Initial load
duke
parents:
diff changeset
1065 PhiNode *phi1 = new (C, size) PhiNode( phi->in(0), Type::TOP );
a61af66fc99e Initial load
duke
parents:
diff changeset
1066 PhiNode *phi2 = new (C, size) PhiNode( phi->in(0), Type::TOP );
a61af66fc99e Initial load
duke
parents:
diff changeset
1067 for( i = 1; i < phi->req(); i++ ) {
a61af66fc99e Initial load
duke
parents:
diff changeset
1068 Node *n1 = phi->in(i)->in(1)->in(1);
a61af66fc99e Initial load
duke
parents:
diff changeset
1069 Node *n2 = phi->in(i)->in(1)->in(2);
a61af66fc99e Initial load
duke
parents:
diff changeset
1070 phi1->set_req( i, n1 );
a61af66fc99e Initial load
duke
parents:
diff changeset
1071 phi2->set_req( i, n2 );
a61af66fc99e Initial load
duke
parents:
diff changeset
1072 phi1->set_type( phi1->type()->meet(n1->bottom_type()) );
a61af66fc99e Initial load
duke
parents:
diff changeset
1073 phi2->set_type( phi2->type()->meet(n2->bottom_type()) );
a61af66fc99e Initial load
duke
parents:
diff changeset
1074 }
a61af66fc99e Initial load
duke
parents:
diff changeset
1075 // See if these Phis have been made before.
a61af66fc99e Initial load
duke
parents:
diff changeset
1076 // Register with optimizer
a61af66fc99e Initial load
duke
parents:
diff changeset
1077 Node *hit1 = _igvn.hash_find_insert(phi1);
a61af66fc99e Initial load
duke
parents:
diff changeset
1078 if( hit1 ) { // Hit, toss just made Phi
a61af66fc99e Initial load
duke
parents:
diff changeset
1079 _igvn.remove_dead_node(phi1); // Remove new phi
a61af66fc99e Initial load
duke
parents:
diff changeset
1080 assert( hit1->is_Phi(), "" );
a61af66fc99e Initial load
duke
parents:
diff changeset
1081 phi1 = (PhiNode*)hit1; // Use existing phi
a61af66fc99e Initial load
duke
parents:
diff changeset
1082 } else { // Miss
a61af66fc99e Initial load
duke
parents:
diff changeset
1083 _igvn.register_new_node_with_optimizer(phi1);
a61af66fc99e Initial load
duke
parents:
diff changeset
1084 }
a61af66fc99e Initial load
duke
parents:
diff changeset
1085 Node *hit2 = _igvn.hash_find_insert(phi2);
a61af66fc99e Initial load
duke
parents:
diff changeset
1086 if( hit2 ) { // Hit, toss just made Phi
a61af66fc99e Initial load
duke
parents:
diff changeset
1087 _igvn.remove_dead_node(phi2); // Remove new phi
a61af66fc99e Initial load
duke
parents:
diff changeset
1088 assert( hit2->is_Phi(), "" );
a61af66fc99e Initial load
duke
parents:
diff changeset
1089 phi2 = (PhiNode*)hit2; // Use existing phi
a61af66fc99e Initial load
duke
parents:
diff changeset
1090 } else { // Miss
a61af66fc99e Initial load
duke
parents:
diff changeset
1091 _igvn.register_new_node_with_optimizer(phi2);
a61af66fc99e Initial load
duke
parents:
diff changeset
1092 }
a61af66fc99e Initial load
duke
parents:
diff changeset
1093 // Register Phis with loop/block info
a61af66fc99e Initial load
duke
parents:
diff changeset
1094 set_ctrl(phi1, phi->in(0));
a61af66fc99e Initial load
duke
parents:
diff changeset
1095 set_ctrl(phi2, phi->in(0));
a61af66fc99e Initial load
duke
parents:
diff changeset
1096 // Make a new Cmp
a61af66fc99e Initial load
duke
parents:
diff changeset
1097 Node *cmp = sample_cmp->clone();
a61af66fc99e Initial load
duke
parents:
diff changeset
1098 cmp->set_req( 1, phi1 );
a61af66fc99e Initial load
duke
parents:
diff changeset
1099 cmp->set_req( 2, phi2 );
a61af66fc99e Initial load
duke
parents:
diff changeset
1100 _igvn.register_new_node_with_optimizer(cmp);
a61af66fc99e Initial load
duke
parents:
diff changeset
1101 set_ctrl(cmp, phi->in(0));
a61af66fc99e Initial load
duke
parents:
diff changeset
1102
a61af66fc99e Initial load
duke
parents:
diff changeset
1103 // Make a new Bool
a61af66fc99e Initial load
duke
parents:
diff changeset
1104 Node *b = sample_bool->clone();
a61af66fc99e Initial load
duke
parents:
diff changeset
1105 b->set_req(1,cmp);
a61af66fc99e Initial load
duke
parents:
diff changeset
1106 _igvn.register_new_node_with_optimizer(b);
a61af66fc99e Initial load
duke
parents:
diff changeset
1107 set_ctrl(b, phi->in(0));
a61af66fc99e Initial load
duke
parents:
diff changeset
1108
a61af66fc99e Initial load
duke
parents:
diff changeset
1109 assert( b->is_Bool(), "" );
a61af66fc99e Initial load
duke
parents:
diff changeset
1110 return (BoolNode*)b;
a61af66fc99e Initial load
duke
parents:
diff changeset
1111 }
a61af66fc99e Initial load
duke
parents:
diff changeset
1112
a61af66fc99e Initial load
duke
parents:
diff changeset
1113 //------------------------------clone_bool-------------------------------------
a61af66fc99e Initial load
duke
parents:
diff changeset
1114 // Passed in a Phi merging (recursively) some nearly equivalent Bool/Cmps.
a61af66fc99e Initial load
duke
parents:
diff changeset
1115 // "Nearly" because all Nodes have been cloned from the original in the loop,
a61af66fc99e Initial load
duke
parents:
diff changeset
1116 // but the fall-in edges to the Cmp are different. Clone bool/Cmp pairs
a61af66fc99e Initial load
duke
parents:
diff changeset
1117 // through the Phi recursively, and return a Bool.
a61af66fc99e Initial load
duke
parents:
diff changeset
1118 CmpNode *PhaseIdealLoop::clone_bool( PhiNode *phi, IdealLoopTree *loop ) {
a61af66fc99e Initial load
duke
parents:
diff changeset
1119 uint i;
a61af66fc99e Initial load
duke
parents:
diff changeset
1120 // Convert this Phi into a Phi merging Bools
a61af66fc99e Initial load
duke
parents:
diff changeset
1121 for( i = 1; i < phi->req(); i++ ) {
a61af66fc99e Initial load
duke
parents:
diff changeset
1122 Node *b = phi->in(i);
a61af66fc99e Initial load
duke
parents:
diff changeset
1123 if( b->is_Phi() ) {
a61af66fc99e Initial load
duke
parents:
diff changeset
1124 _igvn.hash_delete(phi);
a61af66fc99e Initial load
duke
parents:
diff changeset
1125 _igvn._worklist.push(phi);
a61af66fc99e Initial load
duke
parents:
diff changeset
1126 phi->set_req(i, clone_bool( b->as_Phi(), loop ));
a61af66fc99e Initial load
duke
parents:
diff changeset
1127 } else {
a61af66fc99e Initial load
duke
parents:
diff changeset
1128 assert( b->is_Cmp() || b->is_top(), "inputs are all Cmp or TOP" );
a61af66fc99e Initial load
duke
parents:
diff changeset
1129 }
a61af66fc99e Initial load
duke
parents:
diff changeset
1130 }
a61af66fc99e Initial load
duke
parents:
diff changeset
1131
a61af66fc99e Initial load
duke
parents:
diff changeset
1132 Node *sample_cmp = phi->in(1);
a61af66fc99e Initial load
duke
parents:
diff changeset
1133
a61af66fc99e Initial load
duke
parents:
diff changeset
1134 // Make Phis to merge the Cmp's inputs.
a61af66fc99e Initial load
duke
parents:
diff changeset
1135 int size = phi->in(0)->req();
a61af66fc99e Initial load
duke
parents:
diff changeset
1136 PhiNode *phi1 = new (C, size) PhiNode( phi->in(0), Type::TOP );
a61af66fc99e Initial load
duke
parents:
diff changeset
1137 PhiNode *phi2 = new (C, size) PhiNode( phi->in(0), Type::TOP );
a61af66fc99e Initial load
duke
parents:
diff changeset
1138 for( uint j = 1; j < phi->req(); j++ ) {
a61af66fc99e Initial load
duke
parents:
diff changeset
1139 Node *cmp_top = phi->in(j); // Inputs are all Cmp or TOP
a61af66fc99e Initial load
duke
parents:
diff changeset
1140 Node *n1, *n2;
a61af66fc99e Initial load
duke
parents:
diff changeset
1141 if( cmp_top->is_Cmp() ) {
a61af66fc99e Initial load
duke
parents:
diff changeset
1142 n1 = cmp_top->in(1);
a61af66fc99e Initial load
duke
parents:
diff changeset
1143 n2 = cmp_top->in(2);
a61af66fc99e Initial load
duke
parents:
diff changeset
1144 } else {
a61af66fc99e Initial load
duke
parents:
diff changeset
1145 n1 = n2 = cmp_top;
a61af66fc99e Initial load
duke
parents:
diff changeset
1146 }
a61af66fc99e Initial load
duke
parents:
diff changeset
1147 phi1->set_req( j, n1 );
a61af66fc99e Initial load
duke
parents:
diff changeset
1148 phi2->set_req( j, n2 );
a61af66fc99e Initial load
duke
parents:
diff changeset
1149 phi1->set_type( phi1->type()->meet(n1->bottom_type()) );
a61af66fc99e Initial load
duke
parents:
diff changeset
1150 phi2->set_type( phi2->type()->meet(n2->bottom_type()) );
a61af66fc99e Initial load
duke
parents:
diff changeset
1151 }
a61af66fc99e Initial load
duke
parents:
diff changeset
1152
a61af66fc99e Initial load
duke
parents:
diff changeset
1153 // See if these Phis have been made before.
a61af66fc99e Initial load
duke
parents:
diff changeset
1154 // Register with optimizer
a61af66fc99e Initial load
duke
parents:
diff changeset
1155 Node *hit1 = _igvn.hash_find_insert(phi1);
a61af66fc99e Initial load
duke
parents:
diff changeset
1156 if( hit1 ) { // Hit, toss just made Phi
a61af66fc99e Initial load
duke
parents:
diff changeset
1157 _igvn.remove_dead_node(phi1); // Remove new phi
a61af66fc99e Initial load
duke
parents:
diff changeset
1158 assert( hit1->is_Phi(), "" );
a61af66fc99e Initial load
duke
parents:
diff changeset
1159 phi1 = (PhiNode*)hit1; // Use existing phi
a61af66fc99e Initial load
duke
parents:
diff changeset
1160 } else { // Miss
a61af66fc99e Initial load
duke
parents:
diff changeset
1161 _igvn.register_new_node_with_optimizer(phi1);
a61af66fc99e Initial load
duke
parents:
diff changeset
1162 }
a61af66fc99e Initial load
duke
parents:
diff changeset
1163 Node *hit2 = _igvn.hash_find_insert(phi2);
a61af66fc99e Initial load
duke
parents:
diff changeset
1164 if( hit2 ) { // Hit, toss just made Phi
a61af66fc99e Initial load
duke
parents:
diff changeset
1165 _igvn.remove_dead_node(phi2); // Remove new phi
a61af66fc99e Initial load
duke
parents:
diff changeset
1166 assert( hit2->is_Phi(), "" );
a61af66fc99e Initial load
duke
parents:
diff changeset
1167 phi2 = (PhiNode*)hit2; // Use existing phi
a61af66fc99e Initial load
duke
parents:
diff changeset
1168 } else { // Miss
a61af66fc99e Initial load
duke
parents:
diff changeset
1169 _igvn.register_new_node_with_optimizer(phi2);
a61af66fc99e Initial load
duke
parents:
diff changeset
1170 }
a61af66fc99e Initial load
duke
parents:
diff changeset
1171 // Register Phis with loop/block info
a61af66fc99e Initial load
duke
parents:
diff changeset
1172 set_ctrl(phi1, phi->in(0));
a61af66fc99e Initial load
duke
parents:
diff changeset
1173 set_ctrl(phi2, phi->in(0));
a61af66fc99e Initial load
duke
parents:
diff changeset
1174 // Make a new Cmp
a61af66fc99e Initial load
duke
parents:
diff changeset
1175 Node *cmp = sample_cmp->clone();
a61af66fc99e Initial load
duke
parents:
diff changeset
1176 cmp->set_req( 1, phi1 );
a61af66fc99e Initial load
duke
parents:
diff changeset
1177 cmp->set_req( 2, phi2 );
a61af66fc99e Initial load
duke
parents:
diff changeset
1178 _igvn.register_new_node_with_optimizer(cmp);
a61af66fc99e Initial load
duke
parents:
diff changeset
1179 set_ctrl(cmp, phi->in(0));
a61af66fc99e Initial load
duke
parents:
diff changeset
1180
a61af66fc99e Initial load
duke
parents:
diff changeset
1181 assert( cmp->is_Cmp(), "" );
a61af66fc99e Initial load
duke
parents:
diff changeset
1182 return (CmpNode*)cmp;
a61af66fc99e Initial load
duke
parents:
diff changeset
1183 }
a61af66fc99e Initial load
duke
parents:
diff changeset
1184
a61af66fc99e Initial load
duke
parents:
diff changeset
1185 //------------------------------sink_use---------------------------------------
a61af66fc99e Initial load
duke
parents:
diff changeset
1186 // If 'use' was in the loop-exit block, it now needs to be sunk
a61af66fc99e Initial load
duke
parents:
diff changeset
1187 // below the post-loop merge point.
a61af66fc99e Initial load
duke
parents:
diff changeset
1188 void PhaseIdealLoop::sink_use( Node *use, Node *post_loop ) {
a61af66fc99e Initial load
duke
parents:
diff changeset
1189 if (!use->is_CFG() && get_ctrl(use) == post_loop->in(2)) {
a61af66fc99e Initial load
duke
parents:
diff changeset
1190 set_ctrl(use, post_loop);
a61af66fc99e Initial load
duke
parents:
diff changeset
1191 for (DUIterator j = use->outs(); use->has_out(j); j++)
a61af66fc99e Initial load
duke
parents:
diff changeset
1192 sink_use(use->out(j), post_loop);
a61af66fc99e Initial load
duke
parents:
diff changeset
1193 }
a61af66fc99e Initial load
duke
parents:
diff changeset
1194 }
a61af66fc99e Initial load
duke
parents:
diff changeset
1195
a61af66fc99e Initial load
duke
parents:
diff changeset
1196 //------------------------------clone_loop-------------------------------------
a61af66fc99e Initial load
duke
parents:
diff changeset
1197 //
a61af66fc99e Initial load
duke
parents:
diff changeset
1198 // C L O N E A L O O P B O D Y
a61af66fc99e Initial load
duke
parents:
diff changeset
1199 //
a61af66fc99e Initial load
duke
parents:
diff changeset
1200 // This is the basic building block of the loop optimizations. It clones an
a61af66fc99e Initial load
duke
parents:
diff changeset
1201 // entire loop body. It makes an old_new loop body mapping; with this mapping
a61af66fc99e Initial load
duke
parents:
diff changeset
1202 // you can find the new-loop equivalent to an old-loop node. All new-loop
a61af66fc99e Initial load
duke
parents:
diff changeset
1203 // nodes are exactly equal to their old-loop counterparts, all edges are the
a61af66fc99e Initial load
duke
parents:
diff changeset
1204 // same. All exits from the old-loop now have a RegionNode that merges the
a61af66fc99e Initial load
duke
parents:
diff changeset
1205 // equivalent new-loop path. This is true even for the normal "loop-exit"
a61af66fc99e Initial load
duke
parents:
diff changeset
1206 // condition. All uses of loop-invariant old-loop values now come from (one
a61af66fc99e Initial load
duke
parents:
diff changeset
1207 // or more) Phis that merge their new-loop equivalents.
a61af66fc99e Initial load
duke
parents:
diff changeset
1208 //
a61af66fc99e Initial load
duke
parents:
diff changeset
1209 // This operation leaves the graph in an illegal state: there are two valid
a61af66fc99e Initial load
duke
parents:
diff changeset
1210 // control edges coming from the loop pre-header to both loop bodies. I'll
a61af66fc99e Initial load
duke
parents:
diff changeset
1211 // definitely have to hack the graph after running this transform.
a61af66fc99e Initial load
duke
parents:
diff changeset
1212 //
a61af66fc99e Initial load
duke
parents:
diff changeset
1213 // From this building block I will further edit edges to perform loop peeling
a61af66fc99e Initial load
duke
parents:
diff changeset
1214 // or loop unrolling or iteration splitting (Range-Check-Elimination), etc.
a61af66fc99e Initial load
duke
parents:
diff changeset
1215 //
a61af66fc99e Initial load
duke
parents:
diff changeset
1216 // Parameter side_by_size_idom:
a61af66fc99e Initial load
duke
parents:
diff changeset
1217 // When side_by_size_idom is NULL, the dominator tree is constructed for
a61af66fc99e Initial load
duke
parents:
diff changeset
1218 // the clone loop to dominate the original. Used in construction of
a61af66fc99e Initial load
duke
parents:
diff changeset
1219 // pre-main-post loop sequence.
a61af66fc99e Initial load
duke
parents:
diff changeset
1220 // When nonnull, the clone and original are side-by-side, both are
a61af66fc99e Initial load
duke
parents:
diff changeset
1221 // dominated by the side_by_side_idom node. Used in construction of
a61af66fc99e Initial load
duke
parents:
diff changeset
1222 // unswitched loops.
a61af66fc99e Initial load
duke
parents:
diff changeset
1223 void PhaseIdealLoop::clone_loop( IdealLoopTree *loop, Node_List &old_new, int dd,
a61af66fc99e Initial load
duke
parents:
diff changeset
1224 Node* side_by_side_idom) {
a61af66fc99e Initial load
duke
parents:
diff changeset
1225
a61af66fc99e Initial load
duke
parents:
diff changeset
1226 // Step 1: Clone the loop body. Make the old->new mapping.
a61af66fc99e Initial load
duke
parents:
diff changeset
1227 uint i;
a61af66fc99e Initial load
duke
parents:
diff changeset
1228 for( i = 0; i < loop->_body.size(); i++ ) {
a61af66fc99e Initial load
duke
parents:
diff changeset
1229 Node *old = loop->_body.at(i);
a61af66fc99e Initial load
duke
parents:
diff changeset
1230 Node *nnn = old->clone();
a61af66fc99e Initial load
duke
parents:
diff changeset
1231 old_new.map( old->_idx, nnn );
a61af66fc99e Initial load
duke
parents:
diff changeset
1232 _igvn.register_new_node_with_optimizer(nnn);
a61af66fc99e Initial load
duke
parents:
diff changeset
1233 }
a61af66fc99e Initial load
duke
parents:
diff changeset
1234
a61af66fc99e Initial load
duke
parents:
diff changeset
1235
a61af66fc99e Initial load
duke
parents:
diff changeset
1236 // Step 2: Fix the edges in the new body. If the old input is outside the
a61af66fc99e Initial load
duke
parents:
diff changeset
1237 // loop use it. If the old input is INside the loop, use the corresponding
a61af66fc99e Initial load
duke
parents:
diff changeset
1238 // new node instead.
a61af66fc99e Initial load
duke
parents:
diff changeset
1239 for( i = 0; i < loop->_body.size(); i++ ) {
a61af66fc99e Initial load
duke
parents:
diff changeset
1240 Node *old = loop->_body.at(i);
a61af66fc99e Initial load
duke
parents:
diff changeset
1241 Node *nnn = old_new[old->_idx];
a61af66fc99e Initial load
duke
parents:
diff changeset
1242 // Fix CFG/Loop controlling the new node
a61af66fc99e Initial load
duke
parents:
diff changeset
1243 if (has_ctrl(old)) {
a61af66fc99e Initial load
duke
parents:
diff changeset
1244 set_ctrl(nnn, old_new[get_ctrl(old)->_idx]);
a61af66fc99e Initial load
duke
parents:
diff changeset
1245 } else {
a61af66fc99e Initial load
duke
parents:
diff changeset
1246 set_loop(nnn, loop->_parent);
a61af66fc99e Initial load
duke
parents:
diff changeset
1247 if (old->outcnt() > 0) {
a61af66fc99e Initial load
duke
parents:
diff changeset
1248 set_idom( nnn, old_new[idom(old)->_idx], dd );
a61af66fc99e Initial load
duke
parents:
diff changeset
1249 }
a61af66fc99e Initial load
duke
parents:
diff changeset
1250 }
a61af66fc99e Initial load
duke
parents:
diff changeset
1251 // Correct edges to the new node
a61af66fc99e Initial load
duke
parents:
diff changeset
1252 for( uint j = 0; j < nnn->req(); j++ ) {
a61af66fc99e Initial load
duke
parents:
diff changeset
1253 Node *n = nnn->in(j);
a61af66fc99e Initial load
duke
parents:
diff changeset
1254 if( n ) {
a61af66fc99e Initial load
duke
parents:
diff changeset
1255 IdealLoopTree *old_in_loop = get_loop( has_ctrl(n) ? get_ctrl(n) : n );
a61af66fc99e Initial load
duke
parents:
diff changeset
1256 if( loop->is_member( old_in_loop ) )
a61af66fc99e Initial load
duke
parents:
diff changeset
1257 nnn->set_req(j, old_new[n->_idx]);
a61af66fc99e Initial load
duke
parents:
diff changeset
1258 }
a61af66fc99e Initial load
duke
parents:
diff changeset
1259 }
a61af66fc99e Initial load
duke
parents:
diff changeset
1260 _igvn.hash_find_insert(nnn);
a61af66fc99e Initial load
duke
parents:
diff changeset
1261 }
a61af66fc99e Initial load
duke
parents:
diff changeset
1262 Node *newhead = old_new[loop->_head->_idx];
a61af66fc99e Initial load
duke
parents:
diff changeset
1263 set_idom(newhead, newhead->in(LoopNode::EntryControl), dd);
a61af66fc99e Initial load
duke
parents:
diff changeset
1264
a61af66fc99e Initial load
duke
parents:
diff changeset
1265
a61af66fc99e Initial load
duke
parents:
diff changeset
1266 // Step 3: Now fix control uses. Loop varying control uses have already
a61af66fc99e Initial load
duke
parents:
diff changeset
1267 // been fixed up (as part of all input edges in Step 2). Loop invariant
a61af66fc99e Initial load
duke
parents:
diff changeset
1268 // control uses must be either an IfFalse or an IfTrue. Make a merge
a61af66fc99e Initial load
duke
parents:
diff changeset
1269 // point to merge the old and new IfFalse/IfTrue nodes; make the use
a61af66fc99e Initial load
duke
parents:
diff changeset
1270 // refer to this.
a61af66fc99e Initial load
duke
parents:
diff changeset
1271 ResourceArea *area = Thread::current()->resource_area();
a61af66fc99e Initial load
duke
parents:
diff changeset
1272 Node_List worklist(area);
a61af66fc99e Initial load
duke
parents:
diff changeset
1273 uint new_counter = C->unique();
a61af66fc99e Initial load
duke
parents:
diff changeset
1274 for( i = 0; i < loop->_body.size(); i++ ) {
a61af66fc99e Initial load
duke
parents:
diff changeset
1275 Node* old = loop->_body.at(i);
a61af66fc99e Initial load
duke
parents:
diff changeset
1276 if( !old->is_CFG() ) continue;
a61af66fc99e Initial load
duke
parents:
diff changeset
1277 Node* nnn = old_new[old->_idx];
a61af66fc99e Initial load
duke
parents:
diff changeset
1278
a61af66fc99e Initial load
duke
parents:
diff changeset
1279 // Copy uses to a worklist, so I can munge the def-use info
a61af66fc99e Initial load
duke
parents:
diff changeset
1280 // with impunity.
a61af66fc99e Initial load
duke
parents:
diff changeset
1281 for (DUIterator_Fast jmax, j = old->fast_outs(jmax); j < jmax; j++)
a61af66fc99e Initial load
duke
parents:
diff changeset
1282 worklist.push(old->fast_out(j));
a61af66fc99e Initial load
duke
parents:
diff changeset
1283
a61af66fc99e Initial load
duke
parents:
diff changeset
1284 while( worklist.size() ) { // Visit all uses
a61af66fc99e Initial load
duke
parents:
diff changeset
1285 Node *use = worklist.pop();
a61af66fc99e Initial load
duke
parents:
diff changeset
1286 if (!has_node(use)) continue; // Ignore dead nodes
a61af66fc99e Initial load
duke
parents:
diff changeset
1287 IdealLoopTree *use_loop = get_loop( has_ctrl(use) ? get_ctrl(use) : use );
a61af66fc99e Initial load
duke
parents:
diff changeset
1288 if( !loop->is_member( use_loop ) && use->is_CFG() ) {
a61af66fc99e Initial load
duke
parents:
diff changeset
1289 // Both OLD and USE are CFG nodes here.
a61af66fc99e Initial load
duke
parents:
diff changeset
1290 assert( use->is_Proj(), "" );
a61af66fc99e Initial load
duke
parents:
diff changeset
1291
a61af66fc99e Initial load
duke
parents:
diff changeset
1292 // Clone the loop exit control projection
a61af66fc99e Initial load
duke
parents:
diff changeset
1293 Node *newuse = use->clone();
a61af66fc99e Initial load
duke
parents:
diff changeset
1294 newuse->set_req(0,nnn);
a61af66fc99e Initial load
duke
parents:
diff changeset
1295 _igvn.register_new_node_with_optimizer(newuse);
a61af66fc99e Initial load
duke
parents:
diff changeset
1296 set_loop(newuse, use_loop);
a61af66fc99e Initial load
duke
parents:
diff changeset
1297 set_idom(newuse, nnn, dom_depth(nnn) + 1 );
a61af66fc99e Initial load
duke
parents:
diff changeset
1298
a61af66fc99e Initial load
duke
parents:
diff changeset
1299 // We need a Region to merge the exit from the peeled body and the
a61af66fc99e Initial load
duke
parents:
diff changeset
1300 // exit from the old loop body.
a61af66fc99e Initial load
duke
parents:
diff changeset
1301 RegionNode *r = new (C, 3) RegionNode(3);
a61af66fc99e Initial load
duke
parents:
diff changeset
1302 // Map the old use to the new merge point
a61af66fc99e Initial load
duke
parents:
diff changeset
1303 old_new.map( use->_idx, r );
a61af66fc99e Initial load
duke
parents:
diff changeset
1304 uint dd_r = MIN2(dom_depth(newuse),dom_depth(use));
a61af66fc99e Initial load
duke
parents:
diff changeset
1305 assert( dd_r >= dom_depth(dom_lca(newuse,use)), "" );
a61af66fc99e Initial load
duke
parents:
diff changeset
1306
a61af66fc99e Initial load
duke
parents:
diff changeset
1307 // The original user of 'use' uses 'r' instead.
a61af66fc99e Initial load
duke
parents:
diff changeset
1308 for (DUIterator_Last lmin, l = use->last_outs(lmin); l >= lmin;) {
a61af66fc99e Initial load
duke
parents:
diff changeset
1309 Node* useuse = use->last_out(l);
a61af66fc99e Initial load
duke
parents:
diff changeset
1310 _igvn.hash_delete(useuse);
a61af66fc99e Initial load
duke
parents:
diff changeset
1311 _igvn._worklist.push(useuse);
a61af66fc99e Initial load
duke
parents:
diff changeset
1312 uint uses_found = 0;
a61af66fc99e Initial load
duke
parents:
diff changeset
1313 if( useuse->in(0) == use ) {
a61af66fc99e Initial load
duke
parents:
diff changeset
1314 useuse->set_req(0, r);
a61af66fc99e Initial load
duke
parents:
diff changeset
1315 uses_found++;
a61af66fc99e Initial load
duke
parents:
diff changeset
1316 if( useuse->is_CFG() ) {
a61af66fc99e Initial load
duke
parents:
diff changeset
1317 assert( dom_depth(useuse) > dd_r, "" );
a61af66fc99e Initial load
duke
parents:
diff changeset
1318 set_idom(useuse, r, dom_depth(useuse));
a61af66fc99e Initial load
duke
parents:
diff changeset
1319 }
a61af66fc99e Initial load
duke
parents:
diff changeset
1320 }
a61af66fc99e Initial load
duke
parents:
diff changeset
1321 for( uint k = 1; k < useuse->req(); k++ ) {
a61af66fc99e Initial load
duke
parents:
diff changeset
1322 if( useuse->in(k) == use ) {
a61af66fc99e Initial load
duke
parents:
diff changeset
1323 useuse->set_req(k, r);
a61af66fc99e Initial load
duke
parents:
diff changeset
1324 uses_found++;
a61af66fc99e Initial load
duke
parents:
diff changeset
1325 }
a61af66fc99e Initial load
duke
parents:
diff changeset
1326 }
a61af66fc99e Initial load
duke
parents:
diff changeset
1327 l -= uses_found; // we deleted 1 or more copies of this edge
a61af66fc99e Initial load
duke
parents:
diff changeset
1328 }
a61af66fc99e Initial load
duke
parents:
diff changeset
1329
a61af66fc99e Initial load
duke
parents:
diff changeset
1330 // Now finish up 'r'
a61af66fc99e Initial load
duke
parents:
diff changeset
1331 r->set_req( 1, newuse );
a61af66fc99e Initial load
duke
parents:
diff changeset
1332 r->set_req( 2, use );
a61af66fc99e Initial load
duke
parents:
diff changeset
1333 _igvn.register_new_node_with_optimizer(r);
a61af66fc99e Initial load
duke
parents:
diff changeset
1334 set_loop(r, use_loop);
a61af66fc99e Initial load
duke
parents:
diff changeset
1335 set_idom(r, !side_by_side_idom ? newuse->in(0) : side_by_side_idom, dd_r);
a61af66fc99e Initial load
duke
parents:
diff changeset
1336 } // End of if a loop-exit test
a61af66fc99e Initial load
duke
parents:
diff changeset
1337 }
a61af66fc99e Initial load
duke
parents:
diff changeset
1338 }
a61af66fc99e Initial load
duke
parents:
diff changeset
1339
a61af66fc99e Initial load
duke
parents:
diff changeset
1340 // Step 4: If loop-invariant use is not control, it must be dominated by a
a61af66fc99e Initial load
duke
parents:
diff changeset
1341 // loop exit IfFalse/IfTrue. Find "proper" loop exit. Make a Region
a61af66fc99e Initial load
duke
parents:
diff changeset
1342 // there if needed. Make a Phi there merging old and new used values.
a61af66fc99e Initial load
duke
parents:
diff changeset
1343 Node_List *split_if_set = NULL;
a61af66fc99e Initial load
duke
parents:
diff changeset
1344 Node_List *split_bool_set = NULL;
a61af66fc99e Initial load
duke
parents:
diff changeset
1345 Node_List *split_cex_set = NULL;
a61af66fc99e Initial load
duke
parents:
diff changeset
1346 for( i = 0; i < loop->_body.size(); i++ ) {
a61af66fc99e Initial load
duke
parents:
diff changeset
1347 Node* old = loop->_body.at(i);
a61af66fc99e Initial load
duke
parents:
diff changeset
1348 Node* nnn = old_new[old->_idx];
a61af66fc99e Initial load
duke
parents:
diff changeset
1349 // Copy uses to a worklist, so I can munge the def-use info
a61af66fc99e Initial load
duke
parents:
diff changeset
1350 // with impunity.
a61af66fc99e Initial load
duke
parents:
diff changeset
1351 for (DUIterator_Fast jmax, j = old->fast_outs(jmax); j < jmax; j++)
a61af66fc99e Initial load
duke
parents:
diff changeset
1352 worklist.push(old->fast_out(j));
a61af66fc99e Initial load
duke
parents:
diff changeset
1353
a61af66fc99e Initial load
duke
parents:
diff changeset
1354 while( worklist.size() ) {
a61af66fc99e Initial load
duke
parents:
diff changeset
1355 Node *use = worklist.pop();
a61af66fc99e Initial load
duke
parents:
diff changeset
1356 if (!has_node(use)) continue; // Ignore dead nodes
a61af66fc99e Initial load
duke
parents:
diff changeset
1357 if (use->in(0) == C->top()) continue;
a61af66fc99e Initial load
duke
parents:
diff changeset
1358 IdealLoopTree *use_loop = get_loop( has_ctrl(use) ? get_ctrl(use) : use );
a61af66fc99e Initial load
duke
parents:
diff changeset
1359 // Check for data-use outside of loop - at least one of OLD or USE
a61af66fc99e Initial load
duke
parents:
diff changeset
1360 // must not be a CFG node.
a61af66fc99e Initial load
duke
parents:
diff changeset
1361 if( !loop->is_member( use_loop ) && (!old->is_CFG() || !use->is_CFG())) {
a61af66fc99e Initial load
duke
parents:
diff changeset
1362
a61af66fc99e Initial load
duke
parents:
diff changeset
1363 // If the Data use is an IF, that means we have an IF outside of the
a61af66fc99e Initial load
duke
parents:
diff changeset
1364 // loop that is switching on a condition that is set inside of the
a61af66fc99e Initial load
duke
parents:
diff changeset
1365 // loop. Happens if people set a loop-exit flag; then test the flag
a61af66fc99e Initial load
duke
parents:
diff changeset
1366 // in the loop to break the loop, then test is again outside of the
a61af66fc99e Initial load
duke
parents:
diff changeset
1367 // loop to determine which way the loop exited.
a61af66fc99e Initial load
duke
parents:
diff changeset
1368 if( use->is_If() || use->is_CMove() ) {
a61af66fc99e Initial load
duke
parents:
diff changeset
1369 // Since this code is highly unlikely, we lazily build the worklist
a61af66fc99e Initial load
duke
parents:
diff changeset
1370 // of such Nodes to go split.
a61af66fc99e Initial load
duke
parents:
diff changeset
1371 if( !split_if_set )
a61af66fc99e Initial load
duke
parents:
diff changeset
1372 split_if_set = new Node_List(area);
a61af66fc99e Initial load
duke
parents:
diff changeset
1373 split_if_set->push(use);
a61af66fc99e Initial load
duke
parents:
diff changeset
1374 }
a61af66fc99e Initial load
duke
parents:
diff changeset
1375 if( use->is_Bool() ) {
a61af66fc99e Initial load
duke
parents:
diff changeset
1376 if( !split_bool_set )
a61af66fc99e Initial load
duke
parents:
diff changeset
1377 split_bool_set = new Node_List(area);
a61af66fc99e Initial load
duke
parents:
diff changeset
1378 split_bool_set->push(use);
a61af66fc99e Initial load
duke
parents:
diff changeset
1379 }
a61af66fc99e Initial load
duke
parents:
diff changeset
1380 if( use->Opcode() == Op_CreateEx ) {
a61af66fc99e Initial load
duke
parents:
diff changeset
1381 if( !split_cex_set )
a61af66fc99e Initial load
duke
parents:
diff changeset
1382 split_cex_set = new Node_List(area);
a61af66fc99e Initial load
duke
parents:
diff changeset
1383 split_cex_set->push(use);
a61af66fc99e Initial load
duke
parents:
diff changeset
1384 }
a61af66fc99e Initial load
duke
parents:
diff changeset
1385
a61af66fc99e Initial load
duke
parents:
diff changeset
1386
a61af66fc99e Initial load
duke
parents:
diff changeset
1387 // Get "block" use is in
a61af66fc99e Initial load
duke
parents:
diff changeset
1388 uint idx = 0;
a61af66fc99e Initial load
duke
parents:
diff changeset
1389 while( use->in(idx) != old ) idx++;
a61af66fc99e Initial load
duke
parents:
diff changeset
1390 Node *prev = use->is_CFG() ? use : get_ctrl(use);
a61af66fc99e Initial load
duke
parents:
diff changeset
1391 assert( !loop->is_member( get_loop( prev ) ), "" );
a61af66fc99e Initial load
duke
parents:
diff changeset
1392 Node *cfg = prev->_idx >= new_counter
a61af66fc99e Initial load
duke
parents:
diff changeset
1393 ? prev->in(2)
a61af66fc99e Initial load
duke
parents:
diff changeset
1394 : idom(prev);
a61af66fc99e Initial load
duke
parents:
diff changeset
1395 if( use->is_Phi() ) // Phi use is in prior block
a61af66fc99e Initial load
duke
parents:
diff changeset
1396 cfg = prev->in(idx); // NOT in block of Phi itself
a61af66fc99e Initial load
duke
parents:
diff changeset
1397 if (cfg->is_top()) { // Use is dead?
a61af66fc99e Initial load
duke
parents:
diff changeset
1398 _igvn.hash_delete(use);
a61af66fc99e Initial load
duke
parents:
diff changeset
1399 _igvn._worklist.push(use);
a61af66fc99e Initial load
duke
parents:
diff changeset
1400 use->set_req(idx, C->top());
a61af66fc99e Initial load
duke
parents:
diff changeset
1401 continue;
a61af66fc99e Initial load
duke
parents:
diff changeset
1402 }
a61af66fc99e Initial load
duke
parents:
diff changeset
1403
a61af66fc99e Initial load
duke
parents:
diff changeset
1404 while( !loop->is_member( get_loop( cfg ) ) ) {
a61af66fc99e Initial load
duke
parents:
diff changeset
1405 prev = cfg;
a61af66fc99e Initial load
duke
parents:
diff changeset
1406 cfg = cfg->_idx >= new_counter ? cfg->in(2) : idom(cfg);
a61af66fc99e Initial load
duke
parents:
diff changeset
1407 }
a61af66fc99e Initial load
duke
parents:
diff changeset
1408 // If the use occurs after merging several exits from the loop, then
a61af66fc99e Initial load
duke
parents:
diff changeset
1409 // old value must have dominated all those exits. Since the same old
a61af66fc99e Initial load
duke
parents:
diff changeset
1410 // value was used on all those exits we did not need a Phi at this
a61af66fc99e Initial load
duke
parents:
diff changeset
1411 // merge point. NOW we do need a Phi here. Each loop exit value
a61af66fc99e Initial load
duke
parents:
diff changeset
1412 // is now merged with the peeled body exit; each exit gets its own
a61af66fc99e Initial load
duke
parents:
diff changeset
1413 // private Phi and those Phis need to be merged here.
a61af66fc99e Initial load
duke
parents:
diff changeset
1414 Node *phi;
a61af66fc99e Initial load
duke
parents:
diff changeset
1415 if( prev->is_Region() ) {
a61af66fc99e Initial load
duke
parents:
diff changeset
1416 if( idx == 0 ) { // Updating control edge?
a61af66fc99e Initial load
duke
parents:
diff changeset
1417 phi = prev; // Just use existing control
a61af66fc99e Initial load
duke
parents:
diff changeset
1418 } else { // Else need a new Phi
a61af66fc99e Initial load
duke
parents:
diff changeset
1419 phi = PhiNode::make( prev, old );
a61af66fc99e Initial load
duke
parents:
diff changeset
1420 // Now recursively fix up the new uses of old!
a61af66fc99e Initial load
duke
parents:
diff changeset
1421 for( uint i = 1; i < prev->req(); i++ ) {
a61af66fc99e Initial load
duke
parents:
diff changeset
1422 worklist.push(phi); // Onto worklist once for each 'old' input
a61af66fc99e Initial load
duke
parents:
diff changeset
1423 }
a61af66fc99e Initial load
duke
parents:
diff changeset
1424 }
a61af66fc99e Initial load
duke
parents:
diff changeset
1425 } else {
a61af66fc99e Initial load
duke
parents:
diff changeset
1426 // Get new RegionNode merging old and new loop exits
a61af66fc99e Initial load
duke
parents:
diff changeset
1427 prev = old_new[prev->_idx];
a61af66fc99e Initial load
duke
parents:
diff changeset
1428 assert( prev, "just made this in step 7" );
a61af66fc99e Initial load
duke
parents:
diff changeset
1429 if( idx == 0 ) { // Updating control edge?
a61af66fc99e Initial load
duke
parents:
diff changeset
1430 phi = prev; // Just use existing control
a61af66fc99e Initial load
duke
parents:
diff changeset
1431 } else { // Else need a new Phi
a61af66fc99e Initial load
duke
parents:
diff changeset
1432 // Make a new Phi merging data values properly
a61af66fc99e Initial load
duke
parents:
diff changeset
1433 phi = PhiNode::make( prev, old );
a61af66fc99e Initial load
duke
parents:
diff changeset
1434 phi->set_req( 1, nnn );
a61af66fc99e Initial load
duke
parents:
diff changeset
1435 }
a61af66fc99e Initial load
duke
parents:
diff changeset
1436 }
a61af66fc99e Initial load
duke
parents:
diff changeset
1437 // If inserting a new Phi, check for prior hits
a61af66fc99e Initial load
duke
parents:
diff changeset
1438 if( idx != 0 ) {
a61af66fc99e Initial load
duke
parents:
diff changeset
1439 Node *hit = _igvn.hash_find_insert(phi);
a61af66fc99e Initial load
duke
parents:
diff changeset
1440 if( hit == NULL ) {
a61af66fc99e Initial load
duke
parents:
diff changeset
1441 _igvn.register_new_node_with_optimizer(phi); // Register new phi
a61af66fc99e Initial load
duke
parents:
diff changeset
1442 } else { // or
a61af66fc99e Initial load
duke
parents:
diff changeset
1443 // Remove the new phi from the graph and use the hit
a61af66fc99e Initial load
duke
parents:
diff changeset
1444 _igvn.remove_dead_node(phi);
a61af66fc99e Initial load
duke
parents:
diff changeset
1445 phi = hit; // Use existing phi
a61af66fc99e Initial load
duke
parents:
diff changeset
1446 }
a61af66fc99e Initial load
duke
parents:
diff changeset
1447 set_ctrl(phi, prev);
a61af66fc99e Initial load
duke
parents:
diff changeset
1448 }
a61af66fc99e Initial load
duke
parents:
diff changeset
1449 // Make 'use' use the Phi instead of the old loop body exit value
a61af66fc99e Initial load
duke
parents:
diff changeset
1450 _igvn.hash_delete(use);
a61af66fc99e Initial load
duke
parents:
diff changeset
1451 _igvn._worklist.push(use);
a61af66fc99e Initial load
duke
parents:
diff changeset
1452 use->set_req(idx, phi);
a61af66fc99e Initial load
duke
parents:
diff changeset
1453 if( use->_idx >= new_counter ) { // If updating new phis
a61af66fc99e Initial load
duke
parents:
diff changeset
1454 // Not needed for correctness, but prevents a weak assert
a61af66fc99e Initial load
duke
parents:
diff changeset
1455 // in AddPNode from tripping (when we end up with different
a61af66fc99e Initial load
duke
parents:
diff changeset
1456 // base & derived Phis that will become the same after
a61af66fc99e Initial load
duke
parents:
diff changeset
1457 // IGVN does CSE).
a61af66fc99e Initial load
duke
parents:
diff changeset
1458 Node *hit = _igvn.hash_find_insert(use);
a61af66fc99e Initial load
duke
parents:
diff changeset
1459 if( hit ) // Go ahead and re-hash for hits.
1621
6027dddc26c6 6677629: PhaseIterGVN::subsume_node() should call hash_delete() and add_users_to_worklist()
kvn
parents: 1552
diff changeset
1460 _igvn.replace_node( use, hit );
0
a61af66fc99e Initial load
duke
parents:
diff changeset
1461 }
a61af66fc99e Initial load
duke
parents:
diff changeset
1462
a61af66fc99e Initial load
duke
parents:
diff changeset
1463 // If 'use' was in the loop-exit block, it now needs to be sunk
a61af66fc99e Initial load
duke
parents:
diff changeset
1464 // below the post-loop merge point.
a61af66fc99e Initial load
duke
parents:
diff changeset
1465 sink_use( use, prev );
a61af66fc99e Initial load
duke
parents:
diff changeset
1466 }
a61af66fc99e Initial load
duke
parents:
diff changeset
1467 }
a61af66fc99e Initial load
duke
parents:
diff changeset
1468 }
a61af66fc99e Initial load
duke
parents:
diff changeset
1469
a61af66fc99e Initial load
duke
parents:
diff changeset
1470 // Check for IFs that need splitting/cloning. Happens if an IF outside of
a61af66fc99e Initial load
duke
parents:
diff changeset
1471 // the loop uses a condition set in the loop. The original IF probably
a61af66fc99e Initial load
duke
parents:
diff changeset
1472 // takes control from one or more OLD Regions (which in turn get from NEW
a61af66fc99e Initial load
duke
parents:
diff changeset
1473 // Regions). In any case, there will be a set of Phis for each merge point
a61af66fc99e Initial load
duke
parents:
diff changeset
1474 // from the IF up to where the original BOOL def exists the loop.
a61af66fc99e Initial load
duke
parents:
diff changeset
1475 if( split_if_set ) {
a61af66fc99e Initial load
duke
parents:
diff changeset
1476 while( split_if_set->size() ) {
a61af66fc99e Initial load
duke
parents:
diff changeset
1477 Node *iff = split_if_set->pop();
a61af66fc99e Initial load
duke
parents:
diff changeset
1478 if( iff->in(1)->is_Phi() ) {
a61af66fc99e Initial load
duke
parents:
diff changeset
1479 BoolNode *b = clone_iff( iff->in(1)->as_Phi(), loop );
a61af66fc99e Initial load
duke
parents:
diff changeset
1480 _igvn.hash_delete(iff);
a61af66fc99e Initial load
duke
parents:
diff changeset
1481 _igvn._worklist.push(iff);
a61af66fc99e Initial load
duke
parents:
diff changeset
1482 iff->set_req(1, b);
a61af66fc99e Initial load
duke
parents:
diff changeset
1483 }
a61af66fc99e Initial load
duke
parents:
diff changeset
1484 }
a61af66fc99e Initial load
duke
parents:
diff changeset
1485 }
a61af66fc99e Initial load
duke
parents:
diff changeset
1486 if( split_bool_set ) {
a61af66fc99e Initial load
duke
parents:
diff changeset
1487 while( split_bool_set->size() ) {
a61af66fc99e Initial load
duke
parents:
diff changeset
1488 Node *b = split_bool_set->pop();
a61af66fc99e Initial load
duke
parents:
diff changeset
1489 Node *phi = b->in(1);
a61af66fc99e Initial load
duke
parents:
diff changeset
1490 assert( phi->is_Phi(), "" );
a61af66fc99e Initial load
duke
parents:
diff changeset
1491 CmpNode *cmp = clone_bool( (PhiNode*)phi, loop );
a61af66fc99e Initial load
duke
parents:
diff changeset
1492 _igvn.hash_delete(b);
a61af66fc99e Initial load
duke
parents:
diff changeset
1493 _igvn._worklist.push(b);
a61af66fc99e Initial load
duke
parents:
diff changeset
1494 b->set_req(1, cmp);
a61af66fc99e Initial load
duke
parents:
diff changeset
1495 }
a61af66fc99e Initial load
duke
parents:
diff changeset
1496 }
a61af66fc99e Initial load
duke
parents:
diff changeset
1497 if( split_cex_set ) {
a61af66fc99e Initial load
duke
parents:
diff changeset
1498 while( split_cex_set->size() ) {
a61af66fc99e Initial load
duke
parents:
diff changeset
1499 Node *b = split_cex_set->pop();
a61af66fc99e Initial load
duke
parents:
diff changeset
1500 assert( b->in(0)->is_Region(), "" );
a61af66fc99e Initial load
duke
parents:
diff changeset
1501 assert( b->in(1)->is_Phi(), "" );
a61af66fc99e Initial load
duke
parents:
diff changeset
1502 assert( b->in(0)->in(0) == b->in(1)->in(0), "" );
a61af66fc99e Initial load
duke
parents:
diff changeset
1503 split_up( b, b->in(0), NULL );
a61af66fc99e Initial load
duke
parents:
diff changeset
1504 }
a61af66fc99e Initial load
duke
parents:
diff changeset
1505 }
a61af66fc99e Initial load
duke
parents:
diff changeset
1506
a61af66fc99e Initial load
duke
parents:
diff changeset
1507 }
a61af66fc99e Initial load
duke
parents:
diff changeset
1508
a61af66fc99e Initial load
duke
parents:
diff changeset
1509
a61af66fc99e Initial load
duke
parents:
diff changeset
1510 //---------------------- stride_of_possible_iv -------------------------------------
a61af66fc99e Initial load
duke
parents:
diff changeset
1511 // Looks for an iff/bool/comp with one operand of the compare
a61af66fc99e Initial load
duke
parents:
diff changeset
1512 // being a cycle involving an add and a phi,
a61af66fc99e Initial load
duke
parents:
diff changeset
1513 // with an optional truncation (left-shift followed by a right-shift)
a61af66fc99e Initial load
duke
parents:
diff changeset
1514 // of the add. Returns zero if not an iv.
a61af66fc99e Initial load
duke
parents:
diff changeset
1515 int PhaseIdealLoop::stride_of_possible_iv(Node* iff) {
a61af66fc99e Initial load
duke
parents:
diff changeset
1516 Node* trunc1 = NULL;
a61af66fc99e Initial load
duke
parents:
diff changeset
1517 Node* trunc2 = NULL;
a61af66fc99e Initial load
duke
parents:
diff changeset
1518 const TypeInt* ttype = NULL;
a61af66fc99e Initial load
duke
parents:
diff changeset
1519 if (!iff->is_If() || iff->in(1) == NULL || !iff->in(1)->is_Bool()) {
a61af66fc99e Initial load
duke
parents:
diff changeset
1520 return 0;
a61af66fc99e Initial load
duke
parents:
diff changeset
1521 }
a61af66fc99e Initial load
duke
parents:
diff changeset
1522 BoolNode* bl = iff->in(1)->as_Bool();
a61af66fc99e Initial load
duke
parents:
diff changeset
1523 Node* cmp = bl->in(1);
a61af66fc99e Initial load
duke
parents:
diff changeset
1524 if (!cmp || cmp->Opcode() != Op_CmpI && cmp->Opcode() != Op_CmpU) {
a61af66fc99e Initial load
duke
parents:
diff changeset
1525 return 0;
a61af66fc99e Initial load
duke
parents:
diff changeset
1526 }
a61af66fc99e Initial load
duke
parents:
diff changeset
1527 // Must have an invariant operand
a61af66fc99e Initial load
duke
parents:
diff changeset
1528 if (is_member(get_loop(iff), get_ctrl(cmp->in(2)))) {
a61af66fc99e Initial load
duke
parents:
diff changeset
1529 return 0;
a61af66fc99e Initial load
duke
parents:
diff changeset
1530 }
a61af66fc99e Initial load
duke
parents:
diff changeset
1531 Node* add2 = NULL;
a61af66fc99e Initial load
duke
parents:
diff changeset
1532 Node* cmp1 = cmp->in(1);
a61af66fc99e Initial load
duke
parents:
diff changeset
1533 if (cmp1->is_Phi()) {
a61af66fc99e Initial load
duke
parents:
diff changeset
1534 // (If (Bool (CmpX phi:(Phi ...(Optional-trunc(AddI phi add2))) )))
a61af66fc99e Initial load
duke
parents:
diff changeset
1535 Node* phi = cmp1;
a61af66fc99e Initial load
duke
parents:
diff changeset
1536 for (uint i = 1; i < phi->req(); i++) {
a61af66fc99e Initial load
duke
parents:
diff changeset
1537 Node* in = phi->in(i);
a61af66fc99e Initial load
duke
parents:
diff changeset
1538 Node* add = CountedLoopNode::match_incr_with_optional_truncation(in,
a61af66fc99e Initial load
duke
parents:
diff changeset
1539 &trunc1, &trunc2, &ttype);
a61af66fc99e Initial load
duke
parents:
diff changeset
1540 if (add && add->in(1) == phi) {
a61af66fc99e Initial load
duke
parents:
diff changeset
1541 add2 = add->in(2);
a61af66fc99e Initial load
duke
parents:
diff changeset
1542 break;
a61af66fc99e Initial load
duke
parents:
diff changeset
1543 }
a61af66fc99e Initial load
duke
parents:
diff changeset
1544 }
a61af66fc99e Initial load
duke
parents:
diff changeset
1545 } else {
a61af66fc99e Initial load
duke
parents:
diff changeset
1546 // (If (Bool (CmpX addtrunc:(Optional-trunc((AddI (Phi ...addtrunc...) add2)) )))
a61af66fc99e Initial load
duke
parents:
diff changeset
1547 Node* addtrunc = cmp1;
a61af66fc99e Initial load
duke
parents:
diff changeset
1548 Node* add = CountedLoopNode::match_incr_with_optional_truncation(addtrunc,
a61af66fc99e Initial load
duke
parents:
diff changeset
1549 &trunc1, &trunc2, &ttype);
a61af66fc99e Initial load
duke
parents:
diff changeset
1550 if (add && add->in(1)->is_Phi()) {
a61af66fc99e Initial load
duke
parents:
diff changeset
1551 Node* phi = add->in(1);
a61af66fc99e Initial load
duke
parents:
diff changeset
1552 for (uint i = 1; i < phi->req(); i++) {
a61af66fc99e Initial load
duke
parents:
diff changeset
1553 if (phi->in(i) == addtrunc) {
a61af66fc99e Initial load
duke
parents:
diff changeset
1554 add2 = add->in(2);
a61af66fc99e Initial load
duke
parents:
diff changeset
1555 break;
a61af66fc99e Initial load
duke
parents:
diff changeset
1556 }
a61af66fc99e Initial load
duke
parents:
diff changeset
1557 }
a61af66fc99e Initial load
duke
parents:
diff changeset
1558 }
a61af66fc99e Initial load
duke
parents:
diff changeset
1559 }
a61af66fc99e Initial load
duke
parents:
diff changeset
1560 if (add2 != NULL) {
a61af66fc99e Initial load
duke
parents:
diff changeset
1561 const TypeInt* add2t = _igvn.type(add2)->is_int();
a61af66fc99e Initial load
duke
parents:
diff changeset
1562 if (add2t->is_con()) {
a61af66fc99e Initial load
duke
parents:
diff changeset
1563 return add2t->get_con();
a61af66fc99e Initial load
duke
parents:
diff changeset
1564 }
a61af66fc99e Initial load
duke
parents:
diff changeset
1565 }
a61af66fc99e Initial load
duke
parents:
diff changeset
1566 return 0;
a61af66fc99e Initial load
duke
parents:
diff changeset
1567 }
a61af66fc99e Initial load
duke
parents:
diff changeset
1568
a61af66fc99e Initial load
duke
parents:
diff changeset
1569
a61af66fc99e Initial load
duke
parents:
diff changeset
1570 //---------------------- stay_in_loop -------------------------------------
a61af66fc99e Initial load
duke
parents:
diff changeset
1571 // Return the (unique) control output node that's in the loop (if it exists.)
a61af66fc99e Initial load
duke
parents:
diff changeset
1572 Node* PhaseIdealLoop::stay_in_loop( Node* n, IdealLoopTree *loop) {
a61af66fc99e Initial load
duke
parents:
diff changeset
1573 Node* unique = NULL;
a61af66fc99e Initial load
duke
parents:
diff changeset
1574 if (!n) return NULL;
a61af66fc99e Initial load
duke
parents:
diff changeset
1575 for (DUIterator_Fast imax, i = n->fast_outs(imax); i < imax; i++) {
a61af66fc99e Initial load
duke
parents:
diff changeset
1576 Node* use = n->fast_out(i);
a61af66fc99e Initial load
duke
parents:
diff changeset
1577 if (!has_ctrl(use) && loop->is_member(get_loop(use))) {
a61af66fc99e Initial load
duke
parents:
diff changeset
1578 if (unique != NULL) {
a61af66fc99e Initial load
duke
parents:
diff changeset
1579 return NULL;
a61af66fc99e Initial load
duke
parents:
diff changeset
1580 }
a61af66fc99e Initial load
duke
parents:
diff changeset
1581 unique = use;
a61af66fc99e Initial load
duke
parents:
diff changeset
1582 }
a61af66fc99e Initial load
duke
parents:
diff changeset
1583 }
a61af66fc99e Initial load
duke
parents:
diff changeset
1584 return unique;
a61af66fc99e Initial load
duke
parents:
diff changeset
1585 }
a61af66fc99e Initial load
duke
parents:
diff changeset
1586
a61af66fc99e Initial load
duke
parents:
diff changeset
1587 //------------------------------ register_node -------------------------------------
a61af66fc99e Initial load
duke
parents:
diff changeset
1588 // Utility to register node "n" with PhaseIdealLoop
a61af66fc99e Initial load
duke
parents:
diff changeset
1589 void PhaseIdealLoop::register_node(Node* n, IdealLoopTree *loop, Node* pred, int ddepth) {
a61af66fc99e Initial load
duke
parents:
diff changeset
1590 _igvn.register_new_node_with_optimizer(n);
a61af66fc99e Initial load
duke
parents:
diff changeset
1591 loop->_body.push(n);
a61af66fc99e Initial load
duke
parents:
diff changeset
1592 if (n->is_CFG()) {
a61af66fc99e Initial load
duke
parents:
diff changeset
1593 set_loop(n, loop);
a61af66fc99e Initial load
duke
parents:
diff changeset
1594 set_idom(n, pred, ddepth);
a61af66fc99e Initial load
duke
parents:
diff changeset
1595 } else {
a61af66fc99e Initial load
duke
parents:
diff changeset
1596 set_ctrl(n, pred);
a61af66fc99e Initial load
duke
parents:
diff changeset
1597 }
a61af66fc99e Initial load
duke
parents:
diff changeset
1598 }
a61af66fc99e Initial load
duke
parents:
diff changeset
1599
a61af66fc99e Initial load
duke
parents:
diff changeset
1600 //------------------------------ proj_clone -------------------------------------
a61af66fc99e Initial load
duke
parents:
diff changeset
1601 // Utility to create an if-projection
a61af66fc99e Initial load
duke
parents:
diff changeset
1602 ProjNode* PhaseIdealLoop::proj_clone(ProjNode* p, IfNode* iff) {
a61af66fc99e Initial load
duke
parents:
diff changeset
1603 ProjNode* c = p->clone()->as_Proj();
a61af66fc99e Initial load
duke
parents:
diff changeset
1604 c->set_req(0, iff);
a61af66fc99e Initial load
duke
parents:
diff changeset
1605 return c;
a61af66fc99e Initial load
duke
parents:
diff changeset
1606 }
a61af66fc99e Initial load
duke
parents:
diff changeset
1607
a61af66fc99e Initial load
duke
parents:
diff changeset
1608 //------------------------------ short_circuit_if -------------------------------------
a61af66fc99e Initial load
duke
parents:
diff changeset
1609 // Force the iff control output to be the live_proj
a61af66fc99e Initial load
duke
parents:
diff changeset
1610 Node* PhaseIdealLoop::short_circuit_if(IfNode* iff, ProjNode* live_proj) {
a61af66fc99e Initial load
duke
parents:
diff changeset
1611 int proj_con = live_proj->_con;
a61af66fc99e Initial load
duke
parents:
diff changeset
1612 assert(proj_con == 0 || proj_con == 1, "false or true projection");
a61af66fc99e Initial load
duke
parents:
diff changeset
1613 Node *con = _igvn.intcon(proj_con);
a61af66fc99e Initial load
duke
parents:
diff changeset
1614 set_ctrl(con, C->root());
a61af66fc99e Initial load
duke
parents:
diff changeset
1615 if (iff) {
a61af66fc99e Initial load
duke
parents:
diff changeset
1616 iff->set_req(1, con);
a61af66fc99e Initial load
duke
parents:
diff changeset
1617 }
a61af66fc99e Initial load
duke
parents:
diff changeset
1618 return con;
a61af66fc99e Initial load
duke
parents:
diff changeset
1619 }
a61af66fc99e Initial load
duke
parents:
diff changeset
1620
a61af66fc99e Initial load
duke
parents:
diff changeset
1621 //------------------------------ insert_if_before_proj -------------------------------------
a61af66fc99e Initial load
duke
parents:
diff changeset
1622 // Insert a new if before an if projection (* - new node)
a61af66fc99e Initial load
duke
parents:
diff changeset
1623 //
a61af66fc99e Initial load
duke
parents:
diff changeset
1624 // before
a61af66fc99e Initial load
duke
parents:
diff changeset
1625 // if(test)
a61af66fc99e Initial load
duke
parents:
diff changeset
1626 // / \
a61af66fc99e Initial load
duke
parents:
diff changeset
1627 // v v
a61af66fc99e Initial load
duke
parents:
diff changeset
1628 // other-proj proj (arg)
a61af66fc99e Initial load
duke
parents:
diff changeset
1629 //
a61af66fc99e Initial load
duke
parents:
diff changeset
1630 // after
a61af66fc99e Initial load
duke
parents:
diff changeset
1631 // if(test)
a61af66fc99e Initial load
duke
parents:
diff changeset
1632 // / \
a61af66fc99e Initial load
duke
parents:
diff changeset
1633 // / v
a61af66fc99e Initial load
duke
parents:
diff changeset
1634 // | * proj-clone
a61af66fc99e Initial load
duke
parents:
diff changeset
1635 // v |
a61af66fc99e Initial load
duke
parents:
diff changeset
1636 // other-proj v
a61af66fc99e Initial load
duke
parents:
diff changeset
1637 // * new_if(relop(cmp[IU](left,right)))
a61af66fc99e Initial load
duke
parents:
diff changeset
1638 // / \
a61af66fc99e Initial load
duke
parents:
diff changeset
1639 // v v
a61af66fc99e Initial load
duke
parents:
diff changeset
1640 // * new-proj proj
a61af66fc99e Initial load
duke
parents:
diff changeset
1641 // (returned)
a61af66fc99e Initial load
duke
parents:
diff changeset
1642 //
a61af66fc99e Initial load
duke
parents:
diff changeset
1643 ProjNode* PhaseIdealLoop::insert_if_before_proj(Node* left, bool Signed, BoolTest::mask relop, Node* right, ProjNode* proj) {
a61af66fc99e Initial load
duke
parents:
diff changeset
1644 IfNode* iff = proj->in(0)->as_If();
a61af66fc99e Initial load
duke
parents:
diff changeset
1645 IdealLoopTree *loop = get_loop(proj);
a61af66fc99e Initial load
duke
parents:
diff changeset
1646 ProjNode *other_proj = iff->proj_out(!proj->is_IfTrue())->as_Proj();
a61af66fc99e Initial load
duke
parents:
diff changeset
1647 int ddepth = dom_depth(proj);
a61af66fc99e Initial load
duke
parents:
diff changeset
1648
a61af66fc99e Initial load
duke
parents:
diff changeset
1649 _igvn.hash_delete(iff);
a61af66fc99e Initial load
duke
parents:
diff changeset
1650 _igvn._worklist.push(iff);
a61af66fc99e Initial load
duke
parents:
diff changeset
1651 _igvn.hash_delete(proj);
a61af66fc99e Initial load
duke
parents:
diff changeset
1652 _igvn._worklist.push(proj);
a61af66fc99e Initial load
duke
parents:
diff changeset
1653
a61af66fc99e Initial load
duke
parents:
diff changeset
1654 proj->set_req(0, NULL); // temporary disconnect
a61af66fc99e Initial load
duke
parents:
diff changeset
1655 ProjNode* proj2 = proj_clone(proj, iff);
a61af66fc99e Initial load
duke
parents:
diff changeset
1656 register_node(proj2, loop, iff, ddepth);
a61af66fc99e Initial load
duke
parents:
diff changeset
1657
a61af66fc99e Initial load
duke
parents:
diff changeset
1658 Node* cmp = Signed ? (Node*) new (C,3)CmpINode(left, right) : (Node*) new (C,3)CmpUNode(left, right);
a61af66fc99e Initial load
duke
parents:
diff changeset
1659 register_node(cmp, loop, proj2, ddepth);
a61af66fc99e Initial load
duke
parents:
diff changeset
1660
a61af66fc99e Initial load
duke
parents:
diff changeset
1661 BoolNode* bol = new (C,2)BoolNode(cmp, relop);
a61af66fc99e Initial load
duke
parents:
diff changeset
1662 register_node(bol, loop, proj2, ddepth);
a61af66fc99e Initial load
duke
parents:
diff changeset
1663
a61af66fc99e Initial load
duke
parents:
diff changeset
1664 IfNode* new_if = new (C,2)IfNode(proj2, bol, iff->_prob, iff->_fcnt);
a61af66fc99e Initial load
duke
parents:
diff changeset
1665 register_node(new_if, loop, proj2, ddepth);
a61af66fc99e Initial load
duke
parents:
diff changeset
1666
a61af66fc99e Initial load
duke
parents:
diff changeset
1667 proj->set_req(0, new_if); // reattach
a61af66fc99e Initial load
duke
parents:
diff changeset
1668 set_idom(proj, new_if, ddepth);
a61af66fc99e Initial load
duke
parents:
diff changeset
1669
a61af66fc99e Initial load
duke
parents:
diff changeset
1670 ProjNode* new_exit = proj_clone(other_proj, new_if)->as_Proj();
a61af66fc99e Initial load
duke
parents:
diff changeset
1671 register_node(new_exit, get_loop(other_proj), new_if, ddepth);
a61af66fc99e Initial load
duke
parents:
diff changeset
1672
a61af66fc99e Initial load
duke
parents:
diff changeset
1673 return new_exit;
a61af66fc99e Initial load
duke
parents:
diff changeset
1674 }
a61af66fc99e Initial load
duke
parents:
diff changeset
1675
a61af66fc99e Initial load
duke
parents:
diff changeset
1676 //------------------------------ insert_region_before_proj -------------------------------------
a61af66fc99e Initial load
duke
parents:
diff changeset
1677 // Insert a region before an if projection (* - new node)
a61af66fc99e Initial load
duke
parents:
diff changeset
1678 //
a61af66fc99e Initial load
duke
parents:
diff changeset
1679 // before
a61af66fc99e Initial load
duke
parents:
diff changeset
1680 // if(test)
a61af66fc99e Initial load
duke
parents:
diff changeset
1681 // / |
a61af66fc99e Initial load
duke
parents:
diff changeset
1682 // v |
a61af66fc99e Initial load
duke
parents:
diff changeset
1683 // proj v
a61af66fc99e Initial load
duke
parents:
diff changeset
1684 // other-proj
a61af66fc99e Initial load
duke
parents:
diff changeset
1685 //
a61af66fc99e Initial load
duke
parents:
diff changeset
1686 // after
a61af66fc99e Initial load
duke
parents:
diff changeset
1687 // if(test)
a61af66fc99e Initial load
duke
parents:
diff changeset
1688 // / |
a61af66fc99e Initial load
duke
parents:
diff changeset
1689 // v |
a61af66fc99e Initial load
duke
parents:
diff changeset
1690 // * proj-clone v
a61af66fc99e Initial load
duke
parents:
diff changeset
1691 // | other-proj
a61af66fc99e Initial load
duke
parents:
diff changeset
1692 // v
a61af66fc99e Initial load
duke
parents:
diff changeset
1693 // * new-region
a61af66fc99e Initial load
duke
parents:
diff changeset
1694 // |
a61af66fc99e Initial load
duke
parents:
diff changeset
1695 // v
a61af66fc99e Initial load
duke
parents:
diff changeset
1696 // * dum_if
a61af66fc99e Initial load
duke
parents:
diff changeset
1697 // / \
a61af66fc99e Initial load
duke
parents:
diff changeset
1698 // v \
a61af66fc99e Initial load
duke
parents:
diff changeset
1699 // * dum-proj v
a61af66fc99e Initial load
duke
parents:
diff changeset
1700 // proj
a61af66fc99e Initial load
duke
parents:
diff changeset
1701 //
a61af66fc99e Initial load
duke
parents:
diff changeset
1702 RegionNode* PhaseIdealLoop::insert_region_before_proj(ProjNode* proj) {
a61af66fc99e Initial load
duke
parents:
diff changeset
1703 IfNode* iff = proj->in(0)->as_If();
a61af66fc99e Initial load
duke
parents:
diff changeset
1704 IdealLoopTree *loop = get_loop(proj);
a61af66fc99e Initial load
duke
parents:
diff changeset
1705 ProjNode *other_proj = iff->proj_out(!proj->is_IfTrue())->as_Proj();
a61af66fc99e Initial load
duke
parents:
diff changeset
1706 int ddepth = dom_depth(proj);
a61af66fc99e Initial load
duke
parents:
diff changeset
1707
a61af66fc99e Initial load
duke
parents:
diff changeset
1708 _igvn.hash_delete(iff);
a61af66fc99e Initial load
duke
parents:
diff changeset
1709 _igvn._worklist.push(iff);
a61af66fc99e Initial load
duke
parents:
diff changeset
1710 _igvn.hash_delete(proj);
a61af66fc99e Initial load
duke
parents:
diff changeset
1711 _igvn._worklist.push(proj);
a61af66fc99e Initial load
duke
parents:
diff changeset
1712
a61af66fc99e Initial load
duke
parents:
diff changeset
1713 proj->set_req(0, NULL); // temporary disconnect
a61af66fc99e Initial load
duke
parents:
diff changeset
1714 ProjNode* proj2 = proj_clone(proj, iff);
a61af66fc99e Initial load
duke
parents:
diff changeset
1715 register_node(proj2, loop, iff, ddepth);
a61af66fc99e Initial load
duke
parents:
diff changeset
1716
a61af66fc99e Initial load
duke
parents:
diff changeset
1717 RegionNode* reg = new (C,2)RegionNode(2);
a61af66fc99e Initial load
duke
parents:
diff changeset
1718 reg->set_req(1, proj2);
a61af66fc99e Initial load
duke
parents:
diff changeset
1719 register_node(reg, loop, iff, ddepth);
a61af66fc99e Initial load
duke
parents:
diff changeset
1720
a61af66fc99e Initial load
duke
parents:
diff changeset
1721 IfNode* dum_if = new (C,2)IfNode(reg, short_circuit_if(NULL, proj), iff->_prob, iff->_fcnt);
a61af66fc99e Initial load
duke
parents:
diff changeset
1722 register_node(dum_if, loop, reg, ddepth);
a61af66fc99e Initial load
duke
parents:
diff changeset
1723
a61af66fc99e Initial load
duke
parents:
diff changeset
1724 proj->set_req(0, dum_if); // reattach
a61af66fc99e Initial load
duke
parents:
diff changeset
1725 set_idom(proj, dum_if, ddepth);
a61af66fc99e Initial load
duke
parents:
diff changeset
1726
a61af66fc99e Initial load
duke
parents:
diff changeset
1727 ProjNode* dum_proj = proj_clone(other_proj, dum_if);
a61af66fc99e Initial load
duke
parents:
diff changeset
1728 register_node(dum_proj, loop, dum_if, ddepth);
a61af66fc99e Initial load
duke
parents:
diff changeset
1729
a61af66fc99e Initial load
duke
parents:
diff changeset
1730 return reg;
a61af66fc99e Initial load
duke
parents:
diff changeset
1731 }
a61af66fc99e Initial load
duke
parents:
diff changeset
1732
a61af66fc99e Initial load
duke
parents:
diff changeset
1733 //------------------------------ insert_cmpi_loop_exit -------------------------------------
a61af66fc99e Initial load
duke
parents:
diff changeset
1734 // Clone a signed compare loop exit from an unsigned compare and
a61af66fc99e Initial load
duke
parents:
diff changeset
1735 // insert it before the unsigned cmp on the stay-in-loop path.
a61af66fc99e Initial load
duke
parents:
diff changeset
1736 // All new nodes inserted in the dominator tree between the original
a61af66fc99e Initial load
duke
parents:
diff changeset
1737 // if and it's projections. The original if test is replaced with
a61af66fc99e Initial load
duke
parents:
diff changeset
1738 // a constant to force the stay-in-loop path.
a61af66fc99e Initial load
duke
parents:
diff changeset
1739 //
a61af66fc99e Initial load
duke
parents:
diff changeset
1740 // This is done to make sure that the original if and it's projections
a61af66fc99e Initial load
duke
parents:
diff changeset
1741 // still dominate the same set of control nodes, that the ctrl() relation
a61af66fc99e Initial load
duke
parents:
diff changeset
1742 // from data nodes to them is preserved, and that their loop nesting is
a61af66fc99e Initial load
duke
parents:
diff changeset
1743 // preserved.
a61af66fc99e Initial load
duke
parents:
diff changeset
1744 //
a61af66fc99e Initial load
duke
parents:
diff changeset
1745 // before
a61af66fc99e Initial load
duke
parents:
diff changeset
1746 // if(i <u limit) unsigned compare loop exit
a61af66fc99e Initial load
duke
parents:
diff changeset
1747 // / |
a61af66fc99e Initial load
duke
parents:
diff changeset
1748 // v v
a61af66fc99e Initial load
duke
parents:
diff changeset
1749 // exit-proj stay-in-loop-proj
a61af66fc99e Initial load
duke
parents:
diff changeset
1750 //
a61af66fc99e Initial load
duke
parents:
diff changeset
1751 // after
a61af66fc99e Initial load
duke
parents:
diff changeset
1752 // if(stay-in-loop-const) original if
a61af66fc99e Initial load
duke
parents:
diff changeset
1753 // / |
a61af66fc99e Initial load
duke
parents:
diff changeset
1754 // / v
a61af66fc99e Initial load
duke
parents:
diff changeset
1755 // / if(i < limit) new signed test
a61af66fc99e Initial load
duke
parents:
diff changeset
1756 // / / |
a61af66fc99e Initial load
duke
parents:
diff changeset
1757 // / / v
a61af66fc99e Initial load
duke
parents:
diff changeset
1758 // / / if(i <u limit) new cloned unsigned test
a61af66fc99e Initial load
duke
parents:
diff changeset
1759 // / / / |
a61af66fc99e Initial load
duke
parents:
diff changeset
1760 // v v v |
a61af66fc99e Initial load
duke
parents:
diff changeset
1761 // region |
a61af66fc99e Initial load
duke
parents:
diff changeset
1762 // | |
a61af66fc99e Initial load
duke
parents:
diff changeset
1763 // dum-if |
a61af66fc99e Initial load
duke
parents:
diff changeset
1764 // / | |
a61af66fc99e Initial load
duke
parents:
diff changeset
1765 // ether | |
a61af66fc99e Initial load
duke
parents:
diff changeset
1766 // v v
a61af66fc99e Initial load
duke
parents:
diff changeset
1767 // exit-proj stay-in-loop-proj
a61af66fc99e Initial load
duke
parents:
diff changeset
1768 //
a61af66fc99e Initial load
duke
parents:
diff changeset
1769 IfNode* PhaseIdealLoop::insert_cmpi_loop_exit(IfNode* if_cmpu, IdealLoopTree *loop) {
a61af66fc99e Initial load
duke
parents:
diff changeset
1770 const bool Signed = true;
a61af66fc99e Initial load
duke
parents:
diff changeset
1771 const bool Unsigned = false;
a61af66fc99e Initial load
duke
parents:
diff changeset
1772
a61af66fc99e Initial load
duke
parents:
diff changeset
1773 BoolNode* bol = if_cmpu->in(1)->as_Bool();
a61af66fc99e Initial load
duke
parents:
diff changeset
1774 if (bol->_test._test != BoolTest::lt) return NULL;
a61af66fc99e Initial load
duke
parents:
diff changeset
1775 CmpNode* cmpu = bol->in(1)->as_Cmp();
a61af66fc99e Initial load
duke
parents:
diff changeset
1776 if (cmpu->Opcode() != Op_CmpU) return NULL;
a61af66fc99e Initial load
duke
parents:
diff changeset
1777 int stride = stride_of_possible_iv(if_cmpu);
a61af66fc99e Initial load
duke
parents:
diff changeset
1778 if (stride == 0) return NULL;
a61af66fc99e Initial load
duke
parents:
diff changeset
1779
a61af66fc99e Initial load
duke
parents:
diff changeset
1780 ProjNode* lp_continue = stay_in_loop(if_cmpu, loop)->as_Proj();
a61af66fc99e Initial load
duke
parents:
diff changeset
1781 ProjNode* lp_exit = if_cmpu->proj_out(!lp_continue->is_IfTrue())->as_Proj();
a61af66fc99e Initial load
duke
parents:
diff changeset
1782
a61af66fc99e Initial load
duke
parents:
diff changeset
1783 Node* limit = NULL;
a61af66fc99e Initial load
duke
parents:
diff changeset
1784 if (stride > 0) {
a61af66fc99e Initial load
duke
parents:
diff changeset
1785 limit = cmpu->in(2);
a61af66fc99e Initial load
duke
parents:
diff changeset
1786 } else {
a61af66fc99e Initial load
duke
parents:
diff changeset
1787 limit = _igvn.makecon(TypeInt::ZERO);
a61af66fc99e Initial load
duke
parents:
diff changeset
1788 set_ctrl(limit, C->root());
a61af66fc99e Initial load
duke
parents:
diff changeset
1789 }
a61af66fc99e Initial load
duke
parents:
diff changeset
1790 // Create a new region on the exit path
a61af66fc99e Initial load
duke
parents:
diff changeset
1791 RegionNode* reg = insert_region_before_proj(lp_exit);
a61af66fc99e Initial load
duke
parents:
diff changeset
1792
a61af66fc99e Initial load
duke
parents:
diff changeset
1793 // Clone the if-cmpu-true-false using a signed compare
a61af66fc99e Initial load
duke
parents:
diff changeset
1794 BoolTest::mask rel_i = stride > 0 ? bol->_test._test : BoolTest::ge;
a61af66fc99e Initial load
duke
parents:
diff changeset
1795 ProjNode* cmpi_exit = insert_if_before_proj(cmpu->in(1), Signed, rel_i, limit, lp_continue);
a61af66fc99e Initial load
duke
parents:
diff changeset
1796 reg->add_req(cmpi_exit);
a61af66fc99e Initial load
duke
parents:
diff changeset
1797
a61af66fc99e Initial load
duke
parents:
diff changeset
1798 // Clone the if-cmpu-true-false
a61af66fc99e Initial load
duke
parents:
diff changeset
1799 BoolTest::mask rel_u = bol->_test._test;
a61af66fc99e Initial load
duke
parents:
diff changeset
1800 ProjNode* cmpu_exit = insert_if_before_proj(cmpu->in(1), Unsigned, rel_u, cmpu->in(2), lp_continue);
a61af66fc99e Initial load
duke
parents:
diff changeset
1801 reg->add_req(cmpu_exit);
a61af66fc99e Initial load
duke
parents:
diff changeset
1802
a61af66fc99e Initial load
duke
parents:
diff changeset
1803 // Force original if to stay in loop.
a61af66fc99e Initial load
duke
parents:
diff changeset
1804 short_circuit_if(if_cmpu, lp_continue);
a61af66fc99e Initial load
duke
parents:
diff changeset
1805
a61af66fc99e Initial load
duke
parents:
diff changeset
1806 return cmpi_exit->in(0)->as_If();
a61af66fc99e Initial load
duke
parents:
diff changeset
1807 }
a61af66fc99e Initial load
duke
parents:
diff changeset
1808
a61af66fc99e Initial load
duke
parents:
diff changeset
1809 //------------------------------ remove_cmpi_loop_exit -------------------------------------
a61af66fc99e Initial load
duke
parents:
diff changeset
1810 // Remove a previously inserted signed compare loop exit.
a61af66fc99e Initial load
duke
parents:
diff changeset
1811 void PhaseIdealLoop::remove_cmpi_loop_exit(IfNode* if_cmp, IdealLoopTree *loop) {
a61af66fc99e Initial load
duke
parents:
diff changeset
1812 Node* lp_proj = stay_in_loop(if_cmp, loop);
a61af66fc99e Initial load
duke
parents:
diff changeset
1813 assert(if_cmp->in(1)->in(1)->Opcode() == Op_CmpI &&
a61af66fc99e Initial load
duke
parents:
diff changeset
1814 stay_in_loop(lp_proj, loop)->is_If() &&
a61af66fc99e Initial load
duke
parents:
diff changeset
1815 stay_in_loop(lp_proj, loop)->in(1)->in(1)->Opcode() == Op_CmpU, "inserted cmpi before cmpu");
a61af66fc99e Initial load
duke
parents:
diff changeset
1816 Node *con = _igvn.makecon(lp_proj->is_IfTrue() ? TypeInt::ONE : TypeInt::ZERO);
a61af66fc99e Initial load
duke
parents:
diff changeset
1817 set_ctrl(con, C->root());
a61af66fc99e Initial load
duke
parents:
diff changeset
1818 if_cmp->set_req(1, con);
a61af66fc99e Initial load
duke
parents:
diff changeset
1819 }
a61af66fc99e Initial load
duke
parents:
diff changeset
1820
a61af66fc99e Initial load
duke
parents:
diff changeset
1821 //------------------------------ scheduled_nodelist -------------------------------------
a61af66fc99e Initial load
duke
parents:
diff changeset
1822 // Create a post order schedule of nodes that are in the
a61af66fc99e Initial load
duke
parents:
diff changeset
1823 // "member" set. The list is returned in "sched".
a61af66fc99e Initial load
duke
parents:
diff changeset
1824 // The first node in "sched" is the loop head, followed by
a61af66fc99e Initial load
duke
parents:
diff changeset
1825 // nodes which have no inputs in the "member" set, and then
a61af66fc99e Initial load
duke
parents:
diff changeset
1826 // followed by the nodes that have an immediate input dependence
a61af66fc99e Initial load
duke
parents:
diff changeset
1827 // on a node in "sched".
a61af66fc99e Initial load
duke
parents:
diff changeset
1828 void PhaseIdealLoop::scheduled_nodelist( IdealLoopTree *loop, VectorSet& member, Node_List &sched ) {
a61af66fc99e Initial load
duke
parents:
diff changeset
1829
a61af66fc99e Initial load
duke
parents:
diff changeset
1830 assert(member.test(loop->_head->_idx), "loop head must be in member set");
a61af66fc99e Initial load
duke
parents:
diff changeset
1831 Arena *a = Thread::current()->resource_area();
a61af66fc99e Initial load
duke
parents:
diff changeset
1832 VectorSet visited(a);
a61af66fc99e Initial load
duke
parents:
diff changeset
1833 Node_Stack nstack(a, loop->_body.size());
a61af66fc99e Initial load
duke
parents:
diff changeset
1834
a61af66fc99e Initial load
duke
parents:
diff changeset
1835 Node* n = loop->_head; // top of stack is cached in "n"
a61af66fc99e Initial load
duke
parents:
diff changeset
1836 uint idx = 0;
a61af66fc99e Initial load
duke
parents:
diff changeset
1837 visited.set(n->_idx);
a61af66fc99e Initial load
duke
parents:
diff changeset
1838
a61af66fc99e Initial load
duke
parents:
diff changeset
1839 // Initially push all with no inputs from within member set
a61af66fc99e Initial load
duke
parents:
diff changeset
1840 for(uint i = 0; i < loop->_body.size(); i++ ) {
a61af66fc99e Initial load
duke
parents:
diff changeset
1841 Node *elt = loop->_body.at(i);
a61af66fc99e Initial load
duke
parents:
diff changeset
1842 if (member.test(elt->_idx)) {
a61af66fc99e Initial load
duke
parents:
diff changeset
1843 bool found = false;
a61af66fc99e Initial load
duke
parents:
diff changeset
1844 for (uint j = 0; j < elt->req(); j++) {
a61af66fc99e Initial load
duke
parents:
diff changeset
1845 Node* def = elt->in(j);
a61af66fc99e Initial load
duke
parents:
diff changeset
1846 if (def && member.test(def->_idx) && def != elt) {
a61af66fc99e Initial load
duke
parents:
diff changeset
1847 found = true;
a61af66fc99e Initial load
duke
parents:
diff changeset
1848 break;
a61af66fc99e Initial load
duke
parents:
diff changeset
1849 }
a61af66fc99e Initial load
duke
parents:
diff changeset
1850 }
a61af66fc99e Initial load
duke
parents:
diff changeset
1851 if (!found && elt != loop->_head) {
a61af66fc99e Initial load
duke
parents:
diff changeset
1852 nstack.push(n, idx);
a61af66fc99e Initial load
duke
parents:
diff changeset
1853 n = elt;
a61af66fc99e Initial load
duke
parents:
diff changeset
1854 assert(!visited.test(n->_idx), "not seen yet");
a61af66fc99e Initial load
duke
parents:
diff changeset
1855 visited.set(n->_idx);
a61af66fc99e Initial load
duke
parents:
diff changeset
1856 }
a61af66fc99e Initial load
duke
parents:
diff changeset
1857 }
a61af66fc99e Initial load
duke
parents:
diff changeset
1858 }
a61af66fc99e Initial load
duke
parents:
diff changeset
1859
a61af66fc99e Initial load
duke
parents:
diff changeset
1860 // traverse out's that are in the member set
a61af66fc99e Initial load
duke
parents:
diff changeset
1861 while (true) {
a61af66fc99e Initial load
duke
parents:
diff changeset
1862 if (idx < n->outcnt()) {
a61af66fc99e Initial load
duke
parents:
diff changeset
1863 Node* use = n->raw_out(idx);
a61af66fc99e Initial load
duke
parents:
diff changeset
1864 idx++;
a61af66fc99e Initial load
duke
parents:
diff changeset
1865 if (!visited.test_set(use->_idx)) {
a61af66fc99e Initial load
duke
parents:
diff changeset
1866 if (member.test(use->_idx)) {
a61af66fc99e Initial load
duke
parents:
diff changeset
1867 nstack.push(n, idx);
a61af66fc99e Initial load
duke
parents:
diff changeset
1868 n = use;
a61af66fc99e Initial load
duke
parents:
diff changeset
1869 idx = 0;
a61af66fc99e Initial load
duke
parents:
diff changeset
1870 }
a61af66fc99e Initial load
duke
parents:
diff changeset
1871 }
a61af66fc99e Initial load
duke
parents:
diff changeset
1872 } else {
a61af66fc99e Initial load
duke
parents:
diff changeset
1873 // All outputs processed
a61af66fc99e Initial load
duke
parents:
diff changeset
1874 sched.push(n);
a61af66fc99e Initial load
duke
parents:
diff changeset
1875 if (nstack.is_empty()) break;
a61af66fc99e Initial load
duke
parents:
diff changeset
1876 n = nstack.node();
a61af66fc99e Initial load
duke
parents:
diff changeset
1877 idx = nstack.index();
a61af66fc99e Initial load
duke
parents:
diff changeset
1878 nstack.pop();
a61af66fc99e Initial load
duke
parents:
diff changeset
1879 }
a61af66fc99e Initial load
duke
parents:
diff changeset
1880 }
a61af66fc99e Initial load
duke
parents:
diff changeset
1881 }
a61af66fc99e Initial load
duke
parents:
diff changeset
1882
a61af66fc99e Initial load
duke
parents:
diff changeset
1883
a61af66fc99e Initial load
duke
parents:
diff changeset
1884 //------------------------------ has_use_in_set -------------------------------------
a61af66fc99e Initial load
duke
parents:
diff changeset
1885 // Has a use in the vector set
a61af66fc99e Initial load
duke
parents:
diff changeset
1886 bool PhaseIdealLoop::has_use_in_set( Node* n, VectorSet& vset ) {
a61af66fc99e Initial load
duke
parents:
diff changeset
1887 for (DUIterator_Fast jmax, j = n->fast_outs(jmax); j < jmax; j++) {
a61af66fc99e Initial load
duke
parents:
diff changeset
1888 Node* use = n->fast_out(j);
a61af66fc99e Initial load
duke
parents:
diff changeset
1889 if (vset.test(use->_idx)) {
a61af66fc99e Initial load
duke
parents:
diff changeset
1890 return true;
a61af66fc99e Initial load
duke
parents:
diff changeset
1891 }
a61af66fc99e Initial load
duke
parents:
diff changeset
1892 }
a61af66fc99e Initial load
duke
parents:
diff changeset
1893 return false;
a61af66fc99e Initial load
duke
parents:
diff changeset
1894 }
a61af66fc99e Initial load
duke
parents:
diff changeset
1895
a61af66fc99e Initial load
duke
parents:
diff changeset
1896
a61af66fc99e Initial load
duke
parents:
diff changeset
1897 //------------------------------ has_use_internal_to_set -------------------------------------
a61af66fc99e Initial load
duke
parents:
diff changeset
1898 // Has use internal to the vector set (ie. not in a phi at the loop head)
a61af66fc99e Initial load
duke
parents:
diff changeset
1899 bool PhaseIdealLoop::has_use_internal_to_set( Node* n, VectorSet& vset, IdealLoopTree *loop ) {
a61af66fc99e Initial load
duke
parents:
diff changeset
1900 Node* head = loop->_head;
a61af66fc99e Initial load
duke
parents:
diff changeset
1901 for (DUIterator_Fast jmax, j = n->fast_outs(jmax); j < jmax; j++) {
a61af66fc99e Initial load
duke
parents:
diff changeset
1902 Node* use = n->fast_out(j);
a61af66fc99e Initial load
duke
parents:
diff changeset
1903 if (vset.test(use->_idx) && !(use->is_Phi() && use->in(0) == head)) {
a61af66fc99e Initial load
duke
parents:
diff changeset
1904 return true;
a61af66fc99e Initial load
duke
parents:
diff changeset
1905 }
a61af66fc99e Initial load
duke
parents:
diff changeset
1906 }
a61af66fc99e Initial load
duke
parents:
diff changeset
1907 return false;
a61af66fc99e Initial load
duke
parents:
diff changeset
1908 }
a61af66fc99e Initial load
duke
parents:
diff changeset
1909
a61af66fc99e Initial load
duke
parents:
diff changeset
1910
a61af66fc99e Initial load
duke
parents:
diff changeset
1911 //------------------------------ clone_for_use_outside_loop -------------------------------------
a61af66fc99e Initial load
duke
parents:
diff changeset
1912 // clone "n" for uses that are outside of loop
a61af66fc99e Initial load
duke
parents:
diff changeset
1913 void PhaseIdealLoop::clone_for_use_outside_loop( IdealLoopTree *loop, Node* n, Node_List& worklist ) {
a61af66fc99e Initial load
duke
parents:
diff changeset
1914
a61af66fc99e Initial load
duke
parents:
diff changeset
1915 assert(worklist.size() == 0, "should be empty");
a61af66fc99e Initial load
duke
parents:
diff changeset
1916 for (DUIterator_Fast jmax, j = n->fast_outs(jmax); j < jmax; j++) {
a61af66fc99e Initial load
duke
parents:
diff changeset
1917 Node* use = n->fast_out(j);
a61af66fc99e Initial load
duke
parents:
diff changeset
1918 if( !loop->is_member(get_loop(has_ctrl(use) ? get_ctrl(use) : use)) ) {
a61af66fc99e Initial load
duke
parents:
diff changeset
1919 worklist.push(use);
a61af66fc99e Initial load
duke
parents:
diff changeset
1920 }
a61af66fc99e Initial load
duke
parents:
diff changeset
1921 }
a61af66fc99e Initial load
duke
parents:
diff changeset
1922 while( worklist.size() ) {
a61af66fc99e Initial load
duke
parents:
diff changeset
1923 Node *use = worklist.pop();
a61af66fc99e Initial load
duke
parents:
diff changeset
1924 if (!has_node(use) || use->in(0) == C->top()) continue;
a61af66fc99e Initial load
duke
parents:
diff changeset
1925 uint j;
a61af66fc99e Initial load
duke
parents:
diff changeset
1926 for (j = 0; j < use->req(); j++) {
a61af66fc99e Initial load
duke
parents:
diff changeset
1927 if (use->in(j) == n) break;
a61af66fc99e Initial load
duke
parents:
diff changeset
1928 }
a61af66fc99e Initial load
duke
parents:
diff changeset
1929 assert(j < use->req(), "must be there");
a61af66fc99e Initial load
duke
parents:
diff changeset
1930
a61af66fc99e Initial load
duke
parents:
diff changeset
1931 // clone "n" and insert it between the inputs of "n" and the use outside the loop
a61af66fc99e Initial load
duke
parents:
diff changeset
1932 Node* n_clone = n->clone();
a61af66fc99e Initial load
duke
parents:
diff changeset
1933 _igvn.hash_delete(use);
a61af66fc99e Initial load
duke
parents:
diff changeset
1934 use->set_req(j, n_clone);
a61af66fc99e Initial load
duke
parents:
diff changeset
1935 _igvn._worklist.push(use);
251
020a0b730379 6700047: C2 failed in idom_no_update
never
parents: 235
diff changeset
1936 Node* use_c;
0
a61af66fc99e Initial load
duke
parents:
diff changeset
1937 if (!use->is_Phi()) {
251
020a0b730379 6700047: C2 failed in idom_no_update
never
parents: 235
diff changeset
1938 use_c = has_ctrl(use) ? get_ctrl(use) : use->in(0);
0
a61af66fc99e Initial load
duke
parents:
diff changeset
1939 } else {
a61af66fc99e Initial load
duke
parents:
diff changeset
1940 // Use in a phi is considered a use in the associated predecessor block
251
020a0b730379 6700047: C2 failed in idom_no_update
never
parents: 235
diff changeset
1941 use_c = use->in(0)->in(j);
0
a61af66fc99e Initial load
duke
parents:
diff changeset
1942 }
251
020a0b730379 6700047: C2 failed in idom_no_update
never
parents: 235
diff changeset
1943 set_ctrl(n_clone, use_c);
020a0b730379 6700047: C2 failed in idom_no_update
never
parents: 235
diff changeset
1944 assert(!loop->is_member(get_loop(use_c)), "should be outside loop");
020a0b730379 6700047: C2 failed in idom_no_update
never
parents: 235
diff changeset
1945 get_loop(use_c)->_body.push(n_clone);
0
a61af66fc99e Initial load
duke
parents:
diff changeset
1946 _igvn.register_new_node_with_optimizer(n_clone);
a61af66fc99e Initial load
duke
parents:
diff changeset
1947 #if !defined(PRODUCT)
a61af66fc99e Initial load
duke
parents:
diff changeset
1948 if (TracePartialPeeling) {
a61af66fc99e Initial load
duke
parents:
diff changeset
1949 tty->print_cr("loop exit cloning old: %d new: %d newbb: %d", n->_idx, n_clone->_idx, get_ctrl(n_clone)->_idx);
a61af66fc99e Initial load
duke
parents:
diff changeset
1950 }
a61af66fc99e Initial load
duke
parents:
diff changeset
1951 #endif
a61af66fc99e Initial load
duke
parents:
diff changeset
1952 }
a61af66fc99e Initial load
duke
parents:
diff changeset
1953 }
a61af66fc99e Initial load
duke
parents:
diff changeset
1954
a61af66fc99e Initial load
duke
parents:
diff changeset
1955
a61af66fc99e Initial load
duke
parents:
diff changeset
1956 //------------------------------ clone_for_special_use_inside_loop -------------------------------------
a61af66fc99e Initial load
duke
parents:
diff changeset
1957 // clone "n" for special uses that are in the not_peeled region.
a61af66fc99e Initial load
duke
parents:
diff changeset
1958 // If these def-uses occur in separate blocks, the code generator
a61af66fc99e Initial load
duke
parents:
diff changeset
1959 // marks the method as not compilable. For example, if a "BoolNode"
a61af66fc99e Initial load
duke
parents:
diff changeset
1960 // is in a different basic block than the "IfNode" that uses it, then
a61af66fc99e Initial load
duke
parents:
diff changeset
1961 // the compilation is aborted in the code generator.
a61af66fc99e Initial load
duke
parents:
diff changeset
1962 void PhaseIdealLoop::clone_for_special_use_inside_loop( IdealLoopTree *loop, Node* n,
a61af66fc99e Initial load
duke
parents:
diff changeset
1963 VectorSet& not_peel, Node_List& sink_list, Node_List& worklist ) {
a61af66fc99e Initial load
duke
parents:
diff changeset
1964 if (n->is_Phi() || n->is_Load()) {
a61af66fc99e Initial load
duke
parents:
diff changeset
1965 return;
a61af66fc99e Initial load
duke
parents:
diff changeset
1966 }
a61af66fc99e Initial load
duke
parents:
diff changeset
1967 assert(worklist.size() == 0, "should be empty");
a61af66fc99e Initial load
duke
parents:
diff changeset
1968 for (DUIterator_Fast jmax, j = n->fast_outs(jmax); j < jmax; j++) {
a61af66fc99e Initial load
duke
parents:
diff changeset
1969 Node* use = n->fast_out(j);
a61af66fc99e Initial load
duke
parents:
diff changeset
1970 if ( not_peel.test(use->_idx) &&
a61af66fc99e Initial load
duke
parents:
diff changeset
1971 (use->is_If() || use->is_CMove() || use->is_Bool()) &&
a61af66fc99e Initial load
duke
parents:
diff changeset
1972 use->in(1) == n) {
a61af66fc99e Initial load
duke
parents:
diff changeset
1973 worklist.push(use);
a61af66fc99e Initial load
duke
parents:
diff changeset
1974 }
a61af66fc99e Initial load
duke
parents:
diff changeset
1975 }
a61af66fc99e Initial load
duke
parents:
diff changeset
1976 if (worklist.size() > 0) {
a61af66fc99e Initial load
duke
parents:
diff changeset
1977 // clone "n" and insert it between inputs of "n" and the use
a61af66fc99e Initial load
duke
parents:
diff changeset
1978 Node* n_clone = n->clone();
a61af66fc99e Initial load
duke
parents:
diff changeset
1979 loop->_body.push(n_clone);
a61af66fc99e Initial load
duke
parents:
diff changeset
1980 _igvn.register_new_node_with_optimizer(n_clone);
a61af66fc99e Initial load
duke
parents:
diff changeset
1981 set_ctrl(n_clone, get_ctrl(n));
a61af66fc99e Initial load
duke
parents:
diff changeset
1982 sink_list.push(n_clone);
a61af66fc99e Initial load
duke
parents:
diff changeset
1983 not_peel <<= n_clone->_idx; // add n_clone to not_peel set.
a61af66fc99e Initial load
duke
parents:
diff changeset
1984 #if !defined(PRODUCT)
a61af66fc99e Initial load
duke
parents:
diff changeset
1985 if (TracePartialPeeling) {
a61af66fc99e Initial load
duke
parents:
diff changeset
1986 tty->print_cr("special not_peeled cloning old: %d new: %d", n->_idx, n_clone->_idx);
a61af66fc99e Initial load
duke
parents:
diff changeset
1987 }
a61af66fc99e Initial load
duke
parents:
diff changeset
1988 #endif
a61af66fc99e Initial load
duke
parents:
diff changeset
1989 while( worklist.size() ) {
a61af66fc99e Initial load
duke
parents:
diff changeset
1990 Node *use = worklist.pop();
a61af66fc99e Initial load
duke
parents:
diff changeset
1991 _igvn.hash_delete(use);
a61af66fc99e Initial load
duke
parents:
diff changeset
1992 _igvn._worklist.push(use);
a61af66fc99e Initial load
duke
parents:
diff changeset
1993 for (uint j = 1; j < use->req(); j++) {
a61af66fc99e Initial load
duke
parents:
diff changeset
1994 if (use->in(j) == n) {
a61af66fc99e Initial load
duke
parents:
diff changeset
1995 use->set_req(j, n_clone);
a61af66fc99e Initial load
duke
parents:
diff changeset
1996 }
a61af66fc99e Initial load
duke
parents:
diff changeset
1997 }
a61af66fc99e Initial load
duke
parents:
diff changeset
1998 }
a61af66fc99e Initial load
duke
parents:
diff changeset
1999 }
a61af66fc99e Initial load
duke
parents:
diff changeset
2000 }
a61af66fc99e Initial load
duke
parents:
diff changeset
2001
a61af66fc99e Initial load
duke
parents:
diff changeset
2002
a61af66fc99e Initial load
duke
parents:
diff changeset
2003 //------------------------------ insert_phi_for_loop -------------------------------------
a61af66fc99e Initial load
duke
parents:
diff changeset
2004 // Insert phi(lp_entry_val, back_edge_val) at use->in(idx) for loop lp if phi does not already exist
a61af66fc99e Initial load
duke
parents:
diff changeset
2005 void PhaseIdealLoop::insert_phi_for_loop( Node* use, uint idx, Node* lp_entry_val, Node* back_edge_val, LoopNode* lp ) {
a61af66fc99e Initial load
duke
parents:
diff changeset
2006 Node *phi = PhiNode::make(lp, back_edge_val);
a61af66fc99e Initial load
duke
parents:
diff changeset
2007 phi->set_req(LoopNode::EntryControl, lp_entry_val);
a61af66fc99e Initial load
duke
parents:
diff changeset
2008 // Use existing phi if it already exists
a61af66fc99e Initial load
duke
parents:
diff changeset
2009 Node *hit = _igvn.hash_find_insert(phi);
a61af66fc99e Initial load
duke
parents:
diff changeset
2010 if( hit == NULL ) {
a61af66fc99e Initial load
duke
parents:
diff changeset
2011 _igvn.register_new_node_with_optimizer(phi);
a61af66fc99e Initial load
duke
parents:
diff changeset
2012 set_ctrl(phi, lp);
a61af66fc99e Initial load
duke
parents:
diff changeset
2013 } else {
a61af66fc99e Initial load
duke
parents:
diff changeset
2014 // Remove the new phi from the graph and use the hit
a61af66fc99e Initial load
duke
parents:
diff changeset
2015 _igvn.remove_dead_node(phi);
a61af66fc99e Initial load
duke
parents:
diff changeset
2016 phi = hit;
a61af66fc99e Initial load
duke
parents:
diff changeset
2017 }
a61af66fc99e Initial load
duke
parents:
diff changeset
2018 _igvn.hash_delete(use);
a61af66fc99e Initial load
duke
parents:
diff changeset
2019 _igvn._worklist.push(use);
a61af66fc99e Initial load
duke
parents:
diff changeset
2020 use->set_req(idx, phi);
a61af66fc99e Initial load
duke
parents:
diff changeset
2021 }
a61af66fc99e Initial load
duke
parents:
diff changeset
2022
a61af66fc99e Initial load
duke
parents:
diff changeset
2023 #ifdef ASSERT
a61af66fc99e Initial load
duke
parents:
diff changeset
2024 //------------------------------ is_valid_loop_partition -------------------------------------
a61af66fc99e Initial load
duke
parents:
diff changeset
2025 // Validate the loop partition sets: peel and not_peel
a61af66fc99e Initial load
duke
parents:
diff changeset
2026 bool PhaseIdealLoop::is_valid_loop_partition( IdealLoopTree *loop, VectorSet& peel, Node_List& peel_list,
a61af66fc99e Initial load
duke
parents:
diff changeset
2027 VectorSet& not_peel ) {
a61af66fc99e Initial load
duke
parents:
diff changeset
2028 uint i;
a61af66fc99e Initial load
duke
parents:
diff changeset
2029 // Check that peel_list entries are in the peel set
a61af66fc99e Initial load
duke
parents:
diff changeset
2030 for (i = 0; i < peel_list.size(); i++) {
a61af66fc99e Initial load
duke
parents:
diff changeset
2031 if (!peel.test(peel_list.at(i)->_idx)) {
a61af66fc99e Initial load
duke
parents:
diff changeset
2032 return false;
a61af66fc99e Initial load
duke
parents:
diff changeset
2033 }
a61af66fc99e Initial load
duke
parents:
diff changeset
2034 }
a61af66fc99e Initial load
duke
parents:
diff changeset
2035 // Check at loop members are in one of peel set or not_peel set
a61af66fc99e Initial load
duke
parents:
diff changeset
2036 for (i = 0; i < loop->_body.size(); i++ ) {
a61af66fc99e Initial load
duke
parents:
diff changeset
2037 Node *def = loop->_body.at(i);
a61af66fc99e Initial load
duke
parents:
diff changeset
2038 uint di = def->_idx;
a61af66fc99e Initial load
duke
parents:
diff changeset
2039 // Check that peel set elements are in peel_list
a61af66fc99e Initial load
duke
parents:
diff changeset
2040 if (peel.test(di)) {
a61af66fc99e Initial load
duke
parents:
diff changeset
2041 if (not_peel.test(di)) {
a61af66fc99e Initial load
duke
parents:
diff changeset
2042 return false;
a61af66fc99e Initial load
duke
parents:
diff changeset
2043 }
a61af66fc99e Initial load
duke
parents:
diff changeset
2044 // Must be in peel_list also
a61af66fc99e Initial load
duke
parents:
diff changeset
2045 bool found = false;
a61af66fc99e Initial load
duke
parents:
diff changeset
2046 for (uint j = 0; j < peel_list.size(); j++) {
a61af66fc99e Initial load
duke
parents:
diff changeset
2047 if (peel_list.at(j)->_idx == di) {
a61af66fc99e Initial load
duke
parents:
diff changeset
2048 found = true;
a61af66fc99e Initial load
duke
parents:
diff changeset
2049 break;
a61af66fc99e Initial load
duke
parents:
diff changeset
2050 }
a61af66fc99e Initial load
duke
parents:
diff changeset
2051 }
a61af66fc99e Initial load
duke
parents:
diff changeset
2052 if (!found) {
a61af66fc99e Initial load
duke
parents:
diff changeset
2053 return false;
a61af66fc99e Initial load
duke
parents:
diff changeset
2054 }
a61af66fc99e Initial load
duke
parents:
diff changeset
2055 } else if (not_peel.test(di)) {
a61af66fc99e Initial load
duke
parents:
diff changeset
2056 if (peel.test(di)) {
a61af66fc99e Initial load
duke
parents:
diff changeset
2057 return false;
a61af66fc99e Initial load
duke
parents:
diff changeset
2058 }
a61af66fc99e Initial load
duke
parents:
diff changeset
2059 } else {
a61af66fc99e Initial load
duke
parents:
diff changeset
2060 return false;
a61af66fc99e Initial load
duke
parents:
diff changeset
2061 }
a61af66fc99e Initial load
duke
parents:
diff changeset
2062 }
a61af66fc99e Initial load
duke
parents:
diff changeset
2063 return true;
a61af66fc99e Initial load
duke
parents:
diff changeset
2064 }
a61af66fc99e Initial load
duke
parents:
diff changeset
2065
a61af66fc99e Initial load
duke
parents:
diff changeset
2066 //------------------------------ is_valid_clone_loop_exit_use -------------------------------------
a61af66fc99e Initial load
duke
parents:
diff changeset
2067 // Ensure a use outside of loop is of the right form
a61af66fc99e Initial load
duke
parents:
diff changeset
2068 bool PhaseIdealLoop::is_valid_clone_loop_exit_use( IdealLoopTree *loop, Node* use, uint exit_idx) {
a61af66fc99e Initial load
duke
parents:
diff changeset
2069 Node *use_c = has_ctrl(use) ? get_ctrl(use) : use;
a61af66fc99e Initial load
duke
parents:
diff changeset
2070 return (use->is_Phi() &&
a61af66fc99e Initial load
duke
parents:
diff changeset
2071 use_c->is_Region() && use_c->req() == 3 &&
a61af66fc99e Initial load
duke
parents:
diff changeset
2072 (use_c->in(exit_idx)->Opcode() == Op_IfTrue ||
a61af66fc99e Initial load
duke
parents:
diff changeset
2073 use_c->in(exit_idx)->Opcode() == Op_IfFalse ||
a61af66fc99e Initial load
duke
parents:
diff changeset
2074 use_c->in(exit_idx)->Opcode() == Op_JumpProj) &&
a61af66fc99e Initial load
duke
parents:
diff changeset
2075 loop->is_member( get_loop( use_c->in(exit_idx)->in(0) ) ) );
a61af66fc99e Initial load
duke
parents:
diff changeset
2076 }
a61af66fc99e Initial load
duke
parents:
diff changeset
2077
a61af66fc99e Initial load
duke
parents:
diff changeset
2078 //------------------------------ is_valid_clone_loop_form -------------------------------------
a61af66fc99e Initial load
duke
parents:
diff changeset
2079 // Ensure that all uses outside of loop are of the right form
a61af66fc99e Initial load
duke
parents:
diff changeset
2080 bool PhaseIdealLoop::is_valid_clone_loop_form( IdealLoopTree *loop, Node_List& peel_list,
a61af66fc99e Initial load
duke
parents:
diff changeset
2081 uint orig_exit_idx, uint clone_exit_idx) {
a61af66fc99e Initial load
duke
parents:
diff changeset
2082 uint len = peel_list.size();
a61af66fc99e Initial load
duke
parents:
diff changeset
2083 for (uint i = 0; i < len; i++) {
a61af66fc99e Initial load
duke
parents:
diff changeset
2084 Node *def = peel_list.at(i);
a61af66fc99e Initial load
duke
parents:
diff changeset
2085
a61af66fc99e Initial load
duke
parents:
diff changeset
2086 for (DUIterator_Fast jmax, j = def->fast_outs(jmax); j < jmax; j++) {
a61af66fc99e Initial load
duke
parents:
diff changeset
2087 Node *use = def->fast_out(j);
a61af66fc99e Initial load
duke
parents:
diff changeset
2088 Node *use_c = has_ctrl(use) ? get_ctrl(use) : use;
a61af66fc99e Initial load
duke
parents:
diff changeset
2089 if (!loop->is_member(get_loop(use_c))) {
a61af66fc99e Initial load
duke
parents:
diff changeset
2090 // use is not in the loop, check for correct structure
a61af66fc99e Initial load
duke
parents:
diff changeset
2091 if (use->in(0) == def) {
a61af66fc99e Initial load
duke
parents:
diff changeset
2092 // Okay
a61af66fc99e Initial load
duke
parents:
diff changeset
2093 } else if (!is_valid_clone_loop_exit_use(loop, use, orig_exit_idx)) {
a61af66fc99e Initial load
duke
parents:
diff changeset
2094 return false;
a61af66fc99e Initial load
duke
parents:
diff changeset
2095 }
a61af66fc99e Initial load
duke
parents:
diff changeset
2096 }
a61af66fc99e Initial load
duke
parents:
diff changeset
2097 }
a61af66fc99e Initial load
duke
parents:
diff changeset
2098 }
a61af66fc99e Initial load
duke
parents:
diff changeset
2099 return true;
a61af66fc99e Initial load
duke
parents:
diff changeset
2100 }
a61af66fc99e Initial load
duke
parents:
diff changeset
2101 #endif
a61af66fc99e Initial load
duke
parents:
diff changeset
2102
a61af66fc99e Initial load
duke
parents:
diff changeset
2103 //------------------------------ partial_peel -------------------------------------
a61af66fc99e Initial load
duke
parents:
diff changeset
2104 // Partially peel (aka loop rotation) the top portion of a loop (called
a61af66fc99e Initial load
duke
parents:
diff changeset
2105 // the peel section below) by cloning it and placing one copy just before
a61af66fc99e Initial load
duke
parents:
diff changeset
2106 // the new loop head and the other copy at the bottom of the new loop.
a61af66fc99e Initial load
duke
parents:
diff changeset
2107 //
a61af66fc99e Initial load
duke
parents:
diff changeset
2108 // before after where it came from
a61af66fc99e Initial load
duke
parents:
diff changeset
2109 //
a61af66fc99e Initial load
duke
parents:
diff changeset
2110 // stmt1 stmt1
a61af66fc99e Initial load
duke
parents:
diff changeset
2111 // loop: stmt2 clone
a61af66fc99e Initial load
duke
parents:
diff changeset
2112 // stmt2 if condA goto exitA clone
a61af66fc99e Initial load
duke
parents:
diff changeset
2113 // if condA goto exitA new_loop: new
a61af66fc99e Initial load
duke
parents:
diff changeset
2114 // stmt3 stmt3 clone
a61af66fc99e Initial load
duke
parents:
diff changeset
2115 // if !condB goto loop if condB goto exitB clone
a61af66fc99e Initial load
duke
parents:
diff changeset
2116 // exitB: stmt2 orig
a61af66fc99e Initial load
duke
parents:
diff changeset
2117 // stmt4 if !condA goto new_loop orig
a61af66fc99e Initial load
duke
parents:
diff changeset
2118 // exitA: goto exitA
a61af66fc99e Initial load
duke
parents:
diff changeset
2119 // exitB:
a61af66fc99e Initial load
duke
parents:
diff changeset
2120 // stmt4
a61af66fc99e Initial load
duke
parents:
diff changeset
2121 // exitA:
a61af66fc99e Initial load
duke
parents:
diff changeset
2122 //
a61af66fc99e Initial load
duke
parents:
diff changeset
2123 // Step 1: find the cut point: an exit test on probable
a61af66fc99e Initial load
duke
parents:
diff changeset
2124 // induction variable.
a61af66fc99e Initial load
duke
parents:
diff changeset
2125 // Step 2: schedule (with cloning) operations in the peel
a61af66fc99e Initial load
duke
parents:
diff changeset
2126 // section that can be executed after the cut into
a61af66fc99e Initial load
duke
parents:
diff changeset
2127 // the section that is not peeled. This may need
a61af66fc99e Initial load
duke
parents:
diff changeset
2128 // to clone operations into exit blocks. For
a61af66fc99e Initial load
duke
parents:
diff changeset
2129 // instance, a reference to A[i] in the not-peel
a61af66fc99e Initial load
duke
parents:
diff changeset
2130 // section and a reference to B[i] in an exit block
a61af66fc99e Initial load
duke
parents:
diff changeset
2131 // may cause a left-shift of i by 2 to be placed
a61af66fc99e Initial load
duke
parents:
diff changeset
2132 // in the peel block. This step will clone the left
a61af66fc99e Initial load
duke
parents:
diff changeset
2133 // shift into the exit block and sink the left shift
a61af66fc99e Initial load
duke
parents:
diff changeset
2134 // from the peel to the not-peel section.
a61af66fc99e Initial load
duke
parents:
diff changeset
2135 // Step 3: clone the loop, retarget the control, and insert
a61af66fc99e Initial load
duke
parents:
diff changeset
2136 // phis for values that are live across the new loop
a61af66fc99e Initial load
duke
parents:
diff changeset
2137 // head. This is very dependent on the graph structure
a61af66fc99e Initial load
duke
parents:
diff changeset
2138 // from clone_loop. It creates region nodes for
a61af66fc99e Initial load
duke
parents:
diff changeset
2139 // exit control and associated phi nodes for values
a61af66fc99e Initial load
duke
parents:
diff changeset
2140 // flow out of the loop through that exit. The region
a61af66fc99e Initial load
duke
parents:
diff changeset
2141 // node is dominated by the clone's control projection.
a61af66fc99e Initial load
duke
parents:
diff changeset
2142 // So the clone's peel section is placed before the
a61af66fc99e Initial load
duke
parents:
diff changeset
2143 // new loop head, and the clone's not-peel section is
a61af66fc99e Initial load
duke
parents:
diff changeset
2144 // forms the top part of the new loop. The original
a61af66fc99e Initial load
duke
parents:
diff changeset
2145 // peel section forms the tail of the new loop.
a61af66fc99e Initial load
duke
parents:
diff changeset
2146 // Step 4: update the dominator tree and recompute the
a61af66fc99e Initial load
duke
parents:
diff changeset
2147 // dominator depth.
a61af66fc99e Initial load
duke
parents:
diff changeset
2148 //
a61af66fc99e Initial load
duke
parents:
diff changeset
2149 // orig
a61af66fc99e Initial load
duke
parents:
diff changeset
2150 //
2445
08eb13460b3a 7004535: Clone loop predicate during loop unswitch
kvn
parents: 2383
diff changeset
2151 // stmt1
08eb13460b3a 7004535: Clone loop predicate during loop unswitch
kvn
parents: 2383
diff changeset
2152 // |
08eb13460b3a 7004535: Clone loop predicate during loop unswitch
kvn
parents: 2383
diff changeset
2153 // v
08eb13460b3a 7004535: Clone loop predicate during loop unswitch
kvn
parents: 2383
diff changeset
2154 // loop predicate
08eb13460b3a 7004535: Clone loop predicate during loop unswitch
kvn
parents: 2383
diff changeset
2155 // |
08eb13460b3a 7004535: Clone loop predicate during loop unswitch
kvn
parents: 2383
diff changeset
2156 // v
0
a61af66fc99e Initial load
duke
parents:
diff changeset
2157 // loop<----+
a61af66fc99e Initial load
duke
parents:
diff changeset
2158 // | |
a61af66fc99e Initial load
duke
parents:
diff changeset
2159 // stmt2 |
a61af66fc99e Initial load
duke
parents:
diff changeset
2160 // | |
a61af66fc99e Initial load
duke
parents:
diff changeset
2161 // v |
a61af66fc99e Initial load
duke
parents:
diff changeset
2162 // ifA |
a61af66fc99e Initial load
duke
parents:
diff changeset
2163 // / | |
a61af66fc99e Initial load
duke
parents:
diff changeset
2164 // v v |
a61af66fc99e Initial load
duke
parents:
diff changeset
2165 // false true ^ <-- last_peel
a61af66fc99e Initial load
duke
parents:
diff changeset
2166 // / | |
a61af66fc99e Initial load
duke
parents:
diff changeset
2167 // / ===|==cut |
a61af66fc99e Initial load
duke
parents:
diff changeset
2168 // / stmt3 | <-- first_not_peel
a61af66fc99e Initial load
duke
parents:
diff changeset
2169 // / | |
a61af66fc99e Initial load
duke
parents:
diff changeset
2170 // | v |
a61af66fc99e Initial load
duke
parents:
diff changeset
2171 // v ifB |
a61af66fc99e Initial load
duke
parents:
diff changeset
2172 // exitA: / \ |
a61af66fc99e Initial load
duke
parents:
diff changeset
2173 // / \ |
a61af66fc99e Initial load
duke
parents:
diff changeset
2174 // v v |
a61af66fc99e Initial load
duke
parents:
diff changeset
2175 // false true |
a61af66fc99e Initial load
duke
parents:
diff changeset
2176 // / \ |
a61af66fc99e Initial load
duke
parents:
diff changeset
2177 // / ----+
a61af66fc99e Initial load
duke
parents:
diff changeset
2178 // |
a61af66fc99e Initial load
duke
parents:
diff changeset
2179 // v
a61af66fc99e Initial load
duke
parents:
diff changeset
2180 // exitB:
a61af66fc99e Initial load
duke
parents:
diff changeset
2181 // stmt4
a61af66fc99e Initial load
duke
parents:
diff changeset
2182 //
a61af66fc99e Initial load
duke
parents:
diff changeset
2183 //
a61af66fc99e Initial load
duke
parents:
diff changeset
2184 // after clone loop
a61af66fc99e Initial load
duke
parents:
diff changeset
2185 //
a61af66fc99e Initial load
duke
parents:
diff changeset
2186 // stmt1
2445
08eb13460b3a 7004535: Clone loop predicate during loop unswitch
kvn
parents: 2383
diff changeset
2187 // |
08eb13460b3a 7004535: Clone loop predicate during loop unswitch
kvn
parents: 2383
diff changeset
2188 // v
08eb13460b3a 7004535: Clone loop predicate during loop unswitch
kvn
parents: 2383
diff changeset
2189 // loop predicate
0
a61af66fc99e Initial load
duke
parents:
diff changeset
2190 // / \
a61af66fc99e Initial load
duke
parents:
diff changeset
2191 // clone / \ orig
a61af66fc99e Initial load
duke
parents:
diff changeset
2192 // / \
a61af66fc99e Initial load
duke
parents:
diff changeset
2193 // / \
a61af66fc99e Initial load
duke
parents:
diff changeset
2194 // v v
a61af66fc99e Initial load
duke
parents:
diff changeset
2195 // +---->loop loop<----+
a61af66fc99e Initial load
duke
parents:
diff changeset
2196 // | | | |
a61af66fc99e Initial load
duke
parents:
diff changeset
2197 // | stmt2 stmt2 |
a61af66fc99e Initial load
duke
parents:
diff changeset
2198 // | | | |
a61af66fc99e Initial load
duke
parents:
diff changeset
2199 // | v v |
a61af66fc99e Initial load
duke
parents:
diff changeset
2200 // | ifA ifA |
a61af66fc99e Initial load
duke
parents:
diff changeset
2201 // | | \ / | |
a61af66fc99e Initial load
duke
parents:
diff changeset
2202 // | v v v v |
a61af66fc99e Initial load
duke
parents:
diff changeset
2203 // ^ true false false true ^ <-- last_peel
a61af66fc99e Initial load
duke
parents:
diff changeset
2204 // | | ^ \ / | |
a61af66fc99e Initial load
duke
parents:
diff changeset
2205 // | cut==|== \ \ / ===|==cut |
a61af66fc99e Initial load
duke
parents:
diff changeset
2206 // | stmt3 \ \ / stmt3 | <-- first_not_peel
a61af66fc99e Initial load
duke
parents:
diff changeset
2207 // | | dom | | | |
a61af66fc99e Initial load
duke
parents:
diff changeset
2208 // | v \ 1v v2 v |
a61af66fc99e Initial load
duke
parents:
diff changeset
2209 // | ifB regionA ifB |
a61af66fc99e Initial load
duke
parents:
diff changeset
2210 // | / \ | / \ |
a61af66fc99e Initial load
duke
parents:
diff changeset
2211 // | / \ v / \ |
a61af66fc99e Initial load
duke
parents:
diff changeset
2212 // | v v exitA: v v |
a61af66fc99e Initial load
duke
parents:
diff changeset
2213 // | true false false true |
a61af66fc99e Initial load
duke
parents:
diff changeset
2214 // | / ^ \ / \ |
a61af66fc99e Initial load
duke
parents:
diff changeset
2215 // +---- \ \ / ----+
a61af66fc99e Initial load
duke
parents:
diff changeset
2216 // dom \ /
a61af66fc99e Initial load
duke
parents:
diff changeset
2217 // \ 1v v2
a61af66fc99e Initial load
duke
parents:
diff changeset
2218 // regionB
a61af66fc99e Initial load
duke
parents:
diff changeset
2219 // |
a61af66fc99e Initial load
duke
parents:
diff changeset
2220 // v
a61af66fc99e Initial load
duke
parents:
diff changeset
2221 // exitB:
a61af66fc99e Initial load
duke
parents:
diff changeset
2222 // stmt4
a61af66fc99e Initial load
duke
parents:
diff changeset
2223 //
a61af66fc99e Initial load
duke
parents:
diff changeset
2224 //
a61af66fc99e Initial load
duke
parents:
diff changeset
2225 // after partial peel
a61af66fc99e Initial load
duke
parents:
diff changeset
2226 //
a61af66fc99e Initial load
duke
parents:
diff changeset
2227 // stmt1
2445
08eb13460b3a 7004535: Clone loop predicate during loop unswitch
kvn
parents: 2383
diff changeset
2228 // |
08eb13460b3a 7004535: Clone loop predicate during loop unswitch
kvn
parents: 2383
diff changeset
2229 // v
08eb13460b3a 7004535: Clone loop predicate during loop unswitch
kvn
parents: 2383
diff changeset
2230 // loop predicate
0
a61af66fc99e Initial load
duke
parents:
diff changeset
2231 // /
a61af66fc99e Initial load
duke
parents:
diff changeset
2232 // clone / orig
a61af66fc99e Initial load
duke
parents:
diff changeset
2233 // / TOP
a61af66fc99e Initial load
duke
parents:
diff changeset
2234 // / \
a61af66fc99e Initial load
duke
parents:
diff changeset
2235 // v v
2445
08eb13460b3a 7004535: Clone loop predicate during loop unswitch
kvn
parents: 2383
diff changeset
2236 // TOP->loop loop----+
0
a61af66fc99e Initial load
duke
parents:
diff changeset
2237 // | | |
a61af66fc99e Initial load
duke
parents:
diff changeset
2238 // stmt2 stmt2 |
a61af66fc99e Initial load
duke
parents:
diff changeset
2239 // | | |
a61af66fc99e Initial load
duke
parents:
diff changeset
2240 // v v |
a61af66fc99e Initial load
duke
parents:
diff changeset
2241 // ifA ifA |
a61af66fc99e Initial load
duke
parents:
diff changeset
2242 // | \ / | |
a61af66fc99e Initial load
duke
parents:
diff changeset
2243 // v v v v |
a61af66fc99e Initial load
duke
parents:
diff changeset
2244 // true false false true | <-- last_peel
a61af66fc99e Initial load
duke
parents:
diff changeset
2245 // | ^ \ / +------|---+
a61af66fc99e Initial load
duke
parents:
diff changeset
2246 // +->newloop \ \ / === ==cut | |
a61af66fc99e Initial load
duke
parents:
diff changeset
2247 // | stmt3 \ \ / TOP | |
a61af66fc99e Initial load
duke
parents:
diff changeset
2248 // | | dom | | stmt3 | | <-- first_not_peel
a61af66fc99e Initial load
duke
parents:
diff changeset
2249 // | v \ 1v v2 v | |
a61af66fc99e Initial load
duke
parents:
diff changeset
2250 // | ifB regionA ifB ^ v
a61af66fc99e Initial load
duke
parents:
diff changeset
2251 // | / \ | / \ | |
a61af66fc99e Initial load
duke
parents:
diff changeset
2252 // | / \ v / \ | |
a61af66fc99e Initial load
duke
parents:
diff changeset
2253 // | v v exitA: v v | |
a61af66fc99e Initial load
duke
parents:
diff changeset
2254 // | true false false true | |
a61af66fc99e Initial load
duke
parents:
diff changeset
2255 // | / ^ \ / \ | |
a61af66fc99e Initial load
duke
parents:
diff changeset
2256 // | | \ \ / v | |
a61af66fc99e Initial load
duke
parents:
diff changeset
2257 // | | dom \ / TOP | |
a61af66fc99e Initial load
duke
parents:
diff changeset
2258 // | | \ 1v v2 | |
a61af66fc99e Initial load
duke
parents:
diff changeset
2259 // ^ v regionB | |
a61af66fc99e Initial load
duke
parents:
diff changeset
2260 // | | | | |
a61af66fc99e Initial load
duke
parents:
diff changeset
2261 // | | v ^ v
a61af66fc99e Initial load
duke
parents:
diff changeset
2262 // | | exitB: | |
a61af66fc99e Initial load
duke
parents:
diff changeset
2263 // | | stmt4 | |
a61af66fc99e Initial load
duke
parents:
diff changeset
2264 // | +------------>-----------------+ |
a61af66fc99e Initial load
duke
parents:
diff changeset
2265 // | |
a61af66fc99e Initial load
duke
parents:
diff changeset
2266 // +-----------------<---------------------+
a61af66fc99e Initial load
duke
parents:
diff changeset
2267 //
a61af66fc99e Initial load
duke
parents:
diff changeset
2268 //
a61af66fc99e Initial load
duke
parents:
diff changeset
2269 // final graph
a61af66fc99e Initial load
duke
parents:
diff changeset
2270 //
a61af66fc99e Initial load
duke
parents:
diff changeset
2271 // stmt1
a61af66fc99e Initial load
duke
parents:
diff changeset
2272 // |
a61af66fc99e Initial load
duke
parents:
diff changeset
2273 // v
3281
273b56978029 7039586: test/java/util/Collections/Rotate.java failing with hs21-b09
kvn
parents: 2468
diff changeset
2274 // loop predicate
273b56978029 7039586: test/java/util/Collections/Rotate.java failing with hs21-b09
kvn
parents: 2468
diff changeset
2275 // |
273b56978029 7039586: test/java/util/Collections/Rotate.java failing with hs21-b09
kvn
parents: 2468
diff changeset
2276 // v
2445
08eb13460b3a 7004535: Clone loop predicate during loop unswitch
kvn
parents: 2383
diff changeset
2277 // stmt2 clone
08eb13460b3a 7004535: Clone loop predicate during loop unswitch
kvn
parents: 2383
diff changeset
2278 // |
08eb13460b3a 7004535: Clone loop predicate during loop unswitch
kvn
parents: 2383
diff changeset
2279 // v
0
a61af66fc99e Initial load
duke
parents:
diff changeset
2280 // ........> ifA clone
a61af66fc99e Initial load
duke
parents:
diff changeset
2281 // : / |
a61af66fc99e Initial load
duke
parents:
diff changeset
2282 // dom / |
a61af66fc99e Initial load
duke
parents:
diff changeset
2283 // : v v
a61af66fc99e Initial load
duke
parents:
diff changeset
2284 // : false true
a61af66fc99e Initial load
duke
parents:
diff changeset
2285 // : | |
2445
08eb13460b3a 7004535: Clone loop predicate during loop unswitch
kvn
parents: 2383
diff changeset
2286 // : | v
0
a61af66fc99e Initial load
duke
parents:
diff changeset
2287 // : | newloop<-----+
a61af66fc99e Initial load
duke
parents:
diff changeset
2288 // : | | |
a61af66fc99e Initial load
duke
parents:
diff changeset
2289 // : | stmt3 clone |
a61af66fc99e Initial load
duke
parents:
diff changeset
2290 // : | | |
a61af66fc99e Initial load
duke
parents:
diff changeset
2291 // : | v |
a61af66fc99e Initial load
duke
parents:
diff changeset
2292 // : | ifB |
a61af66fc99e Initial load
duke
parents:
diff changeset
2293 // : | / \ |
a61af66fc99e Initial load
duke
parents:
diff changeset
2294 // : | v v |
a61af66fc99e Initial load
duke
parents:
diff changeset
2295 // : | false true |
a61af66fc99e Initial load
duke
parents:
diff changeset
2296 // : | | | |
a61af66fc99e Initial load
duke
parents:
diff changeset
2297 // : | v stmt2 |
a61af66fc99e Initial load
duke
parents:
diff changeset
2298 // : | exitB: | |
a61af66fc99e Initial load
duke
parents:
diff changeset
2299 // : | stmt4 v |
a61af66fc99e Initial load
duke
parents:
diff changeset
2300 // : | ifA orig |
a61af66fc99e Initial load
duke
parents:
diff changeset
2301 // : | / \ |
a61af66fc99e Initial load
duke
parents:
diff changeset
2302 // : | / \ |
a61af66fc99e Initial load
duke
parents:
diff changeset
2303 // : | v v |
a61af66fc99e Initial load
duke
parents:
diff changeset
2304 // : | false true |
a61af66fc99e Initial load
duke
parents:
diff changeset
2305 // : | / \ |
a61af66fc99e Initial load
duke
parents:
diff changeset
2306 // : v v -----+
a61af66fc99e Initial load
duke
parents:
diff changeset
2307 // RegionA
a61af66fc99e Initial load
duke
parents:
diff changeset
2308 // |
a61af66fc99e Initial load
duke
parents:
diff changeset
2309 // v
a61af66fc99e Initial load
duke
parents:
diff changeset
2310 // exitA
a61af66fc99e Initial load
duke
parents:
diff changeset
2311 //
a61af66fc99e Initial load
duke
parents:
diff changeset
2312 bool PhaseIdealLoop::partial_peel( IdealLoopTree *loop, Node_List &old_new ) {
a61af66fc99e Initial load
duke
parents:
diff changeset
2313
2445
08eb13460b3a 7004535: Clone loop predicate during loop unswitch
kvn
parents: 2383
diff changeset
2314 assert(!loop->_head->is_CountedLoop(), "Non-counted loop only");
108
a761c2d3b76a 6684385: Loop unswitching crashes without LoopNode
rasbold
parents: 69
diff changeset
2315 if (!loop->_head->is_Loop()) {
a761c2d3b76a 6684385: Loop unswitching crashes without LoopNode
rasbold
parents: 69
diff changeset
2316 return false; }
a761c2d3b76a 6684385: Loop unswitching crashes without LoopNode
rasbold
parents: 69
diff changeset
2317
0
a61af66fc99e Initial load
duke
parents:
diff changeset
2318 LoopNode *head = loop->_head->as_Loop();
a61af66fc99e Initial load
duke
parents:
diff changeset
2319
a61af66fc99e Initial load
duke
parents:
diff changeset
2320 if (head->is_partial_peel_loop() || head->partial_peel_has_failed()) {
a61af66fc99e Initial load
duke
parents:
diff changeset
2321 return false;
a61af66fc99e Initial load
duke
parents:
diff changeset
2322 }
a61af66fc99e Initial load
duke
parents:
diff changeset
2323
a61af66fc99e Initial load
duke
parents:
diff changeset
2324 // Check for complex exit control
a61af66fc99e Initial load
duke
parents:
diff changeset
2325 for(uint ii = 0; ii < loop->_body.size(); ii++ ) {
a61af66fc99e Initial load
duke
parents:
diff changeset
2326 Node *n = loop->_body.at(ii);
a61af66fc99e Initial load
duke
parents:
diff changeset
2327 int opc = n->Opcode();
a61af66fc99e Initial load
duke
parents:
diff changeset
2328 if (n->is_Call() ||
a61af66fc99e Initial load
duke
parents:
diff changeset
2329 opc == Op_Catch ||
a61af66fc99e Initial load
duke
parents:
diff changeset
2330 opc == Op_CatchProj ||
a61af66fc99e Initial load
duke
parents:
diff changeset
2331 opc == Op_Jump ||
a61af66fc99e Initial load
duke
parents:
diff changeset
2332 opc == Op_JumpProj) {
a61af66fc99e Initial load
duke
parents:
diff changeset
2333 #if !defined(PRODUCT)
a61af66fc99e Initial load
duke
parents:
diff changeset
2334 if (TracePartialPeeling) {
a61af66fc99e Initial load
duke
parents:
diff changeset
2335 tty->print_cr("\nExit control too complex: lp: %d", head->_idx);
a61af66fc99e Initial load
duke
parents:
diff changeset
2336 }
a61af66fc99e Initial load
duke
parents:
diff changeset
2337 #endif
a61af66fc99e Initial load
duke
parents:
diff changeset
2338 return false;
a61af66fc99e Initial load
duke
parents:
diff changeset
2339 }
a61af66fc99e Initial load
duke
parents:
diff changeset
2340 }
a61af66fc99e Initial load
duke
parents:
diff changeset
2341
a61af66fc99e Initial load
duke
parents:
diff changeset
2342 int dd = dom_depth(head);
a61af66fc99e Initial load
duke
parents:
diff changeset
2343
a61af66fc99e Initial load
duke
parents:
diff changeset
2344 // Step 1: find cut point
a61af66fc99e Initial load
duke
parents:
diff changeset
2345
a61af66fc99e Initial load
duke
parents:
diff changeset
2346 // Walk up dominators to loop head looking for first loop exit
a61af66fc99e Initial load
duke
parents:
diff changeset
2347 // which is executed on every path thru loop.
a61af66fc99e Initial load
duke
parents:
diff changeset
2348 IfNode *peel_if = NULL;
a61af66fc99e Initial load
duke
parents:
diff changeset
2349 IfNode *peel_if_cmpu = NULL;
a61af66fc99e Initial load
duke
parents:
diff changeset
2350
a61af66fc99e Initial load
duke
parents:
diff changeset
2351 Node *iff = loop->tail();
a61af66fc99e Initial load
duke
parents:
diff changeset
2352 while( iff != head ) {
a61af66fc99e Initial load
duke
parents:
diff changeset
2353 if( iff->is_If() ) {
a61af66fc99e Initial load
duke
parents:
diff changeset
2354 Node *ctrl = get_ctrl(iff->in(1));
a61af66fc99e Initial load
duke
parents:
diff changeset
2355 if (ctrl->is_top()) return false; // Dead test on live IF.
a61af66fc99e Initial load
duke
parents:
diff changeset
2356 // If loop-varying exit-test, check for induction variable
a61af66fc99e Initial load
duke
parents:
diff changeset
2357 if( loop->is_member(get_loop(ctrl)) &&
a61af66fc99e Initial load
duke
parents:
diff changeset
2358 loop->is_loop_exit(iff) &&
a61af66fc99e Initial load
duke
parents:
diff changeset
2359 is_possible_iv_test(iff)) {
a61af66fc99e Initial load
duke
parents:
diff changeset
2360 Node* cmp = iff->in(1)->in(1);
a61af66fc99e Initial load
duke
parents:
diff changeset
2361 if (cmp->Opcode() == Op_CmpI) {
a61af66fc99e Initial load
duke
parents:
diff changeset
2362 peel_if = iff->as_If();
a61af66fc99e Initial load
duke
parents:
diff changeset
2363 } else {
a61af66fc99e Initial load
duke
parents:
diff changeset
2364 assert(cmp->Opcode() == Op_CmpU, "must be CmpI or CmpU");
a61af66fc99e Initial load
duke
parents:
diff changeset
2365 peel_if_cmpu = iff->as_If();
a61af66fc99e Initial load
duke
parents:
diff changeset
2366 }
a61af66fc99e Initial load
duke
parents:
diff changeset
2367 }
a61af66fc99e Initial load
duke
parents:
diff changeset
2368 }
a61af66fc99e Initial load
duke
parents:
diff changeset
2369 iff = idom(iff);
a61af66fc99e Initial load
duke
parents:
diff changeset
2370 }
a61af66fc99e Initial load
duke
parents:
diff changeset
2371 // Prefer signed compare over unsigned compare.
a61af66fc99e Initial load
duke
parents:
diff changeset
2372 IfNode* new_peel_if = NULL;
a61af66fc99e Initial load
duke
parents:
diff changeset
2373 if (peel_if == NULL) {
a61af66fc99e Initial load
duke
parents:
diff changeset
2374 if (!PartialPeelAtUnsignedTests || peel_if_cmpu == NULL) {
a61af66fc99e Initial load
duke
parents:
diff changeset
2375 return false; // No peel point found
a61af66fc99e Initial load
duke
parents:
diff changeset
2376 }
a61af66fc99e Initial load
duke
parents:
diff changeset
2377 new_peel_if = insert_cmpi_loop_exit(peel_if_cmpu, loop);
a61af66fc99e Initial load
duke
parents:
diff changeset
2378 if (new_peel_if == NULL) {
a61af66fc99e Initial load
duke
parents:
diff changeset
2379 return false; // No peel point found
a61af66fc99e Initial load
duke
parents:
diff changeset
2380 }
a61af66fc99e Initial load
duke
parents:
diff changeset
2381 peel_if = new_peel_if;
a61af66fc99e Initial load
duke
parents:
diff changeset
2382 }
a61af66fc99e Initial load
duke
parents:
diff changeset
2383 Node* last_peel = stay_in_loop(peel_if, loop);
a61af66fc99e Initial load
duke
parents:
diff changeset
2384 Node* first_not_peeled = stay_in_loop(last_peel, loop);
a61af66fc99e Initial load
duke
parents:
diff changeset
2385 if (first_not_peeled == NULL || first_not_peeled == head) {
a61af66fc99e Initial load
duke
parents:
diff changeset
2386 return false;
a61af66fc99e Initial load
duke
parents:
diff changeset
2387 }
a61af66fc99e Initial load
duke
parents:
diff changeset
2388
a61af66fc99e Initial load
duke
parents:
diff changeset
2389 #if !defined(PRODUCT)
2383
9dc311b8473e 7008866: Missing loop predicate for loop with multiple entries
kvn
parents: 1972
diff changeset
2390 if (TraceLoopOpts) {
9dc311b8473e 7008866: Missing loop predicate for loop with multiple entries
kvn
parents: 1972
diff changeset
2391 tty->print("PartialPeel ");
9dc311b8473e 7008866: Missing loop predicate for loop with multiple entries
kvn
parents: 1972
diff changeset
2392 loop->dump_head();
9dc311b8473e 7008866: Missing loop predicate for loop with multiple entries
kvn
parents: 1972
diff changeset
2393 }
9dc311b8473e 7008866: Missing loop predicate for loop with multiple entries
kvn
parents: 1972
diff changeset
2394
0
a61af66fc99e Initial load
duke
parents:
diff changeset
2395 if (TracePartialPeeling) {
a61af66fc99e Initial load
duke
parents:
diff changeset
2396 tty->print_cr("before partial peel one iteration");
a61af66fc99e Initial load
duke
parents:
diff changeset
2397 Node_List wl;
a61af66fc99e Initial load
duke
parents:
diff changeset
2398 Node* t = head->in(2);
a61af66fc99e Initial load
duke
parents:
diff changeset
2399 while (true) {
a61af66fc99e Initial load
duke
parents:
diff changeset
2400 wl.push(t);
a61af66fc99e Initial load
duke
parents:
diff changeset
2401 if (t == head) break;
a61af66fc99e Initial load
duke
parents:
diff changeset
2402 t = idom(t);
a61af66fc99e Initial load
duke
parents:
diff changeset
2403 }
a61af66fc99e Initial load
duke
parents:
diff changeset
2404 while (wl.size() > 0) {
a61af66fc99e Initial load
duke
parents:
diff changeset
2405 Node* tt = wl.pop();
a61af66fc99e Initial load
duke
parents:
diff changeset
2406 tt->dump();
a61af66fc99e Initial load
duke
parents:
diff changeset
2407 if (tt == last_peel) tty->print_cr("-- cut --");
a61af66fc99e Initial load
duke
parents:
diff changeset
2408 }
a61af66fc99e Initial load
duke
parents:
diff changeset
2409 }
a61af66fc99e Initial load
duke
parents:
diff changeset
2410 #endif
a61af66fc99e Initial load
duke
parents:
diff changeset
2411 ResourceArea *area = Thread::current()->resource_area();
a61af66fc99e Initial load
duke
parents:
diff changeset
2412 VectorSet peel(area);
a61af66fc99e Initial load
duke
parents:
diff changeset
2413 VectorSet not_peel(area);
a61af66fc99e Initial load
duke
parents:
diff changeset
2414 Node_List peel_list(area);
a61af66fc99e Initial load
duke
parents:
diff changeset
2415 Node_List worklist(area);
a61af66fc99e Initial load
duke
parents:
diff changeset
2416 Node_List sink_list(area);
a61af66fc99e Initial load
duke
parents:
diff changeset
2417
a61af66fc99e Initial load
duke
parents:
diff changeset
2418 // Set of cfg nodes to peel are those that are executable from
a61af66fc99e Initial load
duke
parents:
diff changeset
2419 // the head through last_peel.
a61af66fc99e Initial load
duke
parents:
diff changeset
2420 assert(worklist.size() == 0, "should be empty");
a61af66fc99e Initial load
duke
parents:
diff changeset
2421 worklist.push(head);
a61af66fc99e Initial load
duke
parents:
diff changeset
2422 peel.set(head->_idx);
a61af66fc99e Initial load
duke
parents:
diff changeset
2423 while (worklist.size() > 0) {
a61af66fc99e Initial load
duke
parents:
diff changeset
2424 Node *n = worklist.pop();
a61af66fc99e Initial load
duke
parents:
diff changeset
2425 if (n != last_peel) {
a61af66fc99e Initial load
duke
parents:
diff changeset
2426 for (DUIterator_Fast jmax, j = n->fast_outs(jmax); j < jmax; j++) {
a61af66fc99e Initial load
duke
parents:
diff changeset
2427 Node* use = n->fast_out(j);
a61af66fc99e Initial load
duke
parents:
diff changeset
2428 if (use->is_CFG() &&
a61af66fc99e Initial load
duke
parents:
diff changeset
2429 loop->is_member(get_loop(use)) &&
a61af66fc99e Initial load
duke
parents:
diff changeset
2430 !peel.test_set(use->_idx)) {
a61af66fc99e Initial load
duke
parents:
diff changeset
2431 worklist.push(use);
a61af66fc99e Initial load
duke
parents:
diff changeset
2432 }
a61af66fc99e Initial load
duke
parents:
diff changeset
2433 }
a61af66fc99e Initial load
duke
parents:
diff changeset
2434 }
a61af66fc99e Initial load
duke
parents:
diff changeset
2435 }
a61af66fc99e Initial load
duke
parents:
diff changeset
2436
a61af66fc99e Initial load
duke
parents:
diff changeset
2437 // Set of non-cfg nodes to peel are those that are control
a61af66fc99e Initial load
duke
parents:
diff changeset
2438 // dependent on the cfg nodes.
a61af66fc99e Initial load
duke
parents:
diff changeset
2439 uint i;
a61af66fc99e Initial load
duke
parents:
diff changeset
2440 for(i = 0; i < loop->_body.size(); i++ ) {
a61af66fc99e Initial load
duke
parents:
diff changeset
2441 Node *n = loop->_body.at(i);
a61af66fc99e Initial load
duke
parents:
diff changeset
2442 Node *n_c = has_ctrl(n) ? get_ctrl(n) : n;
a61af66fc99e Initial load
duke
parents:
diff changeset
2443 if (peel.test(n_c->_idx)) {
a61af66fc99e Initial load
duke
parents:
diff changeset
2444 peel.set(n->_idx);
a61af66fc99e Initial load
duke
parents:
diff changeset
2445 } else {
a61af66fc99e Initial load
duke
parents:
diff changeset
2446 not_peel.set(n->_idx);
a61af66fc99e Initial load
duke
parents:
diff changeset
2447 }
a61af66fc99e Initial load
duke
parents:
diff changeset
2448 }
a61af66fc99e Initial load
duke
parents:
diff changeset
2449
a61af66fc99e Initial load
duke
parents:
diff changeset
2450 // Step 2: move operations from the peeled section down into the
a61af66fc99e Initial load
duke
parents:
diff changeset
2451 // not-peeled section
a61af66fc99e Initial load
duke
parents:
diff changeset
2452
a61af66fc99e Initial load
duke
parents:
diff changeset
2453 // Get a post order schedule of nodes in the peel region
a61af66fc99e Initial load
duke
parents:
diff changeset
2454 // Result in right-most operand.
a61af66fc99e Initial load
duke
parents:
diff changeset
2455 scheduled_nodelist(loop, peel, peel_list );
a61af66fc99e Initial load
duke
parents:
diff changeset
2456
a61af66fc99e Initial load
duke
parents:
diff changeset
2457 assert(is_valid_loop_partition(loop, peel, peel_list, not_peel), "bad partition");
a61af66fc99e Initial load
duke
parents:
diff changeset
2458
a61af66fc99e Initial load
duke
parents:
diff changeset
2459 // For future check for too many new phis
a61af66fc99e Initial load
duke
parents:
diff changeset
2460 uint old_phi_cnt = 0;
a61af66fc99e Initial load
duke
parents:
diff changeset
2461 for (DUIterator_Fast jmax, j = head->fast_outs(jmax); j < jmax; j++) {
a61af66fc99e Initial load
duke
parents:
diff changeset
2462 Node* use = head->fast_out(j);
a61af66fc99e Initial load
duke
parents:
diff changeset
2463 if (use->is_Phi()) old_phi_cnt++;
a61af66fc99e Initial load
duke
parents:
diff changeset
2464 }
a61af66fc99e Initial load
duke
parents:
diff changeset
2465
a61af66fc99e Initial load
duke
parents:
diff changeset
2466 #if !defined(PRODUCT)
a61af66fc99e Initial load
duke
parents:
diff changeset
2467 if (TracePartialPeeling) {
a61af66fc99e Initial load
duke
parents:
diff changeset
2468 tty->print_cr("\npeeled list");
a61af66fc99e Initial load
duke
parents:
diff changeset
2469 }
a61af66fc99e Initial load
duke
parents:
diff changeset
2470 #endif
a61af66fc99e Initial load
duke
parents:
diff changeset
2471
a61af66fc99e Initial load
duke
parents:
diff changeset
2472 // Evacuate nodes in peel region into the not_peeled region if possible
a61af66fc99e Initial load
duke
parents:
diff changeset
2473 uint new_phi_cnt = 0;
a61af66fc99e Initial load
duke
parents:
diff changeset
2474 for (i = 0; i < peel_list.size();) {
a61af66fc99e Initial load
duke
parents:
diff changeset
2475 Node* n = peel_list.at(i);
a61af66fc99e Initial load
duke
parents:
diff changeset
2476 #if !defined(PRODUCT)
a61af66fc99e Initial load
duke
parents:
diff changeset
2477 if (TracePartialPeeling) n->dump();
a61af66fc99e Initial load
duke
parents:
diff changeset
2478 #endif
a61af66fc99e Initial load
duke
parents:
diff changeset
2479 bool incr = true;
a61af66fc99e Initial load
duke
parents:
diff changeset
2480 if ( !n->is_CFG() ) {
a61af66fc99e Initial load
duke
parents:
diff changeset
2481
a61af66fc99e Initial load
duke
parents:
diff changeset
2482 if ( has_use_in_set(n, not_peel) ) {
a61af66fc99e Initial load
duke
parents:
diff changeset
2483
a61af66fc99e Initial load
duke
parents:
diff changeset
2484 // If not used internal to the peeled region,
a61af66fc99e Initial load
duke
parents:
diff changeset
2485 // move "n" from peeled to not_peeled region.
a61af66fc99e Initial load
duke
parents:
diff changeset
2486
a61af66fc99e Initial load
duke
parents:
diff changeset
2487 if ( !has_use_internal_to_set(n, peel, loop) ) {
a61af66fc99e Initial load
duke
parents:
diff changeset
2488
a61af66fc99e Initial load
duke
parents:
diff changeset
2489 // if not pinned and not a load (which maybe anti-dependent on a store)
a61af66fc99e Initial load
duke
parents:
diff changeset
2490 // and not a CMove (Matcher expects only bool->cmove).
a61af66fc99e Initial load
duke
parents:
diff changeset
2491 if ( n->in(0) == NULL && !n->is_Load() && !n->is_CMove() ) {
a61af66fc99e Initial load
duke
parents:
diff changeset
2492 clone_for_use_outside_loop( loop, n, worklist );
a61af66fc99e Initial load
duke
parents:
diff changeset
2493
a61af66fc99e Initial load
duke
parents:
diff changeset
2494 sink_list.push(n);
a61af66fc99e Initial load
duke
parents:
diff changeset
2495 peel >>= n->_idx; // delete n from peel set.
a61af66fc99e Initial load
duke
parents:
diff changeset
2496 not_peel <<= n->_idx; // add n to not_peel set.
a61af66fc99e Initial load
duke
parents:
diff changeset
2497 peel_list.remove(i);
a61af66fc99e Initial load
duke
parents:
diff changeset
2498 incr = false;
a61af66fc99e Initial load
duke
parents:
diff changeset
2499 #if !defined(PRODUCT)
a61af66fc99e Initial load
duke
parents:
diff changeset
2500 if (TracePartialPeeling) {
a61af66fc99e Initial load
duke
parents:
diff changeset
2501 tty->print_cr("sink to not_peeled region: %d newbb: %d",
a61af66fc99e Initial load
duke
parents:
diff changeset
2502 n->_idx, get_ctrl(n)->_idx);
a61af66fc99e Initial load
duke
parents:
diff changeset
2503 }
a61af66fc99e Initial load
duke
parents:
diff changeset
2504 #endif
a61af66fc99e Initial load
duke
parents:
diff changeset
2505 }
a61af66fc99e Initial load
duke
parents:
diff changeset
2506 } else {
a61af66fc99e Initial load
duke
parents:
diff changeset
2507 // Otherwise check for special def-use cases that span
a61af66fc99e Initial load
duke
parents:
diff changeset
2508 // the peel/not_peel boundary such as bool->if
a61af66fc99e Initial load
duke
parents:
diff changeset
2509 clone_for_special_use_inside_loop( loop, n, not_peel, sink_list, worklist );
a61af66fc99e Initial load
duke
parents:
diff changeset
2510 new_phi_cnt++;
a61af66fc99e Initial load
duke
parents:
diff changeset
2511 }
a61af66fc99e Initial load
duke
parents:
diff changeset
2512 }
a61af66fc99e Initial load
duke
parents:
diff changeset
2513 }
a61af66fc99e Initial load
duke
parents:
diff changeset
2514 if (incr) i++;
a61af66fc99e Initial load
duke
parents:
diff changeset
2515 }
a61af66fc99e Initial load
duke
parents:
diff changeset
2516
a61af66fc99e Initial load
duke
parents:
diff changeset
2517 if (new_phi_cnt > old_phi_cnt + PartialPeelNewPhiDelta) {
a61af66fc99e Initial load
duke
parents:
diff changeset
2518 #if !defined(PRODUCT)
a61af66fc99e Initial load
duke
parents:
diff changeset
2519 if (TracePartialPeeling) {
a61af66fc99e Initial load
duke
parents:
diff changeset
2520 tty->print_cr("\nToo many new phis: %d old %d new cmpi: %c",
a61af66fc99e Initial load
duke
parents:
diff changeset
2521 new_phi_cnt, old_phi_cnt, new_peel_if != NULL?'T':'F');
a61af66fc99e Initial load
duke
parents:
diff changeset
2522 }
a61af66fc99e Initial load
duke
parents:
diff changeset
2523 #endif
a61af66fc99e Initial load
duke
parents:
diff changeset
2524 if (new_peel_if != NULL) {
a61af66fc99e Initial load
duke
parents:
diff changeset
2525 remove_cmpi_loop_exit(new_peel_if, loop);
a61af66fc99e Initial load
duke
parents:
diff changeset
2526 }
a61af66fc99e Initial load
duke
parents:
diff changeset
2527 // Inhibit more partial peeling on this loop
a61af66fc99e Initial load
duke
parents:
diff changeset
2528 assert(!head->is_partial_peel_loop(), "not partial peeled");
a61af66fc99e Initial load
duke
parents:
diff changeset
2529 head->mark_partial_peel_failed();
a61af66fc99e Initial load
duke
parents:
diff changeset
2530 return false;
a61af66fc99e Initial load
duke
parents:
diff changeset
2531 }
a61af66fc99e Initial load
duke
parents:
diff changeset
2532
a61af66fc99e Initial load
duke
parents:
diff changeset
2533 // Step 3: clone loop, retarget control, and insert new phis
a61af66fc99e Initial load
duke
parents:
diff changeset
2534
a61af66fc99e Initial load
duke
parents:
diff changeset
2535 // Create new loop head for new phis and to hang
a61af66fc99e Initial load
duke
parents:
diff changeset
2536 // the nodes being moved (sinked) from the peel region.
a61af66fc99e Initial load
duke
parents:
diff changeset
2537 LoopNode* new_head = new (C, 3) LoopNode(last_peel, last_peel);
2383
9dc311b8473e 7008866: Missing loop predicate for loop with multiple entries
kvn
parents: 1972
diff changeset
2538 new_head->set_unswitch_count(head->unswitch_count()); // Preserve
0
a61af66fc99e Initial load
duke
parents:
diff changeset
2539 _igvn.register_new_node_with_optimizer(new_head);
a61af66fc99e Initial load
duke
parents:
diff changeset
2540 assert(first_not_peeled->in(0) == last_peel, "last_peel <- first_not_peeled");
a61af66fc99e Initial load
duke
parents:
diff changeset
2541 first_not_peeled->set_req(0, new_head);
a61af66fc99e Initial load
duke
parents:
diff changeset
2542 set_loop(new_head, loop);
a61af66fc99e Initial load
duke
parents:
diff changeset
2543 loop->_body.push(new_head);
a61af66fc99e Initial load
duke
parents:
diff changeset
2544 not_peel.set(new_head->_idx);
a61af66fc99e Initial load
duke
parents:
diff changeset
2545 set_idom(new_head, last_peel, dom_depth(first_not_peeled));
a61af66fc99e Initial load
duke
parents:
diff changeset
2546 set_idom(first_not_peeled, new_head, dom_depth(first_not_peeled));
a61af66fc99e Initial load
duke
parents:
diff changeset
2547
a61af66fc99e Initial load
duke
parents:
diff changeset
2548 while (sink_list.size() > 0) {
a61af66fc99e Initial load
duke
parents:
diff changeset
2549 Node* n = sink_list.pop();
a61af66fc99e Initial load
duke
parents:
diff changeset
2550 set_ctrl(n, new_head);
a61af66fc99e Initial load
duke
parents:
diff changeset
2551 }
a61af66fc99e Initial load
duke
parents:
diff changeset
2552
a61af66fc99e Initial load
duke
parents:
diff changeset
2553 assert(is_valid_loop_partition(loop, peel, peel_list, not_peel), "bad partition");
a61af66fc99e Initial load
duke
parents:
diff changeset
2554
a61af66fc99e Initial load
duke
parents:
diff changeset
2555 clone_loop( loop, old_new, dd );
a61af66fc99e Initial load
duke
parents:
diff changeset
2556
a61af66fc99e Initial load
duke
parents:
diff changeset
2557 const uint clone_exit_idx = 1;
a61af66fc99e Initial load
duke
parents:
diff changeset
2558 const uint orig_exit_idx = 2;
a61af66fc99e Initial load
duke
parents:
diff changeset
2559 assert(is_valid_clone_loop_form( loop, peel_list, orig_exit_idx, clone_exit_idx ), "bad clone loop");
a61af66fc99e Initial load
duke
parents:
diff changeset
2560
a61af66fc99e Initial load
duke
parents:
diff changeset
2561 Node* head_clone = old_new[head->_idx];
a61af66fc99e Initial load
duke
parents:
diff changeset
2562 LoopNode* new_head_clone = old_new[new_head->_idx]->as_Loop();
a61af66fc99e Initial load
duke
parents:
diff changeset
2563 Node* orig_tail_clone = head_clone->in(2);
a61af66fc99e Initial load
duke
parents:
diff changeset
2564
a61af66fc99e Initial load
duke
parents:
diff changeset
2565 // Add phi if "def" node is in peel set and "use" is not
a61af66fc99e Initial load
duke
parents:
diff changeset
2566
a61af66fc99e Initial load
duke
parents:
diff changeset
2567 for(i = 0; i < peel_list.size(); i++ ) {
a61af66fc99e Initial load
duke
parents:
diff changeset
2568 Node *def = peel_list.at(i);
a61af66fc99e Initial load
duke
parents:
diff changeset
2569 if (!def->is_CFG()) {
a61af66fc99e Initial load
duke
parents:
diff changeset
2570 for (DUIterator_Fast jmax, j = def->fast_outs(jmax); j < jmax; j++) {
a61af66fc99e Initial load
duke
parents:
diff changeset
2571 Node *use = def->fast_out(j);
a61af66fc99e Initial load
duke
parents:
diff changeset
2572 if (has_node(use) && use->in(0) != C->top() &&
a61af66fc99e Initial load
duke
parents:
diff changeset
2573 (!peel.test(use->_idx) ||
a61af66fc99e Initial load
duke
parents:
diff changeset
2574 (use->is_Phi() && use->in(0) == head)) ) {
a61af66fc99e Initial load
duke
parents:
diff changeset
2575 worklist.push(use);
a61af66fc99e Initial load
duke
parents:
diff changeset
2576 }
a61af66fc99e Initial load
duke
parents:
diff changeset
2577 }
a61af66fc99e Initial load
duke
parents:
diff changeset
2578 while( worklist.size() ) {
a61af66fc99e Initial load
duke
parents:
diff changeset
2579 Node *use = worklist.pop();
a61af66fc99e Initial load
duke
parents:
diff changeset
2580 for (uint j = 1; j < use->req(); j++) {
a61af66fc99e Initial load
duke
parents:
diff changeset
2581 Node* n = use->in(j);
a61af66fc99e Initial load
duke
parents:
diff changeset
2582 if (n == def) {
a61af66fc99e Initial load
duke
parents:
diff changeset
2583
a61af66fc99e Initial load
duke
parents:
diff changeset
2584 // "def" is in peel set, "use" is not in peel set
a61af66fc99e Initial load
duke
parents:
diff changeset
2585 // or "use" is in the entry boundary (a phi) of the peel set
a61af66fc99e Initial load
duke
parents:
diff changeset
2586
a61af66fc99e Initial load
duke
parents:
diff changeset
2587 Node* use_c = has_ctrl(use) ? get_ctrl(use) : use;
a61af66fc99e Initial load
duke
parents:
diff changeset
2588
a61af66fc99e Initial load
duke
parents:
diff changeset
2589 if ( loop->is_member(get_loop( use_c )) ) {
a61af66fc99e Initial load
duke
parents:
diff changeset
2590 // use is in loop
a61af66fc99e Initial load
duke
parents:
diff changeset
2591 if (old_new[use->_idx] != NULL) { // null for dead code
a61af66fc99e Initial load
duke
parents:
diff changeset
2592 Node* use_clone = old_new[use->_idx];
a61af66fc99e Initial load
duke
parents:
diff changeset
2593 _igvn.hash_delete(use);
a61af66fc99e Initial load
duke
parents:
diff changeset
2594 use->set_req(j, C->top());
a61af66fc99e Initial load
duke
parents:
diff changeset
2595 _igvn._worklist.push(use);
a61af66fc99e Initial load
duke
parents:
diff changeset
2596 insert_phi_for_loop( use_clone, j, old_new[def->_idx], def, new_head_clone );
a61af66fc99e Initial load
duke
parents:
diff changeset
2597 }
a61af66fc99e Initial load
duke
parents:
diff changeset
2598 } else {
a61af66fc99e Initial load
duke
parents:
diff changeset
2599 assert(is_valid_clone_loop_exit_use(loop, use, orig_exit_idx), "clone loop format");
a61af66fc99e Initial load
duke
parents:
diff changeset
2600 // use is not in the loop, check if the live range includes the cut
a61af66fc99e Initial load
duke
parents:
diff changeset
2601 Node* lp_if = use_c->in(orig_exit_idx)->in(0);
a61af66fc99e Initial load
duke
parents:
diff changeset
2602 if (not_peel.test(lp_if->_idx)) {
a61af66fc99e Initial load
duke
parents:
diff changeset
2603 assert(j == orig_exit_idx, "use from original loop");
a61af66fc99e Initial load
duke
parents:
diff changeset
2604 insert_phi_for_loop( use, clone_exit_idx, old_new[def->_idx], def, new_head_clone );
a61af66fc99e Initial load
duke
parents:
diff changeset
2605 }
a61af66fc99e Initial load
duke
parents:
diff changeset
2606 }
a61af66fc99e Initial load
duke
parents:
diff changeset
2607 }
a61af66fc99e Initial load
duke
parents:
diff changeset
2608 }
a61af66fc99e Initial load
duke
parents:
diff changeset
2609 }
a61af66fc99e Initial load
duke
parents:
diff changeset
2610 }
a61af66fc99e Initial load
duke
parents:
diff changeset
2611 }
a61af66fc99e Initial load
duke
parents:
diff changeset
2612
a61af66fc99e Initial load
duke
parents:
diff changeset
2613 // Step 3b: retarget control
a61af66fc99e Initial load
duke
parents:
diff changeset
2614
a61af66fc99e Initial load
duke
parents:
diff changeset
2615 // Redirect control to the new loop head if a cloned node in
a61af66fc99e Initial load
duke
parents:
diff changeset
2616 // the not_peeled region has control that points into the peeled region.
a61af66fc99e Initial load
duke
parents:
diff changeset
2617 // This necessary because the cloned peeled region will be outside
a61af66fc99e Initial load
duke
parents:
diff changeset
2618 // the loop.
a61af66fc99e Initial load
duke
parents:
diff changeset
2619 // from to
a61af66fc99e Initial load
duke
parents:
diff changeset
2620 // cloned-peeled <---+
a61af66fc99e Initial load
duke
parents:
diff changeset
2621 // new_head_clone: | <--+
a61af66fc99e Initial load
duke
parents:
diff changeset
2622 // cloned-not_peeled in(0) in(0)
a61af66fc99e Initial load
duke
parents:
diff changeset
2623 // orig-peeled
a61af66fc99e Initial load
duke
parents:
diff changeset
2624
a61af66fc99e Initial load
duke
parents:
diff changeset
2625 for(i = 0; i < loop->_body.size(); i++ ) {
a61af66fc99e Initial load
duke
parents:
diff changeset
2626 Node *n = loop->_body.at(i);
a61af66fc99e Initial load
duke
parents:
diff changeset
2627 if (!n->is_CFG() && n->in(0) != NULL &&
a61af66fc99e Initial load
duke
parents:
diff changeset
2628 not_peel.test(n->_idx) && peel.test(n->in(0)->_idx)) {
a61af66fc99e Initial load
duke
parents:
diff changeset
2629 Node* n_clone = old_new[n->_idx];
a61af66fc99e Initial load
duke
parents:
diff changeset
2630 _igvn.hash_delete(n_clone);
a61af66fc99e Initial load
duke
parents:
diff changeset
2631 n_clone->set_req(0, new_head_clone);
a61af66fc99e Initial load
duke
parents:
diff changeset
2632 _igvn._worklist.push(n_clone);
a61af66fc99e Initial load
duke
parents:
diff changeset
2633 }
a61af66fc99e Initial load
duke
parents:
diff changeset
2634 }
a61af66fc99e Initial load
duke
parents:
diff changeset
2635
a61af66fc99e Initial load
duke
parents:
diff changeset
2636 // Backedge of the surviving new_head (the clone) is original last_peel
a61af66fc99e Initial load
duke
parents:
diff changeset
2637 _igvn.hash_delete(new_head_clone);
a61af66fc99e Initial load
duke
parents:
diff changeset
2638 new_head_clone->set_req(LoopNode::LoopBackControl, last_peel);
a61af66fc99e Initial load
duke
parents:
diff changeset
2639 _igvn._worklist.push(new_head_clone);
a61af66fc99e Initial load
duke
parents:
diff changeset
2640
a61af66fc99e Initial load
duke
parents:
diff changeset
2641 // Cut first node in original not_peel set
a61af66fc99e Initial load
duke
parents:
diff changeset
2642 _igvn.hash_delete(new_head);
a61af66fc99e Initial load
duke
parents:
diff changeset
2643 new_head->set_req(LoopNode::EntryControl, C->top());
a61af66fc99e Initial load
duke
parents:
diff changeset
2644 new_head->set_req(LoopNode::LoopBackControl, C->top());
a61af66fc99e Initial load
duke
parents:
diff changeset
2645 _igvn._worklist.push(new_head);
a61af66fc99e Initial load
duke
parents:
diff changeset
2646
a61af66fc99e Initial load
duke
parents:
diff changeset
2647 // Copy head_clone back-branch info to original head
a61af66fc99e Initial load
duke
parents:
diff changeset
2648 // and remove original head's loop entry and
a61af66fc99e Initial load
duke
parents:
diff changeset
2649 // clone head's back-branch
a61af66fc99e Initial load
duke
parents:
diff changeset
2650 _igvn.hash_delete(head);
a61af66fc99e Initial load
duke
parents:
diff changeset
2651 _igvn.hash_delete(head_clone);
a61af66fc99e Initial load
duke
parents:
diff changeset
2652 head->set_req(LoopNode::EntryControl, head_clone->in(LoopNode::LoopBackControl));
a61af66fc99e Initial load
duke
parents:
diff changeset
2653 head->set_req(LoopNode::LoopBackControl, C->top());
a61af66fc99e Initial load
duke
parents:
diff changeset
2654 head_clone->set_req(LoopNode::LoopBackControl, C->top());
a61af66fc99e Initial load
duke
parents:
diff changeset
2655 _igvn._worklist.push(head);
a61af66fc99e Initial load
duke
parents:
diff changeset
2656 _igvn._worklist.push(head_clone);
a61af66fc99e Initial load
duke
parents:
diff changeset
2657
a61af66fc99e Initial load
duke
parents:
diff changeset
2658 // Similarly modify the phis
a61af66fc99e Initial load
duke
parents:
diff changeset
2659 for (DUIterator_Fast kmax, k = head->fast_outs(kmax); k < kmax; k++) {
a61af66fc99e Initial load
duke
parents:
diff changeset
2660 Node* use = head->fast_out(k);
a61af66fc99e Initial load
duke
parents:
diff changeset
2661 if (use->is_Phi() && use->outcnt() > 0) {
a61af66fc99e Initial load
duke
parents:
diff changeset
2662 Node* use_clone = old_new[use->_idx];
a61af66fc99e Initial load
duke
parents:
diff changeset
2663 _igvn.hash_delete(use);
a61af66fc99e Initial load
duke
parents:
diff changeset
2664 _igvn.hash_delete(use_clone);
a61af66fc99e Initial load
duke
parents:
diff changeset
2665 use->set_req(LoopNode::EntryControl, use_clone->in(LoopNode::LoopBackControl));
a61af66fc99e Initial load
duke
parents:
diff changeset
2666 use->set_req(LoopNode::LoopBackControl, C->top());
a61af66fc99e Initial load
duke
parents:
diff changeset
2667 use_clone->set_req(LoopNode::LoopBackControl, C->top());
a61af66fc99e Initial load
duke
parents:
diff changeset
2668 _igvn._worklist.push(use);
a61af66fc99e Initial load
duke
parents:
diff changeset
2669 _igvn._worklist.push(use_clone);
a61af66fc99e Initial load
duke
parents:
diff changeset
2670 }
a61af66fc99e Initial load
duke
parents:
diff changeset
2671 }
a61af66fc99e Initial load
duke
parents:
diff changeset
2672
a61af66fc99e Initial load
duke
parents:
diff changeset
2673 // Step 4: update dominator tree and dominator depth
a61af66fc99e Initial load
duke
parents:
diff changeset
2674
a61af66fc99e Initial load
duke
parents:
diff changeset
2675 set_idom(head, orig_tail_clone, dd);
a61af66fc99e Initial load
duke
parents:
diff changeset
2676 recompute_dom_depth();
a61af66fc99e Initial load
duke
parents:
diff changeset
2677
a61af66fc99e Initial load
duke
parents:
diff changeset
2678 // Inhibit more partial peeling on this loop
a61af66fc99e Initial load
duke
parents:
diff changeset
2679 new_head_clone->set_partial_peel_loop();
a61af66fc99e Initial load
duke
parents:
diff changeset
2680 C->set_major_progress();
a61af66fc99e Initial load
duke
parents:
diff changeset
2681
a61af66fc99e Initial load
duke
parents:
diff changeset
2682 #if !defined(PRODUCT)
a61af66fc99e Initial load
duke
parents:
diff changeset
2683 if (TracePartialPeeling) {
a61af66fc99e Initial load
duke
parents:
diff changeset
2684 tty->print_cr("\nafter partial peel one iteration");
a61af66fc99e Initial load
duke
parents:
diff changeset
2685 Node_List wl(area);
a61af66fc99e Initial load
duke
parents:
diff changeset
2686 Node* t = last_peel;
a61af66fc99e Initial load
duke
parents:
diff changeset
2687 while (true) {
a61af66fc99e Initial load
duke
parents:
diff changeset
2688 wl.push(t);
a61af66fc99e Initial load
duke
parents:
diff changeset
2689 if (t == head_clone) break;
a61af66fc99e Initial load
duke
parents:
diff changeset
2690 t = idom(t);
a61af66fc99e Initial load
duke
parents:
diff changeset
2691 }
a61af66fc99e Initial load
duke
parents:
diff changeset
2692 while (wl.size() > 0) {
a61af66fc99e Initial load
duke
parents:
diff changeset
2693 Node* tt = wl.pop();
a61af66fc99e Initial load
duke
parents:
diff changeset
2694 if (tt == head) tty->print_cr("orig head");
a61af66fc99e Initial load
duke
parents:
diff changeset
2695 else if (tt == new_head_clone) tty->print_cr("new head");
a61af66fc99e Initial load
duke
parents:
diff changeset
2696 else if (tt == head_clone) tty->print_cr("clone head");
a61af66fc99e Initial load
duke
parents:
diff changeset
2697 tt->dump();
a61af66fc99e Initial load
duke
parents:
diff changeset
2698 }
a61af66fc99e Initial load
duke
parents:
diff changeset
2699 }
a61af66fc99e Initial load
duke
parents:
diff changeset
2700 #endif
a61af66fc99e Initial load
duke
parents:
diff changeset
2701 return true;
a61af66fc99e Initial load
duke
parents:
diff changeset
2702 }
a61af66fc99e Initial load
duke
parents:
diff changeset
2703
a61af66fc99e Initial load
duke
parents:
diff changeset
2704 //------------------------------reorg_offsets----------------------------------
a61af66fc99e Initial load
duke
parents:
diff changeset
2705 // Reorganize offset computations to lower register pressure. Mostly
a61af66fc99e Initial load
duke
parents:
diff changeset
2706 // prevent loop-fallout uses of the pre-incremented trip counter (which are
a61af66fc99e Initial load
duke
parents:
diff changeset
2707 // then alive with the post-incremented trip counter forcing an extra
a61af66fc99e Initial load
duke
parents:
diff changeset
2708 // register move)
2383
9dc311b8473e 7008866: Missing loop predicate for loop with multiple entries
kvn
parents: 1972
diff changeset
2709 void PhaseIdealLoop::reorg_offsets(IdealLoopTree *loop) {
9dc311b8473e 7008866: Missing loop predicate for loop with multiple entries
kvn
parents: 1972
diff changeset
2710 // Perform it only for canonical counted loops.
9dc311b8473e 7008866: Missing loop predicate for loop with multiple entries
kvn
parents: 1972
diff changeset
2711 // Loop's shape could be messed up by iteration_split_impl.
9dc311b8473e 7008866: Missing loop predicate for loop with multiple entries
kvn
parents: 1972
diff changeset
2712 if (!loop->_head->is_CountedLoop())
9dc311b8473e 7008866: Missing loop predicate for loop with multiple entries
kvn
parents: 1972
diff changeset
2713 return;
9dc311b8473e 7008866: Missing loop predicate for loop with multiple entries
kvn
parents: 1972
diff changeset
2714 if (!loop->_head->as_Loop()->is_valid_counted_loop())
9dc311b8473e 7008866: Missing loop predicate for loop with multiple entries
kvn
parents: 1972
diff changeset
2715 return;
0
a61af66fc99e Initial load
duke
parents:
diff changeset
2716
a61af66fc99e Initial load
duke
parents:
diff changeset
2717 CountedLoopNode *cl = loop->_head->as_CountedLoop();
a61af66fc99e Initial load
duke
parents:
diff changeset
2718 CountedLoopEndNode *cle = cl->loopexit();
a61af66fc99e Initial load
duke
parents:
diff changeset
2719 Node *exit = cle->proj_out(false);
2383
9dc311b8473e 7008866: Missing loop predicate for loop with multiple entries
kvn
parents: 1972
diff changeset
2720 Node *phi = cl->phi();
0
a61af66fc99e Initial load
duke
parents:
diff changeset
2721
a61af66fc99e Initial load
duke
parents:
diff changeset
2722 // Check for the special case of folks using the pre-incremented
a61af66fc99e Initial load
duke
parents:
diff changeset
2723 // trip-counter on the fall-out path (forces the pre-incremented
a61af66fc99e Initial load
duke
parents:
diff changeset
2724 // and post-incremented trip counter to be live at the same time).
a61af66fc99e Initial load
duke
parents:
diff changeset
2725 // Fix this by adjusting to use the post-increment trip counter.
367
194b8e3a2fc4 6384206: Phis which are later unneeded are impairing our ability to inline based on static types
never
parents: 318
diff changeset
2726
0
a61af66fc99e Initial load
duke
parents:
diff changeset
2727 bool progress = true;
a61af66fc99e Initial load
duke
parents:
diff changeset
2728 while (progress) {
a61af66fc99e Initial load
duke
parents:
diff changeset
2729 progress = false;
a61af66fc99e Initial load
duke
parents:
diff changeset
2730 for (DUIterator_Fast imax, i = phi->fast_outs(imax); i < imax; i++) {
a61af66fc99e Initial load
duke
parents:
diff changeset
2731 Node* use = phi->fast_out(i); // User of trip-counter
a61af66fc99e Initial load
duke
parents:
diff changeset
2732 if (!has_ctrl(use)) continue;
a61af66fc99e Initial load
duke
parents:
diff changeset
2733 Node *u_ctrl = get_ctrl(use);
2383
9dc311b8473e 7008866: Missing loop predicate for loop with multiple entries
kvn
parents: 1972
diff changeset
2734 if (use->is_Phi()) {
0
a61af66fc99e Initial load
duke
parents:
diff changeset
2735 u_ctrl = NULL;
2383
9dc311b8473e 7008866: Missing loop predicate for loop with multiple entries
kvn
parents: 1972
diff changeset
2736 for (uint j = 1; j < use->req(); j++)
9dc311b8473e 7008866: Missing loop predicate for loop with multiple entries
kvn
parents: 1972
diff changeset
2737 if (use->in(j) == phi)
9dc311b8473e 7008866: Missing loop predicate for loop with multiple entries
kvn
parents: 1972
diff changeset
2738 u_ctrl = dom_lca(u_ctrl, use->in(0)->in(j));
0
a61af66fc99e Initial load
duke
parents:
diff changeset
2739 }
a61af66fc99e Initial load
duke
parents:
diff changeset
2740 IdealLoopTree *u_loop = get_loop(u_ctrl);
a61af66fc99e Initial load
duke
parents:
diff changeset
2741 // Look for loop-invariant use
2383
9dc311b8473e 7008866: Missing loop predicate for loop with multiple entries
kvn
parents: 1972
diff changeset
2742 if (u_loop == loop) continue;
9dc311b8473e 7008866: Missing loop predicate for loop with multiple entries
kvn
parents: 1972
diff changeset
2743 if (loop->is_member(u_loop)) continue;
0
a61af66fc99e Initial load
duke
parents:
diff changeset
2744 // Check that use is live out the bottom. Assuming the trip-counter
a61af66fc99e Initial load
duke
parents:
diff changeset
2745 // update is right at the bottom, uses of of the loop middle are ok.
2383
9dc311b8473e 7008866: Missing loop predicate for loop with multiple entries
kvn
parents: 1972
diff changeset
2746 if (dom_lca(exit, u_ctrl) != exit) continue;
0
a61af66fc99e Initial load
duke
parents:
diff changeset
2747 // Hit! Refactor use to use the post-incremented tripcounter.
a61af66fc99e Initial load
duke
parents:
diff changeset
2748 // Compute a post-increment tripcounter.
216
8d191a7697e2 6715633: when matching a memory node the adr_type should not change
kvn
parents: 164
diff changeset
2749 Node *opaq = new (C, 2) Opaque2Node( C, cle->incr() );
0
a61af66fc99e Initial load
duke
parents:
diff changeset
2750 register_new_node( opaq, u_ctrl );
a61af66fc99e Initial load
duke
parents:
diff changeset
2751 Node *neg_stride = _igvn.intcon(-cle->stride_con());
a61af66fc99e Initial load
duke
parents:
diff changeset
2752 set_ctrl(neg_stride, C->root());
a61af66fc99e Initial load
duke
parents:
diff changeset
2753 Node *post = new (C, 3) AddINode( opaq, neg_stride);
a61af66fc99e Initial load
duke
parents:
diff changeset
2754 register_new_node( post, u_ctrl );
a61af66fc99e Initial load
duke
parents:
diff changeset
2755 _igvn.hash_delete(use);
a61af66fc99e Initial load
duke
parents:
diff changeset
2756 _igvn._worklist.push(use);
2383
9dc311b8473e 7008866: Missing loop predicate for loop with multiple entries
kvn
parents: 1972
diff changeset
2757 for (uint j = 1; j < use->req(); j++) {
9dc311b8473e 7008866: Missing loop predicate for loop with multiple entries
kvn
parents: 1972
diff changeset
2758 if (use->in(j) == phi)
0
a61af66fc99e Initial load
duke
parents:
diff changeset
2759 use->set_req(j, post);
2383
9dc311b8473e 7008866: Missing loop predicate for loop with multiple entries
kvn
parents: 1972
diff changeset
2760 }
0
a61af66fc99e Initial load
duke
parents:
diff changeset
2761 // Since DU info changed, rerun loop
a61af66fc99e Initial load
duke
parents:
diff changeset
2762 progress = true;
a61af66fc99e Initial load
duke
parents:
diff changeset
2763 break;
a61af66fc99e Initial load
duke
parents:
diff changeset
2764 }
a61af66fc99e Initial load
duke
parents:
diff changeset
2765 }
a61af66fc99e Initial load
duke
parents:
diff changeset
2766
a61af66fc99e Initial load
duke
parents:
diff changeset
2767 }