src/hotspot/share/oops/cpCache.cpp
author iklam
Tue, 07 Aug 2018 15:45:07 -0700
changeset 51329 9c68699bebe5
parent 50735 2f2af62dfac7
child 51486 67b55f3c45eb
permissions -rw-r--r--
8208999: Some use of Klass* should be replaced by InstanceKlass* Summary: Klass::java_super() => InstanceKlass*; InstanceKlass::{local,transitive}_interfaces() => Array<InstanceKlass*>* Reviewed-by: coleenp, hseigel
Ignore whitespace changes - Everywhere: Within whitespace: At end of lines:
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
     1
/*
49340
4e82736053ae 8191102: Incorrect include file use in classLoader.hpp
hseigel
parents: 48819
diff changeset
     2
 * Copyright (c) 1998, 2018, 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"
47684
c3c04b6e14f8 8174954: Parameter target type is allowed access after a module read edge or a package export has occurred after failed resolution
hseigel
parents: 47634
diff changeset
    26
#include "classfile/resolutionErrors.hpp"
47103
a993ec29ec75 8186842: Use Java class loaders for creating the CDS archive
ccheung
parents: 47095
diff changeset
    27
#include "interpreter/bytecodeStream.hpp"
a993ec29ec75 8186842: Use Java class loaders for creating the CDS archive
ccheung
parents: 47095
diff changeset
    28
#include "interpreter/bytecodes.hpp"
7397
5b173b4ca846 6989984: Use standard include model for Hospot
stefank
parents: 7104
diff changeset
    29
#include "interpreter/interpreter.hpp"
49593
4dd58ecc9912 8200105: Remove cyclic dependency between oop.inline.hpp and collectedHeap.inline.hpp
stefank
parents: 49359
diff changeset
    30
#include "interpreter/linkResolver.hpp"
7397
5b173b4ca846 6989984: Use standard include model for Hospot
stefank
parents: 7104
diff changeset
    31
#include "interpreter/rewriter.hpp"
38259
b495d1cfe673 8153535: Convert TraceRedefineClasses to Unified Logging
rehn
parents: 38033
diff changeset
    32
#include "logging/log.hpp"
47095
9d21da6fe729 8181170: resolved_references array leaks for RedefineClasses
coleenp
parents: 46810
diff changeset
    33
#include "memory/metadataFactory.hpp"
46746
ea379ebb9447 8072061: Automatically determine optimal sizes for the CDS regions
iklam
parents: 46727
diff changeset
    34
#include "memory/metaspaceClosure.hpp"
50532
a18c60527166 8204585: Remove IN_ARCHIVE_ROOT from Access API
kbarrett
parents: 50429
diff changeset
    35
#include "memory/metaspaceShared.hpp"
37248
11a660dbbb8e 8132524: Missing includes to resourceArea.hpp
jprovino
parents: 35913
diff changeset
    36
#include "memory/resourceArea.hpp"
49359
59f6547e151f 8199264: Remove universe.inline.hpp to simplify include dependencies
stefank
parents: 49340
diff changeset
    37
#include "memory/universe.hpp"
48784
899b522ed65c 8194741: Refactor oops in constant pool from CDS to use the Access API
eosterlund
parents: 48463
diff changeset
    38
#include "oops/access.inline.hpp"
49340
4e82736053ae 8191102: Incorrect include file use in classLoader.hpp
hseigel
parents: 48819
diff changeset
    39
#include "oops/constantPool.inline.hpp"
4e82736053ae 8191102: Incorrect include file use in classLoader.hpp
hseigel
parents: 48819
diff changeset
    40
#include "oops/cpCache.inline.hpp"
29081
c61eb4914428 8072911: Remove includes of oop.inline.hpp from .hpp files
stefank
parents: 28019
diff changeset
    41
#include "oops/objArrayOop.inline.hpp"
7397
5b173b4ca846 6989984: Use standard include model for Hospot
stefank
parents: 7104
diff changeset
    42
#include "oops/oop.inline.hpp"
13391
30245956af37 7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents: 11956
diff changeset
    43
#include "prims/methodHandles.hpp"
40655
9f644073d3a0 8157907: Incorrect inclusion of atomic.hpp instead of atomic.inline.hpp
dholmes
parents: 38259
diff changeset
    44
#include "runtime/atomic.hpp"
7397
5b173b4ca846 6989984: Use standard include model for Hospot
stefank
parents: 7104
diff changeset
    45
#include "runtime/handles.inline.hpp"
50429
83aec1d357d4 8204301: Make OrderAccess functions available to hpp rather than inline.hpp files
coleenp
parents: 49773
diff changeset
    46
#include "runtime/orderAccess.hpp"
15482
470d0b0c09f1 8005915: Unify SERIALGC and INCLUDE_ALTERNATE_GCS
jprovino
parents: 14490
diff changeset
    47
#include "utilities/macros.hpp"
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
    48
24424
2658d7834c6e 8037816: Fix for 8036122 breaks build with Xcode5/clang
drchase
parents: 24351
diff changeset
    49
// Implementation of ConstantPoolCacheEntry
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
    50
4429
d7eb4e2099aa 6858164: invokedynamic code needs some cleanup (post-6655638)
jrose
parents: 3261
diff changeset
    51
void ConstantPoolCacheEntry::initialize_entry(int index) {
2570
ecc7862946d4 6655646: dynamic languages need dynamically linked call sites
jrose
parents: 670
diff changeset
    52
  assert(0 < index && index < 0x10000, "sanity check");
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
    53
  _indices = index;
15928
f9d5c6e4107f 8003553: NPG: metaspace objects should be zeroed in constructors
coleenp
parents: 15595
diff changeset
    54
  _f1 = NULL;
f9d5c6e4107f 8003553: NPG: metaspace objects should be zeroed in constructors
coleenp
parents: 15595
diff changeset
    55
  _f2 = _flags = 0;
2570
ecc7862946d4 6655646: dynamic languages need dynamically linked call sites
jrose
parents: 670
diff changeset
    56
  assert(constant_pool_index() == index, "");
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
    57
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
    58
47103
a993ec29ec75 8186842: Use Java class loaders for creating the CDS archive
ccheung
parents: 47095
diff changeset
    59
void ConstantPoolCacheEntry::verify_just_initialized(bool f2_used) {
a993ec29ec75 8186842: Use Java class loaders for creating the CDS archive
ccheung
parents: 47095
diff changeset
    60
  assert((_indices & (~cp_index_mask)) == 0, "sanity");
a993ec29ec75 8186842: Use Java class loaders for creating the CDS archive
ccheung
parents: 47095
diff changeset
    61
  assert(_f1 == NULL, "sanity");
a993ec29ec75 8186842: Use Java class loaders for creating the CDS archive
ccheung
parents: 47095
diff changeset
    62
  assert(_flags == 0, "sanity");
a993ec29ec75 8186842: Use Java class loaders for creating the CDS archive
ccheung
parents: 47095
diff changeset
    63
  if (!f2_used) {
a993ec29ec75 8186842: Use Java class loaders for creating the CDS archive
ccheung
parents: 47095
diff changeset
    64
    assert(_f2 == 0, "sanity");
a993ec29ec75 8186842: Use Java class loaders for creating the CDS archive
ccheung
parents: 47095
diff changeset
    65
  }
a993ec29ec75 8186842: Use Java class loaders for creating the CDS archive
ccheung
parents: 47095
diff changeset
    66
}
a993ec29ec75 8186842: Use Java class loaders for creating the CDS archive
ccheung
parents: 47095
diff changeset
    67
a993ec29ec75 8186842: Use Java class loaders for creating the CDS archive
ccheung
parents: 47095
diff changeset
    68
void ConstantPoolCacheEntry::reinitialize(bool f2_used) {
a993ec29ec75 8186842: Use Java class loaders for creating the CDS archive
ccheung
parents: 47095
diff changeset
    69
  _indices &= cp_index_mask;
a993ec29ec75 8186842: Use Java class loaders for creating the CDS archive
ccheung
parents: 47095
diff changeset
    70
  _f1 = NULL;
a993ec29ec75 8186842: Use Java class loaders for creating the CDS archive
ccheung
parents: 47095
diff changeset
    71
  _flags = 0;
a993ec29ec75 8186842: Use Java class loaders for creating the CDS archive
ccheung
parents: 47095
diff changeset
    72
  if (!f2_used) {
a993ec29ec75 8186842: Use Java class loaders for creating the CDS archive
ccheung
parents: 47095
diff changeset
    73
    _f2 = 0;
a993ec29ec75 8186842: Use Java class loaders for creating the CDS archive
ccheung
parents: 47095
diff changeset
    74
  }
a993ec29ec75 8186842: Use Java class loaders for creating the CDS archive
ccheung
parents: 47095
diff changeset
    75
}
a993ec29ec75 8186842: Use Java class loaders for creating the CDS archive
ccheung
parents: 47095
diff changeset
    76
13391
30245956af37 7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents: 11956
diff changeset
    77
int ConstantPoolCacheEntry::make_flags(TosState state,
30245956af37 7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents: 11956
diff changeset
    78
                                       int option_bits,
30245956af37 7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents: 11956
diff changeset
    79
                                       int field_index_or_method_params) {
30245956af37 7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents: 11956
diff changeset
    80
  assert(state < number_of_states, "Invalid state in make_flags");
30245956af37 7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents: 11956
diff changeset
    81
  int f = ((int)state << tos_state_shift) | option_bits | field_index_or_method_params;
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
    82
  // Preserve existing flag bit values
13391
30245956af37 7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents: 11956
diff changeset
    83
  // The low bits are a field offset, or else the method parameter size.
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
    84
#ifdef ASSERT
13391
30245956af37 7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents: 11956
diff changeset
    85
  TosState old_state = flag_state();
30245956af37 7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents: 11956
diff changeset
    86
  assert(old_state == (TosState)0 || old_state == state,
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
    87
         "inconsistent cpCache flags state");
489c9b5090e2 Initial load
duke
parents:
diff changeset
    88
#endif
489c9b5090e2 Initial load
duke
parents:
diff changeset
    89
  return (_flags | f) ;
489c9b5090e2 Initial load
duke
parents:
diff changeset
    90
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
    91
489c9b5090e2 Initial load
duke
parents:
diff changeset
    92
void ConstantPoolCacheEntry::set_bytecode_1(Bytecodes::Code code) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
    93
#ifdef ASSERT
489c9b5090e2 Initial load
duke
parents:
diff changeset
    94
  // Read once.
489c9b5090e2 Initial load
duke
parents:
diff changeset
    95
  volatile Bytecodes::Code c = bytecode_1();
489c9b5090e2 Initial load
duke
parents:
diff changeset
    96
  assert(c == 0 || c == code || code == 0, "update must be consistent");
489c9b5090e2 Initial load
duke
parents:
diff changeset
    97
#endif
489c9b5090e2 Initial load
duke
parents:
diff changeset
    98
  // Need to flush pending stores here before bytecode is written.
47634
6a0c42c40cd1 8188220: Remove Atomic::*_ptr() uses and overloads from hotspot
coleenp
parents: 47216
diff changeset
    99
  OrderAccess::release_store(&_indices, _indices | ((u_char)code << bytecode_1_shift));
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   100
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
   101
489c9b5090e2 Initial load
duke
parents:
diff changeset
   102
void ConstantPoolCacheEntry::set_bytecode_2(Bytecodes::Code code) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   103
#ifdef ASSERT
489c9b5090e2 Initial load
duke
parents:
diff changeset
   104
  // Read once.
489c9b5090e2 Initial load
duke
parents:
diff changeset
   105
  volatile Bytecodes::Code c = bytecode_2();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   106
  assert(c == 0 || c == code || code == 0, "update must be consistent");
489c9b5090e2 Initial load
duke
parents:
diff changeset
   107
#endif
489c9b5090e2 Initial load
duke
parents:
diff changeset
   108
  // Need to flush pending stores here before bytecode is written.
47634
6a0c42c40cd1 8188220: Remove Atomic::*_ptr() uses and overloads from hotspot
coleenp
parents: 47216
diff changeset
   109
  OrderAccess::release_store(&_indices, _indices | ((u_char)code << bytecode_2_shift));
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   110
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
   111
13391
30245956af37 7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents: 11956
diff changeset
   112
// Sets f1, ordering with previous writes.
13728
882756847a04 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 13391
diff changeset
   113
void ConstantPoolCacheEntry::release_set_f1(Metadata* f1) {
13391
30245956af37 7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents: 11956
diff changeset
   114
  assert(f1 != NULL, "");
47634
6a0c42c40cd1 8188220: Remove Atomic::*_ptr() uses and overloads from hotspot
coleenp
parents: 47216
diff changeset
   115
  OrderAccess::release_store(&_f1, f1);
8316
0816329cc3ee 7012339: JSR 292 crash in G1SATBCardTableModRefBS::write_ref_field_pre_work()
twisti
parents: 7397
diff changeset
   116
}
7104
0c8b519af363 6990192: VM crashes in ciTypeFlow::get_block_for()
twisti
parents: 6064
diff changeset
   117
47684
c3c04b6e14f8 8174954: Parameter target type is allowed access after a module read edge or a package export has occurred after failed resolution
hseigel
parents: 47634
diff changeset
   118
void ConstantPoolCacheEntry::set_indy_resolution_failed() {
c3c04b6e14f8 8174954: Parameter target type is allowed access after a module read edge or a package export has occurred after failed resolution
hseigel
parents: 47634
diff changeset
   119
  OrderAccess::release_store(&_flags, _flags | (1 << indy_resolution_failed_shift));
c3c04b6e14f8 8174954: Parameter target type is allowed access after a module read edge or a package export has occurred after failed resolution
hseigel
parents: 47634
diff changeset
   120
}
c3c04b6e14f8 8174954: Parameter target type is allowed access after a module read edge or a package export has occurred after failed resolution
hseigel
parents: 47634
diff changeset
   121
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   122
// Note that concurrent update of both bytecodes can leave one of them
489c9b5090e2 Initial load
duke
parents:
diff changeset
   123
// reset to zero.  This is harmless; the interpreter will simply re-resolve
489c9b5090e2 Initial load
duke
parents:
diff changeset
   124
// the damaged entry.  More seriously, the memory synchronization is needed
489c9b5090e2 Initial load
duke
parents:
diff changeset
   125
// to flush other fields (f1, f2) completely to memory before the bytecodes
489c9b5090e2 Initial load
duke
parents:
diff changeset
   126
// are updated, lest other processors see a non-zero bytecode but zero f1/f2.
489c9b5090e2 Initial load
duke
parents:
diff changeset
   127
void ConstantPoolCacheEntry::set_field(Bytecodes::Code get_code,
489c9b5090e2 Initial load
duke
parents:
diff changeset
   128
                                       Bytecodes::Code put_code,
46329
53ccc37bda19 8155672: Remove instanceKlassHandles and KlassHandles
coleenp
parents: 46271
diff changeset
   129
                                       Klass* field_holder,
10546
e79347eebbc5 7086585: make Java field injection more flexible
never
parents: 10008
diff changeset
   130
                                       int field_index,
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   131
                                       int field_offset,
489c9b5090e2 Initial load
duke
parents:
diff changeset
   132
                                       TosState field_type,
489c9b5090e2 Initial load
duke
parents:
diff changeset
   133
                                       bool is_final,
13728
882756847a04 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 13391
diff changeset
   134
                                       bool is_volatile,
882756847a04 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 13391
diff changeset
   135
                                       Klass* root_klass) {
46329
53ccc37bda19 8155672: Remove instanceKlassHandles and KlassHandles
coleenp
parents: 46271
diff changeset
   136
  set_f1(field_holder);
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   137
  set_f2(field_offset);
13391
30245956af37 7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents: 11956
diff changeset
   138
  assert((field_index & field_index_mask) == field_index,
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   139
         "field index does not fit in low flag bits");
13391
30245956af37 7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents: 11956
diff changeset
   140
  set_field_flags(field_type,
30245956af37 7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents: 11956
diff changeset
   141
                  ((is_volatile ? 1 : 0) << is_volatile_shift) |
30245956af37 7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents: 11956
diff changeset
   142
                  ((is_final    ? 1 : 0) << is_final_shift),
30245956af37 7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents: 11956
diff changeset
   143
                  field_index);
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   144
  set_bytecode_1(get_code);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   145
  set_bytecode_2(put_code);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   146
  NOT_PRODUCT(verify(tty));
489c9b5090e2 Initial load
duke
parents:
diff changeset
   147
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
   148
13391
30245956af37 7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents: 11956
diff changeset
   149
void ConstantPoolCacheEntry::set_parameter_size(int value) {
30245956af37 7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents: 11956
diff changeset
   150
  // This routine is called only in corner cases where the CPCE is not yet initialized.
30245956af37 7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents: 11956
diff changeset
   151
  // See AbstractInterpreter::deopt_continue_after_entry.
30245956af37 7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents: 11956
diff changeset
   152
  assert(_flags == 0 || parameter_size() == 0 || parameter_size() == value,
33105
294e48b4f704 8080775: Better argument formatting for assert() and friends
david
parents: 31592
diff changeset
   153
         "size must not change: parameter_size=%d, value=%d", parameter_size(), value);
13391
30245956af37 7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents: 11956
diff changeset
   154
  // Setting the parameter size by itself is only safe if the
30245956af37 7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents: 11956
diff changeset
   155
  // current value of _flags is 0, otherwise another thread may have
30245956af37 7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents: 11956
diff changeset
   156
  // updated it and we don't want to overwrite that value.  Don't
30245956af37 7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents: 11956
diff changeset
   157
  // bother trying to update it once it's nonzero but always make
30245956af37 7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents: 11956
diff changeset
   158
  // sure that the final parameter size agrees with what was passed.
30245956af37 7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents: 11956
diff changeset
   159
  if (_flags == 0) {
47634
6a0c42c40cd1 8188220: Remove Atomic::*_ptr() uses and overloads from hotspot
coleenp
parents: 47216
diff changeset
   160
    intx newflags = (value & parameter_size_mask);
6a0c42c40cd1 8188220: Remove Atomic::*_ptr() uses and overloads from hotspot
coleenp
parents: 47216
diff changeset
   161
    Atomic::cmpxchg(newflags, &_flags, (intx)0);
13391
30245956af37 7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents: 11956
diff changeset
   162
  }
30245956af37 7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents: 11956
diff changeset
   163
  guarantee(parameter_size() == value,
33105
294e48b4f704 8080775: Better argument formatting for assert() and friends
david
parents: 31592
diff changeset
   164
            "size must not change: parameter_size=%d, value=%d", parameter_size(), value);
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   165
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
   166
20017
81eba62e9048 8014013: CallInfo structure no longer accurately reports the result of a LinkResolver operation
drchase
parents: 17858
diff changeset
   167
void ConstantPoolCacheEntry::set_direct_or_vtable_call(Bytecodes::Code invoke_code,
46727
6e4a84748e2c 8183039: Re-examine methodHandle methods uninlined by 8144256
coleenp
parents: 46630
diff changeset
   168
                                                       const methodHandle& method,
44738
11431bbc9549 8168699: Validate special case invocations
coleenp
parents: 40655
diff changeset
   169
                                                       int vtable_index,
11431bbc9549 8168699: Validate special case invocations
coleenp
parents: 40655
diff changeset
   170
                                                       bool sender_is_interface) {
20017
81eba62e9048 8014013: CallInfo structure no longer accurately reports the result of a LinkResolver operation
drchase
parents: 17858
diff changeset
   171
  bool is_vtable_call = (vtable_index >= 0);  // FIXME: split this method on this boolean
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   172
  assert(method->interpreter_entry() != NULL, "should have been set at this point");
489c9b5090e2 Initial load
duke
parents:
diff changeset
   173
  assert(!method->is_obsolete(),  "attempt to write obsolete method to cpCache");
489c9b5090e2 Initial load
duke
parents:
diff changeset
   174
489c9b5090e2 Initial load
duke
parents:
diff changeset
   175
  int byte_no = -1;
13391
30245956af37 7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents: 11956
diff changeset
   176
  bool change_to_virtual = false;
50735
2f2af62dfac7 8010319: Implementation of JEP 181: Nest-Based Access Control
dholmes
parents: 50532
diff changeset
   177
  InstanceKlass* holder = NULL;  // have to declare this outside the switch
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   178
  switch (invoke_code) {
13391
30245956af37 7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents: 11956
diff changeset
   179
    case Bytecodes::_invokeinterface:
50735
2f2af62dfac7 8010319: Implementation of JEP 181: Nest-Based Access Control
dholmes
parents: 50532
diff changeset
   180
      holder = method->method_holder();
2f2af62dfac7 8010319: Implementation of JEP 181: Nest-Based Access Control
dholmes
parents: 50532
diff changeset
   181
      // check for private interface method invocations
2f2af62dfac7 8010319: Implementation of JEP 181: Nest-Based Access Control
dholmes
parents: 50532
diff changeset
   182
      if (vtable_index == Method::nonvirtual_vtable_index && holder->is_interface() ) {
2f2af62dfac7 8010319: Implementation of JEP 181: Nest-Based Access Control
dholmes
parents: 50532
diff changeset
   183
        assert(method->is_private(), "unexpected non-private method");
2f2af62dfac7 8010319: Implementation of JEP 181: Nest-Based Access Control
dholmes
parents: 50532
diff changeset
   184
        assert(method->can_be_statically_bound(), "unexpected non-statically-bound method");
2f2af62dfac7 8010319: Implementation of JEP 181: Nest-Based Access Control
dholmes
parents: 50532
diff changeset
   185
        // set_f2_as_vfinal_method checks if is_vfinal flag is true.
2f2af62dfac7 8010319: Implementation of JEP 181: Nest-Based Access Control
dholmes
parents: 50532
diff changeset
   186
        set_method_flags(as_TosState(method->result_type()),
2f2af62dfac7 8010319: Implementation of JEP 181: Nest-Based Access Control
dholmes
parents: 50532
diff changeset
   187
                         (                             1      << is_vfinal_shift) |
2f2af62dfac7 8010319: Implementation of JEP 181: Nest-Based Access Control
dholmes
parents: 50532
diff changeset
   188
                         ((method->is_final_method() ? 1 : 0) << is_final_shift),
2f2af62dfac7 8010319: Implementation of JEP 181: Nest-Based Access Control
dholmes
parents: 50532
diff changeset
   189
                         method()->size_of_parameters());
2f2af62dfac7 8010319: Implementation of JEP 181: Nest-Based Access Control
dholmes
parents: 50532
diff changeset
   190
        set_f2_as_vfinal_method(method());
2f2af62dfac7 8010319: Implementation of JEP 181: Nest-Based Access Control
dholmes
parents: 50532
diff changeset
   191
        byte_no = 2;
2f2af62dfac7 8010319: Implementation of JEP 181: Nest-Based Access Control
dholmes
parents: 50532
diff changeset
   192
        set_f1(holder); // interface klass*
2f2af62dfac7 8010319: Implementation of JEP 181: Nest-Based Access Control
dholmes
parents: 50532
diff changeset
   193
        break;
2f2af62dfac7 8010319: Implementation of JEP 181: Nest-Based Access Control
dholmes
parents: 50532
diff changeset
   194
      }
2f2af62dfac7 8010319: Implementation of JEP 181: Nest-Based Access Control
dholmes
parents: 50532
diff changeset
   195
      else {
2f2af62dfac7 8010319: Implementation of JEP 181: Nest-Based Access Control
dholmes
parents: 50532
diff changeset
   196
        // We get here from InterpreterRuntime::resolve_invoke when an invokeinterface
2f2af62dfac7 8010319: Implementation of JEP 181: Nest-Based Access Control
dholmes
parents: 50532
diff changeset
   197
        // instruction links to a non-interface method (in Object). This can happen when
2f2af62dfac7 8010319: Implementation of JEP 181: Nest-Based Access Control
dholmes
parents: 50532
diff changeset
   198
        // an interface redeclares an Object method (like CharSequence declaring toString())
2f2af62dfac7 8010319: Implementation of JEP 181: Nest-Based Access Control
dholmes
parents: 50532
diff changeset
   199
        // or when invokeinterface is used explicitly.
2f2af62dfac7 8010319: Implementation of JEP 181: Nest-Based Access Control
dholmes
parents: 50532
diff changeset
   200
        // In that case, the method has no itable index and must be invoked as a virtual.
2f2af62dfac7 8010319: Implementation of JEP 181: Nest-Based Access Control
dholmes
parents: 50532
diff changeset
   201
        // Set a flag to keep track of this corner case.
2f2af62dfac7 8010319: Implementation of JEP 181: Nest-Based Access Control
dholmes
parents: 50532
diff changeset
   202
        assert(holder->is_interface() || holder == SystemDictionary::Object_klass(), "unexpected holder class");
2f2af62dfac7 8010319: Implementation of JEP 181: Nest-Based Access Control
dholmes
parents: 50532
diff changeset
   203
        assert(method->is_public(), "Calling non-public method in Object with invokeinterface");
2f2af62dfac7 8010319: Implementation of JEP 181: Nest-Based Access Control
dholmes
parents: 50532
diff changeset
   204
        change_to_virtual = true;
13391
30245956af37 7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents: 11956
diff changeset
   205
50735
2f2af62dfac7 8010319: Implementation of JEP 181: Nest-Based Access Control
dholmes
parents: 50532
diff changeset
   206
        // ...and fall through as if we were handling invokevirtual:
2f2af62dfac7 8010319: Implementation of JEP 181: Nest-Based Access Control
dholmes
parents: 50532
diff changeset
   207
      }
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   208
    case Bytecodes::_invokevirtual:
13391
30245956af37 7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents: 11956
diff changeset
   209
      {
20017
81eba62e9048 8014013: CallInfo structure no longer accurately reports the result of a LinkResolver operation
drchase
parents: 17858
diff changeset
   210
        if (!is_vtable_call) {
81eba62e9048 8014013: CallInfo structure no longer accurately reports the result of a LinkResolver operation
drchase
parents: 17858
diff changeset
   211
          assert(method->can_be_statically_bound(), "");
13391
30245956af37 7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents: 11956
diff changeset
   212
          // set_f2_as_vfinal_method checks if is_vfinal flag is true.
30245956af37 7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents: 11956
diff changeset
   213
          set_method_flags(as_TosState(method->result_type()),
30245956af37 7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents: 11956
diff changeset
   214
                           (                             1      << is_vfinal_shift) |
30245956af37 7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents: 11956
diff changeset
   215
                           ((method->is_final_method() ? 1 : 0) << is_final_shift)  |
30245956af37 7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents: 11956
diff changeset
   216
                           ((change_to_virtual         ? 1 : 0) << is_forced_virtual_shift),
30245956af37 7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents: 11956
diff changeset
   217
                           method()->size_of_parameters());
30245956af37 7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents: 11956
diff changeset
   218
          set_f2_as_vfinal_method(method());
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   219
        } else {
20017
81eba62e9048 8014013: CallInfo structure no longer accurately reports the result of a LinkResolver operation
drchase
parents: 17858
diff changeset
   220
          assert(!method->can_be_statically_bound(), "");
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   221
          assert(vtable_index >= 0, "valid index");
13391
30245956af37 7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents: 11956
diff changeset
   222
          assert(!method->is_final_method(), "sanity");
30245956af37 7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents: 11956
diff changeset
   223
          set_method_flags(as_TosState(method->result_type()),
30245956af37 7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents: 11956
diff changeset
   224
                           ((change_to_virtual ? 1 : 0) << is_forced_virtual_shift),
30245956af37 7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents: 11956
diff changeset
   225
                           method()->size_of_parameters());
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   226
          set_f2(vtable_index);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   227
        }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   228
        byte_no = 2;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   229
        break;
6062
bab93afe9df7 6964498: JSR 292 invokedynamic sites need local bootstrap methods
jrose
parents: 5547
diff changeset
   230
      }
bab93afe9df7 6964498: JSR 292 invokedynamic sites need local bootstrap methods
jrose
parents: 5547
diff changeset
   231
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   232
    case Bytecodes::_invokespecial:
489c9b5090e2 Initial load
duke
parents:
diff changeset
   233
    case Bytecodes::_invokestatic:
20017
81eba62e9048 8014013: CallInfo structure no longer accurately reports the result of a LinkResolver operation
drchase
parents: 17858
diff changeset
   234
      assert(!is_vtable_call, "");
13391
30245956af37 7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents: 11956
diff changeset
   235
      // Note:  Read and preserve the value of the is_vfinal flag on any
30245956af37 7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents: 11956
diff changeset
   236
      // invokevirtual bytecode shared with this constant pool cache entry.
30245956af37 7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents: 11956
diff changeset
   237
      // It is cheap and safe to consult is_vfinal() at all times.
30245956af37 7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents: 11956
diff changeset
   238
      // Once is_vfinal is set, it must stay that way, lest we get a dangling oop.
30245956af37 7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents: 11956
diff changeset
   239
      set_method_flags(as_TosState(method->result_type()),
30245956af37 7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents: 11956
diff changeset
   240
                       ((is_vfinal()               ? 1 : 0) << is_vfinal_shift) |
30245956af37 7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents: 11956
diff changeset
   241
                       ((method->is_final_method() ? 1 : 0) << is_final_shift),
30245956af37 7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents: 11956
diff changeset
   242
                       method()->size_of_parameters());
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   243
      set_f1(method());
489c9b5090e2 Initial load
duke
parents:
diff changeset
   244
      byte_no = 1;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   245
      break;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   246
    default:
489c9b5090e2 Initial load
duke
parents:
diff changeset
   247
      ShouldNotReachHere();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   248
      break;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   249
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   250
489c9b5090e2 Initial load
duke
parents:
diff changeset
   251
  // Note:  byte_no also appears in TemplateTable::resolve.
489c9b5090e2 Initial load
duke
parents:
diff changeset
   252
  if (byte_no == 1) {
13391
30245956af37 7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents: 11956
diff changeset
   253
    assert(invoke_code != Bytecodes::_invokevirtual &&
30245956af37 7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents: 11956
diff changeset
   254
           invoke_code != Bytecodes::_invokeinterface, "");
44738
11431bbc9549 8168699: Validate special case invocations
coleenp
parents: 40655
diff changeset
   255
    // Don't mark invokespecial to method as resolved if sender is an interface.  The receiver
11431bbc9549 8168699: Validate special case invocations
coleenp
parents: 40655
diff changeset
   256
    // has to be checked that it is a subclass of the current class every time this bytecode
11431bbc9549 8168699: Validate special case invocations
coleenp
parents: 40655
diff changeset
   257
    // is executed.
11431bbc9549 8168699: Validate special case invocations
coleenp
parents: 40655
diff changeset
   258
    if (invoke_code != Bytecodes::_invokespecial || !sender_is_interface ||
11431bbc9549 8168699: Validate special case invocations
coleenp
parents: 40655
diff changeset
   259
        method->name() == vmSymbols::object_initializer_name()) {
50735
2f2af62dfac7 8010319: Implementation of JEP 181: Nest-Based Access Control
dholmes
parents: 50532
diff changeset
   260
      set_bytecode_1(invoke_code);
44738
11431bbc9549 8168699: Validate special case invocations
coleenp
parents: 40655
diff changeset
   261
    }
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   262
  } else if (byte_no == 2)  {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   263
    if (change_to_virtual) {
13391
30245956af37 7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents: 11956
diff changeset
   264
      assert(invoke_code == Bytecodes::_invokeinterface, "");
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   265
      // NOTE: THIS IS A HACK - BE VERY CAREFUL!!!
489c9b5090e2 Initial load
duke
parents:
diff changeset
   266
      //
489c9b5090e2 Initial load
duke
parents:
diff changeset
   267
      // Workaround for the case where we encounter an invokeinterface, but we
489c9b5090e2 Initial load
duke
parents:
diff changeset
   268
      // should really have an _invokevirtual since the resolved method is a
489c9b5090e2 Initial load
duke
parents:
diff changeset
   269
      // virtual method in java.lang.Object. This is a corner case in the spec
489c9b5090e2 Initial load
duke
parents:
diff changeset
   270
      // but is presumably legal. javac does not generate this code.
489c9b5090e2 Initial load
duke
parents:
diff changeset
   271
      //
49773
3b0d5f4e1e30 8190478: Improved interface method selection
hseigel
parents: 49593
diff changeset
   272
      // We do not set bytecode_1() to _invokeinterface, because that is the
3b0d5f4e1e30 8190478: Improved interface method selection
hseigel
parents: 49593
diff changeset
   273
      // bytecode # used by the interpreter to see if it is resolved.  In this
3b0d5f4e1e30 8190478: Improved interface method selection
hseigel
parents: 49593
diff changeset
   274
      // case, the method gets reresolved with caller for each interface call
3b0d5f4e1e30 8190478: Improved interface method selection
hseigel
parents: 49593
diff changeset
   275
      // because the actual selected method may not be public.
3b0d5f4e1e30 8190478: Improved interface method selection
hseigel
parents: 49593
diff changeset
   276
      //
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   277
      // We set bytecode_2() to _invokevirtual.
489c9b5090e2 Initial load
duke
parents:
diff changeset
   278
      // See also interpreterRuntime.cpp. (8/25/2000)
489c9b5090e2 Initial load
duke
parents:
diff changeset
   279
    } else {
50735
2f2af62dfac7 8010319: Implementation of JEP 181: Nest-Based Access Control
dholmes
parents: 50532
diff changeset
   280
      assert(invoke_code == Bytecodes::_invokevirtual ||
2f2af62dfac7 8010319: Implementation of JEP 181: Nest-Based Access Control
dholmes
parents: 50532
diff changeset
   281
             (invoke_code == Bytecodes::_invokeinterface &&
2f2af62dfac7 8010319: Implementation of JEP 181: Nest-Based Access Control
dholmes
parents: 50532
diff changeset
   282
              ((method->is_private() ||
2f2af62dfac7 8010319: Implementation of JEP 181: Nest-Based Access Control
dholmes
parents: 50532
diff changeset
   283
                (method->is_final() && method->method_holder() == SystemDictionary::Object_klass())))),
2f2af62dfac7 8010319: Implementation of JEP 181: Nest-Based Access Control
dholmes
parents: 50532
diff changeset
   284
             "unexpected invocation mode");
2f2af62dfac7 8010319: Implementation of JEP 181: Nest-Based Access Control
dholmes
parents: 50532
diff changeset
   285
      if (invoke_code == Bytecodes::_invokeinterface &&
2f2af62dfac7 8010319: Implementation of JEP 181: Nest-Based Access Control
dholmes
parents: 50532
diff changeset
   286
          (method->is_private() || method->is_final())) {
2f2af62dfac7 8010319: Implementation of JEP 181: Nest-Based Access Control
dholmes
parents: 50532
diff changeset
   287
        // We set bytecode_1() to _invokeinterface, because that is the
2f2af62dfac7 8010319: Implementation of JEP 181: Nest-Based Access Control
dholmes
parents: 50532
diff changeset
   288
        // bytecode # used by the interpreter to see if it is resolved.
2f2af62dfac7 8010319: Implementation of JEP 181: Nest-Based Access Control
dholmes
parents: 50532
diff changeset
   289
        // We set bytecode_2() to _invokevirtual.
2f2af62dfac7 8010319: Implementation of JEP 181: Nest-Based Access Control
dholmes
parents: 50532
diff changeset
   290
        set_bytecode_1(invoke_code);
2f2af62dfac7 8010319: Implementation of JEP 181: Nest-Based Access Control
dholmes
parents: 50532
diff changeset
   291
      }
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   292
    }
13391
30245956af37 7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents: 11956
diff changeset
   293
    // set up for invokevirtual, even if linking for invokeinterface also:
30245956af37 7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents: 11956
diff changeset
   294
    set_bytecode_2(Bytecodes::_invokevirtual);
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   295
  } else {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   296
    ShouldNotReachHere();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   297
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   298
  NOT_PRODUCT(verify(tty));
489c9b5090e2 Initial load
duke
parents:
diff changeset
   299
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
   300
46727
6e4a84748e2c 8183039: Re-examine methodHandle methods uninlined by 8144256
coleenp
parents: 46630
diff changeset
   301
void ConstantPoolCacheEntry::set_direct_call(Bytecodes::Code invoke_code, const methodHandle& method,
44738
11431bbc9549 8168699: Validate special case invocations
coleenp
parents: 40655
diff changeset
   302
                                             bool sender_is_interface) {
20017
81eba62e9048 8014013: CallInfo structure no longer accurately reports the result of a LinkResolver operation
drchase
parents: 17858
diff changeset
   303
  int index = Method::nonvirtual_vtable_index;
81eba62e9048 8014013: CallInfo structure no longer accurately reports the result of a LinkResolver operation
drchase
parents: 17858
diff changeset
   304
  // index < 0; FIXME: inline and customize set_direct_or_vtable_call
44738
11431bbc9549 8168699: Validate special case invocations
coleenp
parents: 40655
diff changeset
   305
  set_direct_or_vtable_call(invoke_code, method, index, sender_is_interface);
20017
81eba62e9048 8014013: CallInfo structure no longer accurately reports the result of a LinkResolver operation
drchase
parents: 17858
diff changeset
   306
}
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   307
46727
6e4a84748e2c 8183039: Re-examine methodHandle methods uninlined by 8144256
coleenp
parents: 46630
diff changeset
   308
void ConstantPoolCacheEntry::set_vtable_call(Bytecodes::Code invoke_code, const methodHandle& method, int index) {
20017
81eba62e9048 8014013: CallInfo structure no longer accurately reports the result of a LinkResolver operation
drchase
parents: 17858
diff changeset
   309
  // either the method is a miranda or its holder should accept the given index
81eba62e9048 8014013: CallInfo structure no longer accurately reports the result of a LinkResolver operation
drchase
parents: 17858
diff changeset
   310
  assert(method->method_holder()->is_interface() || method->method_holder()->verify_vtable_index(index), "");
81eba62e9048 8014013: CallInfo structure no longer accurately reports the result of a LinkResolver operation
drchase
parents: 17858
diff changeset
   311
  // index >= 0; FIXME: inline and customize set_direct_or_vtable_call
44738
11431bbc9549 8168699: Validate special case invocations
coleenp
parents: 40655
diff changeset
   312
  set_direct_or_vtable_call(invoke_code, method, index, false);
20017
81eba62e9048 8014013: CallInfo structure no longer accurately reports the result of a LinkResolver operation
drchase
parents: 17858
diff changeset
   313
}
81eba62e9048 8014013: CallInfo structure no longer accurately reports the result of a LinkResolver operation
drchase
parents: 17858
diff changeset
   314
48557
2e867226b914 8174962: Better interface invocations
vlivanov
parents: 47890
diff changeset
   315
void ConstantPoolCacheEntry::set_itable_call(Bytecodes::Code invoke_code,
2e867226b914 8174962: Better interface invocations
vlivanov
parents: 47890
diff changeset
   316
                                             Klass* referenced_klass,
2e867226b914 8174962: Better interface invocations
vlivanov
parents: 47890
diff changeset
   317
                                             const methodHandle& method, int index) {
20017
81eba62e9048 8014013: CallInfo structure no longer accurately reports the result of a LinkResolver operation
drchase
parents: 17858
diff changeset
   318
  assert(method->method_holder()->verify_itable_index(index), "");
81eba62e9048 8014013: CallInfo structure no longer accurately reports the result of a LinkResolver operation
drchase
parents: 17858
diff changeset
   319
  assert(invoke_code == Bytecodes::_invokeinterface, "");
14391
df0a1573d5bd 8000725: NPG: method_holder() and pool_holder() and pool_holder field should be InstanceKlass
coleenp
parents: 13929
diff changeset
   320
  InstanceKlass* interf = method->method_holder();
df0a1573d5bd 8000725: NPG: method_holder() and pool_holder() and pool_holder field should be InstanceKlass
coleenp
parents: 13929
diff changeset
   321
  assert(interf->is_interface(), "must be an interface");
13391
30245956af37 7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents: 11956
diff changeset
   322
  assert(!method->is_final_method(), "interfaces do not have final methods; cannot link to one here");
48557
2e867226b914 8174962: Better interface invocations
vlivanov
parents: 47890
diff changeset
   323
  set_f1(referenced_klass);
2e867226b914 8174962: Better interface invocations
vlivanov
parents: 47890
diff changeset
   324
  set_f2((intx)method());
13391
30245956af37 7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents: 11956
diff changeset
   325
  set_method_flags(as_TosState(method->result_type()),
30245956af37 7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents: 11956
diff changeset
   326
                   0,  // no option bits
30245956af37 7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents: 11956
diff changeset
   327
                   method()->size_of_parameters());
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   328
  set_bytecode_1(Bytecodes::_invokeinterface);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   329
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
   330
489c9b5090e2 Initial load
duke
parents:
diff changeset
   331
33593
60764a78fa5c 8140274: methodHandles and constantPoolHandles should be passed as const references
coleenp
parents: 33148
diff changeset
   332
void ConstantPoolCacheEntry::set_method_handle(const constantPoolHandle& cpool, const CallInfo &call_info) {
14393
93a4dcdbebfd 8001658: No need to pass resolved_references as argument to ConstantPoolCacheEntry::set_method_handle_common
twisti
parents: 13929
diff changeset
   333
  set_method_handle_common(cpool, Bytecodes::_invokehandle, call_info);
6062
bab93afe9df7 6964498: JSR 292 invokedynamic sites need local bootstrap methods
jrose
parents: 5547
diff changeset
   334
}
bab93afe9df7 6964498: JSR 292 invokedynamic sites need local bootstrap methods
jrose
parents: 5547
diff changeset
   335
33593
60764a78fa5c 8140274: methodHandles and constantPoolHandles should be passed as const references
coleenp
parents: 33148
diff changeset
   336
void ConstantPoolCacheEntry::set_dynamic_call(const constantPoolHandle& cpool, const CallInfo &call_info) {
14393
93a4dcdbebfd 8001658: No need to pass resolved_references as argument to ConstantPoolCacheEntry::set_method_handle_common
twisti
parents: 13929
diff changeset
   337
  set_method_handle_common(cpool, Bytecodes::_invokedynamic, call_info);
6062
bab93afe9df7 6964498: JSR 292 invokedynamic sites need local bootstrap methods
jrose
parents: 5547
diff changeset
   338
}
bab93afe9df7 6964498: JSR 292 invokedynamic sites need local bootstrap methods
jrose
parents: 5547
diff changeset
   339
33593
60764a78fa5c 8140274: methodHandles and constantPoolHandles should be passed as const references
coleenp
parents: 33148
diff changeset
   340
void ConstantPoolCacheEntry::set_method_handle_common(const constantPoolHandle& cpool,
13740
ae6179641137 7196242: vm/mlvm/indy/stress/java/loopsAndThreads crashed
twisti
parents: 13728
diff changeset
   341
                                                      Bytecodes::Code invoke_code,
14393
93a4dcdbebfd 8001658: No need to pass resolved_references as argument to ConstantPoolCacheEntry::set_method_handle_common
twisti
parents: 13929
diff changeset
   342
                                                      const CallInfo &call_info) {
13391
30245956af37 7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents: 11956
diff changeset
   343
  // NOTE: This CPCE can be the subject of data races.
13728
882756847a04 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 13391
diff changeset
   344
  // There are three words to update: flags, refs[f2], f1 (in that order).
13391
30245956af37 7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents: 11956
diff changeset
   345
  // Writers must store all other values before f1.
30245956af37 7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents: 11956
diff changeset
   346
  // Readers must test f1 first for non-null before reading other fields.
13740
ae6179641137 7196242: vm/mlvm/indy/stress/java/loopsAndThreads crashed
twisti
parents: 13728
diff changeset
   347
  // Competing writers must acquire exclusive access via a lock.
ae6179641137 7196242: vm/mlvm/indy/stress/java/loopsAndThreads crashed
twisti
parents: 13728
diff changeset
   348
  // A losing writer waits on the lock until the winner writes f1 and leaves
ae6179641137 7196242: vm/mlvm/indy/stress/java/loopsAndThreads crashed
twisti
parents: 13728
diff changeset
   349
  // the lock, so that when the losing writer returns, he can use the linked
ae6179641137 7196242: vm/mlvm/indy/stress/java/loopsAndThreads crashed
twisti
parents: 13728
diff changeset
   350
  // cache entry.
ae6179641137 7196242: vm/mlvm/indy/stress/java/loopsAndThreads crashed
twisti
parents: 13728
diff changeset
   351
46271
979ebd346ecf 8169881: Remove implicit Handle conversions oop->Handle
coleenp
parents: 40655
diff changeset
   352
  objArrayHandle resolved_references(Thread::current(), cpool->resolved_references());
28019
a6303ff71902 8066171: Out of order with Metaspace allocation lock
coleenp
parents: 25354
diff changeset
   353
  // Use the resolved_references() lock for this cpCache entry.
a6303ff71902 8066171: Out of order with Metaspace allocation lock
coleenp
parents: 25354
diff changeset
   354
  // resolved_references are created for all classes with Invokedynamic, MethodHandle
a6303ff71902 8066171: Out of order with Metaspace allocation lock
coleenp
parents: 25354
diff changeset
   355
  // or MethodType constant pool cache entries.
a6303ff71902 8066171: Out of order with Metaspace allocation lock
coleenp
parents: 25354
diff changeset
   356
  assert(resolved_references() != NULL,
a6303ff71902 8066171: Out of order with Metaspace allocation lock
coleenp
parents: 25354
diff changeset
   357
         "a resolved_references array should have been created for this class");
a6303ff71902 8066171: Out of order with Metaspace allocation lock
coleenp
parents: 25354
diff changeset
   358
  ObjectLocker ol(resolved_references, Thread::current());
13740
ae6179641137 7196242: vm/mlvm/indy/stress/java/loopsAndThreads crashed
twisti
parents: 13728
diff changeset
   359
  if (!is_f1_null()) {
ae6179641137 7196242: vm/mlvm/indy/stress/java/loopsAndThreads crashed
twisti
parents: 13728
diff changeset
   360
    return;
ae6179641137 7196242: vm/mlvm/indy/stress/java/loopsAndThreads crashed
twisti
parents: 13728
diff changeset
   361
  }
13391
30245956af37 7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents: 11956
diff changeset
   362
47684
c3c04b6e14f8 8174954: Parameter target type is allowed access after a module read edge or a package export has occurred after failed resolution
hseigel
parents: 47634
diff changeset
   363
  if (indy_resolution_failed()) {
c3c04b6e14f8 8174954: Parameter target type is allowed access after a module read edge or a package export has occurred after failed resolution
hseigel
parents: 47634
diff changeset
   364
    // Before we got here, another thread got a LinkageError exception during
c3c04b6e14f8 8174954: Parameter target type is allowed access after a module read edge or a package export has occurred after failed resolution
hseigel
parents: 47634
diff changeset
   365
    // resolution.  Ignore our success and throw their exception.
c3c04b6e14f8 8174954: Parameter target type is allowed access after a module read edge or a package export has occurred after failed resolution
hseigel
parents: 47634
diff changeset
   366
    ConstantPoolCache* cpCache = cpool->cache();
c3c04b6e14f8 8174954: Parameter target type is allowed access after a module read edge or a package export has occurred after failed resolution
hseigel
parents: 47634
diff changeset
   367
    int index = -1;
c3c04b6e14f8 8174954: Parameter target type is allowed access after a module read edge or a package export has occurred after failed resolution
hseigel
parents: 47634
diff changeset
   368
    for (int i = 0; i < cpCache->length(); i++) {
c3c04b6e14f8 8174954: Parameter target type is allowed access after a module read edge or a package export has occurred after failed resolution
hseigel
parents: 47634
diff changeset
   369
      if (cpCache->entry_at(i) == this) {
c3c04b6e14f8 8174954: Parameter target type is allowed access after a module read edge or a package export has occurred after failed resolution
hseigel
parents: 47634
diff changeset
   370
        index = i;
c3c04b6e14f8 8174954: Parameter target type is allowed access after a module read edge or a package export has occurred after failed resolution
hseigel
parents: 47634
diff changeset
   371
        break;
c3c04b6e14f8 8174954: Parameter target type is allowed access after a module read edge or a package export has occurred after failed resolution
hseigel
parents: 47634
diff changeset
   372
      }
c3c04b6e14f8 8174954: Parameter target type is allowed access after a module read edge or a package export has occurred after failed resolution
hseigel
parents: 47634
diff changeset
   373
    }
c3c04b6e14f8 8174954: Parameter target type is allowed access after a module read edge or a package export has occurred after failed resolution
hseigel
parents: 47634
diff changeset
   374
    guarantee(index >= 0, "Didn't find cpCache entry!");
c3c04b6e14f8 8174954: Parameter target type is allowed access after a module read edge or a package export has occurred after failed resolution
hseigel
parents: 47634
diff changeset
   375
    int encoded_index = ResolutionErrorTable::encode_cpcache_index(
c3c04b6e14f8 8174954: Parameter target type is allowed access after a module read edge or a package export has occurred after failed resolution
hseigel
parents: 47634
diff changeset
   376
                          ConstantPool::encode_invokedynamic_index(index));
c3c04b6e14f8 8174954: Parameter target type is allowed access after a module read edge or a package export has occurred after failed resolution
hseigel
parents: 47634
diff changeset
   377
    Thread* THREAD = Thread::current();
c3c04b6e14f8 8174954: Parameter target type is allowed access after a module read edge or a package export has occurred after failed resolution
hseigel
parents: 47634
diff changeset
   378
    ConstantPool::throw_resolution_error(cpool, encoded_index, THREAD);
c3c04b6e14f8 8174954: Parameter target type is allowed access after a module read edge or a package export has occurred after failed resolution
hseigel
parents: 47634
diff changeset
   379
    return;
c3c04b6e14f8 8174954: Parameter target type is allowed access after a module read edge or a package export has occurred after failed resolution
hseigel
parents: 47634
diff changeset
   380
  }
c3c04b6e14f8 8174954: Parameter target type is allowed access after a module read edge or a package export has occurred after failed resolution
hseigel
parents: 47634
diff changeset
   381
14393
93a4dcdbebfd 8001658: No need to pass resolved_references as argument to ConstantPoolCacheEntry::set_method_handle_common
twisti
parents: 13929
diff changeset
   382
  const methodHandle adapter = call_info.resolved_method();
93a4dcdbebfd 8001658: No need to pass resolved_references as argument to ConstantPoolCacheEntry::set_method_handle_common
twisti
parents: 13929
diff changeset
   383
  const Handle appendix      = call_info.resolved_appendix();
93a4dcdbebfd 8001658: No need to pass resolved_references as argument to ConstantPoolCacheEntry::set_method_handle_common
twisti
parents: 13929
diff changeset
   384
  const Handle method_type   = call_info.resolved_method_type();
13929
8da0dc50a6e4 7200949: JSR 292: rubybench/bench/time/bench_base64.rb fails with jruby.jar not on boot class path
twisti
parents: 13740
diff changeset
   385
  const bool has_appendix    = appendix.not_null();
8da0dc50a6e4 7200949: JSR 292: rubybench/bench/time/bench_base64.rb fails with jruby.jar not on boot class path
twisti
parents: 13740
diff changeset
   386
  const bool has_method_type = method_type.not_null();
13391
30245956af37 7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents: 11956
diff changeset
   387
13728
882756847a04 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 13391
diff changeset
   388
  // Write the flags.
13740
ae6179641137 7196242: vm/mlvm/indy/stress/java/loopsAndThreads crashed
twisti
parents: 13728
diff changeset
   389
  set_method_flags(as_TosState(adapter->result_type()),
13929
8da0dc50a6e4 7200949: JSR 292: rubybench/bench/time/bench_base64.rb fails with jruby.jar not on boot class path
twisti
parents: 13740
diff changeset
   390
                   ((has_appendix    ? 1 : 0) << has_appendix_shift   ) |
8da0dc50a6e4 7200949: JSR 292: rubybench/bench/time/bench_base64.rb fails with jruby.jar not on boot class path
twisti
parents: 13740
diff changeset
   391
                   ((has_method_type ? 1 : 0) << has_method_type_shift) |
8da0dc50a6e4 7200949: JSR 292: rubybench/bench/time/bench_base64.rb fails with jruby.jar not on boot class path
twisti
parents: 13740
diff changeset
   392
                   (                   1      << is_final_shift       ),
13391
30245956af37 7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents: 11956
diff changeset
   393
                   adapter->size_of_parameters());
30245956af37 7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents: 11956
diff changeset
   394
30245956af37 7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents: 11956
diff changeset
   395
  if (TraceInvokeDynamic) {
35543
0961315f4016 8140659: C1: invokedynamic call patching violates JVMS-6.5.invokedynamic
vlivanov
parents: 33611
diff changeset
   396
    ttyLocker ttyl;
31592
43f48e165466 8081202: Hotspot compile warning: "Invalid suffix on literal; C++11 requires a space between literal and identifier"
bpittore
parents: 30289
diff changeset
   397
    tty->print_cr("set_method_handle bc=%d appendix=" PTR_FORMAT "%s method_type=" PTR_FORMAT "%s method=" PTR_FORMAT " ",
13391
30245956af37 7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents: 11956
diff changeset
   398
                  invoke_code,
33148
68fa8b6c4340 8042893: compiler: PRAGMA_FORMAT_MUTE_WARNINGS_FOR_GCC needs to be removed from source files
david
parents: 33105
diff changeset
   399
                  p2i(appendix()),    (has_appendix    ? "" : " (unused)"),
68fa8b6c4340 8042893: compiler: PRAGMA_FORMAT_MUTE_WARNINGS_FOR_GCC needs to be removed from source files
david
parents: 33105
diff changeset
   400
                  p2i(method_type()), (has_method_type ? "" : " (unused)"),
68fa8b6c4340 8042893: compiler: PRAGMA_FORMAT_MUTE_WARNINGS_FOR_GCC needs to be removed from source files
david
parents: 33105
diff changeset
   401
                  p2i(adapter()));
13391
30245956af37 7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents: 11956
diff changeset
   402
    adapter->print();
30245956af37 7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents: 11956
diff changeset
   403
    if (has_appendix)  appendix()->print();
30245956af37 7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents: 11956
diff changeset
   404
  }
30245956af37 7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents: 11956
diff changeset
   405
30245956af37 7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents: 11956
diff changeset
   406
  // Method handle invokes and invokedynamic sites use both cp cache words.
13728
882756847a04 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 13391
diff changeset
   407
  // refs[f2], if not null, contains a value passed as a trailing argument to the adapter.
13391
30245956af37 7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents: 11956
diff changeset
   408
  // In the general case, this could be the call site's MethodType,
30245956af37 7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents: 11956
diff changeset
   409
  // for use with java.lang.Invokers.checkExactType, or else a CallSite object.
13728
882756847a04 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 13391
diff changeset
   410
  // f1 contains the adapter method which manages the actual call.
13391
30245956af37 7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents: 11956
diff changeset
   411
  // In the general case, this is a compiled LambdaForm.
30245956af37 7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents: 11956
diff changeset
   412
  // (The Java code is free to optimize these calls by binding other
30245956af37 7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents: 11956
diff changeset
   413
  // sorts of methods and appendices to call sites.)
13728
882756847a04 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 13391
diff changeset
   414
  // JVM-level linking is via f1, as if for invokespecial, and signatures are erased.
13391
30245956af37 7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents: 11956
diff changeset
   415
  // The appendix argument (if any) is added to the signature, and is counted in the parameter_size bits.
13728
882756847a04 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 13391
diff changeset
   416
  // Even with the appendix, the method will never take more than 255 parameter slots.
13391
30245956af37 7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents: 11956
diff changeset
   417
  //
30245956af37 7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents: 11956
diff changeset
   418
  // This means that given a call site like (List)mh.invoke("foo"),
13728
882756847a04 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 13391
diff changeset
   419
  // the f1 method has signature '(Ljl/Object;Ljl/invoke/MethodType;)Ljl/Object;',
13391
30245956af37 7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents: 11956
diff changeset
   420
  // not '(Ljava/lang/String;)Ljava/util/List;'.
13728
882756847a04 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 13391
diff changeset
   421
  // The fact that String and List are involved is encoded in the MethodType in refs[f2].
23515
f4872ef5df09 8031820: NPG: Fix remaining references to metadata as oops in comments
coleenp
parents: 22859
diff changeset
   422
  // This allows us to create fewer Methods, while keeping type safety.
13391
30245956af37 7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents: 11956
diff changeset
   423
  //
13728
882756847a04 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 13391
diff changeset
   424
13929
8da0dc50a6e4 7200949: JSR 292: rubybench/bench/time/bench_base64.rb fails with jruby.jar not on boot class path
twisti
parents: 13740
diff changeset
   425
  // Store appendix, if any.
13728
882756847a04 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 13391
diff changeset
   426
  if (has_appendix) {
13929
8da0dc50a6e4 7200949: JSR 292: rubybench/bench/time/bench_base64.rb fails with jruby.jar not on boot class path
twisti
parents: 13740
diff changeset
   427
    const int appendix_index = f2_as_index() + _indy_resolved_references_appendix_offset;
8da0dc50a6e4 7200949: JSR 292: rubybench/bench/time/bench_base64.rb fails with jruby.jar not on boot class path
twisti
parents: 13740
diff changeset
   428
    assert(appendix_index >= 0 && appendix_index < resolved_references->length(), "oob");
8da0dc50a6e4 7200949: JSR 292: rubybench/bench/time/bench_base64.rb fails with jruby.jar not on boot class path
twisti
parents: 13740
diff changeset
   429
    assert(resolved_references->obj_at(appendix_index) == NULL, "init just once");
8da0dc50a6e4 7200949: JSR 292: rubybench/bench/time/bench_base64.rb fails with jruby.jar not on boot class path
twisti
parents: 13740
diff changeset
   430
    resolved_references->obj_at_put(appendix_index, appendix());
8da0dc50a6e4 7200949: JSR 292: rubybench/bench/time/bench_base64.rb fails with jruby.jar not on boot class path
twisti
parents: 13740
diff changeset
   431
  }
8da0dc50a6e4 7200949: JSR 292: rubybench/bench/time/bench_base64.rb fails with jruby.jar not on boot class path
twisti
parents: 13740
diff changeset
   432
8da0dc50a6e4 7200949: JSR 292: rubybench/bench/time/bench_base64.rb fails with jruby.jar not on boot class path
twisti
parents: 13740
diff changeset
   433
  // Store MethodType, if any.
8da0dc50a6e4 7200949: JSR 292: rubybench/bench/time/bench_base64.rb fails with jruby.jar not on boot class path
twisti
parents: 13740
diff changeset
   434
  if (has_method_type) {
8da0dc50a6e4 7200949: JSR 292: rubybench/bench/time/bench_base64.rb fails with jruby.jar not on boot class path
twisti
parents: 13740
diff changeset
   435
    const int method_type_index = f2_as_index() + _indy_resolved_references_method_type_offset;
8da0dc50a6e4 7200949: JSR 292: rubybench/bench/time/bench_base64.rb fails with jruby.jar not on boot class path
twisti
parents: 13740
diff changeset
   436
    assert(method_type_index >= 0 && method_type_index < resolved_references->length(), "oob");
8da0dc50a6e4 7200949: JSR 292: rubybench/bench/time/bench_base64.rb fails with jruby.jar not on boot class path
twisti
parents: 13740
diff changeset
   437
    assert(resolved_references->obj_at(method_type_index) == NULL, "init just once");
8da0dc50a6e4 7200949: JSR 292: rubybench/bench/time/bench_base64.rb fails with jruby.jar not on boot class path
twisti
parents: 13740
diff changeset
   438
    resolved_references->obj_at_put(method_type_index, method_type());
13728
882756847a04 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 13391
diff changeset
   439
  }
882756847a04 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 13391
diff changeset
   440
882756847a04 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 13391
diff changeset
   441
  release_set_f1(adapter());  // This must be the last one to set (see NOTE above)!
882756847a04 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 13391
diff changeset
   442
13929
8da0dc50a6e4 7200949: JSR 292: rubybench/bench/time/bench_base64.rb fails with jruby.jar not on boot class path
twisti
parents: 13740
diff changeset
   443
  // The interpreter assembly code does not check byte_2,
8da0dc50a6e4 7200949: JSR 292: rubybench/bench/time/bench_base64.rb fails with jruby.jar not on boot class path
twisti
parents: 13740
diff changeset
   444
  // but it is used by is_resolved, method_if_resolved, etc.
13728
882756847a04 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 13391
diff changeset
   445
  set_bytecode_1(invoke_code);
13391
30245956af37 7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents: 11956
diff changeset
   446
  NOT_PRODUCT(verify(tty));
30245956af37 7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents: 11956
diff changeset
   447
  if (TraceInvokeDynamic) {
35543
0961315f4016 8140659: C1: invokedynamic call patching violates JVMS-6.5.invokedynamic
vlivanov
parents: 33611
diff changeset
   448
    ttyLocker ttyl;
13391
30245956af37 7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents: 11956
diff changeset
   449
    this->print(tty, 0);
30245956af37 7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents: 11956
diff changeset
   450
  }
2570
ecc7862946d4 6655646: dynamic languages need dynamically linked call sites
jrose
parents: 670
diff changeset
   451
}
ecc7862946d4 6655646: dynamic languages need dynamically linked call sites
jrose
parents: 670
diff changeset
   452
47684
c3c04b6e14f8 8174954: Parameter target type is allowed access after a module read edge or a package export has occurred after failed resolution
hseigel
parents: 47634
diff changeset
   453
bool ConstantPoolCacheEntry::save_and_throw_indy_exc(
c3c04b6e14f8 8174954: Parameter target type is allowed access after a module read edge or a package export has occurred after failed resolution
hseigel
parents: 47634
diff changeset
   454
  const constantPoolHandle& cpool, int cpool_index, int index, constantTag tag, TRAPS) {
c3c04b6e14f8 8174954: Parameter target type is allowed access after a module read edge or a package export has occurred after failed resolution
hseigel
parents: 47634
diff changeset
   455
c3c04b6e14f8 8174954: Parameter target type is allowed access after a module read edge or a package export has occurred after failed resolution
hseigel
parents: 47634
diff changeset
   456
  assert(HAS_PENDING_EXCEPTION, "No exception got thrown!");
c3c04b6e14f8 8174954: Parameter target type is allowed access after a module read edge or a package export has occurred after failed resolution
hseigel
parents: 47634
diff changeset
   457
  assert(PENDING_EXCEPTION->is_a(SystemDictionary::LinkageError_klass()),
c3c04b6e14f8 8174954: Parameter target type is allowed access after a module read edge or a package export has occurred after failed resolution
hseigel
parents: 47634
diff changeset
   458
         "No LinkageError exception");
c3c04b6e14f8 8174954: Parameter target type is allowed access after a module read edge or a package export has occurred after failed resolution
hseigel
parents: 47634
diff changeset
   459
c3c04b6e14f8 8174954: Parameter target type is allowed access after a module read edge or a package export has occurred after failed resolution
hseigel
parents: 47634
diff changeset
   460
  // Use the resolved_references() lock for this cpCache entry.
c3c04b6e14f8 8174954: Parameter target type is allowed access after a module read edge or a package export has occurred after failed resolution
hseigel
parents: 47634
diff changeset
   461
  // resolved_references are created for all classes with Invokedynamic, MethodHandle
c3c04b6e14f8 8174954: Parameter target type is allowed access after a module read edge or a package export has occurred after failed resolution
hseigel
parents: 47634
diff changeset
   462
  // or MethodType constant pool cache entries.
c3c04b6e14f8 8174954: Parameter target type is allowed access after a module read edge or a package export has occurred after failed resolution
hseigel
parents: 47634
diff changeset
   463
  objArrayHandle resolved_references(Thread::current(), cpool->resolved_references());
c3c04b6e14f8 8174954: Parameter target type is allowed access after a module read edge or a package export has occurred after failed resolution
hseigel
parents: 47634
diff changeset
   464
  assert(resolved_references() != NULL,
c3c04b6e14f8 8174954: Parameter target type is allowed access after a module read edge or a package export has occurred after failed resolution
hseigel
parents: 47634
diff changeset
   465
         "a resolved_references array should have been created for this class");
c3c04b6e14f8 8174954: Parameter target type is allowed access after a module read edge or a package export has occurred after failed resolution
hseigel
parents: 47634
diff changeset
   466
  ObjectLocker ol(resolved_references, THREAD);
c3c04b6e14f8 8174954: Parameter target type is allowed access after a module read edge or a package export has occurred after failed resolution
hseigel
parents: 47634
diff changeset
   467
c3c04b6e14f8 8174954: Parameter target type is allowed access after a module read edge or a package export has occurred after failed resolution
hseigel
parents: 47634
diff changeset
   468
  // if f1 is not null or the indy_resolution_failed flag is set then another
c3c04b6e14f8 8174954: Parameter target type is allowed access after a module read edge or a package export has occurred after failed resolution
hseigel
parents: 47634
diff changeset
   469
  // thread either succeeded in resolving the method or got a LinkageError
c3c04b6e14f8 8174954: Parameter target type is allowed access after a module read edge or a package export has occurred after failed resolution
hseigel
parents: 47634
diff changeset
   470
  // exception, before this thread was able to record its failure.  So, clear
c3c04b6e14f8 8174954: Parameter target type is allowed access after a module read edge or a package export has occurred after failed resolution
hseigel
parents: 47634
diff changeset
   471
  // this thread's exception and return false so caller can use the earlier
c3c04b6e14f8 8174954: Parameter target type is allowed access after a module read edge or a package export has occurred after failed resolution
hseigel
parents: 47634
diff changeset
   472
  // thread's result.
c3c04b6e14f8 8174954: Parameter target type is allowed access after a module read edge or a package export has occurred after failed resolution
hseigel
parents: 47634
diff changeset
   473
  if (!is_f1_null() || indy_resolution_failed()) {
c3c04b6e14f8 8174954: Parameter target type is allowed access after a module read edge or a package export has occurred after failed resolution
hseigel
parents: 47634
diff changeset
   474
    CLEAR_PENDING_EXCEPTION;
c3c04b6e14f8 8174954: Parameter target type is allowed access after a module read edge or a package export has occurred after failed resolution
hseigel
parents: 47634
diff changeset
   475
    return false;
c3c04b6e14f8 8174954: Parameter target type is allowed access after a module read edge or a package export has occurred after failed resolution
hseigel
parents: 47634
diff changeset
   476
  }
c3c04b6e14f8 8174954: Parameter target type is allowed access after a module read edge or a package export has occurred after failed resolution
hseigel
parents: 47634
diff changeset
   477
c3c04b6e14f8 8174954: Parameter target type is allowed access after a module read edge or a package export has occurred after failed resolution
hseigel
parents: 47634
diff changeset
   478
  Symbol* error = PENDING_EXCEPTION->klass()->name();
c3c04b6e14f8 8174954: Parameter target type is allowed access after a module read edge or a package export has occurred after failed resolution
hseigel
parents: 47634
diff changeset
   479
  Symbol* message = java_lang_Throwable::detail_message(PENDING_EXCEPTION);
47890
2219a502fd45 8191132: assert condition should not be in quotes
hseigel
parents: 47684
diff changeset
   480
  assert(message != NULL, "Missing detail message");
47684
c3c04b6e14f8 8174954: Parameter target type is allowed access after a module read edge or a package export has occurred after failed resolution
hseigel
parents: 47634
diff changeset
   481
c3c04b6e14f8 8174954: Parameter target type is allowed access after a module read edge or a package export has occurred after failed resolution
hseigel
parents: 47634
diff changeset
   482
  SystemDictionary::add_resolution_error(cpool, index, error, message);
c3c04b6e14f8 8174954: Parameter target type is allowed access after a module read edge or a package export has occurred after failed resolution
hseigel
parents: 47634
diff changeset
   483
  set_indy_resolution_failed();
c3c04b6e14f8 8174954: Parameter target type is allowed access after a module read edge or a package export has occurred after failed resolution
hseigel
parents: 47634
diff changeset
   484
  return true;
c3c04b6e14f8 8174954: Parameter target type is allowed access after a module read edge or a package export has occurred after failed resolution
hseigel
parents: 47634
diff changeset
   485
}
c3c04b6e14f8 8174954: Parameter target type is allowed access after a module read edge or a package export has occurred after failed resolution
hseigel
parents: 47634
diff changeset
   486
33593
60764a78fa5c 8140274: methodHandles and constantPoolHandles should be passed as const references
coleenp
parents: 33148
diff changeset
   487
Method* ConstantPoolCacheEntry::method_if_resolved(const constantPoolHandle& cpool) {
10008
d84de97ad847 7056328: JSR 292 invocation sometimes fails in adapters for types not on boot class path
jrose
parents: 9328
diff changeset
   488
  // Decode the action of set_method and set_interface_call
13391
30245956af37 7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents: 11956
diff changeset
   489
  Bytecodes::Code invoke_code = bytecode_1();
30245956af37 7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents: 11956
diff changeset
   490
  if (invoke_code != (Bytecodes::Code)0) {
22859
7b88983393b7 8029396: PPC64 (part 212): Several memory ordering fixes in C-code.
goetz
parents: 21557
diff changeset
   491
    Metadata* f1 = f1_ord();
10008
d84de97ad847 7056328: JSR 292 invocation sometimes fails in adapters for types not on boot class path
jrose
parents: 9328
diff changeset
   492
    if (f1 != NULL) {
d84de97ad847 7056328: JSR 292 invocation sometimes fails in adapters for types not on boot class path
jrose
parents: 9328
diff changeset
   493
      switch (invoke_code) {
d84de97ad847 7056328: JSR 292 invocation sometimes fails in adapters for types not on boot class path
jrose
parents: 9328
diff changeset
   494
      case Bytecodes::_invokeinterface:
d84de97ad847 7056328: JSR 292 invocation sometimes fails in adapters for types not on boot class path
jrose
parents: 9328
diff changeset
   495
        assert(f1->is_klass(), "");
51329
9c68699bebe5 8208999: Some use of Klass* should be replaced by InstanceKlass*
iklam
parents: 50735
diff changeset
   496
        return klassItable::method_for_itable_index((InstanceKlass*)f1, f2_as_index());
10008
d84de97ad847 7056328: JSR 292 invocation sometimes fails in adapters for types not on boot class path
jrose
parents: 9328
diff changeset
   497
      case Bytecodes::_invokestatic:
d84de97ad847 7056328: JSR 292 invocation sometimes fails in adapters for types not on boot class path
jrose
parents: 9328
diff changeset
   498
      case Bytecodes::_invokespecial:
13391
30245956af37 7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents: 11956
diff changeset
   499
        assert(!has_appendix(), "");
13728
882756847a04 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 13391
diff changeset
   500
      case Bytecodes::_invokehandle:
882756847a04 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 13391
diff changeset
   501
      case Bytecodes::_invokedynamic:
10008
d84de97ad847 7056328: JSR 292 invocation sometimes fails in adapters for types not on boot class path
jrose
parents: 9328
diff changeset
   502
        assert(f1->is_method(), "");
13728
882756847a04 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 13391
diff changeset
   503
        return (Method*)f1;
46630
75aa3e39d02c 8182299: Enable disabled clang warnings, build on OSX 10 + Xcode 8
jwilhelm
parents: 46458
diff changeset
   504
      default:
75aa3e39d02c 8182299: Enable disabled clang warnings, build on OSX 10 + Xcode 8
jwilhelm
parents: 46458
diff changeset
   505
        break;
10008
d84de97ad847 7056328: JSR 292 invocation sometimes fails in adapters for types not on boot class path
jrose
parents: 9328
diff changeset
   506
      }
d84de97ad847 7056328: JSR 292 invocation sometimes fails in adapters for types not on boot class path
jrose
parents: 9328
diff changeset
   507
    }
d84de97ad847 7056328: JSR 292 invocation sometimes fails in adapters for types not on boot class path
jrose
parents: 9328
diff changeset
   508
  }
13391
30245956af37 7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents: 11956
diff changeset
   509
  invoke_code = bytecode_2();
30245956af37 7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents: 11956
diff changeset
   510
  if (invoke_code != (Bytecodes::Code)0) {
10008
d84de97ad847 7056328: JSR 292 invocation sometimes fails in adapters for types not on boot class path
jrose
parents: 9328
diff changeset
   511
    switch (invoke_code) {
d84de97ad847 7056328: JSR 292 invocation sometimes fails in adapters for types not on boot class path
jrose
parents: 9328
diff changeset
   512
    case Bytecodes::_invokevirtual:
d84de97ad847 7056328: JSR 292 invocation sometimes fails in adapters for types not on boot class path
jrose
parents: 9328
diff changeset
   513
      if (is_vfinal()) {
d84de97ad847 7056328: JSR 292 invocation sometimes fails in adapters for types not on boot class path
jrose
parents: 9328
diff changeset
   514
        // invokevirtual
13728
882756847a04 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 13391
diff changeset
   515
        Method* m = f2_as_vfinal_method();
10008
d84de97ad847 7056328: JSR 292 invocation sometimes fails in adapters for types not on boot class path
jrose
parents: 9328
diff changeset
   516
        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
   517
        return m;
d84de97ad847 7056328: JSR 292 invocation sometimes fails in adapters for types not on boot class path
jrose
parents: 9328
diff changeset
   518
      } else {
d84de97ad847 7056328: JSR 292 invocation sometimes fails in adapters for types not on boot class path
jrose
parents: 9328
diff changeset
   519
        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
   520
        if (cpool->tag_at(holder_index).is_klass()) {
13728
882756847a04 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 13391
diff changeset
   521
          Klass* klass = cpool->resolved_klass_at(holder_index);
35900
d64cf9290fc4 8148481: Devirtualize Klass::vtable
mgerdin
parents: 33611
diff changeset
   522
          return klass->method_at_vtable(f2_as_index());
10008
d84de97ad847 7056328: JSR 292 invocation sometimes fails in adapters for types not on boot class path
jrose
parents: 9328
diff changeset
   523
        }
d84de97ad847 7056328: JSR 292 invocation sometimes fails in adapters for types not on boot class path
jrose
parents: 9328
diff changeset
   524
      }
13391
30245956af37 7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents: 11956
diff changeset
   525
      break;
46630
75aa3e39d02c 8182299: Enable disabled clang warnings, build on OSX 10 + Xcode 8
jwilhelm
parents: 46458
diff changeset
   526
    default:
75aa3e39d02c 8182299: Enable disabled clang warnings, build on OSX 10 + Xcode 8
jwilhelm
parents: 46458
diff changeset
   527
      break;
10008
d84de97ad847 7056328: JSR 292 invocation sometimes fails in adapters for types not on boot class path
jrose
parents: 9328
diff changeset
   528
    }
d84de97ad847 7056328: JSR 292 invocation sometimes fails in adapters for types not on boot class path
jrose
parents: 9328
diff changeset
   529
  }
d84de97ad847 7056328: JSR 292 invocation sometimes fails in adapters for types not on boot class path
jrose
parents: 9328
diff changeset
   530
  return NULL;
d84de97ad847 7056328: JSR 292 invocation sometimes fails in adapters for types not on boot class path
jrose
parents: 9328
diff changeset
   531
}
d84de97ad847 7056328: JSR 292 invocation sometimes fails in adapters for types not on boot class path
jrose
parents: 9328
diff changeset
   532
d84de97ad847 7056328: JSR 292 invocation sometimes fails in adapters for types not on boot class path
jrose
parents: 9328
diff changeset
   533
33593
60764a78fa5c 8140274: methodHandles and constantPoolHandles should be passed as const references
coleenp
parents: 33148
diff changeset
   534
oop ConstantPoolCacheEntry::appendix_if_resolved(const constantPoolHandle& cpool) {
24926
5ea835dfafaa 8029381: assert(is_method_type()) failed: bad cast
drchase
parents: 24459
diff changeset
   535
  if (!has_appendix())
13728
882756847a04 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 13391
diff changeset
   536
    return NULL;
13929
8da0dc50a6e4 7200949: JSR 292: rubybench/bench/time/bench_base64.rb fails with jruby.jar not on boot class path
twisti
parents: 13740
diff changeset
   537
  const int ref_index = f2_as_index() + _indy_resolved_references_appendix_offset;
8da0dc50a6e4 7200949: JSR 292: rubybench/bench/time/bench_base64.rb fails with jruby.jar not on boot class path
twisti
parents: 13740
diff changeset
   538
  objArrayOop resolved_references = cpool->resolved_references();
8da0dc50a6e4 7200949: JSR 292: rubybench/bench/time/bench_base64.rb fails with jruby.jar not on boot class path
twisti
parents: 13740
diff changeset
   539
  return resolved_references->obj_at(ref_index);
8da0dc50a6e4 7200949: JSR 292: rubybench/bench/time/bench_base64.rb fails with jruby.jar not on boot class path
twisti
parents: 13740
diff changeset
   540
}
8da0dc50a6e4 7200949: JSR 292: rubybench/bench/time/bench_base64.rb fails with jruby.jar not on boot class path
twisti
parents: 13740
diff changeset
   541
8da0dc50a6e4 7200949: JSR 292: rubybench/bench/time/bench_base64.rb fails with jruby.jar not on boot class path
twisti
parents: 13740
diff changeset
   542
33593
60764a78fa5c 8140274: methodHandles and constantPoolHandles should be passed as const references
coleenp
parents: 33148
diff changeset
   543
oop ConstantPoolCacheEntry::method_type_if_resolved(const constantPoolHandle& cpool) {
24926
5ea835dfafaa 8029381: assert(is_method_type()) failed: bad cast
drchase
parents: 24459
diff changeset
   544
  if (!has_method_type())
13929
8da0dc50a6e4 7200949: JSR 292: rubybench/bench/time/bench_base64.rb fails with jruby.jar not on boot class path
twisti
parents: 13740
diff changeset
   545
    return NULL;
8da0dc50a6e4 7200949: JSR 292: rubybench/bench/time/bench_base64.rb fails with jruby.jar not on boot class path
twisti
parents: 13740
diff changeset
   546
  const int ref_index = f2_as_index() + _indy_resolved_references_method_type_offset;
13728
882756847a04 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 13391
diff changeset
   547
  objArrayOop resolved_references = cpool->resolved_references();
882756847a04 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 13391
diff changeset
   548
  return resolved_references->obj_at(ref_index);
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   549
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
   550
489c9b5090e2 Initial load
duke
parents:
diff changeset
   551
15591
b8aa0577f137 7182152: Instrumentation hot swap test incorrect monitor count
dcubed
parents: 14490
diff changeset
   552
#if INCLUDE_JVMTI
48557
2e867226b914 8174962: Better interface invocations
vlivanov
parents: 47890
diff changeset
   553
2e867226b914 8174962: Better interface invocations
vlivanov
parents: 47890
diff changeset
   554
void log_adjust(const char* entry_type, Method* old_method, Method* new_method, bool* trace_name_printed) {
2e867226b914 8174962: Better interface invocations
vlivanov
parents: 47890
diff changeset
   555
  if (log_is_enabled(Info, redefine, class, update)) {
2e867226b914 8174962: Better interface invocations
vlivanov
parents: 47890
diff changeset
   556
    ResourceMark rm;
2e867226b914 8174962: Better interface invocations
vlivanov
parents: 47890
diff changeset
   557
    if (!(*trace_name_printed)) {
2e867226b914 8174962: Better interface invocations
vlivanov
parents: 47890
diff changeset
   558
      log_info(redefine, class, update)("adjust: name=%s", old_method->method_holder()->external_name());
2e867226b914 8174962: Better interface invocations
vlivanov
parents: 47890
diff changeset
   559
      *trace_name_printed = true;
2e867226b914 8174962: Better interface invocations
vlivanov
parents: 47890
diff changeset
   560
    }
2e867226b914 8174962: Better interface invocations
vlivanov
parents: 47890
diff changeset
   561
    log_debug(redefine, class, update, constantpool)
2e867226b914 8174962: Better interface invocations
vlivanov
parents: 47890
diff changeset
   562
          ("cpc %s entry update: %s(%s)", entry_type, new_method->name()->as_C_string(), new_method->signature()->as_C_string());
2e867226b914 8174962: Better interface invocations
vlivanov
parents: 47890
diff changeset
   563
  }
2e867226b914 8174962: Better interface invocations
vlivanov
parents: 47890
diff changeset
   564
}
2e867226b914 8174962: Better interface invocations
vlivanov
parents: 47890
diff changeset
   565
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   566
// RedefineClasses() API support:
15591
b8aa0577f137 7182152: Instrumentation hot swap test incorrect monitor count
dcubed
parents: 14490
diff changeset
   567
// If this ConstantPoolCacheEntry refers to old_method then update it
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   568
// to refer to new_method.
48557
2e867226b914 8174962: Better interface invocations
vlivanov
parents: 47890
diff changeset
   569
void ConstantPoolCacheEntry::adjust_method_entry(Method* old_method,
13728
882756847a04 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 13391
diff changeset
   570
       Method* new_method, bool * trace_name_printed) {
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   571
489c9b5090e2 Initial load
duke
parents:
diff changeset
   572
  if (is_vfinal()) {
13391
30245956af37 7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents: 11956
diff changeset
   573
    // virtual and final so _f2 contains method ptr instead of vtable index
30245956af37 7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents: 11956
diff changeset
   574
    if (f2_as_vfinal_method() == old_method) {
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   575
      // match old_method so need an update
13391
30245956af37 7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents: 11956
diff changeset
   576
      // NOTE: can't use set_f2_as_vfinal_method as it asserts on different values
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   577
      _f2 = (intptr_t)new_method;
48557
2e867226b914 8174962: Better interface invocations
vlivanov
parents: 47890
diff changeset
   578
      log_adjust("vfinal", old_method, new_method, trace_name_printed);
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   579
    }
48557
2e867226b914 8174962: Better interface invocations
vlivanov
parents: 47890
diff changeset
   580
    return;
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   581
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   582
48557
2e867226b914 8174962: Better interface invocations
vlivanov
parents: 47890
diff changeset
   583
  assert (_f1 != NULL, "should not call with uninteresting entry");
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   584
48557
2e867226b914 8174962: Better interface invocations
vlivanov
parents: 47890
diff changeset
   585
  if (!(_f1->is_method())) {
2e867226b914 8174962: Better interface invocations
vlivanov
parents: 47890
diff changeset
   586
    // _f1 is a Klass* for an interface, _f2 is the method
2e867226b914 8174962: Better interface invocations
vlivanov
parents: 47890
diff changeset
   587
    if (f2_as_interface_method() == old_method) {
2e867226b914 8174962: Better interface invocations
vlivanov
parents: 47890
diff changeset
   588
      _f2 = (intptr_t)new_method;
2e867226b914 8174962: Better interface invocations
vlivanov
parents: 47890
diff changeset
   589
      log_adjust("interface", old_method, new_method, trace_name_printed);
2e867226b914 8174962: Better interface invocations
vlivanov
parents: 47890
diff changeset
   590
    }
2e867226b914 8174962: Better interface invocations
vlivanov
parents: 47890
diff changeset
   591
  } else if (_f1 == old_method) {
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   592
    _f1 = new_method;
48557
2e867226b914 8174962: Better interface invocations
vlivanov
parents: 47890
diff changeset
   593
    log_adjust("special, static or dynamic", old_method, new_method, trace_name_printed);
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   594
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   595
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
   596
15591
b8aa0577f137 7182152: Instrumentation hot swap test incorrect monitor count
dcubed
parents: 14490
diff changeset
   597
// a constant pool cache entry should never contain old or obsolete methods
b8aa0577f137 7182152: Instrumentation hot swap test incorrect monitor count
dcubed
parents: 14490
diff changeset
   598
bool ConstantPoolCacheEntry::check_no_old_or_obsolete_entries() {
48557
2e867226b914 8174962: Better interface invocations
vlivanov
parents: 47890
diff changeset
   599
  Method* m = get_interesting_method_entry(NULL);
2e867226b914 8174962: Better interface invocations
vlivanov
parents: 47890
diff changeset
   600
  // return false if m refers to a non-deleted old or obsolete method
2e867226b914 8174962: Better interface invocations
vlivanov
parents: 47890
diff changeset
   601
  if (m != NULL) {
2e867226b914 8174962: Better interface invocations
vlivanov
parents: 47890
diff changeset
   602
    assert(m->is_valid() && m->is_method(), "m is a valid method");
2e867226b914 8174962: Better interface invocations
vlivanov
parents: 47890
diff changeset
   603
    return !m->is_old() && !m->is_obsolete(); // old is always set for old and obsolete
2e867226b914 8174962: Better interface invocations
vlivanov
parents: 47890
diff changeset
   604
  } else {
15591
b8aa0577f137 7182152: Instrumentation hot swap test incorrect monitor count
dcubed
parents: 14490
diff changeset
   605
    return true;
13728
882756847a04 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 13391
diff changeset
   606
  }
882756847a04 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 13391
diff changeset
   607
}
882756847a04 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 13391
diff changeset
   608
29316
5287df8a8972 8046246: the constantPoolCacheOopDesc::adjust_method_entries() used in RedefineClasses does not scale
sspitsyn
parents: 29081
diff changeset
   609
Method* ConstantPoolCacheEntry::get_interesting_method_entry(Klass* k) {
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   610
  if (!is_method_entry()) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   611
    // not a method entry so not interesting by default
29316
5287df8a8972 8046246: the constantPoolCacheOopDesc::adjust_method_entries() used in RedefineClasses does not scale
sspitsyn
parents: 29081
diff changeset
   612
    return NULL;
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   613
  }
13728
882756847a04 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 13391
diff changeset
   614
  Method* m = NULL;
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   615
  if (is_vfinal()) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   616
    // virtual and final so _f2 contains method ptr instead of vtable index
13391
30245956af37 7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents: 11956
diff changeset
   617
    m = f2_as_vfinal_method();
30245956af37 7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents: 11956
diff changeset
   618
  } else if (is_f1_null()) {
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   619
    // NULL _f1 means this is a virtual entry so also not interesting
29316
5287df8a8972 8046246: the constantPoolCacheOopDesc::adjust_method_entries() used in RedefineClasses does not scale
sspitsyn
parents: 29081
diff changeset
   620
    return NULL;
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   621
  } else {
13728
882756847a04 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 13391
diff changeset
   622
    if (!(_f1->is_method())) {
48557
2e867226b914 8174962: Better interface invocations
vlivanov
parents: 47890
diff changeset
   623
      // _f1 is a Klass* for an interface
2e867226b914 8174962: Better interface invocations
vlivanov
parents: 47890
diff changeset
   624
      m = f2_as_interface_method();
2e867226b914 8174962: Better interface invocations
vlivanov
parents: 47890
diff changeset
   625
    } else {
2e867226b914 8174962: Better interface invocations
vlivanov
parents: 47890
diff changeset
   626
      m = f1_as_method();
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   627
    }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   628
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   629
  assert(m != NULL && m->is_method(), "sanity check");
13728
882756847a04 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 13391
diff changeset
   630
  if (m == NULL || !m->is_method() || (k != NULL && m->method_holder() != k)) {
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   631
    // robustness for above sanity checks or method is not in
489c9b5090e2 Initial load
duke
parents:
diff changeset
   632
    // the interesting class
29316
5287df8a8972 8046246: the constantPoolCacheOopDesc::adjust_method_entries() used in RedefineClasses does not scale
sspitsyn
parents: 29081
diff changeset
   633
    return NULL;
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   634
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   635
  // the method is in the interesting class so the entry is interesting
29316
5287df8a8972 8046246: the constantPoolCacheOopDesc::adjust_method_entries() used in RedefineClasses does not scale
sspitsyn
parents: 29081
diff changeset
   636
  return m;
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   637
}
15591
b8aa0577f137 7182152: Instrumentation hot swap test incorrect monitor count
dcubed
parents: 14490
diff changeset
   638
#endif // INCLUDE_JVMTI
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   639
489c9b5090e2 Initial load
duke
parents:
diff changeset
   640
void ConstantPoolCacheEntry::print(outputStream* st, int index) const {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   641
  // print separator
11956
c363dc1ac4b7 7148126: ConstantPoolCacheEntry::print prints to wrong stream
dholmes
parents: 10546
diff changeset
   642
  if (index == 0) st->print_cr("                 -------------");
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   643
  // print entry
31592
43f48e165466 8081202: Hotspot compile warning: "Invalid suffix on literal; C++11 requires a space between literal and identifier"
bpittore
parents: 30289
diff changeset
   644
  st->print("%3d  (" PTR_FORMAT ")  ", index, (intptr_t)this);
15591
b8aa0577f137 7182152: Instrumentation hot swap test incorrect monitor count
dcubed
parents: 14490
diff changeset
   645
  st->print_cr("[%02x|%02x|%5d]", bytecode_2(), bytecode_1(),
b8aa0577f137 7182152: Instrumentation hot swap test incorrect monitor count
dcubed
parents: 14490
diff changeset
   646
               constant_pool_index());
31592
43f48e165466 8081202: Hotspot compile warning: "Invalid suffix on literal; C++11 requires a space between literal and identifier"
bpittore
parents: 30289
diff changeset
   647
  st->print_cr("                 [   " PTR_FORMAT "]", (intptr_t)_f1);
43f48e165466 8081202: Hotspot compile warning: "Invalid suffix on literal; C++11 requires a space between literal and identifier"
bpittore
parents: 30289
diff changeset
   648
  st->print_cr("                 [   " PTR_FORMAT "]", (intptr_t)_f2);
43f48e165466 8081202: Hotspot compile warning: "Invalid suffix on literal; C++11 requires a space between literal and identifier"
bpittore
parents: 30289
diff changeset
   649
  st->print_cr("                 [   " PTR_FORMAT "]", (intptr_t)_flags);
11956
c363dc1ac4b7 7148126: ConstantPoolCacheEntry::print prints to wrong stream
dholmes
parents: 10546
diff changeset
   650
  st->print_cr("                 -------------");
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   651
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
   652
489c9b5090e2 Initial load
duke
parents:
diff changeset
   653
void ConstantPoolCacheEntry::verify(outputStream* st) const {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   654
  // not implemented yet
489c9b5090e2 Initial load
duke
parents:
diff changeset
   655
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
   656
489c9b5090e2 Initial load
duke
parents:
diff changeset
   657
// Implementation of ConstantPoolCache
489c9b5090e2 Initial load
duke
parents:
diff changeset
   658
15928
f9d5c6e4107f 8003553: NPG: metaspace objects should be zeroed in constructors
coleenp
parents: 15595
diff changeset
   659
ConstantPoolCache* ConstantPoolCache::allocate(ClassLoaderData* loader_data,
f9d5c6e4107f 8003553: NPG: metaspace objects should be zeroed in constructors
coleenp
parents: 15595
diff changeset
   660
                                     const intStack& index_map,
21557
55115e0708f1 8025937: assert(existing_f1 == NULL || existing_f1 == f1) failed: illegal field change
coleenp
parents: 21079
diff changeset
   661
                                     const intStack& invokedynamic_index_map,
15928
f9d5c6e4107f 8003553: NPG: metaspace objects should be zeroed in constructors
coleenp
parents: 15595
diff changeset
   662
                                     const intStack& invokedynamic_map, TRAPS) {
21557
55115e0708f1 8025937: assert(existing_f1 == NULL || existing_f1 == f1) failed: illegal field change
coleenp
parents: 21079
diff changeset
   663
55115e0708f1 8025937: assert(existing_f1 == NULL || existing_f1 == f1) failed: illegal field change
coleenp
parents: 21079
diff changeset
   664
  const int length = index_map.length() + invokedynamic_index_map.length();
13728
882756847a04 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 13391
diff changeset
   665
  int size = ConstantPoolCache::size(length);
882756847a04 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 13391
diff changeset
   666
46746
ea379ebb9447 8072061: Automatically determine optimal sizes for the CDS regions
iklam
parents: 46727
diff changeset
   667
  return new (loader_data, size, MetaspaceObj::ConstantPoolCacheType, THREAD)
21557
55115e0708f1 8025937: assert(existing_f1 == NULL || existing_f1 == f1) failed: illegal field change
coleenp
parents: 21079
diff changeset
   668
    ConstantPoolCache(length, index_map, invokedynamic_index_map, invokedynamic_map);
13728
882756847a04 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 13391
diff changeset
   669
}
882756847a04 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 13391
diff changeset
   670
15928
f9d5c6e4107f 8003553: NPG: metaspace objects should be zeroed in constructors
coleenp
parents: 15595
diff changeset
   671
void ConstantPoolCache::initialize(const intArray& inverse_index_map,
21557
55115e0708f1 8025937: assert(existing_f1 == NULL || existing_f1 == f1) failed: illegal field change
coleenp
parents: 21079
diff changeset
   672
                                   const intArray& invokedynamic_inverse_index_map,
15928
f9d5c6e4107f 8003553: NPG: metaspace objects should be zeroed in constructors
coleenp
parents: 15595
diff changeset
   673
                                   const intArray& invokedynamic_references_map) {
21557
55115e0708f1 8025937: assert(existing_f1 == NULL || existing_f1 == f1) failed: illegal field change
coleenp
parents: 21079
diff changeset
   674
  for (int i = 0; i < inverse_index_map.length(); i++) {
4429
d7eb4e2099aa 6858164: invokedynamic code needs some cleanup (post-6655638)
jrose
parents: 3261
diff changeset
   675
    ConstantPoolCacheEntry* e = entry_at(i);
38031
e0b822facc03 8149374: Replace C1-specific collection classes with universal collection classes
fzhinkin
parents: 35913
diff changeset
   676
    int original_index = inverse_index_map.at(i);
13929
8da0dc50a6e4 7200949: JSR 292: rubybench/bench/time/bench_base64.rb fails with jruby.jar not on boot class path
twisti
parents: 13740
diff changeset
   677
    e->initialize_entry(original_index);
13728
882756847a04 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 13391
diff changeset
   678
    assert(entry_at(i) == e, "sanity");
13929
8da0dc50a6e4 7200949: JSR 292: rubybench/bench/time/bench_base64.rb fails with jruby.jar not on boot class path
twisti
parents: 13740
diff changeset
   679
  }
21557
55115e0708f1 8025937: assert(existing_f1 == NULL || existing_f1 == f1) failed: illegal field change
coleenp
parents: 21079
diff changeset
   680
55115e0708f1 8025937: assert(existing_f1 == NULL || existing_f1 == f1) failed: illegal field change
coleenp
parents: 21079
diff changeset
   681
  // Append invokedynamic entries at the end
55115e0708f1 8025937: assert(existing_f1 == NULL || existing_f1 == f1) failed: illegal field change
coleenp
parents: 21079
diff changeset
   682
  int invokedynamic_offset = inverse_index_map.length();
55115e0708f1 8025937: assert(existing_f1 == NULL || existing_f1 == f1) failed: illegal field change
coleenp
parents: 21079
diff changeset
   683
  for (int i = 0; i < invokedynamic_inverse_index_map.length(); i++) {
55115e0708f1 8025937: assert(existing_f1 == NULL || existing_f1 == f1) failed: illegal field change
coleenp
parents: 21079
diff changeset
   684
    int offset = i + invokedynamic_offset;
55115e0708f1 8025937: assert(existing_f1 == NULL || existing_f1 == f1) failed: illegal field change
coleenp
parents: 21079
diff changeset
   685
    ConstantPoolCacheEntry* e = entry_at(offset);
38031
e0b822facc03 8149374: Replace C1-specific collection classes with universal collection classes
fzhinkin
parents: 35913
diff changeset
   686
    int original_index = invokedynamic_inverse_index_map.at(i);
21557
55115e0708f1 8025937: assert(existing_f1 == NULL || existing_f1 == f1) failed: illegal field change
coleenp
parents: 21079
diff changeset
   687
    e->initialize_entry(original_index);
55115e0708f1 8025937: assert(existing_f1 == NULL || existing_f1 == f1) failed: illegal field change
coleenp
parents: 21079
diff changeset
   688
    assert(entry_at(offset) == e, "sanity");
55115e0708f1 8025937: assert(existing_f1 == NULL || existing_f1 == f1) failed: illegal field change
coleenp
parents: 21079
diff changeset
   689
  }
55115e0708f1 8025937: assert(existing_f1 == NULL || existing_f1 == f1) failed: illegal field change
coleenp
parents: 21079
diff changeset
   690
13728
882756847a04 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 13391
diff changeset
   691
  for (int ref = 0; ref < invokedynamic_references_map.length(); ref++) {
38031
e0b822facc03 8149374: Replace C1-specific collection classes with universal collection classes
fzhinkin
parents: 35913
diff changeset
   692
    const int cpci = invokedynamic_references_map.at(ref);
13929
8da0dc50a6e4 7200949: JSR 292: rubybench/bench/time/bench_base64.rb fails with jruby.jar not on boot class path
twisti
parents: 13740
diff changeset
   693
    if (cpci >= 0) {
8da0dc50a6e4 7200949: JSR 292: rubybench/bench/time/bench_base64.rb fails with jruby.jar not on boot class path
twisti
parents: 13740
diff changeset
   694
#ifdef ASSERT
8da0dc50a6e4 7200949: JSR 292: rubybench/bench/time/bench_base64.rb fails with jruby.jar not on boot class path
twisti
parents: 13740
diff changeset
   695
      // invokedynamic and invokehandle have more entries; check if they
8da0dc50a6e4 7200949: JSR 292: rubybench/bench/time/bench_base64.rb fails with jruby.jar not on boot class path
twisti
parents: 13740
diff changeset
   696
      // all point to the same constant pool cache entry.
8da0dc50a6e4 7200949: JSR 292: rubybench/bench/time/bench_base64.rb fails with jruby.jar not on boot class path
twisti
parents: 13740
diff changeset
   697
      for (int entry = 1; entry < ConstantPoolCacheEntry::_indy_resolved_references_entries; entry++) {
38031
e0b822facc03 8149374: Replace C1-specific collection classes with universal collection classes
fzhinkin
parents: 35913
diff changeset
   698
        const int cpci_next = invokedynamic_references_map.at(ref + entry);
33105
294e48b4f704 8080775: Better argument formatting for assert() and friends
david
parents: 31592
diff changeset
   699
        assert(cpci == cpci_next, "%d == %d", cpci, cpci_next);
13929
8da0dc50a6e4 7200949: JSR 292: rubybench/bench/time/bench_base64.rb fails with jruby.jar not on boot class path
twisti
parents: 13740
diff changeset
   700
      }
8da0dc50a6e4 7200949: JSR 292: rubybench/bench/time/bench_base64.rb fails with jruby.jar not on boot class path
twisti
parents: 13740
diff changeset
   701
#endif
13728
882756847a04 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 13391
diff changeset
   702
      entry_at(cpci)->initialize_resolved_reference_index(ref);
13929
8da0dc50a6e4 7200949: JSR 292: rubybench/bench/time/bench_base64.rb fails with jruby.jar not on boot class path
twisti
parents: 13740
diff changeset
   703
      ref += ConstantPoolCacheEntry::_indy_resolved_references_entries - 1;  // skip extra entries
8da0dc50a6e4 7200949: JSR 292: rubybench/bench/time/bench_base64.rb fails with jruby.jar not on boot class path
twisti
parents: 13740
diff changeset
   704
    }
4429
d7eb4e2099aa 6858164: invokedynamic code needs some cleanup (post-6655638)
jrose
parents: 3261
diff changeset
   705
  }
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   706
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
   707
47103
a993ec29ec75 8186842: Use Java class loaders for creating the CDS archive
ccheung
parents: 47095
diff changeset
   708
void ConstantPoolCache::verify_just_initialized() {
a993ec29ec75 8186842: Use Java class loaders for creating the CDS archive
ccheung
parents: 47095
diff changeset
   709
  DEBUG_ONLY(walk_entries_for_initialization(/*check_only = */ true));
