hotspot/src/share/vm/interpreter/linkResolver.cpp
author jwilhelm
Wed, 03 Feb 2016 01:35:25 +0100
changeset 35913 928548a43408
parent 35606 d873b64009cc
parent 35901 f5028c67e7cb
child 36508 5f9eee6b383b
child 36819 bca375d368ed
permissions -rw-r--r--
Merge
Ignore whitespace changes - Everywhere: Within whitespace: At end of lines:
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
     1
/*
35900
d64cf9290fc4 8148481: Devirtualize Klass::vtable
mgerdin
parents: 35495
diff changeset
     2
 * Copyright (c) 1997, 2016, 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: 5421
diff changeset
    19
 * Please contact Oracle, 500 Oracle Parkway, Redwood Shores, CA 94065 USA
f4b087cbb361 6941466: Oracle rebranding changes for Hotspot repositories
trims
parents: 5421
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: 5421
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: 6453
diff changeset
    25
#include "precompiled.hpp"
14385
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents: 13929
diff changeset
    26
#include "classfile/defaultMethods.hpp"
34666
1c7168ea0034 8140485: Class load and creation cleanup
mgronlun
parents: 34238
diff changeset
    27
#include "classfile/symbolTable.hpp"
7397
5b173b4ca846 6989984: Use standard include model for Hospot
stefank
parents: 6453
diff changeset
    28
#include "classfile/systemDictionary.hpp"
5b173b4ca846 6989984: Use standard include model for Hospot
stefank
parents: 6453
diff changeset
    29
#include "classfile/vmSymbols.hpp"
5b173b4ca846 6989984: Use standard include model for Hospot
stefank
parents: 6453
diff changeset
    30
#include "compiler/compileBroker.hpp"
30764
fec48bf5a827 8079792: GC directory structure cleanup
pliden
parents: 30227
diff changeset
    31
#include "gc/shared/collectedHeap.inline.hpp"
7397
5b173b4ca846 6989984: Use standard include model for Hospot
stefank
parents: 6453
diff changeset
    32
#include "interpreter/bytecode.hpp"
5b173b4ca846 6989984: Use standard include model for Hospot
stefank
parents: 6453
diff changeset
    33
#include "interpreter/interpreterRuntime.hpp"
5b173b4ca846 6989984: Use standard include model for Hospot
stefank
parents: 6453
diff changeset
    34
#include "interpreter/linkResolver.hpp"
35463
b32e362563bb 8141564: Convert TraceItables and PrintVtables to Unified Logging
rprotacio
parents: 34666
diff changeset
    35
#include "logging/log.hpp"
7397
5b173b4ca846 6989984: Use standard include model for Hospot
stefank
parents: 6453
diff changeset
    36
#include "memory/resourceArea.hpp"
5b173b4ca846 6989984: Use standard include model for Hospot
stefank
parents: 6453
diff changeset
    37
#include "memory/universe.inline.hpp"
5b173b4ca846 6989984: Use standard include model for Hospot
stefank
parents: 6453
diff changeset
    38
#include "oops/instanceKlass.hpp"
35463
b32e362563bb 8141564: Convert TraceItables and PrintVtables to Unified Logging
rprotacio
parents: 34666
diff changeset
    39
#include "oops/method.hpp"
7397
5b173b4ca846 6989984: Use standard include model for Hospot
stefank
parents: 6453
diff changeset
    40
#include "oops/objArrayOop.hpp"
29081
c61eb4914428 8072911: Remove includes of oop.inline.hpp from .hpp files
stefank
parents: 28738
diff changeset
    41
#include "oops/oop.inline.hpp"
7397
5b173b4ca846 6989984: Use standard include model for Hospot
stefank
parents: 6453
diff changeset
    42
#include "prims/methodHandles.hpp"
5b173b4ca846 6989984: Use standard include model for Hospot
stefank
parents: 6453
diff changeset
    43
#include "prims/nativeLookup.hpp"
5b173b4ca846 6989984: Use standard include model for Hospot
stefank
parents: 6453
diff changeset
    44
#include "runtime/compilationPolicy.hpp"
5b173b4ca846 6989984: Use standard include model for Hospot
stefank
parents: 6453
diff changeset
    45
#include "runtime/fieldDescriptor.hpp"
5b173b4ca846 6989984: Use standard include model for Hospot
stefank
parents: 6453
diff changeset
    46
#include "runtime/frame.inline.hpp"
5b173b4ca846 6989984: Use standard include model for Hospot
stefank
parents: 6453
diff changeset
    47
#include "runtime/handles.inline.hpp"
5b173b4ca846 6989984: Use standard include model for Hospot
stefank
parents: 6453
diff changeset
    48
#include "runtime/reflection.hpp"
5b173b4ca846 6989984: Use standard include model for Hospot
stefank
parents: 6453
diff changeset
    49
#include "runtime/signature.hpp"
14583
d70ee55535f4 8003935: Simplify the needed includes for using Thread::current()
stefank
parents: 14488
diff changeset
    50
#include "runtime/thread.inline.hpp"
7397
5b173b4ca846 6989984: Use standard include model for Hospot
stefank
parents: 6453
diff changeset
    51
#include "runtime/vmThread.hpp"
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
    52
489c9b5090e2 Initial load
duke
parents:
diff changeset
    53
489c9b5090e2 Initial load
duke
parents:
diff changeset
    54
//------------------------------------------------------------------------------------------------------------------------
489c9b5090e2 Initial load
duke
parents:
diff changeset
    55
// Implementation of CallInfo
489c9b5090e2 Initial load
duke
parents:
diff changeset
    56
489c9b5090e2 Initial load
duke
parents:
diff changeset
    57
31019
d05fcdd70109 8029567: Clean up linkResolver code
coleenp
parents: 30773
diff changeset
    58
void CallInfo::set_static(KlassHandle resolved_klass, const methodHandle& resolved_method, TRAPS) {
13728
882756847a04 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 13396
diff changeset
    59
  int vtable_index = Method::nonvirtual_vtable_index;
20017
81eba62e9048 8014013: CallInfo structure no longer accurately reports the result of a LinkResolver operation
drchase
parents: 17024
diff changeset
    60
  set_common(resolved_klass, resolved_klass, resolved_method, resolved_method, CallInfo::direct_call, vtable_index, CHECK);
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
    61
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
    62
489c9b5090e2 Initial load
duke
parents:
diff changeset
    63
31019
d05fcdd70109 8029567: Clean up linkResolver code
coleenp
parents: 30773
diff changeset
    64
void CallInfo::set_interface(KlassHandle resolved_klass,
d05fcdd70109 8029567: Clean up linkResolver code
coleenp
parents: 30773
diff changeset
    65
                             KlassHandle selected_klass,
d05fcdd70109 8029567: Clean up linkResolver code
coleenp
parents: 30773
diff changeset
    66
                             const methodHandle& resolved_method,
d05fcdd70109 8029567: Clean up linkResolver code
coleenp
parents: 30773
diff changeset
    67
                             const methodHandle& selected_method,
d05fcdd70109 8029567: Clean up linkResolver code
coleenp
parents: 30773
diff changeset
    68
                             int itable_index, TRAPS) {
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
    69
  // This is only called for interface methods. If the resolved_method
489c9b5090e2 Initial load
duke
parents:
diff changeset
    70
  // comes from java/lang/Object, it can be the subject of a virtual call, so
489c9b5090e2 Initial load
duke
parents:
diff changeset
    71
  // we should pick the vtable index from the resolved method.
20017
81eba62e9048 8014013: CallInfo structure no longer accurately reports the result of a LinkResolver operation
drchase
parents: 17024
diff changeset
    72
  // In that case, the caller must call set_virtual instead of set_interface.
81eba62e9048 8014013: CallInfo structure no longer accurately reports the result of a LinkResolver operation
drchase
parents: 17024
diff changeset
    73
  assert(resolved_method->method_holder()->is_interface(), "");
81eba62e9048 8014013: CallInfo structure no longer accurately reports the result of a LinkResolver operation
drchase
parents: 17024
diff changeset
    74
  assert(itable_index == resolved_method()->itable_index(), "");
81eba62e9048 8014013: CallInfo structure no longer accurately reports the result of a LinkResolver operation
drchase
parents: 17024
diff changeset
    75
  set_common(resolved_klass, selected_klass, resolved_method, selected_method, CallInfo::itable_call, itable_index, CHECK);
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
    76
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
    77
31019
d05fcdd70109 8029567: Clean up linkResolver code
coleenp
parents: 30773
diff changeset
    78
void CallInfo::set_virtual(KlassHandle resolved_klass,
d05fcdd70109 8029567: Clean up linkResolver code
coleenp
parents: 30773
diff changeset
    79
                           KlassHandle selected_klass,
d05fcdd70109 8029567: Clean up linkResolver code
coleenp
parents: 30773
diff changeset
    80
                           const methodHandle& resolved_method,
d05fcdd70109 8029567: Clean up linkResolver code
coleenp
parents: 30773
diff changeset
    81
                           const methodHandle& selected_method,
d05fcdd70109 8029567: Clean up linkResolver code
coleenp
parents: 30773
diff changeset
    82
                           int vtable_index, TRAPS) {
13728
882756847a04 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 13396
diff changeset
    83
  assert(vtable_index >= 0 || vtable_index == Method::nonvirtual_vtable_index, "valid index");
20017
81eba62e9048 8014013: CallInfo structure no longer accurately reports the result of a LinkResolver operation
drchase
parents: 17024
diff changeset
    84
  assert(vtable_index < 0 || !resolved_method->has_vtable_index() || vtable_index == resolved_method->vtable_index(), "");
81eba62e9048 8014013: CallInfo structure no longer accurately reports the result of a LinkResolver operation
drchase
parents: 17024
diff changeset
    85
  CallKind kind = (vtable_index >= 0 && !resolved_method->can_be_statically_bound() ? CallInfo::vtable_call : CallInfo::direct_call);
81eba62e9048 8014013: CallInfo structure no longer accurately reports the result of a LinkResolver operation
drchase
parents: 17024
diff changeset
    86
  set_common(resolved_klass, selected_klass, resolved_method, selected_method, kind, vtable_index, CHECK);
13391
30245956af37 7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents: 10565
diff changeset
    87
  assert(!resolved_method->is_compiled_lambda_form(), "these must be handled via an invokehandle call");
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
    88
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
    89
31019
d05fcdd70109 8029567: Clean up linkResolver code
coleenp
parents: 30773
diff changeset
    90
void CallInfo::set_handle(const methodHandle& resolved_method,
d05fcdd70109 8029567: Clean up linkResolver code
coleenp
parents: 30773
diff changeset
    91
                          Handle resolved_appendix,
d05fcdd70109 8029567: Clean up linkResolver code
coleenp
parents: 30773
diff changeset
    92
                          Handle resolved_method_type, TRAPS) {
13391
30245956af37 7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents: 10565
diff changeset
    93
  if (resolved_method.is_null()) {
30245956af37 7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents: 10565
diff changeset
    94
    THROW_MSG(vmSymbols::java_lang_InternalError(), "resolved method is null");
30245956af37 7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents: 10565
diff changeset
    95
  }
13728
882756847a04 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 13396
diff changeset
    96
  KlassHandle resolved_klass = SystemDictionary::MethodHandle_klass();
13391
30245956af37 7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents: 10565
diff changeset
    97
  assert(resolved_method->intrinsic_id() == vmIntrinsics::_invokeBasic ||
30245956af37 7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents: 10565
diff changeset
    98
         resolved_method->is_compiled_lambda_form(),
30245956af37 7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents: 10565
diff changeset
    99
         "linkMethod must return one of these");
13728
882756847a04 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 13396
diff changeset
   100
  int vtable_index = Method::nonvirtual_vtable_index;
20017
81eba62e9048 8014013: CallInfo structure no longer accurately reports the result of a LinkResolver operation
drchase
parents: 17024
diff changeset
   101
  assert(!resolved_method->has_vtable_index(), "");
81eba62e9048 8014013: CallInfo structure no longer accurately reports the result of a LinkResolver operation
drchase
parents: 17024
diff changeset
   102
  set_common(resolved_klass, resolved_klass, resolved_method, resolved_method, CallInfo::direct_call, vtable_index, CHECK);
13929
8da0dc50a6e4 7200949: JSR 292: rubybench/bench/time/bench_base64.rb fails with jruby.jar not on boot class path
twisti
parents: 13732
diff changeset
   103
  _resolved_appendix    = resolved_appendix;
8da0dc50a6e4 7200949: JSR 292: rubybench/bench/time/bench_base64.rb fails with jruby.jar not on boot class path
twisti
parents: 13732
diff changeset
   104
  _resolved_method_type = resolved_method_type;
6062
bab93afe9df7 6964498: JSR 292 invokedynamic sites need local bootstrap methods
jrose
parents: 5547
diff changeset
   105
}
bab93afe9df7 6964498: JSR 292 invokedynamic sites need local bootstrap methods
jrose
parents: 5547
diff changeset
   106
20017
81eba62e9048 8014013: CallInfo structure no longer accurately reports the result of a LinkResolver operation
drchase
parents: 17024
diff changeset
   107
void CallInfo::set_common(KlassHandle resolved_klass,
81eba62e9048 8014013: CallInfo structure no longer accurately reports the result of a LinkResolver operation
drchase
parents: 17024
diff changeset
   108
                          KlassHandle selected_klass,
31019
d05fcdd70109 8029567: Clean up linkResolver code
coleenp
parents: 30773
diff changeset
   109
                          const methodHandle& resolved_method,
d05fcdd70109 8029567: Clean up linkResolver code
coleenp
parents: 30773
diff changeset
   110
                          const methodHandle& selected_method,
20017
81eba62e9048 8014013: CallInfo structure no longer accurately reports the result of a LinkResolver operation
drchase
parents: 17024
diff changeset
   111
                          CallKind kind,
81eba62e9048 8014013: CallInfo structure no longer accurately reports the result of a LinkResolver operation
drchase
parents: 17024
diff changeset
   112
                          int index,
81eba62e9048 8014013: CallInfo structure no longer accurately reports the result of a LinkResolver operation
drchase
parents: 17024
diff changeset
   113
                          TRAPS) {
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   114
  assert(resolved_method->signature() == selected_method->signature(), "signatures must correspond");
489c9b5090e2 Initial load
duke
parents:
diff changeset
   115
  _resolved_klass  = resolved_klass;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   116
  _selected_klass  = selected_klass;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   117
  _resolved_method = resolved_method;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   118
  _selected_method = selected_method;
20017
81eba62e9048 8014013: CallInfo structure no longer accurately reports the result of a LinkResolver operation
drchase
parents: 17024
diff changeset
   119
  _call_kind       = kind;
81eba62e9048 8014013: CallInfo structure no longer accurately reports the result of a LinkResolver operation
drchase
parents: 17024
diff changeset
   120
  _call_index      = index;
13391
30245956af37 7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents: 10565
diff changeset
   121
  _resolved_appendix = Handle();
20017
81eba62e9048 8014013: CallInfo structure no longer accurately reports the result of a LinkResolver operation
drchase
parents: 17024
diff changeset
   122
  DEBUG_ONLY(verify());  // verify before making side effects
81eba62e9048 8014013: CallInfo structure no longer accurately reports the result of a LinkResolver operation
drchase
parents: 17024
diff changeset
   123
6453
970dc585ab63 6953144: Tiered compilation
iveresov
parents: 6062
diff changeset
   124
  if (CompilationPolicy::must_be_compiled(selected_method)) {
4564
55dfb20908d0 6893081: method handle & invokedynamic code needs additional cleanup (post 6815692, 6858164)
twisti
parents: 4429
diff changeset
   125
    // This path is unusual, mostly used by the '-Xcomp' stress test mode.
55dfb20908d0 6893081: method handle & invokedynamic code needs additional cleanup (post 6815692, 6858164)
twisti
parents: 4429
diff changeset
   126
6453
970dc585ab63 6953144: Tiered compilation
iveresov
parents: 6062
diff changeset
   127
    // Note: with several active threads, the must_be_compiled may be true
970dc585ab63 6953144: Tiered compilation
iveresov
parents: 6062
diff changeset
   128
    //       while can_be_compiled is false; remove assert
970dc585ab63 6953144: Tiered compilation
iveresov
parents: 6062
diff changeset
   129
    // assert(CompilationPolicy::can_be_compiled(selected_method), "cannot compile");
33160
c59f1676d27e 8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents: 32189
diff changeset
   130
    if (!THREAD->can_call_java()) {
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   131
      // don't force compilation, resolve was on behalf of compiler
489c9b5090e2 Initial load
duke
parents:
diff changeset
   132
      return;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   133
    }
14391
df0a1573d5bd 8000725: NPG: method_holder() and pool_holder() and pool_holder field should be InstanceKlass
coleenp
parents: 14385
diff changeset
   134
    if (selected_method->method_holder()->is_not_initialized()) {
4564
55dfb20908d0 6893081: method handle & invokedynamic code needs additional cleanup (post 6815692, 6858164)
twisti
parents: 4429
diff changeset
   135
      // 'is_not_initialized' means not only '!is_initialized', but also that
55dfb20908d0 6893081: method handle & invokedynamic code needs additional cleanup (post 6815692, 6858164)
twisti
parents: 4429
diff changeset
   136
      // initialization has not been started yet ('!being_initialized')
55dfb20908d0 6893081: method handle & invokedynamic code needs additional cleanup (post 6815692, 6858164)
twisti
parents: 4429
diff changeset
   137
      // Do not force compilation of methods in uninitialized classes.
55dfb20908d0 6893081: method handle & invokedynamic code needs additional cleanup (post 6815692, 6858164)
twisti
parents: 4429
diff changeset
   138
      // Note that doing this would throw an assert later,
55dfb20908d0 6893081: method handle & invokedynamic code needs additional cleanup (post 6815692, 6858164)
twisti
parents: 4429
diff changeset
   139
      // in CompileBroker::compile_method.
55dfb20908d0 6893081: method handle & invokedynamic code needs additional cleanup (post 6815692, 6858164)
twisti
parents: 4429
diff changeset
   140
      // We sometimes use the link resolver to do reflective lookups
55dfb20908d0 6893081: method handle & invokedynamic code needs additional cleanup (post 6815692, 6858164)
twisti
parents: 4429
diff changeset
   141
      // even before classes are initialized.
55dfb20908d0 6893081: method handle & invokedynamic code needs additional cleanup (post 6815692, 6858164)
twisti
parents: 4429
diff changeset
   142
      return;
55dfb20908d0 6893081: method handle & invokedynamic code needs additional cleanup (post 6815692, 6858164)
twisti
parents: 4429
diff changeset
   143
    }
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   144
    CompileBroker::compile_method(selected_method, InvocationEntryBci,
10250
0794cd144834 7066339: Tiered: policy should make consistent decisions about osr levels
iveresov
parents: 10008
diff changeset
   145
                                  CompilationPolicy::policy()->initial_compile_level(),
6453
970dc585ab63 6953144: Tiered compilation
iveresov
parents: 6062
diff changeset
   146
                                  methodHandle(), 0, "must_be_compiled", CHECK);
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   147
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   148
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
   149
20017
81eba62e9048 8014013: CallInfo structure no longer accurately reports the result of a LinkResolver operation
drchase
parents: 17024
diff changeset
   150
// utility query for unreflecting a method
81eba62e9048 8014013: CallInfo structure no longer accurately reports the result of a LinkResolver operation
drchase
parents: 17024
diff changeset
   151
CallInfo::CallInfo(Method* resolved_method, Klass* resolved_klass) {
81eba62e9048 8014013: CallInfo structure no longer accurately reports the result of a LinkResolver operation
drchase
parents: 17024
diff changeset
   152
  Klass* resolved_method_holder = resolved_method->method_holder();
81eba62e9048 8014013: CallInfo structure no longer accurately reports the result of a LinkResolver operation
drchase
parents: 17024
diff changeset
   153
  if (resolved_klass == NULL) { // 2nd argument defaults to holder of 1st
81eba62e9048 8014013: CallInfo structure no longer accurately reports the result of a LinkResolver operation
drchase
parents: 17024
diff changeset
   154
    resolved_klass = resolved_method_holder;
81eba62e9048 8014013: CallInfo structure no longer accurately reports the result of a LinkResolver operation
drchase
parents: 17024
diff changeset
   155
  }
81eba62e9048 8014013: CallInfo structure no longer accurately reports the result of a LinkResolver operation
drchase
parents: 17024
diff changeset
   156
  _resolved_klass  = resolved_klass;
81eba62e9048 8014013: CallInfo structure no longer accurately reports the result of a LinkResolver operation
drchase
parents: 17024
diff changeset
   157
  _selected_klass  = resolved_klass;
81eba62e9048 8014013: CallInfo structure no longer accurately reports the result of a LinkResolver operation
drchase
parents: 17024
diff changeset
   158
  _resolved_method = resolved_method;
81eba62e9048 8014013: CallInfo structure no longer accurately reports the result of a LinkResolver operation
drchase
parents: 17024
diff changeset
   159
  _selected_method = resolved_method;
81eba62e9048 8014013: CallInfo structure no longer accurately reports the result of a LinkResolver operation
drchase
parents: 17024
diff changeset
   160
  // classify:
81eba62e9048 8014013: CallInfo structure no longer accurately reports the result of a LinkResolver operation
drchase
parents: 17024
diff changeset
   161
  CallKind kind = CallInfo::unknown_kind;
81eba62e9048 8014013: CallInfo structure no longer accurately reports the result of a LinkResolver operation
drchase
parents: 17024
diff changeset
   162
  int index = resolved_method->vtable_index();
81eba62e9048 8014013: CallInfo structure no longer accurately reports the result of a LinkResolver operation
drchase
parents: 17024
diff changeset
   163
  if (resolved_method->can_be_statically_bound()) {
81eba62e9048 8014013: CallInfo structure no longer accurately reports the result of a LinkResolver operation
drchase
parents: 17024
diff changeset
   164
    kind = CallInfo::direct_call;
81eba62e9048 8014013: CallInfo structure no longer accurately reports the result of a LinkResolver operation
drchase
parents: 17024
diff changeset
   165
  } else if (!resolved_method_holder->is_interface()) {
81eba62e9048 8014013: CallInfo structure no longer accurately reports the result of a LinkResolver operation
drchase
parents: 17024
diff changeset
   166
    // Could be an Object method inherited into an interface, but still a vtable call.
81eba62e9048 8014013: CallInfo structure no longer accurately reports the result of a LinkResolver operation
drchase
parents: 17024
diff changeset
   167
    kind = CallInfo::vtable_call;
81eba62e9048 8014013: CallInfo structure no longer accurately reports the result of a LinkResolver operation
drchase
parents: 17024
diff changeset
   168
  } else if (!resolved_klass->is_interface()) {
21556
e75cd34a59e0 8027229: ICCE expected for >=2 maximally specific default methods.
acorn
parents: 21078
diff changeset
   169
    // A default or miranda method.  Compute the vtable index.
20017
81eba62e9048 8014013: CallInfo structure no longer accurately reports the result of a LinkResolver operation
drchase
parents: 17024
diff changeset
   170
    ResourceMark rm;
35900
d64cf9290fc4 8148481: Devirtualize Klass::vtable
mgerdin
parents: 35495
diff changeset
   171
    klassVtable* vt = resolved_klass->vtable();
21556
e75cd34a59e0 8027229: ICCE expected for >=2 maximally specific default methods.
acorn
parents: 21078
diff changeset
   172
    index = LinkResolver::vtable_index_of_interface_method(resolved_klass,
e75cd34a59e0 8027229: ICCE expected for >=2 maximally specific default methods.
acorn
parents: 21078
diff changeset
   173
                           resolved_method);
e75cd34a59e0 8027229: ICCE expected for >=2 maximally specific default methods.
acorn
parents: 21078
diff changeset
   174
    assert(index >= 0 , "we should have valid vtable index at this point");
e75cd34a59e0 8027229: ICCE expected for >=2 maximally specific default methods.
acorn
parents: 21078
diff changeset
   175
20017
81eba62e9048 8014013: CallInfo structure no longer accurately reports the result of a LinkResolver operation
drchase
parents: 17024
diff changeset
   176
    kind = CallInfo::vtable_call;
20712
a06e1f01fb6e 8026124: JSR-292 bug: java.nio.file.Path.toString cores dump
drchase
parents: 20391
diff changeset
   177
  } else if (resolved_method->has_vtable_index()) {
a06e1f01fb6e 8026124: JSR-292 bug: java.nio.file.Path.toString cores dump
drchase
parents: 20391
diff changeset
   178
    // Can occur if an interface redeclares a method of Object.
a06e1f01fb6e 8026124: JSR-292 bug: java.nio.file.Path.toString cores dump
drchase
parents: 20391
diff changeset
   179
a06e1f01fb6e 8026124: JSR-292 bug: java.nio.file.Path.toString cores dump
drchase
parents: 20391
diff changeset
   180
#ifdef ASSERT
a06e1f01fb6e 8026124: JSR-292 bug: java.nio.file.Path.toString cores dump
drchase
parents: 20391
diff changeset
   181
    // Ensure that this is really the case.
a06e1f01fb6e 8026124: JSR-292 bug: java.nio.file.Path.toString cores dump
drchase
parents: 20391
diff changeset
   182
    KlassHandle object_klass = SystemDictionary::Object_klass();
a06e1f01fb6e 8026124: JSR-292 bug: java.nio.file.Path.toString cores dump
drchase
parents: 20391
diff changeset
   183
    Method * object_resolved_method = object_klass()->vtable()->method_at(index);
a06e1f01fb6e 8026124: JSR-292 bug: java.nio.file.Path.toString cores dump
drchase
parents: 20391
diff changeset
   184
    assert(object_resolved_method->name() == resolved_method->name(),
33105
294e48b4f704 8080775: Better argument formatting for assert() and friends
david
parents: 32189
diff changeset
   185
      "Object and interface method names should match at vtable index %d, %s != %s",
294e48b4f704 8080775: Better argument formatting for assert() and friends
david
parents: 32189
diff changeset
   186
      index, object_resolved_method->name()->as_C_string(), resolved_method->name()->as_C_string());
20712
a06e1f01fb6e 8026124: JSR-292 bug: java.nio.file.Path.toString cores dump
drchase
parents: 20391
diff changeset
   187
    assert(object_resolved_method->signature() == resolved_method->signature(),
33105
294e48b4f704 8080775: Better argument formatting for assert() and friends
david
parents: 32189
diff changeset
   188
      "Object and interface method signatures should match at vtable index %d, %s != %s",
294e48b4f704 8080775: Better argument formatting for assert() and friends
david
parents: 32189
diff changeset
   189
      index, object_resolved_method->signature()->as_C_string(), resolved_method->signature()->as_C_string());
20712
a06e1f01fb6e 8026124: JSR-292 bug: java.nio.file.Path.toString cores dump
drchase
parents: 20391
diff changeset
   190
#endif // ASSERT
a06e1f01fb6e 8026124: JSR-292 bug: java.nio.file.Path.toString cores dump
drchase
parents: 20391
diff changeset
   191
a06e1f01fb6e 8026124: JSR-292 bug: java.nio.file.Path.toString cores dump
drchase
parents: 20391
diff changeset
   192
    kind = CallInfo::vtable_call;
20017
81eba62e9048 8014013: CallInfo structure no longer accurately reports the result of a LinkResolver operation
drchase
parents: 17024
diff changeset
   193
  } else {
81eba62e9048 8014013: CallInfo structure no longer accurately reports the result of a LinkResolver operation
drchase
parents: 17024
diff changeset
   194
    // A regular interface call.
81eba62e9048 8014013: CallInfo structure no longer accurately reports the result of a LinkResolver operation
drchase
parents: 17024
diff changeset
   195
    kind = CallInfo::itable_call;
81eba62e9048 8014013: CallInfo structure no longer accurately reports the result of a LinkResolver operation
drchase
parents: 17024
diff changeset
   196
    index = resolved_method->itable_index();
81eba62e9048 8014013: CallInfo structure no longer accurately reports the result of a LinkResolver operation
drchase
parents: 17024
diff changeset
   197
  }
33105
294e48b4f704 8080775: Better argument formatting for assert() and friends
david
parents: 32189
diff changeset
   198
  assert(index == Method::nonvirtual_vtable_index || index >= 0, "bad index %d", index);
20017
81eba62e9048 8014013: CallInfo structure no longer accurately reports the result of a LinkResolver operation
drchase
parents: 17024
diff changeset
   199
  _call_kind  = kind;
81eba62e9048 8014013: CallInfo structure no longer accurately reports the result of a LinkResolver operation
drchase
parents: 17024
diff changeset
   200
  _call_index = index;
81eba62e9048 8014013: CallInfo structure no longer accurately reports the result of a LinkResolver operation
drchase
parents: 17024
diff changeset
   201
  _resolved_appendix = Handle();
81eba62e9048 8014013: CallInfo structure no longer accurately reports the result of a LinkResolver operation
drchase
parents: 17024
diff changeset
   202
  DEBUG_ONLY(verify());
81eba62e9048 8014013: CallInfo structure no longer accurately reports the result of a LinkResolver operation
drchase
parents: 17024
diff changeset
   203
}
81eba62e9048 8014013: CallInfo structure no longer accurately reports the result of a LinkResolver operation
drchase
parents: 17024
diff changeset
   204
81eba62e9048 8014013: CallInfo structure no longer accurately reports the result of a LinkResolver operation
drchase
parents: 17024
diff changeset
   205
#ifdef ASSERT
81eba62e9048 8014013: CallInfo structure no longer accurately reports the result of a LinkResolver operation
drchase
parents: 17024
diff changeset
   206
void CallInfo::verify() {
81eba62e9048 8014013: CallInfo structure no longer accurately reports the result of a LinkResolver operation
drchase
parents: 17024
diff changeset
   207
  switch (call_kind()) {  // the meaning and allowed value of index depends on kind
81eba62e9048 8014013: CallInfo structure no longer accurately reports the result of a LinkResolver operation
drchase
parents: 17024
diff changeset
   208
  case CallInfo::direct_call:
81eba62e9048 8014013: CallInfo structure no longer accurately reports the result of a LinkResolver operation
drchase
parents: 17024
diff changeset
   209
    if (_call_index == Method::nonvirtual_vtable_index)  break;
81eba62e9048 8014013: CallInfo structure no longer accurately reports the result of a LinkResolver operation
drchase
parents: 17024
diff changeset
   210
    // else fall through to check vtable index:
81eba62e9048 8014013: CallInfo structure no longer accurately reports the result of a LinkResolver operation
drchase
parents: 17024
diff changeset
   211
  case CallInfo::vtable_call:
81eba62e9048 8014013: CallInfo structure no longer accurately reports the result of a LinkResolver operation
drchase
parents: 17024
diff changeset
   212
    assert(resolved_klass()->verify_vtable_index(_call_index), "");
81eba62e9048 8014013: CallInfo structure no longer accurately reports the result of a LinkResolver operation
drchase
parents: 17024
diff changeset
   213
    break;
81eba62e9048 8014013: CallInfo structure no longer accurately reports the result of a LinkResolver operation
drchase
parents: 17024
diff changeset
   214
  case CallInfo::itable_call:
81eba62e9048 8014013: CallInfo structure no longer accurately reports the result of a LinkResolver operation
drchase
parents: 17024
diff changeset
   215
    assert(resolved_method()->method_holder()->verify_itable_index(_call_index), "");
81eba62e9048 8014013: CallInfo structure no longer accurately reports the result of a LinkResolver operation
drchase
parents: 17024
diff changeset
   216
    break;
81eba62e9048 8014013: CallInfo structure no longer accurately reports the result of a LinkResolver operation
drchase
parents: 17024
diff changeset
   217
  case CallInfo::unknown_kind:
81eba62e9048 8014013: CallInfo structure no longer accurately reports the result of a LinkResolver operation
drchase
parents: 17024
diff changeset
   218
    assert(call_kind() != CallInfo::unknown_kind, "CallInfo must be set");
81eba62e9048 8014013: CallInfo structure no longer accurately reports the result of a LinkResolver operation
drchase
parents: 17024
diff changeset
   219
    break;
81eba62e9048 8014013: CallInfo structure no longer accurately reports the result of a LinkResolver operation
drchase
parents: 17024
diff changeset
   220
  default:
33105
294e48b4f704 8080775: Better argument formatting for assert() and friends
david
parents: 32189
diff changeset
   221
    fatal("Unexpected call kind %d", call_kind());
20017
81eba62e9048 8014013: CallInfo structure no longer accurately reports the result of a LinkResolver operation
drchase
parents: 17024
diff changeset
   222
  }
81eba62e9048 8014013: CallInfo structure no longer accurately reports the result of a LinkResolver operation
drchase
parents: 17024
diff changeset
   223
}
81eba62e9048 8014013: CallInfo structure no longer accurately reports the result of a LinkResolver operation
drchase
parents: 17024
diff changeset
   224
#endif //ASSERT
81eba62e9048 8014013: CallInfo structure no longer accurately reports the result of a LinkResolver operation
drchase
parents: 17024
diff changeset
   225
31019
d05fcdd70109 8029567: Clean up linkResolver code
coleenp
parents: 30773
diff changeset
   226
#ifndef PRODUCT
d05fcdd70109 8029567: Clean up linkResolver code
coleenp
parents: 30773
diff changeset
   227
void CallInfo::print() {
d05fcdd70109 8029567: Clean up linkResolver code
coleenp
parents: 30773
diff changeset
   228
  ResourceMark rm;
d05fcdd70109 8029567: Clean up linkResolver code
coleenp
parents: 30773
diff changeset
   229
  const char* kindstr = "unknown";
d05fcdd70109 8029567: Clean up linkResolver code
coleenp
parents: 30773
diff changeset
   230
  switch (_call_kind) {
d05fcdd70109 8029567: Clean up linkResolver code
coleenp
parents: 30773
diff changeset
   231
  case direct_call: kindstr = "direct"; break;
d05fcdd70109 8029567: Clean up linkResolver code
coleenp
parents: 30773
diff changeset
   232
  case vtable_call: kindstr = "vtable"; break;
d05fcdd70109 8029567: Clean up linkResolver code
coleenp
parents: 30773
diff changeset
   233
  case itable_call: kindstr = "itable"; break;
d05fcdd70109 8029567: Clean up linkResolver code
coleenp
parents: 30773
diff changeset
   234
  }
d05fcdd70109 8029567: Clean up linkResolver code
coleenp
parents: 30773
diff changeset
   235
  tty->print_cr("Call %s@%d %s", kindstr, _call_index,
d05fcdd70109 8029567: Clean up linkResolver code
coleenp
parents: 30773
diff changeset
   236
                _resolved_method.is_null() ? "(none)" : _resolved_method->name_and_sig_as_C_string());
d05fcdd70109 8029567: Clean up linkResolver code
coleenp
parents: 30773
diff changeset
   237
}
d05fcdd70109 8029567: Clean up linkResolver code
coleenp
parents: 30773
diff changeset
   238
#endif
20017
81eba62e9048 8014013: CallInfo structure no longer accurately reports the result of a LinkResolver operation
drchase
parents: 17024
diff changeset
   239
31019
d05fcdd70109 8029567: Clean up linkResolver code
coleenp
parents: 30773
diff changeset
   240
//------------------------------------------------------------------------------------------------------------------------
d05fcdd70109 8029567: Clean up linkResolver code
coleenp
parents: 30773
diff changeset
   241
// Implementation of LinkInfo
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   242
33593
60764a78fa5c 8140274: methodHandles and constantPoolHandles should be passed as const references
coleenp
parents: 33198
diff changeset
   243
LinkInfo::LinkInfo(const constantPoolHandle& pool, int index, TRAPS) {
31019
d05fcdd70109 8029567: Clean up linkResolver code
coleenp
parents: 30773
diff changeset
   244
   // resolve klass
d05fcdd70109 8029567: Clean up linkResolver code
coleenp
parents: 30773
diff changeset
   245
  Klass* result = pool->klass_ref_at(index, CHECK);
d05fcdd70109 8029567: Clean up linkResolver code
coleenp
parents: 30773
diff changeset
   246
  _resolved_klass = KlassHandle(THREAD, result);
d05fcdd70109 8029567: Clean up linkResolver code
coleenp
parents: 30773
diff changeset
   247
d05fcdd70109 8029567: Clean up linkResolver code
coleenp
parents: 30773
diff changeset
   248
  // Get name, signature, and static klass
d05fcdd70109 8029567: Clean up linkResolver code
coleenp
parents: 30773
diff changeset
   249
  _name          = pool->name_ref_at(index);
d05fcdd70109 8029567: Clean up linkResolver code
coleenp
parents: 30773
diff changeset
   250
  _signature     = pool->signature_ref_at(index);
d05fcdd70109 8029567: Clean up linkResolver code
coleenp
parents: 30773
diff changeset
   251
  _current_klass = KlassHandle(THREAD, pool->pool_holder());
d05fcdd70109 8029567: Clean up linkResolver code
coleenp
parents: 30773
diff changeset
   252
d05fcdd70109 8029567: Clean up linkResolver code
coleenp
parents: 30773
diff changeset
   253
  // Coming from the constant pool always checks access
d05fcdd70109 8029567: Clean up linkResolver code
coleenp
parents: 30773
diff changeset
   254
  _check_access  = true;
d05fcdd70109 8029567: Clean up linkResolver code
coleenp
parents: 30773
diff changeset
   255
}
d05fcdd70109 8029567: Clean up linkResolver code
coleenp
parents: 30773
diff changeset
   256
d05fcdd70109 8029567: Clean up linkResolver code
coleenp
parents: 30773
diff changeset
   257
char* LinkInfo::method_string() const {
d05fcdd70109 8029567: Clean up linkResolver code
coleenp
parents: 30773
diff changeset
   258
  return Method::name_and_sig_as_C_string(_resolved_klass(), _name, _signature);
d05fcdd70109 8029567: Clean up linkResolver code
coleenp
parents: 30773
diff changeset
   259
}
d05fcdd70109 8029567: Clean up linkResolver code
coleenp
parents: 30773
diff changeset
   260
d05fcdd70109 8029567: Clean up linkResolver code
coleenp
parents: 30773
diff changeset
   261
#ifndef PRODUCT
d05fcdd70109 8029567: Clean up linkResolver code
coleenp
parents: 30773
diff changeset
   262
void LinkInfo::print() {
d05fcdd70109 8029567: Clean up linkResolver code
coleenp
parents: 30773
diff changeset
   263
  ResourceMark rm;
d05fcdd70109 8029567: Clean up linkResolver code
coleenp
parents: 30773
diff changeset
   264
  tty->print_cr("Link resolved_klass=%s name=%s signature=%s current_klass=%s check_access=%s",
d05fcdd70109 8029567: Clean up linkResolver code
coleenp
parents: 30773
diff changeset
   265
                _resolved_klass->name()->as_C_string(),
d05fcdd70109 8029567: Clean up linkResolver code
coleenp
parents: 30773
diff changeset
   266
                _name->as_C_string(),
d05fcdd70109 8029567: Clean up linkResolver code
coleenp
parents: 30773
diff changeset
   267
                _signature->as_C_string(),
d05fcdd70109 8029567: Clean up linkResolver code
coleenp
parents: 30773
diff changeset
   268
                _current_klass.is_null() ? "(none)" : _current_klass->name()->as_C_string(),
d05fcdd70109 8029567: Clean up linkResolver code
coleenp
parents: 30773
diff changeset
   269
                _check_access ? "true" : "false");
d05fcdd70109 8029567: Clean up linkResolver code
coleenp
parents: 30773
diff changeset
   270
}
d05fcdd70109 8029567: Clean up linkResolver code
coleenp
parents: 30773
diff changeset
   271
#endif // PRODUCT
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   272
//------------------------------------------------------------------------------------------------------------------------
489c9b5090e2 Initial load
duke
parents:
diff changeset
   273
// Klass resolution
489c9b5090e2 Initial load
duke
parents:
diff changeset
   274
489c9b5090e2 Initial load
duke
parents:
diff changeset
   275
void LinkResolver::check_klass_accessability(KlassHandle ref_klass, KlassHandle sel_klass, TRAPS) {
13728
882756847a04 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 13396
diff changeset
   276
  if (!Reflection::verify_class_access(ref_klass(),
882756847a04 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 13396
diff changeset
   277
                                       sel_klass(),
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   278
                                       true)) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   279
    ResourceMark rm(THREAD);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   280
    Exceptions::fthrow(
489c9b5090e2 Initial load
duke
parents:
diff changeset
   281
      THREAD_AND_LOCATION,
8076
96d498ec7ae1 6990754: Use native memory and reference counting to implement SymbolTable
coleenp
parents: 7397
diff changeset
   282
      vmSymbols::java_lang_IllegalAccessError(),
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   283
      "tried to access class %s from class %s",
489c9b5090e2 Initial load
duke
parents:
diff changeset
   284
      sel_klass->external_name(),
489c9b5090e2 Initial load
duke
parents:
diff changeset
   285
      ref_klass->external_name()
489c9b5090e2 Initial load
duke
parents:
diff changeset
   286
    );
489c9b5090e2 Initial load
duke
parents:
diff changeset
   287
    return;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   288
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   289
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
   290
489c9b5090e2 Initial load
duke
parents:
diff changeset
   291
//------------------------------------------------------------------------------------------------------------------------
489c9b5090e2 Initial load
duke
parents:
diff changeset
   292
// Method resolution
489c9b5090e2 Initial load
duke
parents:
diff changeset
   293
//
489c9b5090e2 Initial load
duke
parents:
diff changeset
   294
// According to JVM spec. $5.4.3c & $5.4.3d
489c9b5090e2 Initial load
duke
parents:
diff changeset
   295
20391
7b146c5ebb18 8009130: Lambda: Fix access controls, loader constraints.
acorn
parents: 20284
diff changeset
   296
// Look up method in klasses, including static methods
7b146c5ebb18 8009130: Lambda: Fix access controls, loader constraints.
acorn
parents: 20284
diff changeset
   297
// Then look up local default methods
31019
d05fcdd70109 8029567: Clean up linkResolver code
coleenp
parents: 30773
diff changeset
   298
methodHandle LinkResolver::lookup_method_in_klasses(const LinkInfo& link_info,
d05fcdd70109 8029567: Clean up linkResolver code
coleenp
parents: 30773
diff changeset
   299
                                                    bool checkpolymorphism,
d05fcdd70109 8029567: Clean up linkResolver code
coleenp
parents: 30773
diff changeset
   300
                                                    bool in_imethod_resolve, TRAPS) {
d05fcdd70109 8029567: Clean up linkResolver code
coleenp
parents: 30773
diff changeset
   301
  KlassHandle klass = link_info.resolved_klass();
d05fcdd70109 8029567: Clean up linkResolver code
coleenp
parents: 30773
diff changeset
   302
  Symbol* name = link_info.name();
d05fcdd70109 8029567: Clean up linkResolver code
coleenp
parents: 30773
diff changeset
   303
  Symbol* signature = link_info.signature();
d05fcdd70109 8029567: Clean up linkResolver code
coleenp
parents: 30773
diff changeset
   304
23999
22eb7be3d99d 8033150: invokestatic: IncompatibleClassChangeError trying to invoke static method from a parent in presence of conflicting defaults.
lfoltan
parents: 22739
diff changeset
   305
  // Ignore overpasses so statics can be found during resolution
31019
d05fcdd70109 8029567: Clean up linkResolver code
coleenp
parents: 30773
diff changeset
   306
  Method* result = klass->uncached_lookup_method(name, signature, Klass::skip_overpass);
22219
4bfd5df70189 8028741: Interface Method Resolution should skip static and non-public methods in j.l.Object
hseigel
parents: 22218
diff changeset
   307
33611
9abd65805e19 8139203: Consistent naming for klass type predicates
coleenp
parents: 33602
diff changeset
   308
  if (klass->is_array_klass()) {
27020
a7c8010446c2 8036805: Correct linker method lookup.
lfoltan
parents: 25057
diff changeset
   309
    // Only consider klass and super klass for arrays
31019
d05fcdd70109 8029567: Clean up linkResolver code
coleenp
parents: 30773
diff changeset
   310
    return methodHandle(THREAD, result);
27020
a7c8010446c2 8036805: Correct linker method lookup.
lfoltan
parents: 25057
diff changeset
   311
  }
a7c8010446c2 8036805: Correct linker method lookup.
lfoltan
parents: 25057
diff changeset
   312
33602
16053580a684 8139163: InstanceKlass::cast passes through NULL
coleenp
parents: 33593
diff changeset
   313
  InstanceKlass* ik = InstanceKlass::cast(klass());
16053580a684 8139163: InstanceKlass::cast passes through NULL
coleenp
parents: 33593
diff changeset
   314
22219
4bfd5df70189 8028741: Interface Method Resolution should skip static and non-public methods in j.l.Object
hseigel
parents: 22218
diff changeset
   315
  // JDK 8, JVMS 5.4.3.4: Interface method resolution should
4bfd5df70189 8028741: Interface Method Resolution should skip static and non-public methods in j.l.Object
hseigel
parents: 22218
diff changeset
   316
  // ignore static and non-public methods of java.lang.Object,
4bfd5df70189 8028741: Interface Method Resolution should skip static and non-public methods in j.l.Object
hseigel
parents: 22218
diff changeset
   317
  // like clone, finalize, registerNatives.
4bfd5df70189 8028741: Interface Method Resolution should skip static and non-public methods in j.l.Object
hseigel
parents: 22218
diff changeset
   318
  if (in_imethod_resolve &&
31019
d05fcdd70109 8029567: Clean up linkResolver code
coleenp
parents: 30773
diff changeset
   319
      result != NULL &&
33602
16053580a684 8139163: InstanceKlass::cast passes through NULL
coleenp
parents: 33593
diff changeset
   320
      ik->is_interface() &&
31019
d05fcdd70109 8029567: Clean up linkResolver code
coleenp
parents: 30773
diff changeset
   321
      (result->is_static() || !result->is_public()) &&
d05fcdd70109 8029567: Clean up linkResolver code
coleenp
parents: 30773
diff changeset
   322
      result->method_holder() == SystemDictionary::Object_klass()) {
d05fcdd70109 8029567: Clean up linkResolver code
coleenp
parents: 30773
diff changeset
   323
    result = NULL;
22219
4bfd5df70189 8028741: Interface Method Resolution should skip static and non-public methods in j.l.Object
hseigel
parents: 22218
diff changeset
   324
  }
4bfd5df70189 8028741: Interface Method Resolution should skip static and non-public methods in j.l.Object
hseigel
parents: 22218
diff changeset
   325
23999
22eb7be3d99d 8033150: invokestatic: IncompatibleClassChangeError trying to invoke static method from a parent in presence of conflicting defaults.
lfoltan
parents: 22739
diff changeset
   326
  // Before considering default methods, check for an overpass in the
22eb7be3d99d 8033150: invokestatic: IncompatibleClassChangeError trying to invoke static method from a parent in presence of conflicting defaults.
lfoltan
parents: 22739
diff changeset
   327
  // current class if a method has not been found.
31019
d05fcdd70109 8029567: Clean up linkResolver code
coleenp
parents: 30773
diff changeset
   328
  if (result == NULL) {
33602
16053580a684 8139163: InstanceKlass::cast passes through NULL
coleenp
parents: 33593
diff changeset
   329
    result = ik->find_method(name, signature);
23999
22eb7be3d99d 8033150: invokestatic: IncompatibleClassChangeError trying to invoke static method from a parent in presence of conflicting defaults.
lfoltan
parents: 22739
diff changeset
   330
  }
22eb7be3d99d 8033150: invokestatic: IncompatibleClassChangeError trying to invoke static method from a parent in presence of conflicting defaults.
lfoltan
parents: 22739
diff changeset
   331
31019
d05fcdd70109 8029567: Clean up linkResolver code
coleenp
parents: 30773
diff changeset
   332
  if (result == NULL) {
33602
16053580a684 8139163: InstanceKlass::cast passes through NULL
coleenp
parents: 33593
diff changeset
   333
    Array<Method*>* default_methods = ik->default_methods();
20391
7b146c5ebb18 8009130: Lambda: Fix access controls, loader constraints.
acorn
parents: 20284
diff changeset
   334
    if (default_methods != NULL) {
31019
d05fcdd70109 8029567: Clean up linkResolver code
coleenp
parents: 30773
diff changeset
   335
      result = InstanceKlass::find_method(default_methods, name, signature);
20391
7b146c5ebb18 8009130: Lambda: Fix access controls, loader constraints.
acorn
parents: 20284
diff changeset
   336
    }
7b146c5ebb18 8009130: Lambda: Fix access controls, loader constraints.
acorn
parents: 20284
diff changeset
   337
  }
7b146c5ebb18 8009130: Lambda: Fix access controls, loader constraints.
acorn
parents: 20284
diff changeset
   338
31019
d05fcdd70109 8029567: Clean up linkResolver code
coleenp
parents: 30773
diff changeset
   339
  if (checkpolymorphism && result != NULL) {
d05fcdd70109 8029567: Clean up linkResolver code
coleenp
parents: 30773
diff changeset
   340
    vmIntrinsics::ID iid = result->intrinsic_id();
13391
30245956af37 7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents: 10565
diff changeset
   341
    if (MethodHandles::is_signature_polymorphic(iid)) {
30245956af37 7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents: 10565
diff changeset
   342
      // Do not link directly to these.  The VM must produce a synthetic one using lookup_polymorphic_method.
31019
d05fcdd70109 8029567: Clean up linkResolver code
coleenp
parents: 30773
diff changeset
   343
      return NULL;
5420
586d3988e72b 6939134: JSR 292 adjustments to method handle invocation
jrose
parents: 4571
diff changeset
   344
    }
586d3988e72b 6939134: JSR 292 adjustments to method handle invocation
jrose
parents: 4571
diff changeset
   345
  }
31019
d05fcdd70109 8029567: Clean up linkResolver code
coleenp
parents: 30773
diff changeset
   346
  return methodHandle(THREAD, result);
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   347
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
   348
489c9b5090e2 Initial load
duke
parents:
diff changeset
   349
// returns first instance method
20391
7b146c5ebb18 8009130: Lambda: Fix access controls, loader constraints.
acorn
parents: 20284
diff changeset
   350
// Looks up method in classes, then looks up local default methods
31019
d05fcdd70109 8029567: Clean up linkResolver code
coleenp
parents: 30773
diff changeset
   351
methodHandle LinkResolver::lookup_instance_method_in_klasses(KlassHandle klass,
d05fcdd70109 8029567: Clean up linkResolver code
coleenp
parents: 30773
diff changeset
   352
                                                             Symbol* name,
d05fcdd70109 8029567: Clean up linkResolver code
coleenp
parents: 30773
diff changeset
   353
                                                             Symbol* signature, TRAPS) {
d05fcdd70109 8029567: Clean up linkResolver code
coleenp
parents: 30773
diff changeset
   354
  Method* result = klass->uncached_lookup_method(name, signature, Klass::find_overpass);
d05fcdd70109 8029567: Clean up linkResolver code
coleenp
parents: 30773
diff changeset
   355
d05fcdd70109 8029567: Clean up linkResolver code
coleenp
parents: 30773
diff changeset
   356
  while (result != NULL && result->is_static() && result->method_holder()->super() != NULL) {
d05fcdd70109 8029567: Clean up linkResolver code
coleenp
parents: 30773
diff changeset
   357
    Klass* super_klass = result->method_holder()->super();
d05fcdd70109 8029567: Clean up linkResolver code
coleenp
parents: 30773
diff changeset
   358
    result = super_klass->uncached_lookup_method(name, signature, Klass::find_overpass);
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   359
  }
20391
7b146c5ebb18 8009130: Lambda: Fix access controls, loader constraints.
acorn
parents: 20284
diff changeset
   360
33611
9abd65805e19 8139203: Consistent naming for klass type predicates
coleenp
parents: 33602
diff changeset
   361
  if (klass->is_array_klass()) {
27020
a7c8010446c2 8036805: Correct linker method lookup.
lfoltan
parents: 25057
diff changeset
   362
    // Only consider klass and super klass for arrays
31019
d05fcdd70109 8029567: Clean up linkResolver code
coleenp
parents: 30773
diff changeset
   363
    return methodHandle(THREAD, result);
27020
a7c8010446c2 8036805: Correct linker method lookup.
lfoltan
parents: 25057
diff changeset
   364
  }
a7c8010446c2 8036805: Correct linker method lookup.
lfoltan
parents: 25057
diff changeset
   365
31019
d05fcdd70109 8029567: Clean up linkResolver code
coleenp
parents: 30773
diff changeset
   366
  if (result == NULL) {
20391
7b146c5ebb18 8009130: Lambda: Fix access controls, loader constraints.
acorn
parents: 20284
diff changeset
   367
    Array<Method*>* default_methods = InstanceKlass::cast(klass())->default_methods();
7b146c5ebb18 8009130: Lambda: Fix access controls, loader constraints.
acorn
parents: 20284
diff changeset
   368
    if (default_methods != NULL) {
31019
d05fcdd70109 8029567: Clean up linkResolver code
coleenp
parents: 30773
diff changeset
   369
      result = InstanceKlass::find_method(default_methods, name, signature);
d05fcdd70109 8029567: Clean up linkResolver code
coleenp
parents: 30773
diff changeset
   370
      assert(result == NULL || !result->is_static(), "static defaults not allowed");
20391
7b146c5ebb18 8009130: Lambda: Fix access controls, loader constraints.
acorn
parents: 20284
diff changeset
   371
    }
7b146c5ebb18 8009130: Lambda: Fix access controls, loader constraints.
acorn
parents: 20284
diff changeset
   372
  }
31019
d05fcdd70109 8029567: Clean up linkResolver code
coleenp
parents: 30773
diff changeset
   373
  return methodHandle(THREAD, result);
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   374
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
   375
20391
7b146c5ebb18 8009130: Lambda: Fix access controls, loader constraints.
acorn
parents: 20284
diff changeset
   376
int LinkResolver::vtable_index_of_interface_method(KlassHandle klass,
31019
d05fcdd70109 8029567: Clean up linkResolver code
coleenp
parents: 30773
diff changeset
   377
                                                   const methodHandle& resolved_method) {
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   378
20391
7b146c5ebb18 8009130: Lambda: Fix access controls, loader constraints.
acorn
parents: 20284
diff changeset
   379
  int vtable_index = Method::invalid_vtable_index;
7b146c5ebb18 8009130: Lambda: Fix access controls, loader constraints.
acorn
parents: 20284
diff changeset
   380
  Symbol* name = resolved_method->name();
7b146c5ebb18 8009130: Lambda: Fix access controls, loader constraints.
acorn
parents: 20284
diff changeset
   381
  Symbol* signature = resolved_method->signature();
33602
16053580a684 8139163: InstanceKlass::cast passes through NULL
coleenp
parents: 33593
diff changeset
   382
  InstanceKlass* ik = InstanceKlass::cast(klass());
20391
7b146c5ebb18 8009130: Lambda: Fix access controls, loader constraints.
acorn
parents: 20284
diff changeset
   383
7b146c5ebb18 8009130: Lambda: Fix access controls, loader constraints.
acorn
parents: 20284
diff changeset
   384
  // First check in default method array
33602
16053580a684 8139163: InstanceKlass::cast passes through NULL
coleenp
parents: 33593
diff changeset
   385
  if (!resolved_method->is_abstract() && ik->default_methods() != NULL) {
16053580a684 8139163: InstanceKlass::cast passes through NULL
coleenp
parents: 33593
diff changeset
   386
    int index = InstanceKlass::find_method_index(ik->default_methods(),
32189
5264b560ab1b 8087342: Crash in klassItable::initialize_itable_for_interface
acorn
parents: 31019
diff changeset
   387
                                                 name, signature, Klass::find_overpass,
5264b560ab1b 8087342: Crash in klassItable::initialize_itable_for_interface
acorn
parents: 31019
diff changeset
   388
                                                 Klass::find_static, Klass::find_private);
20391
7b146c5ebb18 8009130: Lambda: Fix access controls, loader constraints.
acorn
parents: 20284
diff changeset
   389
    if (index >= 0 ) {
33602
16053580a684 8139163: InstanceKlass::cast passes through NULL
coleenp
parents: 33593
diff changeset
   390
      vtable_index = ik->default_vtable_indices()->at(index);
20391
7b146c5ebb18 8009130: Lambda: Fix access controls, loader constraints.
acorn
parents: 20284
diff changeset
   391
    }
7b146c5ebb18 8009130: Lambda: Fix access controls, loader constraints.
acorn
parents: 20284
diff changeset
   392
  }
7b146c5ebb18 8009130: Lambda: Fix access controls, loader constraints.
acorn
parents: 20284
diff changeset
   393
  if (vtable_index == Method::invalid_vtable_index) {
7b146c5ebb18 8009130: Lambda: Fix access controls, loader constraints.
acorn
parents: 20284
diff changeset
   394
    // get vtable_index for miranda methods
21556
e75cd34a59e0 8027229: ICCE expected for >=2 maximally specific default methods.
acorn
parents: 21078
diff changeset
   395
    ResourceMark rm;
33602
16053580a684 8139163: InstanceKlass::cast passes through NULL
coleenp
parents: 33593
diff changeset
   396
    klassVtable *vt = ik->vtable();
20391
7b146c5ebb18 8009130: Lambda: Fix access controls, loader constraints.
acorn
parents: 20284
diff changeset
   397
    vtable_index = vt->index_of_miranda(name, signature);
7b146c5ebb18 8009130: Lambda: Fix access controls, loader constraints.
acorn
parents: 20284
diff changeset
   398
  }
7b146c5ebb18 8009130: Lambda: Fix access controls, loader constraints.
acorn
parents: 20284
diff changeset
   399
  return vtable_index;
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   400
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
   401
31019
d05fcdd70109 8029567: Clean up linkResolver code
coleenp
parents: 30773
diff changeset
   402
methodHandle LinkResolver::lookup_method_in_interfaces(const LinkInfo& cp_info, TRAPS) {
d05fcdd70109 8029567: Clean up linkResolver code
coleenp
parents: 30773
diff changeset
   403
  InstanceKlass *ik = InstanceKlass::cast(cp_info.resolved_klass()());
22232
26acfad336c0 8027804: JCK resolveMethod test fails expecting AbstractMethodError
hseigel
parents: 22219
diff changeset
   404
26acfad336c0 8027804: JCK resolveMethod test fails expecting AbstractMethodError
hseigel
parents: 22219
diff changeset
   405
  // Specify 'true' in order to skip default methods when searching the
26acfad336c0 8027804: JCK resolveMethod test fails expecting AbstractMethodError
hseigel
parents: 22219
diff changeset
   406
  // interfaces.  Function lookup_method_in_klasses() already looked for
26acfad336c0 8027804: JCK resolveMethod test fails expecting AbstractMethodError
hseigel
parents: 22219
diff changeset
   407
  // the method in the default methods table.
31019
d05fcdd70109 8029567: Clean up linkResolver code
coleenp
parents: 30773
diff changeset
   408
  return methodHandle(THREAD,
d05fcdd70109 8029567: Clean up linkResolver code
coleenp
parents: 30773
diff changeset
   409
    ik->lookup_method_in_all_interfaces(cp_info.name(), cp_info.signature(),
d05fcdd70109 8029567: Clean up linkResolver code
coleenp
parents: 30773
diff changeset
   410
    Klass::skip_defaults));
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   411
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
   412
31019
d05fcdd70109 8029567: Clean up linkResolver code
coleenp
parents: 30773
diff changeset
   413
methodHandle LinkResolver::lookup_polymorphic_method(
d05fcdd70109 8029567: Clean up linkResolver code
coleenp
parents: 30773
diff changeset
   414
                                             const LinkInfo& link_info,
13929
8da0dc50a6e4 7200949: JSR 292: rubybench/bench/time/bench_base64.rb fails with jruby.jar not on boot class path
twisti
parents: 13732
diff changeset
   415
                                             Handle *appendix_result_or_null,
8da0dc50a6e4 7200949: JSR 292: rubybench/bench/time/bench_base64.rb fails with jruby.jar not on boot class path
twisti
parents: 13732
diff changeset
   416
                                             Handle *method_type_result,
13391
30245956af37 7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents: 10565
diff changeset
   417
                                             TRAPS) {
31019
d05fcdd70109 8029567: Clean up linkResolver code
coleenp
parents: 30773
diff changeset
   418
  KlassHandle klass = link_info.resolved_klass();
d05fcdd70109 8029567: Clean up linkResolver code
coleenp
parents: 30773
diff changeset
   419
  Symbol* name = link_info.name();
d05fcdd70109 8029567: Clean up linkResolver code
coleenp
parents: 30773
diff changeset
   420
  Symbol* full_signature = link_info.signature();
d05fcdd70109 8029567: Clean up linkResolver code
coleenp
parents: 30773
diff changeset
   421
13391
30245956af37 7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents: 10565
diff changeset
   422
  vmIntrinsics::ID iid = MethodHandles::signature_polymorphic_name_id(name);
30245956af37 7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents: 10565
diff changeset
   423
  if (TraceMethodHandles) {
16691
c117eee6d38a 8011952: Missing ResourceMarks in TraceMethodHandles
kmo
parents: 16591
diff changeset
   424
    ResourceMark rm(THREAD);
13391
30245956af37 7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents: 10565
diff changeset
   425
    tty->print_cr("lookup_polymorphic_method iid=%s %s.%s%s",
30245956af37 7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents: 10565
diff changeset
   426
                  vmIntrinsics::name_at(iid), klass->external_name(),
30245956af37 7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents: 10565
diff changeset
   427
                  name->as_C_string(), full_signature->as_C_string());
30245956af37 7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents: 10565
diff changeset
   428
  }
24322
c2978d1578e3 8036956: remove EnableInvokeDynamic flag
anoll
parents: 23999
diff changeset
   429
  if (klass() == SystemDictionary::MethodHandle_klass() &&
13391
30245956af37 7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents: 10565
diff changeset
   430
      iid != vmIntrinsics::_none) {
30245956af37 7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents: 10565
diff changeset
   431
    if (MethodHandles::is_signature_polymorphic_intrinsic(iid)) {
30245956af37 7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents: 10565
diff changeset
   432
      // Most of these do not need an up-call to Java to resolve, so can be done anywhere.
30245956af37 7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents: 10565
diff changeset
   433
      // Do not erase last argument type (MemberName) if it is a static linkTo method.
30245956af37 7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents: 10565
diff changeset
   434
      bool keep_last_arg = MethodHandles::is_signature_polymorphic_static(iid);
30245956af37 7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents: 10565
diff changeset
   435
      TempNewSymbol basic_signature =
31019
d05fcdd70109 8029567: Clean up linkResolver code
coleenp
parents: 30773
diff changeset
   436
        MethodHandles::lookup_basic_type_signature(full_signature, keep_last_arg, CHECK_NULL);
13391
30245956af37 7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents: 10565
diff changeset
   437
      if (TraceMethodHandles) {
16691
c117eee6d38a 8011952: Missing ResourceMarks in TraceMethodHandles
kmo
parents: 16591
diff changeset
   438
        ResourceMark rm(THREAD);
13391
30245956af37 7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents: 10565
diff changeset
   439
        tty->print_cr("lookup_polymorphic_method %s %s => basic %s",
30245956af37 7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents: 10565
diff changeset
   440
                      name->as_C_string(),
30245956af37 7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents: 10565
diff changeset
   441
                      full_signature->as_C_string(),
30245956af37 7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents: 10565
diff changeset
   442
                      basic_signature->as_C_string());
30245956af37 7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents: 10565
diff changeset
   443
      }
31019
d05fcdd70109 8029567: Clean up linkResolver code
coleenp
parents: 30773
diff changeset
   444
      methodHandle result = SystemDictionary::find_method_handle_intrinsic(iid,
13391
30245956af37 7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents: 10565
diff changeset
   445
                                                              basic_signature,
31019
d05fcdd70109 8029567: Clean up linkResolver code
coleenp
parents: 30773
diff changeset
   446
                                                              CHECK_NULL);
13391
30245956af37 7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents: 10565
diff changeset
   447
      if (result.not_null()) {
30245956af37 7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents: 10565
diff changeset
   448
        assert(result->is_method_handle_intrinsic(), "MH.invokeBasic or MH.linkTo* intrinsic");
30245956af37 7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents: 10565
diff changeset
   449
        assert(result->intrinsic_id() != vmIntrinsics::_invokeGeneric, "wrong place to find this");
30245956af37 7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents: 10565
diff changeset
   450
        assert(basic_signature == result->signature(), "predict the result signature");
30245956af37 7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents: 10565
diff changeset
   451
        if (TraceMethodHandles) {
35543
0961315f4016 8140659: C1: invokedynamic call patching violates JVMS-6.5.invokedynamic
vlivanov
parents: 35135
diff changeset
   452
          ttyLocker ttyl;
13391
30245956af37 7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents: 10565
diff changeset
   453
          tty->print("lookup_polymorphic_method => intrinsic ");
30245956af37 7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents: 10565
diff changeset
   454
          result->print_on(tty);
30245956af37 7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents: 10565
diff changeset
   455
        }
6062
bab93afe9df7 6964498: JSR 292 invokedynamic sites need local bootstrap methods
jrose
parents: 5547
diff changeset
   456
      }
31019
d05fcdd70109 8029567: Clean up linkResolver code
coleenp
parents: 30773
diff changeset
   457
      return result;
13391
30245956af37 7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents: 10565
diff changeset
   458
    } else if (iid == vmIntrinsics::_invokeGeneric
33160
c59f1676d27e 8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents: 32189
diff changeset
   459
               && THREAD->can_call_java()
13391
30245956af37 7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents: 10565
diff changeset
   460
               && appendix_result_or_null != NULL) {
30245956af37 7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents: 10565
diff changeset
   461
      // This is a method with type-checking semantics.
30245956af37 7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents: 10565
diff changeset
   462
      // We will ask Java code to spin an adapter method for it.
30245956af37 7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents: 10565
diff changeset
   463
      if (!MethodHandles::enabled()) {
30245956af37 7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents: 10565
diff changeset
   464
        // Make sure the Java part of the runtime has been booted up.
13728
882756847a04 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 13396
diff changeset
   465
        Klass* natives = SystemDictionary::MethodHandleNatives_klass();
882756847a04 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 13396
diff changeset
   466
        if (natives == NULL || InstanceKlass::cast(natives)->is_not_initialized()) {
13391
30245956af37 7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents: 10565
diff changeset
   467
          SystemDictionary::resolve_or_fail(vmSymbols::java_lang_invoke_MethodHandleNatives(),
30245956af37 7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents: 10565
diff changeset
   468
                                            Handle(),
30245956af37 7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents: 10565
diff changeset
   469
                                            Handle(),
30245956af37 7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents: 10565
diff changeset
   470
                                            true,
31019
d05fcdd70109 8029567: Clean up linkResolver code
coleenp
parents: 30773
diff changeset
   471
                                            CHECK_NULL);
13391
30245956af37 7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents: 10565
diff changeset
   472
        }
30245956af37 7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents: 10565
diff changeset
   473
      }
30245956af37 7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents: 10565
diff changeset
   474
30245956af37 7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents: 10565
diff changeset
   475
      Handle appendix;
13929
8da0dc50a6e4 7200949: JSR 292: rubybench/bench/time/bench_base64.rb fails with jruby.jar not on boot class path
twisti
parents: 13732
diff changeset
   476
      Handle method_type;
31019
d05fcdd70109 8029567: Clean up linkResolver code
coleenp
parents: 30773
diff changeset
   477
      methodHandle result = SystemDictionary::find_method_handle_invoker(
d05fcdd70109 8029567: Clean up linkResolver code
coleenp
parents: 30773
diff changeset
   478
                                                            name,
13391
30245956af37 7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents: 10565
diff changeset
   479
                                                            full_signature,
31019
d05fcdd70109 8029567: Clean up linkResolver code
coleenp
parents: 30773
diff changeset
   480
                                                            link_info.current_klass(),
13391
30245956af37 7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents: 10565
diff changeset
   481
                                                            &appendix,
13929
8da0dc50a6e4 7200949: JSR 292: rubybench/bench/time/bench_base64.rb fails with jruby.jar not on boot class path
twisti
parents: 13732
diff changeset
   482
                                                            &method_type,
31019
d05fcdd70109 8029567: Clean up linkResolver code
coleenp
parents: 30773
diff changeset
   483
                                                            CHECK_NULL);
13391
30245956af37 7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents: 10565
diff changeset
   484
      if (TraceMethodHandles) {
35543
0961315f4016 8140659: C1: invokedynamic call patching violates JVMS-6.5.invokedynamic
vlivanov
parents: 35135
diff changeset
   485
        ttyLocker ttyl;
13391
30245956af37 7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents: 10565
diff changeset
   486
        tty->print("lookup_polymorphic_method => (via Java) ");
30245956af37 7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents: 10565
diff changeset
   487
        result->print_on(tty);
30245956af37 7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents: 10565
diff changeset
   488
        tty->print("  lookup_polymorphic_method => appendix = ");
30245956af37 7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents: 10565
diff changeset
   489
        if (appendix.is_null())  tty->print_cr("(none)");
30245956af37 7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents: 10565
diff changeset
   490
        else                     appendix->print_on(tty);
30245956af37 7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents: 10565
diff changeset
   491
      }
30245956af37 7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents: 10565
diff changeset
   492
      if (result.not_null()) {
30245956af37 7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents: 10565
diff changeset
   493
#ifdef ASSERT
16691
c117eee6d38a 8011952: Missing ResourceMarks in TraceMethodHandles
kmo
parents: 16591
diff changeset
   494
        ResourceMark rm(THREAD);
c117eee6d38a 8011952: Missing ResourceMarks in TraceMethodHandles
kmo
parents: 16591
diff changeset
   495
13391
30245956af37 7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents: 10565
diff changeset
   496
        TempNewSymbol basic_signature =
31019
d05fcdd70109 8029567: Clean up linkResolver code
coleenp
parents: 30773
diff changeset
   497
          MethodHandles::lookup_basic_type_signature(full_signature, CHECK_NULL);
13391
30245956af37 7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents: 10565
diff changeset
   498
        int actual_size_of_params = result->size_of_parameters();
30245956af37 7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents: 10565
diff changeset
   499
        int expected_size_of_params = ArgumentSizeComputer(basic_signature).size();
30245956af37 7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents: 10565
diff changeset
   500
        // +1 for MethodHandle.this, +1 for trailing MethodType
30245956af37 7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents: 10565
diff changeset
   501
        if (!MethodHandles::is_signature_polymorphic_static(iid))  expected_size_of_params += 1;
30245956af37 7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents: 10565
diff changeset
   502
        if (appendix.not_null())                                   expected_size_of_params += 1;
30245956af37 7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents: 10565
diff changeset
   503
        if (actual_size_of_params != expected_size_of_params) {
30245956af37 7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents: 10565
diff changeset
   504
          tty->print_cr("*** basic_signature=%s", basic_signature->as_C_string());
30245956af37 7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents: 10565
diff changeset
   505
          tty->print_cr("*** result for %s: ", vmIntrinsics::name_at(iid));
30245956af37 7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents: 10565
diff changeset
   506
          result->print();
30245956af37 7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents: 10565
diff changeset
   507
        }
30245956af37 7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents: 10565
diff changeset
   508
        assert(actual_size_of_params == expected_size_of_params,
33105
294e48b4f704 8080775: Better argument formatting for assert() and friends
david
parents: 32189
diff changeset
   509
               "%d != %d", actual_size_of_params, expected_size_of_params);
13391
30245956af37 7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents: 10565
diff changeset
   510
#endif //ASSERT
30245956af37 7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents: 10565
diff changeset
   511
30245956af37 7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents: 10565
diff changeset
   512
        assert(appendix_result_or_null != NULL, "");
30245956af37 7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents: 10565
diff changeset
   513
        (*appendix_result_or_null) = appendix;
13929
8da0dc50a6e4 7200949: JSR 292: rubybench/bench/time/bench_base64.rb fails with jruby.jar not on boot class path
twisti
parents: 13732
diff changeset
   514
        (*method_type_result)      = method_type;
13391
30245956af37 7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents: 10565
diff changeset
   515
      }
31019
d05fcdd70109 8029567: Clean up linkResolver code
coleenp
parents: 30773
diff changeset
   516
      return result;
2534
08dac9ce0cd7 6655638: dynamic languages need method handles
jrose
parents: 1
diff changeset
   517
    }
08dac9ce0cd7 6655638: dynamic languages need method handles
jrose
parents: 1
diff changeset
   518
  }
31019
d05fcdd70109 8029567: Clean up linkResolver code
coleenp
parents: 30773
diff changeset
   519
  return NULL;
2534
08dac9ce0cd7 6655638: dynamic languages need method handles
jrose
parents: 1
diff changeset
   520
}
08dac9ce0cd7 6655638: dynamic languages need method handles
jrose
parents: 1
diff changeset
   521
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   522
void LinkResolver::check_method_accessability(KlassHandle ref_klass,
489c9b5090e2 Initial load
duke
parents:
diff changeset
   523
                                              KlassHandle resolved_klass,
489c9b5090e2 Initial load
duke
parents:
diff changeset
   524
                                              KlassHandle sel_klass,
31019
d05fcdd70109 8029567: Clean up linkResolver code
coleenp
parents: 30773
diff changeset
   525
                                              const methodHandle& sel_method,
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   526
                                              TRAPS) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   527
489c9b5090e2 Initial load
duke
parents:
diff changeset
   528
  AccessFlags flags = sel_method->access_flags();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   529
22219
4bfd5df70189 8028741: Interface Method Resolution should skip static and non-public methods in j.l.Object
hseigel
parents: 22218
diff changeset
   530
  // Special case:  arrays always override "clone". JVMS 2.15.
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   531
  // If the resolved klass is an array class, and the declaring class
489c9b5090e2 Initial load
duke
parents:
diff changeset
   532
  // is java.lang.Object and the method is "clone", set the flags
489c9b5090e2 Initial load
duke
parents:
diff changeset
   533
  // to public.
489c9b5090e2 Initial load
duke
parents:
diff changeset
   534
  //
22219
4bfd5df70189 8028741: Interface Method Resolution should skip static and non-public methods in j.l.Object
hseigel
parents: 22218
diff changeset
   535
  // We'll check for the method name first, as that's most likely
4bfd5df70189 8028741: Interface Method Resolution should skip static and non-public methods in j.l.Object
hseigel
parents: 22218
diff changeset
   536
  // to be false (so we'll short-circuit out of these tests).
4bfd5df70189 8028741: Interface Method Resolution should skip static and non-public methods in j.l.Object
hseigel
parents: 22218
diff changeset
   537
  if (sel_method->name() == vmSymbols::clone_name() &&
4bfd5df70189 8028741: Interface Method Resolution should skip static and non-public methods in j.l.Object
hseigel
parents: 22218
diff changeset
   538
      sel_klass() == SystemDictionary::Object_klass() &&
33611
9abd65805e19 8139203: Consistent naming for klass type predicates
coleenp
parents: 33602
diff changeset
   539
      resolved_klass->is_array_klass()) {
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   540
    // We need to change "protected" to "public".
22219
4bfd5df70189 8028741: Interface Method Resolution should skip static and non-public methods in j.l.Object
hseigel
parents: 22218
diff changeset
   541
    assert(flags.is_protected(), "clone not protected?");
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   542
    jint new_flags = flags.as_int();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   543
    new_flags = new_flags & (~JVM_ACC_PROTECTED);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   544
    new_flags = new_flags | JVM_ACC_PUBLIC;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   545
    flags.set_flags(new_flags);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   546
  }
13391
30245956af37 7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents: 10565
diff changeset
   547
//  assert(extra_arg_result_or_null != NULL, "must be able to return extra argument");
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   548
13728
882756847a04 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 13396
diff changeset
   549
  if (!Reflection::verify_field_access(ref_klass(),
882756847a04 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 13396
diff changeset
   550
                                       resolved_klass(),
882756847a04 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 13396
diff changeset
   551
                                       sel_klass(),
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   552
                                       flags,
489c9b5090e2 Initial load
duke
parents:
diff changeset
   553
                                       true)) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   554
    ResourceMark rm(THREAD);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   555
    Exceptions::fthrow(
489c9b5090e2 Initial load
duke
parents:
diff changeset
   556
      THREAD_AND_LOCATION,
8076
96d498ec7ae1 6990754: Use native memory and reference counting to implement SymbolTable
coleenp
parents: 7397
diff changeset
   557
      vmSymbols::java_lang_IllegalAccessError(),
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   558
      "tried to access method %s.%s%s from class %s",
489c9b5090e2 Initial load
duke
parents:
diff changeset
   559
      sel_klass->external_name(),
489c9b5090e2 Initial load
duke
parents:
diff changeset
   560
      sel_method->name()->as_C_string(),
489c9b5090e2 Initial load
duke
parents:
diff changeset
   561
      sel_method->signature()->as_C_string(),
489c9b5090e2 Initial load
duke
parents:
diff changeset
   562
      ref_klass->external_name()
489c9b5090e2 Initial load
duke
parents:
diff changeset
   563
    );
489c9b5090e2 Initial load
duke
parents:
diff changeset
   564
    return;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   565
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   566
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
   567
31019
d05fcdd70109 8029567: Clean up linkResolver code
coleenp
parents: 30773
diff changeset
   568
methodHandle LinkResolver::resolve_method_statically(Bytecodes::Code code,
33593
60764a78fa5c 8140274: methodHandles and constantPoolHandles should be passed as const references
coleenp
parents: 33198
diff changeset
   569
                                                     const constantPoolHandle& pool, int index, TRAPS) {
20017
81eba62e9048 8014013: CallInfo structure no longer accurately reports the result of a LinkResolver operation
drchase
parents: 17024
diff changeset
   570
  // This method is used only
81eba62e9048 8014013: CallInfo structure no longer accurately reports the result of a LinkResolver operation
drchase
parents: 17024
diff changeset
   571
  // (1) in C2 from InlineTree::ok_to_inline (via ciMethod::check_call),
81eba62e9048 8014013: CallInfo structure no longer accurately reports the result of a LinkResolver operation
drchase
parents: 17024
diff changeset
   572
  // and
81eba62e9048 8014013: CallInfo structure no longer accurately reports the result of a LinkResolver operation
drchase
parents: 17024
diff changeset
   573
  // (2) in Bytecode_invoke::static_target
81eba62e9048 8014013: CallInfo structure no longer accurately reports the result of a LinkResolver operation
drchase
parents: 17024
diff changeset
   574
  // It appears to fail when applied to an invokeinterface call site.
81eba62e9048 8014013: CallInfo structure no longer accurately reports the result of a LinkResolver operation
drchase
parents: 17024
diff changeset
   575
  // FIXME: Remove this method and ciMethod::check_call; refactor to use the other LinkResolver entry points.
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   576
  // resolve klass
31019
d05fcdd70109 8029567: Clean up linkResolver code
coleenp
parents: 30773
diff changeset
   577
  KlassHandle resolved_klass;
13391
30245956af37 7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents: 10565
diff changeset
   578
  if (code == Bytecodes::_invokedynamic) {
13728
882756847a04 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 13396
diff changeset
   579
    resolved_klass = SystemDictionary::MethodHandle_klass();
13391
30245956af37 7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents: 10565
diff changeset
   580
    Symbol* method_name = vmSymbols::invoke_name();
30245956af37 7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents: 10565
diff changeset
   581
    Symbol* method_signature = pool->signature_ref_at(index);
30245956af37 7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents: 10565
diff changeset
   582
    KlassHandle  current_klass(THREAD, pool->pool_holder());
31019
d05fcdd70109 8029567: Clean up linkResolver code
coleenp
parents: 30773
diff changeset
   583
    LinkInfo link_info(resolved_klass, method_name, method_signature, current_klass);
d05fcdd70109 8029567: Clean up linkResolver code
coleenp
parents: 30773
diff changeset
   584
    return resolve_method(link_info, /*require_methodref*/false, THREAD);
