author | twisti |
Tue, 24 Jul 2012 10:51:00 -0700 | |
changeset 13391 | 30245956af37 |
parent 11956 | c363dc1ac4b7 |
permissions | -rw-r--r-- |
1 | 1 |
/* |
11956
c363dc1ac4b7
7148126: ConstantPoolCacheEntry::print prints to wrong stream
dholmes
parents:
10546
diff
changeset
|
2 |
* Copyright (c) 1998, 2012, 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:
5420
diff
changeset
|
19 |
* Please contact Oracle, 500 Oracle Parkway, Redwood Shores, CA 94065 USA |
f4b087cbb361
6941466: Oracle rebranding changes for Hotspot repositories
trims
parents:
5420
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:
5420
diff
changeset
|
21 |
* questions. |
1 | 22 |
* |
23 |
*/ |
|
24 |
||
7397 | 25 |
#include "precompiled.hpp" |
26 |
#include "gc_implementation/shared/markSweep.inline.hpp" |
|
27 |
#include "interpreter/interpreter.hpp" |
|
28 |
#include "interpreter/rewriter.hpp" |
|
29 |
#include "memory/universe.inline.hpp" |
|
30 |
#include "oops/cpCacheOop.hpp" |
|
31 |
#include "oops/objArrayOop.hpp" |
|
32 |
#include "oops/oop.inline.hpp" |
|
33 |
#include "prims/jvmtiRedefineClassesTrace.hpp" |
|
13391
30245956af37
7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents:
11956
diff
changeset
|
34 |
#include "prims/methodHandles.hpp" |
7397 | 35 |
#include "runtime/handles.inline.hpp" |
1 | 36 |
|
37 |
||
38 |
// Implememtation of ConstantPoolCacheEntry |
|
39 |
||
4429
d7eb4e2099aa
6858164: invokedynamic code needs some cleanup (post-6655638)
jrose
parents:
3261
diff
changeset
|
40 |
void ConstantPoolCacheEntry::initialize_entry(int index) { |
2570
ecc7862946d4
6655646: dynamic languages need dynamically linked call sites
jrose
parents:
670
diff
changeset
|
41 |
assert(0 < index && index < 0x10000, "sanity check"); |
1 | 42 |
_indices = index; |
2570
ecc7862946d4
6655646: dynamic languages need dynamically linked call sites
jrose
parents:
670
diff
changeset
|
43 |
assert(constant_pool_index() == index, ""); |
1 | 44 |
} |
45 |
||
4429
d7eb4e2099aa
6858164: invokedynamic code needs some cleanup (post-6655638)
jrose
parents:
3261
diff
changeset
|
46 |
void ConstantPoolCacheEntry::initialize_secondary_entry(int main_index) { |
d7eb4e2099aa
6858164: invokedynamic code needs some cleanup (post-6655638)
jrose
parents:
3261
diff
changeset
|
47 |
assert(0 <= main_index && main_index < 0x10000, "sanity check"); |
13391
30245956af37
7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents:
11956
diff
changeset
|
48 |
_indices = (main_index << main_cp_index_bits); |
4429
d7eb4e2099aa
6858164: invokedynamic code needs some cleanup (post-6655638)
jrose
parents:
3261
diff
changeset
|
49 |
assert(main_entry_index() == main_index, ""); |
d7eb4e2099aa
6858164: invokedynamic code needs some cleanup (post-6655638)
jrose
parents:
3261
diff
changeset
|
50 |
} |
1 | 51 |
|
13391
30245956af37
7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents:
11956
diff
changeset
|
52 |
int ConstantPoolCacheEntry::make_flags(TosState state, |
30245956af37
7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents:
11956
diff
changeset
|
53 |
int option_bits, |
30245956af37
7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents:
11956
diff
changeset
|
54 |
int field_index_or_method_params) { |
30245956af37
7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents:
11956
diff
changeset
|
55 |
assert(state < number_of_states, "Invalid state in make_flags"); |
30245956af37
7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents:
11956
diff
changeset
|
56 |
int f = ((int)state << tos_state_shift) | option_bits | field_index_or_method_params; |
1 | 57 |
// Preserve existing flag bit values |
13391
30245956af37
7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents:
11956
diff
changeset
|
58 |
// The low bits are a field offset, or else the method parameter size. |
1 | 59 |
#ifdef ASSERT |
13391
30245956af37
7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents:
11956
diff
changeset
|
60 |
TosState old_state = flag_state(); |
30245956af37
7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents:
11956
diff
changeset
|
61 |
assert(old_state == (TosState)0 || old_state == state, |
1 | 62 |
"inconsistent cpCache flags state"); |
63 |
#endif |
|
64 |
return (_flags | f) ; |
|
65 |
} |
|
66 |
||
67 |
void ConstantPoolCacheEntry::set_bytecode_1(Bytecodes::Code code) { |
|
13391
30245956af37
7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents:
11956
diff
changeset
|
68 |
assert(!is_secondary_entry(), "must not overwrite main_entry_index"); |
1 | 69 |
#ifdef ASSERT |
70 |
// Read once. |
|
71 |
volatile Bytecodes::Code c = bytecode_1(); |
|
72 |
assert(c == 0 || c == code || code == 0, "update must be consistent"); |
|
73 |
#endif |
|
74 |
// Need to flush pending stores here before bytecode is written. |
|
13391
30245956af37
7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents:
11956
diff
changeset
|
75 |
OrderAccess::release_store_ptr(&_indices, _indices | ((u_char)code << bytecode_1_shift)); |
1 | 76 |
} |
77 |
||
78 |
void ConstantPoolCacheEntry::set_bytecode_2(Bytecodes::Code code) { |
|
13391
30245956af37
7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents:
11956
diff
changeset
|
79 |
assert(!is_secondary_entry(), "must not overwrite main_entry_index"); |
1 | 80 |
#ifdef ASSERT |
81 |
// Read once. |
|
82 |
volatile Bytecodes::Code c = bytecode_2(); |
|
83 |
assert(c == 0 || c == code || code == 0, "update must be consistent"); |
|
84 |
#endif |
|
85 |
// Need to flush pending stores here before bytecode is written. |
|
13391
30245956af37
7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents:
11956
diff
changeset
|
86 |
OrderAccess::release_store_ptr(&_indices, _indices | ((u_char)code << bytecode_2_shift)); |
1 | 87 |
} |
88 |
||
13391
30245956af37
7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents:
11956
diff
changeset
|
89 |
// Sets f1, ordering with previous writes. |
30245956af37
7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents:
11956
diff
changeset
|
90 |
void ConstantPoolCacheEntry::release_set_f1(oop f1) { |
8316
0816329cc3ee
7012339: JSR 292 crash in G1SATBCardTableModRefBS::write_ref_field_pre_work()
twisti
parents:
7397
diff
changeset
|
91 |
// Use barriers as in oop_store |
13391
30245956af37
7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents:
11956
diff
changeset
|
92 |
assert(f1 != NULL, ""); |
8316
0816329cc3ee
7012339: JSR 292 crash in G1SATBCardTableModRefBS::write_ref_field_pre_work()
twisti
parents:
7397
diff
changeset
|
93 |
oop* f1_addr = (oop*) &_f1; |
0816329cc3ee
7012339: JSR 292 crash in G1SATBCardTableModRefBS::write_ref_field_pre_work()
twisti
parents:
7397
diff
changeset
|
94 |
update_barrier_set_pre(f1_addr, f1); |
13391
30245956af37
7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents:
11956
diff
changeset
|
95 |
OrderAccess::release_store_ptr((intptr_t*)f1_addr, f1); |
30245956af37
7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents:
11956
diff
changeset
|
96 |
update_barrier_set((void*) f1_addr, f1); |
30245956af37
7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents:
11956
diff
changeset
|
97 |
} |
30245956af37
7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents:
11956
diff
changeset
|
98 |
|
30245956af37
7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents:
11956
diff
changeset
|
99 |
// Sets flags, but only if the value was previously zero. |
30245956af37
7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents:
11956
diff
changeset
|
100 |
bool ConstantPoolCacheEntry::init_flags_atomic(intptr_t flags) { |
30245956af37
7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents:
11956
diff
changeset
|
101 |
intptr_t result = Atomic::cmpxchg_ptr(flags, &_flags, 0); |
30245956af37
7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents:
11956
diff
changeset
|
102 |
return (result == 0); |
8316
0816329cc3ee
7012339: JSR 292 crash in G1SATBCardTableModRefBS::write_ref_field_pre_work()
twisti
parents:
7397
diff
changeset
|
103 |
} |
7104 | 104 |
|
1 | 105 |
#ifdef ASSERT |
106 |
// It is possible to have two different dummy methodOops created |
|
107 |
// when the resolve code for invoke interface executes concurrently |
|
108 |
// Hence the assertion below is weakened a bit for the invokeinterface |
|
109 |
// case. |
|
110 |
bool ConstantPoolCacheEntry::same_methodOop(oop cur_f1, oop f1) { |
|
111 |
return (cur_f1 == f1 || ((methodOop)cur_f1)->name() == |
|
112 |
((methodOop)f1)->name() || ((methodOop)cur_f1)->signature() == |
|
113 |
((methodOop)f1)->signature()); |
|
114 |
} |
|
115 |
#endif |
|
116 |
||
117 |
// Note that concurrent update of both bytecodes can leave one of them |
|
118 |
// reset to zero. This is harmless; the interpreter will simply re-resolve |
|
119 |
// the damaged entry. More seriously, the memory synchronization is needed |
|
120 |
// to flush other fields (f1, f2) completely to memory before the bytecodes |
|
121 |
// are updated, lest other processors see a non-zero bytecode but zero f1/f2. |
|
122 |
void ConstantPoolCacheEntry::set_field(Bytecodes::Code get_code, |
|
123 |
Bytecodes::Code put_code, |
|
124 |
KlassHandle field_holder, |
|
10546 | 125 |
int field_index, |
1 | 126 |
int field_offset, |
127 |
TosState field_type, |
|
128 |
bool is_final, |
|
129 |
bool is_volatile) { |
|
8725
8c1e3dd5fe1b
7017732: move static fields into Class to prepare for perm gen removal
never
parents:
8334
diff
changeset
|
130 |
set_f1(field_holder()->java_mirror()); |
1 | 131 |
set_f2(field_offset); |
13391
30245956af37
7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents:
11956
diff
changeset
|
132 |
assert((field_index & field_index_mask) == field_index, |
1 | 133 |
"field index does not fit in low flag bits"); |
13391
30245956af37
7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents:
11956
diff
changeset
|
134 |
set_field_flags(field_type, |
30245956af37
7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents:
11956
diff
changeset
|
135 |
((is_volatile ? 1 : 0) << is_volatile_shift) | |
30245956af37
7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents:
11956
diff
changeset
|
136 |
((is_final ? 1 : 0) << is_final_shift), |
30245956af37
7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents:
11956
diff
changeset
|
137 |
field_index); |
1 | 138 |
set_bytecode_1(get_code); |
139 |
set_bytecode_2(put_code); |
|
140 |
NOT_PRODUCT(verify(tty)); |
|
141 |
} |
|
142 |
||
13391
30245956af37
7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents:
11956
diff
changeset
|
143 |
void ConstantPoolCacheEntry::set_parameter_size(int value) { |
30245956af37
7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents:
11956
diff
changeset
|
144 |
// This routine is called only in corner cases where the CPCE is not yet initialized. |
30245956af37
7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents:
11956
diff
changeset
|
145 |
// See AbstractInterpreter::deopt_continue_after_entry. |
30245956af37
7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents:
11956
diff
changeset
|
146 |
assert(_flags == 0 || parameter_size() == 0 || parameter_size() == value, |
30245956af37
7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents:
11956
diff
changeset
|
147 |
err_msg("size must not change: parameter_size=%d, value=%d", parameter_size(), value)); |
30245956af37
7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents:
11956
diff
changeset
|
148 |
// Setting the parameter size by itself is only safe if the |
30245956af37
7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents:
11956
diff
changeset
|
149 |
// current value of _flags is 0, otherwise another thread may have |
30245956af37
7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents:
11956
diff
changeset
|
150 |
// updated it and we don't want to overwrite that value. Don't |
30245956af37
7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents:
11956
diff
changeset
|
151 |
// bother trying to update it once it's nonzero but always make |
30245956af37
7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents:
11956
diff
changeset
|
152 |
// sure that the final parameter size agrees with what was passed. |
30245956af37
7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents:
11956
diff
changeset
|
153 |
if (_flags == 0) { |
30245956af37
7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents:
11956
diff
changeset
|
154 |
Atomic::cmpxchg_ptr((value & parameter_size_mask), &_flags, 0); |
30245956af37
7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents:
11956
diff
changeset
|
155 |
} |
30245956af37
7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents:
11956
diff
changeset
|
156 |
guarantee(parameter_size() == value, |
30245956af37
7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents:
11956
diff
changeset
|
157 |
err_msg("size must not change: parameter_size=%d, value=%d", parameter_size(), value)); |
1 | 158 |
} |
159 |
||
160 |
void ConstantPoolCacheEntry::set_method(Bytecodes::Code invoke_code, |
|
161 |
methodHandle method, |
|
162 |
int vtable_index) { |
|
6062
bab93afe9df7
6964498: JSR 292 invokedynamic sites need local bootstrap methods
jrose
parents:
5547
diff
changeset
|
163 |
assert(!is_secondary_entry(), ""); |
1 | 164 |
assert(method->interpreter_entry() != NULL, "should have been set at this point"); |
165 |
assert(!method->is_obsolete(), "attempt to write obsolete method to cpCache"); |
|
166 |
||
167 |
int byte_no = -1; |
|
13391
30245956af37
7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents:
11956
diff
changeset
|
168 |
bool change_to_virtual = false; |
30245956af37
7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents:
11956
diff
changeset
|
169 |
|
1 | 170 |
switch (invoke_code) { |
13391
30245956af37
7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents:
11956
diff
changeset
|
171 |
case Bytecodes::_invokeinterface: |
30245956af37
7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents:
11956
diff
changeset
|
172 |
// We get here from InterpreterRuntime::resolve_invoke when an invokeinterface |
30245956af37
7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents:
11956
diff
changeset
|
173 |
// instruction somehow links to a non-interface method (in Object). |
30245956af37
7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents:
11956
diff
changeset
|
174 |
// In that case, the method has no itable index and must be invoked as a virtual. |
30245956af37
7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents:
11956
diff
changeset
|
175 |
// Set a flag to keep track of this corner case. |
30245956af37
7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents:
11956
diff
changeset
|
176 |
change_to_virtual = true; |
30245956af37
7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents:
11956
diff
changeset
|
177 |
|
30245956af37
7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents:
11956
diff
changeset
|
178 |
// ...and fall through as if we were handling invokevirtual: |
1 | 179 |
case Bytecodes::_invokevirtual: |
13391
30245956af37
7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents:
11956
diff
changeset
|
180 |
{ |
1 | 181 |
if (method->can_be_statically_bound()) { |
13391
30245956af37
7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents:
11956
diff
changeset
|
182 |
// set_f2_as_vfinal_method checks if is_vfinal flag is true. |
30245956af37
7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents:
11956
diff
changeset
|
183 |
set_method_flags(as_TosState(method->result_type()), |
30245956af37
7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents:
11956
diff
changeset
|
184 |
( 1 << is_vfinal_shift) | |
30245956af37
7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents:
11956
diff
changeset
|
185 |
((method->is_final_method() ? 1 : 0) << is_final_shift) | |
30245956af37
7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents:
11956
diff
changeset
|
186 |
((change_to_virtual ? 1 : 0) << is_forced_virtual_shift), |
30245956af37
7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents:
11956
diff
changeset
|
187 |
method()->size_of_parameters()); |
30245956af37
7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents:
11956
diff
changeset
|
188 |
set_f2_as_vfinal_method(method()); |
1 | 189 |
} else { |
190 |
assert(vtable_index >= 0, "valid index"); |
|
13391
30245956af37
7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents:
11956
diff
changeset
|
191 |
assert(!method->is_final_method(), "sanity"); |
30245956af37
7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents:
11956
diff
changeset
|
192 |
set_method_flags(as_TosState(method->result_type()), |
30245956af37
7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents:
11956
diff
changeset
|
193 |
((change_to_virtual ? 1 : 0) << is_forced_virtual_shift), |
30245956af37
7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents:
11956
diff
changeset
|
194 |
method()->size_of_parameters()); |
1 | 195 |
set_f2(vtable_index); |
196 |
} |
|
197 |
byte_no = 2; |
|
198 |
break; |
|
6062
bab93afe9df7
6964498: JSR 292 invokedynamic sites need local bootstrap methods
jrose
parents:
5547
diff
changeset
|
199 |
} |
bab93afe9df7
6964498: JSR 292 invokedynamic sites need local bootstrap methods
jrose
parents:
5547
diff
changeset
|
200 |
|
1 | 201 |
case Bytecodes::_invokespecial: |
202 |
case Bytecodes::_invokestatic: |
|
13391
30245956af37
7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents:
11956
diff
changeset
|
203 |
// Note: Read and preserve the value of the is_vfinal flag on any |
30245956af37
7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents:
11956
diff
changeset
|
204 |
// invokevirtual bytecode shared with this constant pool cache entry. |
30245956af37
7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents:
11956
diff
changeset
|
205 |
// It is cheap and safe to consult is_vfinal() at all times. |
30245956af37
7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents:
11956
diff
changeset
|
206 |
// Once is_vfinal is set, it must stay that way, lest we get a dangling oop. |
30245956af37
7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents:
11956
diff
changeset
|
207 |
set_method_flags(as_TosState(method->result_type()), |
30245956af37
7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents:
11956
diff
changeset
|
208 |
((is_vfinal() ? 1 : 0) << is_vfinal_shift) | |
30245956af37
7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents:
11956
diff
changeset
|
209 |
((method->is_final_method() ? 1 : 0) << is_final_shift), |
30245956af37
7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents:
11956
diff
changeset
|
210 |
method()->size_of_parameters()); |
1 | 211 |
set_f1(method()); |
212 |
byte_no = 1; |
|
213 |
break; |
|
214 |
default: |
|
215 |
ShouldNotReachHere(); |
|
216 |
break; |
|
217 |
} |
|
218 |
||
219 |
// Note: byte_no also appears in TemplateTable::resolve. |
|
220 |
if (byte_no == 1) { |
|
13391
30245956af37
7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents:
11956
diff
changeset
|
221 |
assert(invoke_code != Bytecodes::_invokevirtual && |
30245956af37
7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents:
11956
diff
changeset
|
222 |
invoke_code != Bytecodes::_invokeinterface, ""); |
1 | 223 |
set_bytecode_1(invoke_code); |
224 |
} else if (byte_no == 2) { |
|
225 |
if (change_to_virtual) { |
|
13391
30245956af37
7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents:
11956
diff
changeset
|
226 |
assert(invoke_code == Bytecodes::_invokeinterface, ""); |
1 | 227 |
// NOTE: THIS IS A HACK - BE VERY CAREFUL!!! |
228 |
// |
|
229 |
// Workaround for the case where we encounter an invokeinterface, but we |
|
230 |
// should really have an _invokevirtual since the resolved method is a |
|
231 |
// virtual method in java.lang.Object. This is a corner case in the spec |
|
232 |
// but is presumably legal. javac does not generate this code. |
|
233 |
// |
|
234 |
// We set bytecode_1() to _invokeinterface, because that is the |
|
235 |
// bytecode # used by the interpreter to see if it is resolved. |
|
236 |
// We set bytecode_2() to _invokevirtual. |
|
237 |
// See also interpreterRuntime.cpp. (8/25/2000) |
|
238 |
// Only set resolved for the invokeinterface case if method is public. |
|
239 |
// Otherwise, the method needs to be reresolved with caller for each |
|
240 |
// interface call. |
|
241 |
if (method->is_public()) set_bytecode_1(invoke_code); |
|
242 |
} else { |
|
13391
30245956af37
7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents:
11956
diff
changeset
|
243 |
assert(invoke_code == Bytecodes::_invokevirtual, ""); |
1 | 244 |
} |
13391
30245956af37
7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents:
11956
diff
changeset
|
245 |
// set up for invokevirtual, even if linking for invokeinterface also: |
30245956af37
7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents:
11956
diff
changeset
|
246 |
set_bytecode_2(Bytecodes::_invokevirtual); |
1 | 247 |
} else { |
248 |
ShouldNotReachHere(); |
|
249 |
} |
|
250 |
NOT_PRODUCT(verify(tty)); |
|
251 |
} |
|
252 |
||
253 |
||
254 |
void ConstantPoolCacheEntry::set_interface_call(methodHandle method, int index) { |
|
6062
bab93afe9df7
6964498: JSR 292 invokedynamic sites need local bootstrap methods
jrose
parents:
5547
diff
changeset
|
255 |
assert(!is_secondary_entry(), ""); |
1 | 256 |
klassOop interf = method->method_holder(); |
257 |
assert(instanceKlass::cast(interf)->is_interface(), "must be an interface"); |
|
13391
30245956af37
7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents:
11956
diff
changeset
|
258 |
assert(!method->is_final_method(), "interfaces do not have final methods; cannot link to one here"); |
1 | 259 |
set_f1(interf); |
260 |
set_f2(index); |
|
13391
30245956af37
7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents:
11956
diff
changeset
|
261 |
set_method_flags(as_TosState(method->result_type()), |
30245956af37
7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents:
11956
diff
changeset
|
262 |
0, // no option bits |
30245956af37
7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents:
11956
diff
changeset
|
263 |
method()->size_of_parameters()); |
1 | 264 |
set_bytecode_1(Bytecodes::_invokeinterface); |
265 |
} |
|
266 |
||
267 |
||
13391
30245956af37
7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents:
11956
diff
changeset
|
268 |
void ConstantPoolCacheEntry::set_method_handle(methodHandle adapter, Handle appendix) { |
30245956af37
7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents:
11956
diff
changeset
|
269 |
assert(!is_secondary_entry(), ""); |
30245956af37
7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents:
11956
diff
changeset
|
270 |
set_method_handle_common(Bytecodes::_invokehandle, adapter, appendix); |
6062
bab93afe9df7
6964498: JSR 292 invokedynamic sites need local bootstrap methods
jrose
parents:
5547
diff
changeset
|
271 |
} |
bab93afe9df7
6964498: JSR 292 invokedynamic sites need local bootstrap methods
jrose
parents:
5547
diff
changeset
|
272 |
|
13391
30245956af37
7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents:
11956
diff
changeset
|
273 |
void ConstantPoolCacheEntry::set_dynamic_call(methodHandle adapter, Handle appendix) { |
30245956af37
7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents:
11956
diff
changeset
|
274 |
assert(is_secondary_entry(), ""); |
30245956af37
7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents:
11956
diff
changeset
|
275 |
set_method_handle_common(Bytecodes::_invokedynamic, adapter, appendix); |
6062
bab93afe9df7
6964498: JSR 292 invokedynamic sites need local bootstrap methods
jrose
parents:
5547
diff
changeset
|
276 |
} |
bab93afe9df7
6964498: JSR 292 invokedynamic sites need local bootstrap methods
jrose
parents:
5547
diff
changeset
|
277 |
|
13391
30245956af37
7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents:
11956
diff
changeset
|
278 |
void ConstantPoolCacheEntry::set_method_handle_common(Bytecodes::Code invoke_code, methodHandle adapter, Handle appendix) { |
30245956af37
7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents:
11956
diff
changeset
|
279 |
// NOTE: This CPCE can be the subject of data races. |
30245956af37
7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents:
11956
diff
changeset
|
280 |
// There are three words to update: flags, f2, f1 (in that order). |
30245956af37
7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents:
11956
diff
changeset
|
281 |
// Writers must store all other values before f1. |
30245956af37
7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents:
11956
diff
changeset
|
282 |
// Readers must test f1 first for non-null before reading other fields. |
30245956af37
7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents:
11956
diff
changeset
|
283 |
// Competing writers must acquire exclusive access on the first |
30245956af37
7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents:
11956
diff
changeset
|
284 |
// write, to flags, using a compare/exchange. |
30245956af37
7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents:
11956
diff
changeset
|
285 |
// A losing writer must spin until the winner writes f1, |
30245956af37
7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents:
11956
diff
changeset
|
286 |
// so that when he returns, he can use the linked cache entry. |
30245956af37
7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents:
11956
diff
changeset
|
287 |
|
30245956af37
7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents:
11956
diff
changeset
|
288 |
bool has_appendix = appendix.not_null(); |
30245956af37
7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents:
11956
diff
changeset
|
289 |
if (!has_appendix) { |
30245956af37
7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents:
11956
diff
changeset
|
290 |
// The extra argument is not used, but we need a non-null value to signify linkage state. |
30245956af37
7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents:
11956
diff
changeset
|
291 |
// Set it to something benign that will never leak memory. |
30245956af37
7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents:
11956
diff
changeset
|
292 |
appendix = Universe::void_mirror(); |
30245956af37
7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents:
11956
diff
changeset
|
293 |
} |
30245956af37
7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents:
11956
diff
changeset
|
294 |
|
30245956af37
7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents:
11956
diff
changeset
|
295 |
bool owner = |
30245956af37
7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents:
11956
diff
changeset
|
296 |
init_method_flags_atomic(as_TosState(adapter->result_type()), |
30245956af37
7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents:
11956
diff
changeset
|
297 |
((has_appendix ? 1 : 0) << has_appendix_shift) | |
30245956af37
7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents:
11956
diff
changeset
|
298 |
( 1 << is_vfinal_shift) | |
30245956af37
7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents:
11956
diff
changeset
|
299 |
( 1 << is_final_shift), |
30245956af37
7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents:
11956
diff
changeset
|
300 |
adapter->size_of_parameters()); |
30245956af37
7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents:
11956
diff
changeset
|
301 |
if (!owner) { |
30245956af37
7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents:
11956
diff
changeset
|
302 |
while (is_f1_null()) { |
30245956af37
7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents:
11956
diff
changeset
|
303 |
// Pause momentarily on a low-level lock, to allow racing thread to win. |
30245956af37
7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents:
11956
diff
changeset
|
304 |
MutexLockerEx mu(Patching_lock, Mutex::_no_safepoint_check_flag); |
30245956af37
7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents:
11956
diff
changeset
|
305 |
os::yield(); |
30245956af37
7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents:
11956
diff
changeset
|
306 |
} |
30245956af37
7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents:
11956
diff
changeset
|
307 |
return; |
30245956af37
7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents:
11956
diff
changeset
|
308 |
} |
30245956af37
7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents:
11956
diff
changeset
|
309 |
|
30245956af37
7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents:
11956
diff
changeset
|
310 |
if (TraceInvokeDynamic) { |
30245956af37
7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents:
11956
diff
changeset
|
311 |
tty->print_cr("set_method_handle bc=%d appendix="PTR_FORMAT"%s method="PTR_FORMAT" ", |
30245956af37
7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents:
11956
diff
changeset
|
312 |
invoke_code, |
30245956af37
7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents:
11956
diff
changeset
|
313 |
(intptr_t)appendix(), (has_appendix ? "" : " (unused)"), |
30245956af37
7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents:
11956
diff
changeset
|
314 |
(intptr_t)adapter()); |
30245956af37
7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents:
11956
diff
changeset
|
315 |
adapter->print(); |
30245956af37
7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents:
11956
diff
changeset
|
316 |
if (has_appendix) appendix()->print(); |
30245956af37
7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents:
11956
diff
changeset
|
317 |
} |
30245956af37
7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents:
11956
diff
changeset
|
318 |
|
30245956af37
7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents:
11956
diff
changeset
|
319 |
// Method handle invokes and invokedynamic sites use both cp cache words. |
30245956af37
7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents:
11956
diff
changeset
|
320 |
// f1, if not null, contains a value passed as a trailing argument to the adapter. |
30245956af37
7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents:
11956
diff
changeset
|
321 |
// In the general case, this could be the call site's MethodType, |
30245956af37
7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents:
11956
diff
changeset
|
322 |
// for use with java.lang.Invokers.checkExactType, or else a CallSite object. |
30245956af37
7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents:
11956
diff
changeset
|
323 |
// f2 contains the adapter method which manages the actual call. |
30245956af37
7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents:
11956
diff
changeset
|
324 |
// In the general case, this is a compiled LambdaForm. |
30245956af37
7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents:
11956
diff
changeset
|
325 |
// (The Java code is free to optimize these calls by binding other |
30245956af37
7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents:
11956
diff
changeset
|
326 |
// sorts of methods and appendices to call sites.) |
30245956af37
7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents:
11956
diff
changeset
|
327 |
// JVM-level linking is via f2, as if for invokevfinal, and signatures are erased. |
30245956af37
7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents:
11956
diff
changeset
|
328 |
// The appendix argument (if any) is added to the signature, and is counted in the parameter_size bits. |
30245956af37
7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents:
11956
diff
changeset
|
329 |
// In principle this means that the method (with appendix) could take up to 256 parameter slots. |
30245956af37
7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents:
11956
diff
changeset
|
330 |
// |
30245956af37
7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents:
11956
diff
changeset
|
331 |
// This means that given a call site like (List)mh.invoke("foo"), |
30245956af37
7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents:
11956
diff
changeset
|
332 |
// the f2 method has signature '(Ljl/Object;Ljl/invoke/MethodType;)Ljl/Object;', |
30245956af37
7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents:
11956
diff
changeset
|
333 |
// not '(Ljava/lang/String;)Ljava/util/List;'. |
30245956af37
7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents:
11956
diff
changeset
|
334 |
// The fact that String and List are involved is encoded in the MethodType in f1. |
30245956af37
7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents:
11956
diff
changeset
|
335 |
// This allows us to create fewer method oops, while keeping type safety. |
30245956af37
7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents:
11956
diff
changeset
|
336 |
// |
30245956af37
7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents:
11956
diff
changeset
|
337 |
set_f2_as_vfinal_method(adapter()); |
30245956af37
7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents:
11956
diff
changeset
|
338 |
assert(appendix.not_null(), "needed for linkage state"); |
30245956af37
7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents:
11956
diff
changeset
|
339 |
release_set_f1(appendix()); // This must be the last one to set (see NOTE above)! |
30245956af37
7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents:
11956
diff
changeset
|
340 |
if (!is_secondary_entry()) { |
30245956af37
7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents:
11956
diff
changeset
|
341 |
// The interpreter assembly code does not check byte_2, |
30245956af37
7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents:
11956
diff
changeset
|
342 |
// but it is used by is_resolved, method_if_resolved, etc. |
30245956af37
7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents:
11956
diff
changeset
|
343 |
set_bytecode_2(invoke_code); |
30245956af37
7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents:
11956
diff
changeset
|
344 |
} |
30245956af37
7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents:
11956
diff
changeset
|
345 |
NOT_PRODUCT(verify(tty)); |
30245956af37
7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents:
11956
diff
changeset
|
346 |
if (TraceInvokeDynamic) { |
30245956af37
7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents:
11956
diff
changeset
|
347 |
this->print(tty, 0); |
30245956af37
7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents:
11956
diff
changeset
|
348 |
} |
2570
ecc7862946d4
6655646: dynamic languages need dynamically linked call sites
jrose
parents:
670
diff
changeset
|
349 |
} |
ecc7862946d4
6655646: dynamic languages need dynamically linked call sites
jrose
parents:
670
diff
changeset
|
350 |
|
13391
30245956af37
7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents:
11956
diff
changeset
|
351 |
methodOop ConstantPoolCacheEntry::method_if_resolved(constantPoolHandle cpool) { |
10008
d84de97ad847
7056328: JSR 292 invocation sometimes fails in adapters for types not on boot class path
jrose
parents:
9328
diff
changeset
|
352 |
if (is_secondary_entry()) { |
13391
30245956af37
7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents:
11956
diff
changeset
|
353 |
if (!is_f1_null()) |
30245956af37
7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents:
11956
diff
changeset
|
354 |
return f2_as_vfinal_method(); |
30245956af37
7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents:
11956
diff
changeset
|
355 |
return NULL; |
10008
d84de97ad847
7056328: JSR 292 invocation sometimes fails in adapters for types not on boot class path
jrose
parents:
9328
diff
changeset
|
356 |
} |
d84de97ad847
7056328: JSR 292 invocation sometimes fails in adapters for types not on boot class path
jrose
parents:
9328
diff
changeset
|
357 |
// Decode the action of set_method and set_interface_call |
13391
30245956af37
7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents:
11956
diff
changeset
|
358 |
Bytecodes::Code invoke_code = bytecode_1(); |
30245956af37
7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents:
11956
diff
changeset
|
359 |
if (invoke_code != (Bytecodes::Code)0) { |
10008
d84de97ad847
7056328: JSR 292 invocation sometimes fails in adapters for types not on boot class path
jrose
parents:
9328
diff
changeset
|
360 |
oop f1 = _f1; |
d84de97ad847
7056328: JSR 292 invocation sometimes fails in adapters for types not on boot class path
jrose
parents:
9328
diff
changeset
|
361 |
if (f1 != NULL) { |
d84de97ad847
7056328: JSR 292 invocation sometimes fails in adapters for types not on boot class path
jrose
parents:
9328
diff
changeset
|
362 |
switch (invoke_code) { |
d84de97ad847
7056328: JSR 292 invocation sometimes fails in adapters for types not on boot class path
jrose
parents:
9328
diff
changeset
|
363 |
case Bytecodes::_invokeinterface: |
d84de97ad847
7056328: JSR 292 invocation sometimes fails in adapters for types not on boot class path
jrose
parents:
9328
diff
changeset
|
364 |
assert(f1->is_klass(), ""); |
13391
30245956af37
7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents:
11956
diff
changeset
|
365 |
return klassItable::method_for_itable_index(klassOop(f1), f2_as_index()); |
10008
d84de97ad847
7056328: JSR 292 invocation sometimes fails in adapters for types not on boot class path
jrose
parents:
9328
diff
changeset
|
366 |
case Bytecodes::_invokestatic: |
d84de97ad847
7056328: JSR 292 invocation sometimes fails in adapters for types not on boot class path
jrose
parents:
9328
diff
changeset
|
367 |
case Bytecodes::_invokespecial: |
13391
30245956af37
7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents:
11956
diff
changeset
|
368 |
assert(!has_appendix(), ""); |
10008
d84de97ad847
7056328: JSR 292 invocation sometimes fails in adapters for types not on boot class path
jrose
parents:
9328
diff
changeset
|
369 |
assert(f1->is_method(), ""); |
d84de97ad847
7056328: JSR 292 invocation sometimes fails in adapters for types not on boot class path
jrose
parents:
9328
diff
changeset
|
370 |
return methodOop(f1); |
d84de97ad847
7056328: JSR 292 invocation sometimes fails in adapters for types not on boot class path
jrose
parents:
9328
diff
changeset
|
371 |
} |
d84de97ad847
7056328: JSR 292 invocation sometimes fails in adapters for types not on boot class path
jrose
parents:
9328
diff
changeset
|
372 |
} |
d84de97ad847
7056328: JSR 292 invocation sometimes fails in adapters for types not on boot class path
jrose
parents:
9328
diff
changeset
|
373 |
} |
13391
30245956af37
7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents:
11956
diff
changeset
|
374 |
invoke_code = bytecode_2(); |
30245956af37
7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents:
11956
diff
changeset
|
375 |
if (invoke_code != (Bytecodes::Code)0) { |
10008
d84de97ad847
7056328: JSR 292 invocation sometimes fails in adapters for types not on boot class path
jrose
parents:
9328
diff
changeset
|
376 |
switch (invoke_code) { |
d84de97ad847
7056328: JSR 292 invocation sometimes fails in adapters for types not on boot class path
jrose
parents:
9328
diff
changeset
|
377 |
case Bytecodes::_invokevirtual: |
d84de97ad847
7056328: JSR 292 invocation sometimes fails in adapters for types not on boot class path
jrose
parents:
9328
diff
changeset
|
378 |
if (is_vfinal()) { |
d84de97ad847
7056328: JSR 292 invocation sometimes fails in adapters for types not on boot class path
jrose
parents:
9328
diff
changeset
|
379 |
// invokevirtual |
13391
30245956af37
7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents:
11956
diff
changeset
|
380 |
methodOop m = f2_as_vfinal_method(); |
10008
d84de97ad847
7056328: JSR 292 invocation sometimes fails in adapters for types not on boot class path
jrose
parents:
9328
diff
changeset
|
381 |
assert(m->is_method(), ""); |
d84de97ad847
7056328: JSR 292 invocation sometimes fails in adapters for types not on boot class path
jrose
parents:
9328
diff
changeset
|
382 |
return m; |
d84de97ad847
7056328: JSR 292 invocation sometimes fails in adapters for types not on boot class path
jrose
parents:
9328
diff
changeset
|
383 |
} else { |
d84de97ad847
7056328: JSR 292 invocation sometimes fails in adapters for types not on boot class path
jrose
parents:
9328
diff
changeset
|
384 |
int holder_index = cpool->uncached_klass_ref_index_at(constant_pool_index()); |
d84de97ad847
7056328: JSR 292 invocation sometimes fails in adapters for types not on boot class path
jrose
parents:
9328
diff
changeset
|
385 |
if (cpool->tag_at(holder_index).is_klass()) { |
d84de97ad847
7056328: JSR 292 invocation sometimes fails in adapters for types not on boot class path
jrose
parents:
9328
diff
changeset
|
386 |
klassOop klass = cpool->resolved_klass_at(holder_index); |
d84de97ad847
7056328: JSR 292 invocation sometimes fails in adapters for types not on boot class path
jrose
parents:
9328
diff
changeset
|
387 |
if (!Klass::cast(klass)->oop_is_instance()) |
d84de97ad847
7056328: JSR 292 invocation sometimes fails in adapters for types not on boot class path
jrose
parents:
9328
diff
changeset
|
388 |
klass = SystemDictionary::Object_klass(); |
13391
30245956af37
7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents:
11956
diff
changeset
|
389 |
return instanceKlass::cast(klass)->method_at_vtable(f2_as_index()); |
10008
d84de97ad847
7056328: JSR 292 invocation sometimes fails in adapters for types not on boot class path
jrose
parents:
9328
diff
changeset
|
390 |
} |
d84de97ad847
7056328: JSR 292 invocation sometimes fails in adapters for types not on boot class path
jrose
parents:
9328
diff
changeset
|
391 |
} |
13391
30245956af37
7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents:
11956
diff
changeset
|
392 |
break; |
30245956af37
7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents:
11956
diff
changeset
|
393 |
case Bytecodes::_invokehandle: |
30245956af37
7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents:
11956
diff
changeset
|
394 |
case Bytecodes::_invokedynamic: |
30245956af37
7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents:
11956
diff
changeset
|
395 |
return f2_as_vfinal_method(); |
10008
d84de97ad847
7056328: JSR 292 invocation sometimes fails in adapters for types not on boot class path
jrose
parents:
9328
diff
changeset
|
396 |
} |
d84de97ad847
7056328: JSR 292 invocation sometimes fails in adapters for types not on boot class path
jrose
parents:
9328
diff
changeset
|
397 |
} |
d84de97ad847
7056328: JSR 292 invocation sometimes fails in adapters for types not on boot class path
jrose
parents:
9328
diff
changeset
|
398 |
return NULL; |
d84de97ad847
7056328: JSR 292 invocation sometimes fails in adapters for types not on boot class path
jrose
parents:
9328
diff
changeset
|
399 |
} |
d84de97ad847
7056328: JSR 292 invocation sometimes fails in adapters for types not on boot class path
jrose
parents:
9328
diff
changeset
|
400 |
|
d84de97ad847
7056328: JSR 292 invocation sometimes fails in adapters for types not on boot class path
jrose
parents:
9328
diff
changeset
|
401 |
|
1 | 402 |
class LocalOopClosure: public OopClosure { |
403 |
private: |
|
404 |
void (*_f)(oop*); |
|
405 |
||
406 |
public: |
|
407 |
LocalOopClosure(void f(oop*)) { _f = f; } |
|
408 |
virtual void do_oop(oop* o) { _f(o); } |
|
360
21d113ecbf6a
6420645: Create a vm that uses compressed oops for up to 32gb heapsizes
coleenp
parents:
1
diff
changeset
|
409 |
virtual void do_oop(narrowOop *o) { ShouldNotReachHere(); } |
1 | 410 |
}; |
411 |
||
412 |
||
413 |
void ConstantPoolCacheEntry::oops_do(void f(oop*)) { |
|
414 |
LocalOopClosure blk(f); |
|
415 |
oop_iterate(&blk); |
|
416 |
} |
|
417 |
||
418 |
||
419 |
void ConstantPoolCacheEntry::oop_iterate(OopClosure* blk) { |
|
420 |
assert(in_words(size()) == 4, "check code below - may need adjustment"); |
|
421 |
// field[1] is always oop or NULL |
|
422 |
blk->do_oop((oop*)&_f1); |
|
423 |
if (is_vfinal()) { |
|
424 |
blk->do_oop((oop*)&_f2); |
|
425 |
} |
|
426 |
} |
|
427 |
||
428 |
||
429 |
void ConstantPoolCacheEntry::oop_iterate_m(OopClosure* blk, MemRegion mr) { |
|
430 |
assert(in_words(size()) == 4, "check code below - may need adjustment"); |
|
431 |
// field[1] is always oop or NULL |
|
432 |
if (mr.contains((oop *)&_f1)) blk->do_oop((oop*)&_f1); |
|
433 |
if (is_vfinal()) { |
|
434 |
if (mr.contains((oop *)&_f2)) blk->do_oop((oop*)&_f2); |
|
435 |
} |
|
436 |
} |
|
437 |
||
438 |
||
439 |
void ConstantPoolCacheEntry::follow_contents() { |
|
440 |
assert(in_words(size()) == 4, "check code below - may need adjustment"); |
|
441 |
// field[1] is always oop or NULL |
|
442 |
MarkSweep::mark_and_push((oop*)&_f1); |
|
443 |
if (is_vfinal()) { |
|
444 |
MarkSweep::mark_and_push((oop*)&_f2); |
|
445 |
} |
|
446 |
} |
|
447 |
||
448 |
#ifndef SERIALGC |
|
449 |
void ConstantPoolCacheEntry::follow_contents(ParCompactionManager* cm) { |
|
450 |
assert(in_words(size()) == 4, "check code below - may need adjustment"); |
|
451 |
// field[1] is always oop or NULL |
|
452 |
PSParallelCompact::mark_and_push(cm, (oop*)&_f1); |
|
453 |
if (is_vfinal()) { |
|
454 |
PSParallelCompact::mark_and_push(cm, (oop*)&_f2); |
|
455 |
} |
|
456 |
} |
|
457 |
#endif // SERIALGC |
|
458 |
||
459 |
void ConstantPoolCacheEntry::adjust_pointers() { |
|
460 |
assert(in_words(size()) == 4, "check code below - may need adjustment"); |
|
461 |
// field[1] is always oop or NULL |
|
462 |
MarkSweep::adjust_pointer((oop*)&_f1); |
|
463 |
if (is_vfinal()) { |
|
464 |
MarkSweep::adjust_pointer((oop*)&_f2); |
|
465 |
} |
|
466 |
} |
|
467 |
||
468 |
#ifndef SERIALGC |
|
469 |
void ConstantPoolCacheEntry::update_pointers() { |
|
470 |
assert(in_words(size()) == 4, "check code below - may need adjustment"); |
|
471 |
// field[1] is always oop or NULL |
|
472 |
PSParallelCompact::adjust_pointer((oop*)&_f1); |
|
473 |
if (is_vfinal()) { |
|
474 |
PSParallelCompact::adjust_pointer((oop*)&_f2); |
|
475 |
} |
|
476 |
} |
|
477 |
#endif // SERIALGC |
|
478 |
||
479 |
// RedefineClasses() API support: |
|
480 |
// If this constantPoolCacheEntry refers to old_method then update it |
|
481 |
// to refer to new_method. |
|
482 |
bool ConstantPoolCacheEntry::adjust_method_entry(methodOop old_method, |
|
483 |
methodOop new_method, bool * trace_name_printed) { |
|
484 |
||
485 |
if (is_vfinal()) { |
|
13391
30245956af37
7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents:
11956
diff
changeset
|
486 |
// virtual and final so _f2 contains method ptr instead of vtable index |
30245956af37
7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents:
11956
diff
changeset
|
487 |
if (f2_as_vfinal_method() == old_method) { |
1 | 488 |
// match old_method so need an update |
13391
30245956af37
7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents:
11956
diff
changeset
|
489 |
// NOTE: can't use set_f2_as_vfinal_method as it asserts on different values |
1 | 490 |
_f2 = (intptr_t)new_method; |
491 |
if (RC_TRACE_IN_RANGE(0x00100000, 0x00400000)) { |
|
492 |
if (!(*trace_name_printed)) { |
|
493 |
// RC_TRACE_MESG macro has an embedded ResourceMark |
|
494 |
RC_TRACE_MESG(("adjust: name=%s", |
|
495 |
Klass::cast(old_method->method_holder())->external_name())); |
|
496 |
*trace_name_printed = true; |
|
497 |
} |
|
498 |
// RC_TRACE macro has an embedded ResourceMark |
|
499 |
RC_TRACE(0x00400000, ("cpc vf-entry update: %s(%s)", |
|
500 |
new_method->name()->as_C_string(), |
|
501 |
new_method->signature()->as_C_string())); |
|
502 |
} |
|
503 |
||
504 |
return true; |
|
505 |
} |
|
506 |
||
507 |
// f1() is not used with virtual entries so bail out |
|
508 |
return false; |
|
509 |
} |
|
510 |
||
511 |
if ((oop)_f1 == NULL) { |
|
512 |
// NULL f1() means this is a virtual entry so bail out |
|
513 |
// We are assuming that the vtable index does not need change. |
|
514 |
return false; |
|
515 |
} |
|
516 |
||
517 |
if ((oop)_f1 == old_method) { |
|
518 |
_f1 = new_method; |
|
519 |
if (RC_TRACE_IN_RANGE(0x00100000, 0x00400000)) { |
|
520 |
if (!(*trace_name_printed)) { |
|
521 |
// RC_TRACE_MESG macro has an embedded ResourceMark |
|
522 |
RC_TRACE_MESG(("adjust: name=%s", |
|
523 |
Klass::cast(old_method->method_holder())->external_name())); |
|
524 |
*trace_name_printed = true; |
|
525 |
} |
|
526 |
// RC_TRACE macro has an embedded ResourceMark |
|
527 |
RC_TRACE(0x00400000, ("cpc entry update: %s(%s)", |
|
528 |
new_method->name()->as_C_string(), |
|
529 |
new_method->signature()->as_C_string())); |
|
530 |
} |
|
531 |
||
532 |
return true; |
|
533 |
} |
|
534 |
||
535 |
return false; |
|
536 |
} |
|
537 |
||
538 |
bool ConstantPoolCacheEntry::is_interesting_method_entry(klassOop k) { |
|
539 |
if (!is_method_entry()) { |
|
540 |
// not a method entry so not interesting by default |
|
541 |
return false; |
|
542 |
} |
|
543 |
||
544 |
methodOop m = NULL; |
|
545 |
if (is_vfinal()) { |
|
546 |
// virtual and final so _f2 contains method ptr instead of vtable index |
|
13391
30245956af37
7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents:
11956
diff
changeset
|
547 |
m = f2_as_vfinal_method(); |
30245956af37
7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents:
11956
diff
changeset
|
548 |
} else if (is_f1_null()) { |
1 | 549 |
// NULL _f1 means this is a virtual entry so also not interesting |
550 |
return false; |
|
551 |
} else { |
|
13391
30245956af37
7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents:
11956
diff
changeset
|
552 |
oop f1 = _f1; // _f1 is volatile |
30245956af37
7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents:
11956
diff
changeset
|
553 |
if (!f1->is_method()) { |
1 | 554 |
// _f1 can also contain a klassOop for an interface |
555 |
return false; |
|
556 |
} |
|
13391
30245956af37
7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents:
11956
diff
changeset
|
557 |
m = f1_as_method(); |
1 | 558 |
} |
559 |
||
560 |
assert(m != NULL && m->is_method(), "sanity check"); |
|
561 |
if (m == NULL || !m->is_method() || m->method_holder() != k) { |
|
562 |
// robustness for above sanity checks or method is not in |
|
563 |
// the interesting class |
|
564 |
return false; |
|
565 |
} |
|
566 |
||
567 |
// the method is in the interesting class so the entry is interesting |
|
568 |
return true; |
|
569 |
} |
|
570 |
||
571 |
void ConstantPoolCacheEntry::print(outputStream* st, int index) const { |
|
572 |
// print separator |
|
11956
c363dc1ac4b7
7148126: ConstantPoolCacheEntry::print prints to wrong stream
dholmes
parents:
10546
diff
changeset
|
573 |
if (index == 0) st->print_cr(" -------------"); |
1 | 574 |
// print entry |
11956
c363dc1ac4b7
7148126: ConstantPoolCacheEntry::print prints to wrong stream
dholmes
parents:
10546
diff
changeset
|
575 |
st->print("%3d ("PTR_FORMAT") ", index, (intptr_t)this); |
2570
ecc7862946d4
6655646: dynamic languages need dynamically linked call sites
jrose
parents:
670
diff
changeset
|
576 |
if (is_secondary_entry()) |
11956
c363dc1ac4b7
7148126: ConstantPoolCacheEntry::print prints to wrong stream
dholmes
parents:
10546
diff
changeset
|
577 |
st->print_cr("[%5d|secondary]", main_entry_index()); |
2570
ecc7862946d4
6655646: dynamic languages need dynamically linked call sites
jrose
parents:
670
diff
changeset
|
578 |
else |
11956
c363dc1ac4b7
7148126: ConstantPoolCacheEntry::print prints to wrong stream
dholmes
parents:
10546
diff
changeset
|
579 |
st->print_cr("[%02x|%02x|%5d]", bytecode_2(), bytecode_1(), constant_pool_index()); |
c363dc1ac4b7
7148126: ConstantPoolCacheEntry::print prints to wrong stream
dholmes
parents:
10546
diff
changeset
|
580 |
st->print_cr(" [ "PTR_FORMAT"]", (intptr_t)(oop)_f1); |
c363dc1ac4b7
7148126: ConstantPoolCacheEntry::print prints to wrong stream
dholmes
parents:
10546
diff
changeset
|
581 |
st->print_cr(" [ "PTR_FORMAT"]", (intptr_t)_f2); |
c363dc1ac4b7
7148126: ConstantPoolCacheEntry::print prints to wrong stream
dholmes
parents:
10546
diff
changeset
|
582 |
st->print_cr(" [ "PTR_FORMAT"]", (intptr_t)_flags); |
c363dc1ac4b7
7148126: ConstantPoolCacheEntry::print prints to wrong stream
dholmes
parents:
10546
diff
changeset
|
583 |
st->print_cr(" -------------"); |
1 | 584 |
} |
585 |
||
586 |
void ConstantPoolCacheEntry::verify(outputStream* st) const { |
|
587 |
// not implemented yet |
|
588 |
} |
|
589 |
||
590 |
// Implementation of ConstantPoolCache |
|
591 |
||
592 |
void constantPoolCacheOopDesc::initialize(intArray& inverse_index_map) { |
|
593 |
assert(inverse_index_map.length() == length(), "inverse index map must have same length as cache"); |
|
4429
d7eb4e2099aa
6858164: invokedynamic code needs some cleanup (post-6655638)
jrose
parents:
3261
diff
changeset
|
594 |
for (int i = 0; i < length(); i++) { |
d7eb4e2099aa
6858164: invokedynamic code needs some cleanup (post-6655638)
jrose
parents:
3261
diff
changeset
|
595 |
ConstantPoolCacheEntry* e = entry_at(i); |
d7eb4e2099aa
6858164: invokedynamic code needs some cleanup (post-6655638)
jrose
parents:
3261
diff
changeset
|
596 |
int original_index = inverse_index_map[i]; |
d7eb4e2099aa
6858164: invokedynamic code needs some cleanup (post-6655638)
jrose
parents:
3261
diff
changeset
|
597 |
if ((original_index & Rewriter::_secondary_entry_tag) != 0) { |
d7eb4e2099aa
6858164: invokedynamic code needs some cleanup (post-6655638)
jrose
parents:
3261
diff
changeset
|
598 |
int main_index = (original_index - Rewriter::_secondary_entry_tag); |
d7eb4e2099aa
6858164: invokedynamic code needs some cleanup (post-6655638)
jrose
parents:
3261
diff
changeset
|
599 |
assert(!entry_at(main_index)->is_secondary_entry(), "valid main index"); |
d7eb4e2099aa
6858164: invokedynamic code needs some cleanup (post-6655638)
jrose
parents:
3261
diff
changeset
|
600 |
e->initialize_secondary_entry(main_index); |
d7eb4e2099aa
6858164: invokedynamic code needs some cleanup (post-6655638)
jrose
parents:
3261
diff
changeset
|
601 |
} else { |
d7eb4e2099aa
6858164: invokedynamic code needs some cleanup (post-6655638)
jrose
parents:
3261
diff
changeset
|
602 |
e->initialize_entry(original_index); |
d7eb4e2099aa
6858164: invokedynamic code needs some cleanup (post-6655638)
jrose
parents:
3261
diff
changeset
|
603 |
} |
d7eb4e2099aa
6858164: invokedynamic code needs some cleanup (post-6655638)
jrose
parents:
3261
diff
changeset
|
604 |
assert(entry_at(i) == e, "sanity"); |
d7eb4e2099aa
6858164: invokedynamic code needs some cleanup (post-6655638)
jrose
parents:
3261
diff
changeset
|
605 |
} |
1 | 606 |
} |
607 |
||
608 |
// RedefineClasses() API support: |
|
609 |
// If any entry of this constantPoolCache points to any of |
|
610 |
// old_methods, replace it with the corresponding new_method. |
|
611 |
void constantPoolCacheOopDesc::adjust_method_entries(methodOop* old_methods, methodOop* new_methods, |
|
612 |
int methods_length, bool * trace_name_printed) { |
|
613 |
||
614 |
if (methods_length == 0) { |
|
615 |
// nothing to do if there are no methods |
|
616 |
return; |
|
617 |
} |
|
618 |
||
619 |
// get shorthand for the interesting class |
|
620 |
klassOop old_holder = old_methods[0]->method_holder(); |
|
621 |
||
622 |
for (int i = 0; i < length(); i++) { |
|
623 |
if (!entry_at(i)->is_interesting_method_entry(old_holder)) { |
|
624 |
// skip uninteresting methods |
|
625 |
continue; |
|
626 |
} |
|
627 |
||
628 |
// The constantPoolCache contains entries for several different |
|
629 |
// things, but we only care about methods. In fact, we only care |
|
630 |
// about methods in the same class as the one that contains the |
|
631 |
// old_methods. At this point, we have an interesting entry. |
|
632 |
||
633 |
for (int j = 0; j < methods_length; j++) { |
|
634 |
methodOop old_method = old_methods[j]; |
|
635 |
methodOop new_method = new_methods[j]; |
|
636 |
||
637 |
if (entry_at(i)->adjust_method_entry(old_method, new_method, |
|
638 |
trace_name_printed)) { |
|
639 |
// current old_method matched this entry and we updated it so |
|
640 |
// break out and get to the next interesting entry if there one |
|
641 |
break; |
|
642 |
} |
|
643 |
} |
|
644 |
} |
|
645 |
} |