author | coleenp |
Wed, 13 Nov 2019 08:23:23 -0500 | |
changeset 59056 | 15936b142f86 |
parent 58273 | 08a5148e7c4e |
permissions | -rw-r--r-- |
1 | 1 |
/* |
53345
91ab128a65a3
8216556: Unnecessary liveness computation with JVMTI
mdoerr
parents:
52934
diff
changeset
|
2 |
* Copyright (c) 1999, 2019, 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:
4892
diff
changeset
|
19 |
* Please contact Oracle, 500 Oracle Parkway, Redwood Shores, CA 94065 USA |
f4b087cbb361
6941466: Oracle rebranding changes for Hotspot repositories
trims
parents:
4892
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:
4892
diff
changeset
|
21 |
* questions. |
1 | 22 |
* |
23 |
*/ |
|
24 |
||
7397 | 25 |
#include "precompiled.hpp" |
26 |
#include "ci/ciCallProfile.hpp" |
|
27 |
#include "ci/ciExceptionHandler.hpp" |
|
28 |
#include "ci/ciInstanceKlass.hpp" |
|
29 |
#include "ci/ciMethod.hpp" |
|
30 |
#include "ci/ciMethodBlocks.hpp" |
|
31 |
#include "ci/ciMethodData.hpp" |
|
32 |
#include "ci/ciStreams.hpp" |
|
33 |
#include "ci/ciSymbol.hpp" |
|
14477
95e66ea71f71
6830717: replay of compilations would help with debugging
minqi
parents:
14391
diff
changeset
|
34 |
#include "ci/ciReplay.hpp" |
49449
ef5d5d343e2a
8199263: Split interfaceSupport.hpp to not require including .inline.hpp files
coleenp
parents:
49340
diff
changeset
|
35 |
#include "ci/ciUtilities.inline.hpp" |
7397 | 36 |
#include "classfile/systemDictionary.hpp" |
37 |
#include "compiler/abstractCompiler.hpp" |
|
38 |
#include "compiler/methodLiveness.hpp" |
|
39 |
#include "interpreter/interpreter.hpp" |
|
40 |
#include "interpreter/linkResolver.hpp" |
|
41 |
#include "interpreter/oopMapCache.hpp" |
|
42 |
#include "memory/allocation.inline.hpp" |
|
43 |
#include "memory/resourceArea.hpp" |
|
44 |
#include "oops/generateOopMap.hpp" |
|
49340
4e82736053ae
8191102: Incorrect include file use in classLoader.hpp
hseigel
parents:
48024
diff
changeset
|
45 |
#include "oops/method.inline.hpp" |
7397 | 46 |
#include "oops/oop.inline.hpp" |
47 |
#include "prims/nativeLookup.hpp" |
|
48 |
#include "runtime/deoptimization.hpp" |
|
53582 | 49 |
#include "runtime/handles.inline.hpp" |
7397 | 50 |
#include "utilities/bitMap.inline.hpp" |
51 |
#include "utilities/xmlstream.hpp" |
|
52 |
#ifdef COMPILER2 |
|
53 |
#include "ci/bcEscapeAnalyzer.hpp" |
|
54 |
#include "ci/ciTypeFlow.hpp" |
|
13728
882756847a04
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
13522
diff
changeset
|
55 |
#include "oops/method.hpp" |
7397 | 56 |
#endif |
1 | 57 |
|
58 |
// ciMethod |
|
59 |
// |
|
13728
882756847a04
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
13522
diff
changeset
|
60 |
// This class represents a Method* in the HotSpot virtual |
1 | 61 |
// machine. |
62 |
||
63 |
||
64 |
// ------------------------------------------------------------------ |
|
65 |
// ciMethod::ciMethod |
|
66 |
// |
|
67 |
// Loaded method. |
|
46727
6e4a84748e2c
8183039: Re-examine methodHandle methods uninlined by 8144256
coleenp
parents:
46630
diff
changeset
|
68 |
ciMethod::ciMethod(const methodHandle& h_m, ciInstanceKlass* holder) : |
27644
f4aa22a934e4
8060147: SIGSEGV in Metadata::mark_on_stack() while marking metadata in ciEnv
vlivanov
parents:
27143
diff
changeset
|
69 |
ciMetadata(h_m()), |
31228
8e427370cdd1
8074551: GWT can be marked non-compilable due to deopt count pollution
vlivanov
parents:
31019
diff
changeset
|
70 |
_holder(holder) |
27644
f4aa22a934e4
8060147: SIGSEGV in Metadata::mark_on_stack() while marking metadata in ciEnv
vlivanov
parents:
27143
diff
changeset
|
71 |
{ |
1 | 72 |
assert(h_m() != NULL, "no null method"); |
73 |
||
31790 | 74 |
if (LogTouchedMethods) { |
59056
15936b142f86
8233913: Remove implicit conversion from Method* to methodHandle
coleenp
parents:
58273
diff
changeset
|
75 |
h_m->log_touched(Thread::current()); |
31790 | 76 |
} |
1 | 77 |
// These fields are always filled in in loaded methods. |
59056
15936b142f86
8233913: Remove implicit conversion from Method* to methodHandle
coleenp
parents:
58273
diff
changeset
|
78 |
_flags = ciFlags(h_m->access_flags()); |
1 | 79 |
|
80 |
// Easy to compute, so fill them in now. |
|
59056
15936b142f86
8233913: Remove implicit conversion from Method* to methodHandle
coleenp
parents:
58273
diff
changeset
|
81 |
_max_stack = h_m->max_stack(); |
15936b142f86
8233913: Remove implicit conversion from Method* to methodHandle
coleenp
parents:
58273
diff
changeset
|
82 |
_max_locals = h_m->max_locals(); |
15936b142f86
8233913: Remove implicit conversion from Method* to methodHandle
coleenp
parents:
58273
diff
changeset
|
83 |
_code_size = h_m->code_size(); |
15936b142f86
8233913: Remove implicit conversion from Method* to methodHandle
coleenp
parents:
58273
diff
changeset
|
84 |
_intrinsic_id = h_m->intrinsic_id(); |
15936b142f86
8233913: Remove implicit conversion from Method* to methodHandle
coleenp
parents:
58273
diff
changeset
|
85 |
_handler_count = h_m->exception_table_length(); |
15936b142f86
8233913: Remove implicit conversion from Method* to methodHandle
coleenp
parents:
58273
diff
changeset
|
86 |
_size_of_parameters = h_m->size_of_parameters(); |
15936b142f86
8233913: Remove implicit conversion from Method* to methodHandle
coleenp
parents:
58273
diff
changeset
|
87 |
_uses_monitors = h_m->access_flags().has_monitor_bytecodes(); |
15936b142f86
8233913: Remove implicit conversion from Method* to methodHandle
coleenp
parents:
58273
diff
changeset
|
88 |
_balanced_monitors = !_uses_monitors || h_m->access_flags().is_monitor_matching(); |
15936b142f86
8233913: Remove implicit conversion from Method* to methodHandle
coleenp
parents:
58273
diff
changeset
|
89 |
_is_c1_compilable = !h_m->is_not_c1_compilable(); |
15936b142f86
8233913: Remove implicit conversion from Method* to methodHandle
coleenp
parents:
58273
diff
changeset
|
90 |
_is_c2_compilable = !h_m->is_not_c2_compilable(); |
48024
6199dfaf72da
8191688: Assert failed in > 200 tests: failed dependencies, but counter didn't change
dlong
parents:
47687
diff
changeset
|
91 |
_can_be_parsed = true; |
59056
15936b142f86
8233913: Remove implicit conversion from Method* to methodHandle
coleenp
parents:
58273
diff
changeset
|
92 |
_has_reserved_stack_access = h_m->has_reserved_stack_access(); |
15936b142f86
8233913: Remove implicit conversion from Method* to methodHandle
coleenp
parents:
58273
diff
changeset
|
93 |
_is_overpass = h_m->is_overpass(); |
1 | 94 |
// Lazy fields, filled in on demand. Require allocation. |
95 |
_code = NULL; |
|
96 |
_exception_handlers = NULL; |
|
97 |
_liveness = NULL; |
|
98 |
_method_blocks = NULL; |
|
47687 | 99 |
#if defined(COMPILER2) |
1 | 100 |
_flow = NULL; |
5928
f6e69b46e9e3
6968368: SIGSEGV in the BCEscapeAnalyzer::copy_dependencies
kvn
parents:
5702
diff
changeset
|
101 |
_bcea = NULL; |
47687 | 102 |
#endif // COMPILER2 |
1 | 103 |
|
2867
69187054225f
6788527: Server vm intermittently fails with assertion "live value must not be garbage" with fastdebug bits
kvn
parents:
2534
diff
changeset
|
104 |
ciEnv *env = CURRENT_ENV; |
48024
6199dfaf72da
8191688: Assert failed in > 200 tests: failed dependencies, but counter didn't change
dlong
parents:
47687
diff
changeset
|
105 |
if (env->jvmti_can_hotswap_or_post_breakpoint()) { |
1 | 106 |
// 6328518 check hotswap conditions under the right lock. |
107 |
MutexLocker locker(Compile_lock); |
|
108 |
if (Dependencies::check_evol_method(h_m()) != NULL) { |
|
6453 | 109 |
_is_c1_compilable = false; |
110 |
_is_c2_compilable = false; |
|
48024
6199dfaf72da
8191688: Assert failed in > 200 tests: failed dependencies, but counter didn't change
dlong
parents:
47687
diff
changeset
|
111 |
_can_be_parsed = false; |
1 | 112 |
} |
113 |
} else { |
|
57745 | 114 |
DEBUG_ONLY(CompilerThread::current()->check_possible_safepoint()); |
1 | 115 |
} |
116 |
||
59056
15936b142f86
8233913: Remove implicit conversion from Method* to methodHandle
coleenp
parents:
58273
diff
changeset
|
117 |
if (h_m->method_holder()->is_linked()) { |
15936b142f86
8233913: Remove implicit conversion from Method* to methodHandle
coleenp
parents:
58273
diff
changeset
|
118 |
_can_be_statically_bound = h_m->can_be_statically_bound(); |
1 | 119 |
} else { |
120 |
// Have to use a conservative value in this case. |
|
121 |
_can_be_statically_bound = false; |
|
122 |
} |
|
123 |
||
124 |
// Adjust the definition of this condition to be more useful: |
|
125 |
// %%% take these conditions into account in vtable generation |
|
59056
15936b142f86
8233913: Remove implicit conversion from Method* to methodHandle
coleenp
parents:
58273
diff
changeset
|
126 |
if (!_can_be_statically_bound && h_m->is_private()) |
1 | 127 |
_can_be_statically_bound = true; |
59056
15936b142f86
8233913: Remove implicit conversion from Method* to methodHandle
coleenp
parents:
58273
diff
changeset
|
128 |
if (_can_be_statically_bound && h_m->is_abstract()) |
1 | 129 |
_can_be_statically_bound = false; |
130 |
||
131 |
// generating _signature may allow GC and therefore move m. |
|
132 |
// These fields are always filled in. |
|
59056
15936b142f86
8233913: Remove implicit conversion from Method* to methodHandle
coleenp
parents:
58273
diff
changeset
|
133 |
_name = env->get_symbol(h_m->name()); |
15936b142f86
8233913: Remove implicit conversion from Method* to methodHandle
coleenp
parents:
58273
diff
changeset
|
134 |
ciSymbol* sig_symbol = env->get_symbol(h_m->signature()); |
15936b142f86
8233913: Remove implicit conversion from Method* to methodHandle
coleenp
parents:
58273
diff
changeset
|
135 |
constantPoolHandle cpool(Thread::current(), h_m->constants()); |
10008
d84de97ad847
7056328: JSR 292 invocation sometimes fails in adapters for types not on boot class path
jrose
parents:
8676
diff
changeset
|
136 |
_signature = new (env->arena()) ciSignature(_holder, cpool, sig_symbol); |
1 | 137 |
_method_data = NULL; |
59056
15936b142f86
8233913: Remove implicit conversion from Method* to methodHandle
coleenp
parents:
58273
diff
changeset
|
138 |
_nmethod_age = h_m->nmethod_age(); |
1 | 139 |
// Take a snapshot of these values, so they will be commensurate with the MDO. |
6453 | 140 |
if (ProfileInterpreter || TieredCompilation) { |
59056
15936b142f86
8233913: Remove implicit conversion from Method* to methodHandle
coleenp
parents:
58273
diff
changeset
|
141 |
int invcnt = h_m->interpreter_invocation_count(); |
1 | 142 |
// if the value overflowed report it as max int |
143 |
_interpreter_invocation_count = invcnt < 0 ? max_jint : invcnt ; |
|
59056
15936b142f86
8233913: Remove implicit conversion from Method* to methodHandle
coleenp
parents:
58273
diff
changeset
|
144 |
_interpreter_throwout_count = h_m->interpreter_throwout_count(); |
1 | 145 |
} else { |
146 |
_interpreter_invocation_count = 0; |
|
147 |
_interpreter_throwout_count = 0; |
|
148 |
} |
|
149 |
if (_interpreter_invocation_count == 0) |
|
150 |
_interpreter_invocation_count = 1; |
|
14477
95e66ea71f71
6830717: replay of compilations would help with debugging
minqi
parents:
14391
diff
changeset
|
151 |
_instructions_size = -1; |
95e66ea71f71
6830717: replay of compilations would help with debugging
minqi
parents:
14391
diff
changeset
|
152 |
#ifdef ASSERT |
95e66ea71f71
6830717: replay of compilations would help with debugging
minqi
parents:
14391
diff
changeset
|
153 |
if (ReplayCompiles) { |
95e66ea71f71
6830717: replay of compilations would help with debugging
minqi
parents:
14391
diff
changeset
|
154 |
ciReplay::initialize(this); |
95e66ea71f71
6830717: replay of compilations would help with debugging
minqi
parents:
14391
diff
changeset
|
155 |
} |
95e66ea71f71
6830717: replay of compilations would help with debugging
minqi
parents:
14391
diff
changeset
|
156 |
#endif |
1 | 157 |
} |
158 |
||
159 |
||
160 |
// ------------------------------------------------------------------ |
|
161 |
// ciMethod::ciMethod |
|
162 |
// |
|
163 |
// Unloaded method. |
|
164 |
ciMethod::ciMethod(ciInstanceKlass* holder, |
|
10734
065435337883
7092712: JSR 292: unloaded invokedynamic call sites can lead to a crash with signature types not on BCP
twisti
parents:
10506
diff
changeset
|
165 |
ciSymbol* name, |
065435337883
7092712: JSR 292: unloaded invokedynamic call sites can lead to a crash with signature types not on BCP
twisti
parents:
10506
diff
changeset
|
166 |
ciSymbol* signature, |
065435337883
7092712: JSR 292: unloaded invokedynamic call sites can lead to a crash with signature types not on BCP
twisti
parents:
10506
diff
changeset
|
167 |
ciInstanceKlass* accessor) : |
13728
882756847a04
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
13522
diff
changeset
|
168 |
ciMetadata((Metadata*)NULL), |
10734
065435337883
7092712: JSR 292: unloaded invokedynamic call sites can lead to a crash with signature types not on BCP
twisti
parents:
10506
diff
changeset
|
169 |
_name( name), |
065435337883
7092712: JSR 292: unloaded invokedynamic call sites can lead to a crash with signature types not on BCP
twisti
parents:
10506
diff
changeset
|
170 |
_holder( holder), |
51333
f6641fcf7b7e
8208670: Compiler changes to allow enabling -Wreorder
tschatzl
parents:
50113
diff
changeset
|
171 |
_method_data( NULL), |
f6641fcf7b7e
8208670: Compiler changes to allow enabling -Wreorder
tschatzl
parents:
50113
diff
changeset
|
172 |
_method_blocks( NULL), |
10734
065435337883
7092712: JSR 292: unloaded invokedynamic call sites can lead to a crash with signature types not on BCP
twisti
parents:
10506
diff
changeset
|
173 |
_intrinsic_id( vmIntrinsics::_none), |
51333
f6641fcf7b7e
8208670: Compiler changes to allow enabling -Wreorder
tschatzl
parents:
50113
diff
changeset
|
174 |
_instructions_size(-1), |
10734
065435337883
7092712: JSR 292: unloaded invokedynamic call sites can lead to a crash with signature types not on BCP
twisti
parents:
10506
diff
changeset
|
175 |
_can_be_statically_bound(false), |
51333
f6641fcf7b7e
8208670: Compiler changes to allow enabling -Wreorder
tschatzl
parents:
50113
diff
changeset
|
176 |
_liveness( NULL) |
47687 | 177 |
#if defined(COMPILER2) |
10734
065435337883
7092712: JSR 292: unloaded invokedynamic call sites can lead to a crash with signature types not on BCP
twisti
parents:
10506
diff
changeset
|
178 |
, |
065435337883
7092712: JSR 292: unloaded invokedynamic call sites can lead to a crash with signature types not on BCP
twisti
parents:
10506
diff
changeset
|
179 |
_flow( NULL), |
51333
f6641fcf7b7e
8208670: Compiler changes to allow enabling -Wreorder
tschatzl
parents:
50113
diff
changeset
|
180 |
_bcea( NULL) |
47687 | 181 |
#endif // COMPILER2 |
10734
065435337883
7092712: JSR 292: unloaded invokedynamic call sites can lead to a crash with signature types not on BCP
twisti
parents:
10506
diff
changeset
|
182 |
{ |
065435337883
7092712: JSR 292: unloaded invokedynamic call sites can lead to a crash with signature types not on BCP
twisti
parents:
10506
diff
changeset
|
183 |
// Usually holder and accessor are the same type but in some cases |
065435337883
7092712: JSR 292: unloaded invokedynamic call sites can lead to a crash with signature types not on BCP
twisti
parents:
10506
diff
changeset
|
184 |
// the holder has the wrong class loader (e.g. invokedynamic call |
065435337883
7092712: JSR 292: unloaded invokedynamic call sites can lead to a crash with signature types not on BCP
twisti
parents:
10506
diff
changeset
|
185 |
// sites) so we pass the accessor. |
065435337883
7092712: JSR 292: unloaded invokedynamic call sites can lead to a crash with signature types not on BCP
twisti
parents:
10506
diff
changeset
|
186 |
_signature = new (CURRENT_ENV->arena()) ciSignature(accessor, constantPoolHandle(), signature); |
1 | 187 |
} |
188 |
||
189 |
||
190 |
// ------------------------------------------------------------------ |
|
191 |
// ciMethod::load_code |
|
192 |
// |
|
193 |
// Load the bytecodes and exception handler table for this method. |
|
194 |
void ciMethod::load_code() { |
|
195 |
VM_ENTRY_MARK; |
|
196 |
assert(is_loaded(), "only loaded methods have code"); |
|
197 |
||
13728
882756847a04
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
13522
diff
changeset
|
198 |
Method* me = get_Method(); |
1 | 199 |
Arena* arena = CURRENT_THREAD_ENV->arena(); |
200 |
||
201 |
// Load the bytecodes. |
|
202 |
_code = (address)arena->Amalloc(code_size()); |
|
203 |
memcpy(_code, me->code_base(), code_size()); |
|
204 |
||
38059
86ab3f0a9f87
8148195: Some InstanceKlass and MethodCounters fields can be excluded when JVMTI is not supported
cjplummer
parents:
35071
diff
changeset
|
205 |
#if INCLUDE_JVMTI |
1 | 206 |
// Revert any breakpoint bytecodes in ci's copy |
200
88d83617f912
6498878: client compiler crashes on windows when dealing with breakpoint instructions
kvn
parents:
1
diff
changeset
|
207 |
if (me->number_of_breakpoints() > 0) { |
14391
df0a1573d5bd
8000725: NPG: method_holder() and pool_holder() and pool_holder field should be InstanceKlass
coleenp
parents:
13728
diff
changeset
|
208 |
BreakpointInfo* bp = me->method_holder()->breakpoints(); |
1 | 209 |
for (; bp != NULL; bp = bp->next()) { |
210 |
if (bp->match(me)) { |
|
211 |
code_at_put(bp->bci(), bp->orig_bytecode()); |
|
212 |
} |
|
213 |
} |
|
214 |
} |
|
38059
86ab3f0a9f87
8148195: Some InstanceKlass and MethodCounters fields can be excluded when JVMTI is not supported
cjplummer
parents:
35071
diff
changeset
|
215 |
#endif |
1 | 216 |
|
217 |
// And load the exception table. |
|
13282
9872915dd78d
7178145: Change constMethodOop::_exception_table to optionally inlined u2 table.
jiangli
parents:
10734
diff
changeset
|
218 |
ExceptionTable exc_table(me); |
1 | 219 |
|
220 |
// Allocate one extra spot in our list of exceptions. This |
|
221 |
// last entry will be used to represent the possibility that |
|
222 |
// an exception escapes the method. See ciExceptionHandlerStream |
|
223 |
// for details. |
|
224 |
_exception_handlers = |
|
225 |
(ciExceptionHandler**)arena->Amalloc(sizeof(ciExceptionHandler*) |
|
226 |
* (_handler_count + 1)); |
|
227 |
if (_handler_count > 0) { |
|
228 |
for (int i=0; i<_handler_count; i++) { |
|
229 |
_exception_handlers[i] = new (arena) ciExceptionHandler( |
|
230 |
holder(), |
|
13282
9872915dd78d
7178145: Change constMethodOop::_exception_table to optionally inlined u2 table.
jiangli
parents:
10734
diff
changeset
|
231 |
/* start */ exc_table.start_pc(i), |
9872915dd78d
7178145: Change constMethodOop::_exception_table to optionally inlined u2 table.
jiangli
parents:
10734
diff
changeset
|
232 |
/* limit */ exc_table.end_pc(i), |
9872915dd78d
7178145: Change constMethodOop::_exception_table to optionally inlined u2 table.
jiangli
parents:
10734
diff
changeset
|
233 |
/* goto pc */ exc_table.handler_pc(i), |
9872915dd78d
7178145: Change constMethodOop::_exception_table to optionally inlined u2 table.
jiangli
parents:
10734
diff
changeset
|
234 |
/* cp index */ exc_table.catch_type_index(i)); |
1 | 235 |
} |
236 |
} |
|
237 |
||
238 |
// Put an entry at the end of our list to represent the possibility |
|
239 |
// of exceptional exit. |
|
240 |
_exception_handlers[_handler_count] = |
|
241 |
new (arena) ciExceptionHandler(holder(), 0, code_size(), -1, 0); |
|
242 |
||
243 |
if (CIPrintMethodCodes) { |
|
244 |
print_codes(); |
|
245 |
} |
|
246 |
} |
|
247 |
||
248 |
||
249 |
// ------------------------------------------------------------------ |
|
250 |
// ciMethod::has_linenumber_table |
|
251 |
// |
|
252 |
// length unknown until decompression |
|
253 |
bool ciMethod::has_linenumber_table() const { |
|
254 |
check_is_loaded(); |
|
255 |
VM_ENTRY_MARK; |
|
13728
882756847a04
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
13522
diff
changeset
|
256 |
return get_Method()->has_linenumber_table(); |
1 | 257 |
} |
258 |
||
259 |
||
260 |
// ------------------------------------------------------------------ |
|
261 |
// ciMethod::compressed_linenumber_table |
|
262 |
u_char* ciMethod::compressed_linenumber_table() const { |
|
263 |
check_is_loaded(); |
|
264 |
VM_ENTRY_MARK; |
|
13728
882756847a04
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
13522
diff
changeset
|
265 |
return get_Method()->compressed_linenumber_table(); |
1 | 266 |
} |
267 |
||
268 |
||
269 |
// ------------------------------------------------------------------ |
|
270 |
// ciMethod::line_number_from_bci |
|
271 |
int ciMethod::line_number_from_bci(int bci) const { |
|
272 |
check_is_loaded(); |
|
273 |
VM_ENTRY_MARK; |
|
13728
882756847a04
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
13522
diff
changeset
|
274 |
return get_Method()->line_number_from_bci(bci); |
1 | 275 |
} |
276 |
||
277 |
||
278 |
// ------------------------------------------------------------------ |
|
279 |
// ciMethod::vtable_index |
|
280 |
// |
|
281 |
// Get the position of this method's entry in the vtable, if any. |
|
282 |
int ciMethod::vtable_index() { |
|
283 |
check_is_loaded(); |
|
284 |
assert(holder()->is_linked(), "must be linked"); |
|
285 |
VM_ENTRY_MARK; |
|
13728
882756847a04
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
13522
diff
changeset
|
286 |
return get_Method()->vtable_index(); |
1 | 287 |
} |
288 |
||
289 |
||
290 |
// ------------------------------------------------------------------ |
|
291 |
// ciMethod::native_entry |
|
292 |
// |
|
293 |
// Get the address of this method's native code, if any. |
|
294 |
address ciMethod::native_entry() { |
|
295 |
check_is_loaded(); |
|
296 |
assert(flags().is_native(), "must be native method"); |
|
297 |
VM_ENTRY_MARK; |
|
13728
882756847a04
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
13522
diff
changeset
|
298 |
Method* method = get_Method(); |
1 | 299 |
address entry = method->native_function(); |
300 |
assert(entry != NULL, "must be valid entry point"); |
|
301 |
return entry; |
|
302 |
} |
|
303 |
||
304 |
||
305 |
// ------------------------------------------------------------------ |
|
306 |
// ciMethod::interpreter_entry |
|
307 |
// |
|
308 |
// Get the entry point for running this method in the interpreter. |
|
309 |
address ciMethod::interpreter_entry() { |
|
310 |
check_is_loaded(); |
|
311 |
VM_ENTRY_MARK; |
|
13728
882756847a04
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
13522
diff
changeset
|
312 |
methodHandle mh(THREAD, get_Method()); |
1 | 313 |
return Interpreter::entry_for_method(mh); |
314 |
} |
|
315 |
||
316 |
||
317 |
// ------------------------------------------------------------------ |
|
318 |
// ciMethod::uses_balanced_monitors |
|
319 |
// |
|
320 |
// Does this method use monitors in a strict stack-disciplined manner? |
|
321 |
bool ciMethod::has_balanced_monitors() { |
|
322 |
check_is_loaded(); |
|
323 |
if (_balanced_monitors) return true; |
|
324 |
||
325 |
// Analyze the method to see if monitors are used properly. |
|
326 |
VM_ENTRY_MARK; |
|
13728
882756847a04
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
13522
diff
changeset
|
327 |
methodHandle method(THREAD, get_Method()); |
1 | 328 |
assert(method->has_monitor_bytecodes(), "should have checked this"); |
329 |
||
330 |
// Check to see if a previous compilation computed the |
|
331 |
// monitor-matching analysis. |
|
332 |
if (method->guaranteed_monitor_matching()) { |
|
333 |
_balanced_monitors = true; |
|
334 |
return true; |
|
335 |
} |
|
336 |
||
337 |
{ |
|
338 |
EXCEPTION_MARK; |
|
339 |
ResourceMark rm(THREAD); |
|
340 |
GeneratePairingInfo gpi(method); |
|
341 |
gpi.compute_map(CATCH); |
|
342 |
if (!gpi.monitor_safe()) { |
|
343 |
return false; |
|
344 |
} |
|
345 |
method->set_guaranteed_monitor_matching(); |
|
346 |
_balanced_monitors = true; |
|
347 |
} |
|
348 |
return true; |
|
349 |
} |
|
350 |
||
351 |
||
352 |
// ------------------------------------------------------------------ |
|
353 |
// ciMethod::get_flow_analysis |
|
354 |
ciTypeFlow* ciMethod::get_flow_analysis() { |
|
47687 | 355 |
#if defined(COMPILER2) |
1 | 356 |
if (_flow == NULL) { |
357 |
ciEnv* env = CURRENT_ENV; |
|
358 |
_flow = new (env->arena()) ciTypeFlow(env, this); |
|
359 |
_flow->do_flow(); |
|
360 |
} |
|
361 |
return _flow; |
|
47687 | 362 |
#else // COMPILER2 |
1 | 363 |
ShouldNotReachHere(); |
364 |
return NULL; |
|
47687 | 365 |
#endif // COMPILER2 |
1 | 366 |
} |
367 |
||
368 |
||
369 |
// ------------------------------------------------------------------ |
|
370 |
// ciMethod::get_osr_flow_analysis |
|
371 |
ciTypeFlow* ciMethod::get_osr_flow_analysis(int osr_bci) { |
|
47687 | 372 |
#if defined(COMPILER2) |
1 | 373 |
// OSR entry points are always place after a call bytecode of some sort |
374 |
assert(osr_bci >= 0, "must supply valid OSR entry point"); |
|
375 |
ciEnv* env = CURRENT_ENV; |
|
376 |
ciTypeFlow* flow = new (env->arena()) ciTypeFlow(env, this, osr_bci); |
|
377 |
flow->do_flow(); |
|
378 |
return flow; |
|
47687 | 379 |
#else // COMPILER2 |
1 | 380 |
ShouldNotReachHere(); |
381 |
return NULL; |
|
47687 | 382 |
#endif // COMPILER2 |
1 | 383 |
} |
384 |
||
385 |
// ------------------------------------------------------------------ |
|
3910 | 386 |
// ciMethod::raw_liveness_at_bci |
1 | 387 |
// |
388 |
// Which local variables are live at a specific bci? |
|
3910 | 389 |
MethodLivenessResult ciMethod::raw_liveness_at_bci(int bci) { |
1 | 390 |
check_is_loaded(); |
391 |
if (_liveness == NULL) { |
|
392 |
// Create the liveness analyzer. |
|
393 |
Arena* arena = CURRENT_ENV->arena(); |
|
394 |
_liveness = new (arena) MethodLiveness(arena, this); |
|
395 |
_liveness->compute_liveness(); |
|
396 |
} |
|
3910 | 397 |
return _liveness->get_liveness_at(bci); |
398 |
} |
|
399 |
||
400 |
// ------------------------------------------------------------------ |
|
401 |
// ciMethod::liveness_at_bci |
|
402 |
// |
|
403 |
// Which local variables are live at a specific bci? When debugging |
|
404 |
// will return true for all locals in some cases to improve debug |
|
405 |
// information. |
|
406 |
MethodLivenessResult ciMethod::liveness_at_bci(int bci) { |
|
52934
8deeb7bba516
8214917: CTW testlibrary shouldn't ignore errors raised by the library itself
iignatyev
parents:
51333
diff
changeset
|
407 |
if (CURRENT_ENV->should_retain_local_variables() || DeoptimizeALot) { |
1 | 408 |
// Keep all locals live for the user's edification and amusement. |
53345
91ab128a65a3
8216556: Unnecessary liveness computation with JVMTI
mdoerr
parents:
52934
diff
changeset
|
409 |
MethodLivenessResult result(_max_locals); |
91ab128a65a3
8216556: Unnecessary liveness computation with JVMTI
mdoerr
parents:
52934
diff
changeset
|
410 |
result.set_range(0, _max_locals); |
91ab128a65a3
8216556: Unnecessary liveness computation with JVMTI
mdoerr
parents:
52934
diff
changeset
|
411 |
result.set_is_valid(); |
91ab128a65a3
8216556: Unnecessary liveness computation with JVMTI
mdoerr
parents:
52934
diff
changeset
|
412 |
return result; |
1 | 413 |
} |
53345
91ab128a65a3
8216556: Unnecessary liveness computation with JVMTI
mdoerr
parents:
52934
diff
changeset
|
414 |
return raw_liveness_at_bci(bci); |
1 | 415 |
} |
416 |
||
417 |
// ciMethod::live_local_oops_at_bci |
|
418 |
// |
|
419 |
// find all the live oops in the locals array for a particular bci |
|
420 |
// Compute what the interpreter believes by using the interpreter |
|
421 |
// oopmap generator. This is used as a double check during osr to |
|
422 |
// guard against conservative result from MethodLiveness making us |
|
423 |
// think a dead oop is live. MethodLiveness is conservative in the |
|
424 |
// sense that it may consider locals to be live which cannot be live, |
|
425 |
// like in the case where a local could contain an oop or a primitive |
|
426 |
// along different paths. In that case the local must be dead when |
|
427 |
// those paths merge. Since the interpreter's viewpoint is used when |
|
428 |
// gc'ing an interpreter frame we need to use its viewpoint during |
|
429 |
// OSR when loading the locals. |
|
430 |
||
38177 | 431 |
ResourceBitMap ciMethod::live_local_oops_at_bci(int bci) { |
1 | 432 |
VM_ENTRY_MARK; |
433 |
InterpreterOopMap mask; |
|
59056
15936b142f86
8233913: Remove implicit conversion from Method* to methodHandle
coleenp
parents:
58273
diff
changeset
|
434 |
OopMapCache::compute_one_oop_map(methodHandle(THREAD, get_Method()), bci, &mask); |
1 | 435 |
int mask_size = max_locals(); |
38177 | 436 |
ResourceBitMap result(mask_size); |
1 | 437 |
int i; |
438 |
for (i = 0; i < mask_size ; i++ ) { |
|
439 |
if (mask.is_oop(i)) result.set_bit(i); |
|
440 |
} |
|
441 |
return result; |
|
442 |
} |
|
443 |
||
444 |
||
445 |
#ifdef COMPILER1 |
|
446 |
// ------------------------------------------------------------------ |
|
447 |
// ciMethod::bci_block_start |
|
448 |
// |
|
449 |
// Marks all bcis where a new basic block starts |
|
38177 | 450 |
const BitMap& ciMethod::bci_block_start() { |
1 | 451 |
check_is_loaded(); |
452 |
if (_liveness == NULL) { |
|
453 |
// Create the liveness analyzer. |
|
454 |
Arena* arena = CURRENT_ENV->arena(); |
|
455 |
_liveness = new (arena) MethodLiveness(arena, this); |
|
456 |
_liveness->compute_liveness(); |
|
457 |
} |
|
458 |
||
459 |
return _liveness->get_bci_block_start(); |
|
460 |
} |
|
461 |
#endif // COMPILER1 |
|
462 |
||
463 |
||
464 |
// ------------------------------------------------------------------ |
|
55252
6502d6a92fe2
8224162: assert(profile.count() == 0) failed: sanity in InlineTree::is_not_reached
jiefu
parents:
55206
diff
changeset
|
465 |
// ciMethod::check_overflow |
6502d6a92fe2
8224162: assert(profile.count() == 0) failed: sanity in InlineTree::is_not_reached
jiefu
parents:
55206
diff
changeset
|
466 |
// |
6502d6a92fe2
8224162: assert(profile.count() == 0) failed: sanity in InlineTree::is_not_reached
jiefu
parents:
55206
diff
changeset
|
467 |
// Check whether the profile counter is overflowed and adjust if true. |
6502d6a92fe2
8224162: assert(profile.count() == 0) failed: sanity in InlineTree::is_not_reached
jiefu
parents:
55206
diff
changeset
|
468 |
// For invoke* it will turn negative values into max_jint, |
6502d6a92fe2
8224162: assert(profile.count() == 0) failed: sanity in InlineTree::is_not_reached
jiefu
parents:
55206
diff
changeset
|
469 |
// and for checkcast/aastore/instanceof turn positive values into min_jint. |
6502d6a92fe2
8224162: assert(profile.count() == 0) failed: sanity in InlineTree::is_not_reached
jiefu
parents:
55206
diff
changeset
|
470 |
int ciMethod::check_overflow(int c, Bytecodes::Code code) { |
6502d6a92fe2
8224162: assert(profile.count() == 0) failed: sanity in InlineTree::is_not_reached
jiefu
parents:
55206
diff
changeset
|
471 |
switch (code) { |
6502d6a92fe2
8224162: assert(profile.count() == 0) failed: sanity in InlineTree::is_not_reached
jiefu
parents:
55206
diff
changeset
|
472 |
case Bytecodes::_aastore: // fall-through |
6502d6a92fe2
8224162: assert(profile.count() == 0) failed: sanity in InlineTree::is_not_reached
jiefu
parents:
55206
diff
changeset
|
473 |
case Bytecodes::_checkcast: // fall-through |
6502d6a92fe2
8224162: assert(profile.count() == 0) failed: sanity in InlineTree::is_not_reached
jiefu
parents:
55206
diff
changeset
|
474 |
case Bytecodes::_instanceof: { |
6502d6a92fe2
8224162: assert(profile.count() == 0) failed: sanity in InlineTree::is_not_reached
jiefu
parents:
55206
diff
changeset
|
475 |
return (c > 0 ? min_jint : c); // always non-positive |
6502d6a92fe2
8224162: assert(profile.count() == 0) failed: sanity in InlineTree::is_not_reached
jiefu
parents:
55206
diff
changeset
|
476 |
} |
6502d6a92fe2
8224162: assert(profile.count() == 0) failed: sanity in InlineTree::is_not_reached
jiefu
parents:
55206
diff
changeset
|
477 |
default: { |
6502d6a92fe2
8224162: assert(profile.count() == 0) failed: sanity in InlineTree::is_not_reached
jiefu
parents:
55206
diff
changeset
|
478 |
assert(Bytecodes::is_invoke(code), "%s", Bytecodes::name(code)); |
6502d6a92fe2
8224162: assert(profile.count() == 0) failed: sanity in InlineTree::is_not_reached
jiefu
parents:
55206
diff
changeset
|
479 |
return (c < 0 ? max_jint : c); // always non-negative |
6502d6a92fe2
8224162: assert(profile.count() == 0) failed: sanity in InlineTree::is_not_reached
jiefu
parents:
55206
diff
changeset
|
480 |
} |
6502d6a92fe2
8224162: assert(profile.count() == 0) failed: sanity in InlineTree::is_not_reached
jiefu
parents:
55206
diff
changeset
|
481 |
} |
6502d6a92fe2
8224162: assert(profile.count() == 0) failed: sanity in InlineTree::is_not_reached
jiefu
parents:
55206
diff
changeset
|
482 |
} |
6502d6a92fe2
8224162: assert(profile.count() == 0) failed: sanity in InlineTree::is_not_reached
jiefu
parents:
55206
diff
changeset
|
483 |
|
6502d6a92fe2
8224162: assert(profile.count() == 0) failed: sanity in InlineTree::is_not_reached
jiefu
parents:
55206
diff
changeset
|
484 |
|
6502d6a92fe2
8224162: assert(profile.count() == 0) failed: sanity in InlineTree::is_not_reached
jiefu
parents:
55206
diff
changeset
|
485 |
// ------------------------------------------------------------------ |
1 | 486 |
// ciMethod::call_profile_at_bci |
487 |
// |
|
488 |
// Get the ciCallProfile for the invocation of this method. |
|
489 |
// Also reports receiver types for non-call type checks (if TypeProfileCasts). |
|
490 |
ciCallProfile ciMethod::call_profile_at_bci(int bci) { |
|
491 |
ResourceMark rm; |
|
492 |
ciCallProfile result; |
|
493 |
if (method_data() != NULL && method_data()->is_mature()) { |
|
494 |
ciProfileData* data = method_data()->bci_to_data(bci); |
|
495 |
if (data != NULL && data->is_CounterData()) { |
|
496 |
// Every profiled call site has a counter. |
|
55252
6502d6a92fe2
8224162: assert(profile.count() == 0) failed: sanity in InlineTree::is_not_reached
jiefu
parents:
55206
diff
changeset
|
497 |
int count = check_overflow(data->as_CounterData()->count(), java_code_at_bci(bci)); |
1 | 498 |
|
499 |
if (!data->is_ReceiverTypeData()) { |
|
500 |
result._receiver_count[0] = 0; // that's a definite zero |
|
501 |
} else { // ReceiverTypeData is a subclass of CounterData |
|
502 |
ciReceiverTypeData* call = (ciReceiverTypeData*)data->as_ReceiverTypeData(); |
|
503 |
// In addition, virtual call sites have receiver type information |
|
504 |
int receivers_count_total = 0; |
|
505 |
int morphism = 0; |
|
6453 | 506 |
// Precompute morphism for the possible fixup |
1 | 507 |
for (uint i = 0; i < call->row_limit(); i++) { |
508 |
ciKlass* receiver = call->receiver(i); |
|
509 |
if (receiver == NULL) continue; |
|
6453 | 510 |
morphism++; |
511 |
} |
|
512 |
int epsilon = 0; |
|
46326
70de7011f79a
8159734: Consistency check fails with -XX:-ProfileInterpreter
zmajo
parents:
40664
diff
changeset
|
513 |
if (TieredCompilation) { |
70de7011f79a
8159734: Consistency check fails with -XX:-ProfileInterpreter
zmajo
parents:
40664
diff
changeset
|
514 |
// For a call, it is assumed that either the type of the receiver(s) |
70de7011f79a
8159734: Consistency check fails with -XX:-ProfileInterpreter
zmajo
parents:
40664
diff
changeset
|
515 |
// is recorded or an associated counter is incremented, but not both. With |
70de7011f79a
8159734: Consistency check fails with -XX:-ProfileInterpreter
zmajo
parents:
40664
diff
changeset
|
516 |
// tiered compilation, however, both can happen due to the interpreter and |
70de7011f79a
8159734: Consistency check fails with -XX:-ProfileInterpreter
zmajo
parents:
40664
diff
changeset
|
517 |
// C1 profiling invocations differently. Address that inconsistency here. |
6453 | 518 |
if (morphism == 1 && count > 0) { |
519 |
epsilon = count; |
|
520 |
count = 0; |
|
521 |
} |
|
522 |
} |
|
523 |
for (uint i = 0; i < call->row_limit(); i++) { |
|
524 |
ciKlass* receiver = call->receiver(i); |
|
525 |
if (receiver == NULL) continue; |
|
55252
6502d6a92fe2
8224162: assert(profile.count() == 0) failed: sanity in InlineTree::is_not_reached
jiefu
parents:
55206
diff
changeset
|
526 |
int rcount = saturated_add(call->receiver_count(i), epsilon); |
1 | 527 |
if (rcount == 0) rcount = 1; // Should be valid value |
55252
6502d6a92fe2
8224162: assert(profile.count() == 0) failed: sanity in InlineTree::is_not_reached
jiefu
parents:
55206
diff
changeset
|
528 |
receivers_count_total = saturated_add(receivers_count_total, rcount); |
1 | 529 |
// Add the receiver to result data. |
530 |
result.add_receiver(receiver, rcount); |
|
531 |
// If we extend profiling to record methods, |
|
532 |
// we will set result._method also. |
|
533 |
} |
|
534 |
// Determine call site's morphism. |
|
46326
70de7011f79a
8159734: Consistency check fails with -XX:-ProfileInterpreter
zmajo
parents:
40664
diff
changeset
|
535 |
// The call site count is 0 with known morphism (only 1 or 2 receivers) |
70de7011f79a
8159734: Consistency check fails with -XX:-ProfileInterpreter
zmajo
parents:
40664
diff
changeset
|
536 |
// or < 0 in the case of a type check failure for checkcast, aastore, instanceof. |
4754
8aef16f24e16
6614597: Performance variability in jvm2008 xml.validation
kvn
parents:
4581
diff
changeset
|
537 |
// The call site count is > 0 in the case of a polymorphic virtual call. |
8aef16f24e16
6614597: Performance variability in jvm2008 xml.validation
kvn
parents:
4581
diff
changeset
|
538 |
if (morphism > 0 && morphism == result._limit) { |
8aef16f24e16
6614597: Performance variability in jvm2008 xml.validation
kvn
parents:
4581
diff
changeset
|
539 |
// The morphism <= MorphismLimit. |
8aef16f24e16
6614597: Performance variability in jvm2008 xml.validation
kvn
parents:
4581
diff
changeset
|
540 |
if ((morphism < ciCallProfile::MorphismLimit) || |
8aef16f24e16
6614597: Performance variability in jvm2008 xml.validation
kvn
parents:
4581
diff
changeset
|
541 |
(morphism == ciCallProfile::MorphismLimit && count == 0)) { |
8aef16f24e16
6614597: Performance variability in jvm2008 xml.validation
kvn
parents:
4581
diff
changeset
|
542 |
#ifdef ASSERT |
8aef16f24e16
6614597: Performance variability in jvm2008 xml.validation
kvn
parents:
4581
diff
changeset
|
543 |
if (count > 0) { |
4892
e977b527544a
6923002: assert(false,"this call site should not be polymorphic")
kvn
parents:
4754
diff
changeset
|
544 |
this->print_short_name(tty); |
e977b527544a
6923002: assert(false,"this call site should not be polymorphic")
kvn
parents:
4754
diff
changeset
|
545 |
tty->print_cr(" @ bci:%d", bci); |
4754
8aef16f24e16
6614597: Performance variability in jvm2008 xml.validation
kvn
parents:
4581
diff
changeset
|
546 |
this->print_codes(); |
8aef16f24e16
6614597: Performance variability in jvm2008 xml.validation
kvn
parents:
4581
diff
changeset
|
547 |
assert(false, "this call site should not be polymorphic"); |
8aef16f24e16
6614597: Performance variability in jvm2008 xml.validation
kvn
parents:
4581
diff
changeset
|
548 |
} |
8aef16f24e16
6614597: Performance variability in jvm2008 xml.validation
kvn
parents:
4581
diff
changeset
|
549 |
#endif |
1 | 550 |
result._morphism = morphism; |
551 |
} |
|
552 |
} |
|
553 |
// Make the count consistent if this is a call profile. If count is |
|
554 |
// zero or less, presume that this is a typecheck profile and |
|
555 |
// do nothing. Otherwise, increase count to be the sum of all |
|
556 |
// receiver's counts. |
|
4754
8aef16f24e16
6614597: Performance variability in jvm2008 xml.validation
kvn
parents:
4581
diff
changeset
|
557 |
if (count >= 0) { |
55252
6502d6a92fe2
8224162: assert(profile.count() == 0) failed: sanity in InlineTree::is_not_reached
jiefu
parents:
55206
diff
changeset
|
558 |
count = saturated_add(count, receivers_count_total); |
1 | 559 |
} |
560 |
} |
|
561 |
result._count = count; |
|
562 |
} |
|
563 |
} |
|
564 |
return result; |
|
565 |
} |
|
566 |
||
567 |
// ------------------------------------------------------------------ |
|
568 |
// Add new receiver and sort data by receiver's profile count. |
|
569 |
void ciCallProfile::add_receiver(ciKlass* receiver, int receiver_count) { |
|
570 |
// Add new receiver and sort data by receiver's counts when we have space |
|
571 |
// for it otherwise replace the less called receiver (less called receiver |
|
572 |
// is placed to the last array element which is not used). |
|
573 |
// First array's element contains most called receiver. |
|
574 |
int i = _limit; |
|
575 |
for (; i > 0 && receiver_count > _receiver_count[i-1]; i--) { |
|
576 |
_receiver[i] = _receiver[i-1]; |
|
577 |
_receiver_count[i] = _receiver_count[i-1]; |
|
578 |
} |
|
579 |
_receiver[i] = receiver; |
|
580 |
_receiver_count[i] = receiver_count; |
|
581 |
if (_limit < MorphismLimit) _limit++; |
|
582 |
} |
|
583 |
||
21099 | 584 |
|
585 |
void ciMethod::assert_virtual_call_type_ok(int bci) { |
|
586 |
assert(java_code_at_bci(bci) == Bytecodes::_invokevirtual || |
|
33105
294e48b4f704
8080775: Better argument formatting for assert() and friends
david
parents:
31790
diff
changeset
|
587 |
java_code_at_bci(bci) == Bytecodes::_invokeinterface, "unexpected bytecode %s", Bytecodes::name(java_code_at_bci(bci))); |
21099 | 588 |
} |
589 |
||
590 |
void ciMethod::assert_call_type_ok(int bci) { |
|
591 |
assert(java_code_at_bci(bci) == Bytecodes::_invokestatic || |
|
592 |
java_code_at_bci(bci) == Bytecodes::_invokespecial || |
|
33105
294e48b4f704
8080775: Better argument formatting for assert() and friends
david
parents:
31790
diff
changeset
|
593 |
java_code_at_bci(bci) == Bytecodes::_invokedynamic, "unexpected bytecode %s", Bytecodes::name(java_code_at_bci(bci))); |
21099 | 594 |
} |
595 |
||
596 |
/** |
|
597 |
* Check whether profiling provides a type for the argument i to the |
|
598 |
* call at bci bci |
|
599 |
* |
|
23525
e3eb08ead679
8031755: Type speculation should be used to optimize explicit null checks
roland
parents:
23194
diff
changeset
|
600 |
* @param [in]bci bci of the call |
e3eb08ead679
8031755: Type speculation should be used to optimize explicit null checks
roland
parents:
23194
diff
changeset
|
601 |
* @param [in]i argument number |
e3eb08ead679
8031755: Type speculation should be used to optimize explicit null checks
roland
parents:
23194
diff
changeset
|
602 |
* @param [out]type profiled type of argument, NULL if none |
46542
73dd19b96b5d
8181211: C2: Use profiling data to optimize on/off heap unsafe accesses
roland
parents:
46458
diff
changeset
|
603 |
* @param [out]ptr_kind whether always null, never null or maybe null |
23525
e3eb08ead679
8031755: Type speculation should be used to optimize explicit null checks
roland
parents:
23194
diff
changeset
|
604 |
* @return true if profiling exists |
21099 | 605 |
* |
606 |
*/ |
|
46542
73dd19b96b5d
8181211: C2: Use profiling data to optimize on/off heap unsafe accesses
roland
parents:
46458
diff
changeset
|
607 |
bool ciMethod::argument_profiled_type(int bci, int i, ciKlass*& type, ProfilePtrKind& ptr_kind) { |
21099 | 608 |
if (MethodData::profile_parameters() && method_data() != NULL && method_data()->is_mature()) { |
609 |
ciProfileData* data = method_data()->bci_to_data(bci); |
|
610 |
if (data != NULL) { |
|
611 |
if (data->is_VirtualCallTypeData()) { |
|
612 |
assert_virtual_call_type_ok(bci); |
|
613 |
ciVirtualCallTypeData* call = (ciVirtualCallTypeData*)data->as_VirtualCallTypeData(); |
|
614 |
if (i >= call->number_of_arguments()) { |
|
23525
e3eb08ead679
8031755: Type speculation should be used to optimize explicit null checks
roland
parents:
23194
diff
changeset
|
615 |
return false; |
21099 | 616 |
} |
23525
e3eb08ead679
8031755: Type speculation should be used to optimize explicit null checks
roland
parents:
23194
diff
changeset
|
617 |
type = call->valid_argument_type(i); |
46542
73dd19b96b5d
8181211: C2: Use profiling data to optimize on/off heap unsafe accesses
roland
parents:
46458
diff
changeset
|
618 |
ptr_kind = call->argument_ptr_kind(i); |
23525
e3eb08ead679
8031755: Type speculation should be used to optimize explicit null checks
roland
parents:
23194
diff
changeset
|
619 |
return true; |
21099 | 620 |
} else if (data->is_CallTypeData()) { |
621 |
assert_call_type_ok(bci); |
|
622 |
ciCallTypeData* call = (ciCallTypeData*)data->as_CallTypeData(); |
|
623 |
if (i >= call->number_of_arguments()) { |
|
23525
e3eb08ead679
8031755: Type speculation should be used to optimize explicit null checks
roland
parents:
23194
diff
changeset
|
624 |
return false; |
21099 | 625 |
} |
23525
e3eb08ead679
8031755: Type speculation should be used to optimize explicit null checks
roland
parents:
23194
diff
changeset
|
626 |
type = call->valid_argument_type(i); |
46542
73dd19b96b5d
8181211: C2: Use profiling data to optimize on/off heap unsafe accesses
roland
parents:
46458
diff
changeset
|
627 |
ptr_kind = call->argument_ptr_kind(i); |
23525
e3eb08ead679
8031755: Type speculation should be used to optimize explicit null checks
roland
parents:
23194
diff
changeset
|
628 |
return true; |
21099 | 629 |
} |
630 |
} |
|
631 |
} |
|
23525
e3eb08ead679
8031755: Type speculation should be used to optimize explicit null checks
roland
parents:
23194
diff
changeset
|
632 |
return false; |
21099 | 633 |
} |
634 |
||
635 |
/** |
|
636 |
* Check whether profiling provides a type for the return value from |
|
637 |
* the call at bci bci |
|
638 |
* |
|
23525
e3eb08ead679
8031755: Type speculation should be used to optimize explicit null checks
roland
parents:
23194
diff
changeset
|
639 |
* @param [in]bci bci of the call |
e3eb08ead679
8031755: Type speculation should be used to optimize explicit null checks
roland
parents:
23194
diff
changeset
|
640 |
* @param [out]type profiled type of argument, NULL if none |
46542
73dd19b96b5d
8181211: C2: Use profiling data to optimize on/off heap unsafe accesses
roland
parents:
46458
diff
changeset
|
641 |
* @param [out]ptr_kind whether always null, never null or maybe null |
23525
e3eb08ead679
8031755: Type speculation should be used to optimize explicit null checks
roland
parents:
23194
diff
changeset
|
642 |
* @return true if profiling exists |
21099 | 643 |
* |
644 |
*/ |
|
46542
73dd19b96b5d
8181211: C2: Use profiling data to optimize on/off heap unsafe accesses
roland
parents:
46458
diff
changeset
|
645 |
bool ciMethod::return_profiled_type(int bci, ciKlass*& type, ProfilePtrKind& ptr_kind) { |
21099 | 646 |
if (MethodData::profile_return() && method_data() != NULL && method_data()->is_mature()) { |
647 |
ciProfileData* data = method_data()->bci_to_data(bci); |
|
648 |
if (data != NULL) { |
|
649 |
if (data->is_VirtualCallTypeData()) { |
|
650 |
assert_virtual_call_type_ok(bci); |
|
651 |
ciVirtualCallTypeData* call = (ciVirtualCallTypeData*)data->as_VirtualCallTypeData(); |
|
46542
73dd19b96b5d
8181211: C2: Use profiling data to optimize on/off heap unsafe accesses
roland
parents:
46458
diff
changeset
|
652 |
if (call->has_return()) { |
73dd19b96b5d
8181211: C2: Use profiling data to optimize on/off heap unsafe accesses
roland
parents:
46458
diff
changeset
|
653 |
type = call->valid_return_type(); |
73dd19b96b5d
8181211: C2: Use profiling data to optimize on/off heap unsafe accesses
roland
parents:
46458
diff
changeset
|
654 |
ptr_kind = call->return_ptr_kind(); |
73dd19b96b5d
8181211: C2: Use profiling data to optimize on/off heap unsafe accesses
roland
parents:
46458
diff
changeset
|
655 |
return true; |
73dd19b96b5d
8181211: C2: Use profiling data to optimize on/off heap unsafe accesses
roland
parents:
46458
diff
changeset
|
656 |
} |
21099 | 657 |
} else if (data->is_CallTypeData()) { |
658 |
assert_call_type_ok(bci); |
|
659 |
ciCallTypeData* call = (ciCallTypeData*)data->as_CallTypeData(); |
|
46542
73dd19b96b5d
8181211: C2: Use profiling data to optimize on/off heap unsafe accesses
roland
parents:
46458
diff
changeset
|
660 |
if (call->has_return()) { |
73dd19b96b5d
8181211: C2: Use profiling data to optimize on/off heap unsafe accesses
roland
parents:
46458
diff
changeset
|
661 |
type = call->valid_return_type(); |
73dd19b96b5d
8181211: C2: Use profiling data to optimize on/off heap unsafe accesses
roland
parents:
46458
diff
changeset
|
662 |
ptr_kind = call->return_ptr_kind(); |
73dd19b96b5d
8181211: C2: Use profiling data to optimize on/off heap unsafe accesses
roland
parents:
46458
diff
changeset
|
663 |
} |
23525
e3eb08ead679
8031755: Type speculation should be used to optimize explicit null checks
roland
parents:
23194
diff
changeset
|
664 |
return true; |
21099 | 665 |
} |
666 |
} |
|
667 |
} |
|
23525
e3eb08ead679
8031755: Type speculation should be used to optimize explicit null checks
roland
parents:
23194
diff
changeset
|
668 |
return false; |
21099 | 669 |
} |
670 |
||
671 |
/** |
|
672 |
* Check whether profiling provides a type for the parameter i |
|
673 |
* |
|
23525
e3eb08ead679
8031755: Type speculation should be used to optimize explicit null checks
roland
parents:
23194
diff
changeset
|
674 |
* @param [in]i parameter number |
e3eb08ead679
8031755: Type speculation should be used to optimize explicit null checks
roland
parents:
23194
diff
changeset
|
675 |
* @param [out]type profiled type of parameter, NULL if none |
46542
73dd19b96b5d
8181211: C2: Use profiling data to optimize on/off heap unsafe accesses
roland
parents:
46458
diff
changeset
|
676 |
* @param [out]ptr_kind whether always null, never null or maybe null |
23525
e3eb08ead679
8031755: Type speculation should be used to optimize explicit null checks
roland
parents:
23194
diff
changeset
|
677 |
* @return true if profiling exists |
21099 | 678 |
* |
679 |
*/ |
|
46542
73dd19b96b5d
8181211: C2: Use profiling data to optimize on/off heap unsafe accesses
roland
parents:
46458
diff
changeset
|
680 |
bool ciMethod::parameter_profiled_type(int i, ciKlass*& type, ProfilePtrKind& ptr_kind) { |
21099 | 681 |
if (MethodData::profile_parameters() && method_data() != NULL && method_data()->is_mature()) { |
682 |
ciParametersTypeData* parameters = method_data()->parameters_type_data(); |
|
683 |
if (parameters != NULL && i < parameters->number_of_parameters()) { |
|
23525
e3eb08ead679
8031755: Type speculation should be used to optimize explicit null checks
roland
parents:
23194
diff
changeset
|
684 |
type = parameters->valid_parameter_type(i); |
46542
73dd19b96b5d
8181211: C2: Use profiling data to optimize on/off heap unsafe accesses
roland
parents:
46458
diff
changeset
|
685 |
ptr_kind = parameters->parameter_ptr_kind(i); |
23525
e3eb08ead679
8031755: Type speculation should be used to optimize explicit null checks
roland
parents:
23194
diff
changeset
|
686 |
return true; |
21099 | 687 |
} |
688 |
} |
|
23525
e3eb08ead679
8031755: Type speculation should be used to optimize explicit null checks
roland
parents:
23194
diff
changeset
|
689 |
return false; |
21099 | 690 |
} |
691 |
||
692 |
||
1 | 693 |
// ------------------------------------------------------------------ |
694 |
// ciMethod::find_monomorphic_target |
|
695 |
// |
|
696 |
// Given a certain calling environment, find the monomorphic target |
|
697 |
// for the call. Return NULL if the call is not monomorphic in |
|
698 |
// its calling environment, or if there are only abstract methods. |
|
699 |
// The returned method is never abstract. |
|
700 |
// Note: If caller uses a non-null result, it must inform dependencies |
|
701 |
// via assert_unique_concrete_method or assert_leaf_type. |
|
702 |
ciMethod* ciMethod::find_monomorphic_target(ciInstanceKlass* caller, |
|
703 |
ciInstanceKlass* callee_holder, |
|
30223
82ab7b6b4927
8062280: C2: inlining failure due to access checks being too strict
vlivanov
parents:
29582
diff
changeset
|
704 |
ciInstanceKlass* actual_recv, |
82ab7b6b4927
8062280: C2: inlining failure due to access checks being too strict
vlivanov
parents:
29582
diff
changeset
|
705 |
bool check_access) { |
1 | 706 |
check_is_loaded(); |
707 |
||
708 |
if (actual_recv->is_interface()) { |
|
709 |
// %%% We cannot trust interface types, yet. See bug 6312651. |
|
710 |
return NULL; |
|
711 |
} |
|
712 |
||
30223
82ab7b6b4927
8062280: C2: inlining failure due to access checks being too strict
vlivanov
parents:
29582
diff
changeset
|
713 |
ciMethod* root_m = resolve_invoke(caller, actual_recv, check_access); |
1 | 714 |
if (root_m == NULL) { |
715 |
// Something went wrong looking up the actual receiver method. |
|
716 |
return NULL; |
|
717 |
} |
|
718 |
assert(!root_m->is_abstract(), "resolve_invoke promise"); |
|
719 |
||
720 |
// Make certain quick checks even if UseCHA is false. |
|
721 |
||
722 |
// Is it private or final? |
|
723 |
if (root_m->can_be_statically_bound()) { |
|
724 |
return root_m; |
|
725 |
} |
|
726 |
||
727 |
if (actual_recv->is_leaf_type() && actual_recv == root_m->holder()) { |
|
728 |
// Easy case. There is no other place to put a method, so don't bother |
|
729 |
// to go through the VM_ENTRY_MARK and all the rest. |
|
730 |
return root_m; |
|
731 |
} |
|
732 |
||
733 |
// Array methods (clone, hashCode, etc.) are always statically bound. |
|
734 |
// If we were to see an array type here, we'd return root_m. |
|
735 |
// However, this method processes only ciInstanceKlasses. (See 4962591.) |
|
736 |
// The inline_native_clone intrinsic narrows Object to T[] properly, |
|
737 |
// so there is no need to do the same job here. |
|
738 |
||
739 |
if (!UseCHA) return NULL; |
|
740 |
||
741 |
VM_ENTRY_MARK; |
|
742 |
||
23194
e60d7409415b
8036100: Default method returns true for a while, and then returns false
vlivanov
parents:
22893
diff
changeset
|
743 |
// Disable CHA for default methods for now |
53595 | 744 |
if (root_m->is_default_method()) { |
23194
e60d7409415b
8036100: Default method returns true for a while, and then returns false
vlivanov
parents:
22893
diff
changeset
|
745 |
return NULL; |
e60d7409415b
8036100: Default method returns true for a while, and then returns false
vlivanov
parents:
22893
diff
changeset
|
746 |
} |
e60d7409415b
8036100: Default method returns true for a while, and then returns false
vlivanov
parents:
22893
diff
changeset
|
747 |
|
1 | 748 |
methodHandle target; |
749 |
{ |
|
750 |
MutexLocker locker(Compile_lock); |
|
13728
882756847a04
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
13522
diff
changeset
|
751 |
Klass* context = actual_recv->get_Klass(); |
59056
15936b142f86
8233913: Remove implicit conversion from Method* to methodHandle
coleenp
parents:
58273
diff
changeset
|
752 |
target = methodHandle(THREAD, Dependencies::find_unique_concrete_method(context, |
15936b142f86
8233913: Remove implicit conversion from Method* to methodHandle
coleenp
parents:
58273
diff
changeset
|
753 |
root_m->get_Method())); |
1 | 754 |
// %%% Should upgrade this ciMethod API to look for 1 or 2 concrete methods. |
755 |
} |
|
756 |
||
757 |
#ifndef PRODUCT |
|
13728
882756847a04
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
13522
diff
changeset
|
758 |
if (TraceDependencies && target() != NULL && target() != root_m->get_Method()) { |
1 | 759 |
tty->print("found a non-root unique target method"); |
33602 | 760 |
tty->print_cr(" context = %s", actual_recv->get_Klass()->external_name()); |
1 | 761 |
tty->print(" method = "); |
762 |
target->print_short_name(tty); |
|
763 |
tty->cr(); |
|
764 |
} |
|
765 |
#endif //PRODUCT |
|
766 |
||
767 |
if (target() == NULL) { |
|
768 |
return NULL; |
|
769 |
} |
|
13728
882756847a04
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
13522
diff
changeset
|
770 |
if (target() == root_m->get_Method()) { |
1 | 771 |
return root_m; |
772 |
} |
|
773 |
if (!root_m->is_public() && |
|
774 |
!root_m->is_protected()) { |
|
775 |
// If we are going to reason about inheritance, it's easiest |
|
776 |
// if the method in question is public, protected, or private. |
|
777 |
// If the answer is not root_m, it is conservatively correct |
|
778 |
// to return NULL, even if the CHA encountered irrelevant |
|
779 |
// methods in other packages. |
|
780 |
// %%% TO DO: Work out logic for package-private methods |
|
781 |
// with the same name but different vtable indexes. |
|
782 |
return NULL; |
|
783 |
} |
|
53595 | 784 |
assert(!target()->is_abstract(), "not allowed"); |
13728
882756847a04
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
13522
diff
changeset
|
785 |
return CURRENT_THREAD_ENV->get_method(target()); |
1 | 786 |
} |
787 |
||
788 |
// ------------------------------------------------------------------ |
|
54721
3661ad97da8f
8223171: Redundant nmethod dependencies for effectively final methods
vlivanov
parents:
53595
diff
changeset
|
789 |
// ciMethod::can_be_statically_bound |
3661ad97da8f
8223171: Redundant nmethod dependencies for effectively final methods
vlivanov
parents:
53595
diff
changeset
|
790 |
// |
3661ad97da8f
8223171: Redundant nmethod dependencies for effectively final methods
vlivanov
parents:
53595
diff
changeset
|
791 |
// Tries to determine whether a method can be statically bound in some context. |
3661ad97da8f
8223171: Redundant nmethod dependencies for effectively final methods
vlivanov
parents:
53595
diff
changeset
|
792 |
bool ciMethod::can_be_statically_bound(ciInstanceKlass* context) const { |
3661ad97da8f
8223171: Redundant nmethod dependencies for effectively final methods
vlivanov
parents:
53595
diff
changeset
|
793 |
return (holder() == context) && can_be_statically_bound(); |
3661ad97da8f
8223171: Redundant nmethod dependencies for effectively final methods
vlivanov
parents:
53595
diff
changeset
|
794 |
} |
3661ad97da8f
8223171: Redundant nmethod dependencies for effectively final methods
vlivanov
parents:
53595
diff
changeset
|
795 |
|
3661ad97da8f
8223171: Redundant nmethod dependencies for effectively final methods
vlivanov
parents:
53595
diff
changeset
|
796 |
// ------------------------------------------------------------------ |
1 | 797 |
// ciMethod::resolve_invoke |
798 |
// |
|
799 |
// Given a known receiver klass, find the target for the call. |
|
800 |
// Return NULL if the call has no target or the target is abstract. |
|
30223
82ab7b6b4927
8062280: C2: inlining failure due to access checks being too strict
vlivanov
parents:
29582
diff
changeset
|
801 |
ciMethod* ciMethod::resolve_invoke(ciKlass* caller, ciKlass* exact_receiver, bool check_access) { |
1 | 802 |
check_is_loaded(); |
803 |
VM_ENTRY_MARK; |
|
804 |
||
46329
53ccc37bda19
8155672: Remove instanceKlassHandles and KlassHandles
coleenp
parents:
46327
diff
changeset
|
805 |
Klass* caller_klass = caller->get_Klass(); |
53ccc37bda19
8155672: Remove instanceKlassHandles and KlassHandles
coleenp
parents:
46327
diff
changeset
|
806 |
Klass* recv = exact_receiver->get_Klass(); |
53ccc37bda19
8155672: Remove instanceKlassHandles and KlassHandles
coleenp
parents:
46327
diff
changeset
|
807 |
Klass* resolved = holder()->get_Klass(); |
8076
96d498ec7ae1
6990754: Use native memory and reference counting to implement SymbolTable
coleenp
parents:
7432
diff
changeset
|
808 |
Symbol* h_name = name()->get_symbol(); |
96d498ec7ae1
6990754: Use native memory and reference counting to implement SymbolTable
coleenp
parents:
7432
diff
changeset
|
809 |
Symbol* h_signature = signature()->get_symbol(); |
1 | 810 |
|
46329
53ccc37bda19
8155672: Remove instanceKlassHandles and KlassHandles
coleenp
parents:
46327
diff
changeset
|
811 |
LinkInfo link_info(resolved, h_name, h_signature, caller_klass, |
38719
133bf85c3f36
8145148: InterfaceMethod CP entry pointing to a class should cause ICCE
coleenp
parents:
38177
diff
changeset
|
812 |
check_access ? LinkInfo::needs_access_check : LinkInfo::skip_access_check); |
59056
15936b142f86
8233913: Remove implicit conversion from Method* to methodHandle
coleenp
parents:
58273
diff
changeset
|
813 |
Method* m = NULL; |
1 | 814 |
// Only do exact lookup if receiver klass has been linked. Otherwise, |
815 |
// the vtable has not been setup, and the LinkResolver will fail. |
|
46329
53ccc37bda19
8155672: Remove instanceKlassHandles and KlassHandles
coleenp
parents:
46327
diff
changeset
|
816 |
if (recv->is_array_klass() |
1 | 817 |
|| |
46630
75aa3e39d02c
8182299: Enable disabled clang warnings, build on OSX 10 + Xcode 8
jwilhelm
parents:
46542
diff
changeset
|
818 |
(InstanceKlass::cast(recv)->is_linked() && !exact_receiver->is_interface())) { |
1 | 819 |
if (holder()->is_interface()) { |
46329
53ccc37bda19
8155672: Remove instanceKlassHandles and KlassHandles
coleenp
parents:
46327
diff
changeset
|
820 |
m = LinkResolver::resolve_interface_call_or_null(recv, link_info); |
1 | 821 |
} else { |
46329
53ccc37bda19
8155672: Remove instanceKlassHandles and KlassHandles
coleenp
parents:
46327
diff
changeset
|
822 |
m = LinkResolver::resolve_virtual_call_or_null(recv, link_info); |
1 | 823 |
} |
824 |
} |
|
825 |
||
59056
15936b142f86
8233913: Remove implicit conversion from Method* to methodHandle
coleenp
parents:
58273
diff
changeset
|
826 |
if (m == NULL) { |
1 | 827 |
// Return NULL only if there was a problem with lookup (uninitialized class, etc.) |
828 |
return NULL; |
|
829 |
} |
|
830 |
||
831 |
ciMethod* result = this; |
|
59056
15936b142f86
8233913: Remove implicit conversion from Method* to methodHandle
coleenp
parents:
58273
diff
changeset
|
832 |
if (m != get_Method()) { |
15936b142f86
8233913: Remove implicit conversion from Method* to methodHandle
coleenp
parents:
58273
diff
changeset
|
833 |
result = CURRENT_THREAD_ENV->get_method(m); |
1 | 834 |
} |
835 |
||
836 |
// Don't return abstract methods because they aren't |
|
837 |
// optimizable or interesting. |
|
838 |
if (result->is_abstract()) { |
|
839 |
return NULL; |
|
840 |
} else { |
|
841 |
return result; |
|
842 |
} |
|
843 |
} |
|
844 |
||
845 |
// ------------------------------------------------------------------ |
|
846 |
// ciMethod::resolve_vtable_index |
|
847 |
// |
|
848 |
// Given a known receiver klass, find the vtable index for the call. |
|
13728
882756847a04
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
13522
diff
changeset
|
849 |
// Return Method::invalid_vtable_index if the vtable_index is unknown. |
1 | 850 |
int ciMethod::resolve_vtable_index(ciKlass* caller, ciKlass* receiver) { |
851 |
check_is_loaded(); |
|
852 |
||
13728
882756847a04
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
13522
diff
changeset
|
853 |
int vtable_index = Method::invalid_vtable_index; |
1 | 854 |
// Only do lookup if receiver klass has been linked. Otherwise, |
855 |
// the vtable has not been setup, and the LinkResolver will fail. |
|
856 |
if (!receiver->is_interface() |
|
857 |
&& (!receiver->is_instance_klass() || |
|
858 |
receiver->as_instance_klass()->is_linked())) { |
|
859 |
VM_ENTRY_MARK; |
|
860 |
||
46329
53ccc37bda19
8155672: Remove instanceKlassHandles and KlassHandles
coleenp
parents:
46327
diff
changeset
|
861 |
Klass* caller_klass = caller->get_Klass(); |
53ccc37bda19
8155672: Remove instanceKlassHandles and KlassHandles
coleenp
parents:
46327
diff
changeset
|
862 |
Klass* recv = receiver->get_Klass(); |
8076
96d498ec7ae1
6990754: Use native memory and reference counting to implement SymbolTable
coleenp
parents:
7432
diff
changeset
|
863 |
Symbol* h_name = name()->get_symbol(); |
96d498ec7ae1
6990754: Use native memory and reference counting to implement SymbolTable
coleenp
parents:
7432
diff
changeset
|
864 |
Symbol* h_signature = signature()->get_symbol(); |
1 | 865 |
|
46329
53ccc37bda19
8155672: Remove instanceKlassHandles and KlassHandles
coleenp
parents:
46327
diff
changeset
|
866 |
LinkInfo link_info(recv, h_name, h_signature, caller_klass); |
53ccc37bda19
8155672: Remove instanceKlassHandles and KlassHandles
coleenp
parents:
46327
diff
changeset
|
867 |
vtable_index = LinkResolver::resolve_virtual_vtable_index(recv, link_info); |
13728
882756847a04
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
13522
diff
changeset
|
868 |
if (vtable_index == Method::nonvirtual_vtable_index) { |
1 | 869 |
// A statically bound method. Return "no such index". |
13728
882756847a04
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
13522
diff
changeset
|
870 |
vtable_index = Method::invalid_vtable_index; |
1 | 871 |
} |
872 |
} |
|
873 |
||
874 |
return vtable_index; |
|
875 |
} |
|
876 |
||
877 |
// ------------------------------------------------------------------ |
|
878 |
// ciMethod::interpreter_call_site_count |
|
879 |
int ciMethod::interpreter_call_site_count(int bci) { |
|
880 |
if (method_data() != NULL) { |
|
881 |
ResourceMark rm; |
|
882 |
ciProfileData* data = method_data()->bci_to_data(bci); |
|
883 |
if (data != NULL && data->is_CounterData()) { |
|
884 |
return scale_count(data->as_CounterData()->count()); |
|
885 |
} |
|
886 |
} |
|
887 |
return -1; // unknown |
|
888 |
} |
|
889 |
||
890 |
// ------------------------------------------------------------------ |
|
14621
fd9265ab0f67
7172640: C2: instrinsic implementations in LibraryCallKit should use argument() instead of pop()
twisti
parents:
14478
diff
changeset
|
891 |
// ciMethod::get_field_at_bci |
fd9265ab0f67
7172640: C2: instrinsic implementations in LibraryCallKit should use argument() instead of pop()
twisti
parents:
14478
diff
changeset
|
892 |
ciField* ciMethod::get_field_at_bci(int bci, bool &will_link) { |
fd9265ab0f67
7172640: C2: instrinsic implementations in LibraryCallKit should use argument() instead of pop()
twisti
parents:
14478
diff
changeset
|
893 |
ciBytecodeStream iter(this); |
fd9265ab0f67
7172640: C2: instrinsic implementations in LibraryCallKit should use argument() instead of pop()
twisti
parents:
14478
diff
changeset
|
894 |
iter.reset_to_bci(bci); |
fd9265ab0f67
7172640: C2: instrinsic implementations in LibraryCallKit should use argument() instead of pop()
twisti
parents:
14478
diff
changeset
|
895 |
iter.next(); |
fd9265ab0f67
7172640: C2: instrinsic implementations in LibraryCallKit should use argument() instead of pop()
twisti
parents:
14478
diff
changeset
|
896 |
return iter.get_field(will_link); |
fd9265ab0f67
7172640: C2: instrinsic implementations in LibraryCallKit should use argument() instead of pop()
twisti
parents:
14478
diff
changeset
|
897 |
} |
fd9265ab0f67
7172640: C2: instrinsic implementations in LibraryCallKit should use argument() instead of pop()
twisti
parents:
14478
diff
changeset
|
898 |
|
fd9265ab0f67
7172640: C2: instrinsic implementations in LibraryCallKit should use argument() instead of pop()
twisti
parents:
14478
diff
changeset
|
899 |
// ------------------------------------------------------------------ |
fd9265ab0f67
7172640: C2: instrinsic implementations in LibraryCallKit should use argument() instead of pop()
twisti
parents:
14478
diff
changeset
|
900 |
// ciMethod::get_method_at_bci |
fd9265ab0f67
7172640: C2: instrinsic implementations in LibraryCallKit should use argument() instead of pop()
twisti
parents:
14478
diff
changeset
|
901 |
ciMethod* ciMethod::get_method_at_bci(int bci, bool &will_link, ciSignature* *declared_signature) { |
fd9265ab0f67
7172640: C2: instrinsic implementations in LibraryCallKit should use argument() instead of pop()
twisti
parents:
14478
diff
changeset
|
902 |
ciBytecodeStream iter(this); |
fd9265ab0f67
7172640: C2: instrinsic implementations in LibraryCallKit should use argument() instead of pop()
twisti
parents:
14478
diff
changeset
|
903 |
iter.reset_to_bci(bci); |
fd9265ab0f67
7172640: C2: instrinsic implementations in LibraryCallKit should use argument() instead of pop()
twisti
parents:
14478
diff
changeset
|
904 |
iter.next(); |
fd9265ab0f67
7172640: C2: instrinsic implementations in LibraryCallKit should use argument() instead of pop()
twisti
parents:
14478
diff
changeset
|
905 |
return iter.get_method(will_link, declared_signature); |
fd9265ab0f67
7172640: C2: instrinsic implementations in LibraryCallKit should use argument() instead of pop()
twisti
parents:
14478
diff
changeset
|
906 |
} |
fd9265ab0f67
7172640: C2: instrinsic implementations in LibraryCallKit should use argument() instead of pop()
twisti
parents:
14478
diff
changeset
|
907 |
|
fd9265ab0f67
7172640: C2: instrinsic implementations in LibraryCallKit should use argument() instead of pop()
twisti
parents:
14478
diff
changeset
|
908 |
// ------------------------------------------------------------------ |
53595 | 909 |
ciKlass* ciMethod::get_declared_method_holder_at_bci(int bci) { |
910 |
ciBytecodeStream iter(this); |
|
911 |
iter.reset_to_bci(bci); |
|
912 |
iter.next(); |
|
913 |
return iter.get_declared_method_holder(); |
|
914 |
} |
|
915 |
||
916 |
// ------------------------------------------------------------------ |
|
1 | 917 |
// Adjust a CounterData count to be commensurate with |
918 |
// interpreter_invocation_count. If the MDO exists for |
|
919 |
// only 25% of the time the method exists, then the |
|
920 |
// counts in the MDO should be scaled by 4X, so that |
|
921 |
// they can be usefully and stably compared against the |
|
922 |
// invocation counts in methods. |
|
923 |
int ciMethod::scale_count(int count, float prof_factor) { |
|
924 |
if (count > 0 && method_data() != NULL) { |
|
6453 | 925 |
int counter_life; |
1 | 926 |
int method_life = interpreter_invocation_count(); |
6453 | 927 |
if (TieredCompilation) { |
928 |
// In tiered the MDO's life is measured directly, so just use the snapshotted counters |
|
929 |
counter_life = MAX2(method_data()->invocation_count(), method_data()->backedge_count()); |
|
930 |
} else { |
|
931 |
int current_mileage = method_data()->current_mileage(); |
|
932 |
int creation_mileage = method_data()->creation_mileage(); |
|
933 |
counter_life = current_mileage - creation_mileage; |
|
934 |
} |
|
935 |
||
1 | 936 |
// counter_life due to backedge_counter could be > method_life |
937 |
if (counter_life > method_life) |
|
938 |
counter_life = method_life; |
|
939 |
if (0 < counter_life && counter_life <= method_life) { |
|
940 |
count = (int)((double)count * prof_factor * method_life / counter_life + 0.5); |
|
941 |
count = (count > 0) ? count : 1; |
|
942 |
} |
|
943 |
} |
|
944 |
return count; |
|
945 |
} |
|
946 |
||
16617
6235d2c7549f
7198429: need checked categorization of caller-sensitive methods in the JDK
twisti
parents:
15479
diff
changeset
|
947 |
|
6235d2c7549f
7198429: need checked categorization of caller-sensitive methods in the JDK
twisti
parents:
15479
diff
changeset
|
948 |
// ------------------------------------------------------------------ |
6235d2c7549f
7198429: need checked categorization of caller-sensitive methods in the JDK
twisti
parents:
15479
diff
changeset
|
949 |
// ciMethod::is_special_get_caller_class_method |
6235d2c7549f
7198429: need checked categorization of caller-sensitive methods in the JDK
twisti
parents:
15479
diff
changeset
|
950 |
// |
6235d2c7549f
7198429: need checked categorization of caller-sensitive methods in the JDK
twisti
parents:
15479
diff
changeset
|
951 |
bool ciMethod::is_ignored_by_security_stack_walk() const { |
6235d2c7549f
7198429: need checked categorization of caller-sensitive methods in the JDK
twisti
parents:
15479
diff
changeset
|
952 |
check_is_loaded(); |
6235d2c7549f
7198429: need checked categorization of caller-sensitive methods in the JDK
twisti
parents:
15479
diff
changeset
|
953 |
VM_ENTRY_MARK; |
6235d2c7549f
7198429: need checked categorization of caller-sensitive methods in the JDK
twisti
parents:
15479
diff
changeset
|
954 |
return get_Method()->is_ignored_by_security_stack_walk(); |
6235d2c7549f
7198429: need checked categorization of caller-sensitive methods in the JDK
twisti
parents:
15479
diff
changeset
|
955 |
} |
6235d2c7549f
7198429: need checked categorization of caller-sensitive methods in the JDK
twisti
parents:
15479
diff
changeset
|
956 |
|
55105
9ad765641e8f
8223213: Implement fast class initialization checks on x86-64
vlivanov
parents:
54721
diff
changeset
|
957 |
// ------------------------------------------------------------------ |
9ad765641e8f
8223213: Implement fast class initialization checks on x86-64
vlivanov
parents:
54721
diff
changeset
|
958 |
// ciMethod::needs_clinit_barrier |
9ad765641e8f
8223213: Implement fast class initialization checks on x86-64
vlivanov
parents:
54721
diff
changeset
|
959 |
// |
9ad765641e8f
8223213: Implement fast class initialization checks on x86-64
vlivanov
parents:
54721
diff
changeset
|
960 |
bool ciMethod::needs_clinit_barrier() const { |
9ad765641e8f
8223213: Implement fast class initialization checks on x86-64
vlivanov
parents:
54721
diff
changeset
|
961 |
check_is_loaded(); |
9ad765641e8f
8223213: Implement fast class initialization checks on x86-64
vlivanov
parents:
54721
diff
changeset
|
962 |
return is_static() && !holder()->is_initialized(); |
9ad765641e8f
8223213: Implement fast class initialization checks on x86-64
vlivanov
parents:
54721
diff
changeset
|
963 |
} |
16617
6235d2c7549f
7198429: need checked categorization of caller-sensitive methods in the JDK
twisti
parents:
15479
diff
changeset
|
964 |
|
1 | 965 |
// ------------------------------------------------------------------ |
2534 | 966 |
// invokedynamic support |
5687 | 967 |
|
968 |
// ------------------------------------------------------------------ |
|
13391
30245956af37
7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents:
13282
diff
changeset
|
969 |
// ciMethod::is_method_handle_intrinsic |
2534 | 970 |
// |
13391
30245956af37
7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents:
13282
diff
changeset
|
971 |
// Return true if the method is an instance of the JVM-generated |
30245956af37
7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents:
13282
diff
changeset
|
972 |
// signature-polymorphic MethodHandle methods, _invokeBasic, _linkToVirtual, etc. |
30245956af37
7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents:
13282
diff
changeset
|
973 |
bool ciMethod::is_method_handle_intrinsic() const { |
30245956af37
7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents:
13282
diff
changeset
|
974 |
vmIntrinsics::ID iid = _intrinsic_id; // do not check if loaded |
30245956af37
7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents:
13282
diff
changeset
|
975 |
return (MethodHandles::is_signature_polymorphic(iid) && |
30245956af37
7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents:
13282
diff
changeset
|
976 |
MethodHandles::is_signature_polymorphic_intrinsic(iid)); |
2534 | 977 |
} |
978 |
||
5687 | 979 |
// ------------------------------------------------------------------ |
13391
30245956af37
7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents:
13282
diff
changeset
|
980 |
// ciMethod::is_compiled_lambda_form |
5687 | 981 |
// |
982 |
// Return true if the method is a generated MethodHandle adapter. |
|
13391
30245956af37
7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents:
13282
diff
changeset
|
983 |
// These are built by Java code. |
30245956af37
7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents:
13282
diff
changeset
|
984 |
bool ciMethod::is_compiled_lambda_form() const { |
30245956af37
7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents:
13282
diff
changeset
|
985 |
vmIntrinsics::ID iid = _intrinsic_id; // do not check if loaded |
30245956af37
7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents:
13282
diff
changeset
|
986 |
return iid == vmIntrinsics::_compiledLambdaForm; |
4581
e89fbd1bcb3d
6914206: change way of permission checking for generated MethodHandle adapters
twisti
parents:
4566
diff
changeset
|
987 |
} |
e89fbd1bcb3d
6914206: change way of permission checking for generated MethodHandle adapters
twisti
parents:
4566
diff
changeset
|
988 |
|
13391
30245956af37
7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents:
13282
diff
changeset
|
989 |
// ------------------------------------------------------------------ |
44738 | 990 |
// ciMethod::is_object_initializer |
991 |
// |
|
992 |
bool ciMethod::is_object_initializer() const { |
|
993 |
return name() == ciSymbol::object_initializer_name(); |
|
994 |
} |
|
995 |
||
996 |
// ------------------------------------------------------------------ |
|
13391
30245956af37
7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents:
13282
diff
changeset
|
997 |
// ciMethod::has_member_arg |
30245956af37
7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents:
13282
diff
changeset
|
998 |
// |
30245956af37
7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents:
13282
diff
changeset
|
999 |
// Return true if the method is a linker intrinsic like _linkToVirtual. |
30245956af37
7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents:
13282
diff
changeset
|
1000 |
// These are built by the JVM. |
30245956af37
7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents:
13282
diff
changeset
|
1001 |
bool ciMethod::has_member_arg() const { |
30245956af37
7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents:
13282
diff
changeset
|
1002 |
vmIntrinsics::ID iid = _intrinsic_id; // do not check if loaded |
30245956af37
7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents:
13282
diff
changeset
|
1003 |
return (MethodHandles::is_signature_polymorphic(iid) && |
30245956af37
7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents:
13282
diff
changeset
|
1004 |
MethodHandles::has_member_arg(iid)); |
2534 | 1005 |
} |
1006 |
||
1007 |
// ------------------------------------------------------------------ |
|
7432
f06f1253c317
7003554: (tiered) assert(is_null_object() || handle() != NULL) failed: cannot embed null pointer
iveresov
parents:
7397
diff
changeset
|
1008 |
// ciMethod::ensure_method_data |
1 | 1009 |
// |
13728
882756847a04
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
13522
diff
changeset
|
1010 |
// Generate new MethodData* objects at compile time. |
7432
f06f1253c317
7003554: (tiered) assert(is_null_object() || handle() != NULL) failed: cannot embed null pointer
iveresov
parents:
7397
diff
changeset
|
1011 |
// Return true if allocation was successful or no MDO is required. |
46727
6e4a84748e2c
8183039: Re-examine methodHandle methods uninlined by 8144256
coleenp
parents:
46630
diff
changeset
|
1012 |
bool ciMethod::ensure_method_data(const methodHandle& h_m) { |
1 | 1013 |
EXCEPTION_CONTEXT; |
20695
4f5a5e95090b
8025566: EXCEPTION_ACCESS_VIOLATION in compiled by C1 String.valueOf method
twisti
parents:
20692
diff
changeset
|
1014 |
if (is_native() || is_abstract() || h_m()->is_accessor()) { |
4f5a5e95090b
8025566: EXCEPTION_ACCESS_VIOLATION in compiled by C1 String.valueOf method
twisti
parents:
20692
diff
changeset
|
1015 |
return true; |
4f5a5e95090b
8025566: EXCEPTION_ACCESS_VIOLATION in compiled by C1 String.valueOf method
twisti
parents:
20692
diff
changeset
|
1016 |
} |
1 | 1017 |
if (h_m()->method_data() == NULL) { |
13728
882756847a04
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
13522
diff
changeset
|
1018 |
Method::build_interpreter_method_data(h_m, THREAD); |
1 | 1019 |
if (HAS_PENDING_EXCEPTION) { |
1020 |
CLEAR_PENDING_EXCEPTION; |
|
1021 |
} |
|
1022 |
} |
|
1023 |
if (h_m()->method_data() != NULL) { |
|
13728
882756847a04
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
13522
diff
changeset
|
1024 |
_method_data = CURRENT_ENV->get_method_data(h_m()->method_data()); |
1 | 1025 |
_method_data->load_data(); |
7432
f06f1253c317
7003554: (tiered) assert(is_null_object() || handle() != NULL) failed: cannot embed null pointer
iveresov
parents:
7397
diff
changeset
|
1026 |
return true; |
1 | 1027 |
} else { |
1028 |
_method_data = CURRENT_ENV->get_empty_methodData(); |
|
7432
f06f1253c317
7003554: (tiered) assert(is_null_object() || handle() != NULL) failed: cannot embed null pointer
iveresov
parents:
7397
diff
changeset
|
1029 |
return false; |
1 | 1030 |
} |
1031 |
} |
|
1032 |
||
1033 |
// public, retroactive version |
|
7432
f06f1253c317
7003554: (tiered) assert(is_null_object() || handle() != NULL) failed: cannot embed null pointer
iveresov
parents:
7397
diff
changeset
|
1034 |
bool ciMethod::ensure_method_data() { |
f06f1253c317
7003554: (tiered) assert(is_null_object() || handle() != NULL) failed: cannot embed null pointer
iveresov
parents:
7397
diff
changeset
|
1035 |
bool result = true; |
1 | 1036 |
if (_method_data == NULL || _method_data->is_empty()) { |
1037 |
GUARDED_VM_ENTRY({ |
|
59056
15936b142f86
8233913: Remove implicit conversion from Method* to methodHandle
coleenp
parents:
58273
diff
changeset
|
1038 |
methodHandle mh(Thread::current(), get_Method()); |
15936b142f86
8233913: Remove implicit conversion from Method* to methodHandle
coleenp
parents:
58273
diff
changeset
|
1039 |
result = ensure_method_data(mh); |
1 | 1040 |
}); |
1041 |
} |
|
7432
f06f1253c317
7003554: (tiered) assert(is_null_object() || handle() != NULL) failed: cannot embed null pointer
iveresov
parents:
7397
diff
changeset
|
1042 |
return result; |
1 | 1043 |
} |
1044 |
||
1045 |
||
1046 |
// ------------------------------------------------------------------ |
|
1047 |
// ciMethod::method_data |
|
1048 |
// |
|
1049 |
ciMethodData* ciMethod::method_data() { |
|
1050 |
if (_method_data != NULL) { |
|
1051 |
return _method_data; |
|
1052 |
} |
|
1053 |
VM_ENTRY_MARK; |
|
1054 |
ciEnv* env = CURRENT_ENV; |
|
1055 |
Thread* my_thread = JavaThread::current(); |
|
13728
882756847a04
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
13522
diff
changeset
|
1056 |
methodHandle h_m(my_thread, get_Method()); |
1 | 1057 |
|
1058 |
if (h_m()->method_data() != NULL) { |
|
13728
882756847a04
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
13522
diff
changeset
|
1059 |
_method_data = CURRENT_ENV->get_method_data(h_m()->method_data()); |
1 | 1060 |
_method_data->load_data(); |
1061 |
} else { |
|
1062 |
_method_data = CURRENT_ENV->get_empty_methodData(); |
|
1063 |
} |
|
1064 |
return _method_data; |
|
1065 |
||
1066 |
} |
|
1067 |
||
7432
f06f1253c317
7003554: (tiered) assert(is_null_object() || handle() != NULL) failed: cannot embed null pointer
iveresov
parents:
7397
diff
changeset
|
1068 |
// ------------------------------------------------------------------ |
f06f1253c317
7003554: (tiered) assert(is_null_object() || handle() != NULL) failed: cannot embed null pointer
iveresov
parents:
7397
diff
changeset
|
1069 |
// ciMethod::method_data_or_null |
f06f1253c317
7003554: (tiered) assert(is_null_object() || handle() != NULL) failed: cannot embed null pointer
iveresov
parents:
7397
diff
changeset
|
1070 |
// Returns a pointer to ciMethodData if MDO exists on the VM side, |
f06f1253c317
7003554: (tiered) assert(is_null_object() || handle() != NULL) failed: cannot embed null pointer
iveresov
parents:
7397
diff
changeset
|
1071 |
// NULL otherwise. |
f06f1253c317
7003554: (tiered) assert(is_null_object() || handle() != NULL) failed: cannot embed null pointer
iveresov
parents:
7397
diff
changeset
|
1072 |
ciMethodData* ciMethod::method_data_or_null() { |
f06f1253c317
7003554: (tiered) assert(is_null_object() || handle() != NULL) failed: cannot embed null pointer
iveresov
parents:
7397
diff
changeset
|
1073 |
ciMethodData *md = method_data(); |
20695
4f5a5e95090b
8025566: EXCEPTION_ACCESS_VIOLATION in compiled by C1 String.valueOf method
twisti
parents:
20692
diff
changeset
|
1074 |
if (md->is_empty()) { |
4f5a5e95090b
8025566: EXCEPTION_ACCESS_VIOLATION in compiled by C1 String.valueOf method
twisti
parents:
20692
diff
changeset
|
1075 |
return NULL; |
4f5a5e95090b
8025566: EXCEPTION_ACCESS_VIOLATION in compiled by C1 String.valueOf method
twisti
parents:
20692
diff
changeset
|
1076 |
} |
7432
f06f1253c317
7003554: (tiered) assert(is_null_object() || handle() != NULL) failed: cannot embed null pointer
iveresov
parents:
7397
diff
changeset
|
1077 |
return md; |
f06f1253c317
7003554: (tiered) assert(is_null_object() || handle() != NULL) failed: cannot embed null pointer
iveresov
parents:
7397
diff
changeset
|
1078 |
} |
1 | 1079 |
|
1080 |
// ------------------------------------------------------------------ |
|
17000
11bf92e571a2
8010862: The Method counter fields used for profiling can be allocated lazily.
jiangli
parents:
16617
diff
changeset
|
1081 |
// ciMethod::ensure_method_counters |
11bf92e571a2
8010862: The Method counter fields used for profiling can be allocated lazily.
jiangli
parents:
16617
diff
changeset
|
1082 |
// |
20695
4f5a5e95090b
8025566: EXCEPTION_ACCESS_VIOLATION in compiled by C1 String.valueOf method
twisti
parents:
20692
diff
changeset
|
1083 |
MethodCounters* ciMethod::ensure_method_counters() { |
17000
11bf92e571a2
8010862: The Method counter fields used for profiling can be allocated lazily.
jiangli
parents:
16617
diff
changeset
|
1084 |
check_is_loaded(); |
11bf92e571a2
8010862: The Method counter fields used for profiling can be allocated lazily.
jiangli
parents:
16617
diff
changeset
|
1085 |
VM_ENTRY_MARK; |
11bf92e571a2
8010862: The Method counter fields used for profiling can be allocated lazily.
jiangli
parents:
16617
diff
changeset
|
1086 |
methodHandle mh(THREAD, get_Method()); |
20695
4f5a5e95090b
8025566: EXCEPTION_ACCESS_VIOLATION in compiled by C1 String.valueOf method
twisti
parents:
20692
diff
changeset
|
1087 |
MethodCounters* method_counters = mh->get_method_counters(CHECK_NULL); |
4f5a5e95090b
8025566: EXCEPTION_ACCESS_VIOLATION in compiled by C1 String.valueOf method
twisti
parents:
20692
diff
changeset
|
1088 |
return method_counters; |
17000
11bf92e571a2
8010862: The Method counter fields used for profiling can be allocated lazily.
jiangli
parents:
16617
diff
changeset
|
1089 |
} |
11bf92e571a2
8010862: The Method counter fields used for profiling can be allocated lazily.
jiangli
parents:
16617
diff
changeset
|
1090 |
|
11bf92e571a2
8010862: The Method counter fields used for profiling can be allocated lazily.
jiangli
parents:
16617
diff
changeset
|
1091 |
// ------------------------------------------------------------------ |
1 | 1092 |
// ciMethod::has_option |
1093 |
// |
|
1094 |
bool ciMethod::has_option(const char* option) { |
|
1095 |
check_is_loaded(); |
|
1096 |
VM_ENTRY_MARK; |
|
13728
882756847a04
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
13522
diff
changeset
|
1097 |
methodHandle mh(THREAD, get_Method()); |
1 | 1098 |
return CompilerOracle::has_option_string(mh, option); |
1099 |
} |
|
1100 |
||
1101 |
// ------------------------------------------------------------------ |
|
26433 | 1102 |
// ciMethod::has_option_value |
1103 |
// |
|
33451
0712796e4039
8137167: JEP165: Compiler Control: Implementation task
neliasso
parents:
33129
diff
changeset
|
1104 |
bool ciMethod::has_option_value(const char* option, double& value) { |
26433 | 1105 |
check_is_loaded(); |
1106 |
VM_ENTRY_MARK; |
|
1107 |
methodHandle mh(THREAD, get_Method()); |
|
1108 |
return CompilerOracle::has_option_value(mh, option, value); |
|
1109 |
} |
|
1110 |
// ------------------------------------------------------------------ |
|
1 | 1111 |
// ciMethod::can_be_compiled |
1112 |
// |
|
1113 |
// Have previous compilations of this method succeeded? |
|
1114 |
bool ciMethod::can_be_compiled() { |
|
1115 |
check_is_loaded(); |
|
6453 | 1116 |
ciEnv* env = CURRENT_ENV; |
1117 |
if (is_c1_compile(env->comp_level())) { |
|
1118 |
return _is_c1_compilable; |
|
1119 |
} |
|
1120 |
return _is_c2_compilable; |
|
1 | 1121 |
} |
1122 |
||
1123 |
// ------------------------------------------------------------------ |
|
1124 |
// ciMethod::set_not_compilable |
|
1125 |
// |
|
1126 |
// Tell the VM that this method cannot be compiled at all. |
|
15479 | 1127 |
void ciMethod::set_not_compilable(const char* reason) { |
1 | 1128 |
check_is_loaded(); |
1129 |
VM_ENTRY_MARK; |
|
6453 | 1130 |
ciEnv* env = CURRENT_ENV; |
1131 |
if (is_c1_compile(env->comp_level())) { |
|
1132 |
_is_c1_compilable = false; |
|
1133 |
} else { |
|
1134 |
_is_c2_compilable = false; |
|
1135 |
} |
|
55206 | 1136 |
get_Method()->set_not_compilable(reason, env->comp_level()); |
1 | 1137 |
} |
1138 |
||
1139 |
// ------------------------------------------------------------------ |
|
1140 |
// ciMethod::can_be_osr_compiled |
|
1141 |
// |
|
1142 |
// Have previous compilations of this method succeeded? |
|
1143 |
// |
|
1144 |
// Implementation note: the VM does not currently keep track |
|
1145 |
// of failed OSR compilations per bci. The entry_bci parameter |
|
1146 |
// is currently unused. |
|
1147 |
bool ciMethod::can_be_osr_compiled(int entry_bci) { |
|
1148 |
check_is_loaded(); |
|
1149 |
VM_ENTRY_MARK; |
|
6453 | 1150 |
ciEnv* env = CURRENT_ENV; |
13728
882756847a04
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
13522
diff
changeset
|
1151 |
return !get_Method()->is_not_osr_compilable(env->comp_level()); |
1 | 1152 |
} |
1153 |
||
1154 |
// ------------------------------------------------------------------ |
|
1155 |
// ciMethod::has_compiled_code |
|
1156 |
bool ciMethod::has_compiled_code() { |
|
14477
95e66ea71f71
6830717: replay of compilations would help with debugging
minqi
parents:
14391
diff
changeset
|
1157 |
return instructions_size() > 0; |
1 | 1158 |
} |
1159 |
||
6453 | 1160 |
int ciMethod::comp_level() { |
1161 |
check_is_loaded(); |
|
1162 |
VM_ENTRY_MARK; |
|
38133
78b95467b9f1
8151956: Support non-continuous CodeBlobs in HotSpot
rbackman
parents:
35071
diff
changeset
|
1163 |
CompiledMethod* nm = get_Method()->code(); |
6453 | 1164 |
if (nm != NULL) return nm->comp_level(); |
1165 |
return 0; |
|
1166 |
} |
|
1167 |
||
10014
a5c2141ee857
7057120: Tiered: Allow C1 to inline methods with loops
iveresov
parents:
10008
diff
changeset
|
1168 |
int ciMethod::highest_osr_comp_level() { |
a5c2141ee857
7057120: Tiered: Allow C1 to inline methods with loops
iveresov
parents:
10008
diff
changeset
|
1169 |
check_is_loaded(); |
a5c2141ee857
7057120: Tiered: Allow C1 to inline methods with loops
iveresov
parents:
10008
diff
changeset
|
1170 |
VM_ENTRY_MARK; |
13728
882756847a04
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
13522
diff
changeset
|
1171 |
return get_Method()->highest_osr_comp_level(); |
10014
a5c2141ee857
7057120: Tiered: Allow C1 to inline methods with loops
iveresov
parents:
10008
diff
changeset
|
1172 |
} |
a5c2141ee857
7057120: Tiered: Allow C1 to inline methods with loops
iveresov
parents:
10008
diff
changeset
|
1173 |
|
1 | 1174 |
// ------------------------------------------------------------------ |
10506
575ad9bccff5
7078382: JSR 292: don't count method handle adapters against inlining budgets
twisti
parents:
10014
diff
changeset
|
1175 |
// ciMethod::code_size_for_inlining |
575ad9bccff5
7078382: JSR 292: don't count method handle adapters against inlining budgets
twisti
parents:
10014
diff
changeset
|
1176 |
// |
13391
30245956af37
7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents:
13282
diff
changeset
|
1177 |
// Code size for inlining decisions. This method returns a code |
30245956af37
7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents:
13282
diff
changeset
|
1178 |
// size of 1 for methods which has the ForceInline annotation. |
10506
575ad9bccff5
7078382: JSR 292: don't count method handle adapters against inlining budgets
twisti
parents:
10014
diff
changeset
|
1179 |
int ciMethod::code_size_for_inlining() { |
575ad9bccff5
7078382: JSR 292: don't count method handle adapters against inlining budgets
twisti
parents:
10014
diff
changeset
|
1180 |
check_is_loaded(); |
13728
882756847a04
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
13522
diff
changeset
|
1181 |
if (get_Method()->force_inline()) { |
13391
30245956af37
7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents:
13282
diff
changeset
|
1182 |
return 1; |
10506
575ad9bccff5
7078382: JSR 292: don't count method handle adapters against inlining budgets
twisti
parents:
10014
diff
changeset
|
1183 |
} |
575ad9bccff5
7078382: JSR 292: don't count method handle adapters against inlining budgets
twisti
parents:
10014
diff
changeset
|
1184 |
return code_size(); |
575ad9bccff5
7078382: JSR 292: don't count method handle adapters against inlining budgets
twisti
parents:
10014
diff
changeset
|
1185 |
} |
575ad9bccff5
7078382: JSR 292: don't count method handle adapters against inlining budgets
twisti
parents:
10014
diff
changeset
|
1186 |
|
575ad9bccff5
7078382: JSR 292: don't count method handle adapters against inlining budgets
twisti
parents:
10014
diff
changeset
|
1187 |
// ------------------------------------------------------------------ |
1 | 1188 |
// ciMethod::instructions_size |
6418 | 1189 |
// |
1190 |
// This is a rough metric for "fat" methods, compared before inlining |
|
1191 |
// with InlineSmallCode. The CodeBlob::code_size accessor includes |
|
1192 |
// junk like exception handler, stubs, and constant table, which are |
|
1193 |
// not highly relevant to an inlined method. So we use the more |
|
1194 |
// specific accessor nmethod::insts_size. |
|
14477
95e66ea71f71
6830717: replay of compilations would help with debugging
minqi
parents:
14391
diff
changeset
|
1195 |
int ciMethod::instructions_size() { |
95e66ea71f71
6830717: replay of compilations would help with debugging
minqi
parents:
14391
diff
changeset
|
1196 |
if (_instructions_size == -1) { |
95e66ea71f71
6830717: replay of compilations would help with debugging
minqi
parents:
14391
diff
changeset
|
1197 |
GUARDED_VM_ENTRY( |
38133
78b95467b9f1
8151956: Support non-continuous CodeBlobs in HotSpot
rbackman
parents:
35071
diff
changeset
|
1198 |
CompiledMethod* code = get_Method()->code(); |
14477
95e66ea71f71
6830717: replay of compilations would help with debugging
minqi
parents:
14391
diff
changeset
|
1199 |
if (code != NULL && (code->comp_level() == CompLevel_full_optimization)) { |
95e66ea71f71
6830717: replay of compilations would help with debugging
minqi
parents:
14391
diff
changeset
|
1200 |
_instructions_size = code->insts_end() - code->verified_entry_point(); |
95e66ea71f71
6830717: replay of compilations would help with debugging
minqi
parents:
14391
diff
changeset
|
1201 |
} else { |
95e66ea71f71
6830717: replay of compilations would help with debugging
minqi
parents:
14391
diff
changeset
|
1202 |
_instructions_size = 0; |
95e66ea71f71
6830717: replay of compilations would help with debugging
minqi
parents:
14391
diff
changeset
|
1203 |
} |
95e66ea71f71
6830717: replay of compilations would help with debugging
minqi
parents:
14391
diff
changeset
|
1204 |
); |
95e66ea71f71
6830717: replay of compilations would help with debugging
minqi
parents:
14391
diff
changeset
|
1205 |
} |
95e66ea71f71
6830717: replay of compilations would help with debugging
minqi
parents:
14391
diff
changeset
|
1206 |
return _instructions_size; |
1 | 1207 |
} |
1208 |
||
1209 |
// ------------------------------------------------------------------ |
|
1210 |
// ciMethod::log_nmethod_identity |
|
1211 |
void ciMethod::log_nmethod_identity(xmlStream* log) { |
|
1212 |
GUARDED_VM_ENTRY( |
|
38133
78b95467b9f1
8151956: Support non-continuous CodeBlobs in HotSpot
rbackman
parents:
35071
diff
changeset
|
1213 |
CompiledMethod* code = get_Method()->code(); |
1 | 1214 |
if (code != NULL) { |
1215 |
code->log_identity(log); |
|
1216 |
} |
|
1217 |
) |
|
1218 |
} |
|
1219 |
||
1220 |
// ------------------------------------------------------------------ |
|
1221 |
// ciMethod::is_not_reached |
|
1222 |
bool ciMethod::is_not_reached(int bci) { |
|
1223 |
check_is_loaded(); |
|
1224 |
VM_ENTRY_MARK; |
|
1225 |
return Interpreter::is_not_reached( |
|
13728
882756847a04
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
13522
diff
changeset
|
1226 |
methodHandle(THREAD, get_Method()), bci); |
1 | 1227 |
} |
1228 |
||
1229 |
// ------------------------------------------------------------------ |
|
1230 |
// ciMethod::was_never_executed |
|
1231 |
bool ciMethod::was_executed_more_than(int times) { |
|
1232 |
VM_ENTRY_MARK; |
|
13728
882756847a04
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
13522
diff
changeset
|
1233 |
return get_Method()->was_executed_more_than(times); |
1 | 1234 |
} |
1235 |
||
1236 |
// ------------------------------------------------------------------ |
|
1237 |
// ciMethod::has_unloaded_classes_in_signature |
|
1238 |
bool ciMethod::has_unloaded_classes_in_signature() { |
|
1239 |
VM_ENTRY_MARK; |
|
1240 |
{ |
|
1241 |
EXCEPTION_MARK; |
|
13728
882756847a04
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
13522
diff
changeset
|
1242 |
methodHandle m(THREAD, get_Method()); |
882756847a04
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
13522
diff
changeset
|
1243 |
bool has_unloaded = Method::has_unloaded_classes_in_signature(m, (JavaThread *)THREAD); |
1 | 1244 |
if( HAS_PENDING_EXCEPTION ) { |
1245 |
CLEAR_PENDING_EXCEPTION; |
|
1246 |
return true; // Declare that we may have unloaded classes |
|
1247 |
} |
|
1248 |
return has_unloaded; |
|
1249 |
} |
|
1250 |
} |
|
1251 |
||
1252 |
// ------------------------------------------------------------------ |
|
1253 |
// ciMethod::is_klass_loaded |
|
1254 |
bool ciMethod::is_klass_loaded(int refinfo_index, bool must_be_resolved) const { |
|
1255 |
VM_ENTRY_MARK; |
|
13728
882756847a04
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
13522
diff
changeset
|
1256 |
return get_Method()->is_klass_loaded(refinfo_index, must_be_resolved); |
1 | 1257 |
} |
1258 |
||
1259 |
// ------------------------------------------------------------------ |
|
1260 |
// ciMethod::check_call |
|
1261 |
bool ciMethod::check_call(int refinfo_index, bool is_static) const { |
|
20017
81eba62e9048
8014013: CallInfo structure no longer accurately reports the result of a LinkResolver operation
drchase
parents:
17383
diff
changeset
|
1262 |
// This method is used only in C2 from InlineTree::ok_to_inline, |
52934
8deeb7bba516
8214917: CTW testlibrary shouldn't ignore errors raised by the library itself
iignatyev
parents:
51333
diff
changeset
|
1263 |
// and is only used under -Xcomp. |
20017
81eba62e9048
8014013: CallInfo structure no longer accurately reports the result of a LinkResolver operation
drchase
parents:
17383
diff
changeset
|
1264 |
// It appears to fail when applied to an invokeinterface call site. |
81eba62e9048
8014013: CallInfo structure no longer accurately reports the result of a LinkResolver operation
drchase
parents:
17383
diff
changeset
|
1265 |
// FIXME: Remove this method and resolve_method_statically; refactor to use the other LinkResolver entry points. |
1 | 1266 |
VM_ENTRY_MARK; |
1267 |
{ |
|
1268 |
EXCEPTION_MARK; |
|
1269 |
HandleMark hm(THREAD); |
|
13728
882756847a04
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
13522
diff
changeset
|
1270 |
constantPoolHandle pool (THREAD, get_Method()->constants()); |
13391
30245956af37
7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents:
13282
diff
changeset
|
1271 |
Bytecodes::Code code = (is_static ? Bytecodes::_invokestatic : Bytecodes::_invokevirtual); |
59056
15936b142f86
8233913: Remove implicit conversion from Method* to methodHandle
coleenp
parents:
58273
diff
changeset
|
1272 |
Method* spec_method = LinkResolver::resolve_method_statically(code, pool, refinfo_index, THREAD); |
1 | 1273 |
if (HAS_PENDING_EXCEPTION) { |
1274 |
CLEAR_PENDING_EXCEPTION; |
|
1275 |
return false; |
|
1276 |
} else { |
|
1277 |
return (spec_method->is_static() == is_static); |
|
1278 |
} |
|
1279 |
} |
|
1280 |
return false; |
|
1281 |
} |
|
1282 |
||
1283 |
// ------------------------------------------------------------------ |
|
24442
4d4ae31dea26
8032463: VirtualDispatch test timeout with DeoptimizeALot
iveresov
parents:
24018
diff
changeset
|
1284 |
// ciMethod::profile_aging |
4d4ae31dea26
8032463: VirtualDispatch test timeout with DeoptimizeALot
iveresov
parents:
24018
diff
changeset
|
1285 |
// |
4d4ae31dea26
8032463: VirtualDispatch test timeout with DeoptimizeALot
iveresov
parents:
24018
diff
changeset
|
1286 |
// Should the method be compiled with an age counter? |
4d4ae31dea26
8032463: VirtualDispatch test timeout with DeoptimizeALot
iveresov
parents:
24018
diff
changeset
|
1287 |
bool ciMethod::profile_aging() const { |
4d4ae31dea26
8032463: VirtualDispatch test timeout with DeoptimizeALot
iveresov
parents:
24018
diff
changeset
|
1288 |
return UseCodeAging && (!MethodCounters::is_nmethod_hot(nmethod_age()) && |
4d4ae31dea26
8032463: VirtualDispatch test timeout with DeoptimizeALot
iveresov
parents:
24018
diff
changeset
|
1289 |
!MethodCounters::is_nmethod_age_unset(nmethod_age())); |
4d4ae31dea26
8032463: VirtualDispatch test timeout with DeoptimizeALot
iveresov
parents:
24018
diff
changeset
|
1290 |
} |
4d4ae31dea26
8032463: VirtualDispatch test timeout with DeoptimizeALot
iveresov
parents:
24018
diff
changeset
|
1291 |
// ------------------------------------------------------------------ |
1 | 1292 |
// ciMethod::print_codes |
1293 |
// |
|
1294 |
// Print the bytecodes for this method. |
|
1295 |
void ciMethod::print_codes_on(outputStream* st) { |
|
1296 |
check_is_loaded(); |
|
13728
882756847a04
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
13522
diff
changeset
|
1297 |
GUARDED_VM_ENTRY(get_Method()->print_codes_on(st);) |
1 | 1298 |
} |
1299 |
||
1300 |
||
1301 |
#define FETCH_FLAG_FROM_VM(flag_accessor) { \ |
|
1302 |
check_is_loaded(); \ |
|
1303 |
VM_ENTRY_MARK; \ |
|
13728
882756847a04
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
13522
diff
changeset
|
1304 |
return get_Method()->flag_accessor(); \ |
1 | 1305 |
} |
1306 |
||
1307 |
bool ciMethod::is_empty_method() const { FETCH_FLAG_FROM_VM(is_empty_method); } |
|
1308 |
bool ciMethod::is_vanilla_constructor() const { FETCH_FLAG_FROM_VM(is_vanilla_constructor); } |
|
1309 |
bool ciMethod::has_loops () const { FETCH_FLAG_FROM_VM(has_loops); } |
|
1310 |
bool ciMethod::has_jsrs () const { FETCH_FLAG_FROM_VM(has_jsrs); } |
|
34169
b0b7187852b7
8140650: Method::is_accessor should cover getters and setters for all types
shade
parents:
33451
diff
changeset
|
1311 |
bool ciMethod::is_getter () const { FETCH_FLAG_FROM_VM(is_getter); } |
b0b7187852b7
8140650: Method::is_accessor should cover getters and setters for all types
shade
parents:
33451
diff
changeset
|
1312 |
bool ciMethod::is_setter () const { FETCH_FLAG_FROM_VM(is_setter); } |
1 | 1313 |
bool ciMethod::is_accessor () const { FETCH_FLAG_FROM_VM(is_accessor); } |
1314 |
bool ciMethod::is_initializer () const { FETCH_FLAG_FROM_VM(is_initializer); } |
|
1315 |
||
17383 | 1316 |
bool ciMethod::is_boxing_method() const { |
1317 |
if (holder()->is_box_klass()) { |
|
1318 |
switch (intrinsic_id()) { |
|
1319 |
case vmIntrinsics::_Boolean_valueOf: |
|
1320 |
case vmIntrinsics::_Byte_valueOf: |
|
1321 |
case vmIntrinsics::_Character_valueOf: |
|
1322 |
case vmIntrinsics::_Short_valueOf: |
|
1323 |
case vmIntrinsics::_Integer_valueOf: |
|
1324 |
case vmIntrinsics::_Long_valueOf: |
|
1325 |
case vmIntrinsics::_Float_valueOf: |
|
1326 |
case vmIntrinsics::_Double_valueOf: |
|
1327 |
return true; |
|
1328 |
default: |
|
1329 |
return false; |
|
1330 |
} |
|
1331 |
} |
|
1332 |
return false; |
|
1333 |
} |
|
1334 |
||
1335 |
bool ciMethod::is_unboxing_method() const { |
|
1336 |
if (holder()->is_box_klass()) { |
|
1337 |
switch (intrinsic_id()) { |
|
1338 |
case vmIntrinsics::_booleanValue: |
|
1339 |
case vmIntrinsics::_byteValue: |
|
1340 |
case vmIntrinsics::_charValue: |
|
1341 |
case vmIntrinsics::_shortValue: |
|
1342 |
case vmIntrinsics::_intValue: |
|
1343 |
case vmIntrinsics::_longValue: |
|
1344 |
case vmIntrinsics::_floatValue: |
|
1345 |
case vmIntrinsics::_doubleValue: |
|
1346 |
return true; |
|
1347 |
default: |
|
1348 |
return false; |
|
1349 |
} |
|
1350 |
} |
|
1351 |
return false; |
|
1352 |
} |
|
1353 |
||
1 | 1354 |
BCEscapeAnalyzer *ciMethod::get_bcea() { |
5928
f6e69b46e9e3
6968368: SIGSEGV in the BCEscapeAnalyzer::copy_dependencies
kvn
parents:
5702
diff
changeset
|
1355 |
#ifdef COMPILER2 |
1 | 1356 |
if (_bcea == NULL) { |
1357 |
_bcea = new (CURRENT_ENV->arena()) BCEscapeAnalyzer(this, NULL); |
|
1358 |
} |
|
1359 |
return _bcea; |
|
5928
f6e69b46e9e3
6968368: SIGSEGV in the BCEscapeAnalyzer::copy_dependencies
kvn
parents:
5702
diff
changeset
|
1360 |
#else // COMPILER2 |
f6e69b46e9e3
6968368: SIGSEGV in the BCEscapeAnalyzer::copy_dependencies
kvn
parents:
5702
diff
changeset
|
1361 |
ShouldNotReachHere(); |
f6e69b46e9e3
6968368: SIGSEGV in the BCEscapeAnalyzer::copy_dependencies
kvn
parents:
5702
diff
changeset
|
1362 |
return NULL; |
f6e69b46e9e3
6968368: SIGSEGV in the BCEscapeAnalyzer::copy_dependencies
kvn
parents:
5702
diff
changeset
|
1363 |
#endif // COMPILER2 |
1 | 1364 |
} |
1365 |
||
1366 |
ciMethodBlocks *ciMethod::get_method_blocks() { |
|
1367 |
Arena *arena = CURRENT_ENV->arena(); |
|
1368 |
if (_method_blocks == NULL) { |
|
1369 |
_method_blocks = new (arena) ciMethodBlocks(arena, this); |
|
1370 |
} |
|
1371 |
return _method_blocks; |
|
1372 |
} |
|
1373 |
||
1374 |
#undef FETCH_FLAG_FROM_VM |
|
1375 |
||
22243 | 1376 |
void ciMethod::dump_name_as_ascii(outputStream* st) { |
1377 |
Method* method = get_Method(); |
|
1378 |
st->print("%s %s %s", |
|
1379 |
method->klass_name()->as_quoted_ascii(), |
|
1380 |
method->name()->as_quoted_ascii(), |
|
1381 |
method->signature()->as_quoted_ascii()); |
|
1382 |
} |
|
1383 |
||
14477
95e66ea71f71
6830717: replay of compilations would help with debugging
minqi
parents:
14391
diff
changeset
|
1384 |
void ciMethod::dump_replay_data(outputStream* st) { |
15471
41f75023e6a6
8006410: allocating without ResourceMark when CompileCommand was specified
vlivanov
parents:
14621
diff
changeset
|
1385 |
ResourceMark rm; |
14477
95e66ea71f71
6830717: replay of compilations would help with debugging
minqi
parents:
14391
diff
changeset
|
1386 |
Method* method = get_Method(); |
17000
11bf92e571a2
8010862: The Method counter fields used for profiling can be allocated lazily.
jiangli
parents:
16617
diff
changeset
|
1387 |
MethodCounters* mcs = method->method_counters(); |
22243 | 1388 |
st->print("ciMethod "); |
1389 |
dump_name_as_ascii(st); |
|
1390 |
st->print_cr(" %d %d %d %d %d", |
|
17000
11bf92e571a2
8010862: The Method counter fields used for profiling can be allocated lazily.
jiangli
parents:
16617
diff
changeset
|
1391 |
mcs == NULL ? 0 : mcs->invocation_counter()->raw_counter(), |
11bf92e571a2
8010862: The Method counter fields used for profiling can be allocated lazily.
jiangli
parents:
16617
diff
changeset
|
1392 |
mcs == NULL ? 0 : mcs->backedge_counter()->raw_counter(), |
14477
95e66ea71f71
6830717: replay of compilations would help with debugging
minqi
parents:
14391
diff
changeset
|
1393 |
interpreter_invocation_count(), |
95e66ea71f71
6830717: replay of compilations would help with debugging
minqi
parents:
14391
diff
changeset
|
1394 |
interpreter_throwout_count(), |
95e66ea71f71
6830717: replay of compilations would help with debugging
minqi
parents:
14391
diff
changeset
|
1395 |
_instructions_size); |
95e66ea71f71
6830717: replay of compilations would help with debugging
minqi
parents:
14391
diff
changeset
|
1396 |
} |
1 | 1397 |
|
1398 |
// ------------------------------------------------------------------ |
|
1399 |
// ciMethod::print_codes |
|
1400 |
// |
|
1401 |
// Print a range of the bytecodes for this method. |
|
1402 |
void ciMethod::print_codes_on(int from, int to, outputStream* st) { |
|
1403 |
check_is_loaded(); |
|
13728
882756847a04
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
13522
diff
changeset
|
1404 |
GUARDED_VM_ENTRY(get_Method()->print_codes_on(from, to, st);) |
1 | 1405 |
} |
1406 |
||
1407 |
// ------------------------------------------------------------------ |
|
1408 |
// ciMethod::print_name |
|
1409 |
// |
|
1410 |
// Print the name of this method, including signature and some flags. |
|
1411 |
void ciMethod::print_name(outputStream* st) { |
|
1412 |
check_is_loaded(); |
|
13728
882756847a04
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
13522
diff
changeset
|
1413 |
GUARDED_VM_ENTRY(get_Method()->print_name(st);) |
1 | 1414 |
} |
1415 |
||
1416 |
// ------------------------------------------------------------------ |
|
1417 |
// ciMethod::print_short_name |
|
1418 |
// |
|
1419 |
// Print the name of this method, without signature. |
|
1420 |
void ciMethod::print_short_name(outputStream* st) { |
|
13391
30245956af37
7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents:
13282
diff
changeset
|
1421 |
if (is_loaded()) { |
13728
882756847a04
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
13522
diff
changeset
|
1422 |
GUARDED_VM_ENTRY(get_Method()->print_short_name(st);); |
13391
30245956af37
7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents:
13282
diff
changeset
|
1423 |
} else { |
30245956af37
7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents:
13282
diff
changeset
|
1424 |
// Fall back if method is not loaded. |
30245956af37
7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents:
13282
diff
changeset
|
1425 |
holder()->print_name_on(st); |
30245956af37
7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents:
13282
diff
changeset
|
1426 |
st->print("::"); |
30245956af37
7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents:
13282
diff
changeset
|
1427 |
name()->print_symbol_on(st); |
30245956af37
7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents:
13282
diff
changeset
|
1428 |
if (WizardMode) |
30245956af37
7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents:
13282
diff
changeset
|
1429 |
signature()->as_symbol()->print_symbol_on(st); |
30245956af37
7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents:
13282
diff
changeset
|
1430 |
} |
1 | 1431 |
} |
1432 |
||
1433 |
// ------------------------------------------------------------------ |
|
1434 |
// ciMethod::print_impl |
|
1435 |
// |
|
1436 |
// Implementation of the print method. |
|
1437 |
void ciMethod::print_impl(outputStream* st) { |
|
13728
882756847a04
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
13522
diff
changeset
|
1438 |
ciMetadata::print_impl(st); |
1 | 1439 |
st->print(" name="); |
1440 |
name()->print_symbol_on(st); |
|
1441 |
st->print(" holder="); |
|
1442 |
holder()->print_name_on(st); |
|
1443 |
st->print(" signature="); |
|
1444 |
signature()->as_symbol()->print_symbol_on(st); |
|
1445 |
if (is_loaded()) { |
|
13522
5ad4627e792a
7192406: JSR 292: C2 needs exact return type information for invokedynamic and invokehandle call sites
twisti
parents:
13391
diff
changeset
|
1446 |
st->print(" loaded=true"); |
5ad4627e792a
7192406: JSR 292: C2 needs exact return type information for invokedynamic and invokehandle call sites
twisti
parents:
13391
diff
changeset
|
1447 |
st->print(" arg_size=%d", arg_size()); |
5ad4627e792a
7192406: JSR 292: C2 needs exact return type information for invokedynamic and invokehandle call sites
twisti
parents:
13391
diff
changeset
|
1448 |
st->print(" flags="); |
1 | 1449 |
flags().print_member_flags(st); |
1450 |
} else { |
|
1451 |
st->print(" loaded=false"); |
|
1452 |
} |
|
1453 |
} |
|
29582 | 1454 |
|
43947
a52ee13998f3
8174721: C1: Inlining through MH invokers/linkers in unreachable code is unsafe
vlivanov
parents:
40664
diff
changeset
|
1455 |
// ------------------------------------------------------------------ |
a52ee13998f3
8174721: C1: Inlining through MH invokers/linkers in unreachable code is unsafe
vlivanov
parents:
40664
diff
changeset
|
1456 |
|
a52ee13998f3
8174721: C1: Inlining through MH invokers/linkers in unreachable code is unsafe
vlivanov
parents:
40664
diff
changeset
|
1457 |
static BasicType erase_to_word_type(BasicType bt) { |
58273
08a5148e7c4e
8230505: Replace JVM type comparisons to T_OBJECT and T_ARRAY with call to is_reference_type
lfoltan
parents:
57745
diff
changeset
|
1458 |
if (is_subword_type(bt)) return T_INT; |
08a5148e7c4e
8230505: Replace JVM type comparisons to T_OBJECT and T_ARRAY with call to is_reference_type
lfoltan
parents:
57745
diff
changeset
|
1459 |
if (is_reference_type(bt)) return T_OBJECT; |
43947
a52ee13998f3
8174721: C1: Inlining through MH invokers/linkers in unreachable code is unsafe
vlivanov
parents:
40664
diff
changeset
|
1460 |
return bt; |
a52ee13998f3
8174721: C1: Inlining through MH invokers/linkers in unreachable code is unsafe
vlivanov
parents:
40664
diff
changeset
|
1461 |
} |
a52ee13998f3
8174721: C1: Inlining through MH invokers/linkers in unreachable code is unsafe
vlivanov
parents:
40664
diff
changeset
|
1462 |
|
a52ee13998f3
8174721: C1: Inlining through MH invokers/linkers in unreachable code is unsafe
vlivanov
parents:
40664
diff
changeset
|
1463 |
static bool basic_types_match(ciType* t1, ciType* t2) { |
a52ee13998f3
8174721: C1: Inlining through MH invokers/linkers in unreachable code is unsafe
vlivanov
parents:
40664
diff
changeset
|
1464 |
if (t1 == t2) return true; |
a52ee13998f3
8174721: C1: Inlining through MH invokers/linkers in unreachable code is unsafe
vlivanov
parents:
40664
diff
changeset
|
1465 |
return erase_to_word_type(t1->basic_type()) == erase_to_word_type(t2->basic_type()); |
a52ee13998f3
8174721: C1: Inlining through MH invokers/linkers in unreachable code is unsafe
vlivanov
parents:
40664
diff
changeset
|
1466 |
} |
a52ee13998f3
8174721: C1: Inlining through MH invokers/linkers in unreachable code is unsafe
vlivanov
parents:
40664
diff
changeset
|
1467 |
|
a52ee13998f3
8174721: C1: Inlining through MH invokers/linkers in unreachable code is unsafe
vlivanov
parents:
40664
diff
changeset
|
1468 |
bool ciMethod::is_consistent_info(ciMethod* declared_method, ciMethod* resolved_method) { |
a52ee13998f3
8174721: C1: Inlining through MH invokers/linkers in unreachable code is unsafe
vlivanov
parents:
40664
diff
changeset
|
1469 |
bool invoke_through_mh_intrinsic = declared_method->is_method_handle_intrinsic() && |
a52ee13998f3
8174721: C1: Inlining through MH invokers/linkers in unreachable code is unsafe
vlivanov
parents:
40664
diff
changeset
|
1470 |
!resolved_method->is_method_handle_intrinsic(); |
a52ee13998f3
8174721: C1: Inlining through MH invokers/linkers in unreachable code is unsafe
vlivanov
parents:
40664
diff
changeset
|
1471 |
|
a52ee13998f3
8174721: C1: Inlining through MH invokers/linkers in unreachable code is unsafe
vlivanov
parents:
40664
diff
changeset
|
1472 |
if (!invoke_through_mh_intrinsic) { |
a52ee13998f3
8174721: C1: Inlining through MH invokers/linkers in unreachable code is unsafe
vlivanov
parents:
40664
diff
changeset
|
1473 |
// Method name & descriptor should stay the same. |
43951
c23519aaccc1
8174818: bigapps/Weblogic12medrec fails with assert(check_call_consistency(jvms, cg)) failed: inconsistent info
vlivanov
parents:
43947
diff
changeset
|
1474 |
// Signatures may reference unloaded types and thus they may be not strictly equal. |
c23519aaccc1
8174818: bigapps/Weblogic12medrec fails with assert(check_call_consistency(jvms, cg)) failed: inconsistent info
vlivanov
parents:
43947
diff
changeset
|
1475 |
ciSymbol* declared_signature = declared_method->signature()->as_symbol(); |
c23519aaccc1
8174818: bigapps/Weblogic12medrec fails with assert(check_call_consistency(jvms, cg)) failed: inconsistent info
vlivanov
parents:
43947
diff
changeset
|
1476 |
ciSymbol* resolved_signature = resolved_method->signature()->as_symbol(); |
c23519aaccc1
8174818: bigapps/Weblogic12medrec fails with assert(check_call_consistency(jvms, cg)) failed: inconsistent info
vlivanov
parents:
43947
diff
changeset
|
1477 |
|
43947
a52ee13998f3
8174721: C1: Inlining through MH invokers/linkers in unreachable code is unsafe
vlivanov
parents:
40664
diff
changeset
|
1478 |
return (declared_method->name()->equals(resolved_method->name())) && |
43951
c23519aaccc1
8174818: bigapps/Weblogic12medrec fails with assert(check_call_consistency(jvms, cg)) failed: inconsistent info
vlivanov
parents:
43947
diff
changeset
|
1479 |
(declared_signature->equals(resolved_signature)); |
43947
a52ee13998f3
8174721: C1: Inlining through MH invokers/linkers in unreachable code is unsafe
vlivanov
parents:
40664
diff
changeset
|
1480 |
} |
a52ee13998f3
8174721: C1: Inlining through MH invokers/linkers in unreachable code is unsafe
vlivanov
parents:
40664
diff
changeset
|
1481 |
|
a52ee13998f3
8174721: C1: Inlining through MH invokers/linkers in unreachable code is unsafe
vlivanov
parents:
40664
diff
changeset
|
1482 |
ciMethod* linker = declared_method; |
a52ee13998f3
8174721: C1: Inlining through MH invokers/linkers in unreachable code is unsafe
vlivanov
parents:
40664
diff
changeset
|
1483 |
ciMethod* target = resolved_method; |
a52ee13998f3
8174721: C1: Inlining through MH invokers/linkers in unreachable code is unsafe
vlivanov
parents:
40664
diff
changeset
|
1484 |
// Linkers have appendix argument which is not passed to callee. |
a52ee13998f3
8174721: C1: Inlining through MH invokers/linkers in unreachable code is unsafe
vlivanov
parents:
40664
diff
changeset
|
1485 |
int has_appendix = MethodHandles::has_member_arg(linker->intrinsic_id()) ? 1 : 0; |
a52ee13998f3
8174721: C1: Inlining through MH invokers/linkers in unreachable code is unsafe
vlivanov
parents:
40664
diff
changeset
|
1486 |
if (linker->arg_size() != (target->arg_size() + has_appendix)) { |
a52ee13998f3
8174721: C1: Inlining through MH invokers/linkers in unreachable code is unsafe
vlivanov
parents:
40664
diff
changeset
|
1487 |
return false; // argument slot count mismatch |
a52ee13998f3
8174721: C1: Inlining through MH invokers/linkers in unreachable code is unsafe
vlivanov
parents:
40664
diff
changeset
|
1488 |
} |
a52ee13998f3
8174721: C1: Inlining through MH invokers/linkers in unreachable code is unsafe
vlivanov
parents:
40664
diff
changeset
|
1489 |
|
a52ee13998f3
8174721: C1: Inlining through MH invokers/linkers in unreachable code is unsafe
vlivanov
parents:
40664
diff
changeset
|
1490 |
ciSignature* linker_sig = linker->signature(); |
a52ee13998f3
8174721: C1: Inlining through MH invokers/linkers in unreachable code is unsafe
vlivanov
parents:
40664
diff
changeset
|
1491 |
ciSignature* target_sig = target->signature(); |
a52ee13998f3
8174721: C1: Inlining through MH invokers/linkers in unreachable code is unsafe
vlivanov
parents:
40664
diff
changeset
|
1492 |
|
a52ee13998f3
8174721: C1: Inlining through MH invokers/linkers in unreachable code is unsafe
vlivanov
parents:
40664
diff
changeset
|
1493 |
if (linker_sig->count() + (linker->is_static() ? 0 : 1) != |
a52ee13998f3
8174721: C1: Inlining through MH invokers/linkers in unreachable code is unsafe
vlivanov
parents:
40664
diff
changeset
|
1494 |
target_sig->count() + (target->is_static() ? 0 : 1) + has_appendix) { |
a52ee13998f3
8174721: C1: Inlining through MH invokers/linkers in unreachable code is unsafe
vlivanov
parents:
40664
diff
changeset
|
1495 |
return false; // argument count mismatch |
a52ee13998f3
8174721: C1: Inlining through MH invokers/linkers in unreachable code is unsafe
vlivanov
parents:
40664
diff
changeset
|
1496 |
} |
a52ee13998f3
8174721: C1: Inlining through MH invokers/linkers in unreachable code is unsafe
vlivanov
parents:
40664
diff
changeset
|
1497 |
|
a52ee13998f3
8174721: C1: Inlining through MH invokers/linkers in unreachable code is unsafe
vlivanov
parents:
40664
diff
changeset
|
1498 |
int sbase = 0, rbase = 0; |
a52ee13998f3
8174721: C1: Inlining through MH invokers/linkers in unreachable code is unsafe
vlivanov
parents:
40664
diff
changeset
|
1499 |
switch (linker->intrinsic_id()) { |
a52ee13998f3
8174721: C1: Inlining through MH invokers/linkers in unreachable code is unsafe
vlivanov
parents:
40664
diff
changeset
|
1500 |
case vmIntrinsics::_linkToVirtual: |
a52ee13998f3
8174721: C1: Inlining through MH invokers/linkers in unreachable code is unsafe
vlivanov
parents:
40664
diff
changeset
|
1501 |
case vmIntrinsics::_linkToInterface: |
a52ee13998f3
8174721: C1: Inlining through MH invokers/linkers in unreachable code is unsafe
vlivanov
parents:
40664
diff
changeset
|
1502 |
case vmIntrinsics::_linkToSpecial: { |
a52ee13998f3
8174721: C1: Inlining through MH invokers/linkers in unreachable code is unsafe
vlivanov
parents:
40664
diff
changeset
|
1503 |
if (target->is_static()) { |
a52ee13998f3
8174721: C1: Inlining through MH invokers/linkers in unreachable code is unsafe
vlivanov
parents:
40664
diff
changeset
|
1504 |
return false; |
a52ee13998f3
8174721: C1: Inlining through MH invokers/linkers in unreachable code is unsafe
vlivanov
parents:
40664
diff
changeset
|
1505 |
} |
a52ee13998f3
8174721: C1: Inlining through MH invokers/linkers in unreachable code is unsafe
vlivanov
parents:
40664
diff
changeset
|
1506 |
if (linker_sig->type_at(0)->is_primitive_type()) { |
a52ee13998f3
8174721: C1: Inlining through MH invokers/linkers in unreachable code is unsafe
vlivanov
parents:
40664
diff
changeset
|
1507 |
return false; // receiver should be an oop |
a52ee13998f3
8174721: C1: Inlining through MH invokers/linkers in unreachable code is unsafe
vlivanov
parents:
40664
diff
changeset
|
1508 |
} |
a52ee13998f3
8174721: C1: Inlining through MH invokers/linkers in unreachable code is unsafe
vlivanov
parents:
40664
diff
changeset
|
1509 |
sbase = 1; // skip receiver |
a52ee13998f3
8174721: C1: Inlining through MH invokers/linkers in unreachable code is unsafe
vlivanov
parents:
40664
diff
changeset
|
1510 |
break; |
a52ee13998f3
8174721: C1: Inlining through MH invokers/linkers in unreachable code is unsafe
vlivanov
parents:
40664
diff
changeset
|
1511 |
} |
a52ee13998f3
8174721: C1: Inlining through MH invokers/linkers in unreachable code is unsafe
vlivanov
parents:
40664
diff
changeset
|
1512 |
case vmIntrinsics::_linkToStatic: { |
a52ee13998f3
8174721: C1: Inlining through MH invokers/linkers in unreachable code is unsafe
vlivanov
parents:
40664
diff
changeset
|
1513 |
if (!target->is_static()) { |
a52ee13998f3
8174721: C1: Inlining through MH invokers/linkers in unreachable code is unsafe
vlivanov
parents:
40664
diff
changeset
|
1514 |
return false; |
a52ee13998f3
8174721: C1: Inlining through MH invokers/linkers in unreachable code is unsafe
vlivanov
parents:
40664
diff
changeset
|
1515 |
} |
a52ee13998f3
8174721: C1: Inlining through MH invokers/linkers in unreachable code is unsafe
vlivanov
parents:
40664
diff
changeset
|
1516 |
break; |
a52ee13998f3
8174721: C1: Inlining through MH invokers/linkers in unreachable code is unsafe
vlivanov
parents:
40664
diff
changeset
|
1517 |
} |
a52ee13998f3
8174721: C1: Inlining through MH invokers/linkers in unreachable code is unsafe
vlivanov
parents:
40664
diff
changeset
|
1518 |
case vmIntrinsics::_invokeBasic: { |
a52ee13998f3
8174721: C1: Inlining through MH invokers/linkers in unreachable code is unsafe
vlivanov
parents:
40664
diff
changeset
|
1519 |
if (target->is_static()) { |
a52ee13998f3
8174721: C1: Inlining through MH invokers/linkers in unreachable code is unsafe
vlivanov
parents:
40664
diff
changeset
|
1520 |
if (target_sig->type_at(0)->is_primitive_type()) { |
a52ee13998f3
8174721: C1: Inlining through MH invokers/linkers in unreachable code is unsafe
vlivanov
parents:
40664
diff
changeset
|
1521 |
return false; // receiver should be an oop |
a52ee13998f3
8174721: C1: Inlining through MH invokers/linkers in unreachable code is unsafe
vlivanov
parents:
40664
diff
changeset
|
1522 |
} |
a52ee13998f3
8174721: C1: Inlining through MH invokers/linkers in unreachable code is unsafe
vlivanov
parents:
40664
diff
changeset
|
1523 |
rbase = 1; // skip receiver |
a52ee13998f3
8174721: C1: Inlining through MH invokers/linkers in unreachable code is unsafe
vlivanov
parents:
40664
diff
changeset
|
1524 |
} |
a52ee13998f3
8174721: C1: Inlining through MH invokers/linkers in unreachable code is unsafe
vlivanov
parents:
40664
diff
changeset
|
1525 |
break; |
a52ee13998f3
8174721: C1: Inlining through MH invokers/linkers in unreachable code is unsafe
vlivanov
parents:
40664
diff
changeset
|
1526 |
} |
46630
75aa3e39d02c
8182299: Enable disabled clang warnings, build on OSX 10 + Xcode 8
jwilhelm
parents:
46542
diff
changeset
|
1527 |
default: |
75aa3e39d02c
8182299: Enable disabled clang warnings, build on OSX 10 + Xcode 8
jwilhelm
parents:
46542
diff
changeset
|
1528 |
break; |
43947
a52ee13998f3
8174721: C1: Inlining through MH invokers/linkers in unreachable code is unsafe
vlivanov
parents:
40664
diff
changeset
|
1529 |
} |
a52ee13998f3
8174721: C1: Inlining through MH invokers/linkers in unreachable code is unsafe
vlivanov
parents:
40664
diff
changeset
|
1530 |
assert(target_sig->count() - rbase == linker_sig->count() - sbase - has_appendix, "argument count mismatch"); |
a52ee13998f3
8174721: C1: Inlining through MH invokers/linkers in unreachable code is unsafe
vlivanov
parents:
40664
diff
changeset
|
1531 |
int arg_count = target_sig->count() - rbase; |
a52ee13998f3
8174721: C1: Inlining through MH invokers/linkers in unreachable code is unsafe
vlivanov
parents:
40664
diff
changeset
|
1532 |
for (int i = 0; i < arg_count; i++) { |
a52ee13998f3
8174721: C1: Inlining through MH invokers/linkers in unreachable code is unsafe
vlivanov
parents:
40664
diff
changeset
|
1533 |
if (!basic_types_match(linker_sig->type_at(sbase + i), target_sig->type_at(rbase + i))) { |
a52ee13998f3
8174721: C1: Inlining through MH invokers/linkers in unreachable code is unsafe
vlivanov
parents:
40664
diff
changeset
|
1534 |
return false; |
a52ee13998f3
8174721: C1: Inlining through MH invokers/linkers in unreachable code is unsafe
vlivanov
parents:
40664
diff
changeset
|
1535 |
} |
a52ee13998f3
8174721: C1: Inlining through MH invokers/linkers in unreachable code is unsafe
vlivanov
parents:
40664
diff
changeset
|
1536 |
} |
a52ee13998f3
8174721: C1: Inlining through MH invokers/linkers in unreachable code is unsafe
vlivanov
parents:
40664
diff
changeset
|
1537 |
// Only check the return type if the symbolic info has non-void return type. |
a52ee13998f3
8174721: C1: Inlining through MH invokers/linkers in unreachable code is unsafe
vlivanov
parents:
40664
diff
changeset
|
1538 |
// I.e. the return value of the resolved method can be dropped. |
a52ee13998f3
8174721: C1: Inlining through MH invokers/linkers in unreachable code is unsafe
vlivanov
parents:
40664
diff
changeset
|
1539 |
if (!linker->return_type()->is_void() && |
a52ee13998f3
8174721: C1: Inlining through MH invokers/linkers in unreachable code is unsafe
vlivanov
parents:
40664
diff
changeset
|
1540 |
!basic_types_match(linker->return_type(), target->return_type())) { |
a52ee13998f3
8174721: C1: Inlining through MH invokers/linkers in unreachable code is unsafe
vlivanov
parents:
40664
diff
changeset
|
1541 |
return false; |
a52ee13998f3
8174721: C1: Inlining through MH invokers/linkers in unreachable code is unsafe
vlivanov
parents:
40664
diff
changeset
|
1542 |
} |
a52ee13998f3
8174721: C1: Inlining through MH invokers/linkers in unreachable code is unsafe
vlivanov
parents:
40664
diff
changeset
|
1543 |
return true; // no mismatch found |
a52ee13998f3
8174721: C1: Inlining through MH invokers/linkers in unreachable code is unsafe
vlivanov
parents:
40664
diff
changeset
|
1544 |
} |
a52ee13998f3
8174721: C1: Inlining through MH invokers/linkers in unreachable code is unsafe
vlivanov
parents:
40664
diff
changeset
|
1545 |
|
a52ee13998f3
8174721: C1: Inlining through MH invokers/linkers in unreachable code is unsafe
vlivanov
parents:
40664
diff
changeset
|
1546 |
// ------------------------------------------------------------------ |