13391
30245956af37 7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents: 10565
diff changeset
   585
  }
30245956af37 7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents: 10565
diff changeset
   586
31019
d05fcdd70109 8029567: Clean up linkResolver code
coleenp
parents: 30773
diff changeset
   587
  LinkInfo link_info(pool, index, CHECK_NULL);
d05fcdd70109 8029567: Clean up linkResolver code
coleenp
parents: 30773
diff changeset
   588
  resolved_klass = link_info.resolved_klass();
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   589
10008
d84de97ad847 7056328: JSR 292 invocation sometimes fails in adapters for types not on boot class path
jrose
parents: 9967
diff changeset
   590
  if (pool->has_preresolution()
d84de97ad847 7056328: JSR 292 invocation sometimes fails in adapters for types not on boot class path
jrose
parents: 9967
diff changeset
   591
      || (resolved_klass() == SystemDictionary::MethodHandle_klass() &&
31019
d05fcdd70109 8029567: Clean up linkResolver code
coleenp
parents: 30773
diff changeset
   592
          MethodHandles::is_signature_polymorphic_name(resolved_klass(), link_info.name()))) {
d05fcdd70109 8029567: Clean up linkResolver code
coleenp
parents: 30773
diff changeset
   593
    Method* result = ConstantPool::method_at_if_loaded(pool, index);
d05fcdd70109 8029567: Clean up linkResolver code
coleenp
parents: 30773
diff changeset
   594
    if (result != NULL) {
d05fcdd70109 8029567: Clean up linkResolver code
coleenp
parents: 30773
diff changeset
   595
      return methodHandle(THREAD, result);
10008
d84de97ad847 7056328: JSR 292 invocation sometimes fails in adapters for types not on boot class path
jrose
parents: 9967
diff changeset
   596
    }
d84de97ad847 7056328: JSR 292 invocation sometimes fails in adapters for types not on boot class path
jrose
parents: 9967
diff changeset
   597
  }