a993ec29ec75 8186842: Use Java class loaders for creating the CDS archive
ccheung
parents: 47095
diff changeset
   710
}
a993ec29ec75 8186842: Use Java class loaders for creating the CDS archive
ccheung
parents: 47095
diff changeset
   711
a993ec29ec75 8186842: Use Java class loaders for creating the CDS archive
ccheung
parents: 47095
diff changeset
   712
void ConstantPoolCache::remove_unshareable_info() {
a993ec29ec75 8186842: Use Java class loaders for creating the CDS archive
ccheung
parents: 47095
diff changeset
   713
  walk_entries_for_initialization(/*check_only = */ false);
a993ec29ec75 8186842: Use Java class loaders for creating the CDS archive
ccheung
parents: 47095
diff changeset
   714
}
a993ec29ec75 8186842: Use Java class loaders for creating the CDS archive
ccheung
parents: 47095
diff changeset
   715
a993ec29ec75 8186842: Use Java class loaders for creating the CDS archive
ccheung
parents: 47095
diff changeset
   716
void ConstantPoolCache::walk_entries_for_initialization(bool check_only) {
a993ec29ec75 8186842: Use Java class loaders for creating the CDS archive
ccheung
parents: 47095
diff changeset
   717
  assert(DumpSharedSpaces, "sanity");
a993ec29ec75 8186842: Use Java class loaders for creating the CDS archive
ccheung
parents: 47095
diff changeset
   718
  // When dumping the archive, we want to clean up the ConstantPoolCache
a993ec29ec75 8186842: Use Java class loaders for creating the CDS archive
ccheung
parents: 47095
diff changeset
   719
  // to remove any effect of linking due to the execution of Java code --
a993ec29ec75 8186842: Use Java class loaders for creating the CDS archive
ccheung
parents: 47095
diff changeset
   720
  // each ConstantPoolCacheEntry will have the same contents as if
a993ec29ec75 8186842: Use Java class loaders for creating the CDS archive
ccheung
parents: 47095
diff changeset
   721
  // ConstantPoolCache::initialize has just returned:
a993ec29ec75 8186842: Use Java class loaders for creating the CDS archive
ccheung
parents: 47095
diff changeset
   722
  //
a993ec29ec75 8186842: Use Java class loaders for creating the CDS archive
ccheung
parents: 47095
diff changeset
   723
  // - We keep the ConstantPoolCache::constant_pool_index() bits for all entries.
a993ec29ec75 8186842: Use Java class loaders for creating the CDS archive
ccheung
parents: 47095
diff changeset
   724
  // - We keep the "f2" field for entries used by invokedynamic and invokehandle
a993ec29ec75 8186842: Use Java class loaders for creating the CDS archive
ccheung
parents: 47095
diff changeset
   725
  // - All other bits in the entries are cleared to zero.
a993ec29ec75 8186842: Use Java class loaders for creating the CDS archive
ccheung
parents: 47095
diff changeset
   726
  ResourceMark rm;
a993ec29ec75 8186842: Use Java class loaders for creating the CDS archive
ccheung
parents: 47095
diff changeset
   727
a993ec29ec75 8186842: Use Java class loaders for creating the CDS archive
ccheung
parents: 47095
diff changeset
   728
  InstanceKlass* ik = constant_pool()->pool_holder();
a993ec29ec75 8186842: Use Java class loaders for creating the CDS archive
ccheung
parents: 47095
diff changeset
   729
  bool* f2_used = NEW_RESOURCE_ARRAY(bool, length());
a993ec29ec75 8186842: Use Java class loaders for creating the CDS archive
ccheung
parents: 47095
diff changeset
   730
  memset(f2_used, 0, sizeof(bool) * length());
a993ec29ec75 8186842: Use Java class loaders for creating the CDS archive
ccheung
parents: 47095
diff changeset
   731
a993ec29ec75 8186842: Use Java class loaders for creating the CDS archive
ccheung
parents: 47095
diff changeset
   732
  // Find all the slots that we need to preserve f2
a993ec29ec75 8186842: Use Java class loaders for creating the CDS archive
ccheung
parents: 47095
diff changeset
   733
  for (int i = 0; i < ik->methods()->length(); i++) {
a993ec29ec75 8186842: Use Java class loaders for creating the CDS archive
ccheung
parents: 47095
diff changeset
   734
    Method* m = ik->methods()->at(i);
a993ec29ec75 8186842: Use Java class loaders for creating the CDS archive
ccheung
parents: 47095
diff changeset
   735
    RawBytecodeStream bcs(m);
a993ec29ec75 8186842: Use Java class loaders for creating the CDS archive
ccheung
parents: 47095
diff changeset
   736
    while (!bcs.is_last_bytecode()) {
a993ec29ec75 8186842: Use Java class loaders for creating the CDS archive
ccheung
parents: 47095
diff changeset
   737
      Bytecodes::Code opcode = bcs.raw_next();
a993ec29ec75 8186842: Use Java class loaders for creating the CDS archive
ccheung
parents: 47095
diff changeset
   738
      switch (opcode) {
a993ec29ec75 8186842: Use Java class loaders for creating the CDS archive
ccheung
parents: 47095
diff changeset
   739
      case Bytecodes::_invokedynamic: {
a993ec29ec75 8186842: Use Java class loaders for creating the CDS archive
ccheung
parents: 47095
diff changeset
   740
          int index = Bytes::get_native_u4(bcs.bcp() + 1);
a993ec29ec75 8186842: Use Java class loaders for creating the CDS archive
ccheung
parents: 47095
diff changeset
   741
          int cp_cache_index = constant_pool()->invokedynamic_cp_cache_index(index);
a993ec29ec75 8186842: Use Java class loaders for creating the CDS archive
ccheung
parents: 47095
diff changeset
   742
          f2_used[cp_cache_index] = 1;
a993ec29ec75 8186842: Use Java class loaders for creating the CDS archive
ccheung
parents: 47095
diff changeset
   743
        }
a993ec29ec75 8186842: Use Java class loaders for creating the CDS archive
ccheung
parents: 47095
diff changeset
   744
        break;
a993ec29ec75 8186842: Use Java class loaders for creating the CDS archive
ccheung
parents: 47095
diff changeset
   745
      case Bytecodes::_invokehandle: {
a993ec29ec75 8186842: Use Java class loaders for creating the CDS archive
ccheung
parents: 47095
diff changeset
   746
          int cp_cache_index = Bytes::get_native_u2(bcs.bcp() + 1);
a993ec29ec75 8186842: Use Java class loaders for creating the CDS archive
ccheung
parents: 47095
diff changeset
   747
          f2_used[cp_cache_index] = 1;
a993ec29ec75 8186842: Use Java class loaders for creating the CDS archive
ccheung
parents: 47095
diff changeset
   748
        }
a993ec29ec75 8186842: Use Java class loaders for creating the CDS archive
ccheung
parents: 47095
diff changeset
   749
        break;
a993ec29ec75 8186842: Use Java class loaders for creating the CDS archive
ccheung
parents: 47095
diff changeset
   750
      default:
a993ec29ec75 8186842: Use Java class loaders for creating the CDS archive
ccheung
parents: 47095
diff changeset
   751
        break;
a993ec29ec75 8186842: Use Java class loaders for creating the CDS archive
ccheung
parents: 47095
diff changeset
   752
      }
a993ec29ec75 8186842: Use Java class loaders for creating the CDS archive
ccheung
parents: 47095
diff changeset
   753
    }
a993ec29ec75 8186842: Use Java class loaders for creating the CDS archive
ccheung
parents: 47095
diff changeset
   754
  }
a993ec29ec75 8186842: Use Java class loaders for creating the CDS archive
ccheung
parents: 47095
diff changeset
   755
a993ec29ec75 8186842: Use Java class loaders for creating the CDS archive
ccheung
parents: 47095
diff changeset
   756
  if (check_only) {
a993ec29ec75 8186842: Use Java class loaders for creating the CDS archive
ccheung
parents: 47095
diff changeset
   757
    DEBUG_ONLY(
a993ec29ec75 8186842: Use Java class loaders for creating the CDS archive
ccheung
parents: 47095
diff changeset
   758
      for (int i=0; i<length(); i++) {
a993ec29ec75 8186842: Use Java class loaders for creating the CDS archive
ccheung
parents: 47095
diff changeset
   759
        entry_at(i)->verify_just_initialized(f2_used[i]);
a993ec29ec75 8186842: Use Java class loaders for creating the CDS archive
ccheung
parents: 47095
diff changeset
   760
      })
a993ec29ec75 8186842: Use Java class loaders for creating the CDS archive
ccheung
parents: 47095
diff changeset
   761
  } else {
a993ec29ec75 8186842: Use Java class loaders for creating the CDS archive
ccheung
parents: 47095
diff changeset
   762
    for (int i=0; i<length(); i++) {
a993ec29ec75 8186842: Use Java class loaders for creating the CDS archive
ccheung
parents: 47095
diff changeset
   763
      entry_at(i)->reinitialize(f2_used[i]);
a993ec29ec75 8186842: Use Java class loaders for creating the CDS archive
ccheung
parents: 47095
diff changeset
   764
    }
a993ec29ec75 8186842: Use Java class loaders for creating the CDS archive
ccheung
parents: 47095
diff changeset
   765
  }
a993ec29ec75 8186842: Use Java class loaders for creating the CDS archive
ccheung
parents: 47095
diff changeset
   766
}
a993ec29ec75 8186842: Use Java class loaders for creating the CDS archive
ccheung
parents: 47095
diff changeset
   767
