hotspot/src/share/vm/oops/cpCacheOop.cpp
author coleenp
Fri, 23 Mar 2012 11:16:05 -0400
changeset 12263 d20640f4f8fe
parent 11956 c363dc1ac4b7
child 13391 30245956af37
permissions -rw-r--r--
7150058: Allocate symbols from null boot loader to an arena for NMT Summary: Move symbol allocation to an arena so NMT doesn't have to track them at startup. Reviewed-by: never, kamg, zgu
Ignore whitespace changes - Everywhere: Within whitespace: At end of lines:
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
     1
/*
11956
c363dc1ac4b7 7148126: ConstantPoolCacheEntry::print prints to wrong stream
dholmes
parents: 10546
diff changeset
     2
 * Copyright (c) 1998, 2012, Oracle and/or its affiliates. All rights reserved.
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
     3
 * DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS FILE HEADER.
489c9b5090e2 Initial load
duke
parents:
diff changeset
     4
 *
489c9b5090e2 Initial load
duke
parents:
diff changeset
     5
 * This code is free software; you can redistribute it and/or modify it
489c9b5090e2 Initial load
duke
parents:
diff changeset
     6
 * under the terms of the GNU General Public License version 2 only, as
489c9b5090e2 Initial load
duke
parents:
diff changeset
     7
 * published by the Free Software Foundation.
489c9b5090e2 Initial load
duke
parents:
diff changeset
     8
 *
489c9b5090e2 Initial load
duke
parents:
diff changeset
     9
 * This code is distributed in the hope that it will be useful, but WITHOUT
489c9b5090e2 Initial load
duke
parents:
diff changeset
    10
 * ANY WARRANTY; without even the implied warranty of MERCHANTABILITY or
489c9b5090e2 Initial load
duke
parents:
diff changeset
    11
 * FITNESS FOR A PARTICULAR PURPOSE.  See the GNU General Public License
489c9b5090e2 Initial load
duke
parents:
diff changeset
    12
 * version 2 for more details (a copy is included in the LICENSE file that
489c9b5090e2 Initial load
duke
parents:
diff changeset
    13
 * accompanied this code).
489c9b5090e2 Initial load
duke
parents:
diff changeset
    14
 *
489c9b5090e2 Initial load
duke
parents:
diff changeset
    15
 * You should have received a copy of the GNU General Public License version
489c9b5090e2 Initial load
duke
parents:
diff changeset
    16
 * 2 along with this work; if not, write to the Free Software Foundation,
489c9b5090e2 Initial load
duke
parents:
diff changeset
    17
 * Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301 USA.
489c9b5090e2 Initial load
duke
parents:
diff changeset
    18
 *
5547
f4b087cbb361 6941466: Oracle rebranding changes for Hotspot repositories
trims
parents: 5420
diff changeset
    19
 * Please contact Oracle, 500 Oracle Parkway, Redwood Shores, CA 94065 USA
f4b087cbb361 6941466: Oracle rebranding changes for Hotspot repositories
trims
parents: 5420
diff changeset
    20
 * or visit www.oracle.com if you need additional information or have any
f4b087cbb361 6941466: Oracle rebranding changes for Hotspot repositories
trims
parents: 5420
diff changeset
    21
 * questions.
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
    22
 *
489c9b5090e2 Initial load
duke
parents:
diff changeset
    23
 */
489c9b5090e2 Initial load
duke
parents:
diff changeset
    24
7397
5b173b4ca846 6989984: Use standard include model for Hospot
stefank
parents: 7104
diff changeset
    25
#include "precompiled.hpp"
5b173b4ca846 6989984: Use standard include model for Hospot
stefank
parents: 7104
diff changeset
    26
#include "gc_implementation/shared/markSweep.inline.hpp"
5b173b4ca846 6989984: Use standard include model for Hospot
stefank
parents: 7104
diff changeset
    27
#include "interpreter/interpreter.hpp"
5b173b4ca846 6989984: Use standard include model for Hospot
stefank
parents: 7104
diff changeset
    28
#include "interpreter/rewriter.hpp"
5b173b4ca846 6989984: Use standard include model for Hospot
stefank
parents: 7104
diff changeset
    29
#include "memory/universe.inline.hpp"
5b173b4ca846 6989984: Use standard include model for Hospot
stefank
parents: 7104
diff changeset
    30
#include "oops/cpCacheOop.hpp"
5b173b4ca846 6989984: Use standard include model for Hospot
stefank
parents: 7104
diff changeset
    31
#include "oops/objArrayOop.hpp"
5b173b4ca846 6989984: Use standard include model for Hospot
stefank
parents: 7104
diff changeset
    32
#include "oops/oop.inline.hpp"
5b173b4ca846 6989984: Use standard include model for Hospot
stefank
parents: 7104
diff changeset
    33
#include "prims/jvmtiRedefineClassesTrace.hpp"
5b173b4ca846 6989984: Use standard include model for Hospot
stefank
parents: 7104
diff changeset
    34
#include "runtime/handles.inline.hpp"
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
    35
489c9b5090e2 Initial load
duke
parents:
diff changeset
    36
489c9b5090e2 Initial load
duke
parents:
diff changeset
    37
// Implememtation of ConstantPoolCacheEntry
489c9b5090e2 Initial load
duke
parents:
diff changeset
    38
4429
d7eb4e2099aa 6858164: invokedynamic code needs some cleanup (post-6655638)
jrose
parents: 3261
diff changeset
    39
