Skip to content

Commit fa8dce4

Browse files
committed
8256365: Clean up vtable initialization code
Reviewed-by: eosterlund, dholmes
1 parent 4e5116c commit fa8dce4

File tree

4 files changed

+94
-101
lines changed

4 files changed

+94
-101
lines changed

src/hotspot/share/oops/instanceKlass.cpp

Lines changed: 0 additions & 21 deletions
Original file line numberDiff line numberDiff line change
@@ -2972,27 +2972,6 @@ bool InstanceKlass::is_same_class_package(oop other_class_loader,
29722972
}
29732973
}
29742974

2975-
// Returns true iff super_method can be overridden by a method in targetclassname
2976-
// See JLS 3rd edition 8.4.6.1
2977-
// Assumes name-signature match
2978-
// "this" is InstanceKlass of super_method which must exist
2979-
// note that the InstanceKlass of the method in the targetclassname has not always been created yet
2980-
bool InstanceKlass::is_override(const methodHandle& super_method, Handle targetclassloader, Symbol* targetclassname, TRAPS) {
2981-
// Private methods can not be overridden
2982-
if (super_method->is_private()) {
2983-
return false;
2984-
}
2985-
// If super method is accessible, then override
2986-
if ((super_method->is_protected()) ||
2987-
(super_method->is_public())) {
2988-
return true;
2989-
}
2990-
// Package-private methods are not inherited outside of package
2991-
assert(super_method->is_package_private(), "must be package private");
2992-
return(is_same_class_package(targetclassloader(), targetclassname));
2993-
}
2994-
2995-
29962975
static bool is_prohibited_package_slow(Symbol* class_name) {
29972976
// Caller has ResourceMark
29982977
int length;

src/hotspot/share/oops/instanceKlass.hpp

Lines changed: 0 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -512,9 +512,6 @@ class InstanceKlass: public Klass {
512512
enclosing_method_attribute_size = 2
513513
};
514514

515-
// method override check
516-
bool is_override(const methodHandle& super_method, Handle targetclassloader, Symbol* targetclassname, TRAPS);
517-
518515
// package
519516
PackageEntry* package() const { return _package_entry; }
520517
ModuleEntry* module() const;

0 commit comments

Comments
 (0)