annotate src/share/vm/opto/lcm.cpp @ 4710:41406797186b

7113012: G1: rename not-fully-young GCs as "mixed" Summary: Renamed partially-young GCs as mixed and fully-young GCs as young. Change all external output that includes those terms (GC log and GC ergo log) as well as any comments, fields, methods, etc. The changeset also includes very minor code tidying up (added some curly brackets). Reviewed-by: johnc, brutisso
author tonyp
date Fri, 16 Dec 2011 02:14:27 -0500
parents f03a3c8bd5e5
children cf407b7d3d78
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: 2401
diff changeset
2 * Copyright (c) 1998, 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: 1151
diff changeset
19 * Please contact Oracle, 500 Oracle Parkway, Redwood Shores, CA 94065 USA
c18cbe5936b8 6941466: Oracle rebranding changes for Hotspot repositories
trims
parents: 1151
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: 1151
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: 1748
diff changeset
25 #include "precompiled.hpp"
f95d63e2154a 6989984: Use standard include model for Hospot
stefank
parents: 1748
diff changeset
26 #include "memory/allocation.inline.hpp"
f95d63e2154a 6989984: Use standard include model for Hospot
stefank
parents: 1748
diff changeset
27 #include "opto/block.hpp"
f95d63e2154a 6989984: Use standard include model for Hospot
stefank
parents: 1748
diff changeset
28 #include "opto/c2compiler.hpp"
f95d63e2154a 6989984: Use standard include model for Hospot
stefank
parents: 1748
diff changeset
29 #include "opto/callnode.hpp"
f95d63e2154a 6989984: Use standard include model for Hospot
stefank
parents: 1748
diff changeset
30 #include "opto/cfgnode.hpp"
f95d63e2154a 6989984: Use standard include model for Hospot
stefank
parents: 1748
diff changeset
31 #include "opto/machnode.hpp"
f95d63e2154a 6989984: Use standard include model for Hospot
stefank
parents: 1748
diff changeset
32 #include "opto/runtime.hpp"
f95d63e2154a 6989984: Use standard include model for Hospot
stefank
parents: 1748
diff changeset
33 #ifdef TARGET_ARCH_MODEL_x86_32
f95d63e2154a 6989984: Use standard include model for Hospot
stefank
parents: 1748
diff changeset
34 # include "adfiles/ad_x86_32.hpp"
f95d63e2154a 6989984: Use standard include model for Hospot
stefank
parents: 1748
diff changeset
35 #endif
f95d63e2154a 6989984: Use standard include model for Hospot
stefank
parents: 1748
diff changeset
36 #ifdef TARGET_ARCH_MODEL_x86_64
f95d63e2154a 6989984: Use standard include model for Hospot
stefank
parents: 1748
diff changeset
37 # include "adfiles/ad_x86_64.hpp"
f95d63e2154a 6989984: Use standard include model for Hospot
stefank
parents: 1748
diff changeset
38 #endif
f95d63e2154a 6989984: Use standard include model for Hospot
stefank
parents: 1748
diff changeset
39 #ifdef TARGET_ARCH_MODEL_sparc
f95d63e2154a 6989984: Use standard include model for Hospot
stefank
parents: 1748
diff changeset
40 # include "adfiles/ad_sparc.hpp"
f95d63e2154a 6989984: Use standard include model for Hospot
stefank
parents: 1748
diff changeset
41 #endif
f95d63e2154a 6989984: Use standard include model for Hospot
stefank
parents: 1748
diff changeset
42 #ifdef TARGET_ARCH_MODEL_zero
f95d63e2154a 6989984: Use standard include model for Hospot
stefank
parents: 1748
diff changeset
43 # include "adfiles/ad_zero.hpp"
f95d63e2154a 6989984: Use standard include model for Hospot
stefank
parents: 1748
diff changeset
44 #endif
2401
7e88bdae86ec 7029017: Additional architecture support for c2 compiler
roland
parents: 1972
diff changeset
45 #ifdef TARGET_ARCH_MODEL_arm
7e88bdae86ec 7029017: Additional architecture support for c2 compiler
roland
parents: 1972
diff changeset
46 # include "adfiles/ad_arm.hpp"
7e88bdae86ec 7029017: Additional architecture support for c2 compiler
roland
parents: 1972
diff changeset
47 #endif
3796
7d9e451f5416 7061187: need some includes for arm/ppc
jcoomes
parents: 3252
diff changeset
48 #ifdef TARGET_ARCH_MODEL_ppc
7d9e451f5416 7061187: need some includes for arm/ppc
jcoomes
parents: 3252
diff changeset
49 # include "adfiles/ad_ppc.hpp"
7d9e451f5416 7061187: need some includes for arm/ppc
jcoomes
parents: 3252
diff changeset
50 #endif
0
a61af66fc99e Initial load
duke
parents:
diff changeset
51
1972
f95d63e2154a 6989984: Use standard include model for Hospot
stefank
parents: 1748
diff changeset
52 // Optimization - Graph Style
0
a61af66fc99e Initial load
duke
parents:
diff changeset
53
a61af66fc99e Initial load
duke
parents:
diff changeset
54 //------------------------------implicit_null_check----------------------------
a61af66fc99e Initial load
duke
parents:
diff changeset
55 // Detect implicit-null-check opportunities. Basically, find NULL checks
a61af66fc99e Initial load
duke
parents:
diff changeset
56 // with suitable memory ops nearby. Use the memory op to do the NULL check.
a61af66fc99e Initial load
duke
parents:
diff changeset
57 // I can generate a memory op if there is not one nearby.
a61af66fc99e Initial load
duke
parents:
diff changeset
58 // The proj is the control projection for the not-null case.
1575
3657cb01ffc5 6954029: Improve implicit null check generation with compressed oops
kvn
parents: 1151
diff changeset
59 // The val is the pointer being checked for nullness or
3657cb01ffc5 6954029: Improve implicit null check generation with compressed oops
kvn
parents: 1151
diff changeset
60 // decodeHeapOop_not_null node if it did not fold into address.
0
a61af66fc99e Initial load
duke
parents:
diff changeset
61 void Block::implicit_null_check(PhaseCFG *cfg, Node *proj, Node *val, int allowed_reasons) {
a61af66fc99e Initial load
duke
parents:
diff changeset
62 // Assume if null check need for 0 offset then always needed
a61af66fc99e Initial load
duke
parents:
diff changeset
63 // Intel solaris doesn't support any null checks yet and no
a61af66fc99e Initial load
duke
parents:
diff changeset
64 // mechanism exists (yet) to set the switches at an os_cpu level
a61af66fc99e Initial load
duke
parents:
diff changeset
65 if( !ImplicitNullChecks || MacroAssembler::needs_explicit_null_check(0)) return;
a61af66fc99e Initial load
duke
parents:
diff changeset
66
a61af66fc99e Initial load
duke
parents:
diff changeset
67 // Make sure the ptr-is-null path appears to be uncommon!
a61af66fc99e Initial load
duke
parents:
diff changeset
68 float f = end()->as_MachIf()->_prob;
a61af66fc99e Initial load
duke
parents:
diff changeset
69 if( proj->Opcode() == Op_IfTrue ) f = 1.0f - f;
a61af66fc99e Initial load
duke
parents:
diff changeset
70 if( f > PROB_UNLIKELY_MAG(4) ) return;
a61af66fc99e Initial load
duke
parents:
diff changeset
71
a61af66fc99e Initial load
duke
parents:
diff changeset
72 uint bidx = 0; // Capture index of value into memop
a61af66fc99e Initial load
duke
parents:
diff changeset
73 bool was_store; // Memory op is a store op
a61af66fc99e Initial load
duke
parents:
diff changeset
74
a61af66fc99e Initial load
duke
parents:
diff changeset
75 // Get the successor block for if the test ptr is non-null
a61af66fc99e Initial load
duke
parents:
diff changeset
76 Block* not_null_block; // this one goes with the proj
a61af66fc99e Initial load
duke
parents:
diff changeset
77 Block* null_block;
a61af66fc99e Initial load
duke
parents:
diff changeset
78 if (_nodes[_nodes.size()-1] == proj) {
a61af66fc99e Initial load
duke
parents:
diff changeset
79 null_block = _succs[0];
a61af66fc99e Initial load
duke
parents:
diff changeset
80 not_null_block = _succs[1];
a61af66fc99e Initial load
duke
parents:
diff changeset
81 } else {
a61af66fc99e Initial load
duke
parents:
diff changeset
82 assert(_nodes[_nodes.size()-2] == proj, "proj is one or the other");
a61af66fc99e Initial load
duke
parents:
diff changeset
83 not_null_block = _succs[0];
a61af66fc99e Initial load
duke
parents:
diff changeset
84 null_block = _succs[1];
a61af66fc99e Initial load
duke
parents:
diff changeset
85 }
332
c792b641b8bd 6746907: Improve implicit null check generation
kvn
parents: 253
diff changeset
86 while (null_block->is_Empty() == Block::empty_with_goto) {
c792b641b8bd 6746907: Improve implicit null check generation
kvn
parents: 253
diff changeset
87 null_block = null_block->_succs[0];
c792b641b8bd 6746907: Improve implicit null check generation
kvn
parents: 253
diff changeset
88 }
0
a61af66fc99e Initial load
duke
parents:
diff changeset
89
a61af66fc99e Initial load
duke
parents:
diff changeset
90 // Search the exception block for an uncommon trap.
a61af66fc99e Initial load
duke
parents:
diff changeset
91 // (See Parse::do_if and Parse::do_ifnull for the reason
a61af66fc99e Initial load
duke
parents:
diff changeset
92 // we need an uncommon trap. Briefly, we need a way to
a61af66fc99e Initial load
duke
parents:
diff changeset
93 // detect failure of this optimization, as in 6366351.)
a61af66fc99e Initial load
duke
parents:
diff changeset
94 {
a61af66fc99e Initial load
duke
parents:
diff changeset
95 bool found_trap = false;
a61af66fc99e Initial load
duke
parents:
diff changeset
96 for (uint i1 = 0; i1 < null_block->_nodes.size(); i1++) {
a61af66fc99e Initial load
duke
parents:
diff changeset
97 Node* nn = null_block->_nodes[i1];
a61af66fc99e Initial load
duke
parents:
diff changeset
98 if (nn->is_MachCall() &&
1748
3e8fbc61cee8 6978355: renaming for 6961697
twisti
parents: 1693
diff changeset
99 nn->as_MachCall()->entry_point() == SharedRuntime::uncommon_trap_blob()->entry_point()) {
0
a61af66fc99e Initial load
duke
parents:
diff changeset
100 const Type* trtype = nn->in(TypeFunc::Parms)->bottom_type();
a61af66fc99e Initial load
duke
parents:
diff changeset
101 if (trtype->isa_int() && trtype->is_int()->is_con()) {
a61af66fc99e Initial load
duke
parents:
diff changeset
102 jint tr_con = trtype->is_int()->get_con();
a61af66fc99e Initial load
duke
parents:
diff changeset
103 Deoptimization::DeoptReason reason = Deoptimization::trap_request_reason(tr_con);
a61af66fc99e Initial load
duke
parents:
diff changeset
104 Deoptimization::DeoptAction action = Deoptimization::trap_request_action(tr_con);
a61af66fc99e Initial load
duke
parents:
diff changeset
105 assert((int)reason < (int)BitsPerInt, "recode bit map");
a61af66fc99e Initial load
duke
parents:
diff changeset
106 if (is_set_nth_bit(allowed_reasons, (int) reason)
a61af66fc99e Initial load
duke
parents:
diff changeset
107 && action != Deoptimization::Action_none) {
a61af66fc99e Initial load
duke
parents:
diff changeset
108 // This uncommon trap is sure to recompile, eventually.
a61af66fc99e Initial load
duke
parents:
diff changeset
109 // When that happens, C->too_many_traps will prevent
a61af66fc99e Initial load
duke
parents:
diff changeset
110 // this transformation from happening again.
a61af66fc99e Initial load
duke
parents:
diff changeset
111 found_trap = true;
a61af66fc99e Initial load
duke
parents:
diff changeset
112 }
a61af66fc99e Initial load
duke
parents:
diff changeset
113 }
a61af66fc99e Initial load
duke
parents:
diff changeset
114 break;
a61af66fc99e Initial load
duke
parents:
diff changeset
115 }
a61af66fc99e Initial load
duke
parents:
diff changeset
116 }
a61af66fc99e Initial load
duke
parents:
diff changeset
117 if (!found_trap) {
a61af66fc99e Initial load
duke
parents:
diff changeset
118 // We did not find an uncommon trap.
a61af66fc99e Initial load
duke
parents:
diff changeset
119 return;
a61af66fc99e Initial load
duke
parents:
diff changeset
120 }
a61af66fc99e Initial load
duke
parents:
diff changeset
121 }
a61af66fc99e Initial load
duke
parents:
diff changeset
122
1575
3657cb01ffc5 6954029: Improve implicit null check generation with compressed oops
kvn
parents: 1151
diff changeset
123 // Check for decodeHeapOop_not_null node which did not fold into address
3657cb01ffc5 6954029: Improve implicit null check generation with compressed oops
kvn
parents: 1151
diff changeset
124 bool is_decoden = ((intptr_t)val) & 1;
3657cb01ffc5 6954029: Improve implicit null check generation with compressed oops
kvn
parents: 1151
diff changeset
125 val = (Node*)(((intptr_t)val) & ~1);
3657cb01ffc5 6954029: Improve implicit null check generation with compressed oops
kvn
parents: 1151
diff changeset
126
3657cb01ffc5 6954029: Improve implicit null check generation with compressed oops
kvn
parents: 1151
diff changeset
127 assert(!is_decoden || (val->in(0) == NULL) && val->is_Mach() &&
3657cb01ffc5 6954029: Improve implicit null check generation with compressed oops
kvn
parents: 1151
diff changeset
128 (val->as_Mach()->ideal_Opcode() == Op_DecodeN), "sanity");
3657cb01ffc5 6954029: Improve implicit null check generation with compressed oops
kvn
parents: 1151
diff changeset
129
0
a61af66fc99e Initial load
duke
parents:
diff changeset
130 // Search the successor block for a load or store who's base value is also
a61af66fc99e Initial load
duke
parents:
diff changeset
131 // the tested value. There may be several.
a61af66fc99e Initial load
duke
parents:
diff changeset
132 Node_List *out = new Node_List(Thread::current()->resource_area());
a61af66fc99e Initial load
duke
parents:
diff changeset
133 MachNode *best = NULL; // Best found so far
a61af66fc99e Initial load
duke
parents:
diff changeset
134 for (DUIterator i = val->outs(); val->has_out(i); i++) {
a61af66fc99e Initial load
duke
parents:
diff changeset
135 Node *m = val->out(i);
a61af66fc99e Initial load
duke
parents:
diff changeset
136 if( !m->is_Mach() ) continue;
a61af66fc99e Initial load
duke
parents:
diff changeset
137 MachNode *mach = m->as_Mach();
a61af66fc99e Initial load
duke
parents:
diff changeset
138 was_store = false;
1693
6c9cc03d8726 6973329: C2 with Zero based COOP produces code with broken anti-dependency on x86
kvn
parents: 1685
diff changeset
139 int iop = mach->ideal_Opcode();
6c9cc03d8726 6973329: C2 with Zero based COOP produces code with broken anti-dependency on x86
kvn
parents: 1685
diff changeset
140 switch( iop ) {
0
a61af66fc99e Initial load
duke
parents:
diff changeset
141 case Op_LoadB:
558
3b5ac9e7e6ea 6796746: rename LoadC (char) opcode class to LoadUS (unsigned short)
twisti
parents: 365
diff changeset
142 case Op_LoadUS:
0
a61af66fc99e Initial load
duke
parents:
diff changeset
143 case Op_LoadD:
a61af66fc99e Initial load
duke
parents:
diff changeset
144 case Op_LoadF:
a61af66fc99e Initial load
duke
parents:
diff changeset
145 case Op_LoadI:
a61af66fc99e Initial load
duke
parents:
diff changeset
146 case Op_LoadL:
a61af66fc99e Initial load
duke
parents:
diff changeset
147 case Op_LoadP:
113
ba764ed4b6f2 6420645: Create a vm that uses compressed oops for up to 32gb heapsizes
coleenp
parents: 0
diff changeset
148 case Op_LoadN:
0
a61af66fc99e Initial load
duke
parents:
diff changeset
149 case Op_LoadS:
a61af66fc99e Initial load
duke
parents:
diff changeset
150 case Op_LoadKlass:
164
c436414a719e 6703890: Compressed Oops: add LoadNKlass node to generate narrow oops (32-bits) compare instructions
kvn
parents: 125
diff changeset
151 case Op_LoadNKlass:
0
a61af66fc99e Initial load
duke
parents:
diff changeset
152 case Op_LoadRange:
a61af66fc99e Initial load
duke
parents:
diff changeset
153 case Op_LoadD_unaligned:
a61af66fc99e Initial load
duke
parents:
diff changeset
154 case Op_LoadL_unaligned:
1151
1271af4ec18c 6912517: JIT bug compiles out (and stops running) code that needs to be run. Causes NPE.
kvn
parents: 1137
diff changeset
155 assert(mach->in(2) == val, "should be address");
0
a61af66fc99e Initial load
duke
parents:
diff changeset
156 break;
a61af66fc99e Initial load
duke
parents:
diff changeset
157 case Op_StoreB:
a61af66fc99e Initial load
duke
parents:
diff changeset
158 case Op_StoreC:
a61af66fc99e Initial load
duke
parents:
diff changeset
159 case Op_StoreCM:
a61af66fc99e Initial load
duke
parents:
diff changeset
160 case Op_StoreD:
a61af66fc99e Initial load
duke
parents:
diff changeset
161 case Op_StoreF:
a61af66fc99e Initial load
duke
parents:
diff changeset
162 case Op_StoreI:
a61af66fc99e Initial load
duke
parents:
diff changeset
163 case Op_StoreL:
a61af66fc99e Initial load
duke
parents:
diff changeset
164 case Op_StoreP:
113
ba764ed4b6f2 6420645: Create a vm that uses compressed oops for up to 32gb heapsizes
coleenp
parents: 0
diff changeset
165 case Op_StoreN:
0
a61af66fc99e Initial load
duke
parents:
diff changeset
166 was_store = true; // Memory op is a store op
a61af66fc99e Initial load
duke
parents:
diff changeset
167 // Stores will have their address in slot 2 (memory in slot 1).
a61af66fc99e Initial load
duke
parents:
diff changeset
168 // If the value being nul-checked is in another slot, it means we
a61af66fc99e Initial load
duke
parents:
diff changeset
169 // are storing the checked value, which does NOT check the value!
a61af66fc99e Initial load
duke
parents:
diff changeset
170 if( mach->in(2) != val ) continue;
a61af66fc99e Initial load
duke
parents:
diff changeset
171 break; // Found a memory op?
a61af66fc99e Initial load
duke
parents:
diff changeset
172 case Op_StrComp:
681
fbde8ec322d0 6761600: Use sse 4.2 in intrinsics
cfang
parents: 647
diff changeset
173 case Op_StrEquals:
fbde8ec322d0 6761600: Use sse 4.2 in intrinsics
cfang
parents: 647
diff changeset
174 case Op_StrIndexOf:
169
9148c65abefc 6695049: (coll) Create an x86 intrinsic for Arrays.equals
rasbold
parents: 164
diff changeset
175 case Op_AryEq:
0
a61af66fc99e Initial load
duke
parents:
diff changeset
176 // Not a legit memory op for implicit null check regardless of
a61af66fc99e Initial load
duke
parents:
diff changeset
177 // embedded loads
a61af66fc99e Initial load
duke
parents:
diff changeset
178 continue;
a61af66fc99e Initial load
duke
parents:
diff changeset
179 default: // Also check for embedded loads
a61af66fc99e Initial load
duke
parents:
diff changeset
180 if( !mach->needs_anti_dependence_check() )
a61af66fc99e Initial load
duke
parents:
diff changeset
181 continue; // Not an memory op; skip it
1693
6c9cc03d8726 6973329: C2 with Zero based COOP produces code with broken anti-dependency on x86
kvn
parents: 1685
diff changeset
182 if( must_clone[iop] ) {
6c9cc03d8726 6973329: C2 with Zero based COOP produces code with broken anti-dependency on x86
kvn
parents: 1685
diff changeset
183 // Do not move nodes which produce flags because
6c9cc03d8726 6973329: C2 with Zero based COOP produces code with broken anti-dependency on x86
kvn
parents: 1685
diff changeset
184 // RA will try to clone it to place near branch and
6c9cc03d8726 6973329: C2 with Zero based COOP produces code with broken anti-dependency on x86
kvn
parents: 1685
diff changeset
185 // it will cause recompilation, see clone_node().
6c9cc03d8726 6973329: C2 with Zero based COOP produces code with broken anti-dependency on x86
kvn
parents: 1685
diff changeset
186 continue;
6c9cc03d8726 6973329: C2 with Zero based COOP produces code with broken anti-dependency on x86
kvn
parents: 1685
diff changeset
187 }
1151
1271af4ec18c 6912517: JIT bug compiles out (and stops running) code that needs to be run. Causes NPE.
kvn
parents: 1137
diff changeset
188 {
1575
3657cb01ffc5 6954029: Improve implicit null check generation with compressed oops
kvn
parents: 1151
diff changeset
189 // Check that value is used in memory address in
3657cb01ffc5 6954029: Improve implicit null check generation with compressed oops
kvn
parents: 1151
diff changeset
190 // instructions with embedded load (CmpP val1,(val2+off)).
1151
1271af4ec18c 6912517: JIT bug compiles out (and stops running) code that needs to be run. Causes NPE.
kvn
parents: 1137
diff changeset
191 Node* base;
1271af4ec18c 6912517: JIT bug compiles out (and stops running) code that needs to be run. Causes NPE.
kvn
parents: 1137
diff changeset
192 Node* index;
1271af4ec18c 6912517: JIT bug compiles out (and stops running) code that needs to be run. Causes NPE.
kvn
parents: 1137
diff changeset
193 const MachOper* oper = mach->memory_inputs(base, index);
1271af4ec18c 6912517: JIT bug compiles out (and stops running) code that needs to be run. Causes NPE.
kvn
parents: 1137
diff changeset
194 if (oper == NULL || oper == (MachOper*)-1) {
1271af4ec18c 6912517: JIT bug compiles out (and stops running) code that needs to be run. Causes NPE.
kvn
parents: 1137
diff changeset
195 continue; // Not an memory op; skip it
1271af4ec18c 6912517: JIT bug compiles out (and stops running) code that needs to be run. Causes NPE.
kvn
parents: 1137
diff changeset
196 }
1271af4ec18c 6912517: JIT bug compiles out (and stops running) code that needs to be run. Causes NPE.
kvn
parents: 1137
diff changeset
197 if (val == base ||
1271af4ec18c 6912517: JIT bug compiles out (and stops running) code that needs to be run. Causes NPE.
kvn
parents: 1137
diff changeset
198 val == index && val->bottom_type()->isa_narrowoop()) {
1271af4ec18c 6912517: JIT bug compiles out (and stops running) code that needs to be run. Causes NPE.
kvn
parents: 1137
diff changeset
199 break; // Found it
1271af4ec18c 6912517: JIT bug compiles out (and stops running) code that needs to be run. Causes NPE.
kvn
parents: 1137
diff changeset
200 } else {
1271af4ec18c 6912517: JIT bug compiles out (and stops running) code that needs to be run. Causes NPE.
kvn
parents: 1137
diff changeset
201 continue; // Skip it
1271af4ec18c 6912517: JIT bug compiles out (and stops running) code that needs to be run. Causes NPE.
kvn
parents: 1137
diff changeset
202 }
1271af4ec18c 6912517: JIT bug compiles out (and stops running) code that needs to be run. Causes NPE.
kvn
parents: 1137
diff changeset
203 }
0
a61af66fc99e Initial load
duke
parents:
diff changeset
204 break;
a61af66fc99e Initial load
duke
parents:
diff changeset
205 }
a61af66fc99e Initial load
duke
parents:
diff changeset
206 // check if the offset is not too high for implicit exception
a61af66fc99e Initial load
duke
parents:
diff changeset
207 {
a61af66fc99e Initial load
duke
parents:
diff changeset
208 intptr_t offset = 0;
a61af66fc99e Initial load
duke
parents:
diff changeset
209 const TypePtr *adr_type = NULL; // Do not need this return value here
a61af66fc99e Initial load
duke
parents:
diff changeset
210 const Node* base = mach->get_base_and_disp(offset, adr_type);
a61af66fc99e Initial load
duke
parents:
diff changeset
211 if (base == NULL || base == NodeSentinel) {
332
c792b641b8bd 6746907: Improve implicit null check generation
kvn
parents: 253
diff changeset
212 // Narrow oop address doesn't have base, only index
c792b641b8bd 6746907: Improve implicit null check generation
kvn
parents: 253
diff changeset
213 if( val->bottom_type()->isa_narrowoop() &&
c792b641b8bd 6746907: Improve implicit null check generation
kvn
parents: 253
diff changeset
214 MacroAssembler::needs_explicit_null_check(offset) )
c792b641b8bd 6746907: Improve implicit null check generation
kvn
parents: 253
diff changeset
215 continue; // Give up if offset is beyond page size
0
a61af66fc99e Initial load
duke
parents:
diff changeset
216 // cannot reason about it; is probably not implicit null exception
a61af66fc99e Initial load
duke
parents:
diff changeset
217 } else {
642
660978a2a31a 6791178: Specialize for zero as the compressed oop vm heap base
kvn
parents: 558
diff changeset
218 const TypePtr* tptr;
660978a2a31a 6791178: Specialize for zero as the compressed oop vm heap base
kvn
parents: 558
diff changeset
219 if (UseCompressedOops && Universe::narrow_oop_shift() == 0) {
660978a2a31a 6791178: Specialize for zero as the compressed oop vm heap base
kvn
parents: 558
diff changeset
220 // 32-bits narrow oop can be the base of address expressions
660978a2a31a 6791178: Specialize for zero as the compressed oop vm heap base
kvn
parents: 558
diff changeset
221 tptr = base->bottom_type()->make_ptr();
660978a2a31a 6791178: Specialize for zero as the compressed oop vm heap base
kvn
parents: 558
diff changeset
222 } else {
660978a2a31a 6791178: Specialize for zero as the compressed oop vm heap base
kvn
parents: 558
diff changeset
223 // only regular oops are expected here
660978a2a31a 6791178: Specialize for zero as the compressed oop vm heap base
kvn
parents: 558
diff changeset
224 tptr = base->bottom_type()->is_ptr();
660978a2a31a 6791178: Specialize for zero as the compressed oop vm heap base
kvn
parents: 558
diff changeset
225 }
0
a61af66fc99e Initial load
duke
parents:
diff changeset
226 // Give up if offset is not a compile-time constant
a61af66fc99e Initial load
duke
parents:
diff changeset
227 if( offset == Type::OffsetBot || tptr->_offset == Type::OffsetBot )
a61af66fc99e Initial load
duke
parents:
diff changeset
228 continue;
a61af66fc99e Initial load
duke
parents:
diff changeset
229 offset += tptr->_offset; // correct if base is offseted
a61af66fc99e Initial load
duke
parents:
diff changeset
230 if( MacroAssembler::needs_explicit_null_check(offset) )
a61af66fc99e Initial load
duke
parents:
diff changeset
231 continue; // Give up is reference is beyond 4K page size
a61af66fc99e Initial load
duke
parents:
diff changeset
232 }
a61af66fc99e Initial load
duke
parents:
diff changeset
233 }
a61af66fc99e Initial load
duke
parents:
diff changeset
234
a61af66fc99e Initial load
duke
parents:
diff changeset
235 // Check ctrl input to see if the null-check dominates the memory op
a61af66fc99e Initial load
duke
parents:
diff changeset
236 Block *cb = cfg->_bbs[mach->_idx];
a61af66fc99e Initial load
duke
parents:
diff changeset
237 cb = cb->_idom; // Always hoist at least 1 block
a61af66fc99e Initial load
duke
parents:
diff changeset
238 if( !was_store ) { // Stores can be hoisted only one block
a61af66fc99e Initial load
duke
parents:
diff changeset
239 while( cb->_dom_depth > (_dom_depth + 1))
a61af66fc99e Initial load
duke
parents:
diff changeset
240 cb = cb->_idom; // Hoist loads as far as we want
a61af66fc99e Initial load
duke
parents:
diff changeset
241 // The non-null-block should dominate the memory op, too. Live
a61af66fc99e Initial load
duke
parents:
diff changeset
242 // range spilling will insert a spill in the non-null-block if it is
a61af66fc99e Initial load
duke
parents:
diff changeset
243 // needs to spill the memory op for an implicit null check.
a61af66fc99e Initial load
duke
parents:
diff changeset
244 if (cb->_dom_depth == (_dom_depth + 1)) {
a61af66fc99e Initial load
duke
parents:
diff changeset
245 if (cb != not_null_block) continue;
a61af66fc99e Initial load
duke
parents:
diff changeset
246 cb = cb->_idom;
a61af66fc99e Initial load
duke
parents:
diff changeset
247 }
a61af66fc99e Initial load
duke
parents:
diff changeset
248 }
a61af66fc99e Initial load
duke
parents:
diff changeset
249 if( cb != this ) continue;
a61af66fc99e Initial load
duke
parents:
diff changeset
250
a61af66fc99e Initial load
duke
parents:
diff changeset
251 // Found a memory user; see if it can be hoisted to check-block
a61af66fc99e Initial load
duke
parents:
diff changeset
252 uint vidx = 0; // Capture index of value into memop
a61af66fc99e Initial load
duke
parents:
diff changeset
253 uint j;
a61af66fc99e Initial load
duke
parents:
diff changeset
254 for( j = mach->req()-1; j > 0; j-- ) {
1575
3657cb01ffc5 6954029: Improve implicit null check generation with compressed oops
kvn
parents: 1151
diff changeset
255 if( mach->in(j) == val ) {
3657cb01ffc5 6954029: Improve implicit null check generation with compressed oops
kvn
parents: 1151
diff changeset
256 vidx = j;
3657cb01ffc5 6954029: Improve implicit null check generation with compressed oops
kvn
parents: 1151
diff changeset
257 // Ignore DecodeN val which could be hoisted to where needed.
3657cb01ffc5 6954029: Improve implicit null check generation with compressed oops
kvn
parents: 1151
diff changeset
258 if( is_decoden ) continue;
3657cb01ffc5 6954029: Improve implicit null check generation with compressed oops
kvn
parents: 1151
diff changeset
259 }
0
a61af66fc99e Initial load
duke
parents:
diff changeset
260 // Block of memory-op input
a61af66fc99e Initial load
duke
parents:
diff changeset
261 Block *inb = cfg->_bbs[mach->in(j)->_idx];
a61af66fc99e Initial load
duke
parents:
diff changeset
262 Block *b = this; // Start from nul check
a61af66fc99e Initial load
duke
parents:
diff changeset
263 while( b != inb && b->_dom_depth > inb->_dom_depth )
a61af66fc99e Initial load
duke
parents:
diff changeset
264 b = b->_idom; // search upwards for input
a61af66fc99e Initial load
duke
parents:
diff changeset
265 // See if input dominates null check
a61af66fc99e Initial load
duke
parents:
diff changeset
266 if( b != inb )
a61af66fc99e Initial load
duke
parents:
diff changeset
267 break;
a61af66fc99e Initial load
duke
parents:
diff changeset
268 }
a61af66fc99e Initial load
duke
parents:
diff changeset
269 if( j > 0 )
a61af66fc99e Initial load
duke
parents:
diff changeset
270 continue;
a61af66fc99e Initial load
duke
parents:
diff changeset
271 Block *mb = cfg->_bbs[mach->_idx];
a61af66fc99e Initial load
duke
parents:
diff changeset
272 // Hoisting stores requires more checks for the anti-dependence case.
a61af66fc99e Initial load
duke
parents:
diff changeset
273 // Give up hoisting if we have to move the store past any load.
a61af66fc99e Initial load
duke
parents:
diff changeset
274 if( was_store ) {
a61af66fc99e Initial load
duke
parents:
diff changeset
275 Block *b = mb; // Start searching here for a local load
a61af66fc99e Initial load
duke
parents:
diff changeset
276 // mach use (faulting) trying to hoist
a61af66fc99e Initial load
duke
parents:
diff changeset
277 // n might be blocker to hoisting
a61af66fc99e Initial load
duke
parents:
diff changeset
278 while( b != this ) {
a61af66fc99e Initial load
duke
parents:
diff changeset
279 uint k;
a61af66fc99e Initial load
duke
parents:
diff changeset
280 for( k = 1; k < b->_nodes.size(); k++ ) {
a61af66fc99e Initial load
duke
parents:
diff changeset
281 Node *n = b->_nodes[k];
a61af66fc99e Initial load
duke
parents:
diff changeset
282 if( n->needs_anti_dependence_check() &&
a61af66fc99e Initial load
duke
parents:
diff changeset
283 n->in(LoadNode::Memory) == mach->in(StoreNode::Memory) )
a61af66fc99e Initial load
duke
parents:
diff changeset
284 break; // Found anti-dependent load
a61af66fc99e Initial load
duke
parents:
diff changeset
285 }
a61af66fc99e Initial load
duke
parents:
diff changeset
286 if( k < b->_nodes.size() )
a61af66fc99e Initial load
duke
parents:
diff changeset
287 break; // Found anti-dependent load
a61af66fc99e Initial load
duke
parents:
diff changeset
288 // Make sure control does not do a merge (would have to check allpaths)
a61af66fc99e Initial load
duke
parents:
diff changeset
289 if( b->num_preds() != 2 ) break;
a61af66fc99e Initial load
duke
parents:
diff changeset
290 b = cfg->_bbs[b->pred(1)->_idx]; // Move up to predecessor block
a61af66fc99e Initial load
duke
parents:
diff changeset
291 }
a61af66fc99e Initial load
duke
parents:
diff changeset
292 if( b != this ) continue;
a61af66fc99e Initial load
duke
parents:
diff changeset
293 }
a61af66fc99e Initial load
duke
parents:
diff changeset
294
a61af66fc99e Initial load
duke
parents:
diff changeset
295 // Make sure this memory op is not already being used for a NullCheck
a61af66fc99e Initial load
duke
parents:
diff changeset
296 Node *e = mb->end();
a61af66fc99e Initial load
duke
parents:
diff changeset
297 if( e->is_MachNullCheck() && e->in(1) == mach )
a61af66fc99e Initial load
duke
parents:
diff changeset
298 continue; // Already being used as a NULL check
a61af66fc99e Initial load
duke
parents:
diff changeset
299
a61af66fc99e Initial load
duke
parents:
diff changeset
300 // Found a candidate! Pick one with least dom depth - the highest
a61af66fc99e Initial load
duke
parents:
diff changeset
301 // in the dom tree should be closest to the null check.
a61af66fc99e Initial load
duke
parents:
diff changeset
302 if( !best ||
a61af66fc99e Initial load
duke
parents:
diff changeset
303 cfg->_bbs[mach->_idx]->_dom_depth < cfg->_bbs[best->_idx]->_dom_depth ) {
a61af66fc99e Initial load
duke
parents:
diff changeset
304 best = mach;
a61af66fc99e Initial load
duke
parents:
diff changeset
305 bidx = vidx;
a61af66fc99e Initial load
duke
parents:
diff changeset
306
a61af66fc99e Initial load
duke
parents:
diff changeset
307 }
a61af66fc99e Initial load
duke
parents:
diff changeset
308 }
a61af66fc99e Initial load
duke
parents:
diff changeset
309 // No candidate!
a61af66fc99e Initial load
duke
parents:
diff changeset
310 if( !best ) return;
a61af66fc99e Initial load
duke
parents:
diff changeset
311
a61af66fc99e Initial load
duke
parents:
diff changeset
312 // ---- Found an implicit null check
a61af66fc99e Initial load
duke
parents:
diff changeset
313 extern int implicit_null_checks;
a61af66fc99e Initial load
duke
parents:
diff changeset
314 implicit_null_checks++;
a61af66fc99e Initial load
duke
parents:
diff changeset
315
1575
3657cb01ffc5 6954029: Improve implicit null check generation with compressed oops
kvn
parents: 1151
diff changeset
316 if( is_decoden ) {
3657cb01ffc5 6954029: Improve implicit null check generation with compressed oops
kvn
parents: 1151
diff changeset
317 // Check if we need to hoist decodeHeapOop_not_null first.
3657cb01ffc5 6954029: Improve implicit null check generation with compressed oops
kvn
parents: 1151
diff changeset
318 Block *valb = cfg->_bbs[val->_idx];
3657cb01ffc5 6954029: Improve implicit null check generation with compressed oops
kvn
parents: 1151
diff changeset
319 if( this != valb && this->_dom_depth < valb->_dom_depth ) {
3657cb01ffc5 6954029: Improve implicit null check generation with compressed oops
kvn
parents: 1151
diff changeset
320 // Hoist it up to the end of the test block.
3657cb01ffc5 6954029: Improve implicit null check generation with compressed oops
kvn
parents: 1151
diff changeset
321 valb->find_remove(val);
3657cb01ffc5 6954029: Improve implicit null check generation with compressed oops
kvn
parents: 1151
diff changeset
322 this->add_inst(val);
3657cb01ffc5 6954029: Improve implicit null check generation with compressed oops
kvn
parents: 1151
diff changeset
323 cfg->_bbs.map(val->_idx,this);
3657cb01ffc5 6954029: Improve implicit null check generation with compressed oops
kvn
parents: 1151
diff changeset
324 // DecodeN on x86 may kill flags. Check for flag-killing projections
3657cb01ffc5 6954029: Improve implicit null check generation with compressed oops
kvn
parents: 1151
diff changeset
325 // that also need to be hoisted.
3657cb01ffc5 6954029: Improve implicit null check generation with compressed oops
kvn
parents: 1151
diff changeset
326 for (DUIterator_Fast jmax, j = val->fast_outs(jmax); j < jmax; j++) {
3657cb01ffc5 6954029: Improve implicit null check generation with compressed oops
kvn
parents: 1151
diff changeset
327 Node* n = val->fast_out(j);
3842
c7b60b601eb4 7069452: Cleanup NodeFlags
kvn
parents: 3796
diff changeset
328 if( n->is_MachProj() ) {
1575
3657cb01ffc5 6954029: Improve implicit null check generation with compressed oops
kvn
parents: 1151
diff changeset
329 cfg->_bbs[n->_idx]->find_remove(n);
3657cb01ffc5 6954029: Improve implicit null check generation with compressed oops
kvn
parents: 1151
diff changeset
330 this->add_inst(n);
3657cb01ffc5 6954029: Improve implicit null check generation with compressed oops
kvn
parents: 1151
diff changeset
331 cfg->_bbs.map(n->_idx,this);
3657cb01ffc5 6954029: Improve implicit null check generation with compressed oops
kvn
parents: 1151
diff changeset
332 }
3657cb01ffc5 6954029: Improve implicit null check generation with compressed oops
kvn
parents: 1151
diff changeset
333 }
3657cb01ffc5 6954029: Improve implicit null check generation with compressed oops
kvn
parents: 1151
diff changeset
334 }
3657cb01ffc5 6954029: Improve implicit null check generation with compressed oops
kvn
parents: 1151
diff changeset
335 }
0
a61af66fc99e Initial load
duke
parents:
diff changeset
336 // Hoist the memory candidate up to the end of the test block.
a61af66fc99e Initial load
duke
parents:
diff changeset
337 Block *old_block = cfg->_bbs[best->_idx];
a61af66fc99e Initial load
duke
parents:
diff changeset
338 old_block->find_remove(best);
a61af66fc99e Initial load
duke
parents:
diff changeset
339 add_inst(best);
a61af66fc99e Initial load
duke
parents:
diff changeset
340 cfg->_bbs.map(best->_idx,this);
a61af66fc99e Initial load
duke
parents:
diff changeset
341
a61af66fc99e Initial load
duke
parents:
diff changeset
342 // Move the control dependence
a61af66fc99e Initial load
duke
parents:
diff changeset
343 if (best->in(0) && best->in(0) == old_block->_nodes[0])
a61af66fc99e Initial load
duke
parents:
diff changeset
344 best->set_req(0, _nodes[0]);
a61af66fc99e Initial load
duke
parents:
diff changeset
345
a61af66fc99e Initial load
duke
parents:
diff changeset
346 // Check for flag-killing projections that also need to be hoisted
a61af66fc99e Initial load
duke
parents:
diff changeset
347 // Should be DU safe because no edge updates.
a61af66fc99e Initial load
duke
parents:
diff changeset
348 for (DUIterator_Fast jmax, j = best->fast_outs(jmax); j < jmax; j++) {
a61af66fc99e Initial load
duke
parents:
diff changeset
349 Node* n = best->fast_out(j);
3842
c7b60b601eb4 7069452: Cleanup NodeFlags
kvn
parents: 3796
diff changeset
350 if( n->is_MachProj() ) {
0
a61af66fc99e Initial load
duke
parents:
diff changeset
351 cfg->_bbs[n->_idx]->find_remove(n);
a61af66fc99e Initial load
duke
parents:
diff changeset
352 add_inst(n);
a61af66fc99e Initial load
duke
parents:
diff changeset
353 cfg->_bbs.map(n->_idx,this);
a61af66fc99e Initial load
duke
parents:
diff changeset
354 }
a61af66fc99e Initial load
duke
parents:
diff changeset
355 }
a61af66fc99e Initial load
duke
parents:
diff changeset
356
a61af66fc99e Initial load
duke
parents:
diff changeset
357 Compile *C = cfg->C;
a61af66fc99e Initial load
duke
parents:
diff changeset
358 // proj==Op_True --> ne test; proj==Op_False --> eq test.
a61af66fc99e Initial load
duke
parents:
diff changeset
359 // One of two graph shapes got matched:
a61af66fc99e Initial load
duke
parents:
diff changeset
360 // (IfTrue (If (Bool NE (CmpP ptr NULL))))
a61af66fc99e Initial load
duke
parents:
diff changeset
361 // (IfFalse (If (Bool EQ (CmpP ptr NULL))))
a61af66fc99e Initial load
duke
parents:
diff changeset
362 // NULL checks are always branch-if-eq. If we see a IfTrue projection
a61af66fc99e Initial load
duke
parents:
diff changeset
363 // then we are replacing a 'ne' test with a 'eq' NULL check test.
a61af66fc99e Initial load
duke
parents:
diff changeset
364 // We need to flip the projections to keep the same semantics.
a61af66fc99e Initial load
duke
parents:
diff changeset
365 if( proj->Opcode() == Op_IfTrue ) {
a61af66fc99e Initial load
duke
parents:
diff changeset
366 // Swap order of projections in basic block to swap branch targets
a61af66fc99e Initial load
duke
parents:
diff changeset
367 Node *tmp1 = _nodes[end_idx()+1];
a61af66fc99e Initial load
duke
parents:
diff changeset
368 Node *tmp2 = _nodes[end_idx()+2];
a61af66fc99e Initial load
duke
parents:
diff changeset
369 _nodes.map(end_idx()+1, tmp2);
a61af66fc99e Initial load
duke
parents:
diff changeset
370 _nodes.map(end_idx()+2, tmp1);
a61af66fc99e Initial load
duke
parents:
diff changeset
371 Node *tmp = new (C, 1) Node(C->top()); // Use not NULL input
a61af66fc99e Initial load
duke
parents:
diff changeset
372 tmp1->replace_by(tmp);
a61af66fc99e Initial load
duke
parents:
diff changeset
373 tmp2->replace_by(tmp1);
a61af66fc99e Initial load
duke
parents:
diff changeset
374 tmp->replace_by(tmp2);
a61af66fc99e Initial load
duke
parents:
diff changeset
375 tmp->destruct();
a61af66fc99e Initial load
duke
parents:
diff changeset
376 }
a61af66fc99e Initial load
duke
parents:
diff changeset
377
a61af66fc99e Initial load
duke
parents:
diff changeset
378 // Remove the existing null check; use a new implicit null check instead.
a61af66fc99e Initial load
duke
parents:
diff changeset
379 // Since schedule-local needs precise def-use info, we need to correct
a61af66fc99e Initial load
duke
parents:
diff changeset
380 // it as well.
a61af66fc99e Initial load
duke
parents:
diff changeset
381 Node *old_tst = proj->in(0);
a61af66fc99e Initial load
duke
parents:
diff changeset
382 MachNode *nul_chk = new (C) MachNullCheckNode(old_tst->in(0),best,bidx);
a61af66fc99e Initial load
duke
parents:
diff changeset
383 _nodes.map(end_idx(),nul_chk);
a61af66fc99e Initial load
duke
parents:
diff changeset
384 cfg->_bbs.map(nul_chk->_idx,this);
a61af66fc99e Initial load
duke
parents:
diff changeset
385 // Redirect users of old_test to nul_chk
a61af66fc99e Initial load
duke
parents:
diff changeset
386 for (DUIterator_Last i2min, i2 = old_tst->last_outs(i2min); i2 >= i2min; --i2)
a61af66fc99e Initial load
duke
parents:
diff changeset
387 old_tst->last_out(i2)->set_req(0, nul_chk);
a61af66fc99e Initial load
duke
parents:
diff changeset
388 // Clean-up any dead code
a61af66fc99e Initial load
duke
parents:
diff changeset
389 for (uint i3 = 0; i3 < old_tst->req(); i3++)
a61af66fc99e Initial load
duke
parents:
diff changeset
390 old_tst->set_req(i3, NULL);
a61af66fc99e Initial load
duke
parents:
diff changeset
391
a61af66fc99e Initial load
duke
parents:
diff changeset
392 cfg->latency_from_uses(nul_chk);
a61af66fc99e Initial load
duke
parents:
diff changeset
393 cfg->latency_from_uses(best);
a61af66fc99e Initial load
duke
parents:
diff changeset
394 }
a61af66fc99e Initial load
duke
parents:
diff changeset
395
a61af66fc99e Initial load
duke
parents:
diff changeset
396
a61af66fc99e Initial load
duke
parents:
diff changeset
397 //------------------------------select-----------------------------------------
a61af66fc99e Initial load
duke
parents:
diff changeset
398 // Select a nice fellow from the worklist to schedule next. If there is only
a61af66fc99e Initial load
duke
parents:
diff changeset
399 // one choice, then use it. Projections take top priority for correctness
a61af66fc99e Initial load
duke
parents:
diff changeset
400 // reasons - if I see a projection, then it is next. There are a number of
a61af66fc99e Initial load
duke
parents:
diff changeset
401 // other special cases, for instructions that consume condition codes, et al.
a61af66fc99e Initial load
duke
parents:
diff changeset
402 // These are chosen immediately. Some instructions are required to immediately
a61af66fc99e Initial load
duke
parents:
diff changeset
403 // precede the last instruction in the block, and these are taken last. Of the
a61af66fc99e Initial load
duke
parents:
diff changeset
404 // remaining cases (most), choose the instruction with the greatest latency
a61af66fc99e Initial load
duke
parents:
diff changeset
405 // (that is, the most number of pseudo-cycles required to the end of the
a61af66fc99e Initial load
duke
parents:
diff changeset
406 // routine). If there is a tie, choose the instruction with the most inputs.
a61af66fc99e Initial load
duke
parents:
diff changeset
407 Node *Block::select(PhaseCFG *cfg, Node_List &worklist, int *ready_cnt, VectorSet &next_call, uint sched_slot) {
a61af66fc99e Initial load
duke
parents:
diff changeset
408
a61af66fc99e Initial load
duke
parents:
diff changeset
409 // If only a single entry on the stack, use it
a61af66fc99e Initial load
duke
parents:
diff changeset
410 uint cnt = worklist.size();
a61af66fc99e Initial load
duke
parents:
diff changeset
411 if (cnt == 1) {
a61af66fc99e Initial load
duke
parents:
diff changeset
412 Node *n = worklist[0];
a61af66fc99e Initial load
duke
parents:
diff changeset
413 worklist.map(0,worklist.pop());
a61af66fc99e Initial load
duke
parents:
diff changeset
414 return n;
a61af66fc99e Initial load
duke
parents:
diff changeset
415 }
a61af66fc99e Initial load
duke
parents:
diff changeset
416
a61af66fc99e Initial load
duke
parents:
diff changeset
417 uint choice = 0; // Bigger is most important
a61af66fc99e Initial load
duke
parents:
diff changeset
418 uint latency = 0; // Bigger is scheduled first
a61af66fc99e Initial load
duke
parents:
diff changeset
419 uint score = 0; // Bigger is better
253
b0fe4deeb9fb 6726999: nsk/stress/jck12a/jck12a010 assert(n != null,"Bad immediate dominator info.")
kvn
parents: 196
diff changeset
420 int idx = -1; // Index in worklist
0
a61af66fc99e Initial load
duke
parents:
diff changeset
421
a61af66fc99e Initial load
duke
parents:
diff changeset
422 for( uint i=0; i<cnt; i++ ) { // Inspect entire worklist
a61af66fc99e Initial load
duke
parents:
diff changeset
423 // Order in worklist is used to break ties.
a61af66fc99e Initial load
duke
parents:
diff changeset
424 // See caller for how this is used to delay scheduling
a61af66fc99e Initial load
duke
parents:
diff changeset
425 // of induction variable increments to after the other
a61af66fc99e Initial load
duke
parents:
diff changeset
426 // uses of the phi are scheduled.
a61af66fc99e Initial load
duke
parents:
diff changeset
427 Node *n = worklist[i]; // Get Node on worklist
a61af66fc99e Initial load
duke
parents:
diff changeset
428
a61af66fc99e Initial load
duke
parents:
diff changeset
429 int iop = n->is_Mach() ? n->as_Mach()->ideal_Opcode() : 0;
a61af66fc99e Initial load
duke
parents:
diff changeset
430 if( n->is_Proj() || // Projections always win
a61af66fc99e Initial load
duke
parents:
diff changeset
431 n->Opcode()== Op_Con || // So does constant 'Top'
a61af66fc99e Initial load
duke
parents:
diff changeset
432 iop == Op_CreateEx || // Create-exception must start block
a61af66fc99e Initial load
duke
parents:
diff changeset
433 iop == Op_CheckCastPP
a61af66fc99e Initial load
duke
parents:
diff changeset
434 ) {
a61af66fc99e Initial load
duke
parents:
diff changeset
435 worklist.map(i,worklist.pop());
a61af66fc99e Initial load
duke
parents:
diff changeset
436 return n;
a61af66fc99e Initial load
duke
parents:
diff changeset
437 }
a61af66fc99e Initial load
duke
parents:
diff changeset
438
a61af66fc99e Initial load
duke
parents:
diff changeset
439 // Final call in a block must be adjacent to 'catch'
a61af66fc99e Initial load
duke
parents:
diff changeset
440 Node *e = end();
a61af66fc99e Initial load
duke
parents:
diff changeset
441 if( e->is_Catch() && e->in(0)->in(0) == n )
a61af66fc99e Initial load
duke
parents:
diff changeset
442 continue;
a61af66fc99e Initial load
duke
parents:
diff changeset
443
a61af66fc99e Initial load
duke
parents:
diff changeset
444 // Memory op for an implicit null check has to be at the end of the block
a61af66fc99e Initial load
duke
parents:
diff changeset
445 if( e->is_MachNullCheck() && e->in(1) == n )
a61af66fc99e Initial load
duke
parents:
diff changeset
446 continue;
a61af66fc99e Initial load
duke
parents:
diff changeset
447
a61af66fc99e Initial load
duke
parents:
diff changeset
448 uint n_choice = 2;
a61af66fc99e Initial load
duke
parents:
diff changeset
449
a61af66fc99e Initial load
duke
parents:
diff changeset
450 // See if this instruction is consumed by a branch. If so, then (as the
a61af66fc99e Initial load
duke
parents:
diff changeset
451 // branch is the last instruction in the basic block) force it to the
a61af66fc99e Initial load
duke
parents:
diff changeset
452 // end of the basic block
a61af66fc99e Initial load
duke
parents:
diff changeset
453 if ( must_clone[iop] ) {
a61af66fc99e Initial load
duke
parents:
diff changeset
454 // See if any use is a branch
a61af66fc99e Initial load
duke
parents:
diff changeset
455 bool found_machif = false;
a61af66fc99e Initial load
duke
parents:
diff changeset
456
a61af66fc99e Initial load
duke
parents:
diff changeset
457 for (DUIterator_Fast jmax, j = n->fast_outs(jmax); j < jmax; j++) {
a61af66fc99e Initial load
duke
parents:
diff changeset
458 Node* use = n->fast_out(j);
a61af66fc99e Initial load
duke
parents:
diff changeset
459
a61af66fc99e Initial load
duke
parents:
diff changeset
460 // The use is a conditional branch, make them adjacent
a61af66fc99e Initial load
duke
parents:
diff changeset
461 if (use->is_MachIf() && cfg->_bbs[use->_idx]==this ) {
a61af66fc99e Initial load
duke
parents:
diff changeset
462 found_machif = true;
a61af66fc99e Initial load
duke
parents:
diff changeset
463 break;
a61af66fc99e Initial load
duke
parents:
diff changeset
464 }
a61af66fc99e Initial load
duke
parents:
diff changeset
465
a61af66fc99e Initial load
duke
parents:
diff changeset
466 // More than this instruction pending for successor to be ready,
a61af66fc99e Initial load
duke
parents:
diff changeset
467 // don't choose this if other opportunities are ready
a61af66fc99e Initial load
duke
parents:
diff changeset
468 if (ready_cnt[use->_idx] > 1)
a61af66fc99e Initial load
duke
parents:
diff changeset
469 n_choice = 1;
a61af66fc99e Initial load
duke
parents:
diff changeset
470 }
a61af66fc99e Initial load
duke
parents:
diff changeset
471
a61af66fc99e Initial load
duke
parents:
diff changeset
472 // loop terminated, prefer not to use this instruction
a61af66fc99e Initial load
duke
parents:
diff changeset
473 if (found_machif)
a61af66fc99e Initial load
duke
parents:
diff changeset
474 continue;
a61af66fc99e Initial load
duke
parents:
diff changeset
475 }
a61af66fc99e Initial load
duke
parents:
diff changeset
476
a61af66fc99e Initial load
duke
parents:
diff changeset
477 // See if this has a predecessor that is "must_clone", i.e. sets the
a61af66fc99e Initial load
duke
parents:
diff changeset
478 // condition code. If so, choose this first
a61af66fc99e Initial load
duke
parents:
diff changeset
479 for (uint j = 0; j < n->req() ; j++) {
a61af66fc99e Initial load
duke
parents:
diff changeset
480 Node *inn = n->in(j);
a61af66fc99e Initial load
duke
parents:
diff changeset
481 if (inn) {
a61af66fc99e Initial load
duke
parents:
diff changeset
482 if (inn->is_Mach() && must_clone[inn->as_Mach()->ideal_Opcode()] ) {
a61af66fc99e Initial load
duke
parents:
diff changeset
483 n_choice = 3;
a61af66fc99e Initial load
duke
parents:
diff changeset
484 break;
a61af66fc99e Initial load
duke
parents:
diff changeset
485 }
a61af66fc99e Initial load
duke
parents:
diff changeset
486 }
a61af66fc99e Initial load
duke
parents:
diff changeset
487 }
a61af66fc99e Initial load
duke
parents:
diff changeset
488
a61af66fc99e Initial load
duke
parents:
diff changeset
489 // MachTemps should be scheduled last so they are near their uses
a61af66fc99e Initial load
duke
parents:
diff changeset
490 if (n->is_MachTemp()) {
a61af66fc99e Initial load
duke
parents:
diff changeset
491 n_choice = 1;
a61af66fc99e Initial load
duke
parents:
diff changeset
492 }
a61af66fc99e Initial load
duke
parents:
diff changeset
493
1685
0e35fa8ebccd 6973963: SEGV in ciBlock::start_bci() with EA
kvn
parents: 1579
diff changeset
494 uint n_latency = cfg->_node_latency->at_grow(n->_idx);
0
a61af66fc99e Initial load
duke
parents:
diff changeset
495 uint n_score = n->req(); // Many inputs get high score to break ties
a61af66fc99e Initial load
duke
parents:
diff changeset
496
a61af66fc99e Initial load
duke
parents:
diff changeset
497 // Keep best latency found
a61af66fc99e Initial load
duke
parents:
diff changeset
498 if( choice < n_choice ||
a61af66fc99e Initial load
duke
parents:
diff changeset
499 ( choice == n_choice &&
a61af66fc99e Initial load
duke
parents:
diff changeset
500 ( latency < n_latency ||
a61af66fc99e Initial load
duke
parents:
diff changeset
501 ( latency == n_latency &&
a61af66fc99e Initial load
duke
parents:
diff changeset
502 ( score < n_score ))))) {
a61af66fc99e Initial load
duke
parents:
diff changeset
503 choice = n_choice;
a61af66fc99e Initial load
duke
parents:
diff changeset
504 latency = n_latency;
a61af66fc99e Initial load
duke
parents:
diff changeset
505 score = n_score;
a61af66fc99e Initial load
duke
parents:
diff changeset
506 idx = i; // Also keep index in worklist
a61af66fc99e Initial load
duke
parents:
diff changeset
507 }
a61af66fc99e Initial load
duke
parents:
diff changeset
508 } // End of for all ready nodes in worklist
a61af66fc99e Initial load
duke
parents:
diff changeset
509
253
b0fe4deeb9fb 6726999: nsk/stress/jck12a/jck12a010 assert(n != null,"Bad immediate dominator info.")
kvn
parents: 196
diff changeset
510 assert(idx >= 0, "index should be set");
b0fe4deeb9fb 6726999: nsk/stress/jck12a/jck12a010 assert(n != null,"Bad immediate dominator info.")
kvn
parents: 196
diff changeset
511 Node *n = worklist[(uint)idx]; // Get the winner
0
a61af66fc99e Initial load
duke
parents:
diff changeset
512
253
b0fe4deeb9fb 6726999: nsk/stress/jck12a/jck12a010 assert(n != null,"Bad immediate dominator info.")
kvn
parents: 196
diff changeset
513 worklist.map((uint)idx, worklist.pop()); // Compress worklist
0
a61af66fc99e Initial load
duke
parents:
diff changeset
514 return n;
a61af66fc99e Initial load
duke
parents:
diff changeset
515 }
a61af66fc99e Initial load
duke
parents:
diff changeset
516
a61af66fc99e Initial load
duke
parents:
diff changeset
517
a61af66fc99e Initial load
duke
parents:
diff changeset
518 //------------------------------set_next_call----------------------------------
a61af66fc99e Initial load
duke
parents:
diff changeset
519 void Block::set_next_call( Node *n, VectorSet &next_call, Block_Array &bbs ) {
a61af66fc99e Initial load
duke
parents:
diff changeset
520 if( next_call.test_set(n->_idx) ) return;
a61af66fc99e Initial load
duke
parents:
diff changeset
521 for( uint i=0; i<n->len(); i++ ) {
a61af66fc99e Initial load
duke
parents:
diff changeset
522 Node *m = n->in(i);
a61af66fc99e Initial load
duke
parents:
diff changeset
523 if( !m ) continue; // must see all nodes in block that precede call
a61af66fc99e Initial load
duke
parents:
diff changeset
524 if( bbs[m->_idx] == this )
a61af66fc99e Initial load
duke
parents:
diff changeset
525 set_next_call( m, next_call, bbs );
a61af66fc99e Initial load
duke
parents:
diff changeset
526 }
a61af66fc99e Initial load
duke
parents:
diff changeset
527 }
a61af66fc99e Initial load
duke
parents:
diff changeset
528
a61af66fc99e Initial load
duke
parents:
diff changeset
529 //------------------------------needed_for_next_call---------------------------
a61af66fc99e Initial load
duke
parents:
diff changeset
530 // Set the flag 'next_call' for each Node that is needed for the next call to
a61af66fc99e Initial load
duke
parents:
diff changeset
531 // be scheduled. This flag lets me bias scheduling so Nodes needed for the
a61af66fc99e Initial load
duke
parents:
diff changeset
532 // next subroutine call get priority - basically it moves things NOT needed
a61af66fc99e Initial load
duke
parents:
diff changeset
533 // for the next call till after the call. This prevents me from trying to
a61af66fc99e Initial load
duke
parents:
diff changeset
534 // carry lots of stuff live across a call.
a61af66fc99e Initial load
duke
parents:
diff changeset
535 void Block::needed_for_next_call(Node *this_call, VectorSet &next_call, Block_Array &bbs) {
a61af66fc99e Initial load
duke
parents:
diff changeset
536 // Find the next control-defining Node in this block
a61af66fc99e Initial load
duke
parents:
diff changeset
537 Node* call = NULL;
a61af66fc99e Initial load
duke
parents:
diff changeset
538 for (DUIterator_Fast imax, i = this_call->fast_outs(imax); i < imax; i++) {
a61af66fc99e Initial load
duke
parents:
diff changeset
539 Node* m = this_call->fast_out(i);
a61af66fc99e Initial load
duke
parents:
diff changeset
540 if( bbs[m->_idx] == this && // Local-block user
a61af66fc99e Initial load
duke
parents:
diff changeset
541 m != this_call && // Not self-start node
3842
c7b60b601eb4 7069452: Cleanup NodeFlags
kvn
parents: 3796
diff changeset
542 m->is_MachCall() )
0
a61af66fc99e Initial load
duke
parents:
diff changeset
543 call = m;
a61af66fc99e Initial load
duke
parents:
diff changeset
544 break;
a61af66fc99e Initial load
duke
parents:
diff changeset
545 }
a61af66fc99e Initial load
duke
parents:
diff changeset
546 if (call == NULL) return; // No next call (e.g., block end is near)
a61af66fc99e Initial load
duke
parents:
diff changeset
547 // Set next-call for all inputs to this call
a61af66fc99e Initial load
duke
parents:
diff changeset
548 set_next_call(call, next_call, bbs);
a61af66fc99e Initial load
duke
parents:
diff changeset
549 }
a61af66fc99e Initial load
duke
parents:
diff changeset
550
4120
f03a3c8bd5e5 7077312: Provide a CALL effect for instruct declaration in the ad file
roland
parents: 3842
diff changeset
551 //------------------------------add_call_kills-------------------------------------
f03a3c8bd5e5 7077312: Provide a CALL effect for instruct declaration in the ad file
roland
parents: 3842
diff changeset
552 void Block::add_call_kills(MachProjNode *proj, RegMask& regs, const char* save_policy, bool exclude_soe) {
f03a3c8bd5e5 7077312: Provide a CALL effect for instruct declaration in the ad file
roland
parents: 3842
diff changeset
553 // Fill in the kill mask for the call
f03a3c8bd5e5 7077312: Provide a CALL effect for instruct declaration in the ad file
roland
parents: 3842
diff changeset
554 for( OptoReg::Name r = OptoReg::Name(0); r < _last_Mach_Reg; r=OptoReg::add(r,1) ) {
f03a3c8bd5e5 7077312: Provide a CALL effect for instruct declaration in the ad file
roland
parents: 3842
diff changeset
555 if( !regs.Member(r) ) { // Not already defined by the call
f03a3c8bd5e5 7077312: Provide a CALL effect for instruct declaration in the ad file
roland
parents: 3842
diff changeset
556 // Save-on-call register?
f03a3c8bd5e5 7077312: Provide a CALL effect for instruct declaration in the ad file
roland
parents: 3842
diff changeset
557 if ((save_policy[r] == 'C') ||
f03a3c8bd5e5 7077312: Provide a CALL effect for instruct declaration in the ad file
roland
parents: 3842
diff changeset
558 (save_policy[r] == 'A') ||
f03a3c8bd5e5 7077312: Provide a CALL effect for instruct declaration in the ad file
roland
parents: 3842
diff changeset
559 ((save_policy[r] == 'E') && exclude_soe)) {
f03a3c8bd5e5 7077312: Provide a CALL effect for instruct declaration in the ad file
roland
parents: 3842
diff changeset
560 proj->_rout.Insert(r);
f03a3c8bd5e5 7077312: Provide a CALL effect for instruct declaration in the ad file
roland
parents: 3842
diff changeset
561 }
f03a3c8bd5e5 7077312: Provide a CALL effect for instruct declaration in the ad file
roland
parents: 3842
diff changeset
562 }
f03a3c8bd5e5 7077312: Provide a CALL effect for instruct declaration in the ad file
roland
parents: 3842
diff changeset
563 }
f03a3c8bd5e5 7077312: Provide a CALL effect for instruct declaration in the ad file
roland
parents: 3842
diff changeset
564 }
f03a3c8bd5e5 7077312: Provide a CALL effect for instruct declaration in the ad file
roland
parents: 3842
diff changeset
565
f03a3c8bd5e5 7077312: Provide a CALL effect for instruct declaration in the ad file
roland
parents: 3842
diff changeset
566
0
a61af66fc99e Initial load
duke
parents:
diff changeset
567 //------------------------------sched_call-------------------------------------
a61af66fc99e Initial load
duke
parents:
diff changeset
568 uint Block::sched_call( Matcher &matcher, Block_Array &bbs, uint node_cnt, Node_List &worklist, int *ready_cnt, MachCallNode *mcall, VectorSet &next_call ) {
a61af66fc99e Initial load
duke
parents:
diff changeset
569 RegMask regs;
a61af66fc99e Initial load
duke
parents:
diff changeset
570
a61af66fc99e Initial load
duke
parents:
diff changeset
571 // Schedule all the users of the call right now. All the users are
a61af66fc99e Initial load
duke
parents:
diff changeset
572 // projection Nodes, so they must be scheduled next to the call.
a61af66fc99e Initial load
duke
parents:
diff changeset
573 // Collect all the defined registers.
a61af66fc99e Initial load
duke
parents:
diff changeset
574 for (DUIterator_Fast imax, i = mcall->fast_outs(imax); i < imax; i++) {
a61af66fc99e Initial load
duke
parents:
diff changeset
575 Node* n = mcall->fast_out(i);
3842
c7b60b601eb4 7069452: Cleanup NodeFlags
kvn
parents: 3796
diff changeset
576 assert( n->is_MachProj(), "" );
0
a61af66fc99e Initial load
duke
parents:
diff changeset
577 --ready_cnt[n->_idx];
a61af66fc99e Initial load
duke
parents:
diff changeset
578 assert( !ready_cnt[n->_idx], "" );
a61af66fc99e Initial load
duke
parents:
diff changeset
579 // Schedule next to call
a61af66fc99e Initial load
duke
parents:
diff changeset
580 _nodes.map(node_cnt++, n);
a61af66fc99e Initial load
duke
parents:
diff changeset
581 // Collect defined registers
a61af66fc99e Initial load
duke
parents:
diff changeset
582 regs.OR(n->out_RegMask());
a61af66fc99e Initial load
duke
parents:
diff changeset
583 // Check for scheduling the next control-definer
a61af66fc99e Initial load
duke
parents:
diff changeset
584 if( n->bottom_type() == Type::CONTROL )
a61af66fc99e Initial load
duke
parents:
diff changeset
585 // Warm up next pile of heuristic bits
a61af66fc99e Initial load
duke
parents:
diff changeset
586 needed_for_next_call(n, next_call, bbs);
a61af66fc99e Initial load
duke
parents:
diff changeset
587
a61af66fc99e Initial load
duke
parents:
diff changeset
588 // Children of projections are now all ready
a61af66fc99e Initial load
duke
parents:
diff changeset
589 for (DUIterator_Fast jmax, j = n->fast_outs(jmax); j < jmax; j++) {
a61af66fc99e Initial load
duke
parents:
diff changeset
590 Node* m = n->fast_out(j); // Get user
a61af66fc99e Initial load
duke
parents:
diff changeset
591 if( bbs[m->_idx] != this ) continue;
a61af66fc99e Initial load
duke
parents:
diff changeset
592 if( m->is_Phi() ) continue;
a61af66fc99e Initial load
duke
parents:
diff changeset
593 if( !--ready_cnt[m->_idx] )
a61af66fc99e Initial load
duke
parents:
diff changeset
594 worklist.push(m);
a61af66fc99e Initial load
duke
parents:
diff changeset
595 }
a61af66fc99e Initial load
duke
parents:
diff changeset
596
a61af66fc99e Initial load
duke
parents:
diff changeset
597 }
a61af66fc99e Initial load
duke
parents:
diff changeset
598
a61af66fc99e Initial load
duke
parents:
diff changeset
599 // Act as if the call defines the Frame Pointer.
a61af66fc99e Initial load
duke
parents:
diff changeset
600 // Certainly the FP is alive and well after the call.
a61af66fc99e Initial load
duke
parents:
diff changeset
601 regs.Insert(matcher.c_frame_pointer());
a61af66fc99e Initial load
duke
parents:
diff changeset
602
a61af66fc99e Initial load
duke
parents:
diff changeset
603 // Set all registers killed and not already defined by the call.
a61af66fc99e Initial load
duke
parents:
diff changeset
604 uint r_cnt = mcall->tf()->range()->cnt();
a61af66fc99e Initial load
duke
parents:
diff changeset
605 int op = mcall->ideal_Opcode();
a61af66fc99e Initial load
duke
parents:
diff changeset
606 MachProjNode *proj = new (matcher.C, 1) MachProjNode( mcall, r_cnt+1, RegMask::Empty, MachProjNode::fat_proj );
a61af66fc99e Initial load
duke
parents:
diff changeset
607 bbs.map(proj->_idx,this);
a61af66fc99e Initial load
duke
parents:
diff changeset
608 _nodes.insert(node_cnt++, proj);
a61af66fc99e Initial load
duke
parents:
diff changeset
609
a61af66fc99e Initial load
duke
parents:
diff changeset
610 // Select the right register save policy.
a61af66fc99e Initial load
duke
parents:
diff changeset
611 const char * save_policy;
a61af66fc99e Initial load
duke
parents:
diff changeset
612 switch (op) {
a61af66fc99e Initial load
duke
parents:
diff changeset
613 case Op_CallRuntime:
a61af66fc99e Initial load
duke
parents:
diff changeset
614 case Op_CallLeaf:
a61af66fc99e Initial load
duke
parents:
diff changeset
615 case Op_CallLeafNoFP:
a61af66fc99e Initial load
duke
parents:
diff changeset
616 // Calling C code so use C calling convention
a61af66fc99e Initial load
duke
parents:
diff changeset
617 save_policy = matcher._c_reg_save_policy;
a61af66fc99e Initial load
duke
parents:
diff changeset
618 break;
a61af66fc99e Initial load
duke
parents:
diff changeset
619
a61af66fc99e Initial load
duke
parents:
diff changeset
620 case Op_CallStaticJava:
a61af66fc99e Initial load
duke
parents:
diff changeset
621 case Op_CallDynamicJava:
a61af66fc99e Initial load
duke
parents:
diff changeset
622 // Calling Java code so use Java calling convention
a61af66fc99e Initial load
duke
parents:
diff changeset
623 save_policy = matcher._register_save_policy;
a61af66fc99e Initial load
duke
parents:
diff changeset
624 break;
a61af66fc99e Initial load
duke
parents:
diff changeset
625
a61af66fc99e Initial load
duke
parents:
diff changeset
626 default:
a61af66fc99e Initial load
duke
parents:
diff changeset
627 ShouldNotReachHere();
a61af66fc99e Initial load
duke
parents:
diff changeset
628 }
a61af66fc99e Initial load
duke
parents:
diff changeset
629
a61af66fc99e Initial load
duke
parents:
diff changeset
630 // When using CallRuntime mark SOE registers as killed by the call
a61af66fc99e Initial load
duke
parents:
diff changeset
631 // so values that could show up in the RegisterMap aren't live in a
a61af66fc99e Initial load
duke
parents:
diff changeset
632 // callee saved register since the register wouldn't know where to
a61af66fc99e Initial load
duke
parents:
diff changeset
633 // find them. CallLeaf and CallLeafNoFP are ok because they can't
a61af66fc99e Initial load
duke
parents:
diff changeset
634 // have debug info on them. Strictly speaking this only needs to be
a61af66fc99e Initial load
duke
parents:
diff changeset
635 // done for oops since idealreg2debugmask takes care of debug info
a61af66fc99e Initial load
duke
parents:
diff changeset
636 // references but there no way to handle oops differently than other
a61af66fc99e Initial load
duke
parents:
diff changeset
637 // pointers as far as the kill mask goes.
a61af66fc99e Initial load
duke
parents:
diff changeset
638 bool exclude_soe = op == Op_CallRuntime;
a61af66fc99e Initial load
duke
parents:
diff changeset
639
1137
97125851f396 6829187: compiler optimizations required for JSR 292
twisti
parents: 1100
diff changeset
640 // If the call is a MethodHandle invoke, we need to exclude the
97125851f396 6829187: compiler optimizations required for JSR 292
twisti
parents: 1100
diff changeset
641 // register which is used to save the SP value over MH invokes from
97125851f396 6829187: compiler optimizations required for JSR 292
twisti
parents: 1100
diff changeset
642 // the mask. Otherwise this register could be used for
97125851f396 6829187: compiler optimizations required for JSR 292
twisti
parents: 1100
diff changeset
643 // deoptimization information.
97125851f396 6829187: compiler optimizations required for JSR 292
twisti
parents: 1100
diff changeset
644 if (op == Op_CallStaticJava) {
97125851f396 6829187: compiler optimizations required for JSR 292
twisti
parents: 1100
diff changeset
645 MachCallStaticJavaNode* mcallstaticjava = (MachCallStaticJavaNode*) mcall;
97125851f396 6829187: compiler optimizations required for JSR 292
twisti
parents: 1100
diff changeset
646 if (mcallstaticjava->_method_handle_invoke)
97125851f396 6829187: compiler optimizations required for JSR 292
twisti
parents: 1100
diff changeset
647 proj->_rout.OR(Matcher::method_handle_invoke_SP_save_mask());
97125851f396 6829187: compiler optimizations required for JSR 292
twisti
parents: 1100
diff changeset
648 }
97125851f396 6829187: compiler optimizations required for JSR 292
twisti
parents: 1100
diff changeset
649
4120
f03a3c8bd5e5 7077312: Provide a CALL effect for instruct declaration in the ad file
roland
parents: 3842
diff changeset
650 add_call_kills(proj, regs, save_policy, exclude_soe);
0
a61af66fc99e Initial load
duke
parents:
diff changeset
651
a61af66fc99e Initial load
duke
parents:
diff changeset
652 return node_cnt;
a61af66fc99e Initial load
duke
parents:
diff changeset
653 }
a61af66fc99e Initial load
duke
parents:
diff changeset
654
a61af66fc99e Initial load
duke
parents:
diff changeset
655
a61af66fc99e Initial load
duke
parents:
diff changeset
656 //------------------------------schedule_local---------------------------------
a61af66fc99e Initial load
duke
parents:
diff changeset
657 // Topological sort within a block. Someday become a real scheduler.
a61af66fc99e Initial load
duke
parents:
diff changeset
658 bool Block::schedule_local(PhaseCFG *cfg, Matcher &matcher, int *ready_cnt, VectorSet &next_call) {
a61af66fc99e Initial load
duke
parents:
diff changeset
659 // Already "sorted" are the block start Node (as the first entry), and
a61af66fc99e Initial load
duke
parents:
diff changeset
660 // the block-ending Node and any trailing control projections. We leave
a61af66fc99e Initial load
duke
parents:
diff changeset
661 // these alone. PhiNodes and ParmNodes are made to follow the block start
a61af66fc99e Initial load
duke
parents:
diff changeset
662 // Node. Everything else gets topo-sorted.
a61af66fc99e Initial load
duke
parents:
diff changeset
663
a61af66fc99e Initial load
duke
parents:
diff changeset
664 #ifndef PRODUCT
a61af66fc99e Initial load
duke
parents:
diff changeset
665 if (cfg->trace_opto_pipelining()) {
a61af66fc99e Initial load
duke
parents:
diff changeset
666 tty->print_cr("# --- schedule_local B%d, before: ---", _pre_order);
a61af66fc99e Initial load
duke
parents:
diff changeset
667 for (uint i = 0;i < _nodes.size();i++) {
a61af66fc99e Initial load
duke
parents:
diff changeset
668 tty->print("# ");
a61af66fc99e Initial load
duke
parents:
diff changeset
669 _nodes[i]->fast_dump();
a61af66fc99e Initial load
duke
parents:
diff changeset
670 }
a61af66fc99e Initial load
duke
parents:
diff changeset
671 tty->print_cr("#");
a61af66fc99e Initial load
duke
parents:
diff changeset
672 }
a61af66fc99e Initial load
duke
parents:
diff changeset
673 #endif
a61af66fc99e Initial load
duke
parents:
diff changeset
674
a61af66fc99e Initial load
duke
parents:
diff changeset
675 // RootNode is already sorted
a61af66fc99e Initial load
duke
parents:
diff changeset
676 if( _nodes.size() == 1 ) return true;
a61af66fc99e Initial load
duke
parents:
diff changeset
677
a61af66fc99e Initial load
duke
parents:
diff changeset
678 // Move PhiNodes and ParmNodes from 1 to cnt up to the start
a61af66fc99e Initial load
duke
parents:
diff changeset
679 uint node_cnt = end_idx();
a61af66fc99e Initial load
duke
parents:
diff changeset
680 uint phi_cnt = 1;
a61af66fc99e Initial load
duke
parents:
diff changeset
681 uint i;
a61af66fc99e Initial load
duke
parents:
diff changeset
682 for( i = 1; i<node_cnt; i++ ) { // Scan for Phi
a61af66fc99e Initial load
duke
parents:
diff changeset
683 Node *n = _nodes[i];
a61af66fc99e Initial load
duke
parents:
diff changeset
684 if( n->is_Phi() || // Found a PhiNode or ParmNode
a61af66fc99e Initial load
duke
parents:
diff changeset
685 (n->is_Proj() && n->in(0) == head()) ) {
a61af66fc99e Initial load
duke
parents:
diff changeset
686 // Move guy at 'phi_cnt' to the end; makes a hole at phi_cnt
a61af66fc99e Initial load
duke
parents:
diff changeset
687 _nodes.map(i,_nodes[phi_cnt]);
a61af66fc99e Initial load
duke
parents:
diff changeset
688 _nodes.map(phi_cnt++,n); // swap Phi/Parm up front
a61af66fc99e Initial load
duke
parents:
diff changeset
689 } else { // All others
a61af66fc99e Initial load
duke
parents:
diff changeset
690 // Count block-local inputs to 'n'
a61af66fc99e Initial load
duke
parents:
diff changeset
691 uint cnt = n->len(); // Input count
a61af66fc99e Initial load
duke
parents:
diff changeset
692 uint local = 0;
a61af66fc99e Initial load
duke
parents:
diff changeset
693 for( uint j=0; j<cnt; j++ ) {
a61af66fc99e Initial load
duke
parents:
diff changeset
694 Node *m = n->in(j);
a61af66fc99e Initial load
duke
parents:
diff changeset
695 if( m && cfg->_bbs[m->_idx] == this && !m->is_top() )
a61af66fc99e Initial load
duke
parents:
diff changeset
696 local++; // One more block-local input
a61af66fc99e Initial load
duke
parents:
diff changeset
697 }
a61af66fc99e Initial load
duke
parents:
diff changeset
698 ready_cnt[n->_idx] = local; // Count em up
a61af66fc99e Initial load
duke
parents:
diff changeset
699
3248
e6beb62de02d 7032963: StoreCM shouldn't participate in store elimination
never
parents: 1972
diff changeset
700 #ifdef ASSERT
342
37f87013dfd8 6711316: Open source the Garbage-First garbage collector
ysr
parents: 125
diff changeset
701 if( UseConcMarkSweepGC || UseG1GC ) {
0
a61af66fc99e Initial load
duke
parents:
diff changeset
702 if( n->is_Mach() && n->as_Mach()->ideal_Opcode() == Op_StoreCM ) {
3248
e6beb62de02d 7032963: StoreCM shouldn't participate in store elimination
never
parents: 1972
diff changeset
703 // Check the precedence edges
e6beb62de02d 7032963: StoreCM shouldn't participate in store elimination
never
parents: 1972
diff changeset
704 for (uint prec = n->req(); prec < n->len(); prec++) {
e6beb62de02d 7032963: StoreCM shouldn't participate in store elimination
never
parents: 1972
diff changeset
705 Node* oop_store = n->in(prec);
e6beb62de02d 7032963: StoreCM shouldn't participate in store elimination
never
parents: 1972
diff changeset
706 if (oop_store != NULL) {
e6beb62de02d 7032963: StoreCM shouldn't participate in store elimination
never
parents: 1972
diff changeset
707 assert(cfg->_bbs[oop_store->_idx]->_dom_depth <= this->_dom_depth, "oop_store must dominate card-mark");
e6beb62de02d 7032963: StoreCM shouldn't participate in store elimination
never
parents: 1972
diff changeset
708 }
e6beb62de02d 7032963: StoreCM shouldn't participate in store elimination
never
parents: 1972
diff changeset
709 }
0
a61af66fc99e Initial load
duke
parents:
diff changeset
710 }
a61af66fc99e Initial load
duke
parents:
diff changeset
711 }
3248
e6beb62de02d 7032963: StoreCM shouldn't participate in store elimination
never
parents: 1972
diff changeset
712 #endif
e6beb62de02d 7032963: StoreCM shouldn't participate in store elimination
never
parents: 1972
diff changeset
713
e6beb62de02d 7032963: StoreCM shouldn't participate in store elimination
never
parents: 1972
diff changeset
714 // A few node types require changing a required edge to a precedence edge
e6beb62de02d 7032963: StoreCM shouldn't participate in store elimination
never
parents: 1972
diff changeset
715 // before allocation.
1100
f96a1a986f7b 6895383: JCK test throws NPE for method compiled with Escape Analysis
kvn
parents: 681
diff changeset
716 if( n->is_Mach() && n->req() > TypeFunc::Parms &&
f96a1a986f7b 6895383: JCK test throws NPE for method compiled with Escape Analysis
kvn
parents: 681
diff changeset
717 (n->as_Mach()->ideal_Opcode() == Op_MemBarAcquire ||
f96a1a986f7b 6895383: JCK test throws NPE for method compiled with Escape Analysis
kvn
parents: 681
diff changeset
718 n->as_Mach()->ideal_Opcode() == Op_MemBarVolatile) ) {
253
b0fe4deeb9fb 6726999: nsk/stress/jck12a/jck12a010 assert(n != null,"Bad immediate dominator info.")
kvn
parents: 196
diff changeset
719 // MemBarAcquire could be created without Precedent edge.
b0fe4deeb9fb 6726999: nsk/stress/jck12a/jck12a010 assert(n != null,"Bad immediate dominator info.")
kvn
parents: 196
diff changeset
720 // del_req() replaces the specified edge with the last input edge
b0fe4deeb9fb 6726999: nsk/stress/jck12a/jck12a010 assert(n != null,"Bad immediate dominator info.")
kvn
parents: 196
diff changeset
721 // and then removes the last edge. If the specified edge > number of
b0fe4deeb9fb 6726999: nsk/stress/jck12a/jck12a010 assert(n != null,"Bad immediate dominator info.")
kvn
parents: 196
diff changeset
722 // edges the last edge will be moved outside of the input edges array
b0fe4deeb9fb 6726999: nsk/stress/jck12a/jck12a010 assert(n != null,"Bad immediate dominator info.")
kvn
parents: 196
diff changeset
723 // and the edge will be lost. This is why this code should be
b0fe4deeb9fb 6726999: nsk/stress/jck12a/jck12a010 assert(n != null,"Bad immediate dominator info.")
kvn
parents: 196
diff changeset
724 // executed only when Precedent (== TypeFunc::Parms) edge is present.
0
a61af66fc99e Initial load
duke
parents:
diff changeset
725 Node *x = n->in(TypeFunc::Parms);
a61af66fc99e Initial load
duke
parents:
diff changeset
726 n->del_req(TypeFunc::Parms);
a61af66fc99e Initial load
duke
parents:
diff changeset
727 n->add_prec(x);
a61af66fc99e Initial load
duke
parents:
diff changeset
728 }
a61af66fc99e Initial load
duke
parents:
diff changeset
729 }
a61af66fc99e Initial load
duke
parents:
diff changeset
730 }
a61af66fc99e Initial load
duke
parents:
diff changeset
731 for(uint i2=i; i2<_nodes.size(); i2++ ) // Trailing guys get zapped count
a61af66fc99e Initial load
duke
parents:
diff changeset
732 ready_cnt[_nodes[i2]->_idx] = 0;
a61af66fc99e Initial load
duke
parents:
diff changeset
733
a61af66fc99e Initial load
duke
parents:
diff changeset
734 // All the prescheduled guys do not hold back internal nodes
a61af66fc99e Initial load
duke
parents:
diff changeset
735 uint i3;
a61af66fc99e Initial load
duke
parents:
diff changeset
736 for(i3 = 0; i3<phi_cnt; i3++ ) { // For all pre-scheduled
a61af66fc99e Initial load
duke
parents:
diff changeset
737 Node *n = _nodes[i3]; // Get pre-scheduled
a61af66fc99e Initial load
duke
parents:
diff changeset
738 for (DUIterator_Fast jmax, j = n->fast_outs(jmax); j < jmax; j++) {
a61af66fc99e Initial load
duke
parents:
diff changeset
739 Node* m = n->fast_out(j);
a61af66fc99e Initial load
duke
parents:
diff changeset
740 if( cfg->_bbs[m->_idx] ==this ) // Local-block user
a61af66fc99e Initial load
duke
parents:
diff changeset
741 ready_cnt[m->_idx]--; // Fix ready count
a61af66fc99e Initial load
duke
parents:
diff changeset
742 }
a61af66fc99e Initial load
duke
parents:
diff changeset
743 }
a61af66fc99e Initial load
duke
parents:
diff changeset
744
a61af66fc99e Initial load
duke
parents:
diff changeset
745 Node_List delay;
a61af66fc99e Initial load
duke
parents:
diff changeset
746 // Make a worklist
a61af66fc99e Initial load
duke
parents:
diff changeset
747 Node_List worklist;
a61af66fc99e Initial load
duke
parents:
diff changeset
748 for(uint i4=i3; i4<node_cnt; i4++ ) { // Put ready guys on worklist
a61af66fc99e Initial load
duke
parents:
diff changeset
749 Node *m = _nodes[i4];
a61af66fc99e Initial load
duke
parents:
diff changeset
750 if( !ready_cnt[m->_idx] ) { // Zero ready count?
a61af66fc99e Initial load
duke
parents:
diff changeset
751 if (m->is_iteratively_computed()) {
a61af66fc99e Initial load
duke
parents:
diff changeset
752 // Push induction variable increments last to allow other uses
a61af66fc99e Initial load
duke
parents:
diff changeset
753 // of the phi to be scheduled first. The select() method breaks
a61af66fc99e Initial load
duke
parents:
diff changeset
754 // ties in scheduling by worklist order.
a61af66fc99e Initial load
duke
parents:
diff changeset
755 delay.push(m);
125
d942c7e64bd9 6601321: Assert(j == 1 || b->_nodes[j-1]->is_Phi(),"CreateEx must be first instruction in block")
never
parents: 113
diff changeset
756 } else if (m->is_Mach() && m->as_Mach()->ideal_Opcode() == Op_CreateEx) {
d942c7e64bd9 6601321: Assert(j == 1 || b->_nodes[j-1]->is_Phi(),"CreateEx must be first instruction in block")
never
parents: 113
diff changeset
757 // Force the CreateEx to the top of the list so it's processed
d942c7e64bd9 6601321: Assert(j == 1 || b->_nodes[j-1]->is_Phi(),"CreateEx must be first instruction in block")
never
parents: 113
diff changeset
758 // first and ends up at the start of the block.
d942c7e64bd9 6601321: Assert(j == 1 || b->_nodes[j-1]->is_Phi(),"CreateEx must be first instruction in block")
never
parents: 113
diff changeset
759 worklist.insert(0, m);
0
a61af66fc99e Initial load
duke
parents:
diff changeset
760 } else {
a61af66fc99e Initial load
duke
parents:
diff changeset
761 worklist.push(m); // Then on to worklist!
a61af66fc99e Initial load
duke
parents:
diff changeset
762 }
a61af66fc99e Initial load
duke
parents:
diff changeset
763 }
a61af66fc99e Initial load
duke
parents:
diff changeset
764 }
a61af66fc99e Initial load
duke
parents:
diff changeset
765 while (delay.size()) {
a61af66fc99e Initial load
duke
parents:
diff changeset
766 Node* d = delay.pop();
a61af66fc99e Initial load
duke
parents:
diff changeset
767 worklist.push(d);
a61af66fc99e Initial load
duke
parents:
diff changeset
768 }
a61af66fc99e Initial load
duke
parents:
diff changeset
769
a61af66fc99e Initial load
duke
parents:
diff changeset
770 // Warm up the 'next_call' heuristic bits
a61af66fc99e Initial load
duke
parents:
diff changeset
771 needed_for_next_call(_nodes[0], next_call, cfg->_bbs);
a61af66fc99e Initial load
duke
parents:
diff changeset
772
a61af66fc99e Initial load
duke
parents:
diff changeset
773 #ifndef PRODUCT
a61af66fc99e Initial load
duke
parents:
diff changeset
774 if (cfg->trace_opto_pipelining()) {
a61af66fc99e Initial load
duke
parents:
diff changeset
775 for (uint j=0; j<_nodes.size(); j++) {
a61af66fc99e Initial load
duke
parents:
diff changeset
776 Node *n = _nodes[j];
a61af66fc99e Initial load
duke
parents:
diff changeset
777 int idx = n->_idx;
a61af66fc99e Initial load
duke
parents:
diff changeset
778 tty->print("# ready cnt:%3d ", ready_cnt[idx]);
1685
0e35fa8ebccd 6973963: SEGV in ciBlock::start_bci() with EA
kvn
parents: 1579
diff changeset
779 tty->print("latency:%3d ", cfg->_node_latency->at_grow(idx));
0
a61af66fc99e Initial load
duke
parents:
diff changeset
780 tty->print("%4d: %s\n", idx, n->Name());
a61af66fc99e Initial load
duke
parents:
diff changeset
781 }
a61af66fc99e Initial load
duke
parents:
diff changeset
782 }
a61af66fc99e Initial load
duke
parents:
diff changeset
783 #endif
a61af66fc99e Initial load
duke
parents:
diff changeset
784
4120
f03a3c8bd5e5 7077312: Provide a CALL effect for instruct declaration in the ad file
roland
parents: 3842
diff changeset
785 uint max_idx = matcher.C->unique();
0
a61af66fc99e Initial load
duke
parents:
diff changeset
786 // Pull from worklist and schedule
a61af66fc99e Initial load
duke
parents:
diff changeset
787 while( worklist.size() ) { // Worklist is not ready
a61af66fc99e Initial load
duke
parents:
diff changeset
788
a61af66fc99e Initial load
duke
parents:
diff changeset
789 #ifndef PRODUCT
a61af66fc99e Initial load
duke
parents:
diff changeset
790 if (cfg->trace_opto_pipelining()) {
a61af66fc99e Initial load
duke
parents:
diff changeset
791 tty->print("# ready list:");
a61af66fc99e Initial load
duke
parents:
diff changeset
792 for( uint i=0; i<worklist.size(); i++ ) { // Inspect entire worklist
a61af66fc99e Initial load
duke
parents:
diff changeset
793 Node *n = worklist[i]; // Get Node on worklist
a61af66fc99e Initial load
duke
parents:
diff changeset
794 tty->print(" %d", n->_idx);
a61af66fc99e Initial load
duke
parents:
diff changeset
795 }
a61af66fc99e Initial load
duke
parents:
diff changeset
796 tty->cr();
a61af66fc99e Initial load
duke
parents:
diff changeset
797 }
a61af66fc99e Initial load
duke
parents:
diff changeset
798 #endif
a61af66fc99e Initial load
duke
parents:
diff changeset
799
a61af66fc99e Initial load
duke
parents:
diff changeset
800 // Select and pop a ready guy from worklist
a61af66fc99e Initial load
duke
parents:
diff changeset
801 Node* n = select(cfg, worklist, ready_cnt, next_call, phi_cnt);
a61af66fc99e Initial load
duke
parents:
diff changeset
802 _nodes.map(phi_cnt++,n); // Schedule him next
a61af66fc99e Initial load
duke
parents:
diff changeset
803
a61af66fc99e Initial load
duke
parents:
diff changeset
804 #ifndef PRODUCT
a61af66fc99e Initial load
duke
parents:
diff changeset
805 if (cfg->trace_opto_pipelining()) {
a61af66fc99e Initial load
duke
parents:
diff changeset
806 tty->print("# select %d: %s", n->_idx, n->Name());
1685
0e35fa8ebccd 6973963: SEGV in ciBlock::start_bci() with EA
kvn
parents: 1579
diff changeset
807 tty->print(", latency:%d", cfg->_node_latency->at_grow(n->_idx));
0
a61af66fc99e Initial load
duke
parents:
diff changeset
808 n->dump();
a61af66fc99e Initial load
duke
parents:
diff changeset
809 if (Verbose) {
a61af66fc99e Initial load
duke
parents:
diff changeset
810 tty->print("# ready list:");
a61af66fc99e Initial load
duke
parents:
diff changeset
811 for( uint i=0; i<worklist.size(); i++ ) { // Inspect entire worklist
a61af66fc99e Initial load
duke
parents:
diff changeset
812 Node *n = worklist[i]; // Get Node on worklist
a61af66fc99e Initial load
duke
parents:
diff changeset
813 tty->print(" %d", n->_idx);
a61af66fc99e Initial load
duke
parents:
diff changeset
814 }
a61af66fc99e Initial load
duke
parents:
diff changeset
815 tty->cr();
a61af66fc99e Initial load
duke
parents:
diff changeset
816 }
a61af66fc99e Initial load
duke
parents:
diff changeset
817 }
a61af66fc99e Initial load
duke
parents:
diff changeset
818
a61af66fc99e Initial load
duke
parents:
diff changeset
819 #endif
a61af66fc99e Initial load
duke
parents:
diff changeset
820 if( n->is_MachCall() ) {
a61af66fc99e Initial load
duke
parents:
diff changeset
821 MachCallNode *mcall = n->as_MachCall();
a61af66fc99e Initial load
duke
parents:
diff changeset
822 phi_cnt = sched_call(matcher, cfg->_bbs, phi_cnt, worklist, ready_cnt, mcall, next_call);
a61af66fc99e Initial load
duke
parents:
diff changeset
823 continue;
a61af66fc99e Initial load
duke
parents:
diff changeset
824 }
4120
f03a3c8bd5e5 7077312: Provide a CALL effect for instruct declaration in the ad file
roland
parents: 3842
diff changeset
825
f03a3c8bd5e5 7077312: Provide a CALL effect for instruct declaration in the ad file
roland
parents: 3842
diff changeset
826 if (n->is_Mach() && n->as_Mach()->has_call()) {
f03a3c8bd5e5 7077312: Provide a CALL effect for instruct declaration in the ad file
roland
parents: 3842
diff changeset
827 RegMask regs;
f03a3c8bd5e5 7077312: Provide a CALL effect for instruct declaration in the ad file
roland
parents: 3842
diff changeset
828 regs.Insert(matcher.c_frame_pointer());
f03a3c8bd5e5 7077312: Provide a CALL effect for instruct declaration in the ad file
roland
parents: 3842
diff changeset
829 regs.OR(n->out_RegMask());
f03a3c8bd5e5 7077312: Provide a CALL effect for instruct declaration in the ad file
roland
parents: 3842
diff changeset
830
f03a3c8bd5e5 7077312: Provide a CALL effect for instruct declaration in the ad file
roland
parents: 3842
diff changeset
831 MachProjNode *proj = new (matcher.C, 1) MachProjNode( n, 1, RegMask::Empty, MachProjNode::fat_proj );
f03a3c8bd5e5 7077312: Provide a CALL effect for instruct declaration in the ad file
roland
parents: 3842
diff changeset
832 cfg->_bbs.map(proj->_idx,this);
f03a3c8bd5e5 7077312: Provide a CALL effect for instruct declaration in the ad file
roland
parents: 3842
diff changeset
833 _nodes.insert(phi_cnt++, proj);
f03a3c8bd5e5 7077312: Provide a CALL effect for instruct declaration in the ad file
roland
parents: 3842
diff changeset
834
f03a3c8bd5e5 7077312: Provide a CALL effect for instruct declaration in the ad file
roland
parents: 3842
diff changeset
835 add_call_kills(proj, regs, matcher._c_reg_save_policy, false);
f03a3c8bd5e5 7077312: Provide a CALL effect for instruct declaration in the ad file
roland
parents: 3842
diff changeset
836 }
f03a3c8bd5e5 7077312: Provide a CALL effect for instruct declaration in the ad file
roland
parents: 3842
diff changeset
837
0
a61af66fc99e Initial load
duke
parents:
diff changeset
838 // Children are now all ready
a61af66fc99e Initial load
duke
parents:
diff changeset
839 for (DUIterator_Fast i5max, i5 = n->fast_outs(i5max); i5 < i5max; i5++) {
a61af66fc99e Initial load
duke
parents:
diff changeset
840 Node* m = n->fast_out(i5); // Get user
a61af66fc99e Initial load
duke
parents:
diff changeset
841 if( cfg->_bbs[m->_idx] != this ) continue;
a61af66fc99e Initial load
duke
parents:
diff changeset
842 if( m->is_Phi() ) continue;
4120
f03a3c8bd5e5 7077312: Provide a CALL effect for instruct declaration in the ad file
roland
parents: 3842
diff changeset
843 if (m->_idx > max_idx) { // new node, skip it
f03a3c8bd5e5 7077312: Provide a CALL effect for instruct declaration in the ad file
roland
parents: 3842
diff changeset
844 assert(m->is_MachProj() && n->is_Mach() && n->as_Mach()->has_call(), "unexpected node types");
f03a3c8bd5e5 7077312: Provide a CALL effect for instruct declaration in the ad file
roland
parents: 3842
diff changeset
845 continue;
f03a3c8bd5e5 7077312: Provide a CALL effect for instruct declaration in the ad file
roland
parents: 3842
diff changeset
846 }
0
a61af66fc99e Initial load
duke
parents:
diff changeset
847 if( !--ready_cnt[m->_idx] )
a61af66fc99e Initial load
duke
parents:
diff changeset
848 worklist.push(m);
a61af66fc99e Initial load
duke
parents:
diff changeset
849 }
a61af66fc99e Initial load
duke
parents:
diff changeset
850 }
a61af66fc99e Initial load
duke
parents:
diff changeset
851
a61af66fc99e Initial load
duke
parents:
diff changeset
852 if( phi_cnt != end_idx() ) {
a61af66fc99e Initial load
duke
parents:
diff changeset
853 // did not schedule all. Retry, Bailout, or Die
a61af66fc99e Initial load
duke
parents:
diff changeset
854 Compile* C = matcher.C;
a61af66fc99e Initial load
duke
parents:
diff changeset
855 if (C->subsume_loads() == true && !C->failing()) {
a61af66fc99e Initial load
duke
parents:
diff changeset
856 // Retry with subsume_loads == false
a61af66fc99e Initial load
duke
parents:
diff changeset
857 // If this is the first failure, the sentinel string will "stick"
a61af66fc99e Initial load
duke
parents:
diff changeset
858 // to the Compile object, and the C2Compiler will see it and retry.
a61af66fc99e Initial load
duke
parents:
diff changeset
859 C->record_failure(C2Compiler::retry_no_subsuming_loads());
a61af66fc99e Initial load
duke
parents:
diff changeset
860 }
a61af66fc99e Initial load
duke
parents:
diff changeset
861 // assert( phi_cnt == end_idx(), "did not schedule all" );
a61af66fc99e Initial load
duke
parents:
diff changeset
862 return false;
a61af66fc99e Initial load
duke
parents:
diff changeset
863 }
a61af66fc99e Initial load
duke
parents:
diff changeset
864
a61af66fc99e Initial load
duke
parents:
diff changeset
865 #ifndef PRODUCT
a61af66fc99e Initial load
duke
parents:
diff changeset
866 if (cfg->trace_opto_pipelining()) {
a61af66fc99e Initial load
duke
parents:
diff changeset
867 tty->print_cr("#");
a61af66fc99e Initial load
duke
parents:
diff changeset
868 tty->print_cr("# after schedule_local");
a61af66fc99e Initial load
duke
parents:
diff changeset
869 for (uint i = 0;i < _nodes.size();i++) {
a61af66fc99e Initial load
duke
parents:
diff changeset
870 tty->print("# ");
a61af66fc99e Initial load
duke
parents:
diff changeset
871 _nodes[i]->fast_dump();
a61af66fc99e Initial load
duke
parents:
diff changeset
872 }
a61af66fc99e Initial load
duke
parents:
diff changeset
873 tty->cr();
a61af66fc99e Initial load
duke
parents:
diff changeset
874 }
a61af66fc99e Initial load
duke
parents:
diff changeset
875 #endif
a61af66fc99e Initial load
duke
parents:
diff changeset
876
a61af66fc99e Initial load
duke
parents:
diff changeset
877
a61af66fc99e Initial load
duke
parents:
diff changeset
878 return true;
a61af66fc99e Initial load
duke
parents:
diff changeset
879 }
a61af66fc99e Initial load
duke
parents:
diff changeset
880
a61af66fc99e Initial load
duke
parents:
diff changeset
881 //--------------------------catch_cleanup_fix_all_inputs-----------------------
a61af66fc99e Initial load
duke
parents:
diff changeset
882 static void catch_cleanup_fix_all_inputs(Node *use, Node *old_def, Node *new_def) {
a61af66fc99e Initial load
duke
parents:
diff changeset
883 for (uint l = 0; l < use->len(); l++) {
a61af66fc99e Initial load
duke
parents:
diff changeset
884 if (use->in(l) == old_def) {
a61af66fc99e Initial load
duke
parents:
diff changeset
885 if (l < use->req()) {
a61af66fc99e Initial load
duke
parents:
diff changeset
886 use->set_req(l, new_def);
a61af66fc99e Initial load
duke
parents:
diff changeset
887 } else {
a61af66fc99e Initial load
duke
parents:
diff changeset
888 use->rm_prec(l);
a61af66fc99e Initial load
duke
parents:
diff changeset
889 use->add_prec(new_def);
a61af66fc99e Initial load
duke
parents:
diff changeset
890 l--;
a61af66fc99e Initial load
duke
parents:
diff changeset
891 }
a61af66fc99e Initial load
duke
parents:
diff changeset
892 }
a61af66fc99e Initial load
duke
parents:
diff changeset
893 }
a61af66fc99e Initial load
duke
parents:
diff changeset
894 }
a61af66fc99e Initial load
duke
parents:
diff changeset
895
a61af66fc99e Initial load
duke
parents:
diff changeset
896 //------------------------------catch_cleanup_find_cloned_def------------------
a61af66fc99e Initial load
duke
parents:
diff changeset
897 static Node *catch_cleanup_find_cloned_def(Block *use_blk, Node *def, Block *def_blk, Block_Array &bbs, int n_clone_idx) {
a61af66fc99e Initial load
duke
parents:
diff changeset
898 assert( use_blk != def_blk, "Inter-block cleanup only");
a61af66fc99e Initial load
duke
parents:
diff changeset
899
a61af66fc99e Initial load
duke
parents:
diff changeset
900 // The use is some block below the Catch. Find and return the clone of the def
a61af66fc99e Initial load
duke
parents:
diff changeset
901 // that dominates the use. If there is no clone in a dominating block, then
a61af66fc99e Initial load
duke
parents:
diff changeset
902 // create a phi for the def in a dominating block.
a61af66fc99e Initial load
duke
parents:
diff changeset
903
a61af66fc99e Initial load
duke
parents:
diff changeset
904 // Find which successor block dominates this use. The successor
a61af66fc99e Initial load
duke
parents:
diff changeset
905 // blocks must all be single-entry (from the Catch only; I will have
a61af66fc99e Initial load
duke
parents:
diff changeset
906 // split blocks to make this so), hence they all dominate.
a61af66fc99e Initial load
duke
parents:
diff changeset
907 while( use_blk->_dom_depth > def_blk->_dom_depth+1 )
a61af66fc99e Initial load
duke
parents:
diff changeset
908 use_blk = use_blk->_idom;
a61af66fc99e Initial load
duke
parents:
diff changeset
909
a61af66fc99e Initial load
duke
parents:
diff changeset
910 // Find the successor
a61af66fc99e Initial load
duke
parents:
diff changeset
911 Node *fixup = NULL;
a61af66fc99e Initial load
duke
parents:
diff changeset
912
a61af66fc99e Initial load
duke
parents:
diff changeset
913 uint j;
a61af66fc99e Initial load
duke
parents:
diff changeset
914 for( j = 0; j < def_blk->_num_succs; j++ )
a61af66fc99e Initial load
duke
parents:
diff changeset
915 if( use_blk == def_blk->_succs[j] )
a61af66fc99e Initial load
duke
parents:
diff changeset
916 break;
a61af66fc99e Initial load
duke
parents:
diff changeset
917
a61af66fc99e Initial load
duke
parents:
diff changeset
918 if( j == def_blk->_num_succs ) {
a61af66fc99e Initial load
duke
parents:
diff changeset
919 // Block at same level in dom-tree is not a successor. It needs a
a61af66fc99e Initial load
duke
parents:
diff changeset
920 // PhiNode, the PhiNode uses from the def and IT's uses need fixup.
a61af66fc99e Initial load
duke
parents:
diff changeset
921 Node_Array inputs = new Node_List(Thread::current()->resource_area());
a61af66fc99e Initial load
duke
parents:
diff changeset
922 for(uint k = 1; k < use_blk->num_preds(); k++) {
a61af66fc99e Initial load
duke
parents:
diff changeset
923 inputs.map(k, catch_cleanup_find_cloned_def(bbs[use_blk->pred(k)->_idx], def, def_blk, bbs, n_clone_idx));
a61af66fc99e Initial load
duke
parents:
diff changeset
924 }
a61af66fc99e Initial load
duke
parents:
diff changeset
925
a61af66fc99e Initial load
duke
parents:
diff changeset
926 // Check to see if the use_blk already has an identical phi inserted.
a61af66fc99e Initial load
duke
parents:
diff changeset
927 // If it exists, it will be at the first position since all uses of a
a61af66fc99e Initial load
duke
parents:
diff changeset
928 // def are processed together.
a61af66fc99e Initial load
duke
parents:
diff changeset
929 Node *phi = use_blk->_nodes[1];
a61af66fc99e Initial load
duke
parents:
diff changeset
930 if( phi->is_Phi() ) {
a61af66fc99e Initial load
duke
parents:
diff changeset
931 fixup = phi;
a61af66fc99e Initial load
duke
parents:
diff changeset
932 for (uint k = 1; k < use_blk->num_preds(); k++) {
a61af66fc99e Initial load
duke
parents:
diff changeset
933 if (phi->in(k) != inputs[k]) {
a61af66fc99e Initial load
duke
parents:
diff changeset
934 // Not a match
a61af66fc99e Initial load
duke
parents:
diff changeset
935 fixup = NULL;
a61af66fc99e Initial load
duke
parents:
diff changeset
936 break;
a61af66fc99e Initial load
duke
parents:
diff changeset
937 }
a61af66fc99e Initial load
duke
parents:
diff changeset
938 }
a61af66fc99e Initial load
duke
parents:
diff changeset
939 }
a61af66fc99e Initial load
duke
parents:
diff changeset
940
a61af66fc99e Initial load
duke
parents:
diff changeset
941 // If an existing PhiNode was not found, make a new one.
a61af66fc99e Initial load
duke
parents:
diff changeset
942 if (fixup == NULL) {
a61af66fc99e Initial load
duke
parents:
diff changeset
943 Node *new_phi = PhiNode::make(use_blk->head(), def);
a61af66fc99e Initial load
duke
parents:
diff changeset
944 use_blk->_nodes.insert(1, new_phi);
a61af66fc99e Initial load
duke
parents:
diff changeset
945 bbs.map(new_phi->_idx, use_blk);
a61af66fc99e Initial load
duke
parents:
diff changeset
946 for (uint k = 1; k < use_blk->num_preds(); k++) {
a61af66fc99e Initial load
duke
parents:
diff changeset
947 new_phi->set_req(k, inputs[k]);
a61af66fc99e Initial load
duke
parents:
diff changeset
948 }
a61af66fc99e Initial load
duke
parents:
diff changeset
949 fixup = new_phi;
a61af66fc99e Initial load
duke
parents:
diff changeset
950 }
a61af66fc99e Initial load
duke
parents:
diff changeset
951
a61af66fc99e Initial load
duke
parents:
diff changeset
952 } else {
a61af66fc99e Initial load
duke
parents:
diff changeset
953 // Found the use just below the Catch. Make it use the clone.
a61af66fc99e Initial load
duke
parents:
diff changeset
954 fixup = use_blk->_nodes[n_clone_idx];
a61af66fc99e Initial load
duke
parents:
diff changeset
955 }
a61af66fc99e Initial load
duke
parents:
diff changeset
956
a61af66fc99e Initial load
duke
parents:
diff changeset
957 return fixup;
a61af66fc99e Initial load
duke
parents:
diff changeset
958 }
a61af66fc99e Initial load
duke
parents:
diff changeset
959
a61af66fc99e Initial load
duke
parents:
diff changeset
960 //--------------------------catch_cleanup_intra_block--------------------------
a61af66fc99e Initial load
duke
parents:
diff changeset
961 // Fix all input edges in use that reference "def". The use is in the same
a61af66fc99e Initial load
duke
parents:
diff changeset
962 // block as the def and both have been cloned in each successor block.
a61af66fc99e Initial load
duke
parents:
diff changeset
963 static void catch_cleanup_intra_block(Node *use, Node *def, Block *blk, int beg, int n_clone_idx) {
a61af66fc99e Initial load
duke
parents:
diff changeset
964
a61af66fc99e Initial load
duke
parents:
diff changeset
965 // Both the use and def have been cloned. For each successor block,
a61af66fc99e Initial load
duke
parents:
diff changeset
966 // get the clone of the use, and make its input the clone of the def
a61af66fc99e Initial load
duke
parents:
diff changeset
967 // found in that block.
a61af66fc99e Initial load
duke
parents:
diff changeset
968
a61af66fc99e Initial load
duke
parents:
diff changeset
969 uint use_idx = blk->find_node(use);
a61af66fc99e Initial load
duke
parents:
diff changeset
970 uint offset_idx = use_idx - beg;
a61af66fc99e Initial load
duke
parents:
diff changeset
971 for( uint k = 0; k < blk->_num_succs; k++ ) {
a61af66fc99e Initial load
duke
parents:
diff changeset
972 // Get clone in each successor block
a61af66fc99e Initial load
duke
parents:
diff changeset
973 Block *sb = blk->_succs[k];
a61af66fc99e Initial load
duke
parents:
diff changeset
974 Node *clone = sb->_nodes[offset_idx+1];
a61af66fc99e Initial load
duke
parents:
diff changeset
975 assert( clone->Opcode() == use->Opcode(), "" );
a61af66fc99e Initial load
duke
parents:
diff changeset
976
a61af66fc99e Initial load
duke
parents:
diff changeset
977 // Make use-clone reference the def-clone
a61af66fc99e Initial load
duke
parents:
diff changeset
978 catch_cleanup_fix_all_inputs(clone, def, sb->_nodes[n_clone_idx]);
a61af66fc99e Initial load
duke
parents:
diff changeset
979 }
a61af66fc99e Initial load
duke
parents:
diff changeset
980 }
a61af66fc99e Initial load
duke
parents:
diff changeset
981
a61af66fc99e Initial load
duke
parents:
diff changeset
982 //------------------------------catch_cleanup_inter_block---------------------
a61af66fc99e Initial load
duke
parents:
diff changeset
983 // Fix all input edges in use that reference "def". The use is in a different
a61af66fc99e Initial load
duke
parents:
diff changeset
984 // block than the def.
a61af66fc99e Initial load
duke
parents:
diff changeset
985 static void catch_cleanup_inter_block(Node *use, Block *use_blk, Node *def, Block *def_blk, Block_Array &bbs, int n_clone_idx) {
a61af66fc99e Initial load
duke
parents:
diff changeset
986 if( !use_blk ) return; // Can happen if the use is a precedence edge
a61af66fc99e Initial load
duke
parents:
diff changeset
987
a61af66fc99e Initial load
duke
parents:
diff changeset
988 Node *new_def = catch_cleanup_find_cloned_def(use_blk, def, def_blk, bbs, n_clone_idx);
a61af66fc99e Initial load
duke
parents:
diff changeset
989 catch_cleanup_fix_all_inputs(use, def, new_def);
a61af66fc99e Initial load
duke
parents:
diff changeset
990 }
a61af66fc99e Initial load
duke
parents:
diff changeset
991
a61af66fc99e Initial load
duke
parents:
diff changeset
992 //------------------------------call_catch_cleanup-----------------------------
a61af66fc99e Initial load
duke
parents:
diff changeset
993 // If we inserted any instructions between a Call and his CatchNode,
a61af66fc99e Initial load
duke
parents:
diff changeset
994 // clone the instructions on all paths below the Catch.
a61af66fc99e Initial load
duke
parents:
diff changeset
995 void Block::call_catch_cleanup(Block_Array &bbs) {
a61af66fc99e Initial load
duke
parents:
diff changeset
996
a61af66fc99e Initial load
duke
parents:
diff changeset
997 // End of region to clone
a61af66fc99e Initial load
duke
parents:
diff changeset
998 uint end = end_idx();
a61af66fc99e Initial load
duke
parents:
diff changeset
999 if( !_nodes[end]->is_Catch() ) return;
a61af66fc99e Initial load
duke
parents:
diff changeset
1000 // Start of region to clone
a61af66fc99e Initial load
duke
parents:
diff changeset
1001 uint beg = end;
3842
c7b60b601eb4 7069452: Cleanup NodeFlags
kvn
parents: 3796
diff changeset
1002 while(!_nodes[beg-1]->is_MachProj() ||
c7b60b601eb4 7069452: Cleanup NodeFlags
kvn
parents: 3796
diff changeset
1003 !_nodes[beg-1]->in(0)->is_MachCall() ) {
0
a61af66fc99e Initial load
duke
parents:
diff changeset
1004 beg--;
a61af66fc99e Initial load
duke
parents:
diff changeset
1005 assert(beg > 0,"Catch cleanup walking beyond block boundary");
a61af66fc99e Initial load
duke
parents:
diff changeset
1006 }
a61af66fc99e Initial load
duke
parents:
diff changeset
1007 // Range of inserted instructions is [beg, end)
a61af66fc99e Initial load
duke
parents:
diff changeset
1008 if( beg == end ) return;
a61af66fc99e Initial load
duke
parents:
diff changeset
1009
a61af66fc99e Initial load
duke
parents:
diff changeset
1010 // Clone along all Catch output paths. Clone area between the 'beg' and
a61af66fc99e Initial load
duke
parents:
diff changeset
1011 // 'end' indices.
a61af66fc99e Initial load
duke
parents:
diff changeset
1012 for( uint i = 0; i < _num_succs; i++ ) {
a61af66fc99e Initial load
duke
parents:
diff changeset
1013 Block *sb = _succs[i];
a61af66fc99e Initial load
duke
parents:
diff changeset
1014 // Clone the entire area; ignoring the edge fixup for now.
a61af66fc99e Initial load
duke
parents:
diff changeset
1015 for( uint j = end; j > beg; j-- ) {
1693
6c9cc03d8726 6973329: C2 with Zero based COOP produces code with broken anti-dependency on x86
kvn
parents: 1685
diff changeset
1016 // It is safe here to clone a node with anti_dependence
6c9cc03d8726 6973329: C2 with Zero based COOP produces code with broken anti-dependency on x86
kvn
parents: 1685
diff changeset
1017 // since clones dominate on each path.
0
a61af66fc99e Initial load
duke
parents:
diff changeset
1018 Node *clone = _nodes[j-1]->clone();
a61af66fc99e Initial load
duke
parents:
diff changeset
1019 sb->_nodes.insert( 1, clone );
a61af66fc99e Initial load
duke
parents:
diff changeset
1020 bbs.map(clone->_idx,sb);
a61af66fc99e Initial load
duke
parents:
diff changeset
1021 }
a61af66fc99e Initial load
duke
parents:
diff changeset
1022 }
a61af66fc99e Initial load
duke
parents:
diff changeset
1023
a61af66fc99e Initial load
duke
parents:
diff changeset
1024
a61af66fc99e Initial load
duke
parents:
diff changeset
1025 // Fixup edges. Check the def-use info per cloned Node
a61af66fc99e Initial load
duke
parents:
diff changeset
1026 for(uint i2 = beg; i2 < end; i2++ ) {
a61af66fc99e Initial load
duke
parents:
diff changeset
1027 uint n_clone_idx = i2-beg+1; // Index of clone of n in each successor block
a61af66fc99e Initial load
duke
parents:
diff changeset
1028 Node *n = _nodes[i2]; // Node that got cloned
a61af66fc99e Initial load
duke
parents:
diff changeset
1029 // Need DU safe iterator because of edge manipulation in calls.
a61af66fc99e Initial load
duke
parents:
diff changeset
1030 Unique_Node_List *out = new Unique_Node_List(Thread::current()->resource_area());
a61af66fc99e Initial load
duke
parents:
diff changeset
1031 for (DUIterator_Fast j1max, j1 = n->fast_outs(j1max); j1 < j1max; j1++) {
a61af66fc99e Initial load
duke
parents:
diff changeset
1032 out->push(n->fast_out(j1));
a61af66fc99e Initial load
duke
parents:
diff changeset
1033 }
a61af66fc99e Initial load
duke
parents:
diff changeset
1034 uint max = out->size();
a61af66fc99e Initial load
duke
parents:
diff changeset
1035 for (uint j = 0; j < max; j++) {// For all users
a61af66fc99e Initial load
duke
parents:
diff changeset
1036 Node *use = out->pop();
a61af66fc99e Initial load
duke
parents:
diff changeset
1037 Block *buse = bbs[use->_idx];
a61af66fc99e Initial load
duke
parents:
diff changeset
1038 if( use->is_Phi() ) {
a61af66fc99e Initial load
duke
parents:
diff changeset
1039 for( uint k = 1; k < use->req(); k++ )
a61af66fc99e Initial load
duke
parents:
diff changeset
1040 if( use->in(k) == n ) {
a61af66fc99e Initial load
duke
parents:
diff changeset
1041 Node *fixup = catch_cleanup_find_cloned_def(bbs[buse->pred(k)->_idx], n, this, bbs, n_clone_idx);
a61af66fc99e Initial load
duke
parents:
diff changeset
1042 use->set_req(k, fixup);
a61af66fc99e Initial load
duke
parents:
diff changeset
1043 }
a61af66fc99e Initial load
duke
parents:
diff changeset
1044 } else {
a61af66fc99e Initial load
duke
parents:
diff changeset
1045 if (this == buse) {
a61af66fc99e Initial load
duke
parents:
diff changeset
1046 catch_cleanup_intra_block(use, n, this, beg, n_clone_idx);
a61af66fc99e Initial load
duke
parents:
diff changeset
1047 } else {
a61af66fc99e Initial load
duke
parents:
diff changeset
1048 catch_cleanup_inter_block(use, buse, n, this, bbs, n_clone_idx);
a61af66fc99e Initial load
duke
parents:
diff changeset
1049 }
a61af66fc99e Initial load
duke
parents:
diff changeset
1050 }
a61af66fc99e Initial load
duke
parents:
diff changeset
1051 } // End for all users
a61af66fc99e Initial load
duke
parents:
diff changeset
1052
a61af66fc99e Initial load
duke
parents:
diff changeset
1053 } // End of for all Nodes in cloned area
a61af66fc99e Initial load
duke
parents:
diff changeset
1054
a61af66fc99e Initial load
duke
parents:
diff changeset
1055 // Remove the now-dead cloned ops
a61af66fc99e Initial load
duke
parents:
diff changeset
1056 for(uint i3 = beg; i3 < end; i3++ ) {
a61af66fc99e Initial load
duke
parents:
diff changeset
1057 _nodes[beg]->disconnect_inputs(NULL);
a61af66fc99e Initial load
duke
parents:
diff changeset
1058 _nodes.remove(beg);
a61af66fc99e Initial load
duke
parents:
diff changeset
1059 }
a61af66fc99e Initial load
duke
parents:
diff changeset
1060
a61af66fc99e Initial load
duke
parents:
diff changeset
1061 // If the successor blocks have a CreateEx node, move it back to the top
a61af66fc99e Initial load
duke
parents:
diff changeset
1062 for(uint i4 = 0; i4 < _num_succs; i4++ ) {
a61af66fc99e Initial load
duke
parents:
diff changeset
1063 Block *sb = _succs[i4];
a61af66fc99e Initial load
duke
parents:
diff changeset
1064 uint new_cnt = end - beg;
a61af66fc99e Initial load
duke
parents:
diff changeset
1065 // Remove any newly created, but dead, nodes.
a61af66fc99e Initial load
duke
parents:
diff changeset
1066 for( uint j = new_cnt; j > 0; j-- ) {
a61af66fc99e Initial load
duke
parents:
diff changeset
1067 Node *n = sb->_nodes[j];
a61af66fc99e Initial load
duke
parents:
diff changeset
1068 if (n->outcnt() == 0 &&
a61af66fc99e Initial load
duke
parents:
diff changeset
1069 (!n->is_Proj() || n->as_Proj()->in(0)->outcnt() == 1) ){
a61af66fc99e Initial load
duke
parents:
diff changeset
1070 n->disconnect_inputs(NULL);
a61af66fc99e Initial load
duke
parents:
diff changeset
1071 sb->_nodes.remove(j);
a61af66fc99e Initial load
duke
parents:
diff changeset
1072 new_cnt--;
a61af66fc99e Initial load
duke
parents:
diff changeset
1073 }
a61af66fc99e Initial load
duke
parents:
diff changeset
1074 }
a61af66fc99e Initial load
duke
parents:
diff changeset
1075 // If any newly created nodes remain, move the CreateEx node to the top
a61af66fc99e Initial load
duke
parents:
diff changeset
1076 if (new_cnt > 0) {
a61af66fc99e Initial load
duke
parents:
diff changeset
1077 Node *cex = sb->_nodes[1+new_cnt];
a61af66fc99e Initial load
duke
parents:
diff changeset
1078 if( cex->is_Mach() && cex->as_Mach()->ideal_Opcode() == Op_CreateEx ) {
a61af66fc99e Initial load
duke
parents:
diff changeset
1079 sb->_nodes.remove(1+new_cnt);
a61af66fc99e Initial load
duke
parents:
diff changeset
1080 sb->_nodes.insert(1,cex);
a61af66fc99e Initial load
duke
parents:
diff changeset
1081 }
a61af66fc99e Initial load
duke
parents:
diff changeset
1082 }
a61af66fc99e Initial load
duke
parents:
diff changeset
1083 }
a61af66fc99e Initial load
duke
parents:
diff changeset
1084 }