Skip to content

8386874: [lworld] Clarify documentation of InstanceKlass layout in instanceKlass.hpp #2606

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
coleenp wants to merge 1 commit into from
+9 −14
Closed

8386874: [lworld] Clarify documentation of InstanceKlass layout in instanceKlass.hpp #2606

Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
23 changes: 9 additions & 14 deletions src/hotspot/share/oops/instanceKlass.hpp
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -55,10 +55,11 @@ class RecordComponent;

// InstanceKlass embedded field layout (after declared fields):
// [EMBEDDED Java vtable ] size in words = vtable_len
// [EMBEDDED Java itable ] size in words = itable_len
// [EMBEDDED nonstatic oop-map blocks] size in words = nonstatic_oop_map_size
// The embedded nonstatic oop-map blocks are short pairs (offset, length)
// indicating where oops are located in instances of this klass.
// [EMBEDDED implementor of the interface] only exist for interface
// [EMBEDDED implementor of the interface] only exists for interface
// [EMBEDDED InlineKlass::Members] only if is an InlineKlass instance


Expand Down Expand Up @@ -335,19 +336,6 @@ class InstanceKlass: public Klass {
// Located here because sub-klasses can't have their own C++ fields
address _adr_inline_klass_members;

// embedded Java vtable follows here
// embedded Java itables follows here
// embedded static fields follows here
// embedded nonstatic oop-map blocks follows here
// embedded implementor of this interface follows here
// The embedded implementor only exists if the current klass is an
// interface. The possible values of the implementor fall into following
// three cases:
// null: no implementor.
// A Klass* that's not itself: one implementor.
// Itself: more than one implementors.
//

friend class SystemDictionary;

static bool _disable_method_binary_search;
Expand Down Expand Up @@ -1006,6 +994,13 @@ class InstanceKlass: public Klass {
#endif

// Access to the implementor of an interface.
// The embedded implementor only exists if the current klass is an
// interface. The possible values of the implementor fall into following
// three cases:
// null: no implementor.
// A Klass* that's not itself: one implementor.
// Itself: more than one implementors.
//
InstanceKlass* implementor() const;
void set_implementor(InstanceKlass* ik);
int nof_implementors() const;
Expand Down