47095
9d21da6fe729 8181170: resolved_references array leaks for RedefineClasses
coleenp
parents: 46810
diff changeset
   768
void ConstantPoolCache::deallocate_contents(ClassLoaderData* data) {
9d21da6fe729 8181170: resolved_references array leaks for RedefineClasses
coleenp
parents: 46810
diff changeset
   769
  assert(!is_shared(), "shared caches are not deallocated");
9d21da6fe729 8181170: resolved_references array leaks for RedefineClasses
coleenp
parents: 46810
diff changeset
   770
  data->remove_handle(_resolved_references);
9d21da6fe729 8181170: resolved_references array leaks for RedefineClasses
coleenp
parents: 46810
diff changeset
   771
  set_resolved_references(NULL);
9d21da6fe729 8181170: resolved_references array leaks for RedefineClasses
coleenp
parents: 46810
diff changeset
   772
  MetadataFactory::free_array<u2>(data, _reference_map);
9d21da6fe729 8181170: resolved_references array leaks for RedefineClasses
coleenp
parents: 46810
diff changeset
   773
  set_reference_map(NULL);
9d21da6fe729 8181170: resolved_references array leaks for RedefineClasses
coleenp
parents: 46810
diff changeset
   774
}
9d21da6fe729 8181170: resolved_references array leaks for RedefineClasses
coleenp
parents: 46810
diff changeset
   775
