annotate src/share/vm/opto/loopopts.cpp @ 3992:d1bdeef3e3e2

7098282: G1: assert(interval >= 0) failed: Sanity check, referencePolicy.cpp: 76 Summary: There is a race between one thread successfully forwarding and copying the klass mirror for the SoftReference class (including the static master clock) and another thread attempting to use the master clock while attempting to discover a soft reference object. Maintain a shadow copy of the soft reference master clock and use the shadow during reference discovery and reference processing. Reviewed-by: tonyp, brutisso, ysr
author johnc
date Wed, 12 Oct 2011 10:25:51 -0700
parents 4e761e7e6e12
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 }