archive/jdk
1
0
Fork 0
mirror of https://github.com/openjdk/jdk.git synced 2025-09-15 08:34:30 +02:00
Code Issues Projects Releases Packages Wiki Activity Actions

8263992: Remove dead code NativeLookup::base_library_lookup

Browse source 
Reviewed-by: coleenp, dholmes
This commit is contained in:
Ioi Lam 2021-03-23 21:15:38 +00:00
parent 91d86e6ac1
commit 35102cb03f
12 changed files with 22 additions and 57 deletions
Download patch file Download diff file Expand all files Collapse all files

1
src/hotspot/share/c1/c1_Compiler.cpp
View file

@ -37,7 +37,6 @@
#include "memory/allocation.hpp"
#include "memory/allocation.inline.hpp"
#include "memory/resourceArea.hpp"
#include "prims/nativeLookup.hpp"
#include "runtime/arguments.hpp"
#include "runtime/interfaceSupport.inline.hpp"
#include "runtime/sharedRuntime.hpp"

1
src/hotspot/share/ci/ciMethod.cpp
View file

@ -45,7 +45,6 @@
#include "oops/method.inline.hpp"
#include "oops/oop.inline.hpp"
#include "prims/methodHandles.hpp"
#include "prims/nativeLookup.hpp"
#include "runtime/deoptimization.hpp"
#include "runtime/handles.inline.hpp"
#include "utilities/bitMap.inline.hpp"

1
src/hotspot/share/compiler/compilationPolicy.cpp
View file

@ -32,7 +32,6 @@
#include "oops/method.inline.hpp"
#include "oops/oop.inline.hpp"
#include "prims/jvmtiExport.hpp"
#include "prims/nativeLookup.hpp"
#include "runtime/arguments.hpp"
#include "runtime/deoptimization.hpp"
#include "runtime/frame.hpp"

3
src/hotspot/share/compiler/compileBroker.cpp
View file