46810
7dad333205cd 8179302: Pre-resolve constant pool string entries and cache resolved_reference arrays in CDS archive.
jiangli
parents: 46746
diff changeset
   776
#if INCLUDE_CDS_JAVA_HEAP
7dad333205cd 8179302: Pre-resolve constant pool string entries and cache resolved_reference arrays in CDS archive.
jiangli
parents: 46746
diff changeset
   777
oop ConstantPoolCache::archived_references() {
50532
a18c60527166 8204585: Remove IN_ARCHIVE_ROOT from Access API
kbarrett
parents: 50429
diff changeset
   778
  if (CompressedOops::is_null(_archived_references)) {
a18c60527166 8204585: Remove IN_ARCHIVE_ROOT from Access API
kbarrett
parents: 50429
diff changeset
   779
    return NULL;
a18c60527166 8204585: Remove IN_ARCHIVE_ROOT from Access API
kbarrett
parents: 50429
diff changeset
   780
  }
a18c60527166 8204585: Remove IN_ARCHIVE_ROOT from Access API
kbarrett
parents: 50429
diff changeset
   781
  return MetaspaceShared::materialize_archived_object(CompressedOops::decode_not_null(_archived_references));
46810
7dad333205cd 8179302: Pre-resolve constant pool string entries and cache resolved_reference arrays in CDS archive.
jiangli
parents: 46746
diff changeset
   782
}
7dad333205cd 8179302: Pre-resolve constant pool string entries and cache resolved_reference arrays in CDS archive.
jiangli
parents: 46746
diff changeset
   783
