author | thartmann |
Fri, 20 Nov 2015 10:09:42 +0100 | |
changeset 34204 | 5ad1ba3afecc |
parent 34183 | 8b683720a3e4 |
child 38030 | 93f24e7b3c43 |
permissions | -rw-r--r-- |
1 | 1 |
/* |
15480
a9e84da84519
8006807: C2 crash due to out of bounds array access in Parse::do_multianewarray
drchase
parents:
14835
diff
changeset
|
2 |
* Copyright (c) 1998, 2013, Oracle and/or its affiliates. All rights reserved. |
1 | 3 |
* DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS FILE HEADER. |
4 |
* |
|
5 |
* This code is free software; you can redistribute it and/or modify it |
|
6 |
* under the terms of the GNU General Public License version 2 only, as |
|
7 |
* published by the Free Software Foundation. |
|
8 |
* |
|
9 |
* This code is distributed in the hope that it will be useful, but WITHOUT |
|
10 |
* ANY WARRANTY; without even the implied warranty of MERCHANTABILITY or |
|
11 |
* FITNESS FOR A PARTICULAR PURPOSE. See the GNU General Public License |
|
12 |
* version 2 for more details (a copy is included in the LICENSE file that |
|
13 |
* accompanied this code). |
|
14 |
* |
|
15 |
* You should have received a copy of the GNU General Public License version |
|
16 |
* 2 along with this work; if not, write to the Free Software Foundation, |
|
17 |
* Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301 USA. |
|
18 |
* |
|
5547
f4b087cbb361
6941466: Oracle rebranding changes for Hotspot repositories
trims
parents:
4893
diff
changeset
|
19 |
* Please contact Oracle, 500 Oracle Parkway, Redwood Shores, CA 94065 USA |
f4b087cbb361
6941466: Oracle rebranding changes for Hotspot repositories
trims
parents:
4893
diff
changeset
|
20 |
* or visit www.oracle.com if you need additional information or have any |
f4b087cbb361
6941466: Oracle rebranding changes for Hotspot repositories
trims
parents:
4893
diff
changeset
|
21 |
* questions. |
1 | 22 |
* |
23 |
*/ |
|
24 |
||
7397 | 25 |
#include "precompiled.hpp" |
26 |
#include "compiler/compileLog.hpp" |
|
27 |
#include "interpreter/linkResolver.hpp" |
|
28 |
#include "memory/universe.inline.hpp" |
|
29 |
#include "oops/objArrayKlass.hpp" |
|
30 |
#include "opto/addnode.hpp" |
|
23528 | 31 |
#include "opto/castnode.hpp" |
7397 | 32 |
#include "opto/memnode.hpp" |
33 |
#include "opto/parse.hpp" |
|
34 |
#include "opto/rootnode.hpp" |
|
35 |
#include "opto/runtime.hpp" |
|
36 |
#include "opto/subnode.hpp" |
|
37 |
#include "runtime/deoptimization.hpp" |
|
38 |
#include "runtime/handles.inline.hpp" |
|
1 | 39 |
|
40 |
//============================================================================= |
|
41 |
// Helper methods for _get* and _put* bytecodes |
|
42 |
//============================================================================= |
|
43 |
bool Parse::static_field_ok_in_clinit(ciField *field, ciMethod *method) { |
|
44 |
// Could be the field_holder's <clinit> method, or <clinit> for a subklass. |
|
45 |
// Better to check now than to Deoptimize as soon as we execute |
|
46 |
assert( field->is_static(), "Only check if field is static"); |
|
47 |
// is_being_initialized() is too generous. It allows access to statics |
|
48 |
// by threads that are not running the <clinit> before the <clinit> finishes. |
|
49 |
// return field->holder()->is_being_initialized(); |
|
50 |
||
51 |
// The following restriction is correct but conservative. |
|
52 |
// It is also desirable to allow compilation of methods called from <clinit> |
|
53 |
// but this generated code will need to be made safe for execution by |
|
54 |
// other threads, or the transition from interpreted to compiled code would |
|
55 |
// need to be guarded. |
|
56 |
ciInstanceKlass *field_holder = field->holder(); |
|
57 |
||
58 |
bool access_OK = false; |
|
59 |
if (method->holder()->is_subclass_of(field_holder)) { |
|
60 |
if (method->is_static()) { |
|
61 |
if (method->name() == ciSymbol::class_initializer_name()) { |
|
62 |
// OK to access static fields inside initializer |
|
63 |
access_OK = true; |
|
64 |
} |
|
65 |
} else { |
|
66 |
if (method->name() == ciSymbol::object_initializer_name()) { |
|
67 |
// It's also OK to access static fields inside a constructor, |
|
68 |
// because any thread calling the constructor must first have |
|
69 |
// synchronized on the class by executing a '_new' bytecode. |
|
70 |
access_OK = true; |
|
71 |
} |
|
72 |
} |
|
73 |
} |
|
74 |
||
75 |
return access_OK; |
|
76 |
||
77 |
} |
|
78 |
||
79 |
||
80 |
void Parse::do_field_access(bool is_get, bool is_field) { |
|
81 |
bool will_link; |
|
82 |
ciField* field = iter().get_field(will_link); |
|
83 |
assert(will_link, "getfield: typeflow responsibility"); |
|
84 |
||
85 |
ciInstanceKlass* field_holder = field->holder(); |
|
86 |
||
87 |
if (is_field == field->is_static()) { |
|
88 |
// Interpreter will throw java_lang_IncompatibleClassChangeError |
|
89 |
// Check this before allowing <clinit> methods to access static fields |
|
90 |
uncommon_trap(Deoptimization::Reason_unhandled, |
|
91 |
Deoptimization::Action_none); |
|
92 |
return; |
|
93 |
} |
|
94 |
||
95 |
if (!is_field && !field_holder->is_initialized()) { |
|
96 |
if (!static_field_ok_in_clinit(field, method())) { |
|
97 |
uncommon_trap(Deoptimization::Reason_uninitialized, |
|
98 |
Deoptimization::Action_reinterpret, |
|
99 |
NULL, "!static_field_ok_in_clinit"); |
|
100 |
return; |
|
101 |
} |
|
102 |
} |
|
103 |
||
10510
ab626d1bdf53
7085404: JSR 292: VolatileCallSites should have push notification too
twisti
parents:
10265
diff
changeset
|
104 |
// Deoptimize on putfield writes to call site target field. |
10265
4c869854aebd
7071653: JSR 292: call site change notification should be pushed not pulled
twisti
parents:
10028
diff
changeset
|
105 |
if (!is_get && field->is_call_site_target()) { |
4c869854aebd
7071653: JSR 292: call site change notification should be pushed not pulled
twisti
parents:
10028
diff
changeset
|
106 |
uncommon_trap(Deoptimization::Reason_unhandled, |
4c869854aebd
7071653: JSR 292: call site change notification should be pushed not pulled
twisti
parents:
10028
diff
changeset
|
107 |
Deoptimization::Action_reinterpret, |
10510
ab626d1bdf53
7085404: JSR 292: VolatileCallSites should have push notification too
twisti
parents:
10265
diff
changeset
|
108 |
NULL, "put to call site target field"); |
10265
4c869854aebd
7071653: JSR 292: call site change notification should be pushed not pulled
twisti
parents:
10028
diff
changeset
|
109 |
return; |
4c869854aebd
7071653: JSR 292: call site change notification should be pushed not pulled
twisti
parents:
10028
diff
changeset
|
110 |
} |
4c869854aebd
7071653: JSR 292: call site change notification should be pushed not pulled
twisti
parents:
10028
diff
changeset
|
111 |
|
1 | 112 |
assert(field->will_link(method()->holder(), bc()), "getfield: typeflow responsibility"); |
113 |
||
114 |
// Note: We do not check for an unloaded field type here any more. |
|
115 |
||
116 |
// Generate code for the object pointer. |
|
117 |
Node* obj; |
|
118 |
if (is_field) { |
|
119 |
int obj_depth = is_get ? 0 : field->type()->size(); |
|
14621
fd9265ab0f67
7172640: C2: instrinsic implementations in LibraryCallKit should use argument() instead of pop()
twisti
parents:
13963
diff
changeset
|
120 |
obj = null_check(peek(obj_depth)); |
1 | 121 |
// Compile-time detect of null-exception? |
122 |
if (stopped()) return; |
|
123 |
||
8725
8c1e3dd5fe1b
7017732: move static fields into Class to prepare for perm gen removal
never
parents:
8676
diff
changeset
|
124 |
#ifdef ASSERT |
1 | 125 |
const TypeInstPtr *tjp = TypeInstPtr::make(TypePtr::NotNull, iter().get_declared_field_holder()); |
126 |
assert(_gvn.type(obj)->higher_equal(tjp), "cast_up is no longer needed"); |
|
8725
8c1e3dd5fe1b
7017732: move static fields into Class to prepare for perm gen removal
never
parents:
8676
diff
changeset
|
127 |
#endif |
1 | 128 |
|
129 |
if (is_get) { |
|
14621
fd9265ab0f67
7172640: C2: instrinsic implementations in LibraryCallKit should use argument() instead of pop()
twisti
parents:
13963
diff
changeset
|
130 |
(void) pop(); // pop receiver before getting |
8725
8c1e3dd5fe1b
7017732: move static fields into Class to prepare for perm gen removal
never
parents:
8676
diff
changeset
|
131 |
do_get_xxx(obj, field, is_field); |
1 | 132 |
} else { |
8725
8c1e3dd5fe1b
7017732: move static fields into Class to prepare for perm gen removal
never
parents:
8676
diff
changeset
|
133 |
do_put_xxx(obj, field, is_field); |
14621
fd9265ab0f67
7172640: C2: instrinsic implementations in LibraryCallKit should use argument() instead of pop()
twisti
parents:
13963
diff
changeset
|
134 |
(void) pop(); // pop receiver after putting |
1 | 135 |
} |
136 |
} else { |
|
8725
8c1e3dd5fe1b
7017732: move static fields into Class to prepare for perm gen removal
never
parents:
8676
diff
changeset
|
137 |
const TypeInstPtr* tip = TypeInstPtr::make(field_holder->java_mirror()); |
8c1e3dd5fe1b
7017732: move static fields into Class to prepare for perm gen removal
never
parents:
8676
diff
changeset
|
138 |
obj = _gvn.makecon(tip); |
1 | 139 |
if (is_get) { |
8725
8c1e3dd5fe1b
7017732: move static fields into Class to prepare for perm gen removal
never
parents:
8676
diff
changeset
|
140 |
do_get_xxx(obj, field, is_field); |
1 | 141 |
} else { |
8725
8c1e3dd5fe1b
7017732: move static fields into Class to prepare for perm gen removal
never
parents:
8676
diff
changeset
|
142 |
do_put_xxx(obj, field, is_field); |
1 | 143 |
} |
144 |
} |
|
145 |
} |
|
146 |
||
147 |
||
8725
8c1e3dd5fe1b
7017732: move static fields into Class to prepare for perm gen removal
never
parents:
8676
diff
changeset
|
148 |
void Parse::do_get_xxx(Node* obj, ciField* field, bool is_field) { |
1 | 149 |
// Does this field have a constant value? If so, just push the value. |
4567
7fc02fbe5c7a
6893268: additional dynamic language related optimizations in C2
twisti
parents:
4470
diff
changeset
|
150 |
if (field->is_constant()) { |
19770
7cb9f982ea81
8001107: @Stable annotation for constant folding of lazily evaluated variables
vlivanov
parents:
18956
diff
changeset
|
151 |
// final or stable field |
31857
adbf29d9ca43
8078629: VM should constant fold Unsafe.get*() loads from final fields
vlivanov
parents:
31035
diff
changeset
|
152 |
const Type* con_type = Type::make_constant(field, obj); |
adbf29d9ca43
8078629: VM should constant fold Unsafe.get*() loads from final fields
vlivanov
parents:
31035
diff
changeset
|
153 |
if (con_type != NULL) { |
adbf29d9ca43
8078629: VM should constant fold Unsafe.get*() loads from final fields
vlivanov
parents:
31035
diff
changeset
|
154 |
push_node(con_type->basic_type(), makecon(con_type)); |
adbf29d9ca43
8078629: VM should constant fold Unsafe.get*() loads from final fields
vlivanov
parents:
31035
diff
changeset
|
155 |
return; |
4567
7fc02fbe5c7a
6893268: additional dynamic language related optimizations in C2
twisti
parents:
4470
diff
changeset
|
156 |
} |
7fc02fbe5c7a
6893268: additional dynamic language related optimizations in C2
twisti
parents:
4470
diff
changeset
|
157 |
} |
1 | 158 |
|
159 |
ciType* field_klass = field->type(); |
|
160 |
bool is_vol = field->is_volatile(); |
|
161 |
||
162 |
// Compute address and memory type. |
|
163 |
int offset = field->offset_in_bytes(); |
|
164 |
const TypePtr* adr_type = C->alias_type(field)->adr_type(); |
|
165 |
Node *adr = basic_plus_adr(obj, obj, offset); |
|
166 |
BasicType bt = field->layout_type(); |
|
167 |
||
168 |
// Build the resultant type of the load |
|
169 |
const Type *type; |
|
170 |
||
171 |
bool must_assert_null = false; |
|
172 |
||
173 |
if( bt == T_OBJECT ) { |
|
174 |
if (!field->type()->is_loaded()) { |
|
175 |
type = TypeInstPtr::BOTTOM; |
|
176 |
must_assert_null = true; |
|
4567
7fc02fbe5c7a
6893268: additional dynamic language related optimizations in C2
twisti
parents:
4470
diff
changeset
|
177 |
} else if (field->is_constant() && field->is_static()) { |
1 | 178 |
// This can happen if the constant oop is non-perm. |
179 |
ciObject* con = field->constant_value().as_object(); |
|
180 |
// Do not "join" in the previous type; it doesn't add value, |
|
181 |
// and may yield a vacuous result if the field is of interface type. |
|
182 |
type = TypeOopPtr::make_from_constant(con)->isa_oopptr(); |
|
183 |
assert(type != NULL, "field singleton type must be consistent"); |
|
184 |
} else { |
|
185 |
type = TypeOopPtr::make_from_klass(field_klass->as_klass()); |
|
186 |
} |
|
187 |
} else { |
|
188 |
type = Type::get_const_basic_type(bt); |
|
189 |
} |
|
22868
7f6eb436873b
8029101: PPC64 (part 211): ordering of Independent Reads of Independent Writes
goetz
parents:
22845
diff
changeset
|
190 |
if (support_IRIW_for_not_multiple_copy_atomic_cpu && field->is_volatile()) { |
7f6eb436873b
8029101: PPC64 (part 211): ordering of Independent Reads of Independent Writes
goetz
parents:
22845
diff
changeset
|
191 |
insert_mem_bar(Op_MemBarVolatile); // StoreLoad barrier |
7f6eb436873b
8029101: PPC64 (part 211): ordering of Independent Reads of Independent Writes
goetz
parents:
22845
diff
changeset
|
192 |
} |
1 | 193 |
// Build the load. |
22845
d8812d0ff387
8024921: PPC64 (part 113): Extend Load and Store nodes to know about memory ordering
goetz
parents:
19770
diff
changeset
|
194 |
// |
d8812d0ff387
8024921: PPC64 (part 113): Extend Load and Store nodes to know about memory ordering
goetz
parents:
19770
diff
changeset
|
195 |
MemNode::MemOrd mo = is_vol ? MemNode::acquire : MemNode::unordered; |
23189
27cf1316709b
8033380: Experimental VM flag to enforce access atomicity
shade
parents:
22873
diff
changeset
|
196 |
bool needs_atomic_access = is_vol || AlwaysAtomicAccesses; |
31035
0f0743952c41
8077504: Unsafe load can loose control dependency and cause crash
roland
parents:
24923
diff
changeset
|
197 |
Node* ld = make_load(NULL, adr, type, bt, adr_type, mo, LoadNode::DependsOnlyOnTest, needs_atomic_access); |
1 | 198 |
|
199 |
// Adjust Java stack |
|
200 |
if (type2size[bt] == 1) |
|
201 |
push(ld); |
|
202 |
else |
|
203 |
push_pair(ld); |
|
204 |
||
205 |
if (must_assert_null) { |
|
206 |
// Do not take a trap here. It's possible that the program |
|
207 |
// will never load the field's class, and will happily see |
|
208 |
// null values in this field forever. Don't stumble into a |
|
209 |
// trap for such a program, or we might get a long series |
|
210 |
// of useless recompilations. (Or, we might load a class |
|
211 |
// which should not be loaded.) If we ever see a non-null |
|
212 |
// value, we will then trap and recompile. (The trap will |
|
213 |
// not need to mention the class index, since the class will |
|
214 |
// already have been loaded if we ever see a non-null value.) |
|
215 |
// uncommon_trap(iter().get_field_signature_index()); |
|
216 |
if (PrintOpto && (Verbose || WizardMode)) { |
|
217 |
method()->print_name(); tty->print_cr(" asserting nullness of field at bci: %d", bci()); |
|
218 |
} |
|
219 |
if (C->log() != NULL) { |
|
220 |
C->log()->elem("assert_null reason='field' klass='%d'", |
|
221 |
C->log()->identify(field->type())); |
|
222 |
} |
|
223 |
// If there is going to be a trap, put it at the next bytecode: |
|
224 |
set_bci(iter().next_bci()); |
|
14621
fd9265ab0f67
7172640: C2: instrinsic implementations in LibraryCallKit should use argument() instead of pop()
twisti
parents:
13963
diff
changeset
|
225 |
null_assert(peek()); |
1 | 226 |
set_bci(iter().cur_bci()); // put it back |
227 |
} |
|
228 |
||
229 |
// If reference is volatile, prevent following memory ops from |
|
230 |
// floating up past the volatile read. Also prevents commoning |
|
231 |
// another volatile read. |
|
232 |
if (field->is_volatile()) { |
|
233 |
// Memory barrier includes bogus read of value to force load BEFORE membar |
|
234 |
insert_mem_bar(Op_MemBarAcquire, ld); |
|
235 |
} |
|
236 |
} |
|
237 |
||
8725
8c1e3dd5fe1b
7017732: move static fields into Class to prepare for perm gen removal
never
parents:
8676
diff
changeset
|
238 |
void Parse::do_put_xxx(Node* obj, ciField* field, bool is_field) { |
1 | 239 |
bool is_vol = field->is_volatile(); |
240 |
// If reference is volatile, prevent following memory ops from |
|
241 |
// floating down past the volatile write. Also prevents commoning |
|
242 |
// another volatile read. |
|
243 |
if (is_vol) insert_mem_bar(Op_MemBarRelease); |
|
244 |
||
245 |
// Compute address and memory type. |
|
246 |
int offset = field->offset_in_bytes(); |
|
247 |
const TypePtr* adr_type = C->alias_type(field)->adr_type(); |
|
248 |
Node* adr = basic_plus_adr(obj, obj, offset); |
|
249 |
BasicType bt = field->layout_type(); |
|
250 |
// Value to be stored |
|
251 |
Node* val = type2size[bt] == 1 ? pop() : pop_pair(); |
|
252 |
// Round doubles before storing |
|
253 |
if (bt == T_DOUBLE) val = dstore_rounding(val); |
|
254 |
||
22845
d8812d0ff387
8024921: PPC64 (part 113): Extend Load and Store nodes to know about memory ordering
goetz
parents:
19770
diff
changeset
|
255 |
// Conservatively release stores of object references. |
d8812d0ff387
8024921: PPC64 (part 113): Extend Load and Store nodes to know about memory ordering
goetz
parents:
19770
diff
changeset
|
256 |
const MemNode::MemOrd mo = |
d8812d0ff387
8024921: PPC64 (part 113): Extend Load and Store nodes to know about memory ordering
goetz
parents:
19770
diff
changeset
|
257 |
is_vol ? |
d8812d0ff387
8024921: PPC64 (part 113): Extend Load and Store nodes to know about memory ordering
goetz
parents:
19770
diff
changeset
|
258 |
// Volatile fields need releasing stores. |
d8812d0ff387
8024921: PPC64 (part 113): Extend Load and Store nodes to know about memory ordering
goetz
parents:
19770
diff
changeset
|
259 |
MemNode::release : |
d8812d0ff387
8024921: PPC64 (part 113): Extend Load and Store nodes to know about memory ordering
goetz
parents:
19770
diff
changeset
|
260 |
// Non-volatile fields also need releasing stores if they hold an |
d8812d0ff387
8024921: PPC64 (part 113): Extend Load and Store nodes to know about memory ordering
goetz
parents:
19770
diff
changeset
|
261 |
// object reference, because the object reference might point to |
d8812d0ff387
8024921: PPC64 (part 113): Extend Load and Store nodes to know about memory ordering
goetz
parents:
19770
diff
changeset
|
262 |
// a freshly created object. |
d8812d0ff387
8024921: PPC64 (part 113): Extend Load and Store nodes to know about memory ordering
goetz
parents:
19770
diff
changeset
|
263 |
StoreNode::release_if_reference(bt); |
d8812d0ff387
8024921: PPC64 (part 113): Extend Load and Store nodes to know about memory ordering
goetz
parents:
19770
diff
changeset
|
264 |
|
1 | 265 |
// Store the value. |
266 |
Node* store; |
|
267 |
if (bt == T_OBJECT) { |
|
3178 | 268 |
const TypeOopPtr* field_type; |
1 | 269 |
if (!field->type()->is_loaded()) { |
270 |
field_type = TypeInstPtr::BOTTOM; |
|
271 |
} else { |
|
272 |
field_type = TypeOopPtr::make_from_klass(field->type()->as_klass()); |
|
273 |
} |
|
22845
d8812d0ff387
8024921: PPC64 (part 113): Extend Load and Store nodes to know about memory ordering
goetz
parents:
19770
diff
changeset
|
274 |
store = store_oop_to_object(control(), obj, adr, adr_type, val, field_type, bt, mo); |
1 | 275 |
} else { |
23189
27cf1316709b
8033380: Experimental VM flag to enforce access atomicity
shade
parents:
22873
diff
changeset
|
276 |
bool needs_atomic_access = is_vol || AlwaysAtomicAccesses; |
27cf1316709b
8033380: Experimental VM flag to enforce access atomicity
shade
parents:
22873
diff
changeset
|
277 |
store = store_to_memory(control(), adr, val, bt, adr_type, mo, needs_atomic_access); |
1 | 278 |
} |
279 |
||
280 |
// If reference is volatile, prevent following volatiles ops from |
|
281 |
// floating up before the volatile write. |
|
282 |
if (is_vol) { |
|
22868
7f6eb436873b
8029101: PPC64 (part 211): ordering of Independent Reads of Independent Writes
goetz
parents:
22845
diff
changeset
|
283 |
// If not multiple copy atomic, we do the MemBarVolatile before the load. |
7f6eb436873b
8029101: PPC64 (part 211): ordering of Independent Reads of Independent Writes
goetz
parents:
22845
diff
changeset
|
284 |
if (!support_IRIW_for_not_multiple_copy_atomic_cpu) { |
7f6eb436873b
8029101: PPC64 (part 211): ordering of Independent Reads of Independent Writes
goetz
parents:
22845
diff
changeset
|
285 |
insert_mem_bar(Op_MemBarVolatile); // Use fat membar |
7f6eb436873b
8029101: PPC64 (part 211): ordering of Independent Reads of Independent Writes
goetz
parents:
22845
diff
changeset
|
286 |
} |
7f6eb436873b
8029101: PPC64 (part 211): ordering of Independent Reads of Independent Writes
goetz
parents:
22845
diff
changeset
|
287 |
// Remember we wrote a volatile field. |
7f6eb436873b
8029101: PPC64 (part 211): ordering of Independent Reads of Independent Writes
goetz
parents:
22845
diff
changeset
|
288 |
// For not multiple copy atomic cpu (ppc64) a barrier should be issued |
7f6eb436873b
8029101: PPC64 (part 211): ordering of Independent Reads of Independent Writes
goetz
parents:
22845
diff
changeset
|
289 |
// in constructors which have such stores. See do_exits() in parse1.cpp. |
7f6eb436873b
8029101: PPC64 (part 211): ordering of Independent Reads of Independent Writes
goetz
parents:
22845
diff
changeset
|
290 |
if (is_field) { |
7f6eb436873b
8029101: PPC64 (part 211): ordering of Independent Reads of Independent Writes
goetz
parents:
22845
diff
changeset
|
291 |
set_wrote_volatile(true); |
7f6eb436873b
8029101: PPC64 (part 211): ordering of Independent Reads of Independent Writes
goetz
parents:
22845
diff
changeset
|
292 |
} |
1 | 293 |
} |
294 |
||
23190
e8bbf9cd711e
8031818: Experimental VM flag for enforcing safe object construction
shade
parents:
23189
diff
changeset
|
295 |
if (is_field) { |
e8bbf9cd711e
8031818: Experimental VM flag for enforcing safe object construction
shade
parents:
23189
diff
changeset
|
296 |
set_wrote_fields(true); |
e8bbf9cd711e
8031818: Experimental VM flag for enforcing safe object construction
shade
parents:
23189
diff
changeset
|
297 |
} |
e8bbf9cd711e
8031818: Experimental VM flag for enforcing safe object construction
shade
parents:
23189
diff
changeset
|
298 |
|
1 | 299 |
// If the field is final, the rules of Java say we are in <init> or <clinit>. |
300 |
// Note the presence of writes to final non-static fields, so that we |
|
301 |
// can insert a memory barrier later on to keep the writes from floating |
|
302 |
// out of the constructor. |
|
19770
7cb9f982ea81
8001107: @Stable annotation for constant folding of lazily evaluated variables
vlivanov
parents:
18956
diff
changeset
|
303 |
// Any method can write a @Stable field; insert memory barriers after those also. |
7cb9f982ea81
8001107: @Stable annotation for constant folding of lazily evaluated variables
vlivanov
parents:
18956
diff
changeset
|
304 |
if (is_field && (field->is_final() || field->is_stable())) { |
23190
e8bbf9cd711e
8031818: Experimental VM flag for enforcing safe object construction
shade
parents:
23189
diff
changeset
|
305 |
if (field->is_final()) { |
e8bbf9cd711e
8031818: Experimental VM flag for enforcing safe object construction
shade
parents:
23189
diff
changeset
|
306 |
set_wrote_final(true); |
e8bbf9cd711e
8031818: Experimental VM flag for enforcing safe object construction
shade
parents:
23189
diff
changeset
|
307 |
} |
e8bbf9cd711e
8031818: Experimental VM flag for enforcing safe object construction
shade
parents:
23189
diff
changeset
|
308 |
if (field->is_stable()) { |
e8bbf9cd711e
8031818: Experimental VM flag for enforcing safe object construction
shade
parents:
23189
diff
changeset
|
309 |
set_wrote_stable(true); |
e8bbf9cd711e
8031818: Experimental VM flag for enforcing safe object construction
shade
parents:
23189
diff
changeset
|
310 |
} |
e8bbf9cd711e
8031818: Experimental VM flag for enforcing safe object construction
shade
parents:
23189
diff
changeset
|
311 |
|
17383 | 312 |
// Preserve allocation ptr to create precedent edge to it in membar |
313 |
// generated on exit from constructor. |
|
34183
8b683720a3e4
8139758: [REDO] Elide more final field's write memory barrier with escape analysis result
hshi
parents:
34174
diff
changeset
|
314 |
// Can't bind stable with its allocation, only record allocation for final field. |
8b683720a3e4
8139758: [REDO] Elide more final field's write memory barrier with escape analysis result
hshi
parents:
34174
diff
changeset
|
315 |
if (field->is_final() && AllocateNode::Ideal_allocation(obj, &_gvn) != NULL) { |
33196
3d84eedbd82c
8139750: [BACKOUT] Elide more final field's write memory barrier with escape analysis result
roland
parents:
33180
diff
changeset
|
316 |
set_alloc_with_final(obj); |
17383 | 317 |
} |
1 | 318 |
} |
319 |
} |
|
320 |
||
321 |
//============================================================================= |
|
322 |
void Parse::do_anewarray() { |
|
323 |
bool will_link; |
|
324 |
ciKlass* klass = iter().get_klass(will_link); |
|
325 |
||
326 |
// Uncommon Trap when class that array contains is not loaded |
|
327 |
// we need the loaded class for the rest of graph; do not |
|
328 |
// initialize the container class (see Java spec)!!! |
|
329 |
assert(will_link, "anewarray: typeflow responsibility"); |
|
330 |
||
331 |
ciObjArrayKlass* array_klass = ciObjArrayKlass::make(klass); |
|
332 |
// Check that array_klass object is loaded |
|
333 |
if (!array_klass->is_loaded()) { |
|
334 |
// Generate uncommon_trap for unloaded array_class |
|
335 |
uncommon_trap(Deoptimization::Reason_unloaded, |
|
336 |
Deoptimization::Action_reinterpret, |
|
337 |
array_klass); |
|
338 |
return; |
|
339 |
} |
|
340 |
||
341 |
kill_dead_locals(); |
|
342 |
||
343 |
const TypeKlassPtr* array_klass_type = TypeKlassPtr::make(array_klass); |
|
344 |
Node* count_val = pop(); |
|
2574
1d5f85c2d755
6589834: deoptimization problem with -XX:+DeoptimizeALot
cfang
parents:
670
diff
changeset
|
345 |
Node* obj = new_array(makecon(array_klass_type), count_val, 1); |
1 | 346 |
push(obj); |
347 |
} |
|
348 |
||
349 |
||
350 |
void Parse::do_newarray(BasicType elem_type) { |
|
351 |
kill_dead_locals(); |
|
352 |
||
353 |
Node* count_val = pop(); |
|
354 |
const TypeKlassPtr* array_klass = TypeKlassPtr::make(ciTypeArrayKlass::make(elem_type)); |
|
2574
1d5f85c2d755
6589834: deoptimization problem with -XX:+DeoptimizeALot
cfang
parents:
670
diff
changeset
|
355 |
Node* obj = new_array(makecon(array_klass), count_val, 1); |
1 | 356 |
// Push resultant oop onto stack |
357 |
push(obj); |
|
358 |
} |
|
359 |
||
360 |
// Expand simple expressions like new int[3][5] and new Object[2][nonConLen]. |
|
361 |
// Also handle the degenerate 1-dimensional case of anewarray. |
|
2574
1d5f85c2d755
6589834: deoptimization problem with -XX:+DeoptimizeALot
cfang
parents:
670
diff
changeset
|
362 |
Node* Parse::expand_multianewarray(ciArrayKlass* array_klass, Node* *lengths, int ndimensions, int nargs) { |
1 | 363 |
Node* length = lengths[0]; |
364 |
assert(length != NULL, ""); |
|
2574
1d5f85c2d755
6589834: deoptimization problem with -XX:+DeoptimizeALot
cfang
parents:
670
diff
changeset
|
365 |
Node* array = new_array(makecon(TypeKlassPtr::make(array_klass)), length, nargs); |
1 | 366 |
if (ndimensions > 1) { |
367 |
jint length_con = find_int_con(length, -1); |
|
368 |
guarantee(length_con >= 0, "non-constant multianewarray"); |
|
369 |
ciArrayKlass* array_klass_1 = array_klass->as_obj_array_klass()->element_klass()->as_array_klass(); |
|
370 |
const TypePtr* adr_type = TypeAryPtr::OOPS; |
|
3180
c589129153a4
6856025: assert(_base >= OopPtr && _base <= KlassPtr,"Not a Java pointer")
never
parents:
3178
diff
changeset
|
371 |
const TypeOopPtr* elemtype = _gvn.type(array)->is_aryptr()->elem()->make_oopptr(); |
1 | 372 |
const intptr_t header = arrayOopDesc::base_offset_in_bytes(T_OBJECT); |
373 |
for (jint i = 0; i < length_con; i++) { |
|
2574
1d5f85c2d755
6589834: deoptimization problem with -XX:+DeoptimizeALot
cfang
parents:
670
diff
changeset
|
374 |
Node* elem = expand_multianewarray(array_klass_1, &lengths[1], ndimensions-1, nargs); |
360
21d113ecbf6a
6420645: Create a vm that uses compressed oops for up to 32gb heapsizes
coleenp
parents:
353
diff
changeset
|
375 |
intptr_t offset = header + ((intptr_t)i << LogBytesPerHeapOop); |
1 | 376 |
Node* eaddr = basic_plus_adr(array, offset); |
22845
d8812d0ff387
8024921: PPC64 (part 113): Extend Load and Store nodes to know about memory ordering
goetz
parents:
19770
diff
changeset
|
377 |
store_oop_to_array(control(), array, eaddr, adr_type, elem, elemtype, T_OBJECT, MemNode::unordered); |
1 | 378 |
} |
379 |
} |
|
380 |
return array; |
|
381 |
} |
|
382 |
||
383 |
void Parse::do_multianewarray() { |
|
384 |
int ndimensions = iter().get_dimensions(); |
|
385 |
||
386 |
// the m-dimensional array |
|
387 |
bool will_link; |
|
388 |
ciArrayKlass* array_klass = iter().get_klass(will_link)->as_array_klass(); |
|
389 |
assert(will_link, "multianewarray: typeflow responsibility"); |
|
390 |
||
391 |
// Note: Array classes are always initialized; no is_initialized check. |
|
392 |
||
393 |
kill_dead_locals(); |
|
394 |
||
395 |
// get the lengths from the stack (first dimension is on top) |
|
10028
035159a868a1
7058510: multinewarray with 6 dimensions uncommon traps in server compiler
iveresov
parents:
8725
diff
changeset
|
396 |
Node** length = NEW_RESOURCE_ARRAY(Node*, ndimensions + 1); |
1 | 397 |
length[ndimensions] = NULL; // terminating null for make_runtime_call |
398 |
int j; |
|
399 |
for (j = ndimensions-1; j >= 0 ; j--) length[j] = pop(); |
|
400 |
||
401 |
// The original expression was of this form: new T[length0][length1]... |
|
402 |
// It is often the case that the lengths are small (except the last). |
|
403 |
// If that happens, use the fast 1-d creator a constant number of times. |
|
33163
9e128b399e48
8078554: Compiler: implement ranges (optionally constraints) for those flags that have them missing
zmajo
parents:
31857
diff
changeset
|
404 |
const jint expand_limit = MIN2((jint)MultiArrayExpandLimit, 100); |
1 | 405 |
jint expand_count = 1; // count of allocations in the expansion |
406 |
jint expand_fanout = 1; // running total fanout |
|
407 |
for (j = 0; j < ndimensions-1; j++) { |
|
408 |
jint dim_con = find_int_con(length[j], -1); |
|
409 |
expand_fanout *= dim_con; |
|
410 |
expand_count += expand_fanout; // count the level-J sub-arrays |
|
353 | 411 |
if (dim_con <= 0 |
1 | 412 |
|| dim_con > expand_limit |
413 |
|| expand_count > expand_limit) { |
|
414 |
expand_count = 0; |
|
415 |
break; |
|
416 |
} |
|
417 |
} |
|
418 |
||
419 |
// Can use multianewarray instead of [a]newarray if only one dimension, |
|
420 |
// or if all non-final dimensions are small constants. |
|
4893
fedc27b54caa
6910605: C2: NullPointerException/ClassCaseException is thrown when C2 with DeoptimizeALot is used
kvn
parents:
4567
diff
changeset
|
421 |
if (ndimensions == 1 || (1 <= expand_count && expand_count <= expand_limit)) { |
fedc27b54caa
6910605: C2: NullPointerException/ClassCaseException is thrown when C2 with DeoptimizeALot is used
kvn
parents:
4567
diff
changeset
|
422 |
Node* obj = NULL; |
fedc27b54caa
6910605: C2: NullPointerException/ClassCaseException is thrown when C2 with DeoptimizeALot is used
kvn
parents:
4567
diff
changeset
|
423 |
// Set the original stack and the reexecute bit for the interpreter |
fedc27b54caa
6910605: C2: NullPointerException/ClassCaseException is thrown when C2 with DeoptimizeALot is used
kvn
parents:
4567
diff
changeset
|
424 |
// to reexecute the multianewarray bytecode if deoptimization happens. |
fedc27b54caa
6910605: C2: NullPointerException/ClassCaseException is thrown when C2 with DeoptimizeALot is used
kvn
parents:
4567
diff
changeset
|
425 |
// Do it unconditionally even for one dimension multianewarray. |
fedc27b54caa
6910605: C2: NullPointerException/ClassCaseException is thrown when C2 with DeoptimizeALot is used
kvn
parents:
4567
diff
changeset
|
426 |
// Note: the reexecute bit will be set in GraphKit::add_safepoint_edges() |
fedc27b54caa
6910605: C2: NullPointerException/ClassCaseException is thrown when C2 with DeoptimizeALot is used
kvn
parents:
4567
diff
changeset
|
427 |
// when AllocateArray node for newarray is created. |
fedc27b54caa
6910605: C2: NullPointerException/ClassCaseException is thrown when C2 with DeoptimizeALot is used
kvn
parents:
4567
diff
changeset
|
428 |
{ PreserveReexecuteState preexecs(this); |
14621
fd9265ab0f67
7172640: C2: instrinsic implementations in LibraryCallKit should use argument() instead of pop()
twisti
parents:
13963
diff
changeset
|
429 |
inc_sp(ndimensions); |
4893
fedc27b54caa
6910605: C2: NullPointerException/ClassCaseException is thrown when C2 with DeoptimizeALot is used
kvn
parents:
4567
diff
changeset
|
430 |
// Pass 0 as nargs since uncommon trap code does not need to restore stack. |
fedc27b54caa
6910605: C2: NullPointerException/ClassCaseException is thrown when C2 with DeoptimizeALot is used
kvn
parents:
4567
diff
changeset
|
431 |
obj = expand_multianewarray(array_klass, &length[0], ndimensions, 0); |
fedc27b54caa
6910605: C2: NullPointerException/ClassCaseException is thrown when C2 with DeoptimizeALot is used
kvn
parents:
4567
diff
changeset
|
432 |
} //original reexecute and sp are set back here |
1 | 433 |
push(obj); |
434 |
return; |
|
435 |
} |
|
436 |
||
437 |
address fun = NULL; |
|
438 |
switch (ndimensions) { |
|
10028
035159a868a1
7058510: multinewarray with 6 dimensions uncommon traps in server compiler
iveresov
parents:
8725
diff
changeset
|
439 |
case 1: ShouldNotReachHere(); break; |
1 | 440 |
case 2: fun = OptoRuntime::multianewarray2_Java(); break; |
441 |
case 3: fun = OptoRuntime::multianewarray3_Java(); break; |
|
442 |
case 4: fun = OptoRuntime::multianewarray4_Java(); break; |
|
443 |
case 5: fun = OptoRuntime::multianewarray5_Java(); break; |
|
444 |
}; |
|
10028
035159a868a1
7058510: multinewarray with 6 dimensions uncommon traps in server compiler
iveresov
parents:
8725
diff
changeset
|
445 |
Node* c = NULL; |
1 | 446 |
|
10028
035159a868a1
7058510: multinewarray with 6 dimensions uncommon traps in server compiler
iveresov
parents:
8725
diff
changeset
|
447 |
if (fun != NULL) { |
035159a868a1
7058510: multinewarray with 6 dimensions uncommon traps in server compiler
iveresov
parents:
8725
diff
changeset
|
448 |
c = make_runtime_call(RC_NO_LEAF | RC_NO_IO, |
035159a868a1
7058510: multinewarray with 6 dimensions uncommon traps in server compiler
iveresov
parents:
8725
diff
changeset
|
449 |
OptoRuntime::multianewarray_Type(ndimensions), |
035159a868a1
7058510: multinewarray with 6 dimensions uncommon traps in server compiler
iveresov
parents:
8725
diff
changeset
|
450 |
fun, NULL, TypeRawPtr::BOTTOM, |
035159a868a1
7058510: multinewarray with 6 dimensions uncommon traps in server compiler
iveresov
parents:
8725
diff
changeset
|
451 |
makecon(TypeKlassPtr::make(array_klass)), |
035159a868a1
7058510: multinewarray with 6 dimensions uncommon traps in server compiler
iveresov
parents:
8725
diff
changeset
|
452 |
length[0], length[1], length[2], |
15480
a9e84da84519
8006807: C2 crash due to out of bounds array access in Parse::do_multianewarray
drchase
parents:
14835
diff
changeset
|
453 |
(ndimensions > 2) ? length[3] : NULL, |
a9e84da84519
8006807: C2 crash due to out of bounds array access in Parse::do_multianewarray
drchase
parents:
14835
diff
changeset
|
454 |
(ndimensions > 3) ? length[4] : NULL); |
10028
035159a868a1
7058510: multinewarray with 6 dimensions uncommon traps in server compiler
iveresov
parents:
8725
diff
changeset
|
455 |
} else { |
035159a868a1
7058510: multinewarray with 6 dimensions uncommon traps in server compiler
iveresov
parents:
8725
diff
changeset
|
456 |
// Create a java array for dimension sizes |
035159a868a1
7058510: multinewarray with 6 dimensions uncommon traps in server compiler
iveresov
parents:
8725
diff
changeset
|
457 |
Node* dims = NULL; |
035159a868a1
7058510: multinewarray with 6 dimensions uncommon traps in server compiler
iveresov
parents:
8725
diff
changeset
|
458 |
{ PreserveReexecuteState preexecs(this); |
14621
fd9265ab0f67
7172640: C2: instrinsic implementations in LibraryCallKit should use argument() instead of pop()
twisti
parents:
13963
diff
changeset
|
459 |
inc_sp(ndimensions); |
10028
035159a868a1
7058510: multinewarray with 6 dimensions uncommon traps in server compiler
iveresov
parents:
8725
diff
changeset
|
460 |
Node* dims_array_klass = makecon(TypeKlassPtr::make(ciArrayKlass::make(ciType::make(T_INT)))); |
035159a868a1
7058510: multinewarray with 6 dimensions uncommon traps in server compiler
iveresov
parents:
8725
diff
changeset
|
461 |
dims = new_array(dims_array_klass, intcon(ndimensions), 0); |
035159a868a1
7058510: multinewarray with 6 dimensions uncommon traps in server compiler
iveresov
parents:
8725
diff
changeset
|
462 |
|
035159a868a1
7058510: multinewarray with 6 dimensions uncommon traps in server compiler
iveresov
parents:
8725
diff
changeset
|
463 |
// Fill-in it with values |
035159a868a1
7058510: multinewarray with 6 dimensions uncommon traps in server compiler
iveresov
parents:
8725
diff
changeset
|
464 |
for (j = 0; j < ndimensions; j++) { |
035159a868a1
7058510: multinewarray with 6 dimensions uncommon traps in server compiler
iveresov
parents:
8725
diff
changeset
|
465 |
Node *dims_elem = array_element_address(dims, intcon(j), T_INT); |
22845
d8812d0ff387
8024921: PPC64 (part 113): Extend Load and Store nodes to know about memory ordering
goetz
parents:
19770
diff
changeset
|
466 |
store_to_memory(control(), dims_elem, length[j], T_INT, TypeAryPtr::INTS, MemNode::unordered); |
10028
035159a868a1
7058510: multinewarray with 6 dimensions uncommon traps in server compiler
iveresov
parents:
8725
diff
changeset
|
467 |
} |
035159a868a1
7058510: multinewarray with 6 dimensions uncommon traps in server compiler
iveresov
parents:
8725
diff
changeset
|
468 |
} |
035159a868a1
7058510: multinewarray with 6 dimensions uncommon traps in server compiler
iveresov
parents:
8725
diff
changeset
|
469 |
|
035159a868a1
7058510: multinewarray with 6 dimensions uncommon traps in server compiler
iveresov
parents:
8725
diff
changeset
|
470 |
c = make_runtime_call(RC_NO_LEAF | RC_NO_IO, |
035159a868a1
7058510: multinewarray with 6 dimensions uncommon traps in server compiler
iveresov
parents:
8725
diff
changeset
|
471 |
OptoRuntime::multianewarrayN_Type(), |
035159a868a1
7058510: multinewarray with 6 dimensions uncommon traps in server compiler
iveresov
parents:
8725
diff
changeset
|
472 |
OptoRuntime::multianewarrayN_Java(), NULL, TypeRawPtr::BOTTOM, |
035159a868a1
7058510: multinewarray with 6 dimensions uncommon traps in server compiler
iveresov
parents:
8725
diff
changeset
|
473 |
makecon(TypeKlassPtr::make(array_klass)), |
035159a868a1
7058510: multinewarray with 6 dimensions uncommon traps in server compiler
iveresov
parents:
8725
diff
changeset
|
474 |
dims); |
035159a868a1
7058510: multinewarray with 6 dimensions uncommon traps in server compiler
iveresov
parents:
8725
diff
changeset
|
475 |
} |
14835
70896cb93c35
8004741: Missing compiled exception handle table entry for multidimensional array allocation
kvn
parents:
14621
diff
changeset
|
476 |
make_slow_call_ex(c, env()->Throwable_klass(), false); |
10028
035159a868a1
7058510: multinewarray with 6 dimensions uncommon traps in server compiler
iveresov
parents:
8725
diff
changeset
|
477 |
|
24923
9631f7d691dc
8034812: remove IDX_INIT macro hack in Node class
thartmann
parents:
23528
diff
changeset
|
478 |
Node* res = _gvn.transform(new ProjNode(c, TypeFunc::Parms)); |
1 | 479 |
|
480 |
const Type* type = TypeOopPtr::make_from_klass_raw(array_klass); |
|
481 |
||
482 |
// Improve the type: We know it's not null, exact, and of a given length. |
|
483 |
type = type->is_ptr()->cast_to_ptr_type(TypePtr::NotNull); |
|
484 |
type = type->is_aryptr()->cast_to_exactness(true); |
|
485 |
||
486 |
const TypeInt* ltype = _gvn.find_int_type(length[0]); |
|
487 |
if (ltype != NULL) |
|
488 |
type = type->is_aryptr()->cast_to_size(ltype); |
|
489 |
||
10028
035159a868a1
7058510: multinewarray with 6 dimensions uncommon traps in server compiler
iveresov
parents:
8725
diff
changeset
|
490 |
// We cannot sharpen the nested sub-arrays, since the top level is mutable. |
1 | 491 |
|
24923
9631f7d691dc
8034812: remove IDX_INIT macro hack in Node class
thartmann
parents:
23528
diff
changeset
|
492 |
Node* cast = _gvn.transform( new CheckCastPPNode(control(), res, type) ); |
1 | 493 |
push(cast); |
494 |
||
495 |
// Possible improvements: |
|
496 |
// - Make a fast path for small multi-arrays. (W/ implicit init. loops.) |
|
497 |
// - Issue CastII against length[*] values, to TypeInt::POS. |
|
498 |
} |