annotate src/share/vm/opto/block.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 1bd45abaa507
children e626685e9f6c
Ignore whitespace changes - Everywhere: Within whitespace: At end of lines:
rev   line source
0
a61af66fc99e Initial load
duke
parents:
diff changeset
1 /*
1972
f95d63e2154a 6989984: Use standard include model for Hospot
stefank
parents: 1685
diff changeset
2 * Copyright (c) 1997, 2010, 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: 893
diff changeset
19 * Please contact Oracle, 500 Oracle Parkway, Redwood Shores, CA 94065 USA
c18cbe5936b8 6941466: Oracle rebranding changes for Hotspot repositories
trims
parents: 893
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: 893
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: 1685
diff changeset
25 #include "precompiled.hpp"
f95d63e2154a 6989984: Use standard include model for Hospot
stefank
parents: 1685
diff changeset
26 #include "libadt/vectset.hpp"
f95d63e2154a 6989984: Use standard include model for Hospot
stefank
parents: 1685
diff changeset
27 #include "memory/allocation.inline.hpp"
f95d63e2154a 6989984: Use standard include model for Hospot
stefank
parents: 1685
diff changeset
28 #include "opto/block.hpp"
f95d63e2154a 6989984: Use standard include model for Hospot
stefank
parents: 1685
diff changeset
29 #include "opto/cfgnode.hpp"
f95d63e2154a 6989984: Use standard include model for Hospot
stefank
parents: 1685
diff changeset
30 #include "opto/chaitin.hpp"
f95d63e2154a 6989984: Use standard include model for Hospot
stefank
parents: 1685
diff changeset
31 #include "opto/loopnode.hpp"
f95d63e2154a 6989984: Use standard include model for Hospot
stefank
parents: 1685
diff changeset
32 #include "opto/machnode.hpp"
f95d63e2154a 6989984: Use standard include model for Hospot
stefank
parents: 1685
diff changeset
33 #include "opto/matcher.hpp"
f95d63e2154a 6989984: Use standard include model for Hospot
stefank
parents: 1685
diff changeset
34 #include "opto/opcodes.hpp"
f95d63e2154a 6989984: Use standard include model for Hospot
stefank
parents: 1685
diff changeset
35 #include "opto/rootnode.hpp"
f95d63e2154a 6989984: Use standard include model for Hospot
stefank
parents: 1685
diff changeset
36 #include "utilities/copy.hpp"
0
a61af66fc99e Initial load
duke
parents:
diff changeset
37
1972
f95d63e2154a 6989984: Use standard include model for Hospot
stefank
parents: 1685
diff changeset
38 // Optimization - Graph Style
0
a61af66fc99e Initial load
duke
parents:
diff changeset
39
a61af66fc99e Initial load
duke
parents:
diff changeset
40
a61af66fc99e Initial load
duke
parents:
diff changeset
41 //-----------------------------------------------------------------------------
a61af66fc99e Initial load
duke
parents:
diff changeset
42 void Block_Array::grow( uint i ) {
a61af66fc99e Initial load
duke
parents:
diff changeset
43 assert(i >= Max(), "must be an overflow");
a61af66fc99e Initial load
duke
parents:
diff changeset
44 debug_only(_limit = i+1);
a61af66fc99e Initial load
duke
parents:
diff changeset
45 if( i < _size ) return;
a61af66fc99e Initial load
duke
parents:
diff changeset
46 if( !_size ) {
a61af66fc99e Initial load
duke
parents:
diff changeset
47 _size = 1;
a61af66fc99e Initial load
duke
parents:
diff changeset
48 _blocks = (Block**)_arena->Amalloc( _size * sizeof(Block*) );
a61af66fc99e Initial load
duke
parents:
diff changeset
49 _blocks[0] = NULL;
a61af66fc99e Initial load
duke
parents:
diff changeset
50 }
a61af66fc99e Initial load
duke
parents:
diff changeset
51 uint old = _size;
a61af66fc99e Initial load
duke
parents:
diff changeset
52 while( i >= _size ) _size <<= 1; // Double to fit
a61af66fc99e Initial load
duke
parents:
diff changeset
53 _blocks = (Block**)_arena->Arealloc( _blocks, old*sizeof(Block*),_size*sizeof(Block*));
a61af66fc99e Initial load
duke
parents:
diff changeset
54 Copy::zero_to_bytes( &_blocks[old], (_size-old)*sizeof(Block*) );
a61af66fc99e Initial load
duke
parents:
diff changeset
55 }
a61af66fc99e Initial load
duke
parents:
diff changeset
56
a61af66fc99e Initial load
duke
parents:
diff changeset
57 //=============================================================================
a61af66fc99e Initial load
duke
parents:
diff changeset
58 void Block_List::remove(uint i) {
a61af66fc99e Initial load
duke
parents:
diff changeset
59 assert(i < _cnt, "index out of bounds");
a61af66fc99e Initial load
duke
parents:
diff changeset
60 Copy::conjoint_words_to_lower((HeapWord*)&_blocks[i+1], (HeapWord*)&_blocks[i], ((_cnt-i-1)*sizeof(Block*)));
a61af66fc99e Initial load
duke
parents:
diff changeset
61 pop(); // shrink list by one block
a61af66fc99e Initial load
duke
parents:
diff changeset
62 }
a61af66fc99e Initial load
duke
parents:
diff changeset
63
a61af66fc99e Initial load
duke
parents:
diff changeset
64 void Block_List::insert(uint i, Block *b) {
a61af66fc99e Initial load
duke
parents:
diff changeset
65 push(b); // grow list by one block
a61af66fc99e Initial load
duke
parents:
diff changeset
66 Copy::conjoint_words_to_higher((HeapWord*)&_blocks[i], (HeapWord*)&_blocks[i+1], ((_cnt-i-1)*sizeof(Block*)));
a61af66fc99e Initial load
duke
parents:
diff changeset
67 _blocks[i] = b;
a61af66fc99e Initial load
duke
parents:
diff changeset
68 }
a61af66fc99e Initial load
duke
parents:
diff changeset
69
418
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
70 #ifndef PRODUCT
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
71 void Block_List::print() {
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
72 for (uint i=0; i < size(); i++) {
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
73 tty->print("B%d ", _blocks[i]->_pre_order);
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
74 }
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
75 tty->print("size = %d\n", size());
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
76 }
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
77 #endif
0
a61af66fc99e Initial load
duke
parents:
diff changeset
78
a61af66fc99e Initial load
duke
parents:
diff changeset
79 //=============================================================================
a61af66fc99e Initial load
duke
parents:
diff changeset
80
a61af66fc99e Initial load
duke
parents:
diff changeset
81 uint Block::code_alignment() {
a61af66fc99e Initial load
duke
parents:
diff changeset
82 // Check for Root block
3851
95134e034042 7063629: use cbcond in C2 generated code on T4
kvn
parents: 3842
diff changeset
83 if (_pre_order == 0) return CodeEntryAlignment;
0
a61af66fc99e Initial load
duke
parents:
diff changeset
84 // Check for Start block
3851
95134e034042 7063629: use cbcond in C2 generated code on T4
kvn
parents: 3842
diff changeset
85 if (_pre_order == 1) return InteriorEntryAlignment;
0
a61af66fc99e Initial load
duke
parents:
diff changeset
86 // Check for loop alignment
3851
95134e034042 7063629: use cbcond in C2 generated code on T4
kvn
parents: 3842
diff changeset
87 if (has_loop_alignment()) return loop_alignment();
418
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
88
3851
95134e034042 7063629: use cbcond in C2 generated code on T4
kvn
parents: 3842
diff changeset
89 return relocInfo::addr_unit(); // no particular alignment
418
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
90 }
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
91
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
92 uint Block::compute_loop_alignment() {
0
a61af66fc99e Initial load
duke
parents:
diff changeset
93 Node *h = head();
3851
95134e034042 7063629: use cbcond in C2 generated code on T4
kvn
parents: 3842
diff changeset
94 int unit_sz = relocInfo::addr_unit();
95134e034042 7063629: use cbcond in C2 generated code on T4
kvn
parents: 3842
diff changeset
95 if (h->is_Loop() && h->as_Loop()->is_inner_loop()) {
0
a61af66fc99e Initial load
duke
parents:
diff changeset
96 // Pre- and post-loops have low trip count so do not bother with
a61af66fc99e Initial load
duke
parents:
diff changeset
97 // NOPs for align loop head. The constants are hidden from tuning
a61af66fc99e Initial load
duke
parents:
diff changeset
98 // but only because my "divide by 4" heuristic surely gets nearly
a61af66fc99e Initial load
duke
parents:
diff changeset
99 // all possible gain (a "do not align at all" heuristic has a
a61af66fc99e Initial load
duke
parents:
diff changeset
100 // chance of getting a really tiny gain).
3851
95134e034042 7063629: use cbcond in C2 generated code on T4
kvn
parents: 3842
diff changeset
101 if (h->is_CountedLoop() && (h->as_CountedLoop()->is_pre_loop() ||
95134e034042 7063629: use cbcond in C2 generated code on T4
kvn
parents: 3842
diff changeset
102 h->as_CountedLoop()->is_post_loop())) {
95134e034042 7063629: use cbcond in C2 generated code on T4
kvn
parents: 3842
diff changeset
103 return (OptoLoopAlignment > 4*unit_sz) ? (OptoLoopAlignment>>2) : unit_sz;
95134e034042 7063629: use cbcond in C2 generated code on T4
kvn
parents: 3842
diff changeset
104 }
0
a61af66fc99e Initial load
duke
parents:
diff changeset
105 // Loops with low backedge frequency should not be aligned.
a61af66fc99e Initial load
duke
parents:
diff changeset
106 Node *n = h->in(LoopNode::LoopBackControl)->in(0);
3851
95134e034042 7063629: use cbcond in C2 generated code on T4
kvn
parents: 3842
diff changeset
107 if (n->is_MachIf() && n->as_MachIf()->_prob < 0.01) {
95134e034042 7063629: use cbcond in C2 generated code on T4
kvn
parents: 3842
diff changeset
108 return unit_sz; // Loop does not loop, more often than not!
0
a61af66fc99e Initial load
duke
parents:
diff changeset
109 }
a61af66fc99e Initial load
duke
parents:
diff changeset
110 return OptoLoopAlignment; // Otherwise align loop head
a61af66fc99e Initial load
duke
parents:
diff changeset
111 }
418
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
112
3851
95134e034042 7063629: use cbcond in C2 generated code on T4
kvn
parents: 3842
diff changeset
113 return unit_sz; // no particular alignment
0
a61af66fc99e Initial load
duke
parents:
diff changeset
114 }
a61af66fc99e Initial load
duke
parents:
diff changeset
115
a61af66fc99e Initial load
duke
parents:
diff changeset
116 //-----------------------------------------------------------------------------
a61af66fc99e Initial load
duke
parents:
diff changeset
117 // Compute the size of first 'inst_cnt' instructions in this block.
a61af66fc99e Initial load
duke
parents:
diff changeset
118 // Return the number of instructions left to compute if the block has
418
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
119 // less then 'inst_cnt' instructions. Stop, and return 0 if sum_size
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
120 // exceeds OptoLoopAlignment.
0
a61af66fc99e Initial load
duke
parents:
diff changeset
121 uint Block::compute_first_inst_size(uint& sum_size, uint inst_cnt,
a61af66fc99e Initial load
duke
parents:
diff changeset
122 PhaseRegAlloc* ra) {
a61af66fc99e Initial load
duke
parents:
diff changeset
123 uint last_inst = _nodes.size();
a61af66fc99e Initial load
duke
parents:
diff changeset
124 for( uint j = 0; j < last_inst && inst_cnt > 0; j++ ) {
a61af66fc99e Initial load
duke
parents:
diff changeset
125 uint inst_size = _nodes[j]->size(ra);
a61af66fc99e Initial load
duke
parents:
diff changeset
126 if( inst_size > 0 ) {
a61af66fc99e Initial load
duke
parents:
diff changeset
127 inst_cnt--;
a61af66fc99e Initial load
duke
parents:
diff changeset
128 uint sz = sum_size + inst_size;
a61af66fc99e Initial load
duke
parents:
diff changeset
129 if( sz <= (uint)OptoLoopAlignment ) {
a61af66fc99e Initial load
duke
parents:
diff changeset
130 // Compute size of instructions which fit into fetch buffer only
a61af66fc99e Initial load
duke
parents:
diff changeset
131 // since all inst_cnt instructions will not fit even if we align them.
a61af66fc99e Initial load
duke
parents:
diff changeset
132 sum_size = sz;
a61af66fc99e Initial load
duke
parents:
diff changeset
133 } else {
a61af66fc99e Initial load
duke
parents:
diff changeset
134 return 0;
a61af66fc99e Initial load
duke
parents:
diff changeset
135 }
a61af66fc99e Initial load
duke
parents:
diff changeset
136 }
a61af66fc99e Initial load
duke
parents:
diff changeset
137 }
a61af66fc99e Initial load
duke
parents:
diff changeset
138 return inst_cnt;
a61af66fc99e Initial load
duke
parents:
diff changeset
139 }
a61af66fc99e Initial load
duke
parents:
diff changeset
140
a61af66fc99e Initial load
duke
parents:
diff changeset
141 //-----------------------------------------------------------------------------
a61af66fc99e Initial load
duke
parents:
diff changeset
142 uint Block::find_node( const Node *n ) const {
a61af66fc99e Initial load
duke
parents:
diff changeset
143 for( uint i = 0; i < _nodes.size(); i++ ) {
a61af66fc99e Initial load
duke
parents:
diff changeset
144 if( _nodes[i] == n )
a61af66fc99e Initial load
duke
parents:
diff changeset
145 return i;
a61af66fc99e Initial load
duke
parents:
diff changeset
146 }
a61af66fc99e Initial load
duke
parents:
diff changeset
147 ShouldNotReachHere();
a61af66fc99e Initial load
duke
parents:
diff changeset
148 return 0;
a61af66fc99e Initial load
duke
parents:
diff changeset
149 }
a61af66fc99e Initial load
duke
parents:
diff changeset
150
a61af66fc99e Initial load
duke
parents:
diff changeset
151 // Find and remove n from block list
a61af66fc99e Initial load
duke
parents:
diff changeset
152 void Block::find_remove( const Node *n ) {
a61af66fc99e Initial load
duke
parents:
diff changeset
153 _nodes.remove(find_node(n));
a61af66fc99e Initial load
duke
parents:
diff changeset
154 }
a61af66fc99e Initial load
duke
parents:
diff changeset
155
a61af66fc99e Initial load
duke
parents:
diff changeset
156 //------------------------------is_Empty---------------------------------------
a61af66fc99e Initial load
duke
parents:
diff changeset
157 // Return empty status of a block. Empty blocks contain only the head, other
a61af66fc99e Initial load
duke
parents:
diff changeset
158 // ideal nodes, and an optional trailing goto.
a61af66fc99e Initial load
duke
parents:
diff changeset
159 int Block::is_Empty() const {
a61af66fc99e Initial load
duke
parents:
diff changeset
160
a61af66fc99e Initial load
duke
parents:
diff changeset
161 // Root or start block is not considered empty
a61af66fc99e Initial load
duke
parents:
diff changeset
162 if (head()->is_Root() || head()->is_Start()) {
a61af66fc99e Initial load
duke
parents:
diff changeset
163 return not_empty;
a61af66fc99e Initial load
duke
parents:
diff changeset
164 }
a61af66fc99e Initial load
duke
parents:
diff changeset
165
a61af66fc99e Initial load
duke
parents:
diff changeset
166 int success_result = completely_empty;
a61af66fc99e Initial load
duke
parents:
diff changeset
167 int end_idx = _nodes.size()-1;
a61af66fc99e Initial load
duke
parents:
diff changeset
168
a61af66fc99e Initial load
duke
parents:
diff changeset
169 // Check for ending goto
3842
c7b60b601eb4 7069452: Cleanup NodeFlags
kvn
parents: 1972
diff changeset
170 if ((end_idx > 0) && (_nodes[end_idx]->is_MachGoto())) {
0
a61af66fc99e Initial load
duke
parents:
diff changeset
171 success_result = empty_with_goto;
a61af66fc99e Initial load
duke
parents:
diff changeset
172 end_idx--;
a61af66fc99e Initial load
duke
parents:
diff changeset
173 }
a61af66fc99e Initial load
duke
parents:
diff changeset
174
a61af66fc99e Initial load
duke
parents:
diff changeset
175 // Unreachable blocks are considered empty
a61af66fc99e Initial load
duke
parents:
diff changeset
176 if (num_preds() <= 1) {
a61af66fc99e Initial load
duke
parents:
diff changeset
177 return success_result;
a61af66fc99e Initial load
duke
parents:
diff changeset
178 }
a61af66fc99e Initial load
duke
parents:
diff changeset
179
a61af66fc99e Initial load
duke
parents:
diff changeset
180 // Ideal nodes are allowable in empty blocks: skip them Only MachNodes
a61af66fc99e Initial load
duke
parents:
diff changeset
181 // turn directly into code, because only MachNodes have non-trivial
a61af66fc99e Initial load
duke
parents:
diff changeset
182 // emit() functions.
a61af66fc99e Initial load
duke
parents:
diff changeset
183 while ((end_idx > 0) && !_nodes[end_idx]->is_Mach()) {
a61af66fc99e Initial load
duke
parents:
diff changeset
184 end_idx--;
a61af66fc99e Initial load
duke
parents:
diff changeset
185 }
a61af66fc99e Initial load
duke
parents:
diff changeset
186
a61af66fc99e Initial load
duke
parents:
diff changeset
187 // No room for any interesting instructions?
a61af66fc99e Initial load
duke
parents:
diff changeset
188 if (end_idx == 0) {
a61af66fc99e Initial load
duke
parents:
diff changeset
189 return success_result;
a61af66fc99e Initial load
duke
parents:
diff changeset
190 }
a61af66fc99e Initial load
duke
parents:
diff changeset
191
a61af66fc99e Initial load
duke
parents:
diff changeset
192 return not_empty;
a61af66fc99e Initial load
duke
parents:
diff changeset
193 }
a61af66fc99e Initial load
duke
parents:
diff changeset
194
a61af66fc99e Initial load
duke
parents:
diff changeset
195 //------------------------------has_uncommon_code------------------------------
605
98cb887364d3 6810672: Comment typos
twisti
parents: 601
diff changeset
196 // Return true if the block's code implies that it is likely to be
0
a61af66fc99e Initial load
duke
parents:
diff changeset
197 // executed infrequently. Check to see if the block ends in a Halt or
a61af66fc99e Initial load
duke
parents:
diff changeset
198 // a low probability call.
a61af66fc99e Initial load
duke
parents:
diff changeset
199 bool Block::has_uncommon_code() const {
a61af66fc99e Initial load
duke
parents:
diff changeset
200 Node* en = end();
a61af66fc99e Initial load
duke
parents:
diff changeset
201
3842
c7b60b601eb4 7069452: Cleanup NodeFlags
kvn
parents: 1972
diff changeset
202 if (en->is_MachGoto())
0
a61af66fc99e Initial load
duke
parents:
diff changeset
203 en = en->in(0);
a61af66fc99e Initial load
duke
parents:
diff changeset
204 if (en->is_Catch())
a61af66fc99e Initial load
duke
parents:
diff changeset
205 en = en->in(0);
3842
c7b60b601eb4 7069452: Cleanup NodeFlags
kvn
parents: 1972
diff changeset
206 if (en->is_MachProj() && en->in(0)->is_MachCall()) {
0
a61af66fc99e Initial load
duke
parents:
diff changeset
207 MachCallNode* call = en->in(0)->as_MachCall();
a61af66fc99e Initial load
duke
parents:
diff changeset
208 if (call->cnt() != COUNT_UNKNOWN && call->cnt() <= PROB_UNLIKELY_MAG(4)) {
a61af66fc99e Initial load
duke
parents:
diff changeset
209 // This is true for slow-path stubs like new_{instance,array},
a61af66fc99e Initial load
duke
parents:
diff changeset
210 // slow_arraycopy, complete_monitor_locking, uncommon_trap.
a61af66fc99e Initial load
duke
parents:
diff changeset
211 // The magic number corresponds to the probability of an uncommon_trap,
a61af66fc99e Initial load
duke
parents:
diff changeset
212 // even though it is a count not a probability.
a61af66fc99e Initial load
duke
parents:
diff changeset
213 return true;
a61af66fc99e Initial load
duke
parents:
diff changeset
214 }
a61af66fc99e Initial load
duke
parents:
diff changeset
215 }
a61af66fc99e Initial load
duke
parents:
diff changeset
216
a61af66fc99e Initial load
duke
parents:
diff changeset
217 int op = en->is_Mach() ? en->as_Mach()->ideal_Opcode() : en->Opcode();
a61af66fc99e Initial load
duke
parents:
diff changeset
218 return op == Op_Halt;
a61af66fc99e Initial load
duke
parents:
diff changeset
219 }
a61af66fc99e Initial load
duke
parents:
diff changeset
220
a61af66fc99e Initial load
duke
parents:
diff changeset
221 //------------------------------is_uncommon------------------------------------
a61af66fc99e Initial load
duke
parents:
diff changeset
222 // True if block is low enough frequency or guarded by a test which
a61af66fc99e Initial load
duke
parents:
diff changeset
223 // mostly does not go here.
a61af66fc99e Initial load
duke
parents:
diff changeset
224 bool Block::is_uncommon( Block_Array &bbs ) const {
a61af66fc99e Initial load
duke
parents:
diff changeset
225 // Initial blocks must never be moved, so are never uncommon.
a61af66fc99e Initial load
duke
parents:
diff changeset
226 if (head()->is_Root() || head()->is_Start()) return false;
a61af66fc99e Initial load
duke
parents:
diff changeset
227
a61af66fc99e Initial load
duke
parents:
diff changeset
228 // Check for way-low freq
a61af66fc99e Initial load
duke
parents:
diff changeset
229 if( _freq < BLOCK_FREQUENCY(0.00001f) ) return true;
a61af66fc99e Initial load
duke
parents:
diff changeset
230
a61af66fc99e Initial load
duke
parents:
diff changeset
231 // Look for code shape indicating uncommon_trap or slow path
a61af66fc99e Initial load
duke
parents:
diff changeset
232 if (has_uncommon_code()) return true;
a61af66fc99e Initial load
duke
parents:
diff changeset
233
a61af66fc99e Initial load
duke
parents:
diff changeset
234 const float epsilon = 0.05f;
a61af66fc99e Initial load
duke
parents:
diff changeset
235 const float guard_factor = PROB_UNLIKELY_MAG(4) / (1.f - epsilon);
a61af66fc99e Initial load
duke
parents:
diff changeset
236 uint uncommon_preds = 0;
a61af66fc99e Initial load
duke
parents:
diff changeset
237 uint freq_preds = 0;
a61af66fc99e Initial load
duke
parents:
diff changeset
238 uint uncommon_for_freq_preds = 0;
a61af66fc99e Initial load
duke
parents:
diff changeset
239
a61af66fc99e Initial load
duke
parents:
diff changeset
240 for( uint i=1; i<num_preds(); i++ ) {
a61af66fc99e Initial load
duke
parents:
diff changeset
241 Block* guard = bbs[pred(i)->_idx];
a61af66fc99e Initial load
duke
parents:
diff changeset
242 // Check to see if this block follows its guard 1 time out of 10000
a61af66fc99e Initial load
duke
parents:
diff changeset
243 // or less.
a61af66fc99e Initial load
duke
parents:
diff changeset
244 //
a61af66fc99e Initial load
duke
parents:
diff changeset
245 // See list of magnitude-4 unlikely probabilities in cfgnode.hpp which
a61af66fc99e Initial load
duke
parents:
diff changeset
246 // we intend to be "uncommon", such as slow-path TLE allocation,
a61af66fc99e Initial load
duke
parents:
diff changeset
247 // predicted call failure, and uncommon trap triggers.
a61af66fc99e Initial load
duke
parents:
diff changeset
248 //
a61af66fc99e Initial load
duke
parents:
diff changeset
249 // Use an epsilon value of 5% to allow for variability in frequency
a61af66fc99e Initial load
duke
parents:
diff changeset
250 // predictions and floating point calculations. The net effect is
a61af66fc99e Initial load
duke
parents:
diff changeset
251 // that guard_factor is set to 9500.
a61af66fc99e Initial load
duke
parents:
diff changeset
252 //
a61af66fc99e Initial load
duke
parents:
diff changeset
253 // Ignore low-frequency blocks.
a61af66fc99e Initial load
duke
parents:
diff changeset
254 // The next check is (guard->_freq < 1.e-5 * 9500.).
a61af66fc99e Initial load
duke
parents:
diff changeset
255 if(guard->_freq*BLOCK_FREQUENCY(guard_factor) < BLOCK_FREQUENCY(0.00001f)) {
a61af66fc99e Initial load
duke
parents:
diff changeset
256 uncommon_preds++;
a61af66fc99e Initial load
duke
parents:
diff changeset
257 } else {
a61af66fc99e Initial load
duke
parents:
diff changeset
258 freq_preds++;
a61af66fc99e Initial load
duke
parents:
diff changeset
259 if( _freq < guard->_freq * guard_factor ) {
a61af66fc99e Initial load
duke
parents:
diff changeset
260 uncommon_for_freq_preds++;
a61af66fc99e Initial load
duke
parents:
diff changeset
261 }
a61af66fc99e Initial load
duke
parents:
diff changeset
262 }
a61af66fc99e Initial load
duke
parents:
diff changeset
263 }
a61af66fc99e Initial load
duke
parents:
diff changeset
264 if( num_preds() > 1 &&
a61af66fc99e Initial load
duke
parents:
diff changeset
265 // The block is uncommon if all preds are uncommon or
a61af66fc99e Initial load
duke
parents:
diff changeset
266 (uncommon_preds == (num_preds()-1) ||
a61af66fc99e Initial load
duke
parents:
diff changeset
267 // it is uncommon for all frequent preds.
a61af66fc99e Initial load
duke
parents:
diff changeset
268 uncommon_for_freq_preds == freq_preds) ) {
a61af66fc99e Initial load
duke
parents:
diff changeset
269 return true;
a61af66fc99e Initial load
duke
parents:
diff changeset
270 }
a61af66fc99e Initial load
duke
parents:
diff changeset
271 return false;
a61af66fc99e Initial load
duke
parents:
diff changeset
272 }
a61af66fc99e Initial load
duke
parents:
diff changeset
273
a61af66fc99e Initial load
duke
parents:
diff changeset
274 //------------------------------dump-------------------------------------------
a61af66fc99e Initial load
duke
parents:
diff changeset
275 #ifndef PRODUCT
3851
95134e034042 7063629: use cbcond in C2 generated code on T4
kvn
parents: 3842
diff changeset
276 void Block::dump_bidx(const Block* orig, outputStream* st) const {
95134e034042 7063629: use cbcond in C2 generated code on T4
kvn
parents: 3842
diff changeset
277 if (_pre_order) st->print("B%d",_pre_order);
95134e034042 7063629: use cbcond in C2 generated code on T4
kvn
parents: 3842
diff changeset
278 else st->print("N%d", head()->_idx);
0
a61af66fc99e Initial load
duke
parents:
diff changeset
279
a61af66fc99e Initial load
duke
parents:
diff changeset
280 if (Verbose && orig != this) {
a61af66fc99e Initial load
duke
parents:
diff changeset
281 // Dump the original block's idx
3851
95134e034042 7063629: use cbcond in C2 generated code on T4
kvn
parents: 3842
diff changeset
282 st->print(" (");
95134e034042 7063629: use cbcond in C2 generated code on T4
kvn
parents: 3842
diff changeset
283 orig->dump_bidx(orig, st);
95134e034042 7063629: use cbcond in C2 generated code on T4
kvn
parents: 3842
diff changeset
284 st->print(")");
0
a61af66fc99e Initial load
duke
parents:
diff changeset
285 }
a61af66fc99e Initial load
duke
parents:
diff changeset
286 }
a61af66fc99e Initial load
duke
parents:
diff changeset
287
3851
95134e034042 7063629: use cbcond in C2 generated code on T4
kvn
parents: 3842
diff changeset
288 void Block::dump_pred(const Block_Array *bbs, Block* orig, outputStream* st) const {
0
a61af66fc99e Initial load
duke
parents:
diff changeset
289 if (is_connector()) {
a61af66fc99e Initial load
duke
parents:
diff changeset
290 for (uint i=1; i<num_preds(); i++) {
a61af66fc99e Initial load
duke
parents:
diff changeset
291 Block *p = ((*bbs)[pred(i)->_idx]);
3851
95134e034042 7063629: use cbcond in C2 generated code on T4
kvn
parents: 3842
diff changeset
292 p->dump_pred(bbs, orig, st);
0
a61af66fc99e Initial load
duke
parents:
diff changeset
293 }
a61af66fc99e Initial load
duke
parents:
diff changeset
294 } else {
3851
95134e034042 7063629: use cbcond in C2 generated code on T4
kvn
parents: 3842
diff changeset
295 dump_bidx(orig, st);
95134e034042 7063629: use cbcond in C2 generated code on T4
kvn
parents: 3842
diff changeset
296 st->print(" ");
0
a61af66fc99e Initial load
duke
parents:
diff changeset
297 }
a61af66fc99e Initial load
duke
parents:
diff changeset
298 }
a61af66fc99e Initial load
duke
parents:
diff changeset
299
3851
95134e034042 7063629: use cbcond in C2 generated code on T4
kvn
parents: 3842
diff changeset
300 void Block::dump_head( const Block_Array *bbs, outputStream* st ) const {
0
a61af66fc99e Initial load
duke
parents:
diff changeset
301 // Print the basic block
3851
95134e034042 7063629: use cbcond in C2 generated code on T4
kvn
parents: 3842
diff changeset
302 dump_bidx(this, st);
95134e034042 7063629: use cbcond in C2 generated code on T4
kvn
parents: 3842
diff changeset
303 st->print(": #\t");
0
a61af66fc99e Initial load
duke
parents:
diff changeset
304
a61af66fc99e Initial load
duke
parents:
diff changeset
305 // Print the incoming CFG edges and the outgoing CFG edges
a61af66fc99e Initial load
duke
parents:
diff changeset
306 for( uint i=0; i<_num_succs; i++ ) {
3851
95134e034042 7063629: use cbcond in C2 generated code on T4
kvn
parents: 3842
diff changeset
307 non_connector_successor(i)->dump_bidx(_succs[i], st);
95134e034042 7063629: use cbcond in C2 generated code on T4
kvn
parents: 3842
diff changeset
308 st->print(" ");
0
a61af66fc99e Initial load
duke
parents:
diff changeset
309 }
3851
95134e034042 7063629: use cbcond in C2 generated code on T4
kvn
parents: 3842
diff changeset
310 st->print("<- ");
0
a61af66fc99e Initial load
duke
parents:
diff changeset
311 if( head()->is_block_start() ) {
a61af66fc99e Initial load
duke
parents:
diff changeset
312 for (uint i=1; i<num_preds(); i++) {
a61af66fc99e Initial load
duke
parents:
diff changeset
313 Node *s = pred(i);
a61af66fc99e Initial load
duke
parents:
diff changeset
314 if (bbs) {
a61af66fc99e Initial load
duke
parents:
diff changeset
315 Block *p = (*bbs)[s->_idx];
3851
95134e034042 7063629: use cbcond in C2 generated code on T4
kvn
parents: 3842
diff changeset
316 p->dump_pred(bbs, p, st);
0
a61af66fc99e Initial load
duke
parents:
diff changeset
317 } else {
a61af66fc99e Initial load
duke
parents:
diff changeset
318 while (!s->is_block_start())
a61af66fc99e Initial load
duke
parents:
diff changeset
319 s = s->in(0);
3851
95134e034042 7063629: use cbcond in C2 generated code on T4
kvn
parents: 3842
diff changeset
320 st->print("N%d ", s->_idx );
0
a61af66fc99e Initial load
duke
parents:
diff changeset
321 }
a61af66fc99e Initial load
duke
parents:
diff changeset
322 }
a61af66fc99e Initial load
duke
parents:
diff changeset
323 } else
3851
95134e034042 7063629: use cbcond in C2 generated code on T4
kvn
parents: 3842
diff changeset
324 st->print("BLOCK HEAD IS JUNK ");
0
a61af66fc99e Initial load
duke
parents:
diff changeset
325
a61af66fc99e Initial load
duke
parents:
diff changeset
326 // Print loop, if any
a61af66fc99e Initial load
duke
parents:
diff changeset
327 const Block *bhead = this; // Head of self-loop
a61af66fc99e Initial load
duke
parents:
diff changeset
328 Node *bh = bhead->head();
a61af66fc99e Initial load
duke
parents:
diff changeset
329 if( bbs && bh->is_Loop() && !head()->is_Root() ) {
a61af66fc99e Initial load
duke
parents:
diff changeset
330 LoopNode *loop = bh->as_Loop();
a61af66fc99e Initial load
duke
parents:
diff changeset
331 const Block *bx = (*bbs)[loop->in(LoopNode::LoopBackControl)->_idx];
a61af66fc99e Initial load
duke
parents:
diff changeset
332 while (bx->is_connector()) {
a61af66fc99e Initial load
duke
parents:
diff changeset
333 bx = (*bbs)[bx->pred(1)->_idx];
a61af66fc99e Initial load
duke
parents:
diff changeset
334 }
3851
95134e034042 7063629: use cbcond in C2 generated code on T4
kvn
parents: 3842
diff changeset
335 st->print("\tLoop: B%d-B%d ", bhead->_pre_order, bx->_pre_order);
0
a61af66fc99e Initial load
duke
parents:
diff changeset
336 // Dump any loop-specific bits, especially for CountedLoops.
3851
95134e034042 7063629: use cbcond in C2 generated code on T4
kvn
parents: 3842
diff changeset
337 loop->dump_spec(st);
418
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
338 } else if (has_loop_alignment()) {
3851
95134e034042 7063629: use cbcond in C2 generated code on T4
kvn
parents: 3842
diff changeset
339 st->print(" top-of-loop");
0
a61af66fc99e Initial load
duke
parents:
diff changeset
340 }
3851
95134e034042 7063629: use cbcond in C2 generated code on T4
kvn
parents: 3842
diff changeset
341 st->print(" Freq: %g",_freq);
0
a61af66fc99e Initial load
duke
parents:
diff changeset
342 if( Verbose || WizardMode ) {
3851
95134e034042 7063629: use cbcond in C2 generated code on T4
kvn
parents: 3842
diff changeset
343 st->print(" IDom: %d/#%d", _idom ? _idom->_pre_order : 0, _dom_depth);
95134e034042 7063629: use cbcond in C2 generated code on T4
kvn
parents: 3842
diff changeset
344 st->print(" RegPressure: %d",_reg_pressure);
95134e034042 7063629: use cbcond in C2 generated code on T4
kvn
parents: 3842
diff changeset
345 st->print(" IHRP Index: %d",_ihrp_index);
95134e034042 7063629: use cbcond in C2 generated code on T4
kvn
parents: 3842
diff changeset
346 st->print(" FRegPressure: %d",_freg_pressure);
95134e034042 7063629: use cbcond in C2 generated code on T4
kvn
parents: 3842
diff changeset
347 st->print(" FHRP Index: %d",_fhrp_index);
0
a61af66fc99e Initial load
duke
parents:
diff changeset
348 }
3851
95134e034042 7063629: use cbcond in C2 generated code on T4
kvn
parents: 3842
diff changeset
349 st->print_cr("");
0
a61af66fc99e Initial load
duke
parents:
diff changeset
350 }
a61af66fc99e Initial load
duke
parents:
diff changeset
351
3851
95134e034042 7063629: use cbcond in C2 generated code on T4
kvn
parents: 3842
diff changeset
352 void Block::dump() const { dump(NULL); }
0
a61af66fc99e Initial load
duke
parents:
diff changeset
353
a61af66fc99e Initial load
duke
parents:
diff changeset
354 void Block::dump( const Block_Array *bbs ) const {
a61af66fc99e Initial load
duke
parents:
diff changeset
355 dump_head(bbs);
a61af66fc99e Initial load
duke
parents:
diff changeset
356 uint cnt = _nodes.size();
a61af66fc99e Initial load
duke
parents:
diff changeset
357 for( uint i=0; i<cnt; i++ )
a61af66fc99e Initial load
duke
parents:
diff changeset
358 _nodes[i]->dump();
a61af66fc99e Initial load
duke
parents:
diff changeset
359 tty->print("\n");
a61af66fc99e Initial load
duke
parents:
diff changeset
360 }
a61af66fc99e Initial load
duke
parents:
diff changeset
361 #endif
a61af66fc99e Initial load
duke
parents:
diff changeset
362
a61af66fc99e Initial load
duke
parents:
diff changeset
363 //=============================================================================
a61af66fc99e Initial load
duke
parents:
diff changeset
364 //------------------------------PhaseCFG---------------------------------------
a61af66fc99e Initial load
duke
parents:
diff changeset
365 PhaseCFG::PhaseCFG( Arena *a, RootNode *r, Matcher &m ) :
a61af66fc99e Initial load
duke
parents:
diff changeset
366 Phase(CFG),
a61af66fc99e Initial load
duke
parents:
diff changeset
367 _bbs(a),
1685
0e35fa8ebccd 6973963: SEGV in ciBlock::start_bci() with EA
kvn
parents: 1552
diff changeset
368 _root(r),
0e35fa8ebccd 6973963: SEGV in ciBlock::start_bci() with EA
kvn
parents: 1552
diff changeset
369 _node_latency(NULL)
0
a61af66fc99e Initial load
duke
parents:
diff changeset
370 #ifndef PRODUCT
a61af66fc99e Initial load
duke
parents:
diff changeset
371 , _trace_opto_pipelining(TraceOptoPipelining || C->method_has_option("TraceOptoPipelining"))
a61af66fc99e Initial load
duke
parents:
diff changeset
372 #endif
833
acba6af809c8 6840775: Multiple JVM crashes seen with 1.6.0_10 through 1.6.0_14
kvn
parents: 628
diff changeset
373 #ifdef ASSERT
acba6af809c8 6840775: Multiple JVM crashes seen with 1.6.0_10 through 1.6.0_14
kvn
parents: 628
diff changeset
374 , _raw_oops(a)
acba6af809c8 6840775: Multiple JVM crashes seen with 1.6.0_10 through 1.6.0_14
kvn
parents: 628
diff changeset
375 #endif
0
a61af66fc99e Initial load
duke
parents:
diff changeset
376 {
a61af66fc99e Initial load
duke
parents:
diff changeset
377 ResourceMark rm;
a61af66fc99e Initial load
duke
parents:
diff changeset
378 // I'll need a few machine-specific GotoNodes. Make an Ideal GotoNode,
a61af66fc99e Initial load
duke
parents:
diff changeset
379 // then Match it into a machine-specific Node. Then clone the machine
a61af66fc99e Initial load
duke
parents:
diff changeset
380 // Node on demand.
a61af66fc99e Initial load
duke
parents:
diff changeset
381 Node *x = new (C, 1) GotoNode(NULL);
a61af66fc99e Initial load
duke
parents:
diff changeset
382 x->init_req(0, x);
a61af66fc99e Initial load
duke
parents:
diff changeset
383 _goto = m.match_tree(x);
a61af66fc99e Initial load
duke
parents:
diff changeset
384 assert(_goto != NULL, "");
a61af66fc99e Initial load
duke
parents:
diff changeset
385 _goto->set_req(0,_goto);
a61af66fc99e Initial load
duke
parents:
diff changeset
386
a61af66fc99e Initial load
duke
parents:
diff changeset
387 // Build the CFG in Reverse Post Order
a61af66fc99e Initial load
duke
parents:
diff changeset
388 _num_blocks = build_cfg();
a61af66fc99e Initial load
duke
parents:
diff changeset
389 _broot = _bbs[_root->_idx];
a61af66fc99e Initial load
duke
parents:
diff changeset
390 }
a61af66fc99e Initial load
duke
parents:
diff changeset
391
a61af66fc99e Initial load
duke
parents:
diff changeset
392 //------------------------------build_cfg--------------------------------------
a61af66fc99e Initial load
duke
parents:
diff changeset
393 // Build a proper looking CFG. Make every block begin with either a StartNode
a61af66fc99e Initial load
duke
parents:
diff changeset
394 // or a RegionNode. Make every block end with either a Goto, If or Return.
a61af66fc99e Initial load
duke
parents:
diff changeset
395 // The RootNode both starts and ends it's own block. Do this with a recursive
a61af66fc99e Initial load
duke
parents:
diff changeset
396 // backwards walk over the control edges.
a61af66fc99e Initial load
duke
parents:
diff changeset
397 uint PhaseCFG::build_cfg() {
a61af66fc99e Initial load
duke
parents:
diff changeset
398 Arena *a = Thread::current()->resource_area();
a61af66fc99e Initial load
duke
parents:
diff changeset
399 VectorSet visited(a);
a61af66fc99e Initial load
duke
parents:
diff changeset
400
a61af66fc99e Initial load
duke
parents:
diff changeset
401 // Allocate stack with enough space to avoid frequent realloc
a61af66fc99e Initial load
duke
parents:
diff changeset
402 Node_Stack nstack(a, C->unique() >> 1);
a61af66fc99e Initial load
duke
parents:
diff changeset
403 nstack.push(_root, 0);
a61af66fc99e Initial load
duke
parents:
diff changeset
404 uint sum = 0; // Counter for blocks
a61af66fc99e Initial load
duke
parents:
diff changeset
405
a61af66fc99e Initial load
duke
parents:
diff changeset
406 while (nstack.is_nonempty()) {
a61af66fc99e Initial load
duke
parents:
diff changeset
407 // node and in's index from stack's top
a61af66fc99e Initial load
duke
parents:
diff changeset
408 // 'np' is _root (see above) or RegionNode, StartNode: we push on stack
a61af66fc99e Initial load
duke
parents:
diff changeset
409 // only nodes which point to the start of basic block (see below).
a61af66fc99e Initial load
duke
parents:
diff changeset
410 Node *np = nstack.node();
a61af66fc99e Initial load
duke
parents:
diff changeset
411 // idx > 0, except for the first node (_root) pushed on stack
a61af66fc99e Initial load
duke
parents:
diff changeset
412 // at the beginning when idx == 0.
a61af66fc99e Initial load
duke
parents:
diff changeset
413 // We will use the condition (idx == 0) later to end the build.
a61af66fc99e Initial load
duke
parents:
diff changeset
414 uint idx = nstack.index();
a61af66fc99e Initial load
duke
parents:
diff changeset
415 Node *proj = np->in(idx);
a61af66fc99e Initial load
duke
parents:
diff changeset
416 const Node *x = proj->is_block_proj();
a61af66fc99e Initial load
duke
parents:
diff changeset
417 // Does the block end with a proper block-ending Node? One of Return,
a61af66fc99e Initial load
duke
parents:
diff changeset
418 // If or Goto? (This check should be done for visited nodes also).
a61af66fc99e Initial load
duke
parents:
diff changeset
419 if (x == NULL) { // Does not end right...
a61af66fc99e Initial load
duke
parents:
diff changeset
420 Node *g = _goto->clone(); // Force it to end in a Goto
a61af66fc99e Initial load
duke
parents:
diff changeset
421 g->set_req(0, proj);
a61af66fc99e Initial load
duke
parents:
diff changeset
422 np->set_req(idx, g);
a61af66fc99e Initial load
duke
parents:
diff changeset
423 x = proj = g;
a61af66fc99e Initial load
duke
parents:
diff changeset
424 }
a61af66fc99e Initial load
duke
parents:
diff changeset
425 if (!visited.test_set(x->_idx)) { // Visit this block once
a61af66fc99e Initial load
duke
parents:
diff changeset
426 // Skip any control-pinned middle'in stuff
a61af66fc99e Initial load
duke
parents:
diff changeset
427 Node *p = proj;
a61af66fc99e Initial load
duke
parents:
diff changeset
428 do {
a61af66fc99e Initial load
duke
parents:
diff changeset
429 proj = p; // Update pointer to last Control
a61af66fc99e Initial load
duke
parents:
diff changeset
430 p = p->in(0); // Move control forward
a61af66fc99e Initial load
duke
parents:
diff changeset
431 } while( !p->is_block_proj() &&
a61af66fc99e Initial load
duke
parents:
diff changeset
432 !p->is_block_start() );
a61af66fc99e Initial load
duke
parents:
diff changeset
433 // Make the block begin with one of Region or StartNode.
a61af66fc99e Initial load
duke
parents:
diff changeset
434 if( !p->is_block_start() ) {
a61af66fc99e Initial load
duke
parents:
diff changeset
435 RegionNode *r = new (C, 2) RegionNode( 2 );
a61af66fc99e Initial load
duke
parents:
diff changeset
436 r->init_req(1, p); // Insert RegionNode in the way
a61af66fc99e Initial load
duke
parents:
diff changeset
437 proj->set_req(0, r); // Insert RegionNode in the way
a61af66fc99e Initial load
duke
parents:
diff changeset
438 p = r;
a61af66fc99e Initial load
duke
parents:
diff changeset
439 }
a61af66fc99e Initial load
duke
parents:
diff changeset
440 // 'p' now points to the start of this basic block
a61af66fc99e Initial load
duke
parents:
diff changeset
441
a61af66fc99e Initial load
duke
parents:
diff changeset
442 // Put self in array of basic blocks
a61af66fc99e Initial load
duke
parents:
diff changeset
443 Block *bb = new (_bbs._arena) Block(_bbs._arena,p);
a61af66fc99e Initial load
duke
parents:
diff changeset
444 _bbs.map(p->_idx,bb);
a61af66fc99e Initial load
duke
parents:
diff changeset
445 _bbs.map(x->_idx,bb);
3851
95134e034042 7063629: use cbcond in C2 generated code on T4
kvn
parents: 3842
diff changeset
446 if( x != p ) { // Only for root is x == p
0
a61af66fc99e Initial load
duke
parents:
diff changeset
447 bb->_nodes.push((Node*)x);
3851
95134e034042 7063629: use cbcond in C2 generated code on T4
kvn
parents: 3842
diff changeset
448 }
0
a61af66fc99e Initial load
duke
parents:
diff changeset
449 // Now handle predecessors
a61af66fc99e Initial load
duke
parents:
diff changeset
450 ++sum; // Count 1 for self block
a61af66fc99e Initial load
duke
parents:
diff changeset
451 uint cnt = bb->num_preds();
a61af66fc99e Initial load
duke
parents:
diff changeset
452 for (int i = (cnt - 1); i > 0; i-- ) { // For all predecessors
a61af66fc99e Initial load
duke
parents:
diff changeset
453 Node *prevproj = p->in(i); // Get prior input
a61af66fc99e Initial load
duke
parents:
diff changeset
454 assert( !prevproj->is_Con(), "dead input not removed" );
a61af66fc99e Initial load
duke
parents:
diff changeset
455 // Check to see if p->in(i) is a "control-dependent" CFG edge -
a61af66fc99e Initial load
duke
parents:
diff changeset
456 // i.e., it splits at the source (via an IF or SWITCH) and merges
a61af66fc99e Initial load
duke
parents:
diff changeset
457 // at the destination (via a many-input Region).
a61af66fc99e Initial load
duke
parents:
diff changeset
458 // This breaks critical edges. The RegionNode to start the block
a61af66fc99e Initial load
duke
parents:
diff changeset
459 // will be added when <p,i> is pulled off the node stack
a61af66fc99e Initial load
duke
parents:
diff changeset
460 if ( cnt > 2 ) { // Merging many things?
a61af66fc99e Initial load
duke
parents:
diff changeset
461 assert( prevproj== bb->pred(i),"");
a61af66fc99e Initial load
duke
parents:
diff changeset
462 if(prevproj->is_block_proj() != prevproj) { // Control-dependent edge?
a61af66fc99e Initial load
duke
parents:
diff changeset
463 // Force a block on the control-dependent edge
a61af66fc99e Initial load
duke
parents:
diff changeset
464 Node *g = _goto->clone(); // Force it to end in a Goto
a61af66fc99e Initial load
duke
parents:
diff changeset
465 g->set_req(0,prevproj);
a61af66fc99e Initial load
duke
parents:
diff changeset
466 p->set_req(i,g);
a61af66fc99e Initial load
duke
parents:
diff changeset
467 }
a61af66fc99e Initial load
duke
parents:
diff changeset
468 }
a61af66fc99e Initial load
duke
parents:
diff changeset
469 nstack.push(p, i); // 'p' is RegionNode or StartNode
a61af66fc99e Initial load
duke
parents:
diff changeset
470 }
a61af66fc99e Initial load
duke
parents:
diff changeset
471 } else { // Post-processing visited nodes
a61af66fc99e Initial load
duke
parents:
diff changeset
472 nstack.pop(); // remove node from stack
a61af66fc99e Initial load
duke
parents:
diff changeset
473 // Check if it the fist node pushed on stack at the beginning.
a61af66fc99e Initial load
duke
parents:
diff changeset
474 if (idx == 0) break; // end of the build
a61af66fc99e Initial load
duke
parents:
diff changeset
475 // Find predecessor basic block
a61af66fc99e Initial load
duke
parents:
diff changeset
476 Block *pb = _bbs[x->_idx];
a61af66fc99e Initial load
duke
parents:
diff changeset
477 // Insert into nodes array, if not already there
a61af66fc99e Initial load
duke
parents:
diff changeset
478 if( !_bbs.lookup(proj->_idx) ) {
a61af66fc99e Initial load
duke
parents:
diff changeset
479 assert( x != proj, "" );
a61af66fc99e Initial load
duke
parents:
diff changeset
480 // Map basic block of projection
a61af66fc99e Initial load
duke
parents:
diff changeset
481 _bbs.map(proj->_idx,pb);
a61af66fc99e Initial load
duke
parents:
diff changeset
482 pb->_nodes.push(proj);
a61af66fc99e Initial load
duke
parents:
diff changeset
483 }
a61af66fc99e Initial load
duke
parents:
diff changeset
484 // Insert self as a child of my predecessor block
a61af66fc99e Initial load
duke
parents:
diff changeset
485 pb->_succs.map(pb->_num_succs++, _bbs[np->_idx]);
a61af66fc99e Initial load
duke
parents:
diff changeset
486 assert( pb->_nodes[ pb->_nodes.size() - pb->_num_succs ]->is_block_proj(),
a61af66fc99e Initial load
duke
parents:
diff changeset
487 "too many control users, not a CFG?" );
a61af66fc99e Initial load
duke
parents:
diff changeset
488 }
a61af66fc99e Initial load
duke
parents:
diff changeset
489 }
a61af66fc99e Initial load
duke
parents:
diff changeset
490 // Return number of basic blocks for all children and self
a61af66fc99e Initial load
duke
parents:
diff changeset
491 return sum;
a61af66fc99e Initial load
duke
parents:
diff changeset
492 }
a61af66fc99e Initial load
duke
parents:
diff changeset
493
a61af66fc99e Initial load
duke
parents:
diff changeset
494 //------------------------------insert_goto_at---------------------------------
a61af66fc99e Initial load
duke
parents:
diff changeset
495 // Inserts a goto & corresponding basic block between
a61af66fc99e Initial load
duke
parents:
diff changeset
496 // block[block_no] and its succ_no'th successor block
a61af66fc99e Initial load
duke
parents:
diff changeset
497 void PhaseCFG::insert_goto_at(uint block_no, uint succ_no) {
a61af66fc99e Initial load
duke
parents:
diff changeset
498 // get block with block_no
a61af66fc99e Initial load
duke
parents:
diff changeset
499 assert(block_no < _num_blocks, "illegal block number");
a61af66fc99e Initial load
duke
parents:
diff changeset
500 Block* in = _blocks[block_no];
a61af66fc99e Initial load
duke
parents:
diff changeset
501 // get successor block succ_no
a61af66fc99e Initial load
duke
parents:
diff changeset
502 assert(succ_no < in->_num_succs, "illegal successor number");
a61af66fc99e Initial load
duke
parents:
diff changeset
503 Block* out = in->_succs[succ_no];
308
756b58154237 6611837: block frequency is zero
rasbold
parents: 0
diff changeset
504 // Compute frequency of the new block. Do this before inserting
756b58154237 6611837: block frequency is zero
rasbold
parents: 0
diff changeset
505 // new block in case succ_prob() needs to infer the probability from
756b58154237 6611837: block frequency is zero
rasbold
parents: 0
diff changeset
506 // surrounding blocks.
756b58154237 6611837: block frequency is zero
rasbold
parents: 0
diff changeset
507 float freq = in->_freq * in->succ_prob(succ_no);
0
a61af66fc99e Initial load
duke
parents:
diff changeset
508 // get ProjNode corresponding to the succ_no'th successor of the in block
a61af66fc99e Initial load
duke
parents:
diff changeset
509 ProjNode* proj = in->_nodes[in->_nodes.size() - in->_num_succs + succ_no]->as_Proj();
a61af66fc99e Initial load
duke
parents:
diff changeset
510 // create region for basic block
a61af66fc99e Initial load
duke
parents:
diff changeset
511 RegionNode* region = new (C, 2) RegionNode(2);
a61af66fc99e Initial load
duke
parents:
diff changeset
512 region->init_req(1, proj);
a61af66fc99e Initial load
duke
parents:
diff changeset
513 // setup corresponding basic block
a61af66fc99e Initial load
duke
parents:
diff changeset
514 Block* block = new (_bbs._arena) Block(_bbs._arena, region);
a61af66fc99e Initial load
duke
parents:
diff changeset
515 _bbs.map(region->_idx, block);
a61af66fc99e Initial load
duke
parents:
diff changeset
516 C->regalloc()->set_bad(region->_idx);
a61af66fc99e Initial load
duke
parents:
diff changeset
517 // add a goto node
a61af66fc99e Initial load
duke
parents:
diff changeset
518 Node* gto = _goto->clone(); // get a new goto node
a61af66fc99e Initial load
duke
parents:
diff changeset
519 gto->set_req(0, region);
a61af66fc99e Initial load
duke
parents:
diff changeset
520 // add it to the basic block
a61af66fc99e Initial load
duke
parents:
diff changeset
521 block->_nodes.push(gto);
a61af66fc99e Initial load
duke
parents:
diff changeset
522 _bbs.map(gto->_idx, block);
a61af66fc99e Initial load
duke
parents:
diff changeset
523 C->regalloc()->set_bad(gto->_idx);
a61af66fc99e Initial load
duke
parents:
diff changeset
524 // hook up successor block
a61af66fc99e Initial load
duke
parents:
diff changeset
525 block->_succs.map(block->_num_succs++, out);
a61af66fc99e Initial load
duke
parents:
diff changeset
526 // remap successor's predecessors if necessary
a61af66fc99e Initial load
duke
parents:
diff changeset
527 for (uint i = 1; i < out->num_preds(); i++) {
a61af66fc99e Initial load
duke
parents:
diff changeset
528 if (out->pred(i) == proj) out->head()->set_req(i, gto);
a61af66fc99e Initial load
duke
parents:
diff changeset
529 }
a61af66fc99e Initial load
duke
parents:
diff changeset
530 // remap predecessor's successor to new block
a61af66fc99e Initial load
duke
parents:
diff changeset
531 in->_succs.map(succ_no, block);
308
756b58154237 6611837: block frequency is zero
rasbold
parents: 0
diff changeset
532 // Set the frequency of the new block
756b58154237 6611837: block frequency is zero
rasbold
parents: 0
diff changeset
533 block->_freq = freq;
0
a61af66fc99e Initial load
duke
parents:
diff changeset
534 // add new basic block to basic block list
a61af66fc99e Initial load
duke
parents:
diff changeset
535 _blocks.insert(block_no + 1, block);
a61af66fc99e Initial load
duke
parents:
diff changeset
536 _num_blocks++;
a61af66fc99e Initial load
duke
parents:
diff changeset
537 }
a61af66fc99e Initial load
duke
parents:
diff changeset
538
a61af66fc99e Initial load
duke
parents:
diff changeset
539 //------------------------------no_flip_branch---------------------------------
a61af66fc99e Initial load
duke
parents:
diff changeset
540 // Does this block end in a multiway branch that cannot have the default case
a61af66fc99e Initial load
duke
parents:
diff changeset
541 // flipped for another case?
a61af66fc99e Initial load
duke
parents:
diff changeset
542 static bool no_flip_branch( Block *b ) {
a61af66fc99e Initial load
duke
parents:
diff changeset
543 int branch_idx = b->_nodes.size() - b->_num_succs-1;
a61af66fc99e Initial load
duke
parents:
diff changeset
544 if( branch_idx < 1 ) return false;
a61af66fc99e Initial load
duke
parents:
diff changeset
545 Node *bra = b->_nodes[branch_idx];
418
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
546 if( bra->is_Catch() )
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
547 return true;
0
a61af66fc99e Initial load
duke
parents:
diff changeset
548 if( bra->is_Mach() ) {
418
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
549 if( bra->is_MachNullCheck() )
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
550 return true;
0
a61af66fc99e Initial load
duke
parents:
diff changeset
551 int iop = bra->as_Mach()->ideal_Opcode();
a61af66fc99e Initial load
duke
parents:
diff changeset
552 if( iop == Op_FastLock || iop == Op_FastUnlock )
a61af66fc99e Initial load
duke
parents:
diff changeset
553 return true;
a61af66fc99e Initial load
duke
parents:
diff changeset
554 }
a61af66fc99e Initial load
duke
parents:
diff changeset
555 return false;
a61af66fc99e Initial load
duke
parents:
diff changeset
556 }
a61af66fc99e Initial load
duke
parents:
diff changeset
557
a61af66fc99e Initial load
duke
parents:
diff changeset
558 //------------------------------convert_NeverBranch_to_Goto--------------------
a61af66fc99e Initial load
duke
parents:
diff changeset
559 // Check for NeverBranch at block end. This needs to become a GOTO to the
a61af66fc99e Initial load
duke
parents:
diff changeset
560 // true target. NeverBranch are treated as a conditional branch that always
a61af66fc99e Initial load
duke
parents:
diff changeset
561 // goes the same direction for most of the optimizer and are used to give a
a61af66fc99e Initial load
duke
parents:
diff changeset
562 // fake exit path to infinite loops. At this late stage they need to turn
a61af66fc99e Initial load
duke
parents:
diff changeset
563 // into Goto's so that when you enter the infinite loop you indeed hang.
a61af66fc99e Initial load
duke
parents:
diff changeset
564 void PhaseCFG::convert_NeverBranch_to_Goto(Block *b) {
a61af66fc99e Initial load
duke
parents:
diff changeset
565 // Find true target
a61af66fc99e Initial load
duke
parents:
diff changeset
566 int end_idx = b->end_idx();
a61af66fc99e Initial load
duke
parents:
diff changeset
567 int idx = b->_nodes[end_idx+1]->as_Proj()->_con;
a61af66fc99e Initial load
duke
parents:
diff changeset
568 Block *succ = b->_succs[idx];
a61af66fc99e Initial load
duke
parents:
diff changeset
569 Node* gto = _goto->clone(); // get a new goto node
a61af66fc99e Initial load
duke
parents:
diff changeset
570 gto->set_req(0, b->head());
a61af66fc99e Initial load
duke
parents:
diff changeset
571 Node *bp = b->_nodes[end_idx];
a61af66fc99e Initial load
duke
parents:
diff changeset
572 b->_nodes.map(end_idx,gto); // Slam over NeverBranch
a61af66fc99e Initial load
duke
parents:
diff changeset
573 _bbs.map(gto->_idx, b);
a61af66fc99e Initial load
duke
parents:
diff changeset
574 C->regalloc()->set_bad(gto->_idx);
a61af66fc99e Initial load
duke
parents:
diff changeset
575 b->_nodes.pop(); // Yank projections
a61af66fc99e Initial load
duke
parents:
diff changeset
576 b->_nodes.pop(); // Yank projections
a61af66fc99e Initial load
duke
parents:
diff changeset
577 b->_succs.map(0,succ); // Map only successor
a61af66fc99e Initial load
duke
parents:
diff changeset
578 b->_num_succs = 1;
a61af66fc99e Initial load
duke
parents:
diff changeset
579 // remap successor's predecessors if necessary
a61af66fc99e Initial load
duke
parents:
diff changeset
580 uint j;
a61af66fc99e Initial load
duke
parents:
diff changeset
581 for( j = 1; j < succ->num_preds(); j++)
a61af66fc99e Initial load
duke
parents:
diff changeset
582 if( succ->pred(j)->in(0) == bp )
a61af66fc99e Initial load
duke
parents:
diff changeset
583 succ->head()->set_req(j, gto);
a61af66fc99e Initial load
duke
parents:
diff changeset
584 // Kill alternate exit path
a61af66fc99e Initial load
duke
parents:
diff changeset
585 Block *dead = b->_succs[1-idx];
a61af66fc99e Initial load
duke
parents:
diff changeset
586 for( j = 1; j < dead->num_preds(); j++)
a61af66fc99e Initial load
duke
parents:
diff changeset
587 if( dead->pred(j)->in(0) == bp )
a61af66fc99e Initial load
duke
parents:
diff changeset
588 break;
a61af66fc99e Initial load
duke
parents:
diff changeset
589 // Scan through block, yanking dead path from
a61af66fc99e Initial load
duke
parents:
diff changeset
590 // all regions and phis.
a61af66fc99e Initial load
duke
parents:
diff changeset
591 dead->head()->del_req(j);
a61af66fc99e Initial load
duke
parents:
diff changeset
592 for( int k = 1; dead->_nodes[k]->is_Phi(); k++ )
a61af66fc99e Initial load
duke
parents:
diff changeset
593 dead->_nodes[k]->del_req(j);
a61af66fc99e Initial load
duke
parents:
diff changeset
594 }
a61af66fc99e Initial load
duke
parents:
diff changeset
595
418
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
596 //------------------------------move_to_next-----------------------------------
0
a61af66fc99e Initial load
duke
parents:
diff changeset
597 // Helper function to move block bx to the slot following b_index. Return
a61af66fc99e Initial load
duke
parents:
diff changeset
598 // true if the move is successful, otherwise false
418
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
599 bool PhaseCFG::move_to_next(Block* bx, uint b_index) {
0
a61af66fc99e Initial load
duke
parents:
diff changeset
600 if (bx == NULL) return false;
a61af66fc99e Initial load
duke
parents:
diff changeset
601
a61af66fc99e Initial load
duke
parents:
diff changeset
602 // Return false if bx is already scheduled.
a61af66fc99e Initial load
duke
parents:
diff changeset
603 uint bx_index = bx->_pre_order;
a61af66fc99e Initial load
duke
parents:
diff changeset
604 if ((bx_index <= b_index) && (_blocks[bx_index] == bx)) {
a61af66fc99e Initial load
duke
parents:
diff changeset
605 return false;
a61af66fc99e Initial load
duke
parents:
diff changeset
606 }
a61af66fc99e Initial load
duke
parents:
diff changeset
607
a61af66fc99e Initial load
duke
parents:
diff changeset
608 // Find the current index of block bx on the block list
a61af66fc99e Initial load
duke
parents:
diff changeset
609 bx_index = b_index + 1;
a61af66fc99e Initial load
duke
parents:
diff changeset
610 while( bx_index < _num_blocks && _blocks[bx_index] != bx ) bx_index++;
a61af66fc99e Initial load
duke
parents:
diff changeset
611 assert(_blocks[bx_index] == bx, "block not found");
a61af66fc99e Initial load
duke
parents:
diff changeset
612
a61af66fc99e Initial load
duke
parents:
diff changeset
613 // If the previous block conditionally falls into bx, return false,
a61af66fc99e Initial load
duke
parents:
diff changeset
614 // because moving bx will create an extra jump.
a61af66fc99e Initial load
duke
parents:
diff changeset
615 for(uint k = 1; k < bx->num_preds(); k++ ) {
a61af66fc99e Initial load
duke
parents:
diff changeset
616 Block* pred = _bbs[bx->pred(k)->_idx];
a61af66fc99e Initial load
duke
parents:
diff changeset
617 if (pred == _blocks[bx_index-1]) {
a61af66fc99e Initial load
duke
parents:
diff changeset
618 if (pred->_num_succs != 1) {
a61af66fc99e Initial load
duke
parents:
diff changeset
619 return false;
a61af66fc99e Initial load
duke
parents:
diff changeset
620 }
a61af66fc99e Initial load
duke
parents:
diff changeset
621 }
a61af66fc99e Initial load
duke
parents:
diff changeset
622 }
a61af66fc99e Initial load
duke
parents:
diff changeset
623
a61af66fc99e Initial load
duke
parents:
diff changeset
624 // Reinsert bx just past block 'b'
a61af66fc99e Initial load
duke
parents:
diff changeset
625 _blocks.remove(bx_index);
a61af66fc99e Initial load
duke
parents:
diff changeset
626 _blocks.insert(b_index + 1, bx);
a61af66fc99e Initial load
duke
parents:
diff changeset
627 return true;
a61af66fc99e Initial load
duke
parents:
diff changeset
628 }
a61af66fc99e Initial load
duke
parents:
diff changeset
629
418
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
630 //------------------------------move_to_end------------------------------------
0
a61af66fc99e Initial load
duke
parents:
diff changeset
631 // Move empty and uncommon blocks to the end.
418
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
632 void PhaseCFG::move_to_end(Block *b, uint i) {
0
a61af66fc99e Initial load
duke
parents:
diff changeset
633 int e = b->is_Empty();
a61af66fc99e Initial load
duke
parents:
diff changeset
634 if (e != Block::not_empty) {
a61af66fc99e Initial load
duke
parents:
diff changeset
635 if (e == Block::empty_with_goto) {
a61af66fc99e Initial load
duke
parents:
diff changeset
636 // Remove the goto, but leave the block.
a61af66fc99e Initial load
duke
parents:
diff changeset
637 b->_nodes.pop();
a61af66fc99e Initial load
duke
parents:
diff changeset
638 }
a61af66fc99e Initial load
duke
parents:
diff changeset
639 // Mark this block as a connector block, which will cause it to be
a61af66fc99e Initial load
duke
parents:
diff changeset
640 // ignored in certain functions such as non_connector_successor().
a61af66fc99e Initial load
duke
parents:
diff changeset
641 b->set_connector();
a61af66fc99e Initial load
duke
parents:
diff changeset
642 }
a61af66fc99e Initial load
duke
parents:
diff changeset
643 // Move the empty block to the end, and don't recheck.
a61af66fc99e Initial load
duke
parents:
diff changeset
644 _blocks.remove(i);
a61af66fc99e Initial load
duke
parents:
diff changeset
645 _blocks.push(b);
a61af66fc99e Initial load
duke
parents:
diff changeset
646 }
a61af66fc99e Initial load
duke
parents:
diff changeset
647
418
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
648 //---------------------------set_loop_alignment--------------------------------
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
649 // Set loop alignment for every block
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
650 void PhaseCFG::set_loop_alignment() {
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
651 uint last = _num_blocks;
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
652 assert( _blocks[0] == _broot, "" );
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
653
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
654 for (uint i = 1; i < last; i++ ) {
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
655 Block *b = _blocks[i];
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
656 if (b->head()->is_Loop()) {
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
657 b->set_loop_alignment(b);
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
658 }
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
659 }
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
660 }
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
661
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
662 //-----------------------------remove_empty------------------------------------
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
663 // Make empty basic blocks to be "connector" blocks, Move uncommon blocks
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
664 // to the end.
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
665 void PhaseCFG::remove_empty() {
0
a61af66fc99e Initial load
duke
parents:
diff changeset
666 // Move uncommon blocks to the end
a61af66fc99e Initial load
duke
parents:
diff changeset
667 uint last = _num_blocks;
a61af66fc99e Initial load
duke
parents:
diff changeset
668 assert( _blocks[0] == _broot, "" );
418
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
669
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
670 for (uint i = 1; i < last; i++) {
0
a61af66fc99e Initial load
duke
parents:
diff changeset
671 Block *b = _blocks[i];
418
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
672 if (b->is_connector()) break;
0
a61af66fc99e Initial load
duke
parents:
diff changeset
673
a61af66fc99e Initial load
duke
parents:
diff changeset
674 // Check for NeverBranch at block end. This needs to become a GOTO to the
a61af66fc99e Initial load
duke
parents:
diff changeset
675 // true target. NeverBranch are treated as a conditional branch that
a61af66fc99e Initial load
duke
parents:
diff changeset
676 // always goes the same direction for most of the optimizer and are used
a61af66fc99e Initial load
duke
parents:
diff changeset
677 // to give a fake exit path to infinite loops. At this late stage they
a61af66fc99e Initial load
duke
parents:
diff changeset
678 // need to turn into Goto's so that when you enter the infinite loop you
a61af66fc99e Initial load
duke
parents:
diff changeset
679 // indeed hang.
a61af66fc99e Initial load
duke
parents:
diff changeset
680 if( b->_nodes[b->end_idx()]->Opcode() == Op_NeverBranch )
a61af66fc99e Initial load
duke
parents:
diff changeset
681 convert_NeverBranch_to_Goto(b);
a61af66fc99e Initial load
duke
parents:
diff changeset
682
a61af66fc99e Initial load
duke
parents:
diff changeset
683 // Look for uncommon blocks and move to end.
418
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
684 if (!C->do_freq_based_layout()) {
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
685 if( b->is_uncommon(_bbs) ) {
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
686 move_to_end(b, i);
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
687 last--; // No longer check for being uncommon!
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
688 if( no_flip_branch(b) ) { // Fall-thru case must follow?
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
689 b = _blocks[i]; // Find the fall-thru block
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
690 move_to_end(b, i);
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
691 last--;
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
692 }
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
693 i--; // backup block counter post-increment
0
a61af66fc99e Initial load
duke
parents:
diff changeset
694 }
a61af66fc99e Initial load
duke
parents:
diff changeset
695 }
a61af66fc99e Initial load
duke
parents:
diff changeset
696 }
a61af66fc99e Initial load
duke
parents:
diff changeset
697
418
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
698 // Move empty blocks to the end
0
a61af66fc99e Initial load
duke
parents:
diff changeset
699 last = _num_blocks;
418
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
700 for (uint i = 1; i < last; i++) {
0
a61af66fc99e Initial load
duke
parents:
diff changeset
701 Block *b = _blocks[i];
418
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
702 if (b->is_Empty() != Block::not_empty) {
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
703 move_to_end(b, i);
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
704 last--;
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
705 i--;
0
a61af66fc99e Initial load
duke
parents:
diff changeset
706 }
a61af66fc99e Initial load
duke
parents:
diff changeset
707 } // End of for all blocks
418
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
708 }
0
a61af66fc99e Initial load
duke
parents:
diff changeset
709
418
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
710 //-----------------------------fixup_flow--------------------------------------
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
711 // Fix up the final control flow for basic blocks.
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
712 void PhaseCFG::fixup_flow() {
0
a61af66fc99e Initial load
duke
parents:
diff changeset
713 // Fixup final control flow for the blocks. Remove jump-to-next
a61af66fc99e Initial load
duke
parents:
diff changeset
714 // block. If neither arm of a IF follows the conditional branch, we
a61af66fc99e Initial load
duke
parents:
diff changeset
715 // have to add a second jump after the conditional. We place the
a61af66fc99e Initial load
duke
parents:
diff changeset
716 // TRUE branch target in succs[0] for both GOTOs and IFs.
418
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
717 for (uint i=0; i < _num_blocks; i++) {
0
a61af66fc99e Initial load
duke
parents:
diff changeset
718 Block *b = _blocks[i];
a61af66fc99e Initial load
duke
parents:
diff changeset
719 b->_pre_order = i; // turn pre-order into block-index
a61af66fc99e Initial load
duke
parents:
diff changeset
720
a61af66fc99e Initial load
duke
parents:
diff changeset
721 // Connector blocks need no further processing.
a61af66fc99e Initial load
duke
parents:
diff changeset
722 if (b->is_connector()) {
a61af66fc99e Initial load
duke
parents:
diff changeset
723 assert((i+1) == _num_blocks || _blocks[i+1]->is_connector(),
a61af66fc99e Initial load
duke
parents:
diff changeset
724 "All connector blocks should sink to the end");
a61af66fc99e Initial load
duke
parents:
diff changeset
725 continue;
a61af66fc99e Initial load
duke
parents:
diff changeset
726 }
a61af66fc99e Initial load
duke
parents:
diff changeset
727 assert(b->is_Empty() != Block::completely_empty,
a61af66fc99e Initial load
duke
parents:
diff changeset
728 "Empty blocks should be connectors");
a61af66fc99e Initial load
duke
parents:
diff changeset
729
a61af66fc99e Initial load
duke
parents:
diff changeset
730 Block *bnext = (i < _num_blocks-1) ? _blocks[i+1] : NULL;
a61af66fc99e Initial load
duke
parents:
diff changeset
731 Block *bs0 = b->non_connector_successor(0);
a61af66fc99e Initial load
duke
parents:
diff changeset
732
a61af66fc99e Initial load
duke
parents:
diff changeset
733 // Check for multi-way branches where I cannot negate the test to
a61af66fc99e Initial load
duke
parents:
diff changeset
734 // exchange the true and false targets.
a61af66fc99e Initial load
duke
parents:
diff changeset
735 if( no_flip_branch( b ) ) {
a61af66fc99e Initial load
duke
parents:
diff changeset
736 // Find fall through case - if must fall into its target
a61af66fc99e Initial load
duke
parents:
diff changeset
737 int branch_idx = b->_nodes.size() - b->_num_succs;
a61af66fc99e Initial load
duke
parents:
diff changeset
738 for (uint j2 = 0; j2 < b->_num_succs; j2++) {
a61af66fc99e Initial load
duke
parents:
diff changeset
739 const ProjNode* p = b->_nodes[branch_idx + j2]->as_Proj();
a61af66fc99e Initial load
duke
parents:
diff changeset
740 if (p->_con == 0) {
a61af66fc99e Initial load
duke
parents:
diff changeset
741 // successor j2 is fall through case
a61af66fc99e Initial load
duke
parents:
diff changeset
742 if (b->non_connector_successor(j2) != bnext) {
a61af66fc99e Initial load
duke
parents:
diff changeset
743 // but it is not the next block => insert a goto
a61af66fc99e Initial load
duke
parents:
diff changeset
744 insert_goto_at(i, j2);
a61af66fc99e Initial load
duke
parents:
diff changeset
745 }
a61af66fc99e Initial load
duke
parents:
diff changeset
746 // Put taken branch in slot 0
a61af66fc99e Initial load
duke
parents:
diff changeset
747 if( j2 == 0 && b->_num_succs == 2) {
a61af66fc99e Initial load
duke
parents:
diff changeset
748 // Flip targets in succs map
a61af66fc99e Initial load
duke
parents:
diff changeset
749 Block *tbs0 = b->_succs[0];
a61af66fc99e Initial load
duke
parents:
diff changeset
750 Block *tbs1 = b->_succs[1];
a61af66fc99e Initial load
duke
parents:
diff changeset
751 b->_succs.map( 0, tbs1 );
a61af66fc99e Initial load
duke
parents:
diff changeset
752 b->_succs.map( 1, tbs0 );
a61af66fc99e Initial load
duke
parents:
diff changeset
753 }
a61af66fc99e Initial load
duke
parents:
diff changeset
754 break;
a61af66fc99e Initial load
duke
parents:
diff changeset
755 }
a61af66fc99e Initial load
duke
parents:
diff changeset
756 }
a61af66fc99e Initial load
duke
parents:
diff changeset
757 // Remove all CatchProjs
418
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
758 for (uint j1 = 0; j1 < b->_num_succs; j1++) b->_nodes.pop();
0
a61af66fc99e Initial load
duke
parents:
diff changeset
759
a61af66fc99e Initial load
duke
parents:
diff changeset
760 } else if (b->_num_succs == 1) {
a61af66fc99e Initial load
duke
parents:
diff changeset
761 // Block ends in a Goto?
a61af66fc99e Initial load
duke
parents:
diff changeset
762 if (bnext == bs0) {
a61af66fc99e Initial load
duke
parents:
diff changeset
763 // We fall into next block; remove the Goto
a61af66fc99e Initial load
duke
parents:
diff changeset
764 b->_nodes.pop();
a61af66fc99e Initial load
duke
parents:
diff changeset
765 }
a61af66fc99e Initial load
duke
parents:
diff changeset
766
a61af66fc99e Initial load
duke
parents:
diff changeset
767 } else if( b->_num_succs == 2 ) { // Block ends in a If?
a61af66fc99e Initial load
duke
parents:
diff changeset
768 // Get opcode of 1st projection (matches _succs[0])
a61af66fc99e Initial load
duke
parents:
diff changeset
769 // Note: Since this basic block has 2 exits, the last 2 nodes must
a61af66fc99e Initial load
duke
parents:
diff changeset
770 // be projections (in any order), the 3rd last node must be
a61af66fc99e Initial load
duke
parents:
diff changeset
771 // the IfNode (we have excluded other 2-way exits such as
a61af66fc99e Initial load
duke
parents:
diff changeset
772 // CatchNodes already).
a61af66fc99e Initial load
duke
parents:
diff changeset
773 MachNode *iff = b->_nodes[b->_nodes.size()-3]->as_Mach();
a61af66fc99e Initial load
duke
parents:
diff changeset
774 ProjNode *proj0 = b->_nodes[b->_nodes.size()-2]->as_Proj();
a61af66fc99e Initial load
duke
parents:
diff changeset
775 ProjNode *proj1 = b->_nodes[b->_nodes.size()-1]->as_Proj();
a61af66fc99e Initial load
duke
parents:
diff changeset
776
a61af66fc99e Initial load
duke
parents:
diff changeset
777 // Assert that proj0 and succs[0] match up. Similarly for proj1 and succs[1].
a61af66fc99e Initial load
duke
parents:
diff changeset
778 assert(proj0->raw_out(0) == b->_succs[0]->head(), "Mismatch successor 0");
a61af66fc99e Initial load
duke
parents:
diff changeset
779 assert(proj1->raw_out(0) == b->_succs[1]->head(), "Mismatch successor 1");
a61af66fc99e Initial load
duke
parents:
diff changeset
780
a61af66fc99e Initial load
duke
parents:
diff changeset
781 Block *bs1 = b->non_connector_successor(1);
a61af66fc99e Initial load
duke
parents:
diff changeset
782
a61af66fc99e Initial load
duke
parents:
diff changeset
783 // Check for neither successor block following the current
a61af66fc99e Initial load
duke
parents:
diff changeset
784 // block ending in a conditional. If so, move one of the
a61af66fc99e Initial load
duke
parents:
diff changeset
785 // successors after the current one, provided that the
a61af66fc99e Initial load
duke
parents:
diff changeset
786 // successor was previously unscheduled, but moveable
a61af66fc99e Initial load
duke
parents:
diff changeset
787 // (i.e., all paths to it involve a branch).
418
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
788 if( !C->do_freq_based_layout() && bnext != bs0 && bnext != bs1 ) {
0
a61af66fc99e Initial load
duke
parents:
diff changeset
789 // Choose the more common successor based on the probability
a61af66fc99e Initial load
duke
parents:
diff changeset
790 // of the conditional branch.
a61af66fc99e Initial load
duke
parents:
diff changeset
791 Block *bx = bs0;
a61af66fc99e Initial load
duke
parents:
diff changeset
792 Block *by = bs1;
a61af66fc99e Initial load
duke
parents:
diff changeset
793
a61af66fc99e Initial load
duke
parents:
diff changeset
794 // _prob is the probability of taking the true path. Make
a61af66fc99e Initial load
duke
parents:
diff changeset
795 // p the probability of taking successor #1.
a61af66fc99e Initial load
duke
parents:
diff changeset
796 float p = iff->as_MachIf()->_prob;
a61af66fc99e Initial load
duke
parents:
diff changeset
797 if( proj0->Opcode() == Op_IfTrue ) {
a61af66fc99e Initial load
duke
parents:
diff changeset
798 p = 1.0 - p;
a61af66fc99e Initial load
duke
parents:
diff changeset
799 }
a61af66fc99e Initial load
duke
parents:
diff changeset
800
a61af66fc99e Initial load
duke
parents:
diff changeset
801 // Prefer successor #1 if p > 0.5
a61af66fc99e Initial load
duke
parents:
diff changeset
802 if (p > PROB_FAIR) {
a61af66fc99e Initial load
duke
parents:
diff changeset
803 bx = bs1;
a61af66fc99e Initial load
duke
parents:
diff changeset
804 by = bs0;
a61af66fc99e Initial load
duke
parents:
diff changeset
805 }
a61af66fc99e Initial load
duke
parents:
diff changeset
806
a61af66fc99e Initial load
duke
parents:
diff changeset
807 // Attempt the more common successor first
418
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
808 if (move_to_next(bx, i)) {
0
a61af66fc99e Initial load
duke
parents:
diff changeset
809 bnext = bx;
418
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
810 } else if (move_to_next(by, i)) {
0
a61af66fc99e Initial load
duke
parents:
diff changeset
811 bnext = by;
a61af66fc99e Initial load
duke
parents:
diff changeset
812 }
a61af66fc99e Initial load
duke
parents:
diff changeset
813 }
a61af66fc99e Initial load
duke
parents:
diff changeset
814
a61af66fc99e Initial load
duke
parents:
diff changeset
815 // Check for conditional branching the wrong way. Negate
a61af66fc99e Initial load
duke
parents:
diff changeset
816 // conditional, if needed, so it falls into the following block
a61af66fc99e Initial load
duke
parents:
diff changeset
817 // and branches to the not-following block.
a61af66fc99e Initial load
duke
parents:
diff changeset
818
a61af66fc99e Initial load
duke
parents:
diff changeset
819 // Check for the next block being in succs[0]. We are going to branch
a61af66fc99e Initial load
duke
parents:
diff changeset
820 // to succs[0], so we want the fall-thru case as the next block in
a61af66fc99e Initial load
duke
parents:
diff changeset
821 // succs[1].
a61af66fc99e Initial load
duke
parents:
diff changeset
822 if (bnext == bs0) {
a61af66fc99e Initial load
duke
parents:
diff changeset
823 // Fall-thru case in succs[0], so flip targets in succs map
a61af66fc99e Initial load
duke
parents:
diff changeset
824 Block *tbs0 = b->_succs[0];
a61af66fc99e Initial load
duke
parents:
diff changeset
825 Block *tbs1 = b->_succs[1];
a61af66fc99e Initial load
duke
parents:
diff changeset
826 b->_succs.map( 0, tbs1 );
a61af66fc99e Initial load
duke
parents:
diff changeset
827 b->_succs.map( 1, tbs0 );
a61af66fc99e Initial load
duke
parents:
diff changeset
828 // Flip projection for each target
a61af66fc99e Initial load
duke
parents:
diff changeset
829 { ProjNode *tmp = proj0; proj0 = proj1; proj1 = tmp; }
a61af66fc99e Initial load
duke
parents:
diff changeset
830
418
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
831 } else if( bnext != bs1 ) {
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
832 // Need a double-branch
0
a61af66fc99e Initial load
duke
parents:
diff changeset
833 // The existing conditional branch need not change.
a61af66fc99e Initial load
duke
parents:
diff changeset
834 // Add a unconditional branch to the false target.
a61af66fc99e Initial load
duke
parents:
diff changeset
835 // Alas, it must appear in its own block and adding a
a61af66fc99e Initial load
duke
parents:
diff changeset
836 // block this late in the game is complicated. Sigh.
a61af66fc99e Initial load
duke
parents:
diff changeset
837 insert_goto_at(i, 1);
a61af66fc99e Initial load
duke
parents:
diff changeset
838 }
a61af66fc99e Initial load
duke
parents:
diff changeset
839
a61af66fc99e Initial load
duke
parents:
diff changeset
840 // Make sure we TRUE branch to the target
418
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
841 if( proj0->Opcode() == Op_IfFalse ) {
3853
11211f7cb5a0 7079317: Incorrect branch's destination block in PrintoOptoAssembly output
kvn
parents: 3851
diff changeset
842 iff->as_MachIf()->negate();
418
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
843 }
0
a61af66fc99e Initial load
duke
parents:
diff changeset
844
a61af66fc99e Initial load
duke
parents:
diff changeset
845 b->_nodes.pop(); // Remove IfFalse & IfTrue projections
a61af66fc99e Initial load
duke
parents:
diff changeset
846 b->_nodes.pop();
a61af66fc99e Initial load
duke
parents:
diff changeset
847
a61af66fc99e Initial load
duke
parents:
diff changeset
848 } else {
a61af66fc99e Initial load
duke
parents:
diff changeset
849 // Multi-exit block, e.g. a switch statement
a61af66fc99e Initial load
duke
parents:
diff changeset
850 // But we don't need to do anything here
a61af66fc99e Initial load
duke
parents:
diff changeset
851 }
a61af66fc99e Initial load
duke
parents:
diff changeset
852 } // End of for all blocks
a61af66fc99e Initial load
duke
parents:
diff changeset
853 }
a61af66fc99e Initial load
duke
parents:
diff changeset
854
a61af66fc99e Initial load
duke
parents:
diff changeset
855
a61af66fc99e Initial load
duke
parents:
diff changeset
856 //------------------------------dump-------------------------------------------
a61af66fc99e Initial load
duke
parents:
diff changeset
857 #ifndef PRODUCT
a61af66fc99e Initial load
duke
parents:
diff changeset
858 void PhaseCFG::_dump_cfg( const Node *end, VectorSet &visited ) const {
a61af66fc99e Initial load
duke
parents:
diff changeset
859 const Node *x = end->is_block_proj();
a61af66fc99e Initial load
duke
parents:
diff changeset
860 assert( x, "not a CFG" );
a61af66fc99e Initial load
duke
parents:
diff changeset
861
a61af66fc99e Initial load
duke
parents:
diff changeset
862 // Do not visit this block again
a61af66fc99e Initial load
duke
parents:
diff changeset
863 if( visited.test_set(x->_idx) ) return;
a61af66fc99e Initial load
duke
parents:
diff changeset
864
a61af66fc99e Initial load
duke
parents:
diff changeset
865 // Skip through this block
a61af66fc99e Initial load
duke
parents:
diff changeset
866 const Node *p = x;
a61af66fc99e Initial load
duke
parents:
diff changeset
867 do {
a61af66fc99e Initial load
duke
parents:
diff changeset
868 p = p->in(0); // Move control forward
a61af66fc99e Initial load
duke
parents:
diff changeset
869 assert( !p->is_block_proj() || p->is_Root(), "not a CFG" );
a61af66fc99e Initial load
duke
parents:
diff changeset
870 } while( !p->is_block_start() );
a61af66fc99e Initial load
duke
parents:
diff changeset
871
a61af66fc99e Initial load
duke
parents:
diff changeset
872 // Recursively visit
a61af66fc99e Initial load
duke
parents:
diff changeset
873 for( uint i=1; i<p->req(); i++ )
a61af66fc99e Initial load
duke
parents:
diff changeset
874 _dump_cfg(p->in(i),visited);
a61af66fc99e Initial load
duke
parents:
diff changeset
875
a61af66fc99e Initial load
duke
parents:
diff changeset
876 // Dump the block
a61af66fc99e Initial load
duke
parents:
diff changeset
877 _bbs[p->_idx]->dump(&_bbs);
a61af66fc99e Initial load
duke
parents:
diff changeset
878 }
a61af66fc99e Initial load
duke
parents:
diff changeset
879
a61af66fc99e Initial load
duke
parents:
diff changeset
880 void PhaseCFG::dump( ) const {
a61af66fc99e Initial load
duke
parents:
diff changeset
881 tty->print("\n--- CFG --- %d BBs\n",_num_blocks);
a61af66fc99e Initial load
duke
parents:
diff changeset
882 if( _blocks.size() ) { // Did we do basic-block layout?
a61af66fc99e Initial load
duke
parents:
diff changeset
883 for( uint i=0; i<_num_blocks; i++ )
a61af66fc99e Initial load
duke
parents:
diff changeset
884 _blocks[i]->dump(&_bbs);
a61af66fc99e Initial load
duke
parents:
diff changeset
885 } else { // Else do it with a DFS
a61af66fc99e Initial load
duke
parents:
diff changeset
886 VectorSet visited(_bbs._arena);
a61af66fc99e Initial load
duke
parents:
diff changeset
887 _dump_cfg(_root,visited);
a61af66fc99e Initial load
duke
parents:
diff changeset
888 }
a61af66fc99e Initial load
duke
parents:
diff changeset
889 }
a61af66fc99e Initial load
duke
parents:
diff changeset
890
a61af66fc99e Initial load
duke
parents:
diff changeset
891 void PhaseCFG::dump_headers() {
a61af66fc99e Initial load
duke
parents:
diff changeset
892 for( uint i = 0; i < _num_blocks; i++ ) {
a61af66fc99e Initial load
duke
parents:
diff changeset
893 if( _blocks[i] == NULL ) continue;
a61af66fc99e Initial load
duke
parents:
diff changeset
894 _blocks[i]->dump_head(&_bbs);
a61af66fc99e Initial load
duke
parents:
diff changeset
895 }
a61af66fc99e Initial load
duke
parents:
diff changeset
896 }
a61af66fc99e Initial load
duke
parents:
diff changeset
897
a61af66fc99e Initial load
duke
parents:
diff changeset
898 void PhaseCFG::verify( ) const {
566
91263420e1c6 6791852: assert(b->_nodes[insidx] == n,"got insidx set incorrectly")
kvn
parents: 418
diff changeset
899 #ifdef ASSERT
0
a61af66fc99e Initial load
duke
parents:
diff changeset
900 // Verify sane CFG
4115
1bd45abaa507 6890673: Eliminate allocations immediately after EA
kvn
parents: 3929
diff changeset
901 for (uint i = 0; i < _num_blocks; i++) {
0
a61af66fc99e Initial load
duke
parents:
diff changeset
902 Block *b = _blocks[i];
a61af66fc99e Initial load
duke
parents:
diff changeset
903 uint cnt = b->_nodes.size();
a61af66fc99e Initial load
duke
parents:
diff changeset
904 uint j;
4115
1bd45abaa507 6890673: Eliminate allocations immediately after EA
kvn
parents: 3929
diff changeset
905 for (j = 0; j < cnt; j++) {
0
a61af66fc99e Initial load
duke
parents:
diff changeset
906 Node *n = b->_nodes[j];
a61af66fc99e Initial load
duke
parents:
diff changeset
907 assert( _bbs[n->_idx] == b, "" );
4115
1bd45abaa507 6890673: Eliminate allocations immediately after EA
kvn
parents: 3929
diff changeset
908 if (j >= 1 && n->is_Mach() &&
1bd45abaa507 6890673: Eliminate allocations immediately after EA
kvn
parents: 3929
diff changeset
909 n->as_Mach()->ideal_Opcode() == Op_CreateEx) {
1bd45abaa507 6890673: Eliminate allocations immediately after EA
kvn
parents: 3929
diff changeset
910 assert(j == 1 || b->_nodes[j-1]->is_Phi(),
1bd45abaa507 6890673: Eliminate allocations immediately after EA
kvn
parents: 3929
diff changeset
911 "CreateEx must be first instruction in block");
0
a61af66fc99e Initial load
duke
parents:
diff changeset
912 }
4115
1bd45abaa507 6890673: Eliminate allocations immediately after EA
kvn
parents: 3929
diff changeset
913 for (uint k = 0; k < n->req(); k++) {
566
91263420e1c6 6791852: assert(b->_nodes[insidx] == n,"got insidx set incorrectly")
kvn
parents: 418
diff changeset
914 Node *def = n->in(k);
4115
1bd45abaa507 6890673: Eliminate allocations immediately after EA
kvn
parents: 3929
diff changeset
915 if (def && def != n) {
1bd45abaa507 6890673: Eliminate allocations immediately after EA
kvn
parents: 3929
diff changeset
916 assert(_bbs[def->_idx] || def->is_Con(),
1bd45abaa507 6890673: Eliminate allocations immediately after EA
kvn
parents: 3929
diff changeset
917 "must have block; constants for debug info ok");
566
91263420e1c6 6791852: assert(b->_nodes[insidx] == n,"got insidx set incorrectly")
kvn
parents: 418
diff changeset
918 // Verify that instructions in the block is in correct order.
91263420e1c6 6791852: assert(b->_nodes[insidx] == n,"got insidx set incorrectly")
kvn
parents: 418
diff changeset
919 // Uses must follow their definition if they are at the same block.
91263420e1c6 6791852: assert(b->_nodes[insidx] == n,"got insidx set incorrectly")
kvn
parents: 418
diff changeset
920 // Mostly done to check that MachSpillCopy nodes are placed correctly
91263420e1c6 6791852: assert(b->_nodes[insidx] == n,"got insidx set incorrectly")
kvn
parents: 418
diff changeset
921 // when CreateEx node is moved in build_ifg_physical().
4115
1bd45abaa507 6890673: Eliminate allocations immediately after EA
kvn
parents: 3929
diff changeset
922 if (_bbs[def->_idx] == b &&
566
91263420e1c6 6791852: assert(b->_nodes[insidx] == n,"got insidx set incorrectly")
kvn
parents: 418
diff changeset
923 !(b->head()->is_Loop() && n->is_Phi()) &&
91263420e1c6 6791852: assert(b->_nodes[insidx] == n,"got insidx set incorrectly")
kvn
parents: 418
diff changeset
924 // See (+++) comment in reg_split.cpp
4115
1bd45abaa507 6890673: Eliminate allocations immediately after EA
kvn
parents: 3929
diff changeset
925 !(n->jvms() != NULL && n->jvms()->is_monitor_use(k))) {
893
dd0a4e1e219b 6851386: assert(b->find_node(def) < j,"uses must follow definitions")
kvn
parents: 833
diff changeset
926 bool is_loop = false;
dd0a4e1e219b 6851386: assert(b->find_node(def) < j,"uses must follow definitions")
kvn
parents: 833
diff changeset
927 if (n->is_Phi()) {
4115
1bd45abaa507 6890673: Eliminate allocations immediately after EA
kvn
parents: 3929
diff changeset
928 for (uint l = 1; l < def->req(); l++) {
893
dd0a4e1e219b 6851386: assert(b->find_node(def) < j,"uses must follow definitions")
kvn
parents: 833
diff changeset
929 if (n == def->in(l)) {
dd0a4e1e219b 6851386: assert(b->find_node(def) < j,"uses must follow definitions")
kvn
parents: 833
diff changeset
930 is_loop = true;
dd0a4e1e219b 6851386: assert(b->find_node(def) < j,"uses must follow definitions")
kvn
parents: 833
diff changeset
931 break; // Some kind of loop
dd0a4e1e219b 6851386: assert(b->find_node(def) < j,"uses must follow definitions")
kvn
parents: 833
diff changeset
932 }
dd0a4e1e219b 6851386: assert(b->find_node(def) < j,"uses must follow definitions")
kvn
parents: 833
diff changeset
933 }
dd0a4e1e219b 6851386: assert(b->find_node(def) < j,"uses must follow definitions")
kvn
parents: 833
diff changeset
934 }
4115
1bd45abaa507 6890673: Eliminate allocations immediately after EA
kvn
parents: 3929
diff changeset
935 assert(is_loop || b->find_node(def) < j, "uses must follow definitions");
601
523ded093c31 6809798: SafePointScalarObject node placed into incorrect block during GCM
kvn
parents: 566
diff changeset
936 }
0
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 j = b->end_idx();
a61af66fc99e Initial load
duke
parents:
diff changeset
942 Node *bp = (Node*)b->_nodes[b->_nodes.size()-1]->is_block_proj();
a61af66fc99e Initial load
duke
parents:
diff changeset
943 assert( bp, "last instruction must be a block proj" );
a61af66fc99e Initial load
duke
parents:
diff changeset
944 assert( bp == b->_nodes[j], "wrong number of successors for this block" );
4115
1bd45abaa507 6890673: Eliminate allocations immediately after EA
kvn
parents: 3929
diff changeset
945 if (bp->is_Catch()) {
1bd45abaa507 6890673: Eliminate allocations immediately after EA
kvn
parents: 3929
diff changeset
946 while (b->_nodes[--j]->is_MachProj()) ;
1bd45abaa507 6890673: Eliminate allocations immediately after EA
kvn
parents: 3929
diff changeset
947 assert(b->_nodes[j]->is_MachCall(), "CatchProj must follow call");
1bd45abaa507 6890673: Eliminate allocations immediately after EA
kvn
parents: 3929
diff changeset
948 } else if (bp->is_Mach() && bp->as_Mach()->ideal_Opcode() == Op_If) {
1bd45abaa507 6890673: Eliminate allocations immediately after EA
kvn
parents: 3929
diff changeset
949 assert(b->_num_succs == 2, "Conditional branch must have two targets");
0
a61af66fc99e Initial load
duke
parents:
diff changeset
950 }
a61af66fc99e Initial load
duke
parents:
diff changeset
951 }
566
91263420e1c6 6791852: assert(b->_nodes[insidx] == n,"got insidx set incorrectly")
kvn
parents: 418
diff changeset
952 #endif
0
a61af66fc99e Initial load
duke
parents:
diff changeset
953 }
a61af66fc99e Initial load
duke
parents:
diff changeset
954 #endif
a61af66fc99e Initial load
duke
parents:
diff changeset
955
a61af66fc99e Initial load
duke
parents:
diff changeset
956 //=============================================================================
a61af66fc99e Initial load
duke
parents:
diff changeset
957 //------------------------------UnionFind--------------------------------------
a61af66fc99e Initial load
duke
parents:
diff changeset
958 UnionFind::UnionFind( uint max ) : _cnt(max), _max(max), _indices(NEW_RESOURCE_ARRAY(uint,max)) {
a61af66fc99e Initial load
duke
parents:
diff changeset
959 Copy::zero_to_bytes( _indices, sizeof(uint)*max );
a61af66fc99e Initial load
duke
parents:
diff changeset
960 }
a61af66fc99e Initial load
duke
parents:
diff changeset
961
a61af66fc99e Initial load
duke
parents:
diff changeset
962 void UnionFind::extend( uint from_idx, uint to_idx ) {
a61af66fc99e Initial load
duke
parents:
diff changeset
963 _nesting.check();
a61af66fc99e Initial load
duke
parents:
diff changeset
964 if( from_idx >= _max ) {
a61af66fc99e Initial load
duke
parents:
diff changeset
965 uint size = 16;
a61af66fc99e Initial load
duke
parents:
diff changeset
966 while( size <= from_idx ) size <<=1;
a61af66fc99e Initial load
duke
parents:
diff changeset
967 _indices = REALLOC_RESOURCE_ARRAY( uint, _indices, _max, size );
a61af66fc99e Initial load
duke
parents:
diff changeset
968 _max = size;
a61af66fc99e Initial load
duke
parents:
diff changeset
969 }
a61af66fc99e Initial load
duke
parents:
diff changeset
970 while( _cnt <= from_idx ) _indices[_cnt++] = 0;
a61af66fc99e Initial load
duke
parents:
diff changeset
971 _indices[from_idx] = to_idx;
a61af66fc99e Initial load
duke
parents:
diff changeset
972 }
a61af66fc99e Initial load
duke
parents:
diff changeset
973
a61af66fc99e Initial load
duke
parents:
diff changeset
974 void UnionFind::reset( uint max ) {
a61af66fc99e Initial load
duke
parents:
diff changeset
975 assert( max <= max_uint, "Must fit within uint" );
a61af66fc99e Initial load
duke
parents:
diff changeset
976 // Force the Union-Find mapping to be at least this large
a61af66fc99e Initial load
duke
parents:
diff changeset
977 extend(max,0);
a61af66fc99e Initial load
duke
parents:
diff changeset
978 // Initialize to be the ID mapping.
418
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
979 for( uint i=0; i<max; i++ ) map(i,i);
0
a61af66fc99e Initial load
duke
parents:
diff changeset
980 }
a61af66fc99e Initial load
duke
parents:
diff changeset
981
a61af66fc99e Initial load
duke
parents:
diff changeset
982 //------------------------------Find_compress----------------------------------
a61af66fc99e Initial load
duke
parents:
diff changeset
983 // Straight out of Tarjan's union-find algorithm
a61af66fc99e Initial load
duke
parents:
diff changeset
984 uint UnionFind::Find_compress( uint idx ) {
a61af66fc99e Initial load
duke
parents:
diff changeset
985 uint cur = idx;
a61af66fc99e Initial load
duke
parents:
diff changeset
986 uint next = lookup(cur);
a61af66fc99e Initial load
duke
parents:
diff changeset
987 while( next != cur ) { // Scan chain of equivalences
a61af66fc99e Initial load
duke
parents:
diff changeset
988 assert( next < cur, "always union smaller" );
a61af66fc99e Initial load
duke
parents:
diff changeset
989 cur = next; // until find a fixed-point
a61af66fc99e Initial load
duke
parents:
diff changeset
990 next = lookup(cur);
a61af66fc99e Initial load
duke
parents:
diff changeset
991 }
a61af66fc99e Initial load
duke
parents:
diff changeset
992 // Core of union-find algorithm: update chain of
a61af66fc99e Initial load
duke
parents:
diff changeset
993 // equivalences to be equal to the root.
a61af66fc99e Initial load
duke
parents:
diff changeset
994 while( idx != next ) {
a61af66fc99e Initial load
duke
parents:
diff changeset
995 uint tmp = lookup(idx);
a61af66fc99e Initial load
duke
parents:
diff changeset
996 map(idx, next);
a61af66fc99e Initial load
duke
parents:
diff changeset
997 idx = tmp;
a61af66fc99e Initial load
duke
parents:
diff changeset
998 }
a61af66fc99e Initial load
duke
parents:
diff changeset
999 return idx;
a61af66fc99e Initial load
duke
parents:
diff changeset
1000 }
a61af66fc99e Initial load
duke
parents:
diff changeset
1001
a61af66fc99e Initial load
duke
parents:
diff changeset
1002 //------------------------------Find_const-------------------------------------
a61af66fc99e Initial load
duke
parents:
diff changeset
1003 // Like Find above, but no path compress, so bad asymptotic behavior
a61af66fc99e Initial load
duke
parents:
diff changeset
1004 uint UnionFind::Find_const( uint idx ) const {
a61af66fc99e Initial load
duke
parents:
diff changeset
1005 if( idx == 0 ) return idx; // Ignore the zero idx
a61af66fc99e Initial load
duke
parents:
diff changeset
1006 // Off the end? This can happen during debugging dumps
a61af66fc99e Initial load
duke
parents:
diff changeset
1007 // when data structures have not finished being updated.
a61af66fc99e Initial load
duke
parents:
diff changeset
1008 if( idx >= _max ) return idx;
a61af66fc99e Initial load
duke
parents:
diff changeset
1009 uint next = lookup(idx);
a61af66fc99e Initial load
duke
parents:
diff changeset
1010 while( next != idx ) { // Scan chain of equivalences
a61af66fc99e Initial load
duke
parents:
diff changeset
1011 idx = next; // until find a fixed-point
a61af66fc99e Initial load
duke
parents:
diff changeset
1012 next = lookup(idx);
a61af66fc99e Initial load
duke
parents:
diff changeset
1013 }
a61af66fc99e Initial load
duke
parents:
diff changeset
1014 return next;
a61af66fc99e Initial load
duke
parents:
diff changeset
1015 }
a61af66fc99e Initial load
duke
parents:
diff changeset
1016
a61af66fc99e Initial load
duke
parents:
diff changeset
1017 //------------------------------Union------------------------------------------
a61af66fc99e Initial load
duke
parents:
diff changeset
1018 // union 2 sets together.
a61af66fc99e Initial load
duke
parents:
diff changeset
1019 void UnionFind::Union( uint idx1, uint idx2 ) {
a61af66fc99e Initial load
duke
parents:
diff changeset
1020 uint src = Find(idx1);
a61af66fc99e Initial load
duke
parents:
diff changeset
1021 uint dst = Find(idx2);
a61af66fc99e Initial load
duke
parents:
diff changeset
1022 assert( src, "" );
a61af66fc99e Initial load
duke
parents:
diff changeset
1023 assert( dst, "" );
a61af66fc99e Initial load
duke
parents:
diff changeset
1024 assert( src < _max, "oob" );
a61af66fc99e Initial load
duke
parents:
diff changeset
1025 assert( dst < _max, "oob" );
a61af66fc99e Initial load
duke
parents:
diff changeset
1026 assert( src < dst, "always union smaller" );
a61af66fc99e Initial load
duke
parents:
diff changeset
1027 map(dst,src);
a61af66fc99e Initial load
duke
parents:
diff changeset
1028 }
418
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1029
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1030 #ifndef PRODUCT
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1031 static void edge_dump(GrowableArray<CFGEdge *> *edges) {
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1032 tty->print_cr("---- Edges ----");
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1033 for (int i = 0; i < edges->length(); i++) {
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1034 CFGEdge *e = edges->at(i);
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1035 if (e != NULL) {
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1036 edges->at(i)->dump();
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1037 }
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1038 }
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1039 }
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1040
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1041 static void trace_dump(Trace *traces[], int count) {
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1042 tty->print_cr("---- Traces ----");
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1043 for (int i = 0; i < count; i++) {
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1044 Trace *tr = traces[i];
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1045 if (tr != NULL) {
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1046 tr->dump();
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1047 }
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1048 }
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1049 }
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1050
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1051 void Trace::dump( ) const {
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1052 tty->print_cr("Trace (freq %f)", first_block()->_freq);
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1053 for (Block *b = first_block(); b != NULL; b = next(b)) {
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1054 tty->print(" B%d", b->_pre_order);
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1055 if (b->head()->is_Loop()) {
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1056 tty->print(" (L%d)", b->compute_loop_alignment());
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1057 }
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1058 if (b->has_loop_alignment()) {
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1059 tty->print(" (T%d)", b->code_alignment());
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1060 }
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1061 }
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1062 tty->cr();
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1063 }
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1064
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1065 void CFGEdge::dump( ) const {
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1066 tty->print(" B%d --> B%d Freq: %f out:%3d%% in:%3d%% State: ",
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1067 from()->_pre_order, to()->_pre_order, freq(), _from_pct, _to_pct);
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1068 switch(state()) {
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1069 case connected:
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1070 tty->print("connected");
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1071 break;
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1072 case open:
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1073 tty->print("open");
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1074 break;
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1075 case interior:
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1076 tty->print("interior");
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1077 break;
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1078 }
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1079 if (infrequent()) {
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1080 tty->print(" infrequent");
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1081 }
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1082 tty->cr();
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1083 }
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1084 #endif
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1085
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1086 //=============================================================================
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1087
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1088 //------------------------------edge_order-------------------------------------
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1089 // Comparison function for edges
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1090 static int edge_order(CFGEdge **e0, CFGEdge **e1) {
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1091 float freq0 = (*e0)->freq();
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1092 float freq1 = (*e1)->freq();
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1093 if (freq0 != freq1) {
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1094 return freq0 > freq1 ? -1 : 1;
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1095 }
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1096
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1097 int dist0 = (*e0)->to()->_rpo - (*e0)->from()->_rpo;
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1098 int dist1 = (*e1)->to()->_rpo - (*e1)->from()->_rpo;
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1099
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1100 return dist1 - dist0;
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1101 }
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1102
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1103 //------------------------------trace_frequency_order--------------------------
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1104 // Comparison function for edges
3929
f94227b6117b 7090259: Fix hotspot sources to build with old compilers
kvn
parents: 3853
diff changeset
1105 extern "C" int trace_frequency_order(const void *p0, const void *p1) {
418
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1106 Trace *tr0 = *(Trace **) p0;
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1107 Trace *tr1 = *(Trace **) p1;
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1108 Block *b0 = tr0->first_block();
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1109 Block *b1 = tr1->first_block();
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1110
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1111 // The trace of connector blocks goes at the end;
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1112 // we only expect one such trace
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1113 if (b0->is_connector() != b1->is_connector()) {
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1114 return b1->is_connector() ? -1 : 1;
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1115 }
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1116
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1117 // Pull more frequently executed blocks to the beginning
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1118 float freq0 = b0->_freq;
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1119 float freq1 = b1->_freq;
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1120 if (freq0 != freq1) {
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1121 return freq0 > freq1 ? -1 : 1;
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1122 }
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1123
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1124 int diff = tr0->first_block()->_rpo - tr1->first_block()->_rpo;
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1125
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1126 return diff;
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1127 }
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1128
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1129 //------------------------------find_edges-------------------------------------
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1130 // Find edges of interest, i.e, those which can fall through. Presumes that
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1131 // edges which don't fall through are of low frequency and can be generally
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1132 // ignored. Initialize the list of traces.
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1133 void PhaseBlockLayout::find_edges()
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1134 {
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1135 // Walk the blocks, creating edges and Traces
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1136 uint i;
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1137 Trace *tr = NULL;
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1138 for (i = 0; i < _cfg._num_blocks; i++) {
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1139 Block *b = _cfg._blocks[i];
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1140 tr = new Trace(b, next, prev);
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1141 traces[tr->id()] = tr;
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1142
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1143 // All connector blocks should be at the end of the list
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1144 if (b->is_connector()) break;
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1145
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1146 // If this block and the next one have a one-to-one successor
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1147 // predecessor relationship, simply append the next block
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1148 int nfallthru = b->num_fall_throughs();
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1149 while (nfallthru == 1 &&
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1150 b->succ_fall_through(0)) {
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1151 Block *n = b->_succs[0];
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1152
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1153 // Skip over single-entry connector blocks, we don't want to
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1154 // add them to the trace.
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1155 while (n->is_connector() && n->num_preds() == 1) {
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1156 n = n->_succs[0];
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1157 }
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1158
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1159 // We see a merge point, so stop search for the next block
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1160 if (n->num_preds() != 1) break;
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1161
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1162 i++;
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1163 assert(n = _cfg._blocks[i], "expecting next block");
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1164 tr->append(n);
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1165 uf->map(n->_pre_order, tr->id());
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1166 traces[n->_pre_order] = NULL;
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1167 nfallthru = b->num_fall_throughs();
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1168 b = n;
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1169 }
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1170
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1171 if (nfallthru > 0) {
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1172 // Create a CFGEdge for each outgoing
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1173 // edge that could be a fall-through.
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1174 for (uint j = 0; j < b->_num_succs; j++ ) {
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1175 if (b->succ_fall_through(j)) {
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1176 Block *target = b->non_connector_successor(j);
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1177 float freq = b->_freq * b->succ_prob(j);
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1178 int from_pct = (int) ((100 * freq) / b->_freq);
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1179 int to_pct = (int) ((100 * freq) / target->_freq);
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1180 edges->append(new CFGEdge(b, target, freq, from_pct, to_pct));
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1181 }
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1182 }
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1183 }
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1184 }
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1185
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1186 // Group connector blocks into one trace
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1187 for (i++; i < _cfg._num_blocks; i++) {
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1188 Block *b = _cfg._blocks[i];
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1189 assert(b->is_connector(), "connector blocks at the end");
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1190 tr->append(b);
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1191 uf->map(b->_pre_order, tr->id());
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1192 traces[b->_pre_order] = NULL;
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1193 }
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1194 }
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1195
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1196 //------------------------------union_traces----------------------------------
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1197 // Union two traces together in uf, and null out the trace in the list
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1198 void PhaseBlockLayout::union_traces(Trace* updated_trace, Trace* old_trace)
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1199 {
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1200 uint old_id = old_trace->id();
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1201 uint updated_id = updated_trace->id();
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1202
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1203 uint lo_id = updated_id;
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1204 uint hi_id = old_id;
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1205
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1206 // If from is greater than to, swap values to meet
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1207 // UnionFind guarantee.
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1208 if (updated_id > old_id) {
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1209 lo_id = old_id;
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1210 hi_id = updated_id;
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1211
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1212 // Fix up the trace ids
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1213 traces[lo_id] = traces[updated_id];
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1214 updated_trace->set_id(lo_id);
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1215 }
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1216
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1217 // Union the lower with the higher and remove the pointer
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1218 // to the higher.
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1219 uf->Union(lo_id, hi_id);
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1220 traces[hi_id] = NULL;
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1221 }
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1222
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1223 //------------------------------grow_traces-------------------------------------
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1224 // Append traces together via the most frequently executed edges
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1225 void PhaseBlockLayout::grow_traces()
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1226 {
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1227 // Order the edges, and drive the growth of Traces via the most
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1228 // frequently executed edges.
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1229 edges->sort(edge_order);
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1230 for (int i = 0; i < edges->length(); i++) {
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1231 CFGEdge *e = edges->at(i);
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1232
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1233 if (e->state() != CFGEdge::open) continue;
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1234
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1235 Block *src_block = e->from();
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1236 Block *targ_block = e->to();
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1237
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1238 // Don't grow traces along backedges?
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1239 if (!BlockLayoutRotateLoops) {
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1240 if (targ_block->_rpo <= src_block->_rpo) {
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1241 targ_block->set_loop_alignment(targ_block);
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1242 continue;
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1243 }
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1244 }
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1245
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1246 Trace *src_trace = trace(src_block);
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1247 Trace *targ_trace = trace(targ_block);
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1248
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1249 // If the edge in question can join two traces at their ends,
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1250 // append one trace to the other.
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1251 if (src_trace->last_block() == src_block) {
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1252 if (src_trace == targ_trace) {
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1253 e->set_state(CFGEdge::interior);
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1254 if (targ_trace->backedge(e)) {
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1255 // Reset i to catch any newly eligible edge
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1256 // (Or we could remember the first "open" edge, and reset there)
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1257 i = 0;
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1258 }
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1259 } else if (targ_trace->first_block() == targ_block) {
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1260 e->set_state(CFGEdge::connected);
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1261 src_trace->append(targ_trace);
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1262 union_traces(src_trace, targ_trace);
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1263 }
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1264 }
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1265 }
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1266 }
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1267
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1268 //------------------------------merge_traces-----------------------------------
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1269 // Embed one trace into another, if the fork or join points are sufficiently
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1270 // balanced.
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1271 void PhaseBlockLayout::merge_traces(bool fall_thru_only)
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1272 {
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1273 // Walk the edge list a another time, looking at unprocessed edges.
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1274 // Fold in diamonds
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1275 for (int i = 0; i < edges->length(); i++) {
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1276 CFGEdge *e = edges->at(i);
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1277
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1278 if (e->state() != CFGEdge::open) continue;
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1279 if (fall_thru_only) {
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1280 if (e->infrequent()) continue;
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1281 }
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1282
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1283 Block *src_block = e->from();
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1284 Trace *src_trace = trace(src_block);
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1285 bool src_at_tail = src_trace->last_block() == src_block;
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1286
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1287 Block *targ_block = e->to();
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1288 Trace *targ_trace = trace(targ_block);
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1289 bool targ_at_start = targ_trace->first_block() == targ_block;
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1290
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1291 if (src_trace == targ_trace) {
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1292 // This may be a loop, but we can't do much about it.
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1293 e->set_state(CFGEdge::interior);
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1294 continue;
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1295 }
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1296
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1297 if (fall_thru_only) {
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1298 // If the edge links the middle of two traces, we can't do anything.
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1299 // Mark the edge and continue.
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1300 if (!src_at_tail & !targ_at_start) {
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1301 continue;
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1302 }
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1303
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1304 // Don't grow traces along backedges?
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1305 if (!BlockLayoutRotateLoops && (targ_block->_rpo <= src_block->_rpo)) {
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1306 continue;
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1307 }
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1308
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1309 // If both ends of the edge are available, why didn't we handle it earlier?
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1310 assert(src_at_tail ^ targ_at_start, "Should have caught this edge earlier.");
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1311
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1312 if (targ_at_start) {
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1313 // Insert the "targ" trace in the "src" trace if the insertion point
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1314 // is a two way branch.
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1315 // Better profitability check possible, but may not be worth it.
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1316 // Someday, see if the this "fork" has an associated "join";
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1317 // then make a policy on merging this trace at the fork or join.
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1318 // For example, other things being equal, it may be better to place this
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1319 // trace at the join point if the "src" trace ends in a two-way, but
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1320 // the insertion point is one-way.
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1321 assert(src_block->num_fall_throughs() == 2, "unexpected diamond");
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1322 e->set_state(CFGEdge::connected);
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1323 src_trace->insert_after(src_block, targ_trace);
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1324 union_traces(src_trace, targ_trace);
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1325 } else if (src_at_tail) {
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1326 if (src_trace != trace(_cfg._broot)) {
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1327 e->set_state(CFGEdge::connected);
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1328 targ_trace->insert_before(targ_block, src_trace);
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1329 union_traces(targ_trace, src_trace);
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1330 }
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1331 }
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1332 } else if (e->state() == CFGEdge::open) {
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1333 // Append traces, even without a fall-thru connection.
605
98cb887364d3 6810672: Comment typos
twisti
parents: 601
diff changeset
1334 // But leave root entry at the beginning of the block list.
418
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1335 if (targ_trace != trace(_cfg._broot)) {
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1336 e->set_state(CFGEdge::connected);
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1337 src_trace->append(targ_trace);
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1338 union_traces(src_trace, targ_trace);
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1339 }
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1340 }
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1341 }
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1342 }
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1343
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1344 //----------------------------reorder_traces-----------------------------------
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1345 // Order the sequence of the traces in some desirable way, and fixup the
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1346 // jumps at the end of each block.
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1347 void PhaseBlockLayout::reorder_traces(int count)
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1348 {
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1349 ResourceArea *area = Thread::current()->resource_area();
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1350 Trace ** new_traces = NEW_ARENA_ARRAY(area, Trace *, count);
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1351 Block_List worklist;
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1352 int new_count = 0;
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1353
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1354 // Compact the traces.
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1355 for (int i = 0; i < count; i++) {
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1356 Trace *tr = traces[i];
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1357 if (tr != NULL) {
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1358 new_traces[new_count++] = tr;
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1359 }
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1360 }
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1361
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1362 // The entry block should be first on the new trace list.
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1363 Trace *tr = trace(_cfg._broot);
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1364 assert(tr == new_traces[0], "entry trace misplaced");
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1365
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1366 // Sort the new trace list by frequency
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1367 qsort(new_traces + 1, new_count - 1, sizeof(new_traces[0]), trace_frequency_order);
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1368
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1369 // Patch up the successor blocks
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1370 _cfg._blocks.reset();
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1371 _cfg._num_blocks = 0;
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1372 for (int i = 0; i < new_count; i++) {
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1373 Trace *tr = new_traces[i];
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1374 if (tr != NULL) {
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1375 tr->fixup_blocks(_cfg);
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1376 }
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1377 }
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1378 }
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1379
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1380 //------------------------------PhaseBlockLayout-------------------------------
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1381 // Order basic blocks based on frequency
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1382 PhaseBlockLayout::PhaseBlockLayout(PhaseCFG &cfg) :
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1383 Phase(BlockLayout),
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1384 _cfg(cfg)
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1385 {
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1386 ResourceMark rm;
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1387 ResourceArea *area = Thread::current()->resource_area();
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1388
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1389 // List of traces
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1390 int size = _cfg._num_blocks + 1;
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1391 traces = NEW_ARENA_ARRAY(area, Trace *, size);
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1392 memset(traces, 0, size*sizeof(Trace*));
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1393 next = NEW_ARENA_ARRAY(area, Block *, size);
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1394 memset(next, 0, size*sizeof(Block *));
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1395 prev = NEW_ARENA_ARRAY(area, Block *, size);
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1396 memset(prev , 0, size*sizeof(Block *));
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1397
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1398 // List of edges
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1399 edges = new GrowableArray<CFGEdge*>;
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1400
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1401 // Mapping block index --> block_trace
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1402 uf = new UnionFind(size);
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1403 uf->reset(size);
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1404
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1405 // Find edges and create traces.
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1406 find_edges();
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1407
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1408 // Grow traces at their ends via most frequent edges.
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1409 grow_traces();
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1410
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1411 // Merge one trace into another, but only at fall-through points.
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1412 // This may make diamonds and other related shapes in a trace.
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1413 merge_traces(true);
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1414
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1415 // Run merge again, allowing two traces to be catenated, even if
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1416 // one does not fall through into the other. This appends loosely
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1417 // related traces to be near each other.
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1418 merge_traces(false);
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1419
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1420 // Re-order all the remaining traces by frequency
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1421 reorder_traces(size);
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1422
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1423 assert(_cfg._num_blocks >= (uint) (size - 1), "number of blocks can not shrink");
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1424 }
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1425
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1426
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1427 //------------------------------backedge---------------------------------------
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1428 // Edge e completes a loop in a trace. If the target block is head of the
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1429 // loop, rotate the loop block so that the loop ends in a conditional branch.
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1430 bool Trace::backedge(CFGEdge *e) {
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1431 bool loop_rotated = false;
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1432 Block *src_block = e->from();
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1433 Block *targ_block = e->to();
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1434
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1435 assert(last_block() == src_block, "loop discovery at back branch");
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1436 if (first_block() == targ_block) {
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1437 if (BlockLayoutRotateLoops && last_block()->num_fall_throughs() < 2) {
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1438 // Find the last block in the trace that has a conditional
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1439 // branch.
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1440 Block *b;
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1441 for (b = last_block(); b != NULL; b = prev(b)) {
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1442 if (b->num_fall_throughs() == 2) {
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1443 break;
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1444 }
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1445 }
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1446
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1447 if (b != last_block() && b != NULL) {
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1448 loop_rotated = true;
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1449
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1450 // Rotate the loop by doing two-part linked-list surgery.
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1451 append(first_block());
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1452 break_loop_after(b);
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1453 }
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1454 }
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1455
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1456 // Backbranch to the top of a trace
605
98cb887364d3 6810672: Comment typos
twisti
parents: 601
diff changeset
1457 // Scroll forward through the trace from the targ_block. If we find
418
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1458 // a loop head before another loop top, use the the loop head alignment.
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1459 for (Block *b = targ_block; b != NULL; b = next(b)) {
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1460 if (b->has_loop_alignment()) {
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1461 break;
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1462 }
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1463 if (b->head()->is_Loop()) {
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1464 targ_block = b;
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1465 break;
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1466 }
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1467 }
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1468
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1469 first_block()->set_loop_alignment(targ_block);
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1470
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1471 } else {
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1472 // Backbranch into the middle of a trace
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1473 targ_block->set_loop_alignment(targ_block);
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1474 }
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1475
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1476 return loop_rotated;
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1477 }
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1478
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1479 //------------------------------fixup_blocks-----------------------------------
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1480 // push blocks onto the CFG list
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1481 // ensure that blocks have the correct two-way branch sense
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1482 void Trace::fixup_blocks(PhaseCFG &cfg) {
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1483 Block *last = last_block();
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1484 for (Block *b = first_block(); b != NULL; b = next(b)) {
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1485 cfg._blocks.push(b);
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1486 cfg._num_blocks++;
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1487 if (!b->is_connector()) {
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1488 int nfallthru = b->num_fall_throughs();
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1489 if (b != last) {
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1490 if (nfallthru == 2) {
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1491 // Ensure that the sense of the branch is correct
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1492 Block *bnext = next(b);
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1493 Block *bs0 = b->non_connector_successor(0);
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1494
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1495 MachNode *iff = b->_nodes[b->_nodes.size()-3]->as_Mach();
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1496 ProjNode *proj0 = b->_nodes[b->_nodes.size()-2]->as_Proj();
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1497 ProjNode *proj1 = b->_nodes[b->_nodes.size()-1]->as_Proj();
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1498
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1499 if (bnext == bs0) {
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1500 // Fall-thru case in succs[0], should be in succs[1]
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1501
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1502 // Flip targets in _succs map
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1503 Block *tbs0 = b->_succs[0];
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1504 Block *tbs1 = b->_succs[1];
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1505 b->_succs.map( 0, tbs1 );
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1506 b->_succs.map( 1, tbs0 );
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1507
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1508 // Flip projections to match targets
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1509 b->_nodes.map(b->_nodes.size()-2, proj1);
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1510 b->_nodes.map(b->_nodes.size()-1, proj0);
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1511 }
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1512 }
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1513 }
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1514 }
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1515 }
72c5366e5d86 6743900: frequency based block layout
rasbold
parents: 337
diff changeset
1516 }