7dad333205cd 8179302: Pre-resolve constant pool string entries and cache resolved_reference arrays in CDS archive.
jiangli
parents: 46746
diff changeset
   784
void ConstantPoolCache::set_archived_references(oop o) {
7dad333205cd 8179302: Pre-resolve constant pool string entries and cache resolved_reference arrays in CDS archive.
jiangli
parents: 46746
diff changeset
   785
  assert(DumpSharedSpaces, "called only during runtime");
50532
a18c60527166 8204585: Remove IN_ARCHIVE_ROOT from Access API
kbarrett
parents: 50429
diff changeset
   786
  _archived_references = CompressedOops::encode(o);
46810
7dad333205cd 8179302: Pre-resolve constant pool string entries and cache resolved_reference arrays in CDS archive.
jiangli
parents: 46746
diff changeset
   787
}
7dad333205cd 8179302: Pre-resolve constant pool string entries and cache resolved_reference arrays in CDS archive.
jiangli
parents: 46746
diff changeset
   788
#endif
7dad333205cd 8179302: Pre-resolve constant pool string entries and cache resolved_reference arrays in CDS archive.
jiangli
parents: 46746
diff changeset
   789
15591
b8aa0577f137 7182152: Instrumentation hot swap test incorrect monitor count
dcubed
parents: 14490
diff changeset
   790
