annotate src/share/vm/opto/block.cpp @ 1994:6cd6d394f280

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