@ -1432,8 +1432,7 @@ nmethod* CompileBroker::compile_method(const methodHandle& method, int osr_bci,
// Note: A native method implies non-osr compilation which is
// checked with an assertion at the entry of this method.
if (method->is_native() && !method->is_method_handle_intrinsic()) {
bool in_base_library;
address adr = NativeLookup::lookup(method, in_base_library, THREAD);
address adr = NativeLookup::lookup(method, THREAD);
if (HAS_PENDING_EXCEPTION) {
// In case of an exception looking up the method, we just forget
// about it. The interpreter will kick-in and throw the exception.

3
src/hotspot/share/interpreter/interpreterRuntime.cpp
View file

@ -1450,9 +1450,8 @@ JRT_ENTRY(void, InterpreterRuntime::prepare_native_call(JavaThread* thread, Meth
methodHandle m(thread, method);
assert(m->is_native(), "sanity check");
// lookup native function entry point if it doesn't exist
bool in_base_library;
if (!m->has_native_function()) {
NativeLookup::lookup(m, in_base_library, CHECK);
NativeLookup::lookup(m, CHECK);
}
// make sure signature handler is installed
SignatureHandlerLibrary::add(m);

1
src/hotspot/share/interpreter/linkResolver.cpp
View file

@ -51,7 +51,6 @@
#include "oops/objArrayOop.hpp"
#include "oops/oop.inline.hpp"
#include "prims/methodHandles.hpp"
#include "prims/nativeLookup.hpp"
#include "runtime/fieldDescriptor.inline.hpp"
#include "runtime/frame.inline.hpp"
#include "runtime/handles.inline.hpp"

1
src/hotspot/share/oops/method.cpp
View file

@ -59,7 +59,6 @@
#include "oops/symbol.hpp"
#include "prims/jvmtiExport.hpp"
#include "prims/methodHandles.hpp"
#include "prims/nativeLookup.hpp"
#include "runtime/arguments.hpp"
#include "runtime/atomic.hpp"
#include "runtime/frame.inline.hpp"

3
src/hotspot/share/opto/doCall.cpp
View file

@ -1,5 +1,5 @@
/*
* Copyright (c) 1998, 2020, Oracle and/or its affiliates. All rights reserved.
* Copyright (c) 1998, 2021, Oracle and/or its affiliates. All rights reserved.
* DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS FILE HEADER.
*
* This code is free software; you can redistribute it and/or modify it
@ -40,7 +40,6 @@
#include "opto/runtime.hpp"
#include "opto/subnode.hpp"
#include "prims/methodHandles.hpp"
#include "prims/nativeLookup.hpp"
#include "runtime/sharedRuntime.hpp"
void trace_type_profile(Compile* C, ciMethod *method, int depth, int bci, ciMethod *prof_method, ciKlass *prof_klass, int site_count, int receiver_count) {

1
src/hotspot/share/opto/library_call.cpp
View file

@ -50,7 +50,6 @@
#include "opto/runtime.hpp"
#include "opto/rootnode.hpp"
#include "opto/subnode.hpp"
#include "prims/nativeLookup.hpp"
#include "prims/unsafe.hpp"
#include "runtime/objectMonitor.hpp"
#include "runtime/sharedRuntime.hpp"

1
src/hotspot/share/prims/jvm.cpp
View file

@ -65,7 +65,6 @@
#include "prims/jvm_misc.hpp"
#include "prims/jvmtiExport.hpp"
#include "prims/jvmtiThreadState.hpp"
#include "prims/nativeLookup.hpp"
#include "prims/stackwalk.hpp"
#include "runtime/arguments.hpp"
#include "runtime/atomic.hpp"

48
src/hotspot/share/prims/nativeLookup.cpp
View file

@ -266,7 +266,7 @@ static address lookup_special_native(const char* jni_name) {
return NULL;
}
address NativeLookup::lookup_style(const methodHandle& method, char* pure_name, const char* long_name, int args_size, bool os_style, bool& in_base_library, TRAPS) {
address NativeLookup::lookup_style(const methodHandle& method, char* pure_name, const char* long_name, int args_size, bool os_style, TRAPS) {
address entry;
const char* jni_name = compute_complete_jni_name(pure_name, long_name, args_size, os_style);
@ -283,7 +283,6 @@ address NativeLookup::lookup_style(const methodHandle& method, char* pure_name,
entry = (address) os::dll_lookup(os::native_java_library(), jni_name);
}
if (entry != NULL) {
in_base_library = true;
return entry;
}
}
@ -340,9 +339,8 @@ address NativeLookup::lookup_critical_style(void* dll, const char* pure_name, co
// Check all the formats of native implementation name to see if there is one
// for the specified method.
address NativeLookup::lookup_entry(const methodHandle& method, bool& in_base_library, TRAPS) {
address NativeLookup::lookup_entry(const methodHandle& method, TRAPS) {
address entry = NULL;
in_base_library = false;
// Compute pure name
char* pure_name = pure_jni_name(method);
if (pure_name == NULL) {
@ -357,7 +355,7 @@ address NativeLookup::lookup_entry(const methodHandle& method, bool& in_base_lib
+ method->size_of_parameters(); // actual parameters
// 1) Try JNI short style
entry = lookup_style(method, pure_name, "", args_size, true, in_base_library, CHECK_NULL);
entry = lookup_style(method, pure_name, "", args_size, true, CHECK_NULL);
if (entry != NULL) return entry;
// Compute long name
@ -369,15 +367,15 @@ address NativeLookup::lookup_entry(const methodHandle& method, bool& in_base_lib
}
// 2) Try JNI long style
entry = lookup_style(method, pure_name, long_name, args_size, true, in_base_library, CHECK_NULL);
entry = lookup_style(method, pure_name, long_name, args_size, true, CHECK_NULL);
if (entry != NULL) return entry;
// 3) Try JNI short style without os prefix/suffix
entry = lookup_style(method, pure_name, "", args_size, false, in_base_library, CHECK_NULL);
entry = lookup_style(method, pure_name, "", args_size, false, CHECK_NULL);
if (entry != NULL) return entry;
// 4) Try JNI long style without os prefix/suffix
entry = lookup_style(method, pure_name, long_name, args_size, false, in_base_library, CHECK_NULL);
entry = lookup_style(method, pure_name, long_name, args_size, false, CHECK_NULL);
return entry; // NULL indicates not found
}
@ -489,7 +487,7 @@ address NativeLookup::lookup_critical_style(void* dll, const methodHandle& metho
// If any are found, remove them before attemping the look up of the
// native implementation again.
// See SetNativeMethodPrefix in the JVM TI Spec for more details.
address NativeLookup::lookup_entry_prefixed(const methodHandle& method, bool& in_base_library, TRAPS) {
address NativeLookup::lookup_entry_prefixed(const methodHandle& method, TRAPS) {
#if INCLUDE_JVMTI
ResourceMark rm(THREAD);
@ -516,7 +514,7 @@ address NativeLookup::lookup_entry_prefixed(const methodHandle& method, bool& in
if (wrapper_method != NULL && !wrapper_method->is_native()) {
// we found a wrapper method, use its native entry
method->set_is_prefixed_native();
return lookup_entry(methodHandle(THREAD, wrapper_method), in_base_library, THREAD);
return lookup_entry(methodHandle(THREAD, wrapper_method), THREAD);
}
}
}
@ -524,16 +522,16 @@ address NativeLookup::lookup_entry_prefixed(const methodHandle& method, bool& in
return NULL;
}
address NativeLookup::lookup_base(const methodHandle& method, bool& in_base_library, TRAPS) {
address NativeLookup::lookup_base(const methodHandle& method, TRAPS) {
address entry = NULL;
ResourceMark rm(THREAD);
entry = lookup_entry(method, in_base_library, THREAD);
entry = lookup_entry(method, THREAD);
if (entry != NULL) return entry;
// standard native method resolution has failed. Check if there are any
// JVM TI prefixes which have been applied to the native method name.
entry = lookup_entry_prefixed(method, in_base_library, THREAD);
entry = lookup_entry_prefixed(method, THREAD);
if (entry != NULL) return entry;
// Native function not found, throw UnsatisfiedLinkError
@ -545,9 +543,9 @@ address NativeLookup::lookup_base(const methodHandle& method, bool& in_base_libr
}
address NativeLookup::lookup(const methodHandle& method, bool& in_base_library, TRAPS) {
address NativeLookup::lookup(const methodHandle& method, TRAPS) {
if (!method->has_native_function()) {
address entry = lookup_base(method, in_base_library, CHECK_NULL);
address entry = lookup_base(method, CHECK_NULL);
method->set_native_function(entry,
Method::native_bind_event_is_interesting);
// -verbose:jni printing
@ -560,23 +558,3 @@ address NativeLookup::lookup(const methodHandle& method, bool& in_base_library,
}
return method->native_function();
}
address NativeLookup::base_library_lookup(const char* class_name, const char* method_name, const char* signature) {
EXCEPTION_MARK;
bool in_base_library = true; // SharedRuntime inits some math methods.
TempNewSymbol c_name = SymbolTable::new_symbol(class_name);
TempNewSymbol m_name = SymbolTable::new_symbol(method_name);
TempNewSymbol s_name = SymbolTable::new_symbol(signature);
// Find the class
Klass* k = SystemDictionary::resolve_or_fail(c_name, true, CATCH);
InstanceKlass* klass = InstanceKlass::cast(k);
// Find method and invoke standard lookup
methodHandle method (THREAD,
klass->uncached_lookup_method(m_name, s_name, Klass::OverpassLookupMode::find));
address result = lookup(method, in_base_library, CATCH);
assert(in_base_library, "must be in basic library");
guarantee(result != NULL, "must be non NULL");
return result;
}

15
src/hotspot/share/prims/nativeLookup.hpp
View file

@ -1,5 +1,5 @@
/*
* Copyright (c) 1997, 2019, Oracle and/or its affiliates. All rights reserved.
* Copyright (c) 1997, 2021, Oracle and/or its affiliates. All rights reserved.
* DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS FILE HEADER.
*
* This code is free software; you can redistribute it and/or modify it
@ -34,12 +34,12 @@
class NativeLookup : AllStatic {
private:
// Style specific lookup
static address lookup_style(const methodHandle& method, char* pure_name, const char* long_name, int args_size, bool os_style, bool& in_base_library, TRAPS);
static address lookup_style(const methodHandle& method, char* pure_name, const char* long_name, int args_size, bool os_style, TRAPS);
static address lookup_critical_style(void* dll, const char* pure_name, const char* long_name, int args_size, bool os_style);
static address lookup_critical_style(void* dll, const methodHandle& method, int args_size);
static address lookup_base (const methodHandle& method, bool& in_base_library, TRAPS);
static address lookup_entry(const methodHandle& method, bool& in_base_library, TRAPS);
static address lookup_entry_prefixed(const methodHandle& method, bool& in_base_library, TRAPS);
static address lookup_base (const methodHandle& method, TRAPS);
static address lookup_entry(const methodHandle& method, TRAPS);
static address lookup_entry_prefixed(const methodHandle& method, TRAPS);
static void* dll_load(const methodHandle& method);
static const char* compute_complete_jni_name(const char* pure_name, const char* long_name, int args_size, bool os_style);
@ -50,11 +50,8 @@ class NativeLookup : AllStatic {
static char* critical_jni_name(const methodHandle& method);
// Lookup native function. May throw UnsatisfiedLinkError.
static address lookup(const methodHandle& method, bool& in_base_library, TRAPS);
static address lookup(const methodHandle& method, TRAPS);
static address lookup_critical_entry(const methodHandle& method);
// Lookup native functions in base library.
static address base_library_lookup(const char* class_name, const char* method_name, const char* signature);
};
#endif // SHARE_PRIMS_NATIVELOOKUP_HPP