#if INCLUDE_JVMTI
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   791
// RedefineClasses() API support:
15591
b8aa0577f137 7182152: Instrumentation hot swap test incorrect monitor count
dcubed
parents: 14490
diff changeset
   792
// If any entry of this ConstantPoolCache points to any of
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   793
// old_methods, replace it with the corresponding new_method.
29316
5287df8a8972 8046246: the constantPoolCacheOopDesc::adjust_method_entries() used in RedefineClasses does not scale
sspitsyn
parents: 29081
diff changeset
   794
void ConstantPoolCache::adjust_method_entries(InstanceKlass* holder, bool * trace_name_printed) {
5287df8a8972 8046246: the constantPoolCacheOopDesc::adjust_method_entries() used in RedefineClasses does not scale
sspitsyn
parents: 29081
diff changeset
   795
  for (int i = 0; i < length(); i++) {
5287df8a8972 8046246: the constantPoolCacheOopDesc::adjust_method_entries() used in RedefineClasses does not scale
sspitsyn
parents: 29081
diff changeset
   796
    ConstantPoolCacheEntry* entry = entry_at(i);
5287df8a8972 8046246: the constantPoolCacheOopDesc::adjust_method_entries() used in RedefineClasses does not scale
sspitsyn
parents: 29081
diff changeset
   797
    Method* old_method = entry->get_interesting_method_entry(holder);
5287df8a8972 8046246: the constantPoolCacheOopDesc::adjust_method_entries() used in RedefineClasses does not scale
sspitsyn
parents: 29081
diff changeset
   798
    if (old_method == NULL || !old_method->is_old()) {
5287df8a8972 8046246: the constantPoolCacheOopDesc::adjust_method_entries() used in RedefineClasses does not scale
sspitsyn
parents: 29081
diff changeset
   799
      continue; // skip uninteresting entries
5287df8a8972 8046246: the constantPoolCacheOopDesc::adjust_method_entries() used in RedefineClasses does not scale
sspitsyn
parents: 29081
diff changeset
   800
    }
5287df8a8972 8046246: the constantPoolCacheOopDesc::adjust_method_entries() used in RedefineClasses does not scale
sspitsyn
parents: 29081
diff changeset
   801
    if (old_method->is_deleted()) {
5287df8a8972 8046246: the constantPoolCacheOopDesc::adjust_method_entries() used in RedefineClasses does not scale
sspitsyn
parents: 29081
diff changeset
   802
      // clean up entries with deleted methods
5287df8a8972 8046246: the constantPoolCacheOopDesc::adjust_method_entries() used in RedefineClasses does not scale
sspitsyn
parents: 29081
diff changeset
   803
      entry->initialize_entry(entry->constant_pool_index());
5287df8a8972 8046246: the constantPoolCacheOopDesc::adjust_method_entries() used in RedefineClasses does not scale
sspitsyn
parents: 29081
diff changeset
   804
      continue;
5287df8a8972 8046246: the constantPoolCacheOopDesc::adjust_method_entries() used in RedefineClasses does not scale
sspitsyn
parents: 29081
diff changeset
   805
    }
5287df8a8972 8046246: the constantPoolCacheOopDesc::adjust_method_entries() used in RedefineClasses does not scale
sspitsyn
parents: 29081
diff changeset
   806
    Method* new_method = holder->method_with_idnum(old_method->orig_method_idnum());
5287df8a8972 8046246: the constantPoolCacheOopDesc::adjust_method_entries() used in RedefineClasses does not scale
sspitsyn
parents: 29081
diff changeset
   807
5287df8a8972 8046246: the constantPoolCacheOopDesc::adjust_method_entries() used in RedefineClasses does not scale
sspitsyn
parents: 29081
diff changeset
   808
    assert(new_method != NULL, "method_with_idnum() should not be NULL");
5287df8a8972 8046246: the constantPoolCacheOopDesc::adjust_method_entries() used in RedefineClasses does not scale
sspitsyn
parents: 29081
diff changeset
   809
    assert(old_method != new_method, "sanity check");
5287df8a8972 8046246: the constantPoolCacheOopDesc::adjust_method_entries() used in RedefineClasses does not scale
sspitsyn
parents: 29081
diff changeset
   810
5287df8a8972 8046246: the constantPoolCacheOopDesc::adjust_method_entries() used in RedefineClasses does not scale
sspitsyn
parents: 29081
diff changeset
   811
    entry_at(i)->adjust_method_entry(old_method, new_method, trace_name_printed);
5287df8a8972 8046246: the constantPoolCacheOopDesc::adjust_method_entries() used in RedefineClasses does not scale
sspitsyn
parents: 29081
diff changeset
   812
  }
5287df8a8972 8046246: the constantPoolCacheOopDesc::adjust_method_entries() used in RedefineClasses does not scale
sspitsyn
parents: 29081
diff changeset
   813
}
5287df8a8972 8046246: the constantPoolCacheOopDesc::adjust_method_entries() used in RedefineClasses does not scale
sspitsyn
parents: 29081
diff changeset
   814
