src/share/vm/ci/bcEscapeAnalyzer.cpp

Tue, 18 Jun 2013 12:31:07 -0700

author
johnc
date
Tue, 18 Jun 2013 12:31:07 -0700
changeset 5277
01522ca68fc7
parent 4268
bb33c6fdcf0d
child 5436
c90c698831d7
permissions
-rw-r--r--

8015237: Parallelize string table scanning during strong root processing
Summary: Parallelize the scanning of the intern string table by having each GC worker claim a given number of buckets. Changes were also reviewed by Per Liden <per.liden@oracle.com>.
Reviewed-by: tschatzl, stefank, twisti

duke@435 1 /*
kvn@3496 2 * Copyright (c) 2005, 2012, Oracle and/or its affiliates. All rights reserved.
duke@435 3 * DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS FILE HEADER.
duke@435 4 *
duke@435 5 * This code is free software; you can redistribute it and/or modify it
duke@435 6 * under the terms of the GNU General Public License version 2 only, as
duke@435 7 * published by the Free Software Foundation.
duke@435 8 *
duke@435 9 * This code is distributed in the hope that it will be useful, but WITHOUT
duke@435 10 * ANY WARRANTY; without even the implied warranty of MERCHANTABILITY or
duke@435 11 * FITNESS FOR A PARTICULAR PURPOSE. See the GNU General Public License
duke@435 12 * version 2 for more details (a copy is included in the LICENSE file that
duke@435 13 * accompanied this code).
duke@435 14 *
duke@435 15 * You should have received a copy of the GNU General Public License version
duke@435 16 * 2 along with this work; if not, write to the Free Software Foundation,
duke@435 17 * Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301 USA.
duke@435 18 *
trims@1907 19 * Please contact Oracle, 500 Oracle Parkway, Redwood Shores, CA 94065 USA
trims@1907 20 * or visit www.oracle.com if you need additional information or have any
trims@1907 21 * questions.
duke@435 22 *
duke@435 23 */
duke@435 24
stefank@2314 25 #include "precompiled.hpp"
stefank@2314 26 #include "ci/bcEscapeAnalyzer.hpp"
stefank@2314 27 #include "ci/ciConstant.hpp"
stefank@2314 28 #include "ci/ciField.hpp"
stefank@2314 29 #include "ci/ciMethodBlocks.hpp"
stefank@2314 30 #include "ci/ciStreams.hpp"
stefank@2314 31 #include "interpreter/bytecode.hpp"
stefank@2314 32 #include "utilities/bitMap.inline.hpp"
duke@435 33
duke@435 34
duke@435 35
duke@435 36 #ifndef PRODUCT
duke@435 37 #define TRACE_BCEA(level, code) \
duke@435 38 if (EstimateArgEscape && BCEATraceLevel >= level) { \
duke@435 39 code; \
duke@435 40 }
duke@435 41 #else
duke@435 42 #define TRACE_BCEA(level, code)
duke@435 43 #endif
duke@435 44
duke@435 45 // Maintain a map of which aguments a local variable or
duke@435 46 // stack slot may contain. In addition to tracking
duke@435 47 // arguments, it tracks two special values, "allocated"
duke@435 48 // which represents any object allocated in the current
duke@435 49 // method, and "unknown" which is any other object.
duke@435 50 // Up to 30 arguments are handled, with the last one
duke@435 51 // representing summary information for any extra arguments
duke@435 52 class BCEscapeAnalyzer::ArgumentMap {
duke@435 53 uint _bits;
duke@435 54 enum {MAXBIT = 29,
duke@435 55 ALLOCATED = 1,
duke@435 56 UNKNOWN = 2};
duke@435 57
duke@435 58 uint int_to_bit(uint e) const {
duke@435 59 if (e > MAXBIT)
duke@435 60 e = MAXBIT;
duke@435 61 return (1 << (e + 2));
duke@435 62 }
duke@435 63
duke@435 64 public:
duke@435 65 ArgumentMap() { _bits = 0;}
duke@435 66 void set_bits(uint bits) { _bits = bits;}
duke@435 67 uint get_bits() const { return _bits;}
duke@435 68 void clear() { _bits = 0;}
duke@435 69 void set_all() { _bits = ~0u; }
duke@435 70 bool is_empty() const { return _bits == 0; }
duke@435 71 bool contains(uint var) const { return (_bits & int_to_bit(var)) != 0; }
duke@435 72 bool is_singleton(uint var) const { return (_bits == int_to_bit(var)); }
duke@435 73 bool contains_unknown() const { return (_bits & UNKNOWN) != 0; }
duke@435 74 bool contains_allocated() const { return (_bits & ALLOCATED) != 0; }
duke@435 75 bool contains_vars() const { return (_bits & (((1 << MAXBIT) -1) << 2)) != 0; }
duke@435 76 void set(uint var) { _bits = int_to_bit(var); }
duke@435 77 void add(uint var) { _bits |= int_to_bit(var); }
duke@435 78 void add_unknown() { _bits = UNKNOWN; }
duke@435 79 void add_allocated() { _bits = ALLOCATED; }
duke@435 80 void set_union(const ArgumentMap &am) { _bits |= am._bits; }
duke@435 81 void set_intersect(const ArgumentMap &am) { _bits |= am._bits; }
duke@435 82 void set_difference(const ArgumentMap &am) { _bits &= ~am._bits; }
duke@435 83 void operator=(const ArgumentMap &am) { _bits = am._bits; }
duke@435 84 bool operator==(const ArgumentMap &am) { return _bits == am._bits; }
duke@435 85 bool operator!=(const ArgumentMap &am) { return _bits != am._bits; }
duke@435 86 };
duke@435 87
duke@435 88 class BCEscapeAnalyzer::StateInfo {
duke@435 89 public:
duke@435 90 ArgumentMap *_vars;
duke@435 91 ArgumentMap *_stack;
duke@435 92 short _stack_height;
duke@435 93 short _max_stack;
duke@435 94 bool _initialized;
duke@435 95 ArgumentMap empty_map;
duke@435 96
duke@435 97 StateInfo() {
duke@435 98 empty_map.clear();
duke@435 99 }
duke@435 100
kvn@2102 101 ArgumentMap raw_pop() { guarantee(_stack_height > 0, "stack underflow"); return _stack[--_stack_height]; }
duke@435 102 ArgumentMap apop() { return raw_pop(); }
duke@435 103 void spop() { raw_pop(); }
duke@435 104 void lpop() { spop(); spop(); }
kvn@2102 105 void raw_push(ArgumentMap i) { guarantee(_stack_height < _max_stack, "stack overflow"); _stack[_stack_height++] = i; }
duke@435 106 void apush(ArgumentMap i) { raw_push(i); }
duke@435 107 void spush() { raw_push(empty_map); }
duke@435 108 void lpush() { spush(); spush(); }
duke@435 109
duke@435 110 };
duke@435 111
duke@435 112 void BCEscapeAnalyzer::set_returned(ArgumentMap vars) {
kvn@480 113 for (int i = 0; i < _arg_size; i++) {
duke@435 114 if (vars.contains(i))
kvn@2003 115 _arg_returned.set(i);
duke@435 116 }
duke@435 117 _return_local = _return_local && !(vars.contains_unknown() || vars.contains_allocated());
duke@435 118 _return_allocated = _return_allocated && vars.contains_allocated() && !(vars.contains_unknown() || vars.contains_vars());
duke@435 119 }
duke@435 120
duke@435 121 // return true if any element of vars is an argument
duke@435 122 bool BCEscapeAnalyzer::is_argument(ArgumentMap vars) {
kvn@480 123 for (int i = 0; i < _arg_size; i++) {
duke@435 124 if (vars.contains(i))
duke@435 125 return true;
duke@435 126 }
duke@435 127 return false;
duke@435 128 }
duke@435 129
duke@435 130 // return true if any element of vars is an arg_stack argument
duke@435 131 bool BCEscapeAnalyzer::is_arg_stack(ArgumentMap vars){
duke@435 132 if (_conservative)
duke@435 133 return true;
kvn@480 134 for (int i = 0; i < _arg_size; i++) {
kvn@2003 135 if (vars.contains(i) && _arg_stack.test(i))
duke@435 136 return true;
duke@435 137 }
duke@435 138 return false;
duke@435 139 }
duke@435 140
kvn@2003 141 void BCEscapeAnalyzer::clear_bits(ArgumentMap vars, VectorSet &bm) {
kvn@480 142 for (int i = 0; i < _arg_size; i++) {
duke@435 143 if (vars.contains(i)) {
kvn@2003 144 bm >>= i;
duke@435 145 }
duke@435 146 }
duke@435 147 }
kvn@480 148
duke@435 149 void BCEscapeAnalyzer::set_method_escape(ArgumentMap vars) {
duke@435 150 clear_bits(vars, _arg_local);
duke@435 151 }
duke@435 152
kvn@3318 153 void BCEscapeAnalyzer::set_global_escape(ArgumentMap vars, bool merge) {
duke@435 154 clear_bits(vars, _arg_local);
duke@435 155 clear_bits(vars, _arg_stack);
duke@435 156 if (vars.contains_allocated())
duke@435 157 _allocated_escapes = true;
kvn@3318 158
kvn@3318 159 if (merge && !vars.is_empty()) {
kvn@3318 160 // Merge new state into already processed block.
kvn@3318 161 // New state is not taken into account and
kvn@3318 162 // it may invalidate set_returned() result.
kvn@3318 163 if (vars.contains_unknown() || vars.contains_allocated()) {
kvn@3318 164 _return_local = false;
kvn@3318 165 }
kvn@3318 166 if (vars.contains_unknown() || vars.contains_vars()) {
kvn@3318 167 _return_allocated = false;
kvn@3318 168 }
kvn@3318 169 }
duke@435 170 }
duke@435 171
duke@435 172 void BCEscapeAnalyzer::set_dirty(ArgumentMap vars) {
duke@435 173 clear_bits(vars, _dirty);
duke@435 174 }
duke@435 175
kvn@480 176 void BCEscapeAnalyzer::set_modified(ArgumentMap vars, int offs, int size) {
kvn@480 177
kvn@480 178 for (int i = 0; i < _arg_size; i++) {
kvn@480 179 if (vars.contains(i)) {
kvn@480 180 set_arg_modified(i, offs, size);
kvn@480 181 }
kvn@480 182 }
kvn@480 183 if (vars.contains_unknown())
kvn@480 184 _unknown_modified = true;
kvn@480 185 }
kvn@480 186
duke@435 187 bool BCEscapeAnalyzer::is_recursive_call(ciMethod* callee) {
duke@435 188 for (BCEscapeAnalyzer* scope = this; scope != NULL; scope = scope->_parent) {
duke@435 189 if (scope->method() == callee) {
duke@435 190 return true;
duke@435 191 }
duke@435 192 }
duke@435 193 return false;
duke@435 194 }
duke@435 195
kvn@480 196 bool BCEscapeAnalyzer::is_arg_modified(int arg, int offset, int size_in_bytes) {
kvn@480 197 if (offset == OFFSET_ANY)
kvn@480 198 return _arg_modified[arg] != 0;
kvn@480 199 assert(arg >= 0 && arg < _arg_size, "must be an argument.");
kvn@480 200 bool modified = false;
kvn@480 201 int l = offset / HeapWordSize;
kvn@480 202 int h = round_to(offset + size_in_bytes, HeapWordSize) / HeapWordSize;
kvn@480 203 if (l > ARG_OFFSET_MAX)
kvn@480 204 l = ARG_OFFSET_MAX;
kvn@480 205 if (h > ARG_OFFSET_MAX+1)
kvn@480 206 h = ARG_OFFSET_MAX + 1;
kvn@480 207 for (int i = l; i < h; i++) {
kvn@480 208 modified = modified || (_arg_modified[arg] & (1 << i)) != 0;
kvn@480 209 }
kvn@480 210 return modified;
kvn@480 211 }
kvn@480 212
kvn@480 213 void BCEscapeAnalyzer::set_arg_modified(int arg, int offset, int size_in_bytes) {
kvn@480 214 if (offset == OFFSET_ANY) {
kvn@480 215 _arg_modified[arg] = (uint) -1;
kvn@480 216 return;
kvn@480 217 }
kvn@480 218 assert(arg >= 0 && arg < _arg_size, "must be an argument.");
kvn@480 219 int l = offset / HeapWordSize;
kvn@480 220 int h = round_to(offset + size_in_bytes, HeapWordSize) / HeapWordSize;
kvn@480 221 if (l > ARG_OFFSET_MAX)
kvn@480 222 l = ARG_OFFSET_MAX;
kvn@480 223 if (h > ARG_OFFSET_MAX+1)
kvn@480 224 h = ARG_OFFSET_MAX + 1;
kvn@480 225 for (int i = l; i < h; i++) {
kvn@480 226 _arg_modified[arg] |= (1 << i);
kvn@480 227 }
kvn@480 228 }
kvn@480 229
duke@435 230 void BCEscapeAnalyzer::invoke(StateInfo &state, Bytecodes::Code code, ciMethod* target, ciKlass* holder) {
duke@435 231 int i;
duke@435 232
duke@435 233 // retrieve information about the callee
duke@435 234 ciInstanceKlass* klass = target->holder();
duke@435 235 ciInstanceKlass* calling_klass = method()->holder();
duke@435 236 ciInstanceKlass* callee_holder = ciEnv::get_instance_klass_for_declared_method_holder(holder);
duke@435 237 ciInstanceKlass* actual_recv = callee_holder;
duke@435 238
twisti@4021 239 // Some methods are obviously bindable without any type checks so
twisti@4021 240 // convert them directly to an invokespecial or invokestatic.
twisti@3969 241 if (target->is_loaded() && !target->is_abstract() && target->can_be_statically_bound()) {
twisti@3969 242 switch (code) {
twisti@4021 243 case Bytecodes::_invokevirtual:
twisti@4021 244 code = Bytecodes::_invokespecial;
twisti@4021 245 break;
twisti@4021 246 case Bytecodes::_invokehandle:
twisti@4021 247 code = target->is_static() ? Bytecodes::_invokestatic : Bytecodes::_invokespecial;
twisti@4021 248 break;
twisti@3969 249 }
kvn@589 250 }
kvn@589 251
duke@435 252 // compute size of arguments
never@2812 253 int arg_size = target->invoke_arg_size(code);
duke@435 254 int arg_base = MAX2(state._stack_height - arg_size, 0);
duke@435 255
duke@435 256 // direct recursive calls are skipped if they can be bound statically without introducing
duke@435 257 // dependencies and if parameters are passed at the same position as in the current method
duke@435 258 // other calls are skipped if there are no unescaped arguments passed to them
duke@435 259 bool directly_recursive = (method() == target) &&
duke@435 260 (code != Bytecodes::_invokevirtual || target->is_final_method() || state._stack[arg_base] .is_empty());
duke@435 261
duke@435 262 // check if analysis of callee can safely be skipped
duke@435 263 bool skip_callee = true;
duke@435 264 for (i = state._stack_height - 1; i >= arg_base && skip_callee; i--) {
duke@435 265 ArgumentMap arg = state._stack[i];
duke@435 266 skip_callee = !is_argument(arg) || !is_arg_stack(arg) || (directly_recursive && arg.is_singleton(i - arg_base));
duke@435 267 }
twisti@2484 268 // For now we conservatively skip invokedynamic.
twisti@2484 269 if (code == Bytecodes::_invokedynamic) {
twisti@2484 270 skip_callee = true;
twisti@2484 271 }
duke@435 272 if (skip_callee) {
duke@435 273 TRACE_BCEA(3, tty->print_cr("[EA] skipping method %s::%s", holder->name()->as_utf8(), target->name()->as_utf8()));
duke@435 274 for (i = 0; i < arg_size; i++) {
duke@435 275 set_method_escape(state.raw_pop());
duke@435 276 }
kvn@480 277 _unknown_modified = true; // assume the worst since we don't analyze the called method
duke@435 278 return;
duke@435 279 }
duke@435 280
duke@435 281 // determine actual method (use CHA if necessary)
duke@435 282 ciMethod* inline_target = NULL;
duke@435 283 if (target->is_loaded() && klass->is_loaded()
duke@435 284 && (klass->is_initialized() || klass->is_interface() && target->holder()->is_initialized())
bharadwaj@4268 285 && target->is_loaded()) {
duke@435 286 if (code == Bytecodes::_invokestatic
duke@435 287 || code == Bytecodes::_invokespecial
duke@435 288 || code == Bytecodes::_invokevirtual && target->is_final_method()) {
duke@435 289 inline_target = target;
duke@435 290 } else {
duke@435 291 inline_target = target->find_monomorphic_target(calling_klass, callee_holder, actual_recv);
duke@435 292 }
duke@435 293 }
duke@435 294
duke@435 295 if (inline_target != NULL && !is_recursive_call(inline_target)) {
duke@435 296 // analyze callee
duke@435 297 BCEscapeAnalyzer analyzer(inline_target, this);
duke@435 298
duke@435 299 // adjust escape state of actual parameters
duke@435 300 bool must_record_dependencies = false;
duke@435 301 for (i = arg_size - 1; i >= 0; i--) {
duke@435 302 ArgumentMap arg = state.raw_pop();
duke@435 303 if (!is_argument(arg))
duke@435 304 continue;
kvn@480 305 for (int j = 0; j < _arg_size; j++) {
kvn@480 306 if (arg.contains(j)) {
kvn@480 307 _arg_modified[j] |= analyzer._arg_modified[i];
kvn@480 308 }
kvn@480 309 }
duke@435 310 if (!is_arg_stack(arg)) {
duke@435 311 // arguments have already been recognized as escaping
duke@435 312 } else if (analyzer.is_arg_stack(i) && !analyzer.is_arg_returned(i)) {
duke@435 313 set_method_escape(arg);
duke@435 314 must_record_dependencies = true;
duke@435 315 } else {
duke@435 316 set_global_escape(arg);
duke@435 317 }
duke@435 318 }
kvn@480 319 _unknown_modified = _unknown_modified || analyzer.has_non_arg_side_affects();
duke@435 320
duke@435 321 // record dependencies if at least one parameter retained stack-allocatable
duke@435 322 if (must_record_dependencies) {
duke@435 323 if (code == Bytecodes::_invokeinterface || code == Bytecodes::_invokevirtual && !target->is_final_method()) {
duke@435 324 _dependencies.append(actual_recv);
duke@435 325 _dependencies.append(inline_target);
duke@435 326 }
duke@435 327 _dependencies.appendAll(analyzer.dependencies());
duke@435 328 }
duke@435 329 } else {
duke@435 330 TRACE_BCEA(1, tty->print_cr("[EA] virtual method %s is not monomorphic.",
duke@435 331 target->name()->as_utf8()));
duke@435 332 // conservatively mark all actual parameters as escaping globally
duke@435 333 for (i = 0; i < arg_size; i++) {
duke@435 334 ArgumentMap arg = state.raw_pop();
duke@435 335 if (!is_argument(arg))
duke@435 336 continue;
kvn@480 337 set_modified(arg, OFFSET_ANY, type2size[T_INT]*HeapWordSize);
duke@435 338 set_global_escape(arg);
duke@435 339 }
kvn@480 340 _unknown_modified = true; // assume the worst since we don't know the called method
duke@435 341 }
duke@435 342 }
duke@435 343
duke@435 344 bool BCEscapeAnalyzer::contains(uint arg_set1, uint arg_set2) {
duke@435 345 return ((~arg_set1) | arg_set2) == 0;
duke@435 346 }
duke@435 347
duke@435 348
duke@435 349 void BCEscapeAnalyzer::iterate_one_block(ciBlock *blk, StateInfo &state, GrowableArray<ciBlock *> &successors) {
duke@435 350
duke@435 351 blk->set_processed();
duke@435 352 ciBytecodeStream s(method());
duke@435 353 int limit_bci = blk->limit_bci();
duke@435 354 bool fall_through = false;
duke@435 355 ArgumentMap allocated_obj;
duke@435 356 allocated_obj.add_allocated();
duke@435 357 ArgumentMap unknown_obj;
duke@435 358 unknown_obj.add_unknown();
duke@435 359 ArgumentMap empty_map;
duke@435 360
duke@435 361 s.reset_to_bci(blk->start_bci());
duke@435 362 while (s.next() != ciBytecodeStream::EOBC() && s.cur_bci() < limit_bci) {
duke@435 363 fall_through = true;
duke@435 364 switch (s.cur_bc()) {
duke@435 365 case Bytecodes::_nop:
duke@435 366 break;
duke@435 367 case Bytecodes::_aconst_null:
kvn@3496 368 state.apush(unknown_obj);
duke@435 369 break;
duke@435 370 case Bytecodes::_iconst_m1:
duke@435 371 case Bytecodes::_iconst_0:
duke@435 372 case Bytecodes::_iconst_1:
duke@435 373 case Bytecodes::_iconst_2:
duke@435 374 case Bytecodes::_iconst_3:
duke@435 375 case Bytecodes::_iconst_4:
duke@435 376 case Bytecodes::_iconst_5:
duke@435 377 case Bytecodes::_fconst_0:
duke@435 378 case Bytecodes::_fconst_1:
duke@435 379 case Bytecodes::_fconst_2:
duke@435 380 case Bytecodes::_bipush:
duke@435 381 case Bytecodes::_sipush:
duke@435 382 state.spush();
duke@435 383 break;
duke@435 384 case Bytecodes::_lconst_0:
duke@435 385 case Bytecodes::_lconst_1:
duke@435 386 case Bytecodes::_dconst_0:
duke@435 387 case Bytecodes::_dconst_1:
duke@435 388 state.lpush();
duke@435 389 break;
duke@435 390 case Bytecodes::_ldc:
duke@435 391 case Bytecodes::_ldc_w:
duke@435 392 case Bytecodes::_ldc2_w:
kvn@2102 393 {
kvn@2102 394 // Avoid calling get_constant() which will try to allocate
kvn@2102 395 // unloaded constant. We need only constant's type.
kvn@2102 396 int index = s.get_constant_pool_index();
kvn@2102 397 constantTag tag = s.get_constant_pool_tag(index);
kvn@2102 398 if (tag.is_long() || tag.is_double()) {
kvn@2102 399 // Only longs and doubles use 2 stack slots.
kvn@2102 400 state.lpush();
kvn@3496 401 } else if (tag.basic_type() == T_OBJECT) {
kvn@3496 402 state.apush(unknown_obj);
kvn@2102 403 } else {
duke@435 404 state.spush();
duke@435 405 }
duke@435 406 break;
kvn@2102 407 }
duke@435 408 case Bytecodes::_aload:
duke@435 409 state.apush(state._vars[s.get_index()]);
duke@435 410 break;
duke@435 411 case Bytecodes::_iload:
duke@435 412 case Bytecodes::_fload:
duke@435 413 case Bytecodes::_iload_0:
duke@435 414 case Bytecodes::_iload_1:
duke@435 415 case Bytecodes::_iload_2:
duke@435 416 case Bytecodes::_iload_3:
duke@435 417 case Bytecodes::_fload_0:
duke@435 418 case Bytecodes::_fload_1:
duke@435 419 case Bytecodes::_fload_2:
duke@435 420 case Bytecodes::_fload_3:
duke@435 421 state.spush();
duke@435 422 break;
duke@435 423 case Bytecodes::_lload:
duke@435 424 case Bytecodes::_dload:
duke@435 425 case Bytecodes::_lload_0:
duke@435 426 case Bytecodes::_lload_1:
duke@435 427 case Bytecodes::_lload_2:
duke@435 428 case Bytecodes::_lload_3:
duke@435 429 case Bytecodes::_dload_0:
duke@435 430 case Bytecodes::_dload_1:
duke@435 431 case Bytecodes::_dload_2:
duke@435 432 case Bytecodes::_dload_3:
duke@435 433 state.lpush();
duke@435 434 break;
duke@435 435 case Bytecodes::_aload_0:
duke@435 436 state.apush(state._vars[0]);
duke@435 437 break;
duke@435 438 case Bytecodes::_aload_1:
duke@435 439 state.apush(state._vars[1]);
duke@435 440 break;
duke@435 441 case Bytecodes::_aload_2:
duke@435 442 state.apush(state._vars[2]);
duke@435 443 break;
duke@435 444 case Bytecodes::_aload_3:
duke@435 445 state.apush(state._vars[3]);
duke@435 446 break;
duke@435 447 case Bytecodes::_iaload:
duke@435 448 case Bytecodes::_faload:
duke@435 449 case Bytecodes::_baload:
duke@435 450 case Bytecodes::_caload:
duke@435 451 case Bytecodes::_saload:
duke@435 452 state.spop();
duke@435 453 set_method_escape(state.apop());
duke@435 454 state.spush();
duke@435 455 break;
duke@435 456 case Bytecodes::_laload:
duke@435 457 case Bytecodes::_daload:
duke@435 458 state.spop();
duke@435 459 set_method_escape(state.apop());
duke@435 460 state.lpush();
duke@435 461 break;
duke@435 462 case Bytecodes::_aaload:
duke@435 463 { state.spop();
duke@435 464 ArgumentMap array = state.apop();
duke@435 465 set_method_escape(array);
duke@435 466 state.apush(unknown_obj);
duke@435 467 set_dirty(array);
duke@435 468 }
duke@435 469 break;
duke@435 470 case Bytecodes::_istore:
duke@435 471 case Bytecodes::_fstore:
duke@435 472 case Bytecodes::_istore_0:
duke@435 473 case Bytecodes::_istore_1:
duke@435 474 case Bytecodes::_istore_2:
duke@435 475 case Bytecodes::_istore_3:
duke@435 476 case Bytecodes::_fstore_0:
duke@435 477 case Bytecodes::_fstore_1:
duke@435 478 case Bytecodes::_fstore_2:
duke@435 479 case Bytecodes::_fstore_3:
duke@435 480 state.spop();
duke@435 481 break;
duke@435 482 case Bytecodes::_lstore:
duke@435 483 case Bytecodes::_dstore:
duke@435 484 case Bytecodes::_lstore_0:
duke@435 485 case Bytecodes::_lstore_1:
duke@435 486 case Bytecodes::_lstore_2:
duke@435 487 case Bytecodes::_lstore_3:
duke@435 488 case Bytecodes::_dstore_0:
duke@435 489 case Bytecodes::_dstore_1:
duke@435 490 case Bytecodes::_dstore_2:
duke@435 491 case Bytecodes::_dstore_3:
duke@435 492 state.lpop();
duke@435 493 break;
duke@435 494 case Bytecodes::_astore:
duke@435 495 state._vars[s.get_index()] = state.apop();
duke@435 496 break;
duke@435 497 case Bytecodes::_astore_0:
duke@435 498 state._vars[0] = state.apop();
duke@435 499 break;
duke@435 500 case Bytecodes::_astore_1:
duke@435 501 state._vars[1] = state.apop();
duke@435 502 break;
duke@435 503 case Bytecodes::_astore_2:
duke@435 504 state._vars[2] = state.apop();
duke@435 505 break;
duke@435 506 case Bytecodes::_astore_3:
duke@435 507 state._vars[3] = state.apop();
duke@435 508 break;
duke@435 509 case Bytecodes::_iastore:
duke@435 510 case Bytecodes::_fastore:
duke@435 511 case Bytecodes::_bastore:
duke@435 512 case Bytecodes::_castore:
duke@435 513 case Bytecodes::_sastore:
duke@435 514 {
duke@435 515 state.spop();
duke@435 516 state.spop();
duke@435 517 ArgumentMap arr = state.apop();
duke@435 518 set_method_escape(arr);
kvn@480 519 set_modified(arr, OFFSET_ANY, type2size[T_INT]*HeapWordSize);
duke@435 520 break;
duke@435 521 }
duke@435 522 case Bytecodes::_lastore:
duke@435 523 case Bytecodes::_dastore:
duke@435 524 {
duke@435 525 state.lpop();
duke@435 526 state.spop();
duke@435 527 ArgumentMap arr = state.apop();
duke@435 528 set_method_escape(arr);
kvn@480 529 set_modified(arr, OFFSET_ANY, type2size[T_LONG]*HeapWordSize);
duke@435 530 break;
duke@435 531 }
duke@435 532 case Bytecodes::_aastore:
duke@435 533 {
duke@435 534 set_global_escape(state.apop());
duke@435 535 state.spop();
duke@435 536 ArgumentMap arr = state.apop();
kvn@480 537 set_modified(arr, OFFSET_ANY, type2size[T_OBJECT]*HeapWordSize);
duke@435 538 break;
duke@435 539 }
duke@435 540 case Bytecodes::_pop:
duke@435 541 state.raw_pop();
duke@435 542 break;
duke@435 543 case Bytecodes::_pop2:
duke@435 544 state.raw_pop();
duke@435 545 state.raw_pop();
duke@435 546 break;
duke@435 547 case Bytecodes::_dup:
duke@435 548 { ArgumentMap w1 = state.raw_pop();
duke@435 549 state.raw_push(w1);
duke@435 550 state.raw_push(w1);
duke@435 551 }
duke@435 552 break;
duke@435 553 case Bytecodes::_dup_x1:
duke@435 554 { ArgumentMap w1 = state.raw_pop();
duke@435 555 ArgumentMap w2 = state.raw_pop();
duke@435 556 state.raw_push(w1);
duke@435 557 state.raw_push(w2);
duke@435 558 state.raw_push(w1);
duke@435 559 }
duke@435 560 break;
duke@435 561 case Bytecodes::_dup_x2:
duke@435 562 { ArgumentMap w1 = state.raw_pop();
duke@435 563 ArgumentMap w2 = state.raw_pop();
duke@435 564 ArgumentMap w3 = state.raw_pop();
duke@435 565 state.raw_push(w1);
duke@435 566 state.raw_push(w3);
duke@435 567 state.raw_push(w2);
duke@435 568 state.raw_push(w1);
duke@435 569 }
duke@435 570 break;
duke@435 571 case Bytecodes::_dup2:
duke@435 572 { ArgumentMap w1 = state.raw_pop();
duke@435 573 ArgumentMap w2 = state.raw_pop();
duke@435 574 state.raw_push(w2);
duke@435 575 state.raw_push(w1);
duke@435 576 state.raw_push(w2);
duke@435 577 state.raw_push(w1);
duke@435 578 }
duke@435 579 break;
duke@435 580 case Bytecodes::_dup2_x1:
duke@435 581 { ArgumentMap w1 = state.raw_pop();
duke@435 582 ArgumentMap w2 = state.raw_pop();
duke@435 583 ArgumentMap w3 = state.raw_pop();
duke@435 584 state.raw_push(w2);
duke@435 585 state.raw_push(w1);
duke@435 586 state.raw_push(w3);
duke@435 587 state.raw_push(w2);
duke@435 588 state.raw_push(w1);
duke@435 589 }
duke@435 590 break;
duke@435 591 case Bytecodes::_dup2_x2:
duke@435 592 { ArgumentMap w1 = state.raw_pop();
duke@435 593 ArgumentMap w2 = state.raw_pop();
duke@435 594 ArgumentMap w3 = state.raw_pop();
duke@435 595 ArgumentMap w4 = state.raw_pop();
duke@435 596 state.raw_push(w2);
duke@435 597 state.raw_push(w1);
duke@435 598 state.raw_push(w4);
duke@435 599 state.raw_push(w3);
duke@435 600 state.raw_push(w2);
duke@435 601 state.raw_push(w1);
duke@435 602 }
duke@435 603 break;
duke@435 604 case Bytecodes::_swap:
duke@435 605 { ArgumentMap w1 = state.raw_pop();
duke@435 606 ArgumentMap w2 = state.raw_pop();
duke@435 607 state.raw_push(w1);
duke@435 608 state.raw_push(w2);
duke@435 609 }
duke@435 610 break;
duke@435 611 case Bytecodes::_iadd:
duke@435 612 case Bytecodes::_fadd:
duke@435 613 case Bytecodes::_isub:
duke@435 614 case Bytecodes::_fsub:
duke@435 615 case Bytecodes::_imul:
duke@435 616 case Bytecodes::_fmul:
duke@435 617 case Bytecodes::_idiv:
duke@435 618 case Bytecodes::_fdiv:
duke@435 619 case Bytecodes::_irem:
duke@435 620 case Bytecodes::_frem:
duke@435 621 case Bytecodes::_iand:
duke@435 622 case Bytecodes::_ior:
duke@435 623 case Bytecodes::_ixor:
duke@435 624 state.spop();
duke@435 625 state.spop();
duke@435 626 state.spush();
duke@435 627 break;
duke@435 628 case Bytecodes::_ladd:
duke@435 629 case Bytecodes::_dadd:
duke@435 630 case Bytecodes::_lsub:
duke@435 631 case Bytecodes::_dsub:
duke@435 632 case Bytecodes::_lmul:
duke@435 633 case Bytecodes::_dmul:
duke@435 634 case Bytecodes::_ldiv:
duke@435 635 case Bytecodes::_ddiv:
duke@435 636 case Bytecodes::_lrem:
duke@435 637 case Bytecodes::_drem:
duke@435 638 case Bytecodes::_land:
duke@435 639 case Bytecodes::_lor:
duke@435 640 case Bytecodes::_lxor:
duke@435 641 state.lpop();
duke@435 642 state.lpop();
duke@435 643 state.lpush();
duke@435 644 break;
duke@435 645 case Bytecodes::_ishl:
duke@435 646 case Bytecodes::_ishr:
duke@435 647 case Bytecodes::_iushr:
duke@435 648 state.spop();
duke@435 649 state.spop();
duke@435 650 state.spush();
duke@435 651 break;
duke@435 652 case Bytecodes::_lshl:
duke@435 653 case Bytecodes::_lshr:
duke@435 654 case Bytecodes::_lushr:
duke@435 655 state.spop();
duke@435 656 state.lpop();
duke@435 657 state.lpush();
duke@435 658 break;
duke@435 659 case Bytecodes::_ineg:
duke@435 660 case Bytecodes::_fneg:
duke@435 661 state.spop();
duke@435 662 state.spush();
duke@435 663 break;
duke@435 664 case Bytecodes::_lneg:
duke@435 665 case Bytecodes::_dneg:
duke@435 666 state.lpop();
duke@435 667 state.lpush();
duke@435 668 break;
duke@435 669 case Bytecodes::_iinc:
duke@435 670 break;
duke@435 671 case Bytecodes::_i2l:
duke@435 672 case Bytecodes::_i2d:
duke@435 673 case Bytecodes::_f2l:
duke@435 674 case Bytecodes::_f2d:
duke@435 675 state.spop();
duke@435 676 state.lpush();
duke@435 677 break;
duke@435 678 case Bytecodes::_i2f:
duke@435 679 case Bytecodes::_f2i:
duke@435 680 state.spop();
duke@435 681 state.spush();
duke@435 682 break;
duke@435 683 case Bytecodes::_l2i:
duke@435 684 case Bytecodes::_l2f:
duke@435 685 case Bytecodes::_d2i:
duke@435 686 case Bytecodes::_d2f:
duke@435 687 state.lpop();
duke@435 688 state.spush();
duke@435 689 break;
duke@435 690 case Bytecodes::_l2d:
duke@435 691 case Bytecodes::_d2l:
duke@435 692 state.lpop();
duke@435 693 state.lpush();
duke@435 694 break;
duke@435 695 case Bytecodes::_i2b:
duke@435 696 case Bytecodes::_i2c:
duke@435 697 case Bytecodes::_i2s:
duke@435 698 state.spop();
duke@435 699 state.spush();
duke@435 700 break;
duke@435 701 case Bytecodes::_lcmp:
duke@435 702 case Bytecodes::_dcmpl:
duke@435 703 case Bytecodes::_dcmpg:
duke@435 704 state.lpop();
duke@435 705 state.lpop();
duke@435 706 state.spush();
duke@435 707 break;
duke@435 708 case Bytecodes::_fcmpl:
duke@435 709 case Bytecodes::_fcmpg:
duke@435 710 state.spop();
duke@435 711 state.spop();
duke@435 712 state.spush();
duke@435 713 break;
duke@435 714 case Bytecodes::_ifeq:
duke@435 715 case Bytecodes::_ifne:
duke@435 716 case Bytecodes::_iflt:
duke@435 717 case Bytecodes::_ifge:
duke@435 718 case Bytecodes::_ifgt:
duke@435 719 case Bytecodes::_ifle:
duke@435 720 {
duke@435 721 state.spop();
duke@435 722 int dest_bci = s.get_dest();
duke@435 723 assert(_methodBlocks->is_block_start(dest_bci), "branch destination must start a block");
duke@435 724 assert(s.next_bci() == limit_bci, "branch must end block");
duke@435 725 successors.push(_methodBlocks->block_containing(dest_bci));
duke@435 726 break;
duke@435 727 }
duke@435 728 case Bytecodes::_if_icmpeq:
duke@435 729 case Bytecodes::_if_icmpne:
duke@435 730 case Bytecodes::_if_icmplt:
duke@435 731 case Bytecodes::_if_icmpge:
duke@435 732 case Bytecodes::_if_icmpgt:
duke@435 733 case Bytecodes::_if_icmple:
duke@435 734 {
duke@435 735 state.spop();
duke@435 736 state.spop();
duke@435 737 int dest_bci = s.get_dest();
duke@435 738 assert(_methodBlocks->is_block_start(dest_bci), "branch destination must start a block");
duke@435 739 assert(s.next_bci() == limit_bci, "branch must end block");
duke@435 740 successors.push(_methodBlocks->block_containing(dest_bci));
duke@435 741 break;
duke@435 742 }
duke@435 743 case Bytecodes::_if_acmpeq:
duke@435 744 case Bytecodes::_if_acmpne:
duke@435 745 {
duke@435 746 set_method_escape(state.apop());
duke@435 747 set_method_escape(state.apop());
duke@435 748 int dest_bci = s.get_dest();
duke@435 749 assert(_methodBlocks->is_block_start(dest_bci), "branch destination must start a block");
duke@435 750 assert(s.next_bci() == limit_bci, "branch must end block");
duke@435 751 successors.push(_methodBlocks->block_containing(dest_bci));
duke@435 752 break;
duke@435 753 }
duke@435 754 case Bytecodes::_goto:
duke@435 755 {
duke@435 756 int dest_bci = s.get_dest();
duke@435 757 assert(_methodBlocks->is_block_start(dest_bci), "branch destination must start a block");
duke@435 758 assert(s.next_bci() == limit_bci, "branch must end block");
duke@435 759 successors.push(_methodBlocks->block_containing(dest_bci));
duke@435 760 fall_through = false;
duke@435 761 break;
duke@435 762 }
duke@435 763 case Bytecodes::_jsr:
duke@435 764 {
duke@435 765 int dest_bci = s.get_dest();
duke@435 766 assert(_methodBlocks->is_block_start(dest_bci), "branch destination must start a block");
duke@435 767 assert(s.next_bci() == limit_bci, "branch must end block");
duke@435 768 state.apush(empty_map);
duke@435 769 successors.push(_methodBlocks->block_containing(dest_bci));
duke@435 770 fall_through = false;
duke@435 771 break;
duke@435 772 }
duke@435 773 case Bytecodes::_ret:
duke@435 774 // we don't track the destination of a "ret" instruction
duke@435 775 assert(s.next_bci() == limit_bci, "branch must end block");
duke@435 776 fall_through = false;
duke@435 777 break;
duke@435 778 case Bytecodes::_return:
duke@435 779 assert(s.next_bci() == limit_bci, "return must end block");
duke@435 780 fall_through = false;
duke@435 781 break;
duke@435 782 case Bytecodes::_tableswitch:
duke@435 783 {
duke@435 784 state.spop();
never@2462 785 Bytecode_tableswitch sw(&s);
never@2462 786 int len = sw.length();
duke@435 787 int dest_bci;
duke@435 788 for (int i = 0; i < len; i++) {
never@2462 789 dest_bci = s.cur_bci() + sw.dest_offset_at(i);
duke@435 790 assert(_methodBlocks->is_block_start(dest_bci), "branch destination must start a block");
duke@435 791 successors.push(_methodBlocks->block_containing(dest_bci));
duke@435 792 }
never@2462 793 dest_bci = s.cur_bci() + sw.default_offset();
duke@435 794 assert(_methodBlocks->is_block_start(dest_bci), "branch destination must start a block");
duke@435 795 successors.push(_methodBlocks->block_containing(dest_bci));
duke@435 796 assert(s.next_bci() == limit_bci, "branch must end block");
duke@435 797 fall_through = false;
duke@435 798 break;
duke@435 799 }
duke@435 800 case Bytecodes::_lookupswitch:
duke@435 801 {
duke@435 802 state.spop();
never@2462 803 Bytecode_lookupswitch sw(&s);
never@2462 804 int len = sw.number_of_pairs();
duke@435 805 int dest_bci;
duke@435 806 for (int i = 0; i < len; i++) {
never@2462 807 dest_bci = s.cur_bci() + sw.pair_at(i).offset();
duke@435 808 assert(_methodBlocks->is_block_start(dest_bci), "branch destination must start a block");
duke@435 809 successors.push(_methodBlocks->block_containing(dest_bci));
duke@435 810 }
never@2462 811 dest_bci = s.cur_bci() + sw.default_offset();
duke@435 812 assert(_methodBlocks->is_block_start(dest_bci), "branch destination must start a block");
duke@435 813 successors.push(_methodBlocks->block_containing(dest_bci));
duke@435 814 fall_through = false;
duke@435 815 break;
duke@435 816 }
duke@435 817 case Bytecodes::_ireturn:
duke@435 818 case Bytecodes::_freturn:
duke@435 819 state.spop();
duke@435 820 fall_through = false;
duke@435 821 break;
duke@435 822 case Bytecodes::_lreturn:
duke@435 823 case Bytecodes::_dreturn:
duke@435 824 state.lpop();
duke@435 825 fall_through = false;
duke@435 826 break;
duke@435 827 case Bytecodes::_areturn:
duke@435 828 set_returned(state.apop());
duke@435 829 fall_through = false;
duke@435 830 break;
duke@435 831 case Bytecodes::_getstatic:
duke@435 832 case Bytecodes::_getfield:
twisti@4021 833 { bool ignored_will_link;
twisti@4021 834 ciField* field = s.get_field(ignored_will_link);
duke@435 835 BasicType field_type = field->type()->basic_type();
duke@435 836 if (s.cur_bc() != Bytecodes::_getstatic) {
duke@435 837 set_method_escape(state.apop());
duke@435 838 }
duke@435 839 if (field_type == T_OBJECT || field_type == T_ARRAY) {
duke@435 840 state.apush(unknown_obj);
duke@435 841 } else if (type2size[field_type] == 1) {
duke@435 842 state.spush();
duke@435 843 } else {
duke@435 844 state.lpush();
duke@435 845 }
duke@435 846 }
duke@435 847 break;
duke@435 848 case Bytecodes::_putstatic:
duke@435 849 case Bytecodes::_putfield:
duke@435 850 { bool will_link;
duke@435 851 ciField* field = s.get_field(will_link);
duke@435 852 BasicType field_type = field->type()->basic_type();
duke@435 853 if (field_type == T_OBJECT || field_type == T_ARRAY) {
duke@435 854 set_global_escape(state.apop());
duke@435 855 } else if (type2size[field_type] == 1) {
duke@435 856 state.spop();
duke@435 857 } else {
duke@435 858 state.lpop();
duke@435 859 }
duke@435 860 if (s.cur_bc() != Bytecodes::_putstatic) {
duke@435 861 ArgumentMap p = state.apop();
duke@435 862 set_method_escape(p);
kvn@480 863 set_modified(p, will_link ? field->offset() : OFFSET_ANY, type2size[field_type]*HeapWordSize);
duke@435 864 }
duke@435 865 }
duke@435 866 break;
duke@435 867 case Bytecodes::_invokevirtual:
duke@435 868 case Bytecodes::_invokespecial:
duke@435 869 case Bytecodes::_invokestatic:
jrose@1161 870 case Bytecodes::_invokedynamic:
duke@435 871 case Bytecodes::_invokeinterface:
twisti@4021 872 { bool ignored_will_link;
twisti@4021 873 ciSignature* declared_signature = NULL;
twisti@4021 874 ciMethod* target = s.get_method(ignored_will_link, &declared_signature);
twisti@4021 875 ciKlass* holder = s.get_declared_method_holder();
twisti@4021 876 assert(declared_signature != NULL, "cannot be null");
twisti@3969 877 // Push appendix argument, if one.
twisti@3969 878 if (s.has_appendix()) {
twisti@3969 879 state.apush(unknown_obj);
twisti@3969 880 }
twisti@3969 881 // Pass in raw bytecode because we need to see invokehandle instructions.
twisti@3969 882 invoke(state, s.cur_bc_raw(), target, holder);
twisti@4021 883 // We are using the return type of the declared signature here because
twisti@4021 884 // it might be a more concrete type than the one from the target (for
twisti@4021 885 // e.g. invokedynamic and invokehandle).
twisti@4021 886 ciType* return_type = declared_signature->return_type();
duke@435 887 if (!return_type->is_primitive_type()) {
duke@435 888 state.apush(unknown_obj);
duke@435 889 } else if (return_type->is_one_word()) {
duke@435 890 state.spush();
duke@435 891 } else if (return_type->is_two_word()) {
duke@435 892 state.lpush();
duke@435 893 }
duke@435 894 }
duke@435 895 break;
duke@435 896 case Bytecodes::_new:
duke@435 897 state.apush(allocated_obj);
duke@435 898 break;
duke@435 899 case Bytecodes::_newarray:
duke@435 900 case Bytecodes::_anewarray:
duke@435 901 state.spop();
duke@435 902 state.apush(allocated_obj);
duke@435 903 break;
duke@435 904 case Bytecodes::_multianewarray:
duke@435 905 { int i = s.cur_bcp()[3];
duke@435 906 while (i-- > 0) state.spop();
duke@435 907 state.apush(allocated_obj);
duke@435 908 }
duke@435 909 break;
duke@435 910 case Bytecodes::_arraylength:
duke@435 911 set_method_escape(state.apop());
duke@435 912 state.spush();
duke@435 913 break;
duke@435 914 case Bytecodes::_athrow:
duke@435 915 set_global_escape(state.apop());
duke@435 916 fall_through = false;
duke@435 917 break;
duke@435 918 case Bytecodes::_checkcast:
duke@435 919 { ArgumentMap obj = state.apop();
duke@435 920 set_method_escape(obj);
duke@435 921 state.apush(obj);
duke@435 922 }
duke@435 923 break;
duke@435 924 case Bytecodes::_instanceof:
duke@435 925 set_method_escape(state.apop());
duke@435 926 state.spush();
duke@435 927 break;
duke@435 928 case Bytecodes::_monitorenter:
duke@435 929 case Bytecodes::_monitorexit:
duke@435 930 state.apop();
duke@435 931 break;
duke@435 932 case Bytecodes::_wide:
duke@435 933 ShouldNotReachHere();
duke@435 934 break;
duke@435 935 case Bytecodes::_ifnull:
duke@435 936 case Bytecodes::_ifnonnull:
duke@435 937 {
duke@435 938 set_method_escape(state.apop());
duke@435 939 int dest_bci = s.get_dest();
duke@435 940 assert(_methodBlocks->is_block_start(dest_bci), "branch destination must start a block");
duke@435 941 assert(s.next_bci() == limit_bci, "branch must end block");
duke@435 942 successors.push(_methodBlocks->block_containing(dest_bci));
duke@435 943 break;
duke@435 944 }
duke@435 945 case Bytecodes::_goto_w:
duke@435 946 {
duke@435 947 int dest_bci = s.get_far_dest();
duke@435 948 assert(_methodBlocks->is_block_start(dest_bci), "branch destination must start a block");
duke@435 949 assert(s.next_bci() == limit_bci, "branch must end block");
duke@435 950 successors.push(_methodBlocks->block_containing(dest_bci));
duke@435 951 fall_through = false;
duke@435 952 break;
duke@435 953 }
duke@435 954 case Bytecodes::_jsr_w:
duke@435 955 {
duke@435 956 int dest_bci = s.get_far_dest();
duke@435 957 assert(_methodBlocks->is_block_start(dest_bci), "branch destination must start a block");
duke@435 958 assert(s.next_bci() == limit_bci, "branch must end block");
duke@435 959 state.apush(empty_map);
duke@435 960 successors.push(_methodBlocks->block_containing(dest_bci));
duke@435 961 fall_through = false;
duke@435 962 break;
duke@435 963 }
duke@435 964 case Bytecodes::_breakpoint:
duke@435 965 break;
duke@435 966 default:
duke@435 967 ShouldNotReachHere();
duke@435 968 break;
duke@435 969 }
duke@435 970
duke@435 971 }
duke@435 972 if (fall_through) {
duke@435 973 int fall_through_bci = s.cur_bci();
duke@435 974 if (fall_through_bci < _method->code_size()) {
duke@435 975 assert(_methodBlocks->is_block_start(fall_through_bci), "must fall through to block start.");
duke@435 976 successors.push(_methodBlocks->block_containing(fall_through_bci));
duke@435 977 }
duke@435 978 }
duke@435 979 }
duke@435 980
duke@435 981 void BCEscapeAnalyzer::merge_block_states(StateInfo *blockstates, ciBlock *dest, StateInfo *s_state) {
kvn@480 982 StateInfo *d_state = blockstates + dest->index();
duke@435 983 int nlocals = _method->max_locals();
duke@435 984
duke@435 985 // exceptions may cause transfer of control to handlers in the middle of a
duke@435 986 // block, so we don't merge the incoming state of exception handlers
duke@435 987 if (dest->is_handler())
duke@435 988 return;
duke@435 989 if (!d_state->_initialized ) {
duke@435 990 // destination not initialized, just copy
duke@435 991 for (int i = 0; i < nlocals; i++) {
duke@435 992 d_state->_vars[i] = s_state->_vars[i];
duke@435 993 }
duke@435 994 for (int i = 0; i < s_state->_stack_height; i++) {
duke@435 995 d_state->_stack[i] = s_state->_stack[i];
duke@435 996 }
duke@435 997 d_state->_stack_height = s_state->_stack_height;
duke@435 998 d_state->_max_stack = s_state->_max_stack;
duke@435 999 d_state->_initialized = true;
duke@435 1000 } else if (!dest->processed()) {
duke@435 1001 // we have not yet walked the bytecodes of dest, we can merge
duke@435 1002 // the states
duke@435 1003 assert(d_state->_stack_height == s_state->_stack_height, "computed stack heights must match");
duke@435 1004 for (int i = 0; i < nlocals; i++) {
duke@435 1005 d_state->_vars[i].set_union(s_state->_vars[i]);
duke@435 1006 }
duke@435 1007 for (int i = 0; i < s_state->_stack_height; i++) {
duke@435 1008 d_state->_stack[i].set_union(s_state->_stack[i]);
duke@435 1009 }
duke@435 1010 } else {
duke@435 1011 // the bytecodes of dest have already been processed, mark any
duke@435 1012 // arguments in the source state which are not in the dest state
duke@435 1013 // as global escape.
duke@435 1014 // Future refinement: we only need to mark these variable to the
duke@435 1015 // maximum escape of any variables in dest state
duke@435 1016 assert(d_state->_stack_height == s_state->_stack_height, "computed stack heights must match");
duke@435 1017 ArgumentMap extra_vars;
duke@435 1018 for (int i = 0; i < nlocals; i++) {
duke@435 1019 ArgumentMap t;
duke@435 1020 t = s_state->_vars[i];
duke@435 1021 t.set_difference(d_state->_vars[i]);
duke@435 1022 extra_vars.set_union(t);
duke@435 1023 }
duke@435 1024 for (int i = 0; i < s_state->_stack_height; i++) {
duke@435 1025 ArgumentMap t;
kvn@480 1026 //extra_vars |= !d_state->_vars[i] & s_state->_vars[i];
duke@435 1027 t.clear();
duke@435 1028 t = s_state->_stack[i];
duke@435 1029 t.set_difference(d_state->_stack[i]);
duke@435 1030 extra_vars.set_union(t);
duke@435 1031 }
kvn@3318 1032 set_global_escape(extra_vars, true);
duke@435 1033 }
duke@435 1034 }
duke@435 1035
duke@435 1036 void BCEscapeAnalyzer::iterate_blocks(Arena *arena) {
duke@435 1037 int numblocks = _methodBlocks->num_blocks();
duke@435 1038 int stkSize = _method->max_stack();
duke@435 1039 int numLocals = _method->max_locals();
duke@435 1040 StateInfo state;
duke@435 1041
duke@435 1042 int datacount = (numblocks + 1) * (stkSize + numLocals);
duke@435 1043 int datasize = datacount * sizeof(ArgumentMap);
kvn@480 1044 StateInfo *blockstates = (StateInfo *) arena->Amalloc(numblocks * sizeof(StateInfo));
duke@435 1045 ArgumentMap *statedata = (ArgumentMap *) arena->Amalloc(datasize);
duke@435 1046 for (int i = 0; i < datacount; i++) ::new ((void*)&statedata[i]) ArgumentMap();
duke@435 1047 ArgumentMap *dp = statedata;
duke@435 1048 state._vars = dp;
duke@435 1049 dp += numLocals;
duke@435 1050 state._stack = dp;
duke@435 1051 dp += stkSize;
duke@435 1052 state._initialized = false;
duke@435 1053 state._max_stack = stkSize;
duke@435 1054 for (int i = 0; i < numblocks; i++) {
duke@435 1055 blockstates[i]._vars = dp;
duke@435 1056 dp += numLocals;
duke@435 1057 blockstates[i]._stack = dp;
duke@435 1058 dp += stkSize;
duke@435 1059 blockstates[i]._initialized = false;
duke@435 1060 blockstates[i]._stack_height = 0;
duke@435 1061 blockstates[i]._max_stack = stkSize;
duke@435 1062 }
duke@435 1063 GrowableArray<ciBlock *> worklist(arena, numblocks / 4, 0, NULL);
duke@435 1064 GrowableArray<ciBlock *> successors(arena, 4, 0, NULL);
duke@435 1065
duke@435 1066 _methodBlocks->clear_processed();
duke@435 1067
duke@435 1068 // initialize block 0 state from method signature
duke@435 1069 ArgumentMap allVars; // all oop arguments to method
duke@435 1070 ciSignature* sig = method()->signature();
duke@435 1071 int j = 0;
kvn@480 1072 ciBlock* first_blk = _methodBlocks->block_containing(0);
kvn@480 1073 int fb_i = first_blk->index();
duke@435 1074 if (!method()->is_static()) {
duke@435 1075 // record information for "this"
kvn@480 1076 blockstates[fb_i]._vars[j].set(j);
duke@435 1077 allVars.add(j);
duke@435 1078 j++;
duke@435 1079 }
duke@435 1080 for (int i = 0; i < sig->count(); i++) {
duke@435 1081 ciType* t = sig->type_at(i);
duke@435 1082 if (!t->is_primitive_type()) {
kvn@480 1083 blockstates[fb_i]._vars[j].set(j);
duke@435 1084 allVars.add(j);
duke@435 1085 }
duke@435 1086 j += t->size();
duke@435 1087 }
kvn@480 1088 blockstates[fb_i]._initialized = true;
duke@435 1089 assert(j == _arg_size, "just checking");
duke@435 1090
duke@435 1091 ArgumentMap unknown_map;
duke@435 1092 unknown_map.add_unknown();
duke@435 1093
kvn@480 1094 worklist.push(first_blk);
duke@435 1095 while(worklist.length() > 0) {
duke@435 1096 ciBlock *blk = worklist.pop();
kvn@480 1097 StateInfo *blkState = blockstates + blk->index();
duke@435 1098 if (blk->is_handler() || blk->is_ret_target()) {
duke@435 1099 // for an exception handler or a target of a ret instruction, we assume the worst case,
kvn@480 1100 // that any variable could contain any argument
duke@435 1101 for (int i = 0; i < numLocals; i++) {
duke@435 1102 state._vars[i] = allVars;
duke@435 1103 }
duke@435 1104 if (blk->is_handler()) {
duke@435 1105 state._stack_height = 1;
duke@435 1106 } else {
duke@435 1107 state._stack_height = blkState->_stack_height;
duke@435 1108 }
duke@435 1109 for (int i = 0; i < state._stack_height; i++) {
kvn@480 1110 // ??? should this be unknown_map ???
duke@435 1111 state._stack[i] = allVars;
duke@435 1112 }
duke@435 1113 } else {
duke@435 1114 for (int i = 0; i < numLocals; i++) {
duke@435 1115 state._vars[i] = blkState->_vars[i];
duke@435 1116 }
duke@435 1117 for (int i = 0; i < blkState->_stack_height; i++) {
duke@435 1118 state._stack[i] = blkState->_stack[i];
duke@435 1119 }
duke@435 1120 state._stack_height = blkState->_stack_height;
duke@435 1121 }
duke@435 1122 iterate_one_block(blk, state, successors);
duke@435 1123 // if this block has any exception handlers, push them
duke@435 1124 // onto successor list
duke@435 1125 if (blk->has_handler()) {
duke@435 1126 DEBUG_ONLY(int handler_count = 0;)
duke@435 1127 int blk_start = blk->start_bci();
duke@435 1128 int blk_end = blk->limit_bci();
duke@435 1129 for (int i = 0; i < numblocks; i++) {
duke@435 1130 ciBlock *b = _methodBlocks->block(i);
duke@435 1131 if (b->is_handler()) {
duke@435 1132 int ex_start = b->ex_start_bci();
duke@435 1133 int ex_end = b->ex_limit_bci();
duke@435 1134 if ((ex_start >= blk_start && ex_start < blk_end) ||
duke@435 1135 (ex_end > blk_start && ex_end <= blk_end)) {
duke@435 1136 successors.push(b);
duke@435 1137 }
duke@435 1138 DEBUG_ONLY(handler_count++;)
duke@435 1139 }
duke@435 1140 }
duke@435 1141 assert(handler_count > 0, "must find at least one handler");
duke@435 1142 }
duke@435 1143 // merge computed variable state with successors
duke@435 1144 while(successors.length() > 0) {
duke@435 1145 ciBlock *succ = successors.pop();
duke@435 1146 merge_block_states(blockstates, succ, &state);
duke@435 1147 if (!succ->processed())
duke@435 1148 worklist.push(succ);
duke@435 1149 }
duke@435 1150 }
duke@435 1151 }
duke@435 1152
duke@435 1153 bool BCEscapeAnalyzer::do_analysis() {
duke@435 1154 Arena* arena = CURRENT_ENV->arena();
duke@435 1155 // identify basic blocks
duke@435 1156 _methodBlocks = _method->get_method_blocks();
duke@435 1157
duke@435 1158 iterate_blocks(arena);
duke@435 1159 // TEMPORARY
duke@435 1160 return true;
duke@435 1161 }
duke@435 1162
duke@435 1163 vmIntrinsics::ID BCEscapeAnalyzer::known_intrinsic() {
duke@435 1164 vmIntrinsics::ID iid = method()->intrinsic_id();
duke@435 1165
duke@435 1166 if (iid == vmIntrinsics::_getClass ||
kvn@480 1167 iid == vmIntrinsics::_fillInStackTrace ||
duke@435 1168 iid == vmIntrinsics::_hashCode)
duke@435 1169 return iid;
duke@435 1170 else
duke@435 1171 return vmIntrinsics::_none;
duke@435 1172 }
duke@435 1173
duke@435 1174 bool BCEscapeAnalyzer::compute_escape_for_intrinsic(vmIntrinsics::ID iid) {
kvn@480 1175 ArgumentMap arg;
kvn@480 1176 arg.clear();
duke@435 1177 switch (iid) {
duke@435 1178 case vmIntrinsics::_getClass:
duke@435 1179 _return_local = false;
duke@435 1180 break;
kvn@480 1181 case vmIntrinsics::_fillInStackTrace:
kvn@480 1182 arg.set(0); // 'this'
kvn@480 1183 set_returned(arg);
kvn@480 1184 break;
duke@435 1185 case vmIntrinsics::_hashCode:
duke@435 1186 // initialized state is correct
duke@435 1187 break;
duke@435 1188 default:
duke@435 1189 assert(false, "unexpected intrinsic");
duke@435 1190 }
duke@435 1191 return true;
duke@435 1192 }
duke@435 1193
duke@435 1194 void BCEscapeAnalyzer::initialize() {
duke@435 1195 int i;
duke@435 1196
duke@435 1197 // clear escape information (method may have been deoptimized)
duke@435 1198 methodData()->clear_escape_info();
duke@435 1199
duke@435 1200 // initialize escape state of object parameters
duke@435 1201 ciSignature* sig = method()->signature();
duke@435 1202 int j = 0;
duke@435 1203 if (!method()->is_static()) {
kvn@2003 1204 _arg_local.set(0);
kvn@2003 1205 _arg_stack.set(0);
duke@435 1206 j++;
duke@435 1207 }
duke@435 1208 for (i = 0; i < sig->count(); i++) {
duke@435 1209 ciType* t = sig->type_at(i);
duke@435 1210 if (!t->is_primitive_type()) {
kvn@2003 1211 _arg_local.set(j);
kvn@2003 1212 _arg_stack.set(j);
duke@435 1213 }
duke@435 1214 j += t->size();
duke@435 1215 }
duke@435 1216 assert(j == _arg_size, "just checking");
duke@435 1217
duke@435 1218 // start with optimistic assumption
duke@435 1219 ciType *rt = _method->return_type();
duke@435 1220 if (rt->is_primitive_type()) {
duke@435 1221 _return_local = false;
duke@435 1222 _return_allocated = false;
duke@435 1223 } else {
duke@435 1224 _return_local = true;
duke@435 1225 _return_allocated = true;
duke@435 1226 }
duke@435 1227 _allocated_escapes = false;
kvn@480 1228 _unknown_modified = false;
duke@435 1229 }
duke@435 1230
duke@435 1231 void BCEscapeAnalyzer::clear_escape_info() {
duke@435 1232 ciSignature* sig = method()->signature();
duke@435 1233 int arg_count = sig->count();
duke@435 1234 ArgumentMap var;
kvn@480 1235 if (!method()->is_static()) {
kvn@480 1236 arg_count++; // allow for "this"
kvn@480 1237 }
duke@435 1238 for (int i = 0; i < arg_count; i++) {
kvn@480 1239 set_arg_modified(i, OFFSET_ANY, 4);
duke@435 1240 var.clear();
duke@435 1241 var.set(i);
kvn@480 1242 set_modified(var, OFFSET_ANY, 4);
duke@435 1243 set_global_escape(var);
duke@435 1244 }
kvn@2003 1245 _arg_local.Clear();
kvn@2003 1246 _arg_stack.Clear();
kvn@2003 1247 _arg_returned.Clear();
duke@435 1248 _return_local = false;
duke@435 1249 _return_allocated = false;
duke@435 1250 _allocated_escapes = true;
kvn@480 1251 _unknown_modified = true;
duke@435 1252 }
duke@435 1253
duke@435 1254
duke@435 1255 void BCEscapeAnalyzer::compute_escape_info() {
duke@435 1256 int i;
duke@435 1257 assert(!methodData()->has_escape_info(), "do not overwrite escape info");
duke@435 1258
duke@435 1259 vmIntrinsics::ID iid = known_intrinsic();
duke@435 1260
duke@435 1261 // check if method can be analyzed
duke@435 1262 if (iid == vmIntrinsics::_none && (method()->is_abstract() || method()->is_native() || !method()->holder()->is_initialized()
duke@435 1263 || _level > MaxBCEAEstimateLevel
duke@435 1264 || method()->code_size() > MaxBCEAEstimateSize)) {
duke@435 1265 if (BCEATraceLevel >= 1) {
duke@435 1266 tty->print("Skipping method because: ");
duke@435 1267 if (method()->is_abstract())
duke@435 1268 tty->print_cr("method is abstract.");
duke@435 1269 else if (method()->is_native())
duke@435 1270 tty->print_cr("method is native.");
duke@435 1271 else if (!method()->holder()->is_initialized())
duke@435 1272 tty->print_cr("class of method is not initialized.");
duke@435 1273 else if (_level > MaxBCEAEstimateLevel)
duke@435 1274 tty->print_cr("level (%d) exceeds MaxBCEAEstimateLevel (%d).",
duke@435 1275 _level, MaxBCEAEstimateLevel);
duke@435 1276 else if (method()->code_size() > MaxBCEAEstimateSize)
duke@435 1277 tty->print_cr("code size (%d) exceeds MaxBCEAEstimateSize.",
duke@435 1278 method()->code_size(), MaxBCEAEstimateSize);
duke@435 1279 else
duke@435 1280 ShouldNotReachHere();
duke@435 1281 }
duke@435 1282 clear_escape_info();
duke@435 1283
duke@435 1284 return;
duke@435 1285 }
duke@435 1286
duke@435 1287 if (BCEATraceLevel >= 1) {
duke@435 1288 tty->print("[EA] estimating escape information for");
duke@435 1289 if (iid != vmIntrinsics::_none)
duke@435 1290 tty->print(" intrinsic");
duke@435 1291 method()->print_short_name();
duke@435 1292 tty->print_cr(" (%d bytes)", method()->code_size());
duke@435 1293 }
duke@435 1294
duke@435 1295 bool success;
duke@435 1296
duke@435 1297 initialize();
duke@435 1298
kvn@513 1299 // Do not scan method if it has no object parameters and
kvn@513 1300 // does not returns an object (_return_allocated is set in initialize()).
kvn@2003 1301 if (_arg_local.Size() == 0 && !_return_allocated) {
kvn@513 1302 // Clear all info since method's bytecode was not analysed and
kvn@513 1303 // set pessimistic escape information.
kvn@513 1304 clear_escape_info();
coleenp@4037 1305 methodData()->set_eflag(MethodData::allocated_escapes);
coleenp@4037 1306 methodData()->set_eflag(MethodData::unknown_modified);
coleenp@4037 1307 methodData()->set_eflag(MethodData::estimated);
duke@435 1308 return;
duke@435 1309 }
duke@435 1310
duke@435 1311 if (iid != vmIntrinsics::_none)
duke@435 1312 success = compute_escape_for_intrinsic(iid);
duke@435 1313 else {
duke@435 1314 success = do_analysis();
duke@435 1315 }
duke@435 1316
kvn@513 1317 // don't store interprocedural escape information if it introduces
kvn@513 1318 // dependencies or if method data is empty
duke@435 1319 //
duke@435 1320 if (!has_dependencies() && !methodData()->is_empty()) {
duke@435 1321 for (i = 0; i < _arg_size; i++) {
kvn@2003 1322 if (_arg_local.test(i)) {
kvn@2003 1323 assert(_arg_stack.test(i), "inconsistent escape info");
duke@435 1324 methodData()->set_arg_local(i);
duke@435 1325 methodData()->set_arg_stack(i);
kvn@2003 1326 } else if (_arg_stack.test(i)) {
duke@435 1327 methodData()->set_arg_stack(i);
duke@435 1328 }
kvn@2003 1329 if (_arg_returned.test(i)) {
duke@435 1330 methodData()->set_arg_returned(i);
duke@435 1331 }
kvn@480 1332 methodData()->set_arg_modified(i, _arg_modified[i]);
duke@435 1333 }
duke@435 1334 if (_return_local) {
coleenp@4037 1335 methodData()->set_eflag(MethodData::return_local);
duke@435 1336 }
kvn@513 1337 if (_return_allocated) {
coleenp@4037 1338 methodData()->set_eflag(MethodData::return_allocated);
kvn@513 1339 }
kvn@513 1340 if (_allocated_escapes) {
coleenp@4037 1341 methodData()->set_eflag(MethodData::allocated_escapes);
kvn@513 1342 }
kvn@513 1343 if (_unknown_modified) {
coleenp@4037 1344 methodData()->set_eflag(MethodData::unknown_modified);
kvn@513 1345 }
coleenp@4037 1346 methodData()->set_eflag(MethodData::estimated);
duke@435 1347 }
duke@435 1348 }
duke@435 1349
duke@435 1350 void BCEscapeAnalyzer::read_escape_info() {
duke@435 1351 assert(methodData()->has_escape_info(), "no escape info available");
duke@435 1352
duke@435 1353 // read escape information from method descriptor
duke@435 1354 for (int i = 0; i < _arg_size; i++) {
kvn@2003 1355 if (methodData()->is_arg_local(i))
kvn@2003 1356 _arg_local.set(i);
kvn@2003 1357 if (methodData()->is_arg_stack(i))
kvn@2003 1358 _arg_stack.set(i);
kvn@2003 1359 if (methodData()->is_arg_returned(i))
kvn@2003 1360 _arg_returned.set(i);
kvn@480 1361 _arg_modified[i] = methodData()->arg_modified(i);
duke@435 1362 }
coleenp@4037 1363 _return_local = methodData()->eflag_set(MethodData::return_local);
coleenp@4037 1364 _return_allocated = methodData()->eflag_set(MethodData::return_allocated);
coleenp@4037 1365 _allocated_escapes = methodData()->eflag_set(MethodData::allocated_escapes);
coleenp@4037 1366 _unknown_modified = methodData()->eflag_set(MethodData::unknown_modified);
duke@435 1367
duke@435 1368 }
duke@435 1369
kvn@513 1370 #ifndef PRODUCT
kvn@513 1371 void BCEscapeAnalyzer::dump() {
kvn@513 1372 tty->print("[EA] estimated escape information for");
kvn@513 1373 method()->print_short_name();
kvn@513 1374 tty->print_cr(has_dependencies() ? " (not stored)" : "");
kvn@513 1375 tty->print(" non-escaping args: ");
kvn@513 1376 _arg_local.print_on(tty);
kvn@513 1377 tty->print(" stack-allocatable args: ");
kvn@513 1378 _arg_stack.print_on(tty);
kvn@513 1379 if (_return_local) {
kvn@513 1380 tty->print(" returned args: ");
kvn@513 1381 _arg_returned.print_on(tty);
kvn@513 1382 } else if (is_return_allocated()) {
kvn@513 1383 tty->print_cr(" return allocated value");
kvn@513 1384 } else {
kvn@513 1385 tty->print_cr(" return non-local value");
kvn@513 1386 }
kvn@513 1387 tty->print(" modified args: ");
kvn@513 1388 for (int i = 0; i < _arg_size; i++) {
kvn@513 1389 if (_arg_modified[i] == 0)
kvn@513 1390 tty->print(" 0");
kvn@513 1391 else
kvn@513 1392 tty->print(" 0x%x", _arg_modified[i]);
kvn@513 1393 }
kvn@513 1394 tty->cr();
kvn@513 1395 tty->print(" flags: ");
kvn@513 1396 if (_return_allocated)
kvn@513 1397 tty->print(" return_allocated");
kvn@513 1398 if (_allocated_escapes)
kvn@513 1399 tty->print(" allocated_escapes");
kvn@513 1400 if (_unknown_modified)
kvn@513 1401 tty->print(" unknown_modified");
kvn@513 1402 tty->cr();
kvn@513 1403 }
kvn@513 1404 #endif
duke@435 1405
duke@435 1406 BCEscapeAnalyzer::BCEscapeAnalyzer(ciMethod* method, BCEscapeAnalyzer* parent)
duke@435 1407 : _conservative(method == NULL || !EstimateArgEscape)
kvn@2003 1408 , _arena(CURRENT_ENV->arena())
duke@435 1409 , _method(method)
duke@435 1410 , _methodData(method ? method->method_data() : NULL)
duke@435 1411 , _arg_size(method ? method->arg_size() : 0)
kvn@2003 1412 , _arg_local(_arena)
kvn@2003 1413 , _arg_stack(_arena)
kvn@2003 1414 , _arg_returned(_arena)
kvn@2003 1415 , _dirty(_arena)
duke@435 1416 , _return_local(false)
duke@435 1417 , _return_allocated(false)
duke@435 1418 , _allocated_escapes(false)
kvn@480 1419 , _unknown_modified(false)
kvn@2003 1420 , _dependencies(_arena, 4, 0, NULL)
duke@435 1421 , _parent(parent)
duke@435 1422 , _level(parent == NULL ? 0 : parent->level() + 1) {
duke@435 1423 if (!_conservative) {
kvn@2003 1424 _arg_local.Clear();
kvn@2003 1425 _arg_stack.Clear();
kvn@2003 1426 _arg_returned.Clear();
kvn@2003 1427 _dirty.Clear();
duke@435 1428 Arena* arena = CURRENT_ENV->arena();
kvn@480 1429 _arg_modified = (uint *) arena->Amalloc(_arg_size * sizeof(uint));
kvn@480 1430 Copy::zero_to_bytes(_arg_modified, _arg_size * sizeof(uint));
duke@435 1431
duke@435 1432 if (methodData() == NULL)
duke@435 1433 return;
duke@435 1434 bool printit = _method->should_print_assembly();
duke@435 1435 if (methodData()->has_escape_info()) {
duke@435 1436 TRACE_BCEA(2, tty->print_cr("[EA] Reading previous results for %s.%s",
duke@435 1437 method->holder()->name()->as_utf8(),
duke@435 1438 method->name()->as_utf8()));
duke@435 1439 read_escape_info();
duke@435 1440 } else {
duke@435 1441 TRACE_BCEA(2, tty->print_cr("[EA] computing results for %s.%s",
duke@435 1442 method->holder()->name()->as_utf8(),
duke@435 1443 method->name()->as_utf8()));
duke@435 1444
duke@435 1445 compute_escape_info();
duke@435 1446 methodData()->update_escape_info();
duke@435 1447 }
kvn@513 1448 #ifndef PRODUCT
kvn@513 1449 if (BCEATraceLevel >= 3) {
kvn@513 1450 // dump escape information
kvn@513 1451 dump();
kvn@513 1452 }
kvn@513 1453 #endif
duke@435 1454 }
duke@435 1455 }
duke@435 1456
duke@435 1457 void BCEscapeAnalyzer::copy_dependencies(Dependencies *deps) {
never@1832 1458 if (ciEnv::current()->jvmti_can_hotswap_or_post_breakpoint()) {
never@1832 1459 // Also record evol dependencies so redefinition of the
never@1832 1460 // callee will trigger recompilation.
never@1832 1461 deps->assert_evol_method(method());
never@1832 1462 }
duke@435 1463 for (int i = 0; i < _dependencies.length(); i+=2) {
kvn@2003 1464 ciKlass *k = _dependencies.at(i)->as_klass();
kvn@2003 1465 ciMethod *m = _dependencies.at(i+1)->as_method();
duke@435 1466 deps->assert_unique_concrete_method(k, m);
duke@435 1467 }
duke@435 1468 }

mercurial