void ConstantPoolCacheEntry::initialize_entry(int index) {
2570
ecc7862946d4 6655646: dynamic languages need dynamically linked call sites
jrose
parents: 670
diff changeset
    40
  assert(0 < index && index < 0x10000, "sanity check");
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
    41
  _indices = index;
2570
ecc7862946d4 6655646: dynamic languages need dynamically linked call sites
jrose
parents: 670
diff changeset
    42
  assert(constant_pool_index() == index, "");
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
    43
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
    44
4429
d7eb4e2099aa 6858164: invokedynamic code needs some cleanup (post-6655638)
jrose
parents: 3261
diff changeset
    45
void ConstantPoolCacheEntry::initialize_secondary_entry(int main_index) {
d7eb4e2099aa 6858164: invokedynamic code needs some cleanup (post-6655638)
jrose
parents: 3261
diff changeset
    46
  assert(0 <= main_index && main_index < 0x10000, "sanity check");
d7eb4e2099aa 6858164: invokedynamic code needs some cleanup (post-6655638)
jrose
parents: 3261
diff changeset
    47
  _indices = (main_index << 16);
d7eb4e2099aa 6858164: invokedynamic code needs some cleanup (post-6655638)
jrose
parents: 3261
diff changeset
    48
  assert(main_entry_index() == main_index, "");
d7eb4e2099aa 6858164: invokedynamic code needs some cleanup (post-6655638)
jrose
parents: 3261
diff changeset
    49
}
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
    50
489c9b5090e2 Initial load
duke
parents:
diff changeset
    51
int ConstantPoolCacheEntry::as_flags(TosState state, bool is_final,
489c9b5090e2 Initial load
duke
parents:
diff changeset
    52
                    bool is_vfinal, bool is_volatile,
489c9b5090e2 Initial load
duke
parents:
diff changeset
    53
                    bool is_method_interface, bool is_method) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
    54
  int f = state;
489c9b5090e2 Initial load
duke
parents:
diff changeset
    55
489c9b5090e2 Initial load
duke
parents:
diff changeset
    56
  assert( state < number_of_states, "Invalid state in as_flags");
489c9b5090e2 Initial load
duke
parents:
diff changeset
    57
489c9b5090e2 Initial load
duke
parents:
diff changeset
    58
  f <<= 1;
489c9b5090e2 Initial load
duke
parents:
diff changeset
    59
  if (is_final) f |= 1;
489c9b5090e2 Initial load
duke
parents:
diff changeset
    60
  f <<= 1;
489c9b5090e2 Initial load
duke
parents:
diff changeset
    61
  if (is_vfinal) f |= 1;
489c9b5090e2 Initial load
duke
parents:
diff changeset
    62
  f <<= 1;
489c9b5090e2 Initial load
duke
parents:
diff changeset
    63
  if (is_volatile) f |= 1;
489c9b5090e2 Initial load
duke
parents:
diff changeset
    64
  f <<= 1;
489c9b5090e2 Initial load
duke
parents:
diff changeset
    65
  if (is_method_interface) f |= 1;
489c9b5090e2 Initial load
duke
parents:
diff changeset
    66
  f <<= 1;
489c9b5090e2 Initial load
duke
parents:
diff changeset
    67
  if (is_method) f |= 1;
489c9b5090e2 Initial load
duke
parents:
diff changeset
    68
  f <<= ConstantPoolCacheEntry::hotSwapBit;
489c9b5090e2 Initial load
duke
parents:
diff changeset
    69
  // Preserve existing flag bit values
489c9b5090e2 Initial load
duke
parents:
diff changeset
    70
#ifdef ASSERT
489c9b5090e2 Initial load
duke
parents:
diff changeset
    71
  int old_state = ((_flags >> tosBits) & 0x0F);
489c9b5090e2 Initial load
duke
parents:
diff changeset
    72
  assert(old_state == 0 || old_state == state,
489c9b5090e2 Initial load
duke
parents:
diff changeset
    73
         "inconsistent cpCache flags state");
489c9b5090e2 Initial load
duke
parents:
diff changeset
    74
#endif
489c9b5090e2 Initial load
duke
parents:
diff changeset
    75
  return (_flags | f) ;
489c9b5090e2 Initial load
duke
parents:
diff changeset
    76
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
    77
489c9b5090e2 Initial load
duke
parents:
diff changeset
    78
void ConstantPoolCacheEntry::set_bytecode_1(Bytecodes::Code code) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
    79
#ifdef ASSERT
489c9b5090e2 Initial load
duke
parents:
diff changeset
    80
  // Read once.
489c9b5090e2 Initial load
duke
parents:
diff changeset
    81
  volatile Bytecodes::Code c = bytecode_1();
489c9b5090e2 Initial load
duke
parents:
diff changeset
    82
  assert(c == 0 || c == code || code == 0, "update must be consistent");
489c9b5090e2 Initial load
duke
parents:
diff changeset
    83
#endif
489c9b5090e2 Initial load
duke
parents:
diff changeset
    84
  // Need to flush pending stores here before bytecode is written.
489c9b5090e2 Initial load
duke
parents:
diff changeset
    85
  OrderAccess::release_store_ptr(&_indices, _indices | ((u_char)code << 16));
489c9b5090e2 Initial load
duke
parents:
diff changeset
    86
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
    87
489c9b5090e2 Initial load
duke
parents:
diff changeset
    88
void ConstantPoolCacheEntry::set_bytecode_2(Bytecodes::Code code) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
    89
#ifdef ASSERT
489c9b5090e2 Initial load
duke
parents:
diff changeset
    90
  // Read once.
489c9b5090e2 Initial load
duke
parents:
diff changeset
    91
  volatile Bytecodes::Code c = bytecode_2();
489c9b5090e2 Initial load
duke
parents:
diff changeset
    92
  assert(c == 0 || c == code || code == 0, "update must be consistent");
489c9b5090e2 Initial load
duke
parents:
diff changeset
    93
#endif
489c9b5090e2 Initial load
duke
parents:
diff changeset
    94
  // Need to flush pending stores here before bytecode is written.
489c9b5090e2 Initial load
duke
parents:
diff changeset
    95
  OrderAccess::release_store_ptr(&_indices, _indices | ((u_char)code << 24));
489c9b5090e2 Initial load
duke
parents:
diff changeset
    96
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
    97
7104
0c8b519af363 6990192: VM crashes in ciTypeFlow::get_block_for()
twisti
parents: 6064
diff changeset
    98
// Atomically sets f1 if it is still NULL, otherwise it keeps the
0c8b519af363 6990192: VM crashes in ciTypeFlow::get_block_for()
twisti
parents: 6064
diff changeset
    99
// current value.
0c8b519af363 6990192: VM crashes in ciTypeFlow::get_block_for()
twisti
parents: 6064
diff changeset
   100
void ConstantPoolCacheEntry::set_f1_if_null_atomic(oop f1) {
8316
0816329cc3ee 7012339: JSR 292 crash in G1SATBCardTableModRefBS::write_ref_field_pre_work()
twisti
parents: 7397
diff changeset
   101
  // Use barriers as in oop_store
0816329cc3ee 7012339: JSR 292 crash in G1SATBCardTableModRefBS::write_ref_field_pre_work()
twisti
parents: 7397
diff changeset
   102
  oop* f1_addr = (oop*) &_f1;
0816329cc3ee 7012339: JSR 292 crash in G1SATBCardTableModRefBS::write_ref_field_pre_work()
twisti
parents: 7397
diff changeset
   103
  update_barrier_set_pre(f1_addr, f1);
0816329cc3ee 7012339: JSR 292 crash in G1SATBCardTableModRefBS::write_ref_field_pre_work()
twisti
parents: 7397
diff changeset
   104
  void* result = Atomic::cmpxchg_ptr(f1, f1_addr, NULL);
0816329cc3ee 7012339: JSR 292 crash in G1SATBCardTableModRefBS::write_ref_field_pre_work()
twisti
parents: 7397
diff changeset
   105
  bool success = (result == NULL);
0816329cc3ee 7012339: JSR 292 crash in G1SATBCardTableModRefBS::write_ref_field_pre_work()
twisti
parents: 7397
diff changeset
   106
  if (success) {
9328
144894b3384b 6993078: JSR 292 too many pushes: Lesp points into register window
twisti
parents: 9116
diff changeset
   107
    update_barrier_set((void*) f1_addr, f1);
7104
0c8b519af363 6990192: VM crashes in ciTypeFlow::get_block_for()
twisti
parents: 6064
diff changeset
   108
  }
8316
0816329cc3ee 7012339: JSR 292 crash in G1SATBCardTableModRefBS::write_ref_field_pre_work()
twisti
parents: 7397
diff changeset
   109
}
7104
0c8b519af363 6990192: VM crashes in ciTypeFlow::get_block_for()
twisti
parents: 6064
diff changeset
   110
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   111
#ifdef ASSERT
489c9b5090e2 Initial load
duke
parents:
diff changeset
   112
// It is possible to have two different dummy methodOops created
489c9b5090e2 Initial load
duke
parents:
diff changeset
   113
// when the resolve code for invoke interface executes concurrently
489c9b5090e2 Initial load
duke
parents:
diff changeset
   114
// Hence the assertion below is weakened a bit for the invokeinterface
489c9b5090e2 Initial load
duke
parents:
diff changeset
   115
// case.
489c9b5090e2 Initial load
duke
parents:
diff changeset
   116
bool ConstantPoolCacheEntry::same_methodOop(oop cur_f1, oop f1) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   117
  return (cur_f1 == f1 || ((methodOop)cur_f1)->name() ==
489c9b5090e2 Initial load
duke
parents:
diff changeset
   118
         ((methodOop)f1)->name() || ((methodOop)cur_f1)->signature() ==
489c9b5090e2 Initial load
duke
parents:
diff changeset
   119
         ((methodOop)f1)->signature());
489c9b5090e2 Initial load
duke
parents:
diff changeset
   120
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
   121
#endif
489c9b5090e2 Initial load
duke
parents:
diff changeset
   122
489c9b5090e2 Initial load
duke
parents:
diff changeset
   123
// Note that concurrent update of both bytecodes can leave one of them
489c9b5090e2 Initial load
duke
parents:
diff changeset
   124
// reset to zero.  This is harmless; the interpreter will simply re-resolve
489c9b5090e2 Initial load
duke
parents:
diff changeset
   125
// the damaged entry.  More seriously, the memory synchronization is needed
489c9b5090e2 Initial load
duke
parents:
diff changeset
   126
// to flush other fields (f1, f2) completely to memory before the bytecodes
489c9b5090e2 Initial load
duke
parents:
diff changeset
   127
// are updated, lest other processors see a non-zero bytecode but zero f1/f2.
489c9b5090e2 Initial load
duke
parents:
diff changeset
   128
void ConstantPoolCacheEntry::set_field(Bytecodes::Code get_code,
489c9b5090e2 Initial load
duke
parents:
diff changeset
   129
                                       Bytecodes::Code put_code,
489c9b5090e2 Initial load
duke
parents:
diff changeset
   130
                                       KlassHandle field_holder,
10546
e79347eebbc5 7086585: make Java field injection more flexible
never
parents: 10008
diff changeset
   131
                                       int field_index,
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   132
                                       int field_offset,
489c9b5090e2 Initial load
duke
parents:
diff changeset
   133
                                       TosState field_type,
489c9b5090e2 Initial load
duke
parents:
diff changeset
   134
                                       bool is_final,
489c9b5090e2 Initial load
duke
parents:
diff changeset
   135
                                       bool is_volatile) {
8725
8c1e3dd5fe1b 7017732: move static fields into Class to prepare for perm gen removal
never
parents: 8334
diff changeset
   136
  set_f1(field_holder()->java_mirror());
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   137
  set_f2(field_offset);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   138
  assert(field_index <= field_index_mask,
489c9b5090e2 Initial load
duke
parents:
diff changeset
   139
         "field index does not fit in low flag bits");
489c9b5090e2 Initial load
duke
parents:
diff changeset
   140
  set_flags(as_flags(field_type, is_final, false, is_volatile, false, false) |
489c9b5090e2 Initial load
duke
parents:
diff changeset
   141
            (field_index & field_index_mask));
489c9b5090e2 Initial load
duke
parents:
diff changeset
   142
  set_bytecode_1(get_code);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   143
  set_bytecode_2(put_code);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   144
  NOT_PRODUCT(verify(tty));
489c9b5090e2 Initial load
duke
parents:
diff changeset
   145
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
   146
489c9b5090e2 Initial load
duke
parents:
diff changeset
   147
int  ConstantPoolCacheEntry::field_index() const {
10546
e79347eebbc5 7086585: make Java field injection more flexible
never
parents: 10008
diff changeset
   148
  return (_flags & field_index_mask);
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   149
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
   150
489c9b5090e2 Initial load
duke
parents:
diff changeset
   151
void ConstantPoolCacheEntry::set_method(Bytecodes::Code invoke_code,
489c9b5090e2 Initial load
duke
parents:
diff changeset
   152
                                        methodHandle method,
489c9b5090e2 Initial load
duke
parents:
diff changeset
   153
                                        int vtable_index) {
6062
bab93afe9df7 6964498: JSR 292 invokedynamic sites need local bootstrap methods
jrose
parents: 5547
diff changeset
   154
  assert(!is_secondary_entry(), "");
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   155
  assert(method->interpreter_entry() != NULL, "should have been set at this point");
489c9b5090e2 Initial load
duke
parents:
diff changeset
   156
  assert(!method->is_obsolete(),  "attempt to write obsolete method to cpCache");
489c9b5090e2 Initial load
duke
parents:
diff changeset
   157
  bool change_to_virtual = (invoke_code == Bytecodes::_invokeinterface);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   158
489c9b5090e2 Initial load
duke
parents:
diff changeset
   159
  int byte_no = -1;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   160
  bool needs_vfinal_flag = false;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   161
  switch (invoke_code) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   162
    case Bytecodes::_invokevirtual:
489c9b5090e2 Initial load
duke
parents:
diff changeset
   163
    case Bytecodes::_invokeinterface: {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   164
        if (method->can_be_statically_bound()) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   165
          set_f2((intptr_t)method());
489c9b5090e2 Initial load
duke
parents:
diff changeset
   166
          needs_vfinal_flag = true;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   167
        } else {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   168
          assert(vtable_index >= 0, "valid index");
489c9b5090e2 Initial load
duke
parents:
diff changeset
   169
          set_f2(vtable_index);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   170
        }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   171
        byte_no = 2;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   172
        break;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   173
    }
6062
bab93afe9df7 6964498: JSR 292 invokedynamic sites need local bootstrap methods
jrose
parents: 5547
diff changeset
   174
bab93afe9df7 6964498: JSR 292 invokedynamic sites need local bootstrap methods
jrose
parents: 5547
diff changeset
   175
    case Bytecodes::_invokedynamic:  // similar to _invokevirtual
bab93afe9df7 6964498: JSR 292 invokedynamic sites need local bootstrap methods
jrose
parents: 5547
diff changeset
   176
      if (TraceInvokeDynamic) {
bab93afe9df7 6964498: JSR 292 invokedynamic sites need local bootstrap methods
jrose
parents: 5547
diff changeset
   177
        tty->print_cr("InvokeDynamic set_method%s method="PTR_FORMAT" index=%d",
bab93afe9df7 6964498: JSR 292 invokedynamic sites need local bootstrap methods
jrose
parents: 5547
diff changeset
   178
                      (is_secondary_entry() ? " secondary" : ""),
bab93afe9df7 6964498: JSR 292 invokedynamic sites need local bootstrap methods
jrose
parents: 5547
diff changeset
   179
                      (intptr_t)method(), vtable_index);
bab93afe9df7 6964498: JSR 292 invokedynamic sites need local bootstrap methods
jrose
parents: 5547
diff changeset
   180
        method->print();
bab93afe9df7 6964498: JSR 292 invokedynamic sites need local bootstrap methods
jrose
parents: 5547
diff changeset
   181
        this->print(tty, 0);
bab93afe9df7 6964498: JSR 292 invokedynamic sites need local bootstrap methods
jrose
parents: 5547
diff changeset
   182
      }
bab93afe9df7 6964498: JSR 292 invokedynamic sites need local bootstrap methods
jrose
parents: 5547
diff changeset
   183
      assert(method->can_be_statically_bound(), "must be a MH invoker method");
9116
9bc44be338d6 6981791: remove experimental code for JSR 292
jrose
parents: 8725
diff changeset
   184
      assert(_f2 >= constantPoolOopDesc::CPCACHE_INDEX_TAG, "BSM index initialized");
7104
0c8b519af363 6990192: VM crashes in ciTypeFlow::get_block_for()
twisti
parents: 6064
diff changeset
   185
      // SystemDictionary::find_method_handle_invoke only caches
0c8b519af363 6990192: VM crashes in ciTypeFlow::get_block_for()
twisti
parents: 6064
diff changeset
   186
      // methods which signature classes are on the boot classpath,
0c8b519af363 6990192: VM crashes in ciTypeFlow::get_block_for()
twisti
parents: 6064
diff changeset
   187
      // otherwise the newly created method is returned.  To avoid
0c8b519af363 6990192: VM crashes in ciTypeFlow::get_block_for()
twisti
parents: 6064
diff changeset
   188
      // races in that case we store the first one coming in into the
0c8b519af363 6990192: VM crashes in ciTypeFlow::get_block_for()
twisti
parents: 6064
diff changeset
   189
      // cp-cache atomically if it's still unset.
0c8b519af363 6990192: VM crashes in ciTypeFlow::get_block_for()
twisti
parents: 6064
diff changeset
   190
      set_f1_if_null_atomic(method());
6062
bab93afe9df7 6964498: JSR 292 invokedynamic sites need local bootstrap methods
jrose
parents: 5547
diff changeset
   191
      needs_vfinal_flag = false;  // _f2 is not an oop
bab93afe9df7 6964498: JSR 292 invokedynamic sites need local bootstrap methods
jrose
parents: 5547
diff changeset
   192
      assert(!is_vfinal(), "f2 not an oop");
6064
71e316283a85 6969574: invokedynamic call sites deoptimize instead of executing
jrose
parents: 6062
diff changeset
   193
      byte_no = 1;  // coordinate this with bytecode_number & is_resolved
6062
bab93afe9df7 6964498: JSR 292 invokedynamic sites need local bootstrap methods
jrose
parents: 5547
diff changeset
   194
      break;
bab93afe9df7 6964498: JSR 292 invokedynamic sites need local bootstrap methods
jrose
parents: 5547
diff changeset
   195
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   196
    case Bytecodes::_invokespecial:
489c9b5090e2 Initial load
duke
parents:
diff changeset
   197
      // Preserve the value of the vfinal flag on invokevirtual bytecode
489c9b5090e2 Initial load
duke
parents:
diff changeset
   198
      // which may be shared with this constant pool cache entry.
489c9b5090e2 Initial load
duke
parents:
diff changeset
   199
      needs_vfinal_flag = is_resolved(Bytecodes::_invokevirtual) && is_vfinal();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   200
      // fall through
489c9b5090e2 Initial load
duke
parents:
diff changeset
   201
    case Bytecodes::_invokestatic:
489c9b5090e2 Initial load
duke
parents:
diff changeset
   202
      set_f1(method());
489c9b5090e2 Initial load
duke
parents:
diff changeset
   203
      byte_no = 1;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   204
      break;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   205
    default:
489c9b5090e2 Initial load
duke
parents:
diff changeset
   206
      ShouldNotReachHere();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   207
      break;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   208
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   209
489c9b5090e2 Initial load
duke
parents:
diff changeset
   210
  set_flags(as_flags(as_TosState(method->result_type()),
489c9b5090e2 Initial load
duke
parents:
diff changeset
   211
                     method->is_final_method(),
489c9b5090e2 Initial load
duke
parents:
diff changeset
   212
                     needs_vfinal_flag,
489c9b5090e2 Initial load
duke
parents:
diff changeset
   213
                     false,
489c9b5090e2 Initial load
duke
parents:
diff changeset
   214
                     change_to_virtual,
489c9b5090e2 Initial load
duke
parents:
diff changeset
   215
                     true)|
489c9b5090e2 Initial load
duke
parents:
diff changeset
   216
            method()->size_of_parameters());
489c9b5090e2 Initial load
duke
parents:
diff changeset
   217
489c9b5090e2 Initial load
duke
parents:
diff changeset
   218
  // Note:  byte_no also appears in TemplateTable::resolve.
489c9b5090e2 Initial load
duke
parents:
diff changeset
   219
  if (byte_no == 1) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   220
    set_bytecode_1(invoke_code);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   221
  } else if (byte_no == 2)  {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   222
    if (change_to_virtual) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   223
      // NOTE: THIS IS A HACK - BE VERY CAREFUL!!!
489c9b5090e2 Initial load
duke
parents:
diff changeset
   224
      //
489c9b5090e2 Initial load
duke
parents:
diff changeset
   225
      // Workaround for the case where we encounter an invokeinterface, but we
489c9b5090e2 Initial load
duke
parents:
diff changeset
   226
      // should really have an _invokevirtual since the resolved method is a
489c9b5090e2 Initial load
duke
parents:
diff changeset
   227
      // virtual method in java.lang.Object. This is a corner case in the spec
489c9b5090e2 Initial load
duke
parents:
diff changeset
   228
      // but is presumably legal. javac does not generate this code.
489c9b5090e2 Initial load
duke
parents:
diff changeset
   229
      //
489c9b5090e2 Initial load
duke
parents:
diff changeset
   230
      // We set bytecode_1() to _invokeinterface, because that is the
489c9b5090e2 Initial load
duke
parents:
diff changeset
   231
      // bytecode # used by the interpreter to see if it is resolved.
489c9b5090e2 Initial load
duke
parents:
diff changeset
   232
      // We set bytecode_2() to _invokevirtual.
489c9b5090e2 Initial load
duke
parents:
diff changeset
   233
      // See also interpreterRuntime.cpp. (8/25/2000)
489c9b5090e2 Initial load
duke
parents:
diff changeset
   234
      // Only set resolved for the invokeinterface case if method is public.
489c9b5090e2 Initial load
duke
parents:
diff changeset
   235
      // Otherwise, the method needs to be reresolved with caller for each
489c9b5090e2 Initial load
duke
parents:
diff changeset
   236
      // interface call.
489c9b5090e2 Initial load
duke
parents:
diff changeset
   237
      if (method->is_public()) set_bytecode_1(invoke_code);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   238
      set_bytecode_2(Bytecodes::_invokevirtual);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   239
    } else {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   240
      set_bytecode_2(invoke_code);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   241
    }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   242
  } else {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   243
    ShouldNotReachHere();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   244
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   245
  NOT_PRODUCT(verify(tty));
489c9b5090e2 Initial load
duke
parents:
diff changeset
   246
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
   247
489c9b5090e2 Initial load
duke
parents:
diff changeset
   248
489c9b5090e2 Initial load
duke
parents:
diff changeset
   249
void ConstantPoolCacheEntry::set_interface_call(methodHandle method, int index) {
6062
bab93afe9df7 6964498: JSR 292 invokedynamic sites need local bootstrap methods
jrose
parents: 5547
diff changeset
   250
  assert(!is_secondary_entry(), "");
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   251
  klassOop interf = method->method_holder();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   252
  assert(instanceKlass::cast(interf)->is_interface(), "must be an interface");
489c9b5090e2 Initial load
duke
parents:
diff changeset
   253
  set_f1(interf);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   254
  set_f2(index);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   255
  set_flags(as_flags(as_TosState(method->result_type()), method->is_final_method(), false, false, false, true) | method()->size_of_parameters());
489c9b5090e2 Initial load
duke
parents:
diff changeset
   256
  set_bytecode_1(Bytecodes::_invokeinterface);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   257
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
   258
489c9b5090e2 Initial load
duke
parents:
diff changeset
   259
6062
bab93afe9df7 6964498: JSR 292 invokedynamic sites need local bootstrap methods
jrose
parents: 5547
diff changeset
   260
void ConstantPoolCacheEntry::initialize_bootstrap_method_index_in_cache(int bsm_cache_index) {
bab93afe9df7 6964498: JSR 292 invokedynamic sites need local bootstrap methods
jrose
parents: 5547
diff changeset
   261
  assert(!is_secondary_entry(), "only for JVM_CONSTANT_InvokeDynamic main entry");
bab93afe9df7 6964498: JSR 292 invokedynamic sites need local bootstrap methods
jrose
parents: 5547
diff changeset
   262
  assert(_f2 == 0, "initialize once");
bab93afe9df7 6964498: JSR 292 invokedynamic sites need local bootstrap methods
jrose
parents: 5547
diff changeset
   263
  assert(bsm_cache_index == (int)(u2)bsm_cache_index, "oob");
bab93afe9df7 6964498: JSR 292 invokedynamic sites need local bootstrap methods
jrose
parents: 5547
diff changeset
   264
  set_f2(bsm_cache_index + constantPoolOopDesc::CPCACHE_INDEX_TAG);
bab93afe9df7 6964498: JSR 292 invokedynamic sites need local bootstrap methods
jrose
parents: 5547
diff changeset
   265
}
bab93afe9df7 6964498: JSR 292 invokedynamic sites need local bootstrap methods
jrose
parents: 5547
diff changeset
   266
bab93afe9df7 6964498: JSR 292 invokedynamic sites need local bootstrap methods
jrose
parents: 5547
diff changeset
   267
int ConstantPoolCacheEntry::bootstrap_method_index_in_cache() {
bab93afe9df7 6964498: JSR 292 invokedynamic sites need local bootstrap methods
jrose
parents: 5547
diff changeset
   268
  assert(!is_secondary_entry(), "only for JVM_CONSTANT_InvokeDynamic main entry");
bab93afe9df7 6964498: JSR 292 invokedynamic sites need local bootstrap methods
jrose
parents: 5547
diff changeset
   269
  intptr_t bsm_cache_index = (intptr_t) _f2 - constantPoolOopDesc::CPCACHE_INDEX_TAG;
bab93afe9df7 6964498: JSR 292 invokedynamic sites need local bootstrap methods
jrose
parents: 5547
diff changeset
   270
  assert(bsm_cache_index == (intptr_t)(u2)bsm_cache_index, "oob");
bab93afe9df7 6964498: JSR 292 invokedynamic sites need local bootstrap methods
jrose
parents: 5547
diff changeset
   271
  return (int) bsm_cache_index;
bab93afe9df7 6964498: JSR 292 invokedynamic sites need local bootstrap methods
jrose
parents: 5547
diff changeset
   272
}
bab93afe9df7 6964498: JSR 292 invokedynamic sites need local bootstrap methods
jrose
parents: 5547
diff changeset
   273
9328
144894b3384b 6993078: JSR 292 too many pushes: Lesp points into register window
twisti
parents: 9116
diff changeset
   274
void ConstantPoolCacheEntry::set_dynamic_call(Handle call_site, methodHandle signature_invoker) {
6062
bab93afe9df7 6964498: JSR 292 invokedynamic sites need local bootstrap methods
jrose
parents: 5547
diff changeset
   275
  assert(is_secondary_entry(), "");
9328
144894b3384b 6993078: JSR 292 too many pushes: Lesp points into register window
twisti
parents: 9116
diff changeset
   276
  // NOTE: it's important that all other values are set before f1 is
144894b3384b 6993078: JSR 292 too many pushes: Lesp points into register window
twisti
parents: 9116
diff changeset
   277
  // set since some users short circuit on f1 being set
144894b3384b 6993078: JSR 292 too many pushes: Lesp points into register window
twisti
parents: 9116
diff changeset
   278
  // (i.e. non-null) and that may result in uninitialized values for
144894b3384b 6993078: JSR 292 too many pushes: Lesp points into register window
twisti
parents: 9116
diff changeset
   279
  // other racing threads (e.g. flags).
5420
586d3988e72b 6939134: JSR 292 adjustments to method handle invocation
jrose
parents: 4429
diff changeset
   280
  int param_size = signature_invoker->size_of_parameters();
4429
d7eb4e2099aa 6858164: invokedynamic code needs some cleanup (post-6655638)
jrose
parents: 3261
diff changeset
   281
  assert(param_size >= 1, "method argument size must include MH.this");
9328
144894b3384b 6993078: JSR 292 too many pushes: Lesp points into register window
twisti
parents: 9116
diff changeset
   282
  param_size -= 1;  // do not count MH.this; it is not stacked for invokedynamic
5420
586d3988e72b 6939134: JSR 292 adjustments to method handle invocation
jrose
parents: 4429
diff changeset
   283
  bool is_final = true;
586d3988e72b 6939134: JSR 292 adjustments to method handle invocation
jrose
parents: 4429
diff changeset
   284
  assert(signature_invoker->is_final_method(), "is_final");
9328
144894b3384b 6993078: JSR 292 too many pushes: Lesp points into register window
twisti
parents: 9116
diff changeset
   285
  int flags = as_flags(as_TosState(signature_invoker->result_type()), is_final, false, false, false, true) | param_size;
144894b3384b 6993078: JSR 292 too many pushes: Lesp points into register window
twisti
parents: 9116
diff changeset
   286
  assert(_flags == 0 || _flags == flags, "flags should be the same");
144894b3384b 6993078: JSR 292 too many pushes: Lesp points into register window
twisti
parents: 9116
diff changeset
   287
  set_flags(flags);
2570
ecc7862946d4 6655646: dynamic languages need dynamically linked call sites
jrose
parents: 670
diff changeset
   288
  // do not do set_bytecode on a secondary CP cache entry
ecc7862946d4 6655646: dynamic languages need dynamically linked call sites
jrose
parents: 670
diff changeset
   289
  //set_bytecode_1(Bytecodes::_invokedynamic);
9328
144894b3384b 6993078: JSR 292 too many pushes: Lesp points into register window
twisti
parents: 9116
diff changeset
   290
  set_f1_if_null_atomic(call_site());  // This must be the last one to set (see NOTE above)!
2570
ecc7862946d4 6655646: dynamic languages need dynamically linked call sites
jrose
parents: 670
diff changeset
   291
}
ecc7862946d4 6655646: dynamic languages need dynamically linked call sites
jrose
parents: 670
diff changeset
   292
ecc7862946d4 6655646: dynamic languages need dynamically linked call sites
jrose
parents: 670
diff changeset
   293
10008
d84de97ad847 7056328: JSR 292 invocation sometimes fails in adapters for types not on boot class path
jrose
parents: 9328
diff changeset
   294
methodOop ConstantPoolCacheEntry::get_method_if_resolved(Bytecodes::Code invoke_code, constantPoolHandle cpool) {
d84de97ad847 7056328: JSR 292 invocation sometimes fails in adapters for types not on boot class path
jrose
parents: 9328
diff changeset
   295
  assert(invoke_code > (Bytecodes::Code)0, "bad query");
d84de97ad847 7056328: JSR 292 invocation sometimes fails in adapters for types not on boot class path
jrose
parents: 9328
diff changeset
   296
  if (is_secondary_entry()) {
d84de97ad847 7056328: JSR 292 invocation sometimes fails in adapters for types not on boot class path
jrose
parents: 9328
diff changeset
   297
    return cpool->cache()->entry_at(main_entry_index())->get_method_if_resolved(invoke_code, cpool);
d84de97ad847 7056328: JSR 292 invocation sometimes fails in adapters for types not on boot class path
jrose
parents: 9328
diff changeset
   298
  }
d84de97ad847 7056328: JSR 292 invocation sometimes fails in adapters for types not on boot class path
jrose
parents: 9328
diff changeset
   299
  // Decode the action of set_method and set_interface_call
d84de97ad847 7056328: JSR 292 invocation sometimes fails in adapters for types not on boot class path
jrose
parents: 9328
diff changeset
   300
  if (bytecode_1() == invoke_code) {
d84de97ad847 7056328: JSR 292 invocation sometimes fails in adapters for types not on boot class path
jrose
parents: 9328
diff changeset
   301
    oop f1 = _f1;
d84de97ad847 7056328: JSR 292 invocation sometimes fails in adapters for types not on boot class path
jrose
parents: 9328
diff changeset
   302
    if (f1 != NULL) {
d84de97ad847 7056328: JSR 292 invocation sometimes fails in adapters for types not on boot class path
jrose
parents: 9328
diff changeset
   303
      switch (invoke_code) {
d84de97ad847 7056328: JSR 292 invocation sometimes fails in adapters for types not on boot class path
jrose
parents: 9328
diff changeset
   304
      case Bytecodes::_invokeinterface:
d84de97ad847 7056328: JSR 292 invocation sometimes fails in adapters for types not on boot class path
jrose
parents: 9328
diff changeset
   305
        assert(f1->is_klass(), "");
d84de97ad847 7056328: JSR 292 invocation sometimes fails in adapters for types not on boot class path
jrose
parents: 9328
diff changeset
   306
        return klassItable::method_for_itable_index(klassOop(f1), (int) f2());
d84de97ad847 7056328: JSR 292 invocation sometimes fails in adapters for types not on boot class path
jrose
parents: 9328
diff changeset
   307
      case Bytecodes::_invokestatic:
d84de97ad847 7056328: JSR 292 invocation sometimes fails in adapters for types not on boot class path
jrose
parents: 9328
diff changeset
   308
      case Bytecodes::_invokespecial:
d84de97ad847 7056328: JSR 292 invocation sometimes fails in adapters for types not on boot class path
jrose
parents: 9328
diff changeset
   309
        assert(f1->is_method(), "");
d84de97ad847 7056328: JSR 292 invocation sometimes fails in adapters for types not on boot class path
jrose
parents: 9328
diff changeset
   310
        return methodOop(f1);
d84de97ad847 7056328: JSR 292 invocation sometimes fails in adapters for types not on boot class path
jrose
parents: 9328
diff changeset
   311
      }
d84de97ad847 7056328: JSR 292 invocation sometimes fails in adapters for types not on boot class path
jrose
parents: 9328
diff changeset
   312
    }
d84de97ad847 7056328: JSR 292 invocation sometimes fails in adapters for types not on boot class path
jrose
parents: 9328
diff changeset
   313
  }
d84de97ad847 7056328: JSR 292 invocation sometimes fails in adapters for types not on boot class path
jrose
parents: 9328
diff changeset
   314
  if (bytecode_2() == invoke_code) {
d84de97ad847 7056328: JSR 292 invocation sometimes fails in adapters for types not on boot class path
jrose
parents: 9328
diff changeset
   315
    switch (invoke_code) {
d84de97ad847 7056328: JSR 292 invocation sometimes fails in adapters for types not on boot class path
jrose
parents: 9328
diff changeset
   316
    case Bytecodes::_invokevirtual:
d84de97ad847 7056328: JSR 292 invocation sometimes fails in adapters for types not on boot class path
jrose
parents: 9328
diff changeset
   317
      if (is_vfinal()) {
d84de97ad847 7056328: JSR 292 invocation sometimes fails in adapters for types not on boot class path
jrose
parents: 9328
diff changeset
   318
        // invokevirtual
d84de97ad847 7056328: JSR 292 invocation sometimes fails in adapters for types not on boot class path
jrose
parents: 9328
diff changeset
   319
        methodOop m = methodOop((intptr_t) f2());
d84de97ad847 7056328: JSR 292 invocation sometimes fails in adapters for types not on boot class path
jrose
parents: 9328
diff changeset
   320
        assert(m->is_method(), "");
d84de97ad847 7056328: JSR 292 invocation sometimes fails in adapters for types not on boot class path
jrose
parents: 9328
diff changeset
   321
        return m;
d84de97ad847 7056328: JSR 292 invocation sometimes fails in adapters for types not on boot class path
jrose
parents: 9328
diff changeset
   322
      } else {
d84de97ad847 7056328: JSR 292 invocation sometimes fails in adapters for types not on boot class path
jrose
parents: 9328
diff changeset
   323
        int holder_index = cpool->uncached_klass_ref_index_at(constant_pool_index());
d84de97ad847 7056328: JSR 292 invocation sometimes fails in adapters for types not on boot class path
jrose
parents: 9328
diff changeset
   324
        if (cpool->tag_at(holder_index).is_klass()) {
d84de97ad847 7056328: JSR 292 invocation sometimes fails in adapters for types not on boot class path
jrose
parents: 9328
diff changeset
   325
          klassOop klass = cpool->resolved_klass_at(holder_index);
d84de97ad847 7056328: JSR 292 invocation sometimes fails in adapters for types not on boot class path
jrose
parents: 9328
diff changeset
   326
          if (!Klass::cast(klass)->oop_is_instance())
d84de97ad847 7056328: JSR 292 invocation sometimes fails in adapters for types not on boot class path
jrose
parents: 9328
diff changeset
   327
            klass = SystemDictionary::Object_klass();
d84de97ad847 7056328: JSR 292 invocation sometimes fails in adapters for types not on boot class path
jrose
parents: 9328
diff changeset
   328
          return instanceKlass::cast(klass)->method_at_vtable((int) f2());
d84de97ad847 7056328: JSR 292 invocation sometimes fails in adapters for types not on boot class path
jrose
parents: 9328
diff changeset
   329
        }
d84de97ad847 7056328: JSR 292 invocation sometimes fails in adapters for types not on boot class path
jrose
parents: 9328
diff changeset
   330
      }
d84de97ad847 7056328: JSR 292 invocation sometimes fails in adapters for types not on boot class path
jrose
parents: 9328
diff changeset
   331
    }
d84de97ad847 7056328: JSR 292 invocation sometimes fails in adapters for types not on boot class path
jrose
parents: 9328
diff changeset
   332
  }
d84de97ad847 7056328: JSR 292 invocation sometimes fails in adapters for types not on boot class path
jrose
parents: 9328
diff changeset
   333
  return NULL;
d84de97ad847 7056328: JSR 292 invocation sometimes fails in adapters for types not on boot class path
jrose
parents: 9328
diff changeset
   334
}
d84de97ad847 7056328: JSR 292 invocation sometimes fails in adapters for types not on boot class path
jrose
parents: 9328
diff changeset
   335
d84de97ad847 7056328: JSR 292 invocation sometimes fails in adapters for types not on boot class path
jrose
parents: 9328
diff changeset
   336
d84de97ad847 7056328: JSR 292 invocation sometimes fails in adapters for types not on boot class path
jrose
parents: 9328
diff changeset
   337
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   338
class LocalOopClosure: public OopClosure {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   339
 private:
489c9b5090e2 Initial load
duke
parents:
diff changeset
   340
  void (*_f)(oop*);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   341
489c9b5090e2 Initial load
duke
parents:
diff changeset
   342
 public:
489c9b5090e2 Initial load
duke
parents:
diff changeset
   343
  LocalOopClosure(void f(oop*))        { _f = f; }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   344
  virtual void do_oop(oop* o)          { _f(o); }
360
21d113ecbf6a 6420645: Create a vm that uses compressed oops for up to 32gb heapsizes
coleenp
parents: 1
diff changeset
   345
  virtual void do_oop(narrowOop *o)    { ShouldNotReachHere(); }
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   346
};
489c9b5090e2 Initial load
duke
parents:
diff changeset
   347
489c9b5090e2 Initial load
duke
parents:
diff changeset
   348
489c9b5090e2 Initial load
duke
parents:
diff changeset
   349
void ConstantPoolCacheEntry::oops_do(void f(oop*)) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   350
  LocalOopClosure blk(f);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   351
  oop_iterate(&blk);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   352
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
   353
489c9b5090e2 Initial load
duke
parents:
diff changeset
   354
489c9b5090e2 Initial load
duke
parents:
diff changeset
   355
void ConstantPoolCacheEntry::oop_iterate(OopClosure* blk) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   356
  assert(in_words(size()) == 4, "check code below - may need adjustment");
489c9b5090e2 Initial load
duke
parents:
diff changeset
   357
  // field[1] is always oop or NULL
489c9b5090e2 Initial load
duke
parents:
diff changeset
   358
  blk->do_oop((oop*)&_f1);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   359
  if (is_vfinal()) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   360
    blk->do_oop((oop*)&_f2);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   361
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   362
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
   363
489c9b5090e2 Initial load
duke
parents:
diff changeset
   364
489c9b5090e2 Initial load
duke
parents:
diff changeset
   365
void ConstantPoolCacheEntry::oop_iterate_m(OopClosure* blk, MemRegion mr) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   366
  assert(in_words(size()) == 4, "check code below - may need adjustment");
489c9b5090e2 Initial load
duke
parents:
diff changeset
   367
  // field[1] is always oop or NULL
489c9b5090e2 Initial load
duke
parents:
diff changeset
   368
  if (mr.contains((oop *)&_f1)) blk->do_oop((oop*)&_f1);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   369
  if (is_vfinal()) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   370
    if (mr.contains((oop *)&_f2)) blk->do_oop((oop*)&_f2);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   371
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   372
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
   373
489c9b5090e2 Initial load
duke
parents:
diff changeset
   374
489c9b5090e2 Initial load
duke
parents:
diff changeset
   375
void ConstantPoolCacheEntry::follow_contents() {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   376
  assert(in_words(size()) == 4, "check code below - may need adjustment");
489c9b5090e2 Initial load
duke
parents:
diff changeset
   377
  // field[1] is always oop or NULL
489c9b5090e2 Initial load
duke
parents:
diff changeset
   378
  MarkSweep::mark_and_push((oop*)&_f1);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   379
  if (is_vfinal()) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   380
    MarkSweep::mark_and_push((oop*)&_f2);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   381
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   382
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
   383
489c9b5090e2 Initial load
duke
parents:
diff changeset
   384
#ifndef SERIALGC
489c9b5090e2 Initial load
duke
parents:
diff changeset
   385
void ConstantPoolCacheEntry::follow_contents(ParCompactionManager* cm) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   386
  assert(in_words(size()) == 4, "check code below - may need adjustment");
489c9b5090e2 Initial load
duke
parents:
diff changeset
   387
  // field[1] is always oop or NULL
489c9b5090e2 Initial load
duke
parents:
diff changeset
   388
  PSParallelCompact::mark_and_push(cm, (oop*)&_f1);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   389
  if (is_vfinal()) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   390
    PSParallelCompact::mark_and_push(cm, (oop*)&_f2);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   391
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   392
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
   393
#endif // SERIALGC
489c9b5090e2 Initial load
duke
parents:
diff changeset
   394
489c9b5090e2 Initial load
duke
parents:
diff changeset
   395
void ConstantPoolCacheEntry::adjust_pointers() {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   396
  assert(in_words(size()) == 4, "check code below - may need adjustment");
489c9b5090e2 Initial load
duke
parents:
diff changeset
   397
  // field[1] is always oop or NULL
489c9b5090e2 Initial load
duke
parents:
diff changeset
   398
  MarkSweep::adjust_pointer((oop*)&_f1);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   399
  if (is_vfinal()) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   400
    MarkSweep::adjust_pointer((oop*)&_f2);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   401
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   402
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
   403
489c9b5090e2 Initial load
duke
parents:
diff changeset
   404
#ifndef SERIALGC
489c9b5090e2 Initial load
duke
parents:
diff changeset
   405
void ConstantPoolCacheEntry::update_pointers() {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   406
  assert(in_words(size()) == 4, "check code below - may need adjustment");
489c9b5090e2 Initial load
duke
parents:
diff changeset
   407
  // field[1] is always oop or NULL
489c9b5090e2 Initial load
duke
parents:
diff changeset
   408
  PSParallelCompact::adjust_pointer((oop*)&_f1);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   409
  if (is_vfinal()) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   410
    PSParallelCompact::adjust_pointer((oop*)&_f2);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   411
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   412
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
   413
#endif // SERIALGC
489c9b5090e2 Initial load
duke
parents:
diff changeset
   414
489c9b5090e2 Initial load
duke
parents:
diff changeset
   415
// RedefineClasses() API support:
489c9b5090e2 Initial load
duke
parents:
diff changeset
   416
// If this constantPoolCacheEntry refers to old_method then update it
489c9b5090e2 Initial load
duke
parents:
diff changeset
   417
// to refer to new_method.
489c9b5090e2 Initial load
duke
parents:
diff changeset
   418
bool ConstantPoolCacheEntry::adjust_method_entry(methodOop old_method,
489c9b5090e2 Initial load
duke
parents:
diff changeset
   419
       methodOop new_method, bool * trace_name_printed) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   420
489c9b5090e2 Initial load
duke
parents:
diff changeset
   421
  if (is_vfinal()) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   422
    // virtual and final so f2() contains method ptr instead of vtable index
489c9b5090e2 Initial load
duke
parents:
diff changeset
   423
    if (f2() == (intptr_t)old_method) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   424
      // match old_method so need an update
489c9b5090e2 Initial load
duke
parents:
diff changeset
   425
      _f2 = (intptr_t)new_method;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   426
      if (RC_TRACE_IN_RANGE(0x00100000, 0x00400000)) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   427
        if (!(*trace_name_printed)) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   428
          // RC_TRACE_MESG macro has an embedded ResourceMark
489c9b5090e2 Initial load
duke
parents:
diff changeset
   429
          RC_TRACE_MESG(("adjust: name=%s",
489c9b5090e2 Initial load
duke
parents:
diff changeset
   430
            Klass::cast(old_method->method_holder())->external_name()));
489c9b5090e2 Initial load
duke
parents:
diff changeset
   431
          *trace_name_printed = true;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   432
        }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   433
        // RC_TRACE macro has an embedded ResourceMark
489c9b5090e2 Initial load
duke
parents:
diff changeset
   434
        RC_TRACE(0x00400000, ("cpc vf-entry update: %s(%s)",
489c9b5090e2 Initial load
duke
parents:
diff changeset
   435
          new_method->name()->as_C_string(),
489c9b5090e2 Initial load
duke
parents:
diff changeset
   436
          new_method->signature()->as_C_string()));
489c9b5090e2 Initial load
duke
parents:
diff changeset
   437
      }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   438
489c9b5090e2 Initial load
duke
parents:
diff changeset
   439
      return true;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   440
    }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   441
489c9b5090e2 Initial load
duke
parents:
diff changeset
   442
    // f1() is not used with virtual entries so bail out
489c9b5090e2 Initial load
duke
parents:
diff changeset
   443
    return false;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   444
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   445
489c9b5090e2 Initial load
duke
parents:
diff changeset
   446
  if ((oop)_f1 == NULL) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   447
    // NULL f1() means this is a virtual entry so bail out
489c9b5090e2 Initial load
duke
parents:
diff changeset
   448
    // We are assuming that the vtable index does not need change.
489c9b5090e2 Initial load
duke
parents:
diff changeset
   449
    return false;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   450
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   451
489c9b5090e2 Initial load
duke
parents:
diff changeset
   452
  if ((oop)_f1 == old_method) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   453
    _f1 = new_method;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   454
    if (RC_TRACE_IN_RANGE(0x00100000, 0x00400000)) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   455
      if (!(*trace_name_printed)) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   456
        // RC_TRACE_MESG macro has an embedded ResourceMark
489c9b5090e2 Initial load
duke
parents:
diff changeset
   457
        RC_TRACE_MESG(("adjust: name=%s",
489c9b5090e2 Initial load
duke
parents:
diff changeset
   458
          Klass::cast(old_method->method_holder())->external_name()));
489c9b5090e2 Initial load
duke
parents:
diff changeset
   459
        *trace_name_printed = true;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   460
      }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   461
      // RC_TRACE macro has an embedded ResourceMark
489c9b5090e2 Initial load
duke
parents:
diff changeset
   462
      RC_TRACE(0x00400000, ("cpc entry update: %s(%s)",
489c9b5090e2 Initial load
duke
parents:
diff changeset
   463
        new_method->name()->as_C_string(),
489c9b5090e2 Initial load
duke
parents:
diff changeset
   464
        new_method->signature()->as_C_string()));
489c9b5090e2 Initial load
duke
parents:
diff changeset
   465
    }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   466
489c9b5090e2 Initial load
duke
parents:
diff changeset
   467
    return true;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   468
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   469
489c9b5090e2 Initial load
duke
parents:
diff changeset
   470
  return false;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   471
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
   472
489c9b5090e2 Initial load
duke
parents:
diff changeset
   473
bool ConstantPoolCacheEntry::is_interesting_method_entry(klassOop k) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   474
  if (!is_method_entry()) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   475
    // not a method entry so not interesting by default
489c9b5090e2 Initial load
duke
parents:
diff changeset
   476
    return false;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   477
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   478
489c9b5090e2 Initial load
duke
parents:
diff changeset
   479
  methodOop m = NULL;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   480
  if (is_vfinal()) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   481
    // virtual and final so _f2 contains method ptr instead of vtable index
489c9b5090e2 Initial load
duke
parents:
diff changeset
   482
    m = (methodOop)_f2;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   483
  } else if ((oop)_f1 == NULL) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   484
    // NULL _f1 means this is a virtual entry so also not interesting