15591
b8aa0577f137 7182152: Instrumentation hot swap test incorrect monitor count
dcubed
parents: 14490
diff changeset
   815
// the constant pool cache should never contain old or obsolete methods
b8aa0577f137 7182152: Instrumentation hot swap test incorrect monitor count
dcubed
parents: 14490
diff changeset
   816
bool ConstantPoolCache::check_no_old_or_obsolete_entries() {
13728
882756847a04 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 13391
diff changeset
   817
  for (int i = 1; i < length(); i++) {
29316
5287df8a8972 8046246: the constantPoolCacheOopDesc::adjust_method_entries() used in RedefineClasses does not scale
sspitsyn
parents: 29081
diff changeset
   818
    if (entry_at(i)->get_interesting_method_entry(NULL) != NULL &&
15591
b8aa0577f137 7182152: Instrumentation hot swap test incorrect monitor count
dcubed
parents: 14490
diff changeset
   819
        !entry_at(i)->check_no_old_or_obsolete_entries()) {
13728
882756847a04 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 13391
diff changeset
   820
      return false;
882756847a04 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 13391
diff changeset
   821
    }
882756847a04 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 13391
diff changeset
   822
  }
882756847a04 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 13391
diff changeset
   823
  return true;
882756847a04 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 13391
diff changeset
   824
}
15591
b8aa0577f137 7182152: Instrumentation hot swap test incorrect monitor count
dcubed
parents: 14490
diff changeset
   825