d84de97ad847 7056328: JSR 292 invocation sometimes fails in adapters for types not on boot class path
jrose
parents: 9967
diff changeset
   598
13391
30245956af37 7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents: 10565
diff changeset
   599
  if (code == Bytecodes::_invokeinterface) {
31019
d05fcdd70109 8029567: Clean up linkResolver code
coleenp
parents: 30773
diff changeset
   600
    return resolve_interface_method(link_info, true, THREAD);
20685
2feadfa49bb7 8026299: invokespecial gets ICCE when it should get AME.
acorn
parents: 20391
diff changeset
   601
  } else if (code == Bytecodes::_invokevirtual) {
31019
d05fcdd70109 8029567: Clean up linkResolver code
coleenp
parents: 30773
diff changeset
   602
    return resolve_method(link_info, /*require_methodref*/true, THREAD);
21912
8d2924674592 8026066: ICCE for invokeinterface static
acorn
parents: 21768
diff changeset
   603
  } else if (!resolved_klass->is_interface()) {
31019
d05fcdd70109 8029567: Clean up linkResolver code
coleenp
parents: 30773
diff changeset
   604
    return resolve_method(link_info, /*require_methodref*/false, THREAD);
13391
30245956af37 7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents: 10565
diff changeset
   605
  } else {
21912
8d2924674592 8026066: ICCE for invokeinterface static
acorn
parents: 21768
diff changeset
   606
    bool nostatics = (code == Bytecodes::_invokestatic) ? false : true;
31019
d05fcdd70109 8029567: Clean up linkResolver code
coleenp
parents: 30773
diff changeset
   607
    return resolve_interface_method(link_info, nostatics, THREAD);
13391
30245956af37 7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents: 10565
diff changeset
   608
  }
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   609
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
   610