489c9b5090e2 Initial load
duke
parents:
diff changeset
   485
    return false;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   486
  } else {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   487
    if (!((oop)_f1)->is_method()) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   488
      // _f1 can also contain a klassOop for an interface
489c9b5090e2 Initial load
duke
parents:
diff changeset
   489
      return false;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   490
    }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   491
    m = (methodOop)_f1;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   492
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   493
489c9b5090e2 Initial load
duke
parents:
diff changeset
   494
  assert(m != NULL && m->is_method(), "sanity check");
489c9b5090e2 Initial load
duke
parents:
diff changeset
   495
  if (m == NULL || !m->is_method() || m->method_holder() != k) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   496
    // robustness for above sanity checks or method is not in
489c9b5090e2 Initial load
duke
parents:
diff changeset
   497
    // the interesting class
489c9b5090e2 Initial load
duke
parents:
diff changeset
   498
    return false;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   499
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   500
489c9b5090e2 Initial load
duke
parents:
diff changeset
   501
  // the method is in the interesting class so the entry is interesting
489c9b5090e2 Initial load
duke
parents:
diff changeset
   502
  return true;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   503
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
   504
489c9b5090e2 Initial load
duke
parents:
diff changeset
   505
void ConstantPoolCacheEntry::print(outputStream* st, int index) const {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   506
  // print separator
11956
c363dc1ac4b7 7148126: ConstantPoolCacheEntry::print prints to wrong stream
dholmes
parents: 10546
diff changeset
   507
  if (index == 0) st->print_cr("                 -------------");
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   508
  // print entry
11956
c363dc1ac4b7 7148126: ConstantPoolCacheEntry::print prints to wrong stream
dholmes
parents: 10546
diff changeset
   509
  st->print("%3d  ("PTR_FORMAT")  ", index, (intptr_t)this);
2570
ecc7862946d4 6655646: dynamic languages need dynamically linked call sites
jrose
parents: 670
diff changeset
   510
  if (is_secondary_entry())
11956
c363dc1ac4b7 7148126: ConstantPoolCacheEntry::print prints to wrong stream
dholmes
parents: 10546
diff changeset
   511
    st->print_cr("[%5d|secondary]", main_entry_index());
2570
ecc7862946d4 6655646: dynamic languages need dynamically linked call sites
jrose
parents: 670
diff changeset
   512
  else
11956
c363dc1ac4b7 7148126: ConstantPoolCacheEntry::print prints to wrong stream
dholmes
parents: 10546
diff changeset
   513
    st->print_cr("[%02x|%02x|%5d]", bytecode_2(), bytecode_1(), constant_pool_index());
c363dc1ac4b7 7148126: ConstantPoolCacheEntry::print prints to wrong stream
dholmes
parents: 10546
diff changeset
   514
  st->print_cr("                 [   "PTR_FORMAT"]", (intptr_t)(oop)_f1);
c363dc1ac4b7 7148126: ConstantPoolCacheEntry::print prints to wrong stream
dholmes
parents: 10546
diff changeset
   515
  st->print_cr("                 [   "PTR_FORMAT"]", (intptr_t)_f2);
c363dc1ac4b7 7148126: ConstantPoolCacheEntry::print prints to wrong stream
dholmes
parents: 10546
diff changeset
   516
  st->print_cr("                 [   "PTR_FORMAT"]", (intptr_t)_flags);
c363dc1ac4b7 7148126: ConstantPoolCacheEntry::print prints to wrong stream
dholmes
parents: 10546
diff changeset
   517
  st->print_cr("                 -------------");
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   518
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
   519
489c9b5090e2 Initial load
duke
parents:
diff changeset
   520
void ConstantPoolCacheEntry::verify(outputStream* st) const {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   521
  // not implemented yet
489c9b5090e2 Initial load
duke
parents:
diff changeset
   522
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
   523
489c9b5090e2 Initial load
duke
parents:
diff changeset
   524
// Implementation of ConstantPoolCache
489c9b5090e2 Initial load
duke
parents:
diff changeset
   525
489c9b5090e2 Initial load
duke
parents:
diff changeset
   526
void constantPoolCacheOopDesc::initialize(intArray& inverse_index_map) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   527
  assert(inverse_index_map.length() == length(), "inverse index map must have same length as cache");
4429
d7eb4e2099aa 6858164: invokedynamic code needs some cleanup (post-6655638)
jrose
parents: 3261
diff changeset
   528
  for (int i = 0; i < length(); i++) {
d7eb4e2099aa 6858164: invokedynamic code needs some cleanup (post-6655638)
jrose
parents: 3261
diff changeset
   529
    ConstantPoolCacheEntry* e = entry_at(i);
d7eb4e2099aa 6858164: invokedynamic code needs some cleanup (post-6655638)
jrose
parents: 3261
diff changeset
   530
    int original_index = inverse_index_map[i];
d7eb4e2099aa 6858164: invokedynamic code needs some cleanup (post-6655638)
jrose
parents: 3261
diff changeset
   531
    if ((original_index & Rewriter::_secondary_entry_tag) != 0) {
d7eb4e2099aa 6858164: invokedynamic code needs some cleanup (post-6655638)
jrose
parents: 3261
diff changeset
   532
      int main_index = (original_index - Rewriter::_secondary_entry_tag);
d7eb4e2099aa 6858164: invokedynamic code needs some cleanup (post-6655638)
jrose
parents: 3261
diff changeset
   533
      assert(!entry_at(main_index)->is_secondary_entry(), "valid main index");
d7eb4e2099aa 6858164: invokedynamic code needs some cleanup (post-6655638)
jrose
parents: 3261
diff changeset
   534
      e->initialize_secondary_entry(main_index);
d7eb4e2099aa 6858164: invokedynamic code needs some cleanup (post-6655638)
jrose
parents: 3261
diff changeset
   535
    } else {
d7eb4e2099aa 6858164: invokedynamic code needs some cleanup (post-6655638)
jrose
parents: 3261
diff changeset
   536
      e->initialize_entry(original_index);
d7eb4e2099aa 6858164: invokedynamic code needs some cleanup (post-6655638)
jrose
parents: 3261
diff changeset
   537
    }
d7eb4e2099aa 6858164: invokedynamic code needs some cleanup (post-6655638)
jrose
parents: 3261
diff changeset
   538
    assert(entry_at(i) == e, "sanity");
d7eb4e2099aa 6858164: invokedynamic code needs some cleanup (post-6655638)
jrose
parents: 3261
diff changeset
   539
  }
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   540
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
   541
489c9b5090e2 Initial load
duke
parents:
diff changeset
   542
// RedefineClasses() API support:
489c9b5090e2 Initial load
duke
parents:
diff changeset
   543
// If any entry of this constantPoolCache points to any of
489c9b5090e2 Initial load
duke
parents:
diff changeset
   544
// old_methods, replace it with the corresponding new_method.
489c9b5090e2 Initial load
duke
parents:
diff changeset
   545
void constantPoolCacheOopDesc::adjust_method_entries(methodOop* old_methods, methodOop* new_methods,
489c9b5090e2 Initial load
duke
parents:
diff changeset
   546
                                                     int methods_length, bool * trace_name_printed) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   547
489c9b5090e2 Initial load
duke
parents:
diff changeset
   548
  if (methods_length == 0) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   549
    // nothing to do if there are no methods
489c9b5090e2 Initial load
duke
parents:
diff changeset
   550
    return;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   551
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   552
489c9b5090e2 Initial load
duke
parents:
diff changeset
   553
  // get shorthand for the interesting class
489c9b5090e2 Initial load
duke
parents:
diff changeset
   554
  klassOop old_holder = old_methods[0]->method_holder();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   555
489c9b5090e2 Initial load
duke
parents:
diff changeset
   556
  for (int i = 0; i < length(); i++) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   557
    if (!entry_at(i)->is_interesting_method_entry(old_holder)) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   558
      // skip uninteresting methods
489c9b5090e2 Initial load
duke
parents:
diff changeset
   559
      continue;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   560
    }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   561
489c9b5090e2 Initial load
duke
parents:
diff changeset
   562
    // The constantPoolCache contains entries for several different
489c9b5090e2 Initial load
duke
parents:
diff changeset
   563
    // things, but we only care about methods. In fact, we only care
489c9b5090e2 Initial load
duke
parents:
diff changeset
   564
    // about methods in the same class as the one that contains the
489c9b5090e2 Initial load
duke
parents:
diff changeset
   565
    // old_methods. At this point, we have an interesting entry.
489c9b5090e2 Initial load
duke
parents:
diff changeset
   566
489c9b5090e2 Initial load
duke
parents:
diff changeset
   567
    for (int j = 0; j < methods_length; j++) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   568
      methodOop old_method = old_methods[j];
489c9b5090e2 Initial load
duke
parents:
diff changeset
   569
      methodOop new_method = new_methods[j];
489c9b5090e2 Initial load
duke
parents:
diff changeset
   570
489c9b5090e2 Initial load
duke
parents:
diff changeset
   571
      if (entry_at(i)->adjust_method_entry(old_method, new_method,
489c9b5090e2 Initial load
duke
parents:
diff changeset
   572
          trace_name_printed)) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   573
        // current old_method matched this entry and we updated it so
489c9b5090e2 Initial load
duke
parents:
diff changeset
   574
        // break out and get to the next interesting entry if there one
489c9b5090e2 Initial load
duke
parents:
diff changeset
   575
        break;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   576
      }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   577
    }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   578
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   579
}