b8aa0577f137 7182152: Instrumentation hot swap test incorrect monitor count
dcubed
parents: 14490
diff changeset
   826
void ConstantPoolCache::dump_cache() {
b8aa0577f137 7182152: Instrumentation hot swap test incorrect monitor count
dcubed
parents: 14490
diff changeset
   827
  for (int i = 1; i < length(); i++) {
29316
5287df8a8972 8046246: the constantPoolCacheOopDesc::adjust_method_entries() used in RedefineClasses does not scale
sspitsyn
parents: 29081
diff changeset
   828
    if (entry_at(i)->get_interesting_method_entry(NULL) != NULL) {
15591
b8aa0577f137 7182152: Instrumentation hot swap test incorrect monitor count
dcubed
parents: 14490
diff changeset
   829
      entry_at(i)->print(tty, i);
b8aa0577f137 7182152: Instrumentation hot swap test incorrect monitor count
dcubed
parents: 14490
diff changeset
   830
    }
b8aa0577f137 7182152: Instrumentation hot swap test incorrect monitor count
dcubed
parents: 14490
diff changeset
   831
  }
b8aa0577f137 7182152: Instrumentation hot swap test incorrect monitor count
dcubed
parents: 14490
diff changeset
   832
}
b8aa0577f137 7182152: Instrumentation hot swap test incorrect monitor count
dcubed
parents: 14490
diff changeset
   833
#endif // INCLUDE_JVMTI
13728
882756847a04 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 13391
diff changeset
   834
46746
ea379ebb9447 8072061: Automatically determine optimal sizes for the CDS regions
iklam
parents: 46727
diff changeset
   835
void ConstantPoolCache::metaspace_pointers_do(MetaspaceClosure* it) {
ea379ebb9447 8072061: Automatically determine optimal sizes for the CDS regions
iklam
parents: 46727
diff changeset
   836
  log_trace(cds)("Iter(ConstantPoolCache): %p", this);
ea379ebb9447 8072061: Automatically determine optimal sizes for the CDS regions
iklam
parents: 46727
diff changeset
   837
  it->push(&_constant_pool);
ea379ebb9447 8072061: Automatically determine optimal sizes for the CDS regions
iklam
parents: 46727
diff changeset
   838
  it->push(&_reference_map);
ea379ebb9447 8072061: Automatically determine optimal sizes for the CDS regions
iklam
parents: 46727
diff changeset
   839
}
13728
882756847a04 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 13391
diff changeset
   840
882756847a04 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 13391
diff changeset
   841
// Printing
882756847a04 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 13391
diff changeset
   842
882756847a04 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 13391
diff changeset
   843
void ConstantPoolCache::print_on(outputStream* st) const {
882756847a04 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 13391
diff changeset
   844
  assert(is_constantPoolCache(), "obj must be constant pool cache");
24424
2658d7834c6e 8037816: Fix for 8036122 breaks build with Xcode5/clang
drchase
parents: 24351
diff changeset
   845
  st->print_cr("%s", internal_name());
13728
882756847a04 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 13391
diff changeset
   846
  // print constant pool cache entries
882756847a04 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 13391
diff changeset
   847
  for (int i = 0; i < length(); i++) entry_at(i)->print(st, i);
882756847a04 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 13391
diff changeset
   848
}
882756847a04 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 13391
diff changeset
   849
882756847a04 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 13391
diff changeset
   850
void ConstantPoolCache::print_value_on(outputStream* st) const {
882756847a04 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 13391
diff changeset
   851
  assert(is_constantPoolCache(), "obj must be constant pool cache");
882756847a04 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 13391
diff changeset
   852
  st->print("cache [%d]", length());
882756847a04 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 13391
diff changeset
   853
  print_address_on(st);
882756847a04 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 13391
diff changeset
   854
  st->print(" for ");
882756847a04 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 13391
diff changeset
   855
  constant_pool()->print_value_on(st);
882756847a04 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 13391
diff changeset
   856
}
882756847a04 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 13391
diff changeset
   857
882756847a04 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 13391
diff changeset
   858
882756847a04 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 13391
diff changeset
   859
// Verification
882756847a04 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 13391
diff changeset
   860
882756847a04 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 13391
diff changeset
   861
void ConstantPoolCache::verify_on(outputStream* st) {
882756847a04 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 13391
diff changeset
   862
  guarantee(is_constantPoolCache(), "obj must be constant pool cache");
882756847a04 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 13391
diff changeset
   863
  // print constant pool cache entries
882756847a04 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 13391
diff changeset
   864
  for (int i = 0; i < length(); i++) entry_at(i)->verify(st);
882756847a04 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 13391
diff changeset
   865
}