31019
d05fcdd70109 8029567: Clean up linkResolver code
coleenp
parents: 30773
diff changeset
   611
// Check and print a loader constraint violation message for method or interface method
d05fcdd70109 8029567: Clean up linkResolver code
coleenp
parents: 30773
diff changeset
   612
void LinkResolver::check_method_loader_constraints(const LinkInfo& link_info,
d05fcdd70109 8029567: Clean up linkResolver code
coleenp
parents: 30773
diff changeset
   613
                                                   const methodHandle& resolved_method,
d05fcdd70109 8029567: Clean up linkResolver code
coleenp
parents: 30773
diff changeset
   614
                                                   const char* method_type, TRAPS) {
d05fcdd70109 8029567: Clean up linkResolver code
coleenp
parents: 30773
diff changeset
   615
  Handle current_loader(THREAD, link_info.current_klass()->class_loader());
d05fcdd70109 8029567: Clean up linkResolver code
coleenp
parents: 30773
diff changeset
   616
  Handle resolved_loader(THREAD, resolved_method->method_holder()->class_loader());
d05fcdd70109 8029567: Clean up linkResolver code
coleenp
parents: 30773
diff changeset
   617
d05fcdd70109 8029567: Clean up linkResolver code
coleenp
parents: 30773
diff changeset
   618
  ResourceMark rm(THREAD);
d05fcdd70109 8029567: Clean up linkResolver code
coleenp
parents: 30773
diff changeset
   619
  Symbol* failed_type_symbol =
d05fcdd70109 8029567: Clean up linkResolver code
coleenp
parents: 30773
diff changeset
   620
    SystemDictionary::check_signature_loaders(link_info.signature(), current_loader,
d05fcdd70109 8029567: Clean up linkResolver code
coleenp
parents: 30773
diff changeset
   621
                                              resolved_loader, true, CHECK);
d05fcdd70109 8029567: Clean up linkResolver code
coleenp
parents: 30773
diff changeset
   622
  if (failed_type_symbol != NULL) {
d05fcdd70109 8029567: Clean up linkResolver code
coleenp
parents: 30773
diff changeset
   623
    const char* msg = "loader constraint violation: when resolving %s"
d05fcdd70109 8029567: Clean up linkResolver code
coleenp
parents: 30773
diff changeset
   624
      " \"%s\" the class loader (instance of %s) of the current class, %s,"
d05fcdd70109 8029567: Clean up linkResolver code
coleenp
parents: 30773
diff changeset
   625
      " and the class loader (instance of %s) for the method's defining class, %s, have"
d05fcdd70109 8029567: Clean up linkResolver code
coleenp
parents: 30773
diff changeset
   626
      " different Class objects for the type %s used in the signature";
d05fcdd70109 8029567: Clean up linkResolver code
coleenp
parents: 30773
diff changeset
   627
    char* sig = link_info.method_string();
d05fcdd70109 8029567: Clean up linkResolver code
coleenp
parents: 30773
diff changeset
   628
    const char* loader1_name = SystemDictionary::loader_name(current_loader());
d05fcdd70109 8029567: Clean up linkResolver code
coleenp
parents: 30773
diff changeset
   629
    char* current = link_info.current_klass()->name()->as_C_string();
d05fcdd70109 8029567: Clean up linkResolver code
coleenp
parents: 30773
diff changeset
   630
    const char* loader2_name = SystemDictionary::loader_name(resolved_loader());
d05fcdd70109 8029567: Clean up linkResolver code
coleenp
parents: 30773
diff changeset
   631
    char* target = resolved_method->method_holder()->name()->as_C_string();
d05fcdd70109 8029567: Clean up linkResolver code
coleenp
parents: 30773
diff changeset
   632
    char* failed_type_name = failed_type_symbol->as_C_string();
d05fcdd70109 8029567: Clean up linkResolver code
coleenp
parents: 30773
diff changeset
   633
    size_t buflen = strlen(msg) + strlen(sig) + strlen(loader1_name) +
d05fcdd70109 8029567: Clean up linkResolver code
coleenp
parents: 30773
diff changeset
   634
      strlen(current) + strlen(loader2_name) + strlen(target) +
d05fcdd70109 8029567: Clean up linkResolver code
coleenp
parents: 30773
diff changeset
   635
      strlen(failed_type_name) + strlen(method_type) + 1;
d05fcdd70109 8029567: Clean up linkResolver code
coleenp
parents: 30773
diff changeset
   636
    char* buf = NEW_RESOURCE_ARRAY_IN_THREAD(THREAD, char, buflen);
d05fcdd70109 8029567: Clean up linkResolver code
coleenp
parents: 30773
diff changeset
   637
    jio_snprintf(buf, buflen, msg, method_type, sig, loader1_name, current, loader2_name,
d05fcdd70109 8029567: Clean up linkResolver code
coleenp
parents: 30773
diff changeset
   638
                 target, failed_type_name);
d05fcdd70109 8029567: Clean up linkResolver code
coleenp
parents: 30773
diff changeset
   639
    THROW_MSG(vmSymbols::java_lang_LinkageError(), buf);
d05fcdd70109 8029567: Clean up linkResolver code
coleenp
parents: 30773
diff changeset
   640
  }
d05fcdd70109 8029567: Clean up linkResolver code
coleenp
parents: 30773
diff changeset
   641
}
d05fcdd70109 8029567: Clean up linkResolver code
coleenp
parents: 30773
diff changeset
   642
d05fcdd70109 8029567: Clean up linkResolver code
coleenp
parents: 30773
diff changeset
   643
void LinkResolver::check_field_loader_constraints(Symbol* field, Symbol* sig,
d05fcdd70109 8029567: Clean up linkResolver code
coleenp
parents: 30773
diff changeset
   644
                                                  KlassHandle current_klass,
d05fcdd70109 8029567: Clean up linkResolver code
coleenp
parents: 30773
diff changeset
   645
                                                  KlassHandle sel_klass, TRAPS) {
d05fcdd70109 8029567: Clean up linkResolver code
coleenp
parents: 30773
diff changeset
   646
  Handle ref_loader(THREAD, current_klass->class_loader());
d05fcdd70109 8029567: Clean up linkResolver code
coleenp
parents: 30773
diff changeset
   647
  Handle sel_loader(THREAD, sel_klass->class_loader());
d05fcdd70109 8029567: Clean up linkResolver code
coleenp
parents: 30773
diff changeset
   648
d05fcdd70109 8029567: Clean up linkResolver code
coleenp
parents: 30773
diff changeset
   649
  ResourceMark rm(THREAD);  // needed for check_signature_loaders
d05fcdd70109 8029567: Clean up linkResolver code
coleenp
parents: 30773
diff changeset
   650
  Symbol* failed_type_symbol =
d05fcdd70109 8029567: Clean up linkResolver code
coleenp
parents: 30773
diff changeset
   651
    SystemDictionary::check_signature_loaders(sig,
d05fcdd70109 8029567: Clean up linkResolver code
coleenp
parents: 30773
diff changeset
   652
                                              ref_loader, sel_loader,
d05fcdd70109 8029567: Clean up linkResolver code
coleenp
parents: 30773
diff changeset
   653
                                              false,
d05fcdd70109 8029567: Clean up linkResolver code
coleenp
parents: 30773
diff changeset
   654
                                              CHECK);
d05fcdd70109 8029567: Clean up linkResolver code
coleenp
parents: 30773
diff changeset
   655
  if (failed_type_symbol != NULL) {
d05fcdd70109 8029567: Clean up linkResolver code
coleenp
parents: 30773
diff changeset
   656
    const char* msg = "loader constraint violation: when resolving field"
d05fcdd70109 8029567: Clean up linkResolver code
coleenp
parents: 30773
diff changeset
   657
      " \"%s\" the class loader (instance of %s) of the referring class, "
d05fcdd70109 8029567: Clean up linkResolver code
coleenp
parents: 30773
diff changeset
   658
      "%s, and the class loader (instance of %s) for the field's resolved "
d05fcdd70109 8029567: Clean up linkResolver code
coleenp
parents: 30773
diff changeset
   659
      "type, %s, have different Class objects for that type";
d05fcdd70109 8029567: Clean up linkResolver code
coleenp
parents: 30773
diff changeset
   660
    char* field_name = field->as_C_string();
d05fcdd70109 8029567: Clean up linkResolver code
coleenp
parents: 30773
diff changeset
   661
    const char* loader1_name = SystemDictionary::loader_name(ref_loader());
d05fcdd70109 8029567: Clean up linkResolver code
coleenp
parents: 30773
diff changeset
   662
    char* sel = sel_klass->name()->as_C_string();
d05fcdd70109 8029567: Clean up linkResolver code
coleenp
parents: 30773
diff changeset
   663
    const char* loader2_name = SystemDictionary::loader_name(sel_loader());
d05fcdd70109 8029567: Clean up linkResolver code
coleenp
parents: 30773
diff changeset
   664
    char* failed_type_name = failed_type_symbol->as_C_string();
d05fcdd70109 8029567: Clean up linkResolver code
coleenp
parents: 30773
diff changeset
   665
    size_t buflen = strlen(msg) + strlen(field_name) + strlen(loader1_name) +
d05fcdd70109 8029567: Clean up linkResolver code
coleenp
parents: 30773
diff changeset
   666
                    strlen(sel) + strlen(loader2_name) + strlen(failed_type_name) + 1;
d05fcdd70109 8029567: Clean up linkResolver code
coleenp
parents: 30773
diff changeset
   667
    char* buf = NEW_RESOURCE_ARRAY_IN_THREAD(THREAD, char, buflen);
d05fcdd70109 8029567: Clean up linkResolver code
coleenp
parents: 30773
diff changeset
   668
    jio_snprintf(buf, buflen, msg, field_name, loader1_name, sel, loader2_name,
d05fcdd70109 8029567: Clean up linkResolver code
coleenp
parents: 30773
diff changeset
   669
                     failed_type_name);
d05fcdd70109 8029567: Clean up linkResolver code
coleenp
parents: 30773
diff changeset
   670
    THROW_MSG(vmSymbols::java_lang_LinkageError(), buf);
d05fcdd70109 8029567: Clean up linkResolver code
coleenp
parents: 30773
diff changeset
   671
  }
d05fcdd70109 8029567: Clean up linkResolver code
coleenp
parents: 30773
diff changeset
   672
}
d05fcdd70109 8029567: Clean up linkResolver code
coleenp
parents: 30773
diff changeset
   673
d05fcdd70109 8029567: Clean up linkResolver code
coleenp
parents: 30773
diff changeset
   674
methodHandle LinkResolver::resolve_method(const LinkInfo& link_info,
d05fcdd70109 8029567: Clean up linkResolver code
coleenp
parents: 30773
diff changeset
   675
                                          bool require_methodref, TRAPS) {
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   676
13391
30245956af37 7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents: 10565
diff changeset
   677
  Handle nested_exception;
31019
d05fcdd70109 8029567: Clean up linkResolver code
coleenp
parents: 30773
diff changeset
   678
  KlassHandle resolved_klass = link_info.resolved_klass();
13391
30245956af37 7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents: 10565
diff changeset
   679
20685
2feadfa49bb7 8026299: invokespecial gets ICCE when it should get AME.
acorn
parents: 20391
diff changeset
   680
  // 1. check if methodref required, that resolved_klass is not interfacemethodref
2feadfa49bb7 8026299: invokespecial gets ICCE when it should get AME.
acorn
parents: 20391
diff changeset
   681
  if (require_methodref && resolved_klass->is_interface()) {
2feadfa49bb7 8026299: invokespecial gets ICCE when it should get AME.
acorn
parents: 20391
diff changeset
   682
    ResourceMark rm(THREAD);
2feadfa49bb7 8026299: invokespecial gets ICCE when it should get AME.
acorn
parents: 20391
diff changeset
   683
    char buf[200];
2feadfa49bb7 8026299: invokespecial gets ICCE when it should get AME.
acorn
parents: 20391
diff changeset
   684
    jio_snprintf(buf, sizeof(buf), "Found interface %s, but class was expected",
2feadfa49bb7 8026299: invokespecial gets ICCE when it should get AME.
acorn
parents: 20391
diff changeset
   685
        resolved_klass()->external_name());
31019
d05fcdd70109 8029567: Clean up linkResolver code
coleenp
parents: 30773
diff changeset
   686
    THROW_MSG_NULL(vmSymbols::java_lang_IncompatibleClassChangeError(), buf);
20685
2feadfa49bb7 8026299: invokespecial gets ICCE when it should get AME.
acorn
parents: 20391
diff changeset
   687
  }
2feadfa49bb7 8026299: invokespecial gets ICCE when it should get AME.
acorn
parents: 20391
diff changeset
   688
2feadfa49bb7 8026299: invokespecial gets ICCE when it should get AME.
acorn
parents: 20391
diff changeset
   689
  // 2. lookup method in resolved klass and its super klasses
31019
d05fcdd70109 8029567: Clean up linkResolver code
coleenp
parents: 30773
diff changeset
   690
  methodHandle resolved_method = lookup_method_in_klasses(link_info, true, false, CHECK_NULL);
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   691
33611
9abd65805e19 8139203: Consistent naming for klass type predicates
coleenp
parents: 33602
diff changeset
   692
  if (resolved_method.is_null() && !resolved_klass->is_array_klass()) { // not found in the class hierarchy
20685
2feadfa49bb7 8026299: invokespecial gets ICCE when it should get AME.
acorn
parents: 20391
diff changeset
   693
    // 3. lookup method in all the interfaces implemented by the resolved klass
31019
d05fcdd70109 8029567: Clean up linkResolver code
coleenp
parents: 30773
diff changeset
   694
    resolved_method = lookup_method_in_interfaces(link_info, CHECK_NULL);
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   695
489c9b5090e2 Initial load
duke
parents:
diff changeset
   696
    if (resolved_method.is_null()) {
13391
30245956af37 7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents: 10565
diff changeset
   697
      // JSR 292:  see if this is an implicitly generated method MethodHandle.linkToVirtual(*...), etc
31019
d05fcdd70109 8029567: Clean up linkResolver code
coleenp
parents: 30773
diff changeset
   698
      resolved_method = lookup_polymorphic_method(link_info, (Handle*)NULL, (Handle*)NULL, THREAD);
13391
30245956af37 7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents: 10565
diff changeset
   699
      if (HAS_PENDING_EXCEPTION) {
30245956af37 7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents: 10565
diff changeset
   700
        nested_exception = Handle(THREAD, PENDING_EXCEPTION);
30245956af37 7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents: 10565
diff changeset
   701
        CLEAR_PENDING_EXCEPTION;
30245956af37 7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents: 10565
diff changeset
   702
      }
2534
08dac9ce0cd7 6655638: dynamic languages need method handles
jrose
parents: 1
diff changeset
   703
    }
27020
a7c8010446c2 8036805: Correct linker method lookup.
lfoltan
parents: 25057
diff changeset
   704
  }
2534
08dac9ce0cd7 6655638: dynamic languages need method handles
jrose
parents: 1
diff changeset
   705
27020
a7c8010446c2 8036805: Correct linker method lookup.
lfoltan
parents: 25057
diff changeset
   706
  if (resolved_method.is_null()) {
a7c8010446c2 8036805: Correct linker method lookup.
lfoltan
parents: 25057
diff changeset
   707
    // 4. method lookup failed
a7c8010446c2 8036805: Correct linker method lookup.
lfoltan
parents: 25057
diff changeset
   708
    ResourceMark rm(THREAD);
31019
d05fcdd70109 8029567: Clean up linkResolver code
coleenp
parents: 30773
diff changeset
   709
    THROW_MSG_CAUSE_(vmSymbols::java_lang_NoSuchMethodError(),
27020
a7c8010446c2 8036805: Correct linker method lookup.
lfoltan
parents: 25057
diff changeset
   710
                    Method::name_and_sig_as_C_string(resolved_klass(),
31019
d05fcdd70109 8029567: Clean up linkResolver code
coleenp
parents: 30773
diff changeset
   711
                                                     link_info.name(),
d05fcdd70109 8029567: Clean up linkResolver code
coleenp
parents: 30773
diff changeset
   712
                                                     link_info.signature()),
d05fcdd70109 8029567: Clean up linkResolver code
coleenp
parents: 30773
diff changeset
   713
                    nested_exception, NULL);
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   714
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   715
22739
74105c330330 8032010: Attempt to resolve abstract method in concrete class fails with AbstractMethodError
ccheung
parents: 22521
diff changeset
   716
  // 5. access checks, access checking may be turned off when calling from within the VM.
31019
d05fcdd70109 8029567: Clean up linkResolver code
coleenp
parents: 30773
diff changeset
   717
  KlassHandle current_klass = link_info.current_klass();
d05fcdd70109 8029567: Clean up linkResolver code
coleenp
parents: 30773
diff changeset
   718
  if (link_info.check_access()) {
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   719
    assert(current_klass.not_null() , "current_klass should not be null");
489c9b5090e2 Initial load
duke
parents:
diff changeset
   720
489c9b5090e2 Initial load
duke
parents:
diff changeset
   721
    // check if method can be accessed by the referring class
489c9b5090e2 Initial load
duke
parents:
diff changeset
   722
    check_method_accessability(current_klass,
489c9b5090e2 Initial load
duke
parents:
diff changeset
   723
                               resolved_klass,
489c9b5090e2 Initial load
duke
parents:
diff changeset
   724
                               KlassHandle(THREAD, resolved_method->method_holder()),
489c9b5090e2 Initial load
duke
parents:
diff changeset
   725
                               resolved_method,
31019
d05fcdd70109 8029567: Clean up linkResolver code
coleenp
parents: 30773
diff changeset
   726
                               CHECK_NULL);
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   727
489c9b5090e2 Initial load
duke
parents:
diff changeset
   728
    // check loader constraints
31019
d05fcdd70109 8029567: Clean up linkResolver code
coleenp
parents: 30773
diff changeset
   729
    check_method_loader_constraints(link_info, resolved_method, "method", CHECK_NULL);
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   730
  }
31019
d05fcdd70109 8029567: Clean up linkResolver code
coleenp
parents: 30773
diff changeset
   731
d05fcdd70109 8029567: Clean up linkResolver code
coleenp
parents: 30773
diff changeset
   732
  return resolved_method;
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   733
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
   734
35463
b32e362563bb 8141564: Convert TraceItables and PrintVtables to Unified Logging
rprotacio
parents: 34666
diff changeset
   735
static void trace_method_resolution(const char* prefix,
b32e362563bb 8141564: Convert TraceItables and PrintVtables to Unified Logging
rprotacio
parents: 34666
diff changeset
   736
                                    KlassHandle klass,
b32e362563bb 8141564: Convert TraceItables and PrintVtables to Unified Logging
rprotacio
parents: 34666
diff changeset
   737
                                    KlassHandle resolved_klass,
b32e362563bb 8141564: Convert TraceItables and PrintVtables to Unified Logging
rprotacio
parents: 34666
diff changeset
   738
                                    const methodHandle& method,
b32e362563bb 8141564: Convert TraceItables and PrintVtables to Unified Logging
rprotacio
parents: 34666
diff changeset
   739
                                    bool logitables,
b32e362563bb 8141564: Convert TraceItables and PrintVtables to Unified Logging
rprotacio
parents: 34666
diff changeset
   740
                                    int index = -1) {
b32e362563bb 8141564: Convert TraceItables and PrintVtables to Unified Logging
rprotacio
parents: 34666
diff changeset
   741
#ifndef PRODUCT
b32e362563bb 8141564: Convert TraceItables and PrintVtables to Unified Logging
rprotacio
parents: 34666
diff changeset
   742
  ResourceMark rm;
b32e362563bb 8141564: Convert TraceItables and PrintVtables to Unified Logging
rprotacio
parents: 34666
diff changeset
   743
  outputStream* st;
b32e362563bb 8141564: Convert TraceItables and PrintVtables to Unified Logging
rprotacio
parents: 34666
diff changeset
   744
  if (logitables) {
b32e362563bb 8141564: Convert TraceItables and PrintVtables to Unified Logging
rprotacio
parents: 34666
diff changeset
   745
    st = LogHandle(itables)::trace_stream();
b32e362563bb 8141564: Convert TraceItables and PrintVtables to Unified Logging
rprotacio
parents: 34666
diff changeset
   746
  } else {
b32e362563bb 8141564: Convert TraceItables and PrintVtables to Unified Logging
rprotacio
parents: 34666
diff changeset
   747
    st = LogHandle(vtables)::trace_stream();
b32e362563bb 8141564: Convert TraceItables and PrintVtables to Unified Logging
rprotacio
parents: 34666
diff changeset
   748
  }
b32e362563bb 8141564: Convert TraceItables and PrintVtables to Unified Logging
rprotacio
parents: 34666
diff changeset
   749
  st->print("%s%s, compile-time-class:%s, method:%s, method_holder:%s, access_flags: ",
b32e362563bb 8141564: Convert TraceItables and PrintVtables to Unified Logging
rprotacio
parents: 34666
diff changeset
   750
            prefix,
b32e362563bb 8141564: Convert TraceItables and PrintVtables to Unified Logging
rprotacio
parents: 34666
diff changeset
   751
            (klass.is_null() ? "<NULL>" : klass->internal_name()),
b32e362563bb 8141564: Convert TraceItables and PrintVtables to Unified Logging
rprotacio
parents: 34666
diff changeset
   752
            (resolved_klass.is_null() ? "<NULL>" : resolved_klass->internal_name()),
b32e362563bb 8141564: Convert TraceItables and PrintVtables to Unified Logging
rprotacio
parents: 34666
diff changeset
   753
            Method::name_and_sig_as_C_string(resolved_klass(),
b32e362563bb 8141564: Convert TraceItables and PrintVtables to Unified Logging
rprotacio
parents: 34666
diff changeset
   754
                                             method->name(),
b32e362563bb 8141564: Convert TraceItables and PrintVtables to Unified Logging
rprotacio
parents: 34666
diff changeset
   755
                                             method->signature()),
b32e362563bb 8141564: Convert TraceItables and PrintVtables to Unified Logging
rprotacio
parents: 34666
diff changeset
   756
            method->method_holder()->internal_name());
b32e362563bb 8141564: Convert TraceItables and PrintVtables to Unified Logging
rprotacio
parents: 34666
diff changeset
   757
  method->print_linkage_flags(st);
b32e362563bb 8141564: Convert TraceItables and PrintVtables to Unified Logging
rprotacio
parents: 34666
diff changeset
   758
  if (index != -1) {
b32e362563bb 8141564: Convert TraceItables and PrintVtables to Unified Logging
rprotacio
parents: 34666
diff changeset
   759
    st->print("vtable_index:%d", index);
b32e362563bb 8141564: Convert TraceItables and PrintVtables to Unified Logging
rprotacio
parents: 34666
diff changeset
   760
  }
b32e362563bb 8141564: Convert TraceItables and PrintVtables to Unified Logging
rprotacio
parents: 34666
diff changeset
   761
  st->cr();
b32e362563bb 8141564: Convert TraceItables and PrintVtables to Unified Logging
rprotacio
parents: 34666
diff changeset
   762
#endif // PRODUCT
b32e362563bb 8141564: Convert TraceItables and PrintVtables to Unified Logging
rprotacio
parents: 34666
diff changeset
   763
}
b32e362563bb 8141564: Convert TraceItables and PrintVtables to Unified Logging
rprotacio
parents: 34666
diff changeset
   764
31019
d05fcdd70109 8029567: Clean up linkResolver code
coleenp
parents: 30773
diff changeset
   765
methodHandle LinkResolver::resolve_interface_method(const LinkInfo& link_info,
d05fcdd70109 8029567: Clean up linkResolver code
coleenp
parents: 30773
diff changeset
   766
                                                    bool nostatics, TRAPS) {
d05fcdd70109 8029567: Clean up linkResolver code
coleenp
parents: 30773
diff changeset
   767
d05fcdd70109 8029567: Clean up linkResolver code
coleenp
parents: 30773
diff changeset
   768
  KlassHandle resolved_klass = link_info.resolved_klass();
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   769
22232
26acfad336c0 8027804: JCK resolveMethod test fails expecting AbstractMethodError
hseigel
parents: 22219
diff changeset
   770
  // check if klass is interface
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   771
  if (!resolved_klass->is_interface()) {
9425
6da66b869573 6728025: LinkResolver is missing some ResourceMarks
jcoomes
parents: 9116
diff changeset
   772
    ResourceMark rm(THREAD);
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   773
    char buf[200];
14488
ab48109f7d1b 8001471: Klass::cast() does nothing
hseigel
parents: 14391
diff changeset
   774
    jio_snprintf(buf, sizeof(buf), "Found class %s, but interface was expected", resolved_klass()->external_name());
31019
d05fcdd70109 8029567: Clean up linkResolver code
coleenp
parents: 30773
diff changeset
   775
    THROW_MSG_NULL(vmSymbols::java_lang_IncompatibleClassChangeError(), buf);
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   776
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   777
489c9b5090e2 Initial load
duke
parents:
diff changeset
   778
  // lookup method in this interface or its super, java.lang.Object
21912
8d2924674592 8026066: ICCE for invokeinterface static
acorn
parents: 21768
diff changeset
   779
  // JDK8: also look for static methods
31019
d05fcdd70109 8029567: Clean up linkResolver code
coleenp
parents: 30773
diff changeset
   780
  methodHandle resolved_method = lookup_method_in_klasses(link_info, false, true, CHECK_NULL);
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   781
33611
9abd65805e19 8139203: Consistent naming for klass type predicates
coleenp
parents: 33602
diff changeset
   782
  if (resolved_method.is_null() && !resolved_klass->is_array_klass()) {
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   783
    // lookup method in all the super-interfaces
31019
d05fcdd70109 8029567: Clean up linkResolver code
coleenp
parents: 30773
diff changeset
   784
    resolved_method = lookup_method_in_interfaces(link_info, CHECK_NULL);
27020
a7c8010446c2 8036805: Correct linker method lookup.
lfoltan
parents: 25057
diff changeset
   785
  }
a7c8010446c2 8036805: Correct linker method lookup.
lfoltan
parents: 25057
diff changeset
   786
a7c8010446c2 8036805: Correct linker method lookup.
lfoltan
parents: 25057
diff changeset
   787
  if (resolved_method.is_null()) {
a7c8010446c2 8036805: Correct linker method lookup.
lfoltan
parents: 25057
diff changeset
   788
    // no method found
a7c8010446c2 8036805: Correct linker method lookup.
lfoltan
parents: 25057
diff changeset
   789
    ResourceMark rm(THREAD);
31019
d05fcdd70109 8029567: Clean up linkResolver code
coleenp
parents: 30773
diff changeset
   790
    THROW_MSG_NULL(vmSymbols::java_lang_NoSuchMethodError(),
d05fcdd70109 8029567: Clean up linkResolver code
coleenp
parents: 30773
diff changeset
   791
                   Method::name_and_sig_as_C_string(resolved_klass(),
d05fcdd70109 8029567: Clean up linkResolver code
coleenp
parents: 30773
diff changeset
   792
                                                    link_info.name(),
d05fcdd70109 8029567: Clean up linkResolver code
coleenp
parents: 30773
diff changeset
   793
                                                    link_info.signature()));
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   794
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   795
31019
d05fcdd70109 8029567: Clean up linkResolver code
coleenp
parents: 30773
diff changeset
   796
  if (link_info.check_access()) {
20284
595a25ab9474 8011311: Private interface methods. Default conflicts:ICCE. no erased_super_default.
acorn
parents: 20282
diff changeset
   797
    // JDK8 adds non-public interface methods, and accessability check requirement
31019
d05fcdd70109 8029567: Clean up linkResolver code
coleenp
parents: 30773
diff changeset
   798
    KlassHandle current_klass = link_info.current_klass();
d05fcdd70109 8029567: Clean up linkResolver code
coleenp
parents: 30773
diff changeset
   799
20284
595a25ab9474 8011311: Private interface methods. Default conflicts:ICCE. no erased_super_default.
acorn
parents: 20282
diff changeset
   800
    assert(current_klass.not_null() , "current_klass should not be null");
595a25ab9474 8011311: Private interface methods. Default conflicts:ICCE. no erased_super_default.
acorn
parents: 20282
diff changeset
   801
595a25ab9474 8011311: Private interface methods. Default conflicts:ICCE. no erased_super_default.
acorn
parents: 20282
diff changeset
   802
    // check if method can be accessed by the referring class
595a25ab9474 8011311: Private interface methods. Default conflicts:ICCE. no erased_super_default.
acorn
parents: 20282
diff changeset
   803
    check_method_accessability(current_klass,
595a25ab9474 8011311: Private interface methods. Default conflicts:ICCE. no erased_super_default.
acorn
parents: 20282
diff changeset
   804
                               resolved_klass,
595a25ab9474 8011311: Private interface methods. Default conflicts:ICCE. no erased_super_default.
acorn
parents: 20282
diff changeset
   805
                               KlassHandle(THREAD, resolved_method->method_holder()),
595a25ab9474 8011311: Private interface methods. Default conflicts:ICCE. no erased_super_default.
acorn
parents: 20282
diff changeset
   806
                               resolved_method,
31019
d05fcdd70109 8029567: Clean up linkResolver code
coleenp
parents: 30773
diff changeset
   807
                               CHECK_NULL);
20284
595a25ab9474 8011311: Private interface methods. Default conflicts:ICCE. no erased_super_default.
acorn
parents: 20282
diff changeset
   808
31019
d05fcdd70109 8029567: Clean up linkResolver code
coleenp
parents: 30773
diff changeset
   809
    check_method_loader_constraints(link_info, resolved_method, "interface method", CHECK_NULL);
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   810
  }
20284
595a25ab9474 8011311: Private interface methods. Default conflicts:ICCE. no erased_super_default.
acorn
parents: 20282
diff changeset
   811
22521
5387725ea99d 8031045: Access checks should precede additional per-instruction checks
hseigel
parents: 22232
diff changeset
   812
  if (nostatics && resolved_method->is_static()) {
5387725ea99d 8031045: Access checks should precede additional per-instruction checks
hseigel
parents: 22232
diff changeset
   813
    ResourceMark rm(THREAD);
5387725ea99d 8031045: Access checks should precede additional per-instruction checks
hseigel
parents: 22232
diff changeset
   814
    char buf[200];
5387725ea99d 8031045: Access checks should precede additional per-instruction checks
hseigel
parents: 22232
diff changeset
   815
    jio_snprintf(buf, sizeof(buf), "Expected instance not static method %s",
5387725ea99d 8031045: Access checks should precede additional per-instruction checks
hseigel
parents: 22232
diff changeset
   816
                 Method::name_and_sig_as_C_string(resolved_klass(),
5387725ea99d 8031045: Access checks should precede additional per-instruction checks
hseigel
parents: 22232
diff changeset
   817
                 resolved_method->name(), resolved_method->signature()));
31019
d05fcdd70109 8029567: Clean up linkResolver code
coleenp
parents: 30773
diff changeset
   818
    THROW_MSG_NULL(vmSymbols::java_lang_IncompatibleClassChangeError(), buf);
22521
5387725ea99d 8031045: Access checks should precede additional per-instruction checks
hseigel
parents: 22232
diff changeset
   819
  }
5387725ea99d 8031045: Access checks should precede additional per-instruction checks
hseigel
parents: 22232
diff changeset
   820
35901
f5028c67e7cb 8147918: Rename develop_log_is_enabled() to log_develop_is_enabled()
pliden
parents: 35900
diff changeset
   821
  if (log_develop_is_enabled(Trace, itables)) {
31019
d05fcdd70109 8029567: Clean up linkResolver code
coleenp
parents: 30773
diff changeset
   822
    trace_method_resolution("invokeinterface resolved method: caller-class",
35463
b32e362563bb 8141564: Convert TraceItables and PrintVtables to Unified Logging
rprotacio
parents: 34666
diff changeset
   823
                            link_info.current_klass(), resolved_klass,
b32e362563bb 8141564: Convert TraceItables and PrintVtables to Unified Logging
rprotacio
parents: 34666
diff changeset
   824
                            resolved_method, true);
20284
595a25ab9474 8011311: Private interface methods. Default conflicts:ICCE. no erased_super_default.
acorn
parents: 20282
diff changeset
   825
  }
31019
d05fcdd70109 8029567: Clean up linkResolver code
coleenp
parents: 30773
diff changeset
   826
d05fcdd70109 8029567: Clean up linkResolver code
coleenp
parents: 30773
diff changeset
   827
  return resolved_method;
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   828
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
   829
489c9b5090e2 Initial load
duke
parents:
diff changeset
   830
//------------------------------------------------------------------------------------------------------------------------
489c9b5090e2 Initial load
duke
parents:
diff changeset
   831
// Field resolution
489c9b5090e2 Initial load
duke
parents:
diff changeset
   832
489c9b5090e2 Initial load
duke
parents:
diff changeset
   833
void LinkResolver::check_field_accessability(KlassHandle ref_klass,
489c9b5090e2 Initial load
duke
parents:
diff changeset
   834
                                             KlassHandle resolved_klass,
489c9b5090e2 Initial load
duke
parents:
diff changeset
   835
                                             KlassHandle sel_klass,
31019
d05fcdd70109 8029567: Clean up linkResolver code
coleenp
parents: 30773
diff changeset
   836
                                             const fieldDescriptor& fd,
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   837
                                             TRAPS) {
13728
882756847a04 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 13396
diff changeset
   838
  if (!Reflection::verify_field_access(ref_klass(),
882756847a04 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 13396
diff changeset
   839
                                       resolved_klass(),
882756847a04 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 13396
diff changeset
   840
                                       sel_klass(),
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   841
                                       fd.access_flags(),
489c9b5090e2 Initial load
duke
parents:
diff changeset
   842
                                       true)) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   843
    ResourceMark rm(THREAD);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   844
    Exceptions::fthrow(
489c9b5090e2 Initial load
duke
parents:
diff changeset
   845
      THREAD_AND_LOCATION,
8076
96d498ec7ae1 6990754: Use native memory and reference counting to implement SymbolTable
coleenp
parents: 7397
diff changeset
   846
      vmSymbols::java_lang_IllegalAccessError(),
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   847
      "tried to access field %s.%s from class %s",
489c9b5090e2 Initial load
duke
parents:
diff changeset
   848
      sel_klass->external_name(),
489c9b5090e2 Initial load
duke
parents:
diff changeset
   849
      fd.name()->as_C_string(),
489c9b5090e2 Initial load
duke
parents:
diff changeset
   850
      ref_klass->external_name()
489c9b5090e2 Initial load
duke
parents:
diff changeset
   851
    );
489c9b5090e2 Initial load
duke
parents:
diff changeset
   852
    return;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   853
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   854
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
   855
33593
60764a78fa5c 8140274: methodHandles and constantPoolHandles should be passed as const references
coleenp
parents: 33198
diff changeset
   856
void LinkResolver::resolve_field_access(fieldDescriptor& fd, const constantPoolHandle& pool, int index, Bytecodes::Code byte, TRAPS) {
31019
d05fcdd70109 8029567: Clean up linkResolver code
coleenp
parents: 30773
diff changeset
   857
  LinkInfo link_info(pool, index, CHECK);
d05fcdd70109 8029567: Clean up linkResolver code
coleenp
parents: 30773
diff changeset
   858
  resolve_field(fd, link_info, byte, true, CHECK);
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   859
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
   860
31019
d05fcdd70109 8029567: Clean up linkResolver code
coleenp
parents: 30773
diff changeset
   861
void LinkResolver::resolve_field(fieldDescriptor& fd,
d05fcdd70109 8029567: Clean up linkResolver code
coleenp
parents: 30773
diff changeset
   862
                                 const LinkInfo& link_info,
d05fcdd70109 8029567: Clean up linkResolver code
coleenp
parents: 30773
diff changeset
   863
                                 Bytecodes::Code byte, bool initialize_class,
20017
81eba62e9048 8014013: CallInfo structure no longer accurately reports the result of a LinkResolver operation
drchase
parents: 17024
diff changeset
   864
                                 TRAPS) {
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   865
  assert(byte == Bytecodes::_getstatic || byte == Bytecodes::_putstatic ||
20017
81eba62e9048 8014013: CallInfo structure no longer accurately reports the result of a LinkResolver operation
drchase
parents: 17024
diff changeset
   866
         byte == Bytecodes::_getfield  || byte == Bytecodes::_putfield  ||
30117
cce2cdac56dc 8074345: Enable RewriteBytecodes when VM runs with CDS
minqi
parents: 29081
diff changeset
   867
         byte == Bytecodes::_nofast_getfield  || byte == Bytecodes::_nofast_putfield  ||
31019
d05fcdd70109 8029567: Clean up linkResolver code
coleenp
parents: 30773
diff changeset
   868
         (byte == Bytecodes::_nop && !link_info.check_access()), "bad field access bytecode");
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   869
489c9b5090e2 Initial load
duke
parents:
diff changeset
   870
  bool is_static = (byte == Bytecodes::_getstatic || byte == Bytecodes::_putstatic);
30117
cce2cdac56dc 8074345: Enable RewriteBytecodes when VM runs with CDS
minqi
parents: 29081
diff changeset
   871
  bool is_put    = (byte == Bytecodes::_putfield  || byte == Bytecodes::_putstatic || byte == Bytecodes::_nofast_putfield);
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   872
  // Check if there's a resolved klass containing the field
31019
d05fcdd70109 8029567: Clean up linkResolver code
coleenp
parents: 30773
diff changeset
   873
  KlassHandle resolved_klass = link_info.resolved_klass();
d05fcdd70109 8029567: Clean up linkResolver code
coleenp
parents: 30773
diff changeset
   874
  Symbol* field = link_info.name();
d05fcdd70109 8029567: Clean up linkResolver code
coleenp
parents: 30773
diff changeset
   875
  Symbol* sig = link_info.signature();
d05fcdd70109 8029567: Clean up linkResolver code
coleenp
parents: 30773
diff changeset
   876
20017
81eba62e9048 8014013: CallInfo structure no longer accurately reports the result of a LinkResolver operation
drchase
parents: 17024
diff changeset
   877
  if (resolved_klass.is_null()) {
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   878
    ResourceMark rm(THREAD);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   879
    THROW_MSG(vmSymbols::java_lang_NoSuchFieldError(), field->as_C_string());
489c9b5090e2 Initial load
duke
parents:
diff changeset
   880
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   881
489c9b5090e2 Initial load
duke
parents:
diff changeset
   882
  // Resolve instance field
27020
a7c8010446c2 8036805: Correct linker method lookup.
lfoltan
parents: 25057
diff changeset
   883
  KlassHandle sel_klass(THREAD, resolved_klass->find_field(field, sig, &fd));
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   884
  // check if field exists; i.e., if a klass containing the field def has been selected
20017
81eba62e9048 8014013: CallInfo structure no longer accurately reports the result of a LinkResolver operation
drchase
parents: 17024
diff changeset
   885
  if (sel_klass.is_null()) {
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   886
    ResourceMark rm(THREAD);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   887
    THROW_MSG(vmSymbols::java_lang_NoSuchFieldError(), field->as_C_string());
489c9b5090e2 Initial load
duke
parents:
diff changeset
   888
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   889
31019
d05fcdd70109 8029567: Clean up linkResolver code
coleenp
parents: 30773
diff changeset
   890
  if (!link_info.check_access())
20017
81eba62e9048 8014013: CallInfo structure no longer accurately reports the result of a LinkResolver operation
drchase
parents: 17024
diff changeset
   891
    // Access checking may be turned off when calling from within the VM.
81eba62e9048 8014013: CallInfo structure no longer accurately reports the result of a LinkResolver operation
drchase
parents: 17024
diff changeset
   892
    return;
81eba62e9048 8014013: CallInfo structure no longer accurately reports the result of a LinkResolver operation
drchase
parents: 17024
diff changeset
   893
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   894
  // check access
31019
d05fcdd70109 8029567: Clean up linkResolver code
coleenp
parents: 30773
diff changeset
   895
  KlassHandle current_klass = link_info.current_klass();
20017
81eba62e9048 8014013: CallInfo structure no longer accurately reports the result of a LinkResolver operation
drchase
parents: 17024
diff changeset
   896
  check_field_accessability(current_klass, resolved_klass, sel_klass, fd, CHECK);
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   897
489c9b5090e2 Initial load
duke
parents:
diff changeset
   898
  // check for errors
489c9b5090e2 Initial load
duke
parents:
diff changeset
   899
  if (is_static != fd.is_static()) {
9425
6da66b869573 6728025: LinkResolver is missing some ResourceMarks
jcoomes
parents: 9116
diff changeset
   900
    ResourceMark rm(THREAD);
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   901
    char msg[200];
14488
ab48109f7d1b 8001471: Klass::cast() does nothing
hseigel
parents: 14391
diff changeset
   902
    jio_snprintf(msg, sizeof(msg), "Expected %s field %s.%s", is_static ? "static" : "non-static", resolved_klass()->external_name(), fd.name()->as_C_string());
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   903
    THROW_MSG(vmSymbols::java_lang_IncompatibleClassChangeError(), msg);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   904
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   905
489c9b5090e2 Initial load
duke
parents:
diff changeset
   906
  // Final fields can only be accessed from its own class.
20017
81eba62e9048 8014013: CallInfo structure no longer accurately reports the result of a LinkResolver operation
drchase
parents: 17024
diff changeset
   907
  if (is_put && fd.access_flags().is_final() && sel_klass() != current_klass()) {
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   908
    THROW(vmSymbols::java_lang_IllegalAccessError());
489c9b5090e2 Initial load
duke
parents:
diff changeset
   909
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   910
489c9b5090e2 Initial load
duke
parents:
diff changeset
   911
  // initialize resolved_klass if necessary
489c9b5090e2 Initial load
duke
parents:
diff changeset
   912
  // note 1: the klass which declared the field must be initialized (i.e, sel_klass)
489c9b5090e2 Initial load
duke
parents:
diff changeset
   913
  //         according to the newest JVM spec (5.5, p.170) - was bug (gri 7/28/99)
489c9b5090e2 Initial load
duke
parents:
diff changeset
   914
  //
489c9b5090e2 Initial load
duke
parents:
diff changeset
   915
  // note 2: we don't want to force initialization if we are just checking
489c9b5090e2 Initial load
duke
parents:
diff changeset
   916
  //         if the field access is legal; e.g., during compilation
20017
81eba62e9048 8014013: CallInfo structure no longer accurately reports the result of a LinkResolver operation
drchase
parents: 17024
diff changeset
   917
  if (is_static && initialize_class) {
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   918
    sel_klass->initialize(CHECK);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   919
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   920
20017
81eba62e9048 8014013: CallInfo structure no longer accurately reports the result of a LinkResolver operation
drchase
parents: 17024
diff changeset
   921
  if (sel_klass() != current_klass()) {
31019
d05fcdd70109 8029567: Clean up linkResolver code
coleenp
parents: 30773
diff changeset
   922
    check_field_loader_constraints(field, sig, current_klass, sel_klass, CHECK);
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   923
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   924
489c9b5090e2 Initial load
duke
parents:
diff changeset
   925
  // return information. note that the klass is set to the actual klass containing the
489c9b5090e2 Initial load
duke
parents:
diff changeset
   926
  // field, otherwise access of static fields in superclasses will not work.
489c9b5090e2 Initial load
duke
parents:
diff changeset
   927
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
   928
489c9b5090e2 Initial load
duke
parents:
diff changeset
   929
489c9b5090e2 Initial load
duke
parents:
diff changeset
   930
//------------------------------------------------------------------------------------------------------------------------
489c9b5090e2 Initial load
duke
parents:
diff changeset
   931
// Invoke resolution
489c9b5090e2 Initial load
duke
parents:
diff changeset
   932
//
489c9b5090e2 Initial load
duke
parents:
diff changeset
   933
// Naming conventions:
489c9b5090e2 Initial load
duke
parents:
diff changeset
   934
//
489c9b5090e2 Initial load
duke
parents:
diff changeset
   935
// resolved_method    the specified method (i.e., static receiver specified via constant pool index)
489c9b5090e2 Initial load
duke
parents:
diff changeset
   936
// sel_method         the selected method  (selected via run-time lookup; e.g., based on dynamic receiver class)
489c9b5090e2 Initial load
duke
parents:
diff changeset
   937
// resolved_klass     the specified klass  (i.e., specified via constant pool index)
489c9b5090e2 Initial load
duke
parents:
diff changeset
   938
// recv_klass         the receiver klass
489c9b5090e2 Initial load
duke
parents:
diff changeset
   939
489c9b5090e2 Initial load
duke
parents:
diff changeset
   940
31019
d05fcdd70109 8029567: Clean up linkResolver code
coleenp
parents: 30773
diff changeset
   941
void LinkResolver::resolve_static_call(CallInfo& result,
d05fcdd70109 8029567: Clean up linkResolver code
coleenp
parents: 30773
diff changeset
   942
                                       const LinkInfo& link_info,
d05fcdd70109 8029567: Clean up linkResolver code
coleenp
parents: 30773
diff changeset
   943
                                       bool initialize_class, TRAPS) {
d05fcdd70109 8029567: Clean up linkResolver code
coleenp
parents: 30773
diff changeset
   944
  methodHandle resolved_method = linktime_resolve_static_method(link_info, CHECK);
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   945
31019
d05fcdd70109 8029567: Clean up linkResolver code
coleenp
parents: 30773
diff changeset
   946
  // The resolved class can change as a result of this resolution.
d05fcdd70109 8029567: Clean up linkResolver code
coleenp
parents: 30773
diff changeset
   947
  KlassHandle resolved_klass = KlassHandle(THREAD, resolved_method->method_holder());
d05fcdd70109 8029567: Clean up linkResolver code
coleenp
parents: 30773
diff changeset
   948
d05fcdd70109 8029567: Clean up linkResolver code
coleenp
parents: 30773
diff changeset
   949
  Method* save_resolved_method = resolved_method();
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   950
  // Initialize klass (this should only happen if everything is ok)
489c9b5090e2 Initial load
duke
parents:
diff changeset
   951
  if (initialize_class && resolved_klass->should_be_initialized()) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   952
    resolved_klass->initialize(CHECK);
31019
d05fcdd70109 8029567: Clean up linkResolver code
coleenp
parents: 30773
diff changeset
   953
    // Use updated LinkInfo (to reresolve with resolved_klass as method_holder?)
d05fcdd70109 8029567: Clean up linkResolver code
coleenp
parents: 30773
diff changeset
   954
    LinkInfo new_info(resolved_klass, link_info.name(), link_info.signature(),
34238
60e87df08f1e 8143324: Backout JDK-8087223
ctornqvi
parents: 34229
diff changeset
   955
                      link_info.current_klass(), link_info.check_access());
31019
d05fcdd70109 8029567: Clean up linkResolver code
coleenp
parents: 30773
diff changeset
   956
    resolved_method = linktime_resolve_static_method(new_info, CHECK);
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   957
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   958
31019
d05fcdd70109 8029567: Clean up linkResolver code
coleenp
parents: 30773
diff changeset
   959
  assert(save_resolved_method == resolved_method(), "does this change?");
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   960
  // setup result
489c9b5090e2 Initial load
duke
parents:
diff changeset
   961
  result.set_static(resolved_klass, resolved_method, CHECK);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   962
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
   963
489c9b5090e2 Initial load
duke
parents:
diff changeset
   964
// throws linktime exceptions
31019
d05fcdd70109 8029567: Clean up linkResolver code
coleenp
parents: 30773
diff changeset
   965
methodHandle LinkResolver::linktime_resolve_static_method(const LinkInfo& link_info, TRAPS) {
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   966
31019
d05fcdd70109 8029567: Clean up linkResolver code
coleenp
parents: 30773
diff changeset
   967
  KlassHandle resolved_klass = link_info.resolved_klass();
d05fcdd70109 8029567: Clean up linkResolver code
coleenp
parents: 30773
diff changeset
   968
  methodHandle resolved_method;
21912
8d2924674592 8026066: ICCE for invokeinterface static
acorn
parents: 21768
diff changeset
   969
  if (!resolved_klass->is_interface()) {
31019
d05fcdd70109 8029567: Clean up linkResolver code
coleenp
parents: 30773
diff changeset
   970
    resolved_method = resolve_method(link_info, /*require_methodref*/false, CHECK_NULL);
21912
8d2924674592 8026066: ICCE for invokeinterface static
acorn
parents: 21768
diff changeset
   971
  } else {
31019
d05fcdd70109 8029567: Clean up linkResolver code
coleenp
parents: 30773
diff changeset
   972
    resolved_method = resolve_interface_method(link_info, /*nostatics*/false, CHECK_NULL);
21912
8d2924674592 8026066: ICCE for invokeinterface static
acorn
parents: 21768
diff changeset
   973
  }
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   974
  assert(resolved_method->name() != vmSymbols::class_initializer_name(), "should have been checked in verifier");
489c9b5090e2 Initial load
duke
parents:
diff changeset
   975
489c9b5090e2 Initial load
duke
parents:
diff changeset
   976
  // check if static
489c9b5090e2 Initial load
duke
parents:
diff changeset
   977
  if (!resolved_method->is_static()) {
9425
6da66b869573 6728025: LinkResolver is missing some ResourceMarks
jcoomes
parents: 9116
diff changeset
   978
    ResourceMark rm(THREAD);
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   979
    char buf[200];
14488
ab48109f7d1b 8001471: Klass::cast() does nothing
hseigel
parents: 14391
diff changeset
   980
    jio_snprintf(buf, sizeof(buf), "Expected static method %s", Method::name_and_sig_as_C_string(resolved_klass(),
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   981
                                                      resolved_method->name(),
489c9b5090e2 Initial load
duke
parents:
diff changeset
   982
                                                      resolved_method->signature()));
31019
d05fcdd70109 8029567: Clean up linkResolver code
coleenp
parents: 30773
diff changeset
   983
    THROW_MSG_NULL(vmSymbols::java_lang_IncompatibleClassChangeError(), buf);
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   984
  }
31019
d05fcdd70109 8029567: Clean up linkResolver code
coleenp
parents: 30773
diff changeset
   985
  return resolved_method;
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   986
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
   987
489c9b5090e2 Initial load
duke
parents:
diff changeset
   988
31019
d05fcdd70109 8029567: Clean up linkResolver code
coleenp
parents: 30773
diff changeset
   989
void LinkResolver::resolve_special_call(CallInfo& result,
d05fcdd70109 8029567: Clean up linkResolver code
coleenp
parents: 30773
diff changeset
   990
                                        const LinkInfo& link_info,
d05fcdd70109 8029567: Clean up linkResolver code
coleenp
parents: 30773
diff changeset
   991
                                        TRAPS) {
d05fcdd70109 8029567: Clean up linkResolver code
coleenp
parents: 30773
diff changeset
   992
  methodHandle resolved_method = linktime_resolve_special_method(link_info, CHECK);
d05fcdd70109 8029567: Clean up linkResolver code
coleenp
parents: 30773
diff changeset
   993
  runtime_resolve_special_method(result, resolved_method,
d05fcdd70109 8029567: Clean up linkResolver code
coleenp
parents: 30773
diff changeset
   994
                                 link_info.resolved_klass(),
d05fcdd70109 8029567: Clean up linkResolver code
coleenp
parents: 30773
diff changeset
   995
                                 link_info.current_klass(),
d05fcdd70109 8029567: Clean up linkResolver code
coleenp
parents: 30773
diff changeset
   996
                                 link_info.check_access(), CHECK);
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   997
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
   998
489c9b5090e2 Initial load
duke
parents:
diff changeset
   999
// throws linktime exceptions
31019
d05fcdd70109 8029567: Clean up linkResolver code
coleenp
parents: 30773
diff changeset
  1000
methodHandle LinkResolver::linktime_resolve_special_method(const LinkInfo& link_info,
d05fcdd70109 8029567: Clean up linkResolver code
coleenp
parents: 30773
diff changeset
  1001
                                                           TRAPS) {
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1002
20284
595a25ab9474 8011311: Private interface methods. Default conflicts:ICCE. no erased_super_default.
acorn
parents: 20282
diff changeset
  1003
  // Invokespecial is called for multiple special reasons:
595a25ab9474 8011311: Private interface methods. Default conflicts:ICCE. no erased_super_default.
acorn
parents: 20282
diff changeset
  1004
  // <init>
595a25ab9474 8011311: Private interface methods. Default conflicts:ICCE. no erased_super_default.
acorn
parents: 20282
diff changeset
  1005
  // local private method invocation, for classes and interfaces
595a25ab9474 8011311: Private interface methods. Default conflicts:ICCE. no erased_super_default.
acorn
parents: 20282
diff changeset
  1006
  // superclass.method, which can also resolve to a default method
595a25ab9474 8011311: Private interface methods. Default conflicts:ICCE. no erased_super_default.
acorn
parents: 20282
diff changeset
  1007
  // and the selected method is recalculated relative to the direct superclass
595a25ab9474 8011311: Private interface methods. Default conflicts:ICCE. no erased_super_default.
acorn
parents: 20282
diff changeset
  1008
  // superinterface.method, which explicitly does not check shadowing
31019
d05fcdd70109 8029567: Clean up linkResolver code
coleenp
parents: 30773
diff changeset
  1009
  KlassHandle resolved_klass = link_info.resolved_klass();
d05fcdd70109 8029567: Clean up linkResolver code
coleenp
parents: 30773
diff changeset
  1010
  methodHandle resolved_method;
14385
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents: 13929
diff changeset
  1011
21912
8d2924674592 8026066: ICCE for invokeinterface static
acorn
parents: 21768
diff changeset
  1012
  if (!resolved_klass->is_interface()) {
31019
d05fcdd70109 8029567: Clean up linkResolver code
coleenp
parents: 30773
diff changeset
  1013
    resolved_method = resolve_method(link_info, /*require_methodref*/false, CHECK_NULL);
21912
8d2924674592 8026066: ICCE for invokeinterface static
acorn
parents: 21768
diff changeset
  1014
  } else {
31019
d05fcdd70109 8029567: Clean up linkResolver code
coleenp
parents: 30773
diff changeset
  1015
    resolved_method = resolve_interface_method(link_info, /*nostatics*/true, CHECK_NULL);
21912
8d2924674592 8026066: ICCE for invokeinterface static
acorn
parents: 21768
diff changeset
  1016
  }
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1017
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1018
  // check if method name is <init>, that it is found in same klass as static type
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1019
  if (resolved_method->name() == vmSymbols::object_initializer_name() &&
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1020
      resolved_method->method_holder() != resolved_klass()) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1021
    ResourceMark rm(THREAD);
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1022
    Exceptions::fthrow(
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1023
      THREAD_AND_LOCATION,
8076
96d498ec7ae1 6990754: Use native memory and reference counting to implement SymbolTable
coleenp
parents: 7397
diff changeset
  1024
      vmSymbols::java_lang_NoSuchMethodError(),
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1025
      "%s: method %s%s not found",
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1026
      resolved_klass->external_name(),
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1027
      resolved_method->name()->as_C_string(),
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1028
      resolved_method->signature()->as_C_string()
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1029
    );
31019
d05fcdd70109 8029567: Clean up linkResolver code
coleenp
parents: 30773
diff changeset
  1030
    return NULL;
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1031
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1032
21768
b7dba4cde1c6 8026065: InterfaceMethodref for invokespecial must name a direct superinterface
hseigel
parents: 21556
diff changeset
  1033
  // check if invokespecial's interface method reference is in an indirect superinterface
31019
d05fcdd70109 8029567: Clean up linkResolver code
coleenp
parents: 30773
diff changeset
  1034
  KlassHandle current_klass = link_info.current_klass();
21768
b7dba4cde1c6 8026065: InterfaceMethodref for invokespecial must name a direct superinterface
hseigel
parents: 21556
diff changeset
  1035
  if (!current_klass.is_null() && resolved_klass->is_interface()) {
b7dba4cde1c6 8026065: InterfaceMethodref for invokespecial must name a direct superinterface
hseigel
parents: 21556
diff changeset
  1036
    Klass *klass_to_check = !InstanceKlass::cast(current_klass())->is_anonymous() ?
b7dba4cde1c6 8026065: InterfaceMethodref for invokespecial must name a direct superinterface
hseigel
parents: 21556
diff changeset
  1037
                                  current_klass() :
b7dba4cde1c6 8026065: InterfaceMethodref for invokespecial must name a direct superinterface
hseigel
parents: 21556
diff changeset
  1038
                                  InstanceKlass::cast(current_klass())->host_klass();
25057
f38210f84f8c 8031819: Remove legacy jdk checks and code
hseigel
parents: 24456
diff changeset
  1039
    // Disable verification for the dynamically-generated reflection bytecodes.
f38210f84f8c 8031819: Remove legacy jdk checks and code
hseigel
parents: 24456
diff changeset
  1040
    bool is_reflect = klass_to_check->is_subclass_of(
22218
71cff6a30ec1 8029415: java/lang/reflect/Method/invoke/TestPrivateInterfaceMethodReflect.java fails on all platforms with hs25-b61
hseigel
parents: 21913
diff changeset
  1041
                        SystemDictionary::reflect_MagicAccessorImpl_klass());
21768
b7dba4cde1c6 8026065: InterfaceMethodref for invokespecial must name a direct superinterface
hseigel
parents: 21556
diff changeset
  1042
22218
71cff6a30ec1 8029415: java/lang/reflect/Method/invoke/TestPrivateInterfaceMethodReflect.java fails on all platforms with hs25-b61
hseigel
parents: 21913
diff changeset
  1043
    if (!is_reflect &&
71cff6a30ec1 8029415: java/lang/reflect/Method/invoke/TestPrivateInterfaceMethodReflect.java fails on all platforms with hs25-b61
hseigel
parents: 21913
diff changeset
  1044
        !InstanceKlass::cast(klass_to_check)->is_same_or_direct_interface(resolved_klass())) {
21768
b7dba4cde1c6 8026065: InterfaceMethodref for invokespecial must name a direct superinterface
hseigel
parents: 21556
diff changeset
  1045
      ResourceMark rm(THREAD);
b7dba4cde1c6 8026065: InterfaceMethodref for invokespecial must name a direct superinterface
hseigel
parents: 21556
diff changeset
  1046
      char buf[200];
b7dba4cde1c6 8026065: InterfaceMethodref for invokespecial must name a direct superinterface
hseigel
parents: 21556
diff changeset
  1047
      jio_snprintf(buf, sizeof(buf),
b7dba4cde1c6 8026065: InterfaceMethodref for invokespecial must name a direct superinterface
hseigel
parents: 21556
diff changeset
  1048
                   "Interface method reference: %s, is in an indirect superinterface of %s",
b7dba4cde1c6 8026065: InterfaceMethodref for invokespecial must name a direct superinterface
hseigel
parents: 21556
diff changeset
  1049
                   Method::name_and_sig_as_C_string(resolved_klass(),
b7dba4cde1c6 8026065: InterfaceMethodref for invokespecial must name a direct superinterface
hseigel
parents: 21556
diff changeset
  1050
                                                         resolved_method->name(),
b7dba4cde1c6 8026065: InterfaceMethodref for invokespecial must name a direct superinterface
hseigel
parents: 21556
diff changeset
  1051
                                                         resolved_method->signature()),
b7dba4cde1c6 8026065: InterfaceMethodref for invokespecial must name a direct superinterface
hseigel
parents: 21556
diff changeset
  1052
                   current_klass->external_name());
31019
d05fcdd70109 8029567: Clean up linkResolver code
coleenp
parents: 30773
diff changeset
  1053
      THROW_MSG_NULL(vmSymbols::java_lang_IncompatibleClassChangeError(), buf);
21768
b7dba4cde1c6 8026065: InterfaceMethodref for invokespecial must name a direct superinterface
hseigel
parents: 21556
diff changeset
  1054
    }
b7dba4cde1c6 8026065: InterfaceMethodref for invokespecial must name a direct superinterface
hseigel
parents: 21556
diff changeset
  1055
  }
b7dba4cde1c6 8026065: InterfaceMethodref for invokespecial must name a direct superinterface
hseigel
parents: 21556
diff changeset
  1056
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1057
  // check if not static
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1058
  if (resolved_method->is_static()) {
9425
6da66b869573 6728025: LinkResolver is missing some ResourceMarks
jcoomes
parents: 9116
diff changeset
  1059
    ResourceMark rm(THREAD);
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1060
    char buf[200];
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1061
    jio_snprintf(buf, sizeof(buf),
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1062
                 "Expecting non-static method %s",
14488
ab48109f7d1b 8001471: Klass::cast() does nothing
hseigel
parents: 14391
diff changeset
  1063
                 Method::name_and_sig_as_C_string(resolved_klass(),
31019
d05fcdd70109 8029567: Clean up linkResolver code
coleenp
parents: 30773
diff changeset
  1064
                                                  resolved_method->name(),
d05fcdd70109 8029567: Clean up linkResolver code
coleenp
parents: 30773
diff changeset
  1065
                                                  resolved_method->signature()));
d05fcdd70109 8029567: Clean up linkResolver code
coleenp
parents: 30773
diff changeset
  1066
    THROW_MSG_NULL(vmSymbols::java_lang_IncompatibleClassChangeError(), buf);
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1067
  }
20391
7b146c5ebb18 8009130: Lambda: Fix access controls, loader constraints.
acorn
parents: 20284
diff changeset
  1068
35901
f5028c67e7cb 8147918: Rename develop_log_is_enabled() to log_develop_is_enabled()
pliden
parents: 35900
diff changeset
  1069
  if (log_develop_is_enabled(Trace, itables)) {
31019
d05fcdd70109 8029567: Clean up linkResolver code
coleenp
parents: 30773
diff changeset
  1070
    trace_method_resolution("invokespecial resolved method: caller-class:",
35463
b32e362563bb 8141564: Convert TraceItables and PrintVtables to Unified Logging
rprotacio
parents: 34666
diff changeset
  1071
                            current_klass, resolved_klass, resolved_method, true);
20284
595a25ab9474 8011311: Private interface methods. Default conflicts:ICCE. no erased_super_default.
acorn
parents: 20282
diff changeset
  1072
  }
31019
d05fcdd70109 8029567: Clean up linkResolver code
coleenp
parents: 30773
diff changeset
  1073
d05fcdd70109 8029567: Clean up linkResolver code
coleenp
parents: 30773
diff changeset
  1074
  return resolved_method;
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1075
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1076
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1077
// throws runtime exceptions
31019
d05fcdd70109 8029567: Clean up linkResolver code
coleenp
parents: 30773
diff changeset
  1078
void LinkResolver::runtime_resolve_special_method(CallInfo& result,
d05fcdd70109 8029567: Clean up linkResolver code
coleenp
parents: 30773
diff changeset
  1079
                                                  const methodHandle& resolved_method,
d05fcdd70109 8029567: Clean up linkResolver code
coleenp
parents: 30773
diff changeset
  1080
                                                  KlassHandle resolved_klass,
d05fcdd70109 8029567: Clean up linkResolver code
coleenp
parents: 30773
diff changeset
  1081
                                                  KlassHandle current_klass,
d05fcdd70109 8029567: Clean up linkResolver code
coleenp
parents: 30773
diff changeset
  1082
                                                  bool check_access, TRAPS) {
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1083
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1084
  // resolved method is selected method unless we have an old-style lookup
20284
595a25ab9474 8011311: Private interface methods. Default conflicts:ICCE. no erased_super_default.
acorn
parents: 20282
diff changeset
  1085
  // for a superclass method
595a25ab9474 8011311: Private interface methods. Default conflicts:ICCE. no erased_super_default.
acorn
parents: 20282
diff changeset
  1086
  // Invokespecial for a superinterface, resolved method is selected method,
595a25ab9474 8011311: Private interface methods. Default conflicts:ICCE. no erased_super_default.
acorn
parents: 20282
diff changeset
  1087
  // no checks for shadowing
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1088
  methodHandle sel_method(THREAD, resolved_method());
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1089
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1090
  // check if this is an old-style super call and do a new lookup if so
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1091
  { KlassHandle method_klass  = KlassHandle(THREAD,
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1092
                                            resolved_method->method_holder());
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1093
20284
595a25ab9474 8011311: Private interface methods. Default conflicts:ICCE. no erased_super_default.
acorn
parents: 20282
diff changeset
  1094
    if (check_access &&
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1095
        // a) check if ACC_SUPER flag is set for the current class
15918
3fcc894aca8c 8001307: Modify ACC_SUPER behavior
kamg
parents: 13929
diff changeset
  1096
        (current_klass->is_super() || !AllowNonVirtualCalls) &&
20284
595a25ab9474 8011311: Private interface methods. Default conflicts:ICCE. no erased_super_default.
acorn
parents: 20282
diff changeset
  1097
        // b) check if the class of the resolved_klass is a superclass
595a25ab9474 8011311: Private interface methods. Default conflicts:ICCE. no erased_super_default.
acorn
parents: 20282
diff changeset
  1098
        // (not supertype in order to exclude interface classes) of the current class.
595a25ab9474 8011311: Private interface methods. Default conflicts:ICCE. no erased_super_default.
acorn
parents: 20282
diff changeset
  1099
        // This check is not performed for super.invoke for interface methods
595a25ab9474 8011311: Private interface methods. Default conflicts:ICCE. no erased_super_default.
acorn
parents: 20282
diff changeset
  1100
        // in super interfaces.
595a25ab9474 8011311: Private interface methods. Default conflicts:ICCE. no erased_super_default.
acorn
parents: 20282
diff changeset
  1101
        current_klass->is_subclass_of(resolved_klass()) &&
595a25ab9474 8011311: Private interface methods. Default conflicts:ICCE. no erased_super_default.
acorn
parents: 20282
diff changeset
  1102
        current_klass() != resolved_klass() &&
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1103
        // c) check if the method is not <init>
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1104
        resolved_method->name() != vmSymbols::object_initializer_name()) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1105
      // Lookup super method
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1106
      KlassHandle super_klass(THREAD, current_klass->super());
31019
d05fcdd70109 8029567: Clean up linkResolver code
coleenp
parents: 30773
diff changeset
  1107
      sel_method = lookup_instance_method_in_klasses(super_klass,
8076
96d498ec7ae1 6990754: Use native memory and reference counting to implement SymbolTable
coleenp
parents: 7397
diff changeset
  1108
                           resolved_method->name(),
96d498ec7ae1 6990754: Use native memory and reference counting to implement SymbolTable
coleenp
parents: 7397
diff changeset
  1109
                           resolved_method->signature(), CHECK);
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1110
      // check if found
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1111
      if (sel_method.is_null()) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1112
        ResourceMark rm(THREAD);
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1113
        THROW_MSG(vmSymbols::java_lang_AbstractMethodError(),
14488
ab48109f7d1b 8001471: Klass::cast() does nothing
hseigel
parents: 14391
diff changeset
  1114
                  Method::name_and_sig_as_C_string(resolved_klass(),
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1115
                                            resolved_method->name(),
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1116
                                            resolved_method->signature()));
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1117
      }
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1118
    }
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1119
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1120
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1121
  // check if not static
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1122
  if (sel_method->is_static()) {
9425
6da66b869573 6728025: LinkResolver is missing some ResourceMarks
jcoomes
parents: 9116
diff changeset
  1123
    ResourceMark rm(THREAD);
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1124
    char buf[200];
14488
ab48109f7d1b 8001471: Klass::cast() does nothing
hseigel
parents: 14391
diff changeset
  1125
    jio_snprintf(buf, sizeof(buf), "Expecting non-static method %s", Method::name_and_sig_as_C_string(resolved_klass(),
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1126
                                                                                                             resolved_method->name(),
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1127
                                                                                                             resolved_method->signature()));
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1128
    THROW_MSG(vmSymbols::java_lang_IncompatibleClassChangeError(), buf);
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1129
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1130
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1131
  // check if abstract
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1132
  if (sel_method->is_abstract()) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1133
    ResourceMark rm(THREAD);
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1134
    THROW_MSG(vmSymbols::java_lang_AbstractMethodError(),
14488
ab48109f7d1b 8001471: Klass::cast() does nothing
hseigel
parents: 14391
diff changeset
  1135
              Method::name_and_sig_as_C_string(resolved_klass(),
31019
d05fcdd70109 8029567: Clean up linkResolver code
coleenp
parents: 30773
diff changeset
  1136
                                               sel_method->name(),
d05fcdd70109 8029567: Clean up linkResolver code
coleenp
parents: 30773
diff changeset
  1137
                                               sel_method->signature()));
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1138
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1139
35901
f5028c67e7cb 8147918: Rename develop_log_is_enabled() to log_develop_is_enabled()
pliden
parents: 35900
diff changeset
  1140
  if (log_develop_is_enabled(Trace, itables)) {
31019
d05fcdd70109 8029567: Clean up linkResolver code
coleenp
parents: 30773
diff changeset
  1141
    trace_method_resolution("invokespecial selected method: resolved-class:",
35463
b32e362563bb 8141564: Convert TraceItables and PrintVtables to Unified Logging
rprotacio
parents: 34666
diff changeset
  1142
                            resolved_klass, resolved_klass, sel_method, true);
20284
595a25ab9474 8011311: Private interface methods. Default conflicts:ICCE. no erased_super_default.
acorn
parents: 20282
diff changeset
  1143
  }
595a25ab9474 8011311: Private interface methods. Default conflicts:ICCE. no erased_super_default.
acorn
parents: 20282
diff changeset
  1144
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1145
  // setup result
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1146
  result.set_static(resolved_klass, sel_method, CHECK);
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1147
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1148
31019
d05fcdd70109 8029567: Clean up linkResolver code
coleenp
parents: 30773
diff changeset
  1149
void LinkResolver::resolve_virtual_call(CallInfo& result, Handle recv, KlassHandle receiver_klass,
d05fcdd70109 8029567: Clean up linkResolver code
coleenp
parents: 30773
diff changeset
  1150
                                        const LinkInfo& link_info,
d05fcdd70109 8029567: Clean up linkResolver code
coleenp
parents: 30773
diff changeset
  1151
                                        bool check_null_and_abstract, TRAPS) {
d05fcdd70109 8029567: Clean up linkResolver code
coleenp
parents: 30773
diff changeset
  1152
  methodHandle resolved_method = linktime_resolve_virtual_method(link_info, CHECK);
d05fcdd70109 8029567: Clean up linkResolver code
coleenp
parents: 30773
diff changeset
  1153
  runtime_resolve_virtual_method(result, resolved_method,
d05fcdd70109 8029567: Clean up linkResolver code
coleenp
parents: 30773
diff changeset
  1154
                                 link_info.resolved_klass(),
d05fcdd70109 8029567: Clean up linkResolver code
coleenp
parents: 30773
diff changeset
  1155
                                 recv, receiver_klass,
d05fcdd70109 8029567: Clean up linkResolver code
coleenp
parents: 30773
diff changeset
  1156
                                 check_null_and_abstract, CHECK);
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1157
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1158
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1159
// throws linktime exceptions
31019
d05fcdd70109 8029567: Clean up linkResolver code
coleenp
parents: 30773
diff changeset
  1160
methodHandle LinkResolver::linktime_resolve_virtual_method(const LinkInfo& link_info,
d05fcdd70109 8029567: Clean up linkResolver code
coleenp
parents: 30773
diff changeset
  1161
                                                           TRAPS) {
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1162
  // normal method resolution
31019
d05fcdd70109 8029567: Clean up linkResolver code
coleenp
parents: 30773
diff changeset
  1163
  methodHandle resolved_method = resolve_method(link_info, /*require_methodref*/true, CHECK_NULL);
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1164
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1165
  assert(resolved_method->name() != vmSymbols::object_initializer_name(), "should have been checked in verifier");
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1166
  assert(resolved_method->name() != vmSymbols::class_initializer_name (), "should have been checked in verifier");
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1167
20284
595a25ab9474 8011311: Private interface methods. Default conflicts:ICCE. no erased_super_default.
acorn
parents: 20282
diff changeset
  1168
  // check if private interface method
31019
d05fcdd70109 8029567: Clean up linkResolver code
coleenp
parents: 30773
diff changeset
  1169
  KlassHandle resolved_klass = link_info.resolved_klass();
d05fcdd70109 8029567: Clean up linkResolver code
coleenp
parents: 30773
diff changeset
  1170
  KlassHandle current_klass = link_info.current_klass();
d05fcdd70109 8029567: Clean up linkResolver code
coleenp
parents: 30773
diff changeset
  1171
20284
595a25ab9474 8011311: Private interface methods. Default conflicts:ICCE. no erased_super_default.
acorn
parents: 20282
diff changeset
  1172
  if (resolved_klass->is_interface() && resolved_method->is_private()) {
595a25ab9474 8011311: Private interface methods. Default conflicts:ICCE. no erased_super_default.
acorn
parents: 20282
diff changeset
  1173
    ResourceMark rm(THREAD);
595a25ab9474 8011311: Private interface methods. Default conflicts:ICCE. no erased_super_default.
acorn
parents: 20282
diff changeset
  1174
    char buf[200];
595a25ab9474 8011311: Private interface methods. Default conflicts:ICCE. no erased_super_default.
acorn
parents: 20282
diff changeset
  1175
    jio_snprintf(buf, sizeof(buf), "private interface method requires invokespecial, not invokevirtual: method %s, caller-class:%s",
595a25ab9474 8011311: Private interface methods. Default conflicts:ICCE. no erased_super_default.
acorn
parents: 20282
diff changeset
  1176
                 Method::name_and_sig_as_C_string(resolved_klass(),
595a25ab9474 8011311: Private interface methods. Default conflicts:ICCE. no erased_super_default.
acorn
parents: 20282
diff changeset
  1177
                                                  resolved_method->name(),
595a25ab9474 8011311: Private interface methods. Default conflicts:ICCE. no erased_super_default.
acorn
parents: 20282
diff changeset
  1178
                                                  resolved_method->signature()),
595a25ab9474 8011311: Private interface methods. Default conflicts:ICCE. no erased_super_default.
acorn
parents: 20282
diff changeset
  1179
                   (current_klass.is_null() ? "<NULL>" : current_klass->internal_name()));
31019
d05fcdd70109 8029567: Clean up linkResolver code
coleenp
parents: 30773
diff changeset
  1180
    THROW_MSG_NULL(vmSymbols::java_lang_IncompatibleClassChangeError(), buf);
20284
595a25ab9474 8011311: Private interface methods. Default conflicts:ICCE. no erased_super_default.
acorn
parents: 20282
diff changeset
  1181
  }
595a25ab9474 8011311: Private interface methods. Default conflicts:ICCE. no erased_super_default.
acorn
parents: 20282
diff changeset
  1182
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1183
  // check if not static
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1184
  if (resolved_method->is_static()) {
9425
6da66b869573 6728025: LinkResolver is missing some ResourceMarks
jcoomes
parents: 9116
diff changeset
  1185
    ResourceMark rm(THREAD);
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1186
    char buf[200];
14488
ab48109f7d1b 8001471: Klass::cast() does nothing
hseigel
parents: 14391
diff changeset
  1187
    jio_snprintf(buf, sizeof(buf), "Expecting non-static method %s", Method::name_and_sig_as_C_string(resolved_klass(),
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1188
                                                                                                             resolved_method->name(),
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1189
                                                                                                             resolved_method->signature()));
31019
d05fcdd70109 8029567: Clean up linkResolver code
coleenp
parents: 30773
diff changeset
  1190
    THROW_MSG_NULL(vmSymbols::java_lang_IncompatibleClassChangeError(), buf);
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1191
  }
20284
595a25ab9474 8011311: Private interface methods. Default conflicts:ICCE. no erased_super_default.
acorn
parents: 20282
diff changeset
  1192
35901
f5028c67e7cb 8147918: Rename develop_log_is_enabled() to log_develop_is_enabled()
pliden
parents: 35900
diff changeset
  1193
  if (log_develop_is_enabled(Trace, vtables)) {
31019
d05fcdd70109 8029567: Clean up linkResolver code
coleenp
parents: 30773
diff changeset
  1194
    trace_method_resolution("invokevirtual resolved method: caller-class:",
35463
b32e362563bb 8141564: Convert TraceItables and PrintVtables to Unified Logging
rprotacio
parents: 34666
diff changeset
  1195
                            current_klass, resolved_klass, resolved_method, false);
20391
7b146c5ebb18 8009130: Lambda: Fix access controls, loader constraints.
acorn
parents: 20284
diff changeset
  1196
  }
31019
d05fcdd70109 8029567: Clean up linkResolver code
coleenp
parents: 30773
diff changeset
  1197
d05fcdd70109 8029567: Clean up linkResolver code
coleenp
parents: 30773
diff changeset
  1198
  return resolved_method;
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1199
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1200
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1201
// throws runtime exceptions
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1202
void LinkResolver::runtime_resolve_virtual_method(CallInfo& result,
31019
d05fcdd70109 8029567: Clean up linkResolver code
coleenp
parents: 30773
diff changeset
  1203
                                                  const methodHandle& resolved_method,
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1204
                                                  KlassHandle resolved_klass,
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1205
                                                  Handle recv,
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1206
                                                  KlassHandle recv_klass,
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1207
                                                  bool check_null_and_abstract,
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1208
                                                  TRAPS) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1209
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1210
  // setup default return values
13728
882756847a04 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 13396
diff changeset
  1211
  int vtable_index = Method::invalid_vtable_index;
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1212
  methodHandle selected_method;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1213
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1214
  assert(recv.is_null() || recv->is_oop(), "receiver is not an oop");
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1215
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1216
  // runtime method resolution
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1217
  if (check_null_and_abstract && recv.is_null()) { // check if receiver exists
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1218
    THROW(vmSymbols::java_lang_NullPointerException());
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1219
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1220
13728
882756847a04 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 13396
diff changeset
  1221
  // Virtual methods cannot be resolved before its klass has been linked, for otherwise the Method*'s
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1222
  // has not been rewritten, and the vtable initialized. Make sure to do this after the nullcheck, since
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1223
  // a missing receiver might result in a bogus lookup.
14391
df0a1573d5bd 8000725: NPG: method_holder() and pool_holder() and pool_holder field should be InstanceKlass
coleenp
parents: 14385
diff changeset
  1224
  assert(resolved_method->method_holder()->is_linked(), "must be linked");
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1225
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1226
  // do lookup based on receiver klass using the vtable index
32189
5264b560ab1b 8087342: Crash in klassItable::initialize_itable_for_interface
acorn
parents: 31019
diff changeset
  1227
  if (resolved_method->method_holder()->is_interface()) { // default or miranda method
20391
7b146c5ebb18 8009130: Lambda: Fix access controls, loader constraints.
acorn
parents: 20284
diff changeset
  1228
    vtable_index = vtable_index_of_interface_method(resolved_klass,
21556
e75cd34a59e0 8027229: ICCE expected for >=2 maximally specific default methods.
acorn
parents: 21078
diff changeset
  1229
                           resolved_method);
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1230
    assert(vtable_index >= 0 , "we should have valid vtable index at this point");
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1231
35900
d64cf9290fc4 8148481: Devirtualize Klass::vtable
mgerdin
parents: 35495
diff changeset
  1232
    selected_method = methodHandle(THREAD, recv_klass->method_at_vtable(vtable_index));
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1233
  } else {
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1234
    // at this point we are sure that resolved_method is virtual and not
32189
5264b560ab1b 8087342: Crash in klassItable::initialize_itable_for_interface
acorn
parents: 31019
diff changeset
  1235
    // a default or miranda method; therefore, it must have a valid vtable index.
20017
81eba62e9048 8014013: CallInfo structure no longer accurately reports the result of a LinkResolver operation
drchase
parents: 17024
diff changeset
  1236
    assert(!resolved_method->has_itable_index(), "");
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1237
    vtable_index = resolved_method->vtable_index();
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1238
    // We could get a negative vtable_index for final methods,
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1239
    // because as an optimization they are they are never put in the vtable,
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1240
    // unless they override an existing method.
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1241
    // If we do get a negative, it means the resolved method is the the selected
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1242
    // method, and it can never be changed by an override.
13728
882756847a04 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 13396
diff changeset
  1243
    if (vtable_index == Method::nonvirtual_vtable_index) {
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1244
      assert(resolved_method->can_be_statically_bound(), "cannot override this method");
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1245
      selected_method = resolved_method;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1246
    } else {
35900
d64cf9290fc4 8148481: Devirtualize Klass::vtable
mgerdin
parents: 35495
diff changeset
  1247
      selected_method = methodHandle(THREAD, recv_klass->method_at_vtable(vtable_index));
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1248
    }
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1249
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1250
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1251
  // check if method exists
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1252
  if (selected_method.is_null()) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1253
    ResourceMark rm(THREAD);
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1254
    THROW_MSG(vmSymbols::java_lang_AbstractMethodError(),
14488
ab48109f7d1b 8001471: Klass::cast() does nothing
hseigel
parents: 14391
diff changeset
  1255
              Method::name_and_sig_as_C_string(resolved_klass(),
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1256
                                                      resolved_method->name(),
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1257
                                                      resolved_method->signature()));
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1258
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1259
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1260
  // check if abstract
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1261
  if (check_null_and_abstract && selected_method->is_abstract()) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1262
    ResourceMark rm(THREAD);
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1263
    THROW_MSG(vmSymbols::java_lang_AbstractMethodError(),
14488
ab48109f7d1b 8001471: Klass::cast() does nothing
hseigel
parents: 14391
diff changeset
  1264
              Method::name_and_sig_as_C_string(resolved_klass(),
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1265
                                                      selected_method->name(),
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1266
                                                      selected_method->signature()));
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1267
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1268
35901
f5028c67e7cb 8147918: Rename develop_log_is_enabled() to log_develop_is_enabled()
pliden
parents: 35900
diff changeset
  1269
  if (log_develop_is_enabled(Trace, vtables)) {
31019
d05fcdd70109 8029567: Clean up linkResolver code
coleenp
parents: 30773
diff changeset
  1270
    trace_method_resolution("invokevirtual selected method: receiver-class:",
35463
b32e362563bb 8141564: Convert TraceItables and PrintVtables to Unified Logging
rprotacio
parents: 34666
diff changeset
  1271
                            recv_klass, resolved_klass, selected_method,
b32e362563bb 8141564: Convert TraceItables and PrintVtables to Unified Logging
rprotacio
parents: 34666
diff changeset
  1272
                            false, vtable_index);
20284
595a25ab9474 8011311: Private interface methods. Default conflicts:ICCE. no erased_super_default.
acorn
parents: 20282
diff changeset
  1273
  }
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1274
  // setup result
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1275
  result.set_virtual(resolved_klass, recv_klass, resolved_method, selected_method, vtable_index, CHECK);
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1276
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1277
31019
d05fcdd70109 8029567: Clean up linkResolver code
coleenp
parents: 30773
diff changeset
  1278
void LinkResolver::resolve_interface_call(CallInfo& result, Handle recv, KlassHandle recv_klass,
d05fcdd70109 8029567: Clean up linkResolver code
coleenp
parents: 30773
diff changeset
  1279
                                          const LinkInfo& link_info,
d05fcdd70109 8029567: Clean up linkResolver code
coleenp
parents: 30773
diff changeset
  1280
                                          bool check_null_and_abstract, TRAPS) {
d05fcdd70109 8029567: Clean up linkResolver code
coleenp
parents: 30773
diff changeset
  1281
  // throws linktime exceptions
d05fcdd70109 8029567: Clean up linkResolver code
coleenp
parents: 30773
diff changeset
  1282
  methodHandle resolved_method = linktime_resolve_interface_method(link_info, CHECK);
d05fcdd70109 8029567: Clean up linkResolver code
coleenp
parents: 30773
diff changeset
  1283
  runtime_resolve_interface_method(result, resolved_method,link_info.resolved_klass(),
d05fcdd70109 8029567: Clean up linkResolver code
coleenp
parents: 30773
diff changeset
  1284
                                   recv, recv_klass, check_null_and_abstract, CHECK);
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1285
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1286
31019
d05fcdd70109 8029567: Clean up linkResolver code
coleenp
parents: 30773
diff changeset
  1287
methodHandle LinkResolver::linktime_resolve_interface_method(const LinkInfo& link_info,
d05fcdd70109 8029567: Clean up linkResolver code
coleenp
parents: 30773
diff changeset
  1288
                                                             TRAPS) {
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1289
  // normal interface method resolution
31019
d05fcdd70109 8029567: Clean up linkResolver code
coleenp
parents: 30773
diff changeset
  1290
  methodHandle resolved_method = resolve_interface_method(link_info, true, CHECK_NULL);
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1291
  assert(resolved_method->name() != vmSymbols::object_initializer_name(), "should have been checked in verifier");
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1292
  assert(resolved_method->name() != vmSymbols::class_initializer_name (), "should have been checked in verifier");
31019
d05fcdd70109 8029567: Clean up linkResolver code
coleenp
parents: 30773
diff changeset
  1293
d05fcdd70109 8029567: Clean up linkResolver code
coleenp
parents: 30773
diff changeset
  1294
  return resolved_method;
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1295
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1296
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1297
// throws runtime exceptions
31019
d05fcdd70109 8029567: Clean up linkResolver code
coleenp
parents: 30773
diff changeset
  1298
void LinkResolver::runtime_resolve_interface_method(CallInfo& result,
d05fcdd70109 8029567: Clean up linkResolver code
coleenp
parents: 30773
diff changeset
  1299
                                                    const methodHandle& resolved_method,
d05fcdd70109 8029567: Clean up linkResolver code
coleenp
parents: 30773
diff changeset
  1300
                                                    KlassHandle resolved_klass,
d05fcdd70109 8029567: Clean up linkResolver code
coleenp
parents: 30773
diff changeset
  1301
                                                    Handle recv,
d05fcdd70109 8029567: Clean up linkResolver code
coleenp
parents: 30773
diff changeset
  1302
                                                    KlassHandle recv_klass,
d05fcdd70109 8029567: Clean up linkResolver code
coleenp
parents: 30773
diff changeset
  1303
                                                    bool check_null_and_abstract, TRAPS) {
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1304
  // check if receiver exists
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1305
  if (check_null_and_abstract && recv.is_null()) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1306
    THROW(vmSymbols::java_lang_NullPointerException());
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1307
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1308
20284
595a25ab9474 8011311: Private interface methods. Default conflicts:ICCE. no erased_super_default.
acorn
parents: 20282
diff changeset
  1309
  // check if private interface method
595a25ab9474 8011311: Private interface methods. Default conflicts:ICCE. no erased_super_default.
acorn
parents: 20282
diff changeset
  1310
  if (resolved_klass->is_interface() && resolved_method->is_private()) {
595a25ab9474 8011311: Private interface methods. Default conflicts:ICCE. no erased_super_default.
acorn
parents: 20282
diff changeset
  1311
    ResourceMark rm(THREAD);
595a25ab9474 8011311: Private interface methods. Default conflicts:ICCE. no erased_super_default.
acorn
parents: 20282
diff changeset
  1312
    char buf[200];
595a25ab9474 8011311: Private interface methods. Default conflicts:ICCE. no erased_super_default.
acorn
parents: 20282
diff changeset
  1313
    jio_snprintf(buf, sizeof(buf), "private interface method requires invokespecial, not invokeinterface: method %s",
595a25ab9474 8011311: Private interface methods. Default conflicts:ICCE. no erased_super_default.
acorn
parents: 20282
diff changeset
  1314
                 Method::name_and_sig_as_C_string(resolved_klass(),
595a25ab9474 8011311: Private interface methods. Default conflicts:ICCE. no erased_super_default.
acorn
parents: 20282
diff changeset
  1315
                                                  resolved_method->name(),
595a25ab9474 8011311: Private interface methods. Default conflicts:ICCE. no erased_super_default.
acorn
parents: 20282
diff changeset
  1316
                                                  resolved_method->signature()));
595a25ab9474 8011311: Private interface methods. Default conflicts:ICCE. no erased_super_default.
acorn
parents: 20282
diff changeset
  1317
    THROW_MSG(vmSymbols::java_lang_IncompatibleClassChangeError(), buf);
595a25ab9474 8011311: Private interface methods. Default conflicts:ICCE. no erased_super_default.
acorn
parents: 20282
diff changeset
  1318
  }
595a25ab9474 8011311: Private interface methods. Default conflicts:ICCE. no erased_super_default.
acorn
parents: 20282
diff changeset
  1319
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1320
  // check if receiver klass implements the resolved interface
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1321
  if (!recv_klass->is_subtype_of(resolved_klass())) {
9425
6da66b869573 6728025: LinkResolver is missing some ResourceMarks
jcoomes
parents: 9116
diff changeset
  1322
    ResourceMark rm(THREAD);
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1323
    char buf[200];
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1324
    jio_snprintf(buf, sizeof(buf), "Class %s does not implement the requested interface %s",
14488
ab48109f7d1b 8001471: Klass::cast() does nothing
hseigel
parents: 14391
diff changeset
  1325
                 recv_klass()->external_name(),
ab48109f7d1b 8001471: Klass::cast() does nothing
hseigel
parents: 14391
diff changeset
  1326
                 resolved_klass()->external_name());
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1327
    THROW_MSG(vmSymbols::java_lang_IncompatibleClassChangeError(), buf);
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1328
  }
22232
26acfad336c0 8027804: JCK resolveMethod test fails expecting AbstractMethodError
hseigel
parents: 22219
diff changeset
  1329
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1330
  // do lookup based on receiver klass
22232
26acfad336c0 8027804: JCK resolveMethod test fails expecting AbstractMethodError
hseigel
parents: 22219
diff changeset
  1331
  // This search must match the linktime preparation search for itable initialization
26acfad336c0 8027804: JCK resolveMethod test fails expecting AbstractMethodError
hseigel
parents: 22219
diff changeset
  1332
  // to correctly enforce loader constraints for interface method inheritance
31019
d05fcdd70109 8029567: Clean up linkResolver code
coleenp
parents: 30773
diff changeset
  1333
  methodHandle sel_method = lookup_instance_method_in_klasses(recv_klass,
d05fcdd70109 8029567: Clean up linkResolver code
coleenp
parents: 30773
diff changeset
  1334
                                                  resolved_method->name(),
d05fcdd70109 8029567: Clean up linkResolver code
coleenp
parents: 30773
diff changeset
  1335
                                                  resolved_method->signature(), CHECK);
20017
81eba62e9048 8014013: CallInfo structure no longer accurately reports the result of a LinkResolver operation
drchase
parents: 17024
diff changeset
  1336
  if (sel_method.is_null() && !check_null_and_abstract) {
81eba62e9048 8014013: CallInfo structure no longer accurately reports the result of a LinkResolver operation
drchase
parents: 17024
diff changeset
  1337
    // In theory this is a harmless placeholder value, but
81eba62e9048 8014013: CallInfo structure no longer accurately reports the result of a LinkResolver operation
drchase
parents: 17024
diff changeset
  1338
    // in practice leaving in null affects the nsk default method tests.
81eba62e9048 8014013: CallInfo structure no longer accurately reports the result of a LinkResolver operation
drchase
parents: 17024
diff changeset
  1339
    // This needs further study.
81eba62e9048 8014013: CallInfo structure no longer accurately reports the result of a LinkResolver operation
drchase
parents: 17024
diff changeset
  1340
    sel_method = resolved_method;
81eba62e9048 8014013: CallInfo structure no longer accurately reports the result of a LinkResolver operation
drchase
parents: 17024
diff changeset
  1341
  }
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1342
  // check if method exists
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1343
  if (sel_method.is_null()) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1344
    ResourceMark rm(THREAD);
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1345
    THROW_MSG(vmSymbols::java_lang_AbstractMethodError(),
31019
d05fcdd70109 8029567: Clean up linkResolver code
coleenp
parents: 30773
diff changeset
  1346
                   Method::name_and_sig_as_C_string(recv_klass(),
d05fcdd70109 8029567: Clean up linkResolver code
coleenp
parents: 30773
diff changeset
  1347
                                                    resolved_method->name(),
d05fcdd70109 8029567: Clean up linkResolver code
coleenp
parents: 30773
diff changeset
  1348
                                                    resolved_method->signature()));
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1349
  }
17024
49cbeca23983 8006267: InterfaceMethod_ref should allow invokestatic and invokespecial
bharadwaj
parents: 16691
diff changeset
  1350
  // check access
20284
595a25ab9474 8011311: Private interface methods. Default conflicts:ICCE. no erased_super_default.
acorn
parents: 20282
diff changeset
  1351
  // Throw Illegal Access Error if sel_method is not public.
595a25ab9474 8011311: Private interface methods. Default conflicts:ICCE. no erased_super_default.
acorn
parents: 20282
diff changeset
  1352
  if (!sel_method->is_public()) {
595a25ab9474 8011311: Private interface methods. Default conflicts:ICCE. no erased_super_default.
acorn
parents: 20282
diff changeset
  1353
    ResourceMark rm(THREAD);
595a25ab9474 8011311: Private interface methods. Default conflicts:ICCE. no erased_super_default.
acorn
parents: 20282
diff changeset
  1354
    THROW_MSG(vmSymbols::java_lang_IllegalAccessError(),
595a25ab9474 8011311: Private interface methods. Default conflicts:ICCE. no erased_super_default.
acorn
parents: 20282
diff changeset
  1355
              Method::name_and_sig_as_C_string(recv_klass(),
595a25ab9474 8011311: Private interface methods. Default conflicts:ICCE. no erased_super_default.
acorn
parents: 20282
diff changeset
  1356
                                               sel_method->name(),
595a25ab9474 8011311: Private interface methods. Default conflicts:ICCE. no erased_super_default.
acorn
parents: 20282
diff changeset
  1357
                                               sel_method->signature()));
17024
49cbeca23983 8006267: InterfaceMethod_ref should allow invokestatic and invokespecial
bharadwaj
parents: 16691
diff changeset
  1358
  }
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1359
  // check if abstract
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1360
  if (check_null_and_abstract && sel_method->is_abstract()) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1361
    ResourceMark rm(THREAD);
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1362
    THROW_MSG(vmSymbols::java_lang_AbstractMethodError(),
14488
ab48109f7d1b 8001471: Klass::cast() does nothing
hseigel
parents: 14391
diff changeset
  1363
              Method::name_and_sig_as_C_string(recv_klass(),
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1364
                                                      sel_method->name(),
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1365
                                                      sel_method->signature()));
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1366
  }
20284
595a25ab9474 8011311: Private interface methods. Default conflicts:ICCE. no erased_super_default.
acorn
parents: 20282
diff changeset
  1367
35901
f5028c67e7cb 8147918: Rename develop_log_is_enabled() to log_develop_is_enabled()
pliden
parents: 35900
diff changeset
  1368
  if (log_develop_is_enabled(Trace, itables)) {
31019
d05fcdd70109 8029567: Clean up linkResolver code
coleenp
parents: 30773
diff changeset
  1369
    trace_method_resolution("invokeinterface selected method: receiver-class",
35463
b32e362563bb 8141564: Convert TraceItables and PrintVtables to Unified Logging
rprotacio
parents: 34666
diff changeset
  1370
                            recv_klass, resolved_klass, sel_method, true);
20284
595a25ab9474 8011311: Private interface methods. Default conflicts:ICCE. no erased_super_default.
acorn
parents: 20282
diff changeset
  1371
  }
21556
e75cd34a59e0 8027229: ICCE expected for >=2 maximally specific default methods.
acorn
parents: 21078
diff changeset
  1372
  // setup result
e75cd34a59e0 8027229: ICCE expected for >=2 maximally specific default methods.
acorn
parents: 21078
diff changeset
  1373
  if (!resolved_method->has_itable_index()) {
e75cd34a59e0 8027229: ICCE expected for >=2 maximally specific default methods.
acorn
parents: 21078
diff changeset
  1374
    int vtable_index = resolved_method->vtable_index();
e75cd34a59e0 8027229: ICCE expected for >=2 maximally specific default methods.
acorn
parents: 21078
diff changeset
  1375
    assert(vtable_index == sel_method->vtable_index(), "sanity check");
e75cd34a59e0 8027229: ICCE expected for >=2 maximally specific default methods.
acorn
parents: 21078
diff changeset
  1376
    result.set_virtual(resolved_klass, recv_klass, resolved_method, sel_method, vtable_index, CHECK);
e75cd34a59e0 8027229: ICCE expected for >=2 maximally specific default methods.
acorn
parents: 21078
diff changeset
  1377
  } else {
e75cd34a59e0 8027229: ICCE expected for >=2 maximally specific default methods.
acorn
parents: 21078
diff changeset
  1378
    int itable_index = resolved_method()->itable_index();
e75cd34a59e0 8027229: ICCE expected for >=2 maximally specific default methods.
acorn
parents: 21078
diff changeset
  1379
    result.set_interface(resolved_klass, recv_klass, resolved_method, sel_method, itable_index, CHECK);
e75cd34a59e0 8027229: ICCE expected for >=2 maximally specific default methods.
acorn
parents: 21078
diff changeset
  1380
  }
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1381
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1382
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1383
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1384
methodHandle LinkResolver::linktime_resolve_interface_method_or_null(
31019
d05fcdd70109 8029567: Clean up linkResolver code
coleenp
parents: 30773
diff changeset
  1385
                                                 const LinkInfo& link_info) {
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1386
  EXCEPTION_MARK;
31019
d05fcdd70109 8029567: Clean up linkResolver code
coleenp
parents: 30773
diff changeset
  1387
  methodHandle method_result = linktime_resolve_interface_method(link_info, THREAD);
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1388
  if (HAS_PENDING_EXCEPTION) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1389
    CLEAR_PENDING_EXCEPTION;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1390
    return methodHandle();
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1391
  } else {
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1392
    return method_result;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1393
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1394
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1395
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1396
methodHandle LinkResolver::linktime_resolve_virtual_method_or_null(
31019
d05fcdd70109 8029567: Clean up linkResolver code
coleenp
parents: 30773
diff changeset
  1397
                                                 const LinkInfo& link_info) {
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1398
  EXCEPTION_MARK;
31019
d05fcdd70109 8029567: Clean up linkResolver code
coleenp
parents: 30773
diff changeset
  1399
  methodHandle method_result = linktime_resolve_virtual_method(link_info, THREAD);
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1400
  if (HAS_PENDING_EXCEPTION) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1401
    CLEAR_PENDING_EXCEPTION;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1402
    return methodHandle();
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1403
  } else {
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1404
    return method_result;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1405
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1406
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1407
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1408
methodHandle LinkResolver::resolve_virtual_call_or_null(
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1409
                                                 KlassHandle receiver_klass,
31019
d05fcdd70109 8029567: Clean up linkResolver code
coleenp
parents: 30773
diff changeset
  1410
                                                 const LinkInfo& link_info) {
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1411
  EXCEPTION_MARK;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1412
  CallInfo info;
31019
d05fcdd70109 8029567: Clean up linkResolver code
coleenp
parents: 30773
diff changeset
  1413
  resolve_virtual_call(info, Handle(), receiver_klass, link_info, false, THREAD);
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1414
  if (HAS_PENDING_EXCEPTION) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1415
    CLEAR_PENDING_EXCEPTION;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1416
    return methodHandle();
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1417
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1418
  return info.selected_method();
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1419
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1420
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1421
methodHandle LinkResolver::resolve_interface_call_or_null(
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1422
                                                 KlassHandle receiver_klass,
31019
d05fcdd70109 8029567: Clean up linkResolver code
coleenp
parents: 30773
diff changeset
  1423
                                                 const LinkInfo& link_info) {
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1424
  EXCEPTION_MARK;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1425
  CallInfo info;
31019
d05fcdd70109 8029567: Clean up linkResolver code
coleenp
parents: 30773
diff changeset
  1426
  resolve_interface_call(info, Handle(), receiver_klass, link_info, false, THREAD);
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1427
  if (HAS_PENDING_EXCEPTION) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1428
    CLEAR_PENDING_EXCEPTION;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1429
    return methodHandle();
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1430
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1431
  return info.selected_method();
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1432
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1433
31019
d05fcdd70109 8029567: Clean up linkResolver code
coleenp
parents: 30773
diff changeset
  1434
int LinkResolver::resolve_virtual_vtable_index(KlassHandle receiver_klass,
d05fcdd70109 8029567: Clean up linkResolver code
coleenp
parents: 30773
diff changeset
  1435
                                               const LinkInfo& link_info) {
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1436
  EXCEPTION_MARK;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1437
  CallInfo info;
31019
d05fcdd70109 8029567: Clean up linkResolver code
coleenp
parents: 30773
diff changeset
  1438
  resolve_virtual_call(info, Handle(), receiver_klass, link_info,
d05fcdd70109 8029567: Clean up linkResolver code
coleenp
parents: 30773
diff changeset
  1439
                       /*check_null_or_abstract*/false, THREAD);
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1440
  if (HAS_PENDING_EXCEPTION) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1441
    CLEAR_PENDING_EXCEPTION;
13728
882756847a04 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 13396
diff changeset
  1442
    return Method::invalid_vtable_index;
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1443
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1444
  return info.vtable_index();
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1445
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1446
31019
d05fcdd70109 8029567: Clean up linkResolver code
coleenp
parents: 30773
diff changeset
  1447
methodHandle LinkResolver::resolve_static_call_or_null(const LinkInfo& link_info) {
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1448
  EXCEPTION_MARK;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1449
  CallInfo info;
31019
d05fcdd70109 8029567: Clean up linkResolver code
coleenp
parents: 30773
diff changeset
  1450
  resolve_static_call(info, link_info, /*initialize_class*/false, THREAD);
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1451
  if (HAS_PENDING_EXCEPTION) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1452
    CLEAR_PENDING_EXCEPTION;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1453
    return methodHandle();
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1454
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1455
  return info.selected_method();
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1456
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1457
31019
d05fcdd70109 8029567: Clean up linkResolver code
coleenp
parents: 30773
diff changeset
  1458
methodHandle LinkResolver::resolve_special_call_or_null(const LinkInfo& link_info) {
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1459
  EXCEPTION_MARK;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1460
  CallInfo info;
31019
d05fcdd70109 8029567: Clean up linkResolver code
coleenp
parents: 30773
diff changeset
  1461
  resolve_special_call(info, link_info, THREAD);
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1462
  if (HAS_PENDING_EXCEPTION) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1463
    CLEAR_PENDING_EXCEPTION;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1464
    return methodHandle();
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1465
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1466
  return info.selected_method();
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1467
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1468
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1469
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1470
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1471
//------------------------------------------------------------------------------------------------------------------------
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1472
// ConstantPool entries
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1473
33593
60764a78fa5c 8140274: methodHandles and constantPoolHandles should be passed as const references
coleenp
parents: 33198
diff changeset
  1474
void LinkResolver::resolve_invoke(CallInfo& result, Handle recv, const constantPoolHandle& pool, int index, Bytecodes::Code byte, TRAPS) {
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1475
  switch (byte) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1476
    case Bytecodes::_invokestatic   : resolve_invokestatic   (result,       pool, index, CHECK); break;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1477
    case Bytecodes::_invokespecial  : resolve_invokespecial  (result,       pool, index, CHECK); break;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1478
    case Bytecodes::_invokevirtual  : resolve_invokevirtual  (result, recv, pool, index, CHECK); break;
13391
30245956af37 7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents: 10565
diff changeset
  1479
    case Bytecodes::_invokehandle   : resolve_invokehandle   (result,       pool, index, CHECK); break;
2570
ecc7862946d4 6655646: dynamic languages need dynamically linked call sites
jrose
parents: 2534
diff changeset
  1480
    case Bytecodes::_invokedynamic  : resolve_invokedynamic  (result,       pool, index, CHECK); break;
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1481
    case Bytecodes::_invokeinterface: resolve_invokeinterface(result, recv, pool, index, CHECK); break;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1482
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1483
  return;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1484
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1485
35086
bbf32241d851 8072008: Emit direct call instead of linkTo* for recursive indy/MH.invoke* calls
vlivanov
parents: 34238
diff changeset
  1486
void LinkResolver::resolve_invoke(CallInfo& result, Handle& recv,
bbf32241d851 8072008: Emit direct call instead of linkTo* for recursive indy/MH.invoke* calls
vlivanov
parents: 34238
diff changeset
  1487
                             const methodHandle& attached_method,
bbf32241d851 8072008: Emit direct call instead of linkTo* for recursive indy/MH.invoke* calls
vlivanov
parents: 34238
diff changeset
  1488
                             Bytecodes::Code byte, TRAPS) {
bbf32241d851 8072008: Emit direct call instead of linkTo* for recursive indy/MH.invoke* calls
vlivanov
parents: 34238
diff changeset
  1489
  KlassHandle defc = attached_method->method_holder();
bbf32241d851 8072008: Emit direct call instead of linkTo* for recursive indy/MH.invoke* calls
vlivanov
parents: 34238
diff changeset
  1490
  Symbol* name = attached_method->name();
bbf32241d851 8072008: Emit direct call instead of linkTo* for recursive indy/MH.invoke* calls
vlivanov
parents: 34238
diff changeset
  1491
  Symbol* type = attached_method->signature();
bbf32241d851 8072008: Emit direct call instead of linkTo* for recursive indy/MH.invoke* calls
vlivanov
parents: 34238
diff changeset
  1492
  LinkInfo link_info(defc, name, type, KlassHandle(), /*check_access=*/false);
bbf32241d851 8072008: Emit direct call instead of linkTo* for recursive indy/MH.invoke* calls
vlivanov
parents: 34238
diff changeset
  1493
  switch(byte) {
bbf32241d851 8072008: Emit direct call instead of linkTo* for recursive indy/MH.invoke* calls
vlivanov
parents: 34238
diff changeset
  1494
    case Bytecodes::_invokevirtual:
bbf32241d851 8072008: Emit direct call instead of linkTo* for recursive indy/MH.invoke* calls
vlivanov
parents: 34238
diff changeset
  1495
      resolve_virtual_call(result, recv, recv->klass(), link_info,
bbf32241d851 8072008: Emit direct call instead of linkTo* for recursive indy/MH.invoke* calls
vlivanov
parents: 34238
diff changeset
  1496
                           /*check_null_and_abstract=*/true, CHECK);
bbf32241d851 8072008: Emit direct call instead of linkTo* for recursive indy/MH.invoke* calls
vlivanov
parents: 34238
diff changeset
  1497
      break;
bbf32241d851 8072008: Emit direct call instead of linkTo* for recursive indy/MH.invoke* calls
vlivanov
parents: 34238
diff changeset
  1498
    case Bytecodes::_invokeinterface:
bbf32241d851 8072008: Emit direct call instead of linkTo* for recursive indy/MH.invoke* calls
vlivanov
parents: 34238
diff changeset
  1499
      resolve_interface_call(result, recv, recv->klass(), link_info,
bbf32241d851 8072008: Emit direct call instead of linkTo* for recursive indy/MH.invoke* calls
vlivanov
parents: 34238
diff changeset
  1500
                             /*check_null_and_abstract=*/true, CHECK);
bbf32241d851 8072008: Emit direct call instead of linkTo* for recursive indy/MH.invoke* calls
vlivanov
parents: 34238
diff changeset
  1501
      break;
bbf32241d851 8072008: Emit direct call instead of linkTo* for recursive indy/MH.invoke* calls
vlivanov
parents: 34238
diff changeset
  1502
    case Bytecodes::_invokestatic:
bbf32241d851 8072008: Emit direct call instead of linkTo* for recursive indy/MH.invoke* calls
vlivanov
parents: 34238
diff changeset
  1503
      resolve_static_call(result, link_info, /*initialize_class=*/false, CHECK);
bbf32241d851 8072008: Emit direct call instead of linkTo* for recursive indy/MH.invoke* calls
vlivanov
parents: 34238
diff changeset
  1504
      break;
bbf32241d851 8072008: Emit direct call instead of linkTo* for recursive indy/MH.invoke* calls
vlivanov
parents: 34238
diff changeset
  1505
    case Bytecodes::_invokespecial:
bbf32241d851 8072008: Emit direct call instead of linkTo* for recursive indy/MH.invoke* calls
vlivanov
parents: 34238
diff changeset
  1506
      resolve_special_call(result, link_info, CHECK);
bbf32241d851 8072008: Emit direct call instead of linkTo* for recursive indy/MH.invoke* calls
vlivanov
parents: 34238
diff changeset
  1507
      break;
bbf32241d851 8072008: Emit direct call instead of linkTo* for recursive indy/MH.invoke* calls
vlivanov
parents: 34238
diff changeset
  1508
    default:
bbf32241d851 8072008: Emit direct call instead of linkTo* for recursive indy/MH.invoke* calls
vlivanov
parents: 34238
diff changeset
  1509
      fatal("bad call: %s", Bytecodes::name(byte));
bbf32241d851 8072008: Emit direct call instead of linkTo* for recursive indy/MH.invoke* calls
vlivanov
parents: 34238
diff changeset
  1510
  }
bbf32241d851 8072008: Emit direct call instead of linkTo* for recursive indy/MH.invoke* calls
vlivanov
parents: 34238
diff changeset
  1511
}
bbf32241d851 8072008: Emit direct call instead of linkTo* for recursive indy/MH.invoke* calls
vlivanov
parents: 34238
diff changeset
  1512
33593
60764a78fa5c 8140274: methodHandles and constantPoolHandles should be passed as const references
coleenp
parents: 33198
diff changeset
  1513
void LinkResolver::resolve_invokestatic(CallInfo& result, const constantPoolHandle& pool, int index, TRAPS) {
31019
d05fcdd70109 8029567: Clean up linkResolver code
coleenp
parents: 30773
diff changeset
  1514
  LinkInfo link_info(pool, index, CHECK);
d05fcdd70109 8029567: Clean up linkResolver code
coleenp
parents: 30773
diff changeset
  1515
  resolve_static_call(result, link_info, /*initialize_class*/true, CHECK);
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1516
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1517
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1518
33593
60764a78fa5c 8140274: methodHandles and constantPoolHandles should be passed as const references
coleenp
parents: 33198
diff changeset
  1519
void LinkResolver::resolve_invokespecial(CallInfo& result, const constantPoolHandle& pool, int index, TRAPS) {
31019
d05fcdd70109 8029567: Clean up linkResolver code
coleenp
parents: 30773
diff changeset
  1520
  LinkInfo link_info(pool, index, CHECK);
d05fcdd70109 8029567: Clean up linkResolver code
coleenp
parents: 30773
diff changeset
  1521
  resolve_special_call(result, link_info, CHECK);
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1522
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1523
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1524
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1525
void LinkResolver::resolve_invokevirtual(CallInfo& result, Handle recv,
33593
60764a78fa5c 8140274: methodHandles and constantPoolHandles should be passed as const references
coleenp
parents: 33198
diff changeset
  1526
                                          const constantPoolHandle& pool, int index,
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1527
                                          TRAPS) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1528
31019
d05fcdd70109 8029567: Clean up linkResolver code
coleenp
parents: 30773
diff changeset
  1529
  LinkInfo link_info(pool, index, CHECK);
13728
882756847a04 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 13396
diff changeset
  1530
  KlassHandle recvrKlass (THREAD, recv.is_null() ? (Klass*)NULL : recv->klass());
31019
d05fcdd70109 8029567: Clean up linkResolver code
coleenp
parents: 30773
diff changeset
  1531
  resolve_virtual_call(result, recv, recvrKlass, link_info, /*check_null_or_abstract*/true, CHECK);
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1532
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1533
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1534
33593
60764a78fa5c 8140274: methodHandles and constantPoolHandles should be passed as const references
coleenp
parents: 33198
diff changeset
  1535
void LinkResolver::resolve_invokeinterface(CallInfo& result, Handle recv, const constantPoolHandle& pool, int index, TRAPS) {
31019
d05fcdd70109 8029567: Clean up linkResolver code
coleenp
parents: 30773
diff changeset
  1536
  LinkInfo link_info(pool, index, CHECK);
13728
882756847a04 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 13396
diff changeset
  1537
  KlassHandle recvrKlass (THREAD, recv.is_null() ? (Klass*)NULL : recv->klass());
31019
d05fcdd70109 8029567: Clean up linkResolver code
coleenp
parents: 30773
diff changeset
  1538
  resolve_interface_call(result, recv, recvrKlass, link_info, true, CHECK);
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1539
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1540
2570
ecc7862946d4 6655646: dynamic languages need dynamically linked call sites
jrose
parents: 2534
diff changeset
  1541
33593
60764a78fa5c 8140274: methodHandles and constantPoolHandles should be passed as const references
coleenp
parents: 33198
diff changeset
  1542
void LinkResolver::resolve_invokehandle(CallInfo& result, const constantPoolHandle& pool, int index, TRAPS) {
13391
30245956af37 7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents: 10565
diff changeset
  1543
  // This guy is reached from InterpreterRuntime::resolve_invokehandle.
31019
d05fcdd70109 8029567: Clean up linkResolver code
coleenp
parents: 30773
diff changeset
  1544
  LinkInfo link_info(pool, index, CHECK);
16691
c117eee6d38a 8011952: Missing ResourceMarks in TraceMethodHandles
kmo
parents: 16591
diff changeset
  1545
  if (TraceMethodHandles) {
c117eee6d38a 8011952: Missing ResourceMarks in TraceMethodHandles
kmo
parents: 16591
diff changeset
  1546
    ResourceMark rm(THREAD);
31019
d05fcdd70109 8029567: Clean up linkResolver code
coleenp
parents: 30773
diff changeset
  1547
    tty->print_cr("resolve_invokehandle %s %s", link_info.name()->as_C_string(),
d05fcdd70109 8029567: Clean up linkResolver code
coleenp
parents: 30773
diff changeset
  1548
                  link_info.signature()->as_C_string());
16691
c117eee6d38a 8011952: Missing ResourceMarks in TraceMethodHandles
kmo
parents: 16591
diff changeset
  1549
  }
31019
d05fcdd70109 8029567: Clean up linkResolver code
coleenp
parents: 30773
diff changeset
  1550
  resolve_handle_call(result, link_info, CHECK);
13391
30245956af37 7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents: 10565
diff changeset
  1551
}
2570
ecc7862946d4 6655646: dynamic languages need dynamically linked call sites
jrose
parents: 2534
diff changeset
  1552
31019
d05fcdd70109 8029567: Clean up linkResolver code
coleenp
parents: 30773
diff changeset
  1553
void LinkResolver::resolve_handle_call(CallInfo& result,
d05fcdd70109 8029567: Clean up linkResolver code
coleenp
parents: 30773
diff changeset
  1554
                                       const LinkInfo& link_info,
13391
30245956af37 7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents: 10565
diff changeset
  1555
                                       TRAPS) {
30245956af37 7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents: 10565
diff changeset
  1556
  // JSR 292:  this must be an implicitly generated method MethodHandle.invokeExact(*...) or similar
31019
d05fcdd70109 8029567: Clean up linkResolver code
coleenp
parents: 30773
diff changeset
  1557
  assert(link_info.resolved_klass()() == SystemDictionary::MethodHandle_klass(), "");
d05fcdd70109 8029567: Clean up linkResolver code
coleenp
parents: 30773
diff changeset
  1558
  assert(MethodHandles::is_signature_polymorphic_name(link_info.name()), "");
13929
8da0dc50a6e4 7200949: JSR 292: rubybench/bench/time/bench_base64.rb fails with jruby.jar not on boot class path
twisti
parents: 13732
diff changeset
  1559
  Handle       resolved_appendix;
8da0dc50a6e4 7200949: JSR 292: rubybench/bench/time/bench_base64.rb fails with jruby.jar not on boot class path
twisti
parents: 13732
diff changeset
  1560
  Handle       resolved_method_type;
31019
d05fcdd70109 8029567: Clean up linkResolver code
coleenp
parents: 30773
diff changeset
  1561
  methodHandle resolved_method = lookup_polymorphic_method(link_info,
d05fcdd70109 8029567: Clean up linkResolver code
coleenp
parents: 30773
diff changeset
  1562
                                       &resolved_appendix, &resolved_method_type, CHECK);
13929
8da0dc50a6e4 7200949: JSR 292: rubybench/bench/time/bench_base64.rb fails with jruby.jar not on boot class path
twisti
parents: 13732
diff changeset
  1563
  result.set_handle(resolved_method, resolved_appendix, resolved_method_type, CHECK);
13391
30245956af37 7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents: 10565
diff changeset
  1564
}
30245956af37 7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents: 10565
diff changeset
  1565
30746
dfce1db72058 8051045: HotSpot fails to wrap Exceptions from invokedynamic in a BootstrapMethodError
aeriksso
parents: 30227
diff changeset
  1566
static void wrap_invokedynamic_exception(TRAPS) {
dfce1db72058 8051045: HotSpot fails to wrap Exceptions from invokedynamic in a BootstrapMethodError
aeriksso
parents: 30227
diff changeset
  1567
  if (HAS_PENDING_EXCEPTION) {
dfce1db72058 8051045: HotSpot fails to wrap Exceptions from invokedynamic in a BootstrapMethodError
aeriksso
parents: 30227
diff changeset
  1568
    if (TraceMethodHandles) {
dfce1db72058 8051045: HotSpot fails to wrap Exceptions from invokedynamic in a BootstrapMethodError
aeriksso
parents: 30227
diff changeset
  1569
      tty->print_cr("invokedynamic throws BSME for " INTPTR_FORMAT, p2i((void *)PENDING_EXCEPTION));
dfce1db72058 8051045: HotSpot fails to wrap Exceptions from invokedynamic in a BootstrapMethodError
aeriksso
parents: 30227
diff changeset
  1570
      PENDING_EXCEPTION->print();
dfce1db72058 8051045: HotSpot fails to wrap Exceptions from invokedynamic in a BootstrapMethodError
aeriksso
parents: 30227
diff changeset
  1571
    }
dfce1db72058 8051045: HotSpot fails to wrap Exceptions from invokedynamic in a BootstrapMethodError
aeriksso
parents: 30227
diff changeset
  1572
    if (PENDING_EXCEPTION->is_a(SystemDictionary::BootstrapMethodError_klass())) {
dfce1db72058 8051045: HotSpot fails to wrap Exceptions from invokedynamic in a BootstrapMethodError
aeriksso
parents: 30227
diff changeset
  1573
      // throw these guys, since they are already wrapped
dfce1db72058 8051045: HotSpot fails to wrap Exceptions from invokedynamic in a BootstrapMethodError
aeriksso
parents: 30227
diff changeset
  1574
      return;
dfce1db72058 8051045: HotSpot fails to wrap Exceptions from invokedynamic in a BootstrapMethodError
aeriksso
parents: 30227
diff changeset
  1575
    }
dfce1db72058 8051045: HotSpot fails to wrap Exceptions from invokedynamic in a BootstrapMethodError
aeriksso
parents: 30227
diff changeset
  1576
    if (!PENDING_EXCEPTION->is_a(SystemDictionary::LinkageError_klass())) {
dfce1db72058 8051045: HotSpot fails to wrap Exceptions from invokedynamic in a BootstrapMethodError
aeriksso
parents: 30227
diff changeset
  1577
      // intercept only LinkageErrors which might have failed to wrap
dfce1db72058 8051045: HotSpot fails to wrap Exceptions from invokedynamic in a BootstrapMethodError
aeriksso
parents: 30227
diff changeset
  1578
      return;
dfce1db72058 8051045: HotSpot fails to wrap Exceptions from invokedynamic in a BootstrapMethodError
aeriksso
parents: 30227
diff changeset
  1579
    }
dfce1db72058 8051045: HotSpot fails to wrap Exceptions from invokedynamic in a BootstrapMethodError
aeriksso
parents: 30227
diff changeset
  1580
    // See the "Linking Exceptions" section for the invokedynamic instruction in the JVMS.
dfce1db72058 8051045: HotSpot fails to wrap Exceptions from invokedynamic in a BootstrapMethodError
aeriksso
parents: 30227
diff changeset
  1581
    Handle nested_exception(THREAD, PENDING_EXCEPTION);
dfce1db72058 8051045: HotSpot fails to wrap Exceptions from invokedynamic in a BootstrapMethodError
aeriksso
parents: 30227
diff changeset
  1582
    CLEAR_PENDING_EXCEPTION;
dfce1db72058 8051045: HotSpot fails to wrap Exceptions from invokedynamic in a BootstrapMethodError
aeriksso
parents: 30227
diff changeset
  1583
    THROW_CAUSE(vmSymbols::java_lang_BootstrapMethodError(), nested_exception)
dfce1db72058 8051045: HotSpot fails to wrap Exceptions from invokedynamic in a BootstrapMethodError
aeriksso
parents: 30227
diff changeset
  1584
  }
dfce1db72058 8051045: HotSpot fails to wrap Exceptions from invokedynamic in a BootstrapMethodError
aeriksso
parents: 30227
diff changeset
  1585
}
13391
30245956af37 7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents: 10565
diff changeset
  1586
33593
60764a78fa5c 8140274: methodHandles and constantPoolHandles should be passed as const references
coleenp
parents: 33198
diff changeset
  1587
void LinkResolver::resolve_invokedynamic(CallInfo& result, const constantPoolHandle& pool, int index, TRAPS) {
13391
30245956af37 7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents: 10565
diff changeset
  1588
  Symbol* method_name       = pool->name_ref_at(index);
30245956af37 7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents: 10565
diff changeset
  1589
  Symbol* method_signature  = pool->signature_ref_at(index);
30245956af37 7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents: 10565
diff changeset
  1590
  KlassHandle current_klass = KlassHandle(THREAD, pool->pool_holder());
2570
ecc7862946d4 6655646: dynamic languages need dynamically linked call sites
jrose
parents: 2534
diff changeset
  1591
13391
30245956af37 7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents: 10565
diff changeset
  1592
  // Resolve the bootstrap specifier (BSM + optional arguments).
30245956af37 7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents: 10565
diff changeset
  1593
  Handle bootstrap_specifier;
30245956af37 7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents: 10565
diff changeset
  1594
  // Check if CallSite has been bound already:
13728
882756847a04 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 13396
diff changeset
  1595
  ConstantPoolCacheEntry* cpce = pool->invokedynamic_cp_cache_entry_at(index);
13391
30245956af37 7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents: 10565
diff changeset
  1596
  if (cpce->is_f1_null()) {
13728
882756847a04 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 13396
diff changeset
  1597
    int pool_index = cpce->constant_pool_index();
30746
dfce1db72058 8051045: HotSpot fails to wrap Exceptions from invokedynamic in a BootstrapMethodError
aeriksso
parents: 30227
diff changeset
  1598
    oop bsm_info = pool->resolve_bootstrap_specifier_at(pool_index, THREAD);
dfce1db72058 8051045: HotSpot fails to wrap Exceptions from invokedynamic in a BootstrapMethodError
aeriksso
parents: 30227
diff changeset
  1599
    wrap_invokedynamic_exception(CHECK);
13391
30245956af37 7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents: 10565
diff changeset
  1600
    assert(bsm_info != NULL, "");
30245956af37 7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents: 10565
diff changeset
  1601
    // FIXME: Cache this once per BootstrapMethods entry, not once per CONSTANT_InvokeDynamic.
30245956af37 7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents: 10565
diff changeset
  1602
    bootstrap_specifier = Handle(THREAD, bsm_info);
30245956af37 7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents: 10565
diff changeset
  1603
  }
30245956af37 7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents: 10565
diff changeset
  1604
  if (!cpce->is_f1_null()) {
13929
8da0dc50a6e4 7200949: JSR 292: rubybench/bench/time/bench_base64.rb fails with jruby.jar not on boot class path
twisti
parents: 13732
diff changeset
  1605
    methodHandle method(     THREAD, cpce->f1_as_method());
8da0dc50a6e4 7200949: JSR 292: rubybench/bench/time/bench_base64.rb fails with jruby.jar not on boot class path
twisti
parents: 13732
diff changeset
  1606
    Handle       appendix(   THREAD, cpce->appendix_if_resolved(pool));
8da0dc50a6e4 7200949: JSR 292: rubybench/bench/time/bench_base64.rb fails with jruby.jar not on boot class path
twisti
parents: 13732
diff changeset
  1607
    Handle       method_type(THREAD, cpce->method_type_if_resolved(pool));
30746
dfce1db72058 8051045: HotSpot fails to wrap Exceptions from invokedynamic in a BootstrapMethodError
aeriksso
parents: 30227
diff changeset
  1608
    result.set_handle(method, appendix, method_type, THREAD);
dfce1db72058 8051045: HotSpot fails to wrap Exceptions from invokedynamic in a BootstrapMethodError
aeriksso
parents: 30227
diff changeset
  1609
    wrap_invokedynamic_exception(CHECK);
13391
30245956af37 7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents: 10565
diff changeset
  1610
    return;
30245956af37 7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents: 10565
diff changeset
  1611
  }
2570
ecc7862946d4 6655646: dynamic languages need dynamically linked call sites
jrose
parents: 2534
diff changeset
  1612
13391
30245956af37 7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents: 10565
diff changeset
  1613
  if (TraceMethodHandles) {
35543
0961315f4016 8140659: C1: invokedynamic call patching violates JVMS-6.5.invokedynamic
vlivanov
parents: 35135
diff changeset
  1614
    ResourceMark rm(THREAD);
0961315f4016 8140659: C1: invokedynamic call patching violates JVMS-6.5.invokedynamic
vlivanov
parents: 35135
diff changeset
  1615
    tty->print_cr("resolve_invokedynamic #%d %s %s in %s",
13728
882756847a04 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 13396
diff changeset
  1616
                  ConstantPool::decode_invokedynamic_index(index),
35543
0961315f4016 8140659: C1: invokedynamic call patching violates JVMS-6.5.invokedynamic
vlivanov
parents: 35135
diff changeset
  1617
                  method_name->as_C_string(), method_signature->as_C_string(),
0961315f4016 8140659: C1: invokedynamic call patching violates JVMS-6.5.invokedynamic
vlivanov
parents: 35135
diff changeset
  1618
                  current_klass->name()->as_C_string());
13391
30245956af37 7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents: 10565
diff changeset
  1619
    tty->print("  BSM info: "); bootstrap_specifier->print();
30245956af37 7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents: 10565
diff changeset
  1620
  }
30245956af37 7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents: 10565
diff changeset
  1621
30245956af37 7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents: 10565
diff changeset
  1622
  resolve_dynamic_call(result, bootstrap_specifier, method_name, method_signature, current_klass, CHECK);
30245956af37 7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents: 10565
diff changeset
  1623
}
30245956af37 7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents: 10565
diff changeset
  1624
30245956af37 7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents: 10565
diff changeset
  1625
void LinkResolver::resolve_dynamic_call(CallInfo& result,
30245956af37 7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents: 10565
diff changeset
  1626
                                        Handle bootstrap_specifier,
30245956af37 7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents: 10565
diff changeset
  1627
                                        Symbol* method_name, Symbol* method_signature,
30245956af37 7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents: 10565
diff changeset
  1628
                                        KlassHandle current_klass,
30245956af37 7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents: 10565
diff changeset
  1629
                                        TRAPS) {
30245956af37 7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents: 10565
diff changeset
  1630
  // JSR 292:  this must resolve to an implicitly generated method MH.linkToCallSite(*...)
30245956af37 7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents: 10565
diff changeset
  1631
  // The appendix argument is likely to be a freshly-created CallSite.
30245956af37 7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents: 10565
diff changeset
  1632
  Handle       resolved_appendix;
13929
8da0dc50a6e4 7200949: JSR 292: rubybench/bench/time/bench_base64.rb fails with jruby.jar not on boot class path
twisti
parents: 13732
diff changeset
  1633
  Handle       resolved_method_type;
13391
30245956af37 7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents: 10565
diff changeset
  1634
  methodHandle resolved_method =
30245956af37 7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents: 10565
diff changeset
  1635
    SystemDictionary::find_dynamic_call_site_invoker(current_klass,
30245956af37 7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents: 10565
diff changeset
  1636
                                                     bootstrap_specifier,
30245956af37 7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents: 10565
diff changeset
  1637
                                                     method_name, method_signature,
30245956af37 7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents: 10565
diff changeset
  1638
                                                     &resolved_appendix,
13929
8da0dc50a6e4 7200949: JSR 292: rubybench/bench/time/bench_base64.rb fails with jruby.jar not on boot class path
twisti
parents: 13732
diff changeset
  1639
                                                     &resolved_method_type,
13732
a9865f5514fc 7196120: NPG: JSR 2292 test fails because missing fix for 7188911
twisti
parents: 13728
diff changeset
  1640
                                                     THREAD);
30746
dfce1db72058 8051045: HotSpot fails to wrap Exceptions from invokedynamic in a BootstrapMethodError
aeriksso
parents: 30227
diff changeset
  1641
  wrap_invokedynamic_exception(CHECK);
dfce1db72058 8051045: HotSpot fails to wrap Exceptions from invokedynamic in a BootstrapMethodError
aeriksso
parents: 30227
diff changeset
  1642
  result.set_handle(resolved_method, resolved_appendix, resolved_method_type, THREAD);
dfce1db72058 8051045: HotSpot fails to wrap Exceptions from invokedynamic in a BootstrapMethodError
aeriksso
parents: 30227
diff changeset
  1643
  wrap_invokedynamic_exception(CHECK);
2570
ecc7862946d4 6655646: dynamic languages need dynamically linked call sites
jrose
parents: 2534
diff changeset
  1644
}