annotate src/share/vm/opto/loopopts.cpp @ 4062:83d0b5cd1438

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