annotate src/share/vm/c1/c1_ValueMap.cpp @ 1721:413ad0331a0c

6977924: Changes for 6975078 produce build error with certain gcc versions Summary: The changes introduced for 6975078 assign badHeapOopVal to the _allocation field in the ResourceObj class. In 32 bit linux builds with certain versions of gcc this assignment will be flagged as an error while compiling allocation.cpp. In 32 bit builds the constant value badHeapOopVal (which is cast to an intptr_t) is negative. The _allocation field is typed as an unsigned intptr_t and gcc catches this as an error. Reviewed-by: jcoomes, ysr, phh
author johnc
date Wed, 18 Aug 2010 10:59:06 -0700
parents c18cbe5936b8
children f95d63e2154a
Ignore whitespace changes - Everywhere: Within whitespace: At end of lines:
rev   line source
0
a61af66fc99e Initial load
duke
parents:
diff changeset
1 /*
1552
c18cbe5936b8 6941466: Oracle rebranding changes for Hotspot repositories
trims
parents: 0
diff changeset
2 * Copyright (c) 1999, 2005, 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
a61af66fc99e Initial load
duke
parents:
diff changeset
25 #include "incls/_precompiled.incl"
a61af66fc99e Initial load
duke
parents:
diff changeset
26 #include "incls/_c1_ValueMap.cpp.incl"
a61af66fc99e Initial load
duke
parents:
diff changeset
27
a61af66fc99e Initial load
duke
parents:
diff changeset
28
a61af66fc99e Initial load
duke
parents:
diff changeset
29 #ifndef PRODUCT
a61af66fc99e Initial load
duke
parents:
diff changeset
30
a61af66fc99e Initial load
duke
parents:
diff changeset
31 int ValueMap::_number_of_finds = 0;
a61af66fc99e Initial load
duke
parents:
diff changeset
32 int ValueMap::_number_of_hits = 0;
a61af66fc99e Initial load
duke
parents:
diff changeset
33 int ValueMap::_number_of_kills = 0;
a61af66fc99e Initial load
duke
parents:
diff changeset
34
a61af66fc99e Initial load
duke
parents:
diff changeset
35 #define TRACE_VALUE_NUMBERING(code) if (PrintValueNumbering) { code; }
a61af66fc99e Initial load
duke
parents:
diff changeset
36
a61af66fc99e Initial load
duke
parents:
diff changeset
37 #else
a61af66fc99e Initial load
duke
parents:
diff changeset
38
a61af66fc99e Initial load
duke
parents:
diff changeset
39 #define TRACE_VALUE_NUMBERING(code)
a61af66fc99e Initial load
duke
parents:
diff changeset
40
a61af66fc99e Initial load
duke
parents:
diff changeset
41 #endif
a61af66fc99e Initial load
duke
parents:
diff changeset
42
a61af66fc99e Initial load
duke
parents:
diff changeset
43
a61af66fc99e Initial load
duke
parents:
diff changeset
44 ValueMap::ValueMap()
a61af66fc99e Initial load
duke
parents:
diff changeset
45 : _nesting(0)
a61af66fc99e Initial load
duke
parents:
diff changeset
46 , _entries(ValueMapInitialSize, NULL)
a61af66fc99e Initial load
duke
parents:
diff changeset
47 , _killed_values()
a61af66fc99e Initial load
duke
parents:
diff changeset
48 , _entry_count(0)
a61af66fc99e Initial load
duke
parents:
diff changeset
49 {
a61af66fc99e Initial load
duke
parents:
diff changeset
50 NOT_PRODUCT(reset_statistics());
a61af66fc99e Initial load
duke
parents:
diff changeset
51 }
a61af66fc99e Initial load
duke
parents:
diff changeset
52
a61af66fc99e Initial load
duke
parents:
diff changeset
53
a61af66fc99e Initial load
duke
parents:
diff changeset
54 ValueMap::ValueMap(ValueMap* old)
a61af66fc99e Initial load
duke
parents:
diff changeset
55 : _nesting(old->_nesting + 1)
a61af66fc99e Initial load
duke
parents:
diff changeset
56 , _entries(old->_entries.length())
a61af66fc99e Initial load
duke
parents:
diff changeset
57 , _killed_values()
a61af66fc99e Initial load
duke
parents:
diff changeset
58 , _entry_count(old->_entry_count)
a61af66fc99e Initial load
duke
parents:
diff changeset
59 {
a61af66fc99e Initial load
duke
parents:
diff changeset
60 for (int i = size() - 1; i >= 0; i--) {
a61af66fc99e Initial load
duke
parents:
diff changeset
61 _entries.at_put(i, old->entry_at(i));
a61af66fc99e Initial load
duke
parents:
diff changeset
62 }
a61af66fc99e Initial load
duke
parents:
diff changeset
63 _killed_values.set_from(&old->_killed_values);
a61af66fc99e Initial load
duke
parents:
diff changeset
64 }
a61af66fc99e Initial load
duke
parents:
diff changeset
65
a61af66fc99e Initial load
duke
parents:
diff changeset
66
a61af66fc99e Initial load
duke
parents:
diff changeset
67 void ValueMap::increase_table_size() {
a61af66fc99e Initial load
duke
parents:
diff changeset
68 int old_size = size();
a61af66fc99e Initial load
duke
parents:
diff changeset
69 int new_size = old_size * 2 + 1;
a61af66fc99e Initial load
duke
parents:
diff changeset
70
a61af66fc99e Initial load
duke
parents:
diff changeset
71 ValueMapEntryList worklist(8);
a61af66fc99e Initial load
duke
parents:
diff changeset
72 ValueMapEntryArray new_entries(new_size, NULL);
a61af66fc99e Initial load
duke
parents:
diff changeset
73 int new_entry_count = 0;
a61af66fc99e Initial load
duke
parents:
diff changeset
74
a61af66fc99e Initial load
duke
parents:
diff changeset
75 TRACE_VALUE_NUMBERING(tty->print_cr("increasing table size from %d to %d", old_size, new_size));
a61af66fc99e Initial load
duke
parents:
diff changeset
76
a61af66fc99e Initial load
duke
parents:
diff changeset
77 for (int i = old_size - 1; i >= 0; i--) {
a61af66fc99e Initial load
duke
parents:
diff changeset
78 ValueMapEntry* entry;
a61af66fc99e Initial load
duke
parents:
diff changeset
79 for (entry = entry_at(i); entry != NULL; entry = entry->next()) {
a61af66fc99e Initial load
duke
parents:
diff changeset
80 if (!is_killed(entry->value())) {
a61af66fc99e Initial load
duke
parents:
diff changeset
81 worklist.push(entry);
a61af66fc99e Initial load
duke
parents:
diff changeset
82 }
a61af66fc99e Initial load
duke
parents:
diff changeset
83 }
a61af66fc99e Initial load
duke
parents:
diff changeset
84
a61af66fc99e Initial load
duke
parents:
diff changeset
85 while (!worklist.is_empty()) {
a61af66fc99e Initial load
duke
parents:
diff changeset
86 entry = worklist.pop();
a61af66fc99e Initial load
duke
parents:
diff changeset
87 int new_index = entry_index(entry->hash(), new_size);
a61af66fc99e Initial load
duke
parents:
diff changeset
88
a61af66fc99e Initial load
duke
parents:
diff changeset
89 if (entry->nesting() != nesting() && new_entries.at(new_index) != entry->next()) {
a61af66fc99e Initial load
duke
parents:
diff changeset
90 // changing entries with a lower nesting than the current nesting of the table
a61af66fc99e Initial load
duke
parents:
diff changeset
91 // is not allowed because then the same entry is contained in multiple value maps.
a61af66fc99e Initial load
duke
parents:
diff changeset
92 // clone entry when next-pointer must be changed
a61af66fc99e Initial load
duke
parents:
diff changeset
93 entry = new ValueMapEntry(entry->hash(), entry->value(), entry->nesting(), NULL);
a61af66fc99e Initial load
duke
parents:
diff changeset
94 }
a61af66fc99e Initial load
duke
parents:
diff changeset
95 entry->set_next(new_entries.at(new_index));
a61af66fc99e Initial load
duke
parents:
diff changeset
96 new_entries.at_put(new_index, entry);
a61af66fc99e Initial load
duke
parents:
diff changeset
97 new_entry_count++;
a61af66fc99e Initial load
duke
parents:
diff changeset
98 }
a61af66fc99e Initial load
duke
parents:
diff changeset
99 }
a61af66fc99e Initial load
duke
parents:
diff changeset
100
a61af66fc99e Initial load
duke
parents:
diff changeset
101 _entries = new_entries;
a61af66fc99e Initial load
duke
parents:
diff changeset
102 _entry_count = new_entry_count;
a61af66fc99e Initial load
duke
parents:
diff changeset
103 }
a61af66fc99e Initial load
duke
parents:
diff changeset
104
a61af66fc99e Initial load
duke
parents:
diff changeset
105
a61af66fc99e Initial load
duke
parents:
diff changeset
106 Value ValueMap::find_insert(Value x) {
a61af66fc99e Initial load
duke
parents:
diff changeset
107 const intx hash = x->hash();
a61af66fc99e Initial load
duke
parents:
diff changeset
108 if (hash != 0) {
a61af66fc99e Initial load
duke
parents:
diff changeset
109 // 0 hash means: exclude from value numbering
a61af66fc99e Initial load
duke
parents:
diff changeset
110 NOT_PRODUCT(_number_of_finds++);
a61af66fc99e Initial load
duke
parents:
diff changeset
111
a61af66fc99e Initial load
duke
parents:
diff changeset
112 for (ValueMapEntry* entry = entry_at(entry_index(hash, size())); entry != NULL; entry = entry->next()) {
a61af66fc99e Initial load
duke
parents:
diff changeset
113 if (entry->hash() == hash) {
a61af66fc99e Initial load
duke
parents:
diff changeset
114 Value f = entry->value();
a61af66fc99e Initial load
duke
parents:
diff changeset
115
a61af66fc99e Initial load
duke
parents:
diff changeset
116 if (!is_killed(f) && f->is_equal(x)) {
a61af66fc99e Initial load
duke
parents:
diff changeset
117 NOT_PRODUCT(_number_of_hits++);
a61af66fc99e Initial load
duke
parents:
diff changeset
118 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
119
a61af66fc99e Initial load
duke
parents:
diff changeset
120 if (entry->nesting() != nesting() && f->as_Constant() == NULL) {
a61af66fc99e Initial load
duke
parents:
diff changeset
121 // non-constant values of of another block must be pinned,
a61af66fc99e Initial load
duke
parents:
diff changeset
122 // otherwise it is possible that they are not evaluated
a61af66fc99e Initial load
duke
parents:
diff changeset
123 f->pin(Instruction::PinGlobalValueNumbering);
a61af66fc99e Initial load
duke
parents:
diff changeset
124 }
a61af66fc99e Initial load
duke
parents:
diff changeset
125
a61af66fc99e Initial load
duke
parents:
diff changeset
126 return f;
a61af66fc99e Initial load
duke
parents:
diff changeset
127
a61af66fc99e Initial load
duke
parents:
diff changeset
128 }
a61af66fc99e Initial load
duke
parents:
diff changeset
129 }
a61af66fc99e Initial load
duke
parents:
diff changeset
130 }
a61af66fc99e Initial load
duke
parents:
diff changeset
131
a61af66fc99e Initial load
duke
parents:
diff changeset
132 // x not found, so insert it
a61af66fc99e Initial load
duke
parents:
diff changeset
133 if (entry_count() >= size_threshold()) {
a61af66fc99e Initial load
duke
parents:
diff changeset
134 increase_table_size();
a61af66fc99e Initial load
duke
parents:
diff changeset
135 }
a61af66fc99e Initial load
duke
parents:
diff changeset
136 int idx = entry_index(hash, size());
a61af66fc99e Initial load
duke
parents:
diff changeset
137 _entries.at_put(idx, new ValueMapEntry(hash, x, nesting(), entry_at(idx)));
a61af66fc99e Initial load
duke
parents:
diff changeset
138 _entry_count++;
a61af66fc99e Initial load
duke
parents:
diff changeset
139
a61af66fc99e Initial load
duke
parents:
diff changeset
140 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
141 }
a61af66fc99e Initial load
duke
parents:
diff changeset
142
a61af66fc99e Initial load
duke
parents:
diff changeset
143 return x;
a61af66fc99e Initial load
duke
parents:
diff changeset
144 }
a61af66fc99e Initial load
duke
parents:
diff changeset
145
a61af66fc99e Initial load
duke
parents:
diff changeset
146
a61af66fc99e Initial load
duke
parents:
diff changeset
147 #define GENERIC_KILL_VALUE(must_kill_implementation) \
a61af66fc99e Initial load
duke
parents:
diff changeset
148 NOT_PRODUCT(_number_of_kills++); \
a61af66fc99e Initial load
duke
parents:
diff changeset
149 \
a61af66fc99e Initial load
duke
parents:
diff changeset
150 for (int i = size() - 1; i >= 0; i--) { \
a61af66fc99e Initial load
duke
parents:
diff changeset
151 ValueMapEntry* prev_entry = NULL; \
a61af66fc99e Initial load
duke
parents:
diff changeset
152 for (ValueMapEntry* entry = entry_at(i); entry != NULL; entry = entry->next()) { \
a61af66fc99e Initial load
duke
parents:
diff changeset
153 Value value = entry->value(); \
a61af66fc99e Initial load
duke
parents:
diff changeset
154 \
a61af66fc99e Initial load
duke
parents:
diff changeset
155 must_kill_implementation(must_kill, entry, value) \
a61af66fc99e Initial load
duke
parents:
diff changeset
156 \
a61af66fc99e Initial load
duke
parents:
diff changeset
157 if (must_kill) { \
a61af66fc99e Initial load
duke
parents:
diff changeset
158 kill_value(value); \
a61af66fc99e Initial load
duke
parents:
diff changeset
159 \
a61af66fc99e Initial load
duke
parents:
diff changeset
160 if (prev_entry == NULL) { \
a61af66fc99e Initial load
duke
parents:
diff changeset
161 _entries.at_put(i, entry->next()); \
a61af66fc99e Initial load
duke
parents:
diff changeset
162 _entry_count--; \
a61af66fc99e Initial load
duke
parents:
diff changeset
163 } else if (prev_entry->nesting() == nesting()) { \
a61af66fc99e Initial load
duke
parents:
diff changeset
164 prev_entry->set_next(entry->next()); \
a61af66fc99e Initial load
duke
parents:
diff changeset
165 _entry_count--; \
a61af66fc99e Initial load
duke
parents:
diff changeset
166 } else { \
a61af66fc99e Initial load
duke
parents:
diff changeset
167 prev_entry = entry; \
a61af66fc99e Initial load
duke
parents:
diff changeset
168 } \
a61af66fc99e Initial load
duke
parents:
diff changeset
169 \
a61af66fc99e Initial load
duke
parents:
diff changeset
170 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
171 } else { \
a61af66fc99e Initial load
duke
parents:
diff changeset
172 prev_entry = entry; \
a61af66fc99e Initial load
duke
parents:
diff changeset
173 } \
a61af66fc99e Initial load
duke
parents:
diff changeset
174 } \
a61af66fc99e Initial load
duke
parents:
diff changeset
175 } \
a61af66fc99e Initial load
duke
parents:
diff changeset
176
a61af66fc99e Initial load
duke
parents:
diff changeset
177 #define MUST_KILL_MEMORY(must_kill, entry, value) \
a61af66fc99e Initial load
duke
parents:
diff changeset
178 bool must_kill = value->as_LoadField() != NULL || value->as_LoadIndexed() != NULL;
a61af66fc99e Initial load
duke
parents:
diff changeset
179
a61af66fc99e Initial load
duke
parents:
diff changeset
180 #define MUST_KILL_ARRAY(must_kill, entry, value) \
a61af66fc99e Initial load
duke
parents:
diff changeset
181 bool must_kill = value->as_LoadIndexed() != NULL \
a61af66fc99e Initial load
duke
parents:
diff changeset
182 && value->type()->tag() == type->tag();
a61af66fc99e Initial load
duke
parents:
diff changeset
183
a61af66fc99e Initial load
duke
parents:
diff changeset
184 #define MUST_KILL_FIELD(must_kill, entry, value) \
a61af66fc99e Initial load
duke
parents:
diff changeset
185 /* ciField's are not unique; must compare their contents */ \
a61af66fc99e Initial load
duke
parents:
diff changeset
186 LoadField* lf = value->as_LoadField(); \
a61af66fc99e Initial load
duke
parents:
diff changeset
187 bool must_kill = lf != NULL \
a61af66fc99e Initial load
duke
parents:
diff changeset
188 && lf->field()->holder() == field->holder() \
a61af66fc99e Initial load
duke
parents:
diff changeset
189 && lf->field()->offset() == field->offset();
a61af66fc99e Initial load
duke
parents:
diff changeset
190
a61af66fc99e Initial load
duke
parents:
diff changeset
191 #define MUST_KILL_EXCEPTION(must_kill, entry, value) \
a61af66fc99e Initial load
duke
parents:
diff changeset
192 assert(entry->nesting() < nesting(), "must not find bigger nesting than current"); \
a61af66fc99e Initial load
duke
parents:
diff changeset
193 bool must_kill = (entry->nesting() == nesting() - 1);
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
a61af66fc99e Initial load
duke
parents:
diff changeset
204 void ValueMap::kill_field(ciField* field) {
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_exception() {
a61af66fc99e Initial load
duke
parents:
diff changeset
209 GENERIC_KILL_VALUE(MUST_KILL_EXCEPTION);
a61af66fc99e Initial load
duke
parents:
diff changeset
210 }
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_map(ValueMap* map) {
a61af66fc99e Initial load
duke
parents:
diff changeset
214 assert(is_global_value_numbering(), "only for global value numbering");
a61af66fc99e Initial load
duke
parents:
diff changeset
215 _killed_values.set_union(&map->_killed_values);
a61af66fc99e Initial load
duke
parents:
diff changeset
216 }
a61af66fc99e Initial load
duke
parents:
diff changeset
217
a61af66fc99e Initial load
duke
parents:
diff changeset
218 void ValueMap::kill_all() {
a61af66fc99e Initial load
duke
parents:
diff changeset
219 assert(is_local_value_numbering(), "only for local value numbering");
a61af66fc99e Initial load
duke
parents:
diff changeset
220 for (int i = size() - 1; i >= 0; i--) {
a61af66fc99e Initial load
duke
parents:
diff changeset
221 _entries.at_put(i, NULL);
a61af66fc99e Initial load
duke
parents:
diff changeset
222 }
a61af66fc99e Initial load
duke
parents:
diff changeset
223 _entry_count = 0;
a61af66fc99e Initial load
duke
parents:
diff changeset
224 }
a61af66fc99e Initial load
duke
parents:
diff changeset
225
a61af66fc99e Initial load
duke
parents:
diff changeset
226
a61af66fc99e Initial load
duke
parents:
diff changeset
227 #ifndef PRODUCT
a61af66fc99e Initial load
duke
parents:
diff changeset
228
a61af66fc99e Initial load
duke
parents:
diff changeset
229 void ValueMap::print() {
a61af66fc99e Initial load
duke
parents:
diff changeset
230 tty->print_cr("(size %d, entries %d, nesting %d)", size(), entry_count(), nesting());
a61af66fc99e Initial load
duke
parents:
diff changeset
231
a61af66fc99e Initial load
duke
parents:
diff changeset
232 int entries = 0;
a61af66fc99e Initial load
duke
parents:
diff changeset
233 for (int i = 0; i < size(); i++) {
a61af66fc99e Initial load
duke
parents:
diff changeset
234 if (entry_at(i) != NULL) {
a61af66fc99e Initial load
duke
parents:
diff changeset
235 tty->print(" %2d: ", i);
a61af66fc99e Initial load
duke
parents:
diff changeset
236 for (ValueMapEntry* entry = entry_at(i); entry != NULL; entry = entry->next()) {
a61af66fc99e Initial load
duke
parents:
diff changeset
237 Value value = entry->value();
a61af66fc99e Initial load
duke
parents:
diff changeset
238 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
239 entries++;
a61af66fc99e Initial load
duke
parents:
diff changeset
240 }
a61af66fc99e Initial load
duke
parents:
diff changeset
241 tty->print_cr("NULL");
a61af66fc99e Initial load
duke
parents:
diff changeset
242 }
a61af66fc99e Initial load
duke
parents:
diff changeset
243 }
a61af66fc99e Initial load
duke
parents:
diff changeset
244
a61af66fc99e Initial load
duke
parents:
diff changeset
245 _killed_values.print();
a61af66fc99e Initial load
duke
parents:
diff changeset
246 assert(entry_count() == entries, "entry_count incorrect");
a61af66fc99e Initial load
duke
parents:
diff changeset
247 }
a61af66fc99e Initial load
duke
parents:
diff changeset
248
a61af66fc99e Initial load
duke
parents:
diff changeset
249 void ValueMap::reset_statistics() {
a61af66fc99e Initial load
duke
parents:
diff changeset
250 _number_of_finds = 0;
a61af66fc99e Initial load
duke
parents:
diff changeset
251 _number_of_hits = 0;
a61af66fc99e Initial load
duke
parents:
diff changeset
252 _number_of_kills = 0;
a61af66fc99e Initial load
duke
parents:
diff changeset
253 }
a61af66fc99e Initial load
duke
parents:
diff changeset
254
a61af66fc99e Initial load
duke
parents:
diff changeset
255 void ValueMap::print_statistics() {
a61af66fc99e Initial load
duke
parents:
diff changeset
256 float hit_rate = 0;
a61af66fc99e Initial load
duke
parents:
diff changeset
257 if (_number_of_finds != 0) {
a61af66fc99e Initial load
duke
parents:
diff changeset
258 hit_rate = (float)_number_of_hits / _number_of_finds;
a61af66fc99e Initial load
duke
parents:
diff changeset
259 }
a61af66fc99e Initial load
duke
parents:
diff changeset
260
a61af66fc99e Initial load
duke
parents:
diff changeset
261 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
262 }
a61af66fc99e Initial load
duke
parents:
diff changeset
263
a61af66fc99e Initial load
duke
parents:
diff changeset
264 #endif
a61af66fc99e Initial load
duke
parents:
diff changeset
265
a61af66fc99e Initial load
duke
parents:
diff changeset
266
a61af66fc99e Initial load
duke
parents:
diff changeset
267
a61af66fc99e Initial load
duke
parents:
diff changeset
268 class ShortLoopOptimizer : public ValueNumberingVisitor {
a61af66fc99e Initial load
duke
parents:
diff changeset
269 private:
a61af66fc99e Initial load
duke
parents:
diff changeset
270 GlobalValueNumbering* _gvn;
a61af66fc99e Initial load
duke
parents:
diff changeset
271 BlockList _loop_blocks;
a61af66fc99e Initial load
duke
parents:
diff changeset
272 bool _too_complicated_loop;
a61af66fc99e Initial load
duke
parents:
diff changeset
273
a61af66fc99e Initial load
duke
parents:
diff changeset
274 // simplified access to methods of GlobalValueNumbering
a61af66fc99e Initial load
duke
parents:
diff changeset
275 ValueMap* current_map() { return _gvn->current_map(); }
a61af66fc99e Initial load
duke
parents:
diff changeset
276 ValueMap* value_map_of(BlockBegin* block) { return _gvn->value_map_of(block); }
a61af66fc99e Initial load
duke
parents:
diff changeset
277
a61af66fc99e Initial load
duke
parents:
diff changeset
278 // implementation for abstract methods of ValueNumberingVisitor
a61af66fc99e Initial load
duke
parents:
diff changeset
279 void kill_memory() { _too_complicated_loop = true; }
a61af66fc99e Initial load
duke
parents:
diff changeset
280 void kill_field(ciField* field) { current_map()->kill_field(field); };
a61af66fc99e Initial load
duke
parents:
diff changeset
281 void kill_array(ValueType* type) { current_map()->kill_array(type); };
a61af66fc99e Initial load
duke
parents:
diff changeset
282
a61af66fc99e Initial load
duke
parents:
diff changeset
283 public:
a61af66fc99e Initial load
duke
parents:
diff changeset
284 ShortLoopOptimizer(GlobalValueNumbering* gvn)
a61af66fc99e Initial load
duke
parents:
diff changeset
285 : _gvn(gvn)
a61af66fc99e Initial load
duke
parents:
diff changeset
286 , _loop_blocks(ValueMapMaxLoopSize)
a61af66fc99e Initial load
duke
parents:
diff changeset
287 , _too_complicated_loop(false)
a61af66fc99e Initial load
duke
parents:
diff changeset
288 {
a61af66fc99e Initial load
duke
parents:
diff changeset
289 }
a61af66fc99e Initial load
duke
parents:
diff changeset
290
a61af66fc99e Initial load
duke
parents:
diff changeset
291 bool process(BlockBegin* loop_header);
a61af66fc99e Initial load
duke
parents:
diff changeset
292 };
a61af66fc99e Initial load
duke
parents:
diff changeset
293
a61af66fc99e Initial load
duke
parents:
diff changeset
294
a61af66fc99e Initial load
duke
parents:
diff changeset
295 bool ShortLoopOptimizer::process(BlockBegin* loop_header) {
a61af66fc99e Initial load
duke
parents:
diff changeset
296 TRACE_VALUE_NUMBERING(tty->print_cr("** loop header block"));
a61af66fc99e Initial load
duke
parents:
diff changeset
297
a61af66fc99e Initial load
duke
parents:
diff changeset
298 _too_complicated_loop = false;
a61af66fc99e Initial load
duke
parents:
diff changeset
299 _loop_blocks.clear();
a61af66fc99e Initial load
duke
parents:
diff changeset
300 _loop_blocks.append(loop_header);
a61af66fc99e Initial load
duke
parents:
diff changeset
301
a61af66fc99e Initial load
duke
parents:
diff changeset
302 for (int i = 0; i < _loop_blocks.length(); i++) {
a61af66fc99e Initial load
duke
parents:
diff changeset
303 BlockBegin* block = _loop_blocks.at(i);
a61af66fc99e Initial load
duke
parents:
diff changeset
304 TRACE_VALUE_NUMBERING(tty->print_cr("processing loop block B%d", block->block_id()));
a61af66fc99e Initial load
duke
parents:
diff changeset
305
a61af66fc99e Initial load
duke
parents:
diff changeset
306 if (block->is_set(BlockBegin::exception_entry_flag)) {
a61af66fc99e Initial load
duke
parents:
diff changeset
307 // this would be too complicated
a61af66fc99e Initial load
duke
parents:
diff changeset
308 return false;
a61af66fc99e Initial load
duke
parents:
diff changeset
309 }
a61af66fc99e Initial load
duke
parents:
diff changeset
310
a61af66fc99e Initial load
duke
parents:
diff changeset
311 // add predecessors to worklist
a61af66fc99e Initial load
duke
parents:
diff changeset
312 for (int j = block->number_of_preds() - 1; j >= 0; j--) {
a61af66fc99e Initial load
duke
parents:
diff changeset
313 BlockBegin* pred = block->pred_at(j);
a61af66fc99e Initial load
duke
parents:
diff changeset
314
a61af66fc99e Initial load
duke
parents:
diff changeset
315 ValueMap* pred_map = value_map_of(pred);
a61af66fc99e Initial load
duke
parents:
diff changeset
316 if (pred_map != NULL) {
a61af66fc99e Initial load
duke
parents:
diff changeset
317 current_map()->kill_map(pred_map);
a61af66fc99e Initial load
duke
parents:
diff changeset
318 } else if (!_loop_blocks.contains(pred)) {
a61af66fc99e Initial load
duke
parents:
diff changeset
319 if (_loop_blocks.length() >= ValueMapMaxLoopSize) {
a61af66fc99e Initial load
duke
parents:
diff changeset
320 return false;
a61af66fc99e Initial load
duke
parents:
diff changeset
321 }
a61af66fc99e Initial load
duke
parents:
diff changeset
322 _loop_blocks.append(pred);
a61af66fc99e Initial load
duke
parents:
diff changeset
323 }
a61af66fc99e Initial load
duke
parents:
diff changeset
324 }
a61af66fc99e Initial load
duke
parents:
diff changeset
325
a61af66fc99e Initial load
duke
parents:
diff changeset
326 // use the instruction visitor for killing values
a61af66fc99e Initial load
duke
parents:
diff changeset
327 for (Value instr = block->next(); instr != NULL; instr = instr->next()) {
a61af66fc99e Initial load
duke
parents:
diff changeset
328 instr->visit(this);
a61af66fc99e Initial load
duke
parents:
diff changeset
329 if (_too_complicated_loop) {
a61af66fc99e Initial load
duke
parents:
diff changeset
330 return false;
a61af66fc99e Initial load
duke
parents:
diff changeset
331 }
a61af66fc99e Initial load
duke
parents:
diff changeset
332 }
a61af66fc99e Initial load
duke
parents:
diff changeset
333 }
a61af66fc99e Initial load
duke
parents:
diff changeset
334
a61af66fc99e Initial load
duke
parents:
diff changeset
335 TRACE_VALUE_NUMBERING(tty->print_cr("** loop successfully optimized"));
a61af66fc99e Initial load
duke
parents:
diff changeset
336 return true;
a61af66fc99e Initial load
duke
parents:
diff changeset
337 }
a61af66fc99e Initial load
duke
parents:
diff changeset
338
a61af66fc99e Initial load
duke
parents:
diff changeset
339
a61af66fc99e Initial load
duke
parents:
diff changeset
340 GlobalValueNumbering::GlobalValueNumbering(IR* ir)
a61af66fc99e Initial load
duke
parents:
diff changeset
341 : _current_map(NULL)
a61af66fc99e Initial load
duke
parents:
diff changeset
342 , _value_maps(ir->linear_scan_order()->length(), NULL)
a61af66fc99e Initial load
duke
parents:
diff changeset
343 {
a61af66fc99e Initial load
duke
parents:
diff changeset
344 TRACE_VALUE_NUMBERING(tty->print_cr("****** start of global value numbering"));
a61af66fc99e Initial load
duke
parents:
diff changeset
345
a61af66fc99e Initial load
duke
parents:
diff changeset
346 ShortLoopOptimizer short_loop_optimizer(this);
a61af66fc99e Initial load
duke
parents:
diff changeset
347 int subst_count = 0;
a61af66fc99e Initial load
duke
parents:
diff changeset
348
a61af66fc99e Initial load
duke
parents:
diff changeset
349 BlockList* blocks = ir->linear_scan_order();
a61af66fc99e Initial load
duke
parents:
diff changeset
350 int num_blocks = blocks->length();
a61af66fc99e Initial load
duke
parents:
diff changeset
351
a61af66fc99e Initial load
duke
parents:
diff changeset
352 BlockBegin* start_block = blocks->at(0);
a61af66fc99e Initial load
duke
parents:
diff changeset
353 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
354 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
355
a61af66fc99e Initial load
duke
parents:
diff changeset
356 // initial, empty value map with nesting 0
a61af66fc99e Initial load
duke
parents:
diff changeset
357 set_value_map_of(start_block, new ValueMap());
a61af66fc99e Initial load
duke
parents:
diff changeset
358
a61af66fc99e Initial load
duke
parents:
diff changeset
359 for (int i = 1; i < num_blocks; i++) {
a61af66fc99e Initial load
duke
parents:
diff changeset
360 BlockBegin* block = blocks->at(i);
a61af66fc99e Initial load
duke
parents:
diff changeset
361 TRACE_VALUE_NUMBERING(tty->print_cr("**** processing block B%d", block->block_id()));
a61af66fc99e Initial load
duke
parents:
diff changeset
362
a61af66fc99e Initial load
duke
parents:
diff changeset
363 int num_preds = block->number_of_preds();
a61af66fc99e Initial load
duke
parents:
diff changeset
364 assert(num_preds > 0, "block must have predecessors");
a61af66fc99e Initial load
duke
parents:
diff changeset
365
a61af66fc99e Initial load
duke
parents:
diff changeset
366 BlockBegin* dominator = block->dominator();
a61af66fc99e Initial load
duke
parents:
diff changeset
367 assert(dominator != NULL, "dominator must exist");
a61af66fc99e Initial load
duke
parents:
diff changeset
368 assert(value_map_of(dominator) != NULL, "value map of dominator must exist");
a61af66fc99e Initial load
duke
parents:
diff changeset
369
a61af66fc99e Initial load
duke
parents:
diff changeset
370 // create new value map with increased nesting
a61af66fc99e Initial load
duke
parents:
diff changeset
371 _current_map = new ValueMap(value_map_of(dominator));
a61af66fc99e Initial load
duke
parents:
diff changeset
372
a61af66fc99e Initial load
duke
parents:
diff changeset
373 if (num_preds == 1) {
a61af66fc99e Initial load
duke
parents:
diff changeset
374 assert(dominator == block->pred_at(0), "dominator must be equal to predecessor");
a61af66fc99e Initial load
duke
parents:
diff changeset
375 // nothing to do here
a61af66fc99e Initial load
duke
parents:
diff changeset
376
a61af66fc99e Initial load
duke
parents:
diff changeset
377 } else if (block->is_set(BlockBegin::linear_scan_loop_header_flag)) {
a61af66fc99e Initial load
duke
parents:
diff changeset
378 // block has incoming backward branches -> try to optimize short loops
a61af66fc99e Initial load
duke
parents:
diff changeset
379 if (!short_loop_optimizer.process(block)) {
a61af66fc99e Initial load
duke
parents:
diff changeset
380 // loop is too complicated, so kill all memory loads because there might be
a61af66fc99e Initial load
duke
parents:
diff changeset
381 // stores to them in the loop
a61af66fc99e Initial load
duke
parents:
diff changeset
382 current_map()->kill_memory();
a61af66fc99e Initial load
duke
parents:
diff changeset
383 }
a61af66fc99e Initial load
duke
parents:
diff changeset
384
a61af66fc99e Initial load
duke
parents:
diff changeset
385 } else {
a61af66fc99e Initial load
duke
parents:
diff changeset
386 // only incoming forward branches that are already processed
a61af66fc99e Initial load
duke
parents:
diff changeset
387 for (int j = 0; j < num_preds; j++) {
a61af66fc99e Initial load
duke
parents:
diff changeset
388 BlockBegin* pred = block->pred_at(j);
a61af66fc99e Initial load
duke
parents:
diff changeset
389 ValueMap* pred_map = value_map_of(pred);
a61af66fc99e Initial load
duke
parents:
diff changeset
390
a61af66fc99e Initial load
duke
parents:
diff changeset
391 if (pred_map != NULL) {
a61af66fc99e Initial load
duke
parents:
diff changeset
392 // propagate killed values of the predecessor to this block
a61af66fc99e Initial load
duke
parents:
diff changeset
393 current_map()->kill_map(value_map_of(pred));
a61af66fc99e Initial load
duke
parents:
diff changeset
394 } else {
a61af66fc99e Initial load
duke
parents:
diff changeset
395 // kill all memory loads because predecessor not yet processed
a61af66fc99e Initial load
duke
parents:
diff changeset
396 // (this can happen with non-natural loops and OSR-compiles)
a61af66fc99e Initial load
duke
parents:
diff changeset
397 current_map()->kill_memory();
a61af66fc99e Initial load
duke
parents:
diff changeset
398 }
a61af66fc99e Initial load
duke
parents:
diff changeset
399 }
a61af66fc99e Initial load
duke
parents:
diff changeset
400 }
a61af66fc99e Initial load
duke
parents:
diff changeset
401
a61af66fc99e Initial load
duke
parents:
diff changeset
402 if (block->is_set(BlockBegin::exception_entry_flag)) {
a61af66fc99e Initial load
duke
parents:
diff changeset
403 current_map()->kill_exception();
a61af66fc99e Initial load
duke
parents:
diff changeset
404 }
a61af66fc99e Initial load
duke
parents:
diff changeset
405
a61af66fc99e Initial load
duke
parents:
diff changeset
406 TRACE_VALUE_NUMBERING(tty->print("value map before processing block: "); current_map()->print());
a61af66fc99e Initial load
duke
parents:
diff changeset
407
a61af66fc99e Initial load
duke
parents:
diff changeset
408 // visit all instructions of this block
a61af66fc99e Initial load
duke
parents:
diff changeset
409 for (Value instr = block->next(); instr != NULL; instr = instr->next()) {
a61af66fc99e Initial load
duke
parents:
diff changeset
410 assert(!instr->has_subst(), "substitution already set");
a61af66fc99e Initial load
duke
parents:
diff changeset
411
a61af66fc99e Initial load
duke
parents:
diff changeset
412 // check if instruction kills any values
a61af66fc99e Initial load
duke
parents:
diff changeset
413 instr->visit(this);
a61af66fc99e Initial load
duke
parents:
diff changeset
414
a61af66fc99e Initial load
duke
parents:
diff changeset
415 if (instr->hash() != 0) {
a61af66fc99e Initial load
duke
parents:
diff changeset
416 Value f = current_map()->find_insert(instr);
a61af66fc99e Initial load
duke
parents:
diff changeset
417 if (f != instr) {
a61af66fc99e Initial load
duke
parents:
diff changeset
418 assert(!f->has_subst(), "can't have a substitution");
a61af66fc99e Initial load
duke
parents:
diff changeset
419 instr->set_subst(f);
a61af66fc99e Initial load
duke
parents:
diff changeset
420 subst_count++;
a61af66fc99e Initial load
duke
parents:
diff changeset
421 }
a61af66fc99e Initial load
duke
parents:
diff changeset
422 }
a61af66fc99e Initial load
duke
parents:
diff changeset
423 }
a61af66fc99e Initial load
duke
parents:
diff changeset
424
a61af66fc99e Initial load
duke
parents:
diff changeset
425 // remember value map for successors
a61af66fc99e Initial load
duke
parents:
diff changeset
426 set_value_map_of(block, current_map());
a61af66fc99e Initial load
duke
parents:
diff changeset
427 }
a61af66fc99e Initial load
duke
parents:
diff changeset
428
a61af66fc99e Initial load
duke
parents:
diff changeset
429 if (subst_count != 0) {
a61af66fc99e Initial load
duke
parents:
diff changeset
430 SubstitutionResolver resolver(ir);
a61af66fc99e Initial load
duke
parents:
diff changeset
431 }
a61af66fc99e Initial load
duke
parents:
diff changeset
432
a61af66fc99e Initial load
duke
parents:
diff changeset
433 TRACE_VALUE_NUMBERING(tty->print("****** end of global value numbering. "); ValueMap::print_statistics());
a61af66fc99e Initial load
duke
parents:
diff changeset
434 }