annotate src/share/vm/c1/c1_ValueMap.cpp @ 20543:e7d0505c8a30

8059758: Footprint regressions with JDK-8038423 Summary: Changes in JDK-8038423 always initialize (zero out) virtual memory used for auxiliary data structures. This causes a footprint regression for G1 in startup benchmarks. This is because they do not touch that memory at all, so the operating system does not actually commit these pages. The fix is to, if the initialization value of the data structures matches the default value of just committed memory (=0), do not do anything. Reviewed-by: jwilhelm, brutisso
author tschatzl
date Fri, 10 Oct 2014 15:51:58 +0200
parents 55fb97c4c58d
children 4ca6dc0799b6
Ignore whitespace changes - Everywhere: Within whitespace: At end of lines:
rev   line source
0
a61af66fc99e Initial load
duke
parents:
diff changeset
1 /*
17467
55fb97c4c58d 8029233: Update copyright year to match last edit in jdk8 hotspot repository for 2013
mikael
parents: 9081
diff changeset
2 * Copyright (c) 1999, 2013, 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: 0
diff changeset
19 * Please contact Oracle, 500 Oracle Parkway, Redwood Shores, CA 94065 USA
c18cbe5936b8 6941466: Oracle rebranding changes for Hotspot repositories
trims
parents: 0
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: 0
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: 1552
diff changeset
25 #include "precompiled.hpp"
f95d63e2154a 6989984: Use standard include model for Hospot
stefank
parents: 1552
diff changeset
26 #include "c1/c1_Canonicalizer.hpp"
f95d63e2154a 6989984: Use standard include model for Hospot
stefank
parents: 1552
diff changeset
27 #include "c1/c1_IR.hpp"
f95d63e2154a 6989984: Use standard include model for Hospot
stefank
parents: 1552
diff changeset
28 #include "c1/c1_ValueMap.hpp"
8860
46f6f063b272 7153771: array bound check elimination for c1
roland
parents: 6842
diff changeset
29 #include "c1/c1_ValueStack.hpp"
1972
f95d63e2154a 6989984: Use standard include model for Hospot
stefank
parents: 1552
diff changeset
30 #include "utilities/bitMap.inline.hpp"
0
a61af66fc99e Initial load
duke
parents:
diff changeset
31
a61af66fc99e Initial load
duke
parents:
diff changeset
32 #ifndef PRODUCT
a61af66fc99e Initial load
duke
parents:
diff changeset
33
a61af66fc99e Initial load
duke
parents:
diff changeset
34 int ValueMap::_number_of_finds = 0;
a61af66fc99e Initial load
duke
parents:
diff changeset
35 int ValueMap::_number_of_hits = 0;
a61af66fc99e Initial load
duke
parents:
diff changeset
36 int ValueMap::_number_of_kills = 0;
a61af66fc99e Initial load
duke
parents:
diff changeset
37
a61af66fc99e Initial load
duke
parents:
diff changeset
38 #define TRACE_VALUE_NUMBERING(code) if (PrintValueNumbering) { code; }
a61af66fc99e Initial load
duke
parents:
diff changeset
39
a61af66fc99e Initial load
duke
parents:
diff changeset
40 #else
a61af66fc99e Initial load
duke
parents:
diff changeset
41
a61af66fc99e Initial load
duke
parents:
diff changeset
42 #define TRACE_VALUE_NUMBERING(code)
a61af66fc99e Initial load
duke
parents:
diff changeset
43
a61af66fc99e Initial load
duke
parents:
diff changeset
44 #endif
a61af66fc99e Initial load
duke
parents:
diff changeset
45
a61af66fc99e Initial load
duke
parents:
diff changeset
46
a61af66fc99e Initial load
duke
parents:
diff changeset
47 ValueMap::ValueMap()
a61af66fc99e Initial load
duke
parents:
diff changeset
48 : _nesting(0)
a61af66fc99e Initial load
duke
parents:
diff changeset
49 , _entries(ValueMapInitialSize, NULL)
a61af66fc99e Initial load
duke
parents:
diff changeset
50 , _killed_values()
a61af66fc99e Initial load
duke
parents:
diff changeset
51 , _entry_count(0)
a61af66fc99e Initial load
duke
parents:
diff changeset
52 {
a61af66fc99e Initial load
duke
parents:
diff changeset
53 NOT_PRODUCT(reset_statistics());
a61af66fc99e Initial load
duke
parents:
diff changeset
54 }
a61af66fc99e Initial load
duke
parents:
diff changeset
55
a61af66fc99e Initial load
duke
parents:
diff changeset
56
a61af66fc99e Initial load
duke
parents:
diff changeset
57 ValueMap::ValueMap(ValueMap* old)
a61af66fc99e Initial load
duke
parents:
diff changeset
58 : _nesting(old->_nesting + 1)
a61af66fc99e Initial load
duke
parents:
diff changeset
59 , _entries(old->_entries.length())
a61af66fc99e Initial load
duke
parents:
diff changeset
60 , _killed_values()
a61af66fc99e Initial load
duke
parents:
diff changeset
61 , _entry_count(old->_entry_count)
a61af66fc99e Initial load
duke
parents:
diff changeset
62 {
a61af66fc99e Initial load
duke
parents:
diff changeset
63 for (int i = size() - 1; i >= 0; i--) {
a61af66fc99e Initial load
duke
parents:
diff changeset
64 _entries.at_put(i, old->entry_at(i));
a61af66fc99e Initial load
duke
parents:
diff changeset
65 }
a61af66fc99e Initial load
duke
parents:
diff changeset
66 _killed_values.set_from(&old->_killed_values);
a61af66fc99e Initial load
duke
parents:
diff changeset
67 }
a61af66fc99e Initial load
duke
parents:
diff changeset
68
a61af66fc99e Initial load
duke
parents:
diff changeset
69
a61af66fc99e Initial load
duke
parents:
diff changeset
70 void ValueMap::increase_table_size() {
a61af66fc99e Initial load
duke
parents:
diff changeset
71 int old_size = size();
a61af66fc99e Initial load
duke
parents:
diff changeset
72 int new_size = old_size * 2 + 1;
a61af66fc99e Initial load
duke
parents:
diff changeset
73
a61af66fc99e Initial load
duke
parents:
diff changeset
74 ValueMapEntryList worklist(8);
a61af66fc99e Initial load
duke
parents:
diff changeset
75 ValueMapEntryArray new_entries(new_size, NULL);
a61af66fc99e Initial load
duke
parents:
diff changeset
76 int new_entry_count = 0;
a61af66fc99e Initial load
duke
parents:
diff changeset
77
a61af66fc99e Initial load
duke
parents:
diff changeset
78 TRACE_VALUE_NUMBERING(tty->print_cr("increasing table size from %d to %d", old_size, new_size));
a61af66fc99e Initial load
duke
parents:
diff changeset
79
a61af66fc99e Initial load
duke
parents:
diff changeset
80 for (int i = old_size - 1; i >= 0; i--) {
a61af66fc99e Initial load
duke
parents:
diff changeset
81 ValueMapEntry* entry;
a61af66fc99e Initial load
duke
parents:
diff changeset
82 for (entry = entry_at(i); entry != NULL; entry = entry->next()) {
a61af66fc99e Initial load
duke
parents:
diff changeset
83 if (!is_killed(entry->value())) {
a61af66fc99e Initial load
duke
parents:
diff changeset
84 worklist.push(entry);
a61af66fc99e Initial load
duke
parents:
diff changeset
85 }
a61af66fc99e Initial load
duke
parents:
diff changeset
86 }
a61af66fc99e Initial load
duke
parents:
diff changeset
87
a61af66fc99e Initial load
duke
parents:
diff changeset
88 while (!worklist.is_empty()) {
a61af66fc99e Initial load
duke
parents:
diff changeset
89 entry = worklist.pop();
a61af66fc99e Initial load
duke
parents:
diff changeset
90 int new_index = entry_index(entry->hash(), new_size);
a61af66fc99e Initial load
duke
parents:
diff changeset
91
a61af66fc99e Initial load
duke
parents:
diff changeset
92 if (entry->nesting() != nesting() && new_entries.at(new_index) != entry->next()) {
a61af66fc99e Initial load
duke
parents:
diff changeset
93 // changing entries with a lower nesting than the current nesting of the table
a61af66fc99e Initial load
duke
parents:
diff changeset
94 // is not allowed because then the same entry is contained in multiple value maps.
a61af66fc99e Initial load
duke
parents:
diff changeset
95 // clone entry when next-pointer must be changed
a61af66fc99e Initial load
duke
parents:
diff changeset
96 entry = new ValueMapEntry(entry->hash(), entry->value(), entry->nesting(), NULL);
a61af66fc99e Initial load
duke
parents:
diff changeset
97 }
a61af66fc99e Initial load
duke
parents:
diff changeset
98 entry->set_next(new_entries.at(new_index));
a61af66fc99e Initial load
duke
parents:
diff changeset
99 new_entries.at_put(new_index, entry);
a61af66fc99e Initial load
duke
parents:
diff changeset
100 new_entry_count++;
a61af66fc99e Initial load
duke
parents:
diff changeset
101 }
a61af66fc99e Initial load
duke
parents:
diff changeset
102 }
a61af66fc99e Initial load
duke
parents:
diff changeset
103
a61af66fc99e Initial load
duke
parents:
diff changeset
104 _entries = new_entries;
a61af66fc99e Initial load
duke
parents:
diff changeset
105 _entry_count = new_entry_count;
a61af66fc99e Initial load
duke
parents:
diff changeset
106 }
a61af66fc99e Initial load
duke
parents:
diff changeset
107
a61af66fc99e Initial load
duke
parents:
diff changeset
108
a61af66fc99e Initial load
duke
parents:
diff changeset
109 Value ValueMap::find_insert(Value x) {
a61af66fc99e Initial load
duke
parents:
diff changeset
110 const intx hash = x->hash();
a61af66fc99e Initial load
duke
parents:
diff changeset
111 if (hash != 0) {
a61af66fc99e Initial load
duke
parents:
diff changeset
112 // 0 hash means: exclude from value numbering
a61af66fc99e Initial load
duke
parents:
diff changeset
113 NOT_PRODUCT(_number_of_finds++);
a61af66fc99e Initial load
duke
parents:
diff changeset
114
a61af66fc99e Initial load
duke
parents:
diff changeset
115 for (ValueMapEntry* entry = entry_at(entry_index(hash, size())); entry != NULL; entry = entry->next()) {
a61af66fc99e Initial load
duke
parents:
diff changeset
116 if (entry->hash() == hash) {
a61af66fc99e Initial load
duke
parents:
diff changeset
117 Value f = entry->value();
a61af66fc99e Initial load
duke
parents:
diff changeset
118
a61af66fc99e Initial load
duke
parents:
diff changeset
119 if (!is_killed(f) && f->is_equal(x)) {
a61af66fc99e Initial load
duke
parents:
diff changeset
120 NOT_PRODUCT(_number_of_hits++);
a61af66fc99e Initial load
duke
parents:
diff changeset
121 TRACE_VALUE_NUMBERING(tty->print_cr("Value Numbering: %s %c%d equal to %c%d (size %d, entries %d, nesting-diff %d)", x->name(), x->type()->tchar(), x->id(), f->type()->tchar(), f->id(), size(), entry_count(), nesting() - entry->nesting()));
a61af66fc99e Initial load
duke
parents:
diff changeset
122
a61af66fc99e Initial load
duke
parents:
diff changeset
123 if (entry->nesting() != nesting() && f->as_Constant() == NULL) {
a61af66fc99e Initial load
duke
parents:
diff changeset
124 // non-constant values of of another block must be pinned,
a61af66fc99e Initial load
duke
parents:
diff changeset
125 // otherwise it is possible that they are not evaluated
a61af66fc99e Initial load
duke
parents:
diff changeset
126 f->pin(Instruction::PinGlobalValueNumbering);
a61af66fc99e Initial load
duke
parents:
diff changeset
127 }
4871
f067b4e0e04b 7090976: Eclipse/CDT causes a JVM crash while indexing C++ code
roland
parents: 1972
diff changeset
128 assert(x->type()->tag() == f->type()->tag(), "should have same type");
0
a61af66fc99e Initial load
duke
parents:
diff changeset
129
a61af66fc99e Initial load
duke
parents:
diff changeset
130 return f;
a61af66fc99e Initial load
duke
parents:
diff changeset
131
a61af66fc99e Initial load
duke
parents:
diff changeset
132 }
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 // x not found, so insert it
a61af66fc99e Initial load
duke
parents:
diff changeset
137 if (entry_count() >= size_threshold()) {
a61af66fc99e Initial load
duke
parents:
diff changeset
138 increase_table_size();
a61af66fc99e Initial load
duke
parents:
diff changeset
139 }
a61af66fc99e Initial load
duke
parents:
diff changeset
140 int idx = entry_index(hash, size());
a61af66fc99e Initial load
duke
parents:
diff changeset
141 _entries.at_put(idx, new ValueMapEntry(hash, x, nesting(), entry_at(idx)));
a61af66fc99e Initial load
duke
parents:
diff changeset
142 _entry_count++;
a61af66fc99e Initial load
duke
parents:
diff changeset
143
a61af66fc99e Initial load
duke
parents:
diff changeset
144 TRACE_VALUE_NUMBERING(tty->print_cr("Value Numbering: insert %s %c%d (size %d, entries %d, nesting %d)", x->name(), x->type()->tchar(), x->id(), size(), entry_count(), nesting()));
a61af66fc99e Initial load
duke
parents:
diff changeset
145 }
a61af66fc99e Initial load
duke
parents:
diff changeset
146
a61af66fc99e Initial load
duke
parents:
diff changeset
147 return x;
a61af66fc99e Initial load
duke
parents:
diff changeset
148 }
a61af66fc99e Initial load
duke
parents:
diff changeset
149
a61af66fc99e Initial load
duke
parents:
diff changeset
150
a61af66fc99e Initial load
duke
parents:
diff changeset
151 #define GENERIC_KILL_VALUE(must_kill_implementation) \
a61af66fc99e Initial load
duke
parents:
diff changeset
152 NOT_PRODUCT(_number_of_kills++); \
a61af66fc99e Initial load
duke
parents:
diff changeset
153 \
a61af66fc99e Initial load
duke
parents:
diff changeset
154 for (int i = size() - 1; i >= 0; i--) { \
a61af66fc99e Initial load
duke
parents:
diff changeset
155 ValueMapEntry* prev_entry = NULL; \
a61af66fc99e Initial load
duke
parents:
diff changeset
156 for (ValueMapEntry* entry = entry_at(i); entry != NULL; entry = entry->next()) { \
a61af66fc99e Initial load
duke
parents:
diff changeset
157 Value value = entry->value(); \
a61af66fc99e Initial load
duke
parents:
diff changeset
158 \
a61af66fc99e Initial load
duke
parents:
diff changeset
159 must_kill_implementation(must_kill, entry, value) \
a61af66fc99e Initial load
duke
parents:
diff changeset
160 \
a61af66fc99e Initial load
duke
parents:
diff changeset
161 if (must_kill) { \
a61af66fc99e Initial load
duke
parents:
diff changeset
162 kill_value(value); \
a61af66fc99e Initial load
duke
parents:
diff changeset
163 \
a61af66fc99e Initial load
duke
parents:
diff changeset
164 if (prev_entry == NULL) { \
a61af66fc99e Initial load
duke
parents:
diff changeset
165 _entries.at_put(i, entry->next()); \
a61af66fc99e Initial load
duke
parents:
diff changeset
166 _entry_count--; \
a61af66fc99e Initial load
duke
parents:
diff changeset
167 } else if (prev_entry->nesting() == nesting()) { \
a61af66fc99e Initial load
duke
parents:
diff changeset
168 prev_entry->set_next(entry->next()); \
a61af66fc99e Initial load
duke
parents:
diff changeset
169 _entry_count--; \
a61af66fc99e Initial load
duke
parents:
diff changeset
170 } else { \
a61af66fc99e Initial load
duke
parents:
diff changeset
171 prev_entry = entry; \
a61af66fc99e Initial load
duke
parents:
diff changeset
172 } \
a61af66fc99e Initial load
duke
parents:
diff changeset
173 \
a61af66fc99e Initial load
duke
parents:
diff changeset
174 TRACE_VALUE_NUMBERING(tty->print_cr("Value Numbering: killed %s %c%d (size %d, entries %d, nesting-diff %d)", value->name(), value->type()->tchar(), value->id(), size(), entry_count(), nesting() - entry->nesting())); \
a61af66fc99e Initial load
duke
parents:
diff changeset
175 } else { \
a61af66fc99e Initial load
duke
parents:
diff changeset
176 prev_entry = entry; \
a61af66fc99e Initial load
duke
parents:
diff changeset
177 } \
a61af66fc99e Initial load
duke
parents:
diff changeset
178 } \
a61af66fc99e Initial load
duke
parents:
diff changeset
179 } \
a61af66fc99e Initial load
duke
parents:
diff changeset
180
a61af66fc99e Initial load
duke
parents:
diff changeset
181 #define MUST_KILL_MEMORY(must_kill, entry, value) \
a61af66fc99e Initial load
duke
parents:
diff changeset
182 bool must_kill = value->as_LoadField() != NULL || value->as_LoadIndexed() != NULL;
a61af66fc99e Initial load
duke
parents:
diff changeset
183
a61af66fc99e Initial load
duke
parents:
diff changeset
184 #define MUST_KILL_ARRAY(must_kill, entry, value) \
a61af66fc99e Initial load
duke
parents:
diff changeset
185 bool must_kill = value->as_LoadIndexed() != NULL \
a61af66fc99e Initial load
duke
parents:
diff changeset
186 && value->type()->tag() == type->tag();
a61af66fc99e Initial load
duke
parents:
diff changeset
187
a61af66fc99e Initial load
duke
parents:
diff changeset
188 #define MUST_KILL_FIELD(must_kill, entry, value) \
a61af66fc99e Initial load
duke
parents:
diff changeset
189 /* ciField's are not unique; must compare their contents */ \
a61af66fc99e Initial load
duke
parents:
diff changeset
190 LoadField* lf = value->as_LoadField(); \
a61af66fc99e Initial load
duke
parents:
diff changeset
191 bool must_kill = lf != NULL \
a61af66fc99e Initial load
duke
parents:
diff changeset
192 && lf->field()->holder() == field->holder() \
6618
0bfcb7a3e12d 7171824: assert(_offset >= 1) failed: illegal call to offset()
roland
parents: 4871
diff changeset
193 && (all_offsets || lf->field()->offset() == field->offset());
0
a61af66fc99e Initial load
duke
parents:
diff changeset
194
a61af66fc99e Initial load
duke
parents:
diff changeset
195
a61af66fc99e Initial load
duke
parents:
diff changeset
196 void ValueMap::kill_memory() {
a61af66fc99e Initial load
duke
parents:
diff changeset
197 GENERIC_KILL_VALUE(MUST_KILL_MEMORY);
a61af66fc99e Initial load
duke
parents:
diff changeset
198 }
a61af66fc99e Initial load
duke
parents:
diff changeset
199
a61af66fc99e Initial load
duke
parents:
diff changeset
200 void ValueMap::kill_array(ValueType* type) {
a61af66fc99e Initial load
duke
parents:
diff changeset
201 GENERIC_KILL_VALUE(MUST_KILL_ARRAY);
a61af66fc99e Initial load
duke
parents:
diff changeset
202 }
a61af66fc99e Initial load
duke
parents:
diff changeset
203
6618
0bfcb7a3e12d 7171824: assert(_offset >= 1) failed: illegal call to offset()
roland
parents: 4871
diff changeset
204 void ValueMap::kill_field(ciField* field, bool all_offsets) {
0
a61af66fc99e Initial load
duke
parents:
diff changeset
205 GENERIC_KILL_VALUE(MUST_KILL_FIELD);
a61af66fc99e Initial load
duke
parents:
diff changeset
206 }
a61af66fc99e Initial load
duke
parents:
diff changeset
207
a61af66fc99e Initial load
duke
parents:
diff changeset
208 void ValueMap::kill_map(ValueMap* map) {
a61af66fc99e Initial load
duke
parents:
diff changeset
209 assert(is_global_value_numbering(), "only for global value numbering");
a61af66fc99e Initial load
duke
parents:
diff changeset
210 _killed_values.set_union(&map->_killed_values);
a61af66fc99e Initial load
duke
parents:
diff changeset
211 }
a61af66fc99e Initial load
duke
parents:
diff changeset
212
a61af66fc99e Initial load
duke
parents:
diff changeset
213 void ValueMap::kill_all() {
a61af66fc99e Initial load
duke
parents:
diff changeset
214 assert(is_local_value_numbering(), "only for local value numbering");
a61af66fc99e Initial load
duke
parents:
diff changeset
215 for (int i = size() - 1; i >= 0; i--) {
a61af66fc99e Initial load
duke
parents:
diff changeset
216 _entries.at_put(i, NULL);
a61af66fc99e Initial load
duke
parents:
diff changeset
217 }
a61af66fc99e Initial load
duke
parents:
diff changeset
218 _entry_count = 0;
a61af66fc99e Initial load
duke
parents:
diff changeset
219 }
a61af66fc99e Initial load
duke
parents:
diff changeset
220
a61af66fc99e Initial load
duke
parents:
diff changeset
221
a61af66fc99e Initial load
duke
parents:
diff changeset
222 #ifndef PRODUCT
a61af66fc99e Initial load
duke
parents:
diff changeset
223
a61af66fc99e Initial load
duke
parents:
diff changeset
224 void ValueMap::print() {
a61af66fc99e Initial load
duke
parents:
diff changeset
225 tty->print_cr("(size %d, entries %d, nesting %d)", size(), entry_count(), nesting());
a61af66fc99e Initial load
duke
parents:
diff changeset
226
a61af66fc99e Initial load
duke
parents:
diff changeset
227 int entries = 0;
a61af66fc99e Initial load
duke
parents:
diff changeset
228 for (int i = 0; i < size(); i++) {
a61af66fc99e Initial load
duke
parents:
diff changeset
229 if (entry_at(i) != NULL) {
a61af66fc99e Initial load
duke
parents:
diff changeset
230 tty->print(" %2d: ", i);
a61af66fc99e Initial load
duke
parents:
diff changeset
231 for (ValueMapEntry* entry = entry_at(i); entry != NULL; entry = entry->next()) {
a61af66fc99e Initial load
duke
parents:
diff changeset
232 Value value = entry->value();
a61af66fc99e Initial load
duke
parents:
diff changeset
233 tty->print("%s %c%d (%s%d) -> ", value->name(), value->type()->tchar(), value->id(), is_killed(value) ? "x" : "", entry->nesting());
a61af66fc99e Initial load
duke
parents:
diff changeset
234 entries++;
a61af66fc99e Initial load
duke
parents:
diff changeset
235 }
a61af66fc99e Initial load
duke
parents:
diff changeset
236 tty->print_cr("NULL");
a61af66fc99e Initial load
duke
parents:
diff changeset
237 }
a61af66fc99e Initial load
duke
parents:
diff changeset
238 }
a61af66fc99e Initial load
duke
parents:
diff changeset
239
a61af66fc99e Initial load
duke
parents:
diff changeset
240 _killed_values.print();
a61af66fc99e Initial load
duke
parents:
diff changeset
241 assert(entry_count() == entries, "entry_count incorrect");
a61af66fc99e Initial load
duke
parents:
diff changeset
242 }
a61af66fc99e Initial load
duke
parents:
diff changeset
243
a61af66fc99e Initial load
duke
parents:
diff changeset
244 void ValueMap::reset_statistics() {
a61af66fc99e Initial load
duke
parents:
diff changeset
245 _number_of_finds = 0;
a61af66fc99e Initial load
duke
parents:
diff changeset
246 _number_of_hits = 0;
a61af66fc99e Initial load
duke
parents:
diff changeset
247 _number_of_kills = 0;
a61af66fc99e Initial load
duke
parents:
diff changeset
248 }
a61af66fc99e Initial load
duke
parents:
diff changeset
249
a61af66fc99e Initial load
duke
parents:
diff changeset
250 void ValueMap::print_statistics() {
a61af66fc99e Initial load
duke
parents:
diff changeset
251 float hit_rate = 0;
a61af66fc99e Initial load
duke
parents:
diff changeset
252 if (_number_of_finds != 0) {
a61af66fc99e Initial load
duke
parents:
diff changeset
253 hit_rate = (float)_number_of_hits / _number_of_finds;
a61af66fc99e Initial load
duke
parents:
diff changeset
254 }
a61af66fc99e Initial load
duke
parents:
diff changeset
255
a61af66fc99e Initial load
duke
parents:
diff changeset
256 tty->print_cr("finds:%3d hits:%3d kills:%3d hit rate: %1.4f", _number_of_finds, _number_of_hits, _number_of_kills, hit_rate);
a61af66fc99e Initial load
duke
parents:
diff changeset
257 }
a61af66fc99e Initial load
duke
parents:
diff changeset
258
a61af66fc99e Initial load
duke
parents:
diff changeset
259 #endif
a61af66fc99e Initial load
duke
parents:
diff changeset
260
a61af66fc99e Initial load
duke
parents:
diff changeset
261
a61af66fc99e Initial load
duke
parents:
diff changeset
262
a61af66fc99e Initial load
duke
parents:
diff changeset
263 class ShortLoopOptimizer : public ValueNumberingVisitor {
a61af66fc99e Initial load
duke
parents:
diff changeset
264 private:
a61af66fc99e Initial load
duke
parents:
diff changeset
265 GlobalValueNumbering* _gvn;
a61af66fc99e Initial load
duke
parents:
diff changeset
266 BlockList _loop_blocks;
a61af66fc99e Initial load
duke
parents:
diff changeset
267 bool _too_complicated_loop;
8860
46f6f063b272 7153771: array bound check elimination for c1
roland
parents: 6842
diff changeset
268 bool _has_field_store[T_ARRAY + 1];
46f6f063b272 7153771: array bound check elimination for c1
roland
parents: 6842
diff changeset
269 bool _has_indexed_store[T_ARRAY + 1];
0
a61af66fc99e Initial load
duke
parents:
diff changeset
270
a61af66fc99e Initial load
duke
parents:
diff changeset
271 // simplified access to methods of GlobalValueNumbering
a61af66fc99e Initial load
duke
parents:
diff changeset
272 ValueMap* current_map() { return _gvn->current_map(); }
a61af66fc99e Initial load
duke
parents:
diff changeset
273 ValueMap* value_map_of(BlockBegin* block) { return _gvn->value_map_of(block); }
a61af66fc99e Initial load
duke
parents:
diff changeset
274
a61af66fc99e Initial load
duke
parents:
diff changeset
275 // implementation for abstract methods of ValueNumberingVisitor
6618
0bfcb7a3e12d 7171824: assert(_offset >= 1) failed: illegal call to offset()
roland
parents: 4871
diff changeset
276 void kill_memory() { _too_complicated_loop = true; }
8860
46f6f063b272 7153771: array bound check elimination for c1
roland
parents: 6842
diff changeset
277 void kill_field(ciField* field, bool all_offsets) {
46f6f063b272 7153771: array bound check elimination for c1
roland
parents: 6842
diff changeset
278 current_map()->kill_field(field, all_offsets);
46f6f063b272 7153771: array bound check elimination for c1
roland
parents: 6842
diff changeset
279 assert(field->type()->basic_type() >= 0 && field->type()->basic_type() <= T_ARRAY, "Invalid type");
46f6f063b272 7153771: array bound check elimination for c1
roland
parents: 6842
diff changeset
280 _has_field_store[field->type()->basic_type()] = true;
46f6f063b272 7153771: array bound check elimination for c1
roland
parents: 6842
diff changeset
281 }
46f6f063b272 7153771: array bound check elimination for c1
roland
parents: 6842
diff changeset
282 void kill_array(ValueType* type) {
46f6f063b272 7153771: array bound check elimination for c1
roland
parents: 6842
diff changeset
283 current_map()->kill_array(type);
46f6f063b272 7153771: array bound check elimination for c1
roland
parents: 6842
diff changeset
284 BasicType basic_type = as_BasicType(type); assert(basic_type >= 0 && basic_type <= T_ARRAY, "Invalid type");
46f6f063b272 7153771: array bound check elimination for c1
roland
parents: 6842
diff changeset
285 _has_indexed_store[basic_type] = true;
46f6f063b272 7153771: array bound check elimination for c1
roland
parents: 6842
diff changeset
286 }
0
a61af66fc99e Initial load
duke
parents:
diff changeset
287
a61af66fc99e Initial load
duke
parents:
diff changeset
288 public:
a61af66fc99e Initial load
duke
parents:
diff changeset
289 ShortLoopOptimizer(GlobalValueNumbering* gvn)
a61af66fc99e Initial load
duke
parents:
diff changeset
290 : _gvn(gvn)
a61af66fc99e Initial load
duke
parents:
diff changeset
291 , _loop_blocks(ValueMapMaxLoopSize)
a61af66fc99e Initial load
duke
parents:
diff changeset
292 , _too_complicated_loop(false)
a61af66fc99e Initial load
duke
parents:
diff changeset
293 {
8860
46f6f063b272 7153771: array bound check elimination for c1
roland
parents: 6842
diff changeset
294 for (int i=0; i<= T_ARRAY; i++){
46f6f063b272 7153771: array bound check elimination for c1
roland
parents: 6842
diff changeset
295 _has_field_store[i] = false;
46f6f063b272 7153771: array bound check elimination for c1
roland
parents: 6842
diff changeset
296 _has_indexed_store[i] = false;
46f6f063b272 7153771: array bound check elimination for c1
roland
parents: 6842
diff changeset
297 }
46f6f063b272 7153771: array bound check elimination for c1
roland
parents: 6842
diff changeset
298 }
46f6f063b272 7153771: array bound check elimination for c1
roland
parents: 6842
diff changeset
299
46f6f063b272 7153771: array bound check elimination for c1
roland
parents: 6842
diff changeset
300 bool has_field_store(BasicType type) {
46f6f063b272 7153771: array bound check elimination for c1
roland
parents: 6842
diff changeset
301 assert(type >= 0 && type <= T_ARRAY, "Invalid type");
46f6f063b272 7153771: array bound check elimination for c1
roland
parents: 6842
diff changeset
302 return _has_field_store[type];
46f6f063b272 7153771: array bound check elimination for c1
roland
parents: 6842
diff changeset
303 }
46f6f063b272 7153771: array bound check elimination for c1
roland
parents: 6842
diff changeset
304
46f6f063b272 7153771: array bound check elimination for c1
roland
parents: 6842
diff changeset
305 bool has_indexed_store(BasicType type) {
46f6f063b272 7153771: array bound check elimination for c1
roland
parents: 6842
diff changeset
306 assert(type >= 0 && type <= T_ARRAY, "Invalid type");
46f6f063b272 7153771: array bound check elimination for c1
roland
parents: 6842
diff changeset
307 return _has_indexed_store[type];
0
a61af66fc99e Initial load
duke
parents:
diff changeset
308 }
a61af66fc99e Initial load
duke
parents:
diff changeset
309
a61af66fc99e Initial load
duke
parents:
diff changeset
310 bool process(BlockBegin* loop_header);
a61af66fc99e Initial load
duke
parents:
diff changeset
311 };
a61af66fc99e Initial load
duke
parents:
diff changeset
312
8860
46f6f063b272 7153771: array bound check elimination for c1
roland
parents: 6842
diff changeset
313 class LoopInvariantCodeMotion : public StackObj {
46f6f063b272 7153771: array bound check elimination for c1
roland
parents: 6842
diff changeset
314 private:
46f6f063b272 7153771: array bound check elimination for c1
roland
parents: 6842
diff changeset
315 GlobalValueNumbering* _gvn;
46f6f063b272 7153771: array bound check elimination for c1
roland
parents: 6842
diff changeset
316 ShortLoopOptimizer* _short_loop_optimizer;
46f6f063b272 7153771: array bound check elimination for c1
roland
parents: 6842
diff changeset
317 Instruction* _insertion_point;
46f6f063b272 7153771: array bound check elimination for c1
roland
parents: 6842
diff changeset
318 ValueStack * _state;
9081
84ab5667f290 8011706: specjvm2008 test xml.transform gets array bound exception with c1
roland
parents: 8860
diff changeset
319 bool _insert_is_pred;
8860
46f6f063b272 7153771: array bound check elimination for c1
roland
parents: 6842
diff changeset
320
46f6f063b272 7153771: array bound check elimination for c1
roland
parents: 6842
diff changeset
321 void set_invariant(Value v) const { _gvn->set_processed(v); }
46f6f063b272 7153771: array bound check elimination for c1
roland
parents: 6842
diff changeset
322 bool is_invariant(Value v) const { return _gvn->is_processed(v); }
46f6f063b272 7153771: array bound check elimination for c1
roland
parents: 6842
diff changeset
323
46f6f063b272 7153771: array bound check elimination for c1
roland
parents: 6842
diff changeset
324 void process_block(BlockBegin* block);
46f6f063b272 7153771: array bound check elimination for c1
roland
parents: 6842
diff changeset
325
46f6f063b272 7153771: array bound check elimination for c1
roland
parents: 6842
diff changeset
326 public:
46f6f063b272 7153771: array bound check elimination for c1
roland
parents: 6842
diff changeset
327 LoopInvariantCodeMotion(ShortLoopOptimizer *slo, GlobalValueNumbering* gvn, BlockBegin* loop_header, BlockList* loop_blocks);
46f6f063b272 7153771: array bound check elimination for c1
roland
parents: 6842
diff changeset
328 };
46f6f063b272 7153771: array bound check elimination for c1
roland
parents: 6842
diff changeset
329
46f6f063b272 7153771: array bound check elimination for c1
roland
parents: 6842
diff changeset
330 LoopInvariantCodeMotion::LoopInvariantCodeMotion(ShortLoopOptimizer *slo, GlobalValueNumbering* gvn, BlockBegin* loop_header, BlockList* loop_blocks)
46f6f063b272 7153771: array bound check elimination for c1
roland
parents: 6842
diff changeset
331 : _gvn(gvn), _short_loop_optimizer(slo) {
46f6f063b272 7153771: array bound check elimination for c1
roland
parents: 6842
diff changeset
332
46f6f063b272 7153771: array bound check elimination for c1
roland
parents: 6842
diff changeset
333 TRACE_VALUE_NUMBERING(tty->print_cr("using loop invariant code motion loop_header = %d", loop_header->block_id()));
46f6f063b272 7153771: array bound check elimination for c1
roland
parents: 6842
diff changeset
334 TRACE_VALUE_NUMBERING(tty->print_cr("** loop invariant code motion for short loop B%d", loop_header->block_id()));
46f6f063b272 7153771: array bound check elimination for c1
roland
parents: 6842
diff changeset
335
46f6f063b272 7153771: array bound check elimination for c1
roland
parents: 6842
diff changeset
336 BlockBegin* insertion_block = loop_header->dominator();
46f6f063b272 7153771: array bound check elimination for c1
roland
parents: 6842
diff changeset
337 if (insertion_block->number_of_preds() == 0) {
46f6f063b272 7153771: array bound check elimination for c1
roland
parents: 6842
diff changeset
338 return; // only the entry block does not have a predecessor
46f6f063b272 7153771: array bound check elimination for c1
roland
parents: 6842
diff changeset
339 }
46f6f063b272 7153771: array bound check elimination for c1
roland
parents: 6842
diff changeset
340
46f6f063b272 7153771: array bound check elimination for c1
roland
parents: 6842
diff changeset
341 assert(insertion_block->end()->as_Base() == NULL, "cannot insert into entry block");
46f6f063b272 7153771: array bound check elimination for c1
roland
parents: 6842
diff changeset
342 _insertion_point = insertion_block->end()->prev();
9081
84ab5667f290 8011706: specjvm2008 test xml.transform gets array bound exception with c1
roland
parents: 8860
diff changeset
343 _insert_is_pred = loop_header->is_predecessor(insertion_block);
8860
46f6f063b272 7153771: array bound check elimination for c1
roland
parents: 6842
diff changeset
344
46f6f063b272 7153771: array bound check elimination for c1
roland
parents: 6842
diff changeset
345 BlockEnd *block_end = insertion_block->end();
46f6f063b272 7153771: array bound check elimination for c1
roland
parents: 6842
diff changeset
346 _state = block_end->state_before();
46f6f063b272 7153771: array bound check elimination for c1
roland
parents: 6842
diff changeset
347
46f6f063b272 7153771: array bound check elimination for c1
roland
parents: 6842
diff changeset
348 if (!_state) {
46f6f063b272 7153771: array bound check elimination for c1
roland
parents: 6842
diff changeset
349 // If, TableSwitch and LookupSwitch always have state_before when
46f6f063b272 7153771: array bound check elimination for c1
roland
parents: 6842
diff changeset
350 // loop invariant code motion happens..
46f6f063b272 7153771: array bound check elimination for c1
roland
parents: 6842
diff changeset
351 assert(block_end->as_Goto(), "Block has to be goto");
46f6f063b272 7153771: array bound check elimination for c1
roland
parents: 6842
diff changeset
352 _state = block_end->state();
46f6f063b272 7153771: array bound check elimination for c1
roland
parents: 6842
diff changeset
353 }
46f6f063b272 7153771: array bound check elimination for c1
roland
parents: 6842
diff changeset
354
46f6f063b272 7153771: array bound check elimination for c1
roland
parents: 6842
diff changeset
355 // the loop_blocks are filled by going backward from the loop header, so this processing order is best
46f6f063b272 7153771: array bound check elimination for c1
roland
parents: 6842
diff changeset
356 assert(loop_blocks->at(0) == loop_header, "loop header must be first loop block");
46f6f063b272 7153771: array bound check elimination for c1
roland
parents: 6842
diff changeset
357 process_block(loop_header);
46f6f063b272 7153771: array bound check elimination for c1
roland
parents: 6842
diff changeset
358 for (int i = loop_blocks->length() - 1; i >= 1; i--) {
46f6f063b272 7153771: array bound check elimination for c1
roland
parents: 6842
diff changeset
359 process_block(loop_blocks->at(i));
46f6f063b272 7153771: array bound check elimination for c1
roland
parents: 6842
diff changeset
360 }
46f6f063b272 7153771: array bound check elimination for c1
roland
parents: 6842
diff changeset
361 }
46f6f063b272 7153771: array bound check elimination for c1
roland
parents: 6842
diff changeset
362
46f6f063b272 7153771: array bound check elimination for c1
roland
parents: 6842
diff changeset
363 void LoopInvariantCodeMotion::process_block(BlockBegin* block) {
46f6f063b272 7153771: array bound check elimination for c1
roland
parents: 6842
diff changeset
364 TRACE_VALUE_NUMBERING(tty->print_cr("processing block B%d", block->block_id()));
46f6f063b272 7153771: array bound check elimination for c1
roland
parents: 6842
diff changeset
365
46f6f063b272 7153771: array bound check elimination for c1
roland
parents: 6842
diff changeset
366 Instruction* prev = block;
46f6f063b272 7153771: array bound check elimination for c1
roland
parents: 6842
diff changeset
367 Instruction* cur = block->next();
46f6f063b272 7153771: array bound check elimination for c1
roland
parents: 6842
diff changeset
368
46f6f063b272 7153771: array bound check elimination for c1
roland
parents: 6842
diff changeset
369 while (cur != NULL) {
46f6f063b272 7153771: array bound check elimination for c1
roland
parents: 6842
diff changeset
370
46f6f063b272 7153771: array bound check elimination for c1
roland
parents: 6842
diff changeset
371 // determine if cur instruction is loop invariant
46f6f063b272 7153771: array bound check elimination for c1
roland
parents: 6842
diff changeset
372 // only selected instruction types are processed here
46f6f063b272 7153771: array bound check elimination for c1
roland
parents: 6842
diff changeset
373 bool cur_invariant = false;
46f6f063b272 7153771: array bound check elimination for c1
roland
parents: 6842
diff changeset
374
46f6f063b272 7153771: array bound check elimination for c1
roland
parents: 6842
diff changeset
375 if (cur->as_Constant() != NULL) {
46f6f063b272 7153771: array bound check elimination for c1
roland
parents: 6842
diff changeset
376 cur_invariant = !cur->can_trap();
46f6f063b272 7153771: array bound check elimination for c1
roland
parents: 6842
diff changeset
377 } else if (cur->as_ArithmeticOp() != NULL || cur->as_LogicOp() != NULL || cur->as_ShiftOp() != NULL) {
46f6f063b272 7153771: array bound check elimination for c1
roland
parents: 6842
diff changeset
378 assert(cur->as_Op2() != NULL, "must be Op2");
46f6f063b272 7153771: array bound check elimination for c1
roland
parents: 6842
diff changeset
379 Op2* op2 = (Op2*)cur;
46f6f063b272 7153771: array bound check elimination for c1
roland
parents: 6842
diff changeset
380 cur_invariant = !op2->can_trap() && is_invariant(op2->x()) && is_invariant(op2->y());
46f6f063b272 7153771: array bound check elimination for c1
roland
parents: 6842
diff changeset
381 } else if (cur->as_LoadField() != NULL) {
46f6f063b272 7153771: array bound check elimination for c1
roland
parents: 6842
diff changeset
382 LoadField* lf = (LoadField*)cur;
46f6f063b272 7153771: array bound check elimination for c1
roland
parents: 6842
diff changeset
383 // deoptimizes on NullPointerException
9081
84ab5667f290 8011706: specjvm2008 test xml.transform gets array bound exception with c1
roland
parents: 8860
diff changeset
384 cur_invariant = !lf->needs_patching() && !lf->field()->is_volatile() && !_short_loop_optimizer->has_field_store(lf->field()->type()->basic_type()) && is_invariant(lf->obj()) && _insert_is_pred;
8860
46f6f063b272 7153771: array bound check elimination for c1
roland
parents: 6842
diff changeset
385 } else if (cur->as_ArrayLength() != NULL) {
46f6f063b272 7153771: array bound check elimination for c1
roland
parents: 6842
diff changeset
386 ArrayLength *length = cur->as_ArrayLength();
46f6f063b272 7153771: array bound check elimination for c1
roland
parents: 6842
diff changeset
387 cur_invariant = is_invariant(length->array());
46f6f063b272 7153771: array bound check elimination for c1
roland
parents: 6842
diff changeset
388 } else if (cur->as_LoadIndexed() != NULL) {
46f6f063b272 7153771: array bound check elimination for c1
roland
parents: 6842
diff changeset
389 LoadIndexed *li = (LoadIndexed *)cur->as_LoadIndexed();
9081
84ab5667f290 8011706: specjvm2008 test xml.transform gets array bound exception with c1
roland
parents: 8860
diff changeset
390 cur_invariant = !_short_loop_optimizer->has_indexed_store(as_BasicType(cur->type())) && is_invariant(li->array()) && is_invariant(li->index()) && _insert_is_pred;
8860
46f6f063b272 7153771: array bound check elimination for c1
roland
parents: 6842
diff changeset
391 }
46f6f063b272 7153771: array bound check elimination for c1
roland
parents: 6842
diff changeset
392
46f6f063b272 7153771: array bound check elimination for c1
roland
parents: 6842
diff changeset
393 if (cur_invariant) {
46f6f063b272 7153771: array bound check elimination for c1
roland
parents: 6842
diff changeset
394 // perform value numbering and mark instruction as loop-invariant
46f6f063b272 7153771: array bound check elimination for c1
roland
parents: 6842
diff changeset
395 _gvn->substitute(cur);
46f6f063b272 7153771: array bound check elimination for c1
roland
parents: 6842
diff changeset
396
46f6f063b272 7153771: array bound check elimination for c1
roland
parents: 6842
diff changeset
397 if (cur->as_Constant() == NULL) {
46f6f063b272 7153771: array bound check elimination for c1
roland
parents: 6842
diff changeset
398 // ensure that code for non-constant instructions is always generated
46f6f063b272 7153771: array bound check elimination for c1
roland
parents: 6842
diff changeset
399 cur->pin();
46f6f063b272 7153771: array bound check elimination for c1
roland
parents: 6842
diff changeset
400 }
46f6f063b272 7153771: array bound check elimination for c1
roland
parents: 6842
diff changeset
401
46f6f063b272 7153771: array bound check elimination for c1
roland
parents: 6842
diff changeset
402 // remove cur instruction from loop block and append it to block before loop
46f6f063b272 7153771: array bound check elimination for c1
roland
parents: 6842
diff changeset
403 Instruction* next = cur->next();
46f6f063b272 7153771: array bound check elimination for c1
roland
parents: 6842
diff changeset
404 Instruction* in = _insertion_point->next();
46f6f063b272 7153771: array bound check elimination for c1
roland
parents: 6842
diff changeset
405 _insertion_point = _insertion_point->set_next(cur);
46f6f063b272 7153771: array bound check elimination for c1
roland
parents: 6842
diff changeset
406 cur->set_next(in);
46f6f063b272 7153771: array bound check elimination for c1
roland
parents: 6842
diff changeset
407
46f6f063b272 7153771: array bound check elimination for c1
roland
parents: 6842
diff changeset
408 // Deoptimize on exception
46f6f063b272 7153771: array bound check elimination for c1
roland
parents: 6842
diff changeset
409 cur->set_flag(Instruction::DeoptimizeOnException, true);
46f6f063b272 7153771: array bound check elimination for c1
roland
parents: 6842
diff changeset
410
46f6f063b272 7153771: array bound check elimination for c1
roland
parents: 6842
diff changeset
411 // Clear exception handlers
46f6f063b272 7153771: array bound check elimination for c1
roland
parents: 6842
diff changeset
412 cur->set_exception_handlers(NULL);
46f6f063b272 7153771: array bound check elimination for c1
roland
parents: 6842
diff changeset
413
46f6f063b272 7153771: array bound check elimination for c1
roland
parents: 6842
diff changeset
414 TRACE_VALUE_NUMBERING(tty->print_cr("Instruction %c%d is loop invariant", cur->type()->tchar(), cur->id()));
46f6f063b272 7153771: array bound check elimination for c1
roland
parents: 6842
diff changeset
415
46f6f063b272 7153771: array bound check elimination for c1
roland
parents: 6842
diff changeset
416 if (cur->state_before() != NULL) {
46f6f063b272 7153771: array bound check elimination for c1
roland
parents: 6842
diff changeset
417 cur->set_state_before(_state->copy());
46f6f063b272 7153771: array bound check elimination for c1
roland
parents: 6842
diff changeset
418 }
46f6f063b272 7153771: array bound check elimination for c1
roland
parents: 6842
diff changeset
419 if (cur->exception_state() != NULL) {
46f6f063b272 7153771: array bound check elimination for c1
roland
parents: 6842
diff changeset
420 cur->set_exception_state(_state->copy());
46f6f063b272 7153771: array bound check elimination for c1
roland
parents: 6842
diff changeset
421 }
46f6f063b272 7153771: array bound check elimination for c1
roland
parents: 6842
diff changeset
422
46f6f063b272 7153771: array bound check elimination for c1
roland
parents: 6842
diff changeset
423 cur = prev->set_next(next);
46f6f063b272 7153771: array bound check elimination for c1
roland
parents: 6842
diff changeset
424
46f6f063b272 7153771: array bound check elimination for c1
roland
parents: 6842
diff changeset
425 } else {
46f6f063b272 7153771: array bound check elimination for c1
roland
parents: 6842
diff changeset
426 prev = cur;
46f6f063b272 7153771: array bound check elimination for c1
roland
parents: 6842
diff changeset
427 cur = cur->next();
46f6f063b272 7153771: array bound check elimination for c1
roland
parents: 6842
diff changeset
428 }
46f6f063b272 7153771: array bound check elimination for c1
roland
parents: 6842
diff changeset
429 }
46f6f063b272 7153771: array bound check elimination for c1
roland
parents: 6842
diff changeset
430 }
0
a61af66fc99e Initial load
duke
parents:
diff changeset
431
a61af66fc99e Initial load
duke
parents:
diff changeset
432 bool ShortLoopOptimizer::process(BlockBegin* loop_header) {
a61af66fc99e Initial load
duke
parents:
diff changeset
433 TRACE_VALUE_NUMBERING(tty->print_cr("** loop header block"));
a61af66fc99e Initial load
duke
parents:
diff changeset
434
a61af66fc99e Initial load
duke
parents:
diff changeset
435 _too_complicated_loop = false;
a61af66fc99e Initial load
duke
parents:
diff changeset
436 _loop_blocks.clear();
a61af66fc99e Initial load
duke
parents:
diff changeset
437 _loop_blocks.append(loop_header);
a61af66fc99e Initial load
duke
parents:
diff changeset
438
a61af66fc99e Initial load
duke
parents:
diff changeset
439 for (int i = 0; i < _loop_blocks.length(); i++) {
a61af66fc99e Initial load
duke
parents:
diff changeset
440 BlockBegin* block = _loop_blocks.at(i);
a61af66fc99e Initial load
duke
parents:
diff changeset
441 TRACE_VALUE_NUMBERING(tty->print_cr("processing loop block B%d", block->block_id()));
a61af66fc99e Initial load
duke
parents:
diff changeset
442
a61af66fc99e Initial load
duke
parents:
diff changeset
443 if (block->is_set(BlockBegin::exception_entry_flag)) {
a61af66fc99e Initial load
duke
parents:
diff changeset
444 // this would be too complicated
a61af66fc99e Initial load
duke
parents:
diff changeset
445 return false;
a61af66fc99e Initial load
duke
parents:
diff changeset
446 }
a61af66fc99e Initial load
duke
parents:
diff changeset
447
a61af66fc99e Initial load
duke
parents:
diff changeset
448 // add predecessors to worklist
a61af66fc99e Initial load
duke
parents:
diff changeset
449 for (int j = block->number_of_preds() - 1; j >= 0; j--) {
a61af66fc99e Initial load
duke
parents:
diff changeset
450 BlockBegin* pred = block->pred_at(j);
a61af66fc99e Initial load
duke
parents:
diff changeset
451
8860
46f6f063b272 7153771: array bound check elimination for c1
roland
parents: 6842
diff changeset
452 if (pred->is_set(BlockBegin::osr_entry_flag)) {
46f6f063b272 7153771: array bound check elimination for c1
roland
parents: 6842
diff changeset
453 return false;
46f6f063b272 7153771: array bound check elimination for c1
roland
parents: 6842
diff changeset
454 }
46f6f063b272 7153771: array bound check elimination for c1
roland
parents: 6842
diff changeset
455
0
a61af66fc99e Initial load
duke
parents:
diff changeset
456 ValueMap* pred_map = value_map_of(pred);
a61af66fc99e Initial load
duke
parents:
diff changeset
457 if (pred_map != NULL) {
a61af66fc99e Initial load
duke
parents:
diff changeset
458 current_map()->kill_map(pred_map);
a61af66fc99e Initial load
duke
parents:
diff changeset
459 } else if (!_loop_blocks.contains(pred)) {
a61af66fc99e Initial load
duke
parents:
diff changeset
460 if (_loop_blocks.length() >= ValueMapMaxLoopSize) {
a61af66fc99e Initial load
duke
parents:
diff changeset
461 return false;
a61af66fc99e Initial load
duke
parents:
diff changeset
462 }
a61af66fc99e Initial load
duke
parents:
diff changeset
463 _loop_blocks.append(pred);
a61af66fc99e Initial load
duke
parents:
diff changeset
464 }
a61af66fc99e Initial load
duke
parents:
diff changeset
465 }
a61af66fc99e Initial load
duke
parents:
diff changeset
466
a61af66fc99e Initial load
duke
parents:
diff changeset
467 // use the instruction visitor for killing values
a61af66fc99e Initial load
duke
parents:
diff changeset
468 for (Value instr = block->next(); instr != NULL; instr = instr->next()) {
a61af66fc99e Initial load
duke
parents:
diff changeset
469 instr->visit(this);
a61af66fc99e Initial load
duke
parents:
diff changeset
470 if (_too_complicated_loop) {
a61af66fc99e Initial load
duke
parents:
diff changeset
471 return false;
a61af66fc99e Initial load
duke
parents:
diff changeset
472 }
a61af66fc99e Initial load
duke
parents:
diff changeset
473 }
a61af66fc99e Initial load
duke
parents:
diff changeset
474 }
a61af66fc99e Initial load
duke
parents:
diff changeset
475
8860
46f6f063b272 7153771: array bound check elimination for c1
roland
parents: 6842
diff changeset
476 bool optimistic = this->_gvn->compilation()->is_optimistic();
46f6f063b272 7153771: array bound check elimination for c1
roland
parents: 6842
diff changeset
477
46f6f063b272 7153771: array bound check elimination for c1
roland
parents: 6842
diff changeset
478 if (UseLoopInvariantCodeMotion && optimistic) {
46f6f063b272 7153771: array bound check elimination for c1
roland
parents: 6842
diff changeset
479 LoopInvariantCodeMotion code_motion(this, _gvn, loop_header, &_loop_blocks);
46f6f063b272 7153771: array bound check elimination for c1
roland
parents: 6842
diff changeset
480 }
46f6f063b272 7153771: array bound check elimination for c1
roland
parents: 6842
diff changeset
481
0
a61af66fc99e Initial load
duke
parents:
diff changeset
482 TRACE_VALUE_NUMBERING(tty->print_cr("** loop successfully optimized"));
a61af66fc99e Initial load
duke
parents:
diff changeset
483 return true;
a61af66fc99e Initial load
duke
parents:
diff changeset
484 }
a61af66fc99e Initial load
duke
parents:
diff changeset
485
a61af66fc99e Initial load
duke
parents:
diff changeset
486
a61af66fc99e Initial load
duke
parents:
diff changeset
487 GlobalValueNumbering::GlobalValueNumbering(IR* ir)
a61af66fc99e Initial load
duke
parents:
diff changeset
488 : _current_map(NULL)
a61af66fc99e Initial load
duke
parents:
diff changeset
489 , _value_maps(ir->linear_scan_order()->length(), NULL)
8860
46f6f063b272 7153771: array bound check elimination for c1
roland
parents: 6842
diff changeset
490 , _compilation(ir->compilation())
0
a61af66fc99e Initial load
duke
parents:
diff changeset
491 {
a61af66fc99e Initial load
duke
parents:
diff changeset
492 TRACE_VALUE_NUMBERING(tty->print_cr("****** start of global value numbering"));
a61af66fc99e Initial load
duke
parents:
diff changeset
493
a61af66fc99e Initial load
duke
parents:
diff changeset
494 ShortLoopOptimizer short_loop_optimizer(this);
a61af66fc99e Initial load
duke
parents:
diff changeset
495
a61af66fc99e Initial load
duke
parents:
diff changeset
496 BlockList* blocks = ir->linear_scan_order();
a61af66fc99e Initial load
duke
parents:
diff changeset
497 int num_blocks = blocks->length();
a61af66fc99e Initial load
duke
parents:
diff changeset
498
a61af66fc99e Initial load
duke
parents:
diff changeset
499 BlockBegin* start_block = blocks->at(0);
a61af66fc99e Initial load
duke
parents:
diff changeset
500 assert(start_block == ir->start() && start_block->number_of_preds() == 0 && start_block->dominator() == NULL, "must be start block");
a61af66fc99e Initial load
duke
parents:
diff changeset
501 assert(start_block->next()->as_Base() != NULL && start_block->next()->next() == NULL, "start block must not have instructions");
a61af66fc99e Initial load
duke
parents:
diff changeset
502
8860
46f6f063b272 7153771: array bound check elimination for c1
roland
parents: 6842
diff changeset
503 // method parameters are not linked in instructions list, so process them separateley
46f6f063b272 7153771: array bound check elimination for c1
roland
parents: 6842
diff changeset
504 for_each_state_value(start_block->state(), value,
46f6f063b272 7153771: array bound check elimination for c1
roland
parents: 6842
diff changeset
505 assert(value->as_Local() != NULL, "only method parameters allowed");
46f6f063b272 7153771: array bound check elimination for c1
roland
parents: 6842
diff changeset
506 set_processed(value);
46f6f063b272 7153771: array bound check elimination for c1
roland
parents: 6842
diff changeset
507 );
46f6f063b272 7153771: array bound check elimination for c1
roland
parents: 6842
diff changeset
508
0
a61af66fc99e Initial load
duke
parents:
diff changeset
509 // initial, empty value map with nesting 0
a61af66fc99e Initial load
duke
parents:
diff changeset
510 set_value_map_of(start_block, new ValueMap());
a61af66fc99e Initial load
duke
parents:
diff changeset
511
a61af66fc99e Initial load
duke
parents:
diff changeset
512 for (int i = 1; i < num_blocks; i++) {
a61af66fc99e Initial load
duke
parents:
diff changeset
513 BlockBegin* block = blocks->at(i);
a61af66fc99e Initial load
duke
parents:
diff changeset
514 TRACE_VALUE_NUMBERING(tty->print_cr("**** processing block B%d", block->block_id()));
a61af66fc99e Initial load
duke
parents:
diff changeset
515
a61af66fc99e Initial load
duke
parents:
diff changeset
516 int num_preds = block->number_of_preds();
a61af66fc99e Initial load
duke
parents:
diff changeset
517 assert(num_preds > 0, "block must have predecessors");
a61af66fc99e Initial load
duke
parents:
diff changeset
518
a61af66fc99e Initial load
duke
parents:
diff changeset
519 BlockBegin* dominator = block->dominator();
a61af66fc99e Initial load
duke
parents:
diff changeset
520 assert(dominator != NULL, "dominator must exist");
a61af66fc99e Initial load
duke
parents:
diff changeset
521 assert(value_map_of(dominator) != NULL, "value map of dominator must exist");
a61af66fc99e Initial load
duke
parents:
diff changeset
522
a61af66fc99e Initial load
duke
parents:
diff changeset
523 // create new value map with increased nesting
a61af66fc99e Initial load
duke
parents:
diff changeset
524 _current_map = new ValueMap(value_map_of(dominator));
a61af66fc99e Initial load
duke
parents:
diff changeset
525
8860
46f6f063b272 7153771: array bound check elimination for c1
roland
parents: 6842
diff changeset
526 if (num_preds == 1 && !block->is_set(BlockBegin::exception_entry_flag)) {
0
a61af66fc99e Initial load
duke
parents:
diff changeset
527 assert(dominator == block->pred_at(0), "dominator must be equal to predecessor");
a61af66fc99e Initial load
duke
parents:
diff changeset
528 // nothing to do here
a61af66fc99e Initial load
duke
parents:
diff changeset
529
a61af66fc99e Initial load
duke
parents:
diff changeset
530 } else if (block->is_set(BlockBegin::linear_scan_loop_header_flag)) {
a61af66fc99e Initial load
duke
parents:
diff changeset
531 // block has incoming backward branches -> try to optimize short loops
a61af66fc99e Initial load
duke
parents:
diff changeset
532 if (!short_loop_optimizer.process(block)) {
a61af66fc99e Initial load
duke
parents:
diff changeset
533 // loop is too complicated, so kill all memory loads because there might be
a61af66fc99e Initial load
duke
parents:
diff changeset
534 // stores to them in the loop
a61af66fc99e Initial load
duke
parents:
diff changeset
535 current_map()->kill_memory();
a61af66fc99e Initial load
duke
parents:
diff changeset
536 }
a61af66fc99e Initial load
duke
parents:
diff changeset
537
a61af66fc99e Initial load
duke
parents:
diff changeset
538 } else {
a61af66fc99e Initial load
duke
parents:
diff changeset
539 // only incoming forward branches that are already processed
a61af66fc99e Initial load
duke
parents:
diff changeset
540 for (int j = 0; j < num_preds; j++) {
a61af66fc99e Initial load
duke
parents:
diff changeset
541 BlockBegin* pred = block->pred_at(j);
a61af66fc99e Initial load
duke
parents:
diff changeset
542 ValueMap* pred_map = value_map_of(pred);
a61af66fc99e Initial load
duke
parents:
diff changeset
543
a61af66fc99e Initial load
duke
parents:
diff changeset
544 if (pred_map != NULL) {
a61af66fc99e Initial load
duke
parents:
diff changeset
545 // propagate killed values of the predecessor to this block
a61af66fc99e Initial load
duke
parents:
diff changeset
546 current_map()->kill_map(value_map_of(pred));
a61af66fc99e Initial load
duke
parents:
diff changeset
547 } else {
a61af66fc99e Initial load
duke
parents:
diff changeset
548 // kill all memory loads because predecessor not yet processed
a61af66fc99e Initial load
duke
parents:
diff changeset
549 // (this can happen with non-natural loops and OSR-compiles)
a61af66fc99e Initial load
duke
parents:
diff changeset
550 current_map()->kill_memory();
a61af66fc99e Initial load
duke
parents:
diff changeset
551 }
a61af66fc99e Initial load
duke
parents:
diff changeset
552 }
a61af66fc99e Initial load
duke
parents:
diff changeset
553 }
a61af66fc99e Initial load
duke
parents:
diff changeset
554
8860
46f6f063b272 7153771: array bound check elimination for c1
roland
parents: 6842
diff changeset
555 // phi functions are not linked in instructions list, so process them separateley
46f6f063b272 7153771: array bound check elimination for c1
roland
parents: 6842
diff changeset
556 for_each_phi_fun(block, phi,
46f6f063b272 7153771: array bound check elimination for c1
roland
parents: 6842
diff changeset
557 set_processed(phi);
46f6f063b272 7153771: array bound check elimination for c1
roland
parents: 6842
diff changeset
558 );
0
a61af66fc99e Initial load
duke
parents:
diff changeset
559
a61af66fc99e Initial load
duke
parents:
diff changeset
560 TRACE_VALUE_NUMBERING(tty->print("value map before processing block: "); current_map()->print());
a61af66fc99e Initial load
duke
parents:
diff changeset
561
a61af66fc99e Initial load
duke
parents:
diff changeset
562 // visit all instructions of this block
a61af66fc99e Initial load
duke
parents:
diff changeset
563 for (Value instr = block->next(); instr != NULL; instr = instr->next()) {
a61af66fc99e Initial load
duke
parents:
diff changeset
564 // check if instruction kills any values
a61af66fc99e Initial load
duke
parents:
diff changeset
565 instr->visit(this);
8860
46f6f063b272 7153771: array bound check elimination for c1
roland
parents: 6842
diff changeset
566 // perform actual value numbering
46f6f063b272 7153771: array bound check elimination for c1
roland
parents: 6842
diff changeset
567 substitute(instr);
0
a61af66fc99e Initial load
duke
parents:
diff changeset
568 }
a61af66fc99e Initial load
duke
parents:
diff changeset
569
a61af66fc99e Initial load
duke
parents:
diff changeset
570 // remember value map for successors
a61af66fc99e Initial load
duke
parents:
diff changeset
571 set_value_map_of(block, current_map());
a61af66fc99e Initial load
duke
parents:
diff changeset
572 }
a61af66fc99e Initial load
duke
parents:
diff changeset
573
8860
46f6f063b272 7153771: array bound check elimination for c1
roland
parents: 6842
diff changeset
574 if (_has_substitutions) {
0
a61af66fc99e Initial load
duke
parents:
diff changeset
575 SubstitutionResolver resolver(ir);
a61af66fc99e Initial load
duke
parents:
diff changeset
576 }
a61af66fc99e Initial load
duke
parents:
diff changeset
577
a61af66fc99e Initial load
duke
parents:
diff changeset
578 TRACE_VALUE_NUMBERING(tty->print("****** end of global value numbering. "); ValueMap::print_statistics());
a61af66fc99e Initial load
duke
parents:
diff changeset
579 }
8860
46f6f063b272 7153771: array bound check elimination for c1
roland
parents: 6842
diff changeset
580
46f6f063b272 7153771: array bound check elimination for c1
roland
parents: 6842
diff changeset
581 void GlobalValueNumbering::substitute(Instruction* instr) {
46f6f063b272 7153771: array bound check elimination for c1
roland
parents: 6842
diff changeset
582 assert(!instr->has_subst(), "substitution already set");
46f6f063b272 7153771: array bound check elimination for c1
roland
parents: 6842
diff changeset
583 Value subst = current_map()->find_insert(instr);
46f6f063b272 7153771: array bound check elimination for c1
roland
parents: 6842
diff changeset
584 if (subst != instr) {
46f6f063b272 7153771: array bound check elimination for c1
roland
parents: 6842
diff changeset
585 assert(!subst->has_subst(), "can't have a substitution");
46f6f063b272 7153771: array bound check elimination for c1
roland
parents: 6842
diff changeset
586
46f6f063b272 7153771: array bound check elimination for c1
roland
parents: 6842
diff changeset
587 TRACE_VALUE_NUMBERING(tty->print_cr("substitution for %d set to %d", instr->id(), subst->id()));
46f6f063b272 7153771: array bound check elimination for c1
roland
parents: 6842
diff changeset
588 instr->set_subst(subst);
46f6f063b272 7153771: array bound check elimination for c1
roland
parents: 6842
diff changeset
589 _has_substitutions = true;
46f6f063b272 7153771: array bound check elimination for c1
roland
parents: 6842
diff changeset
590 }
46f6f063b272 7153771: array bound check elimination for c1
roland
parents: 6842
diff changeset
591 set_processed(instr);
46f6f063b272 7153771: array bound check elimination for c1
roland
parents: 6842
diff changeset
592 }