Skip to content
Permalink
Browse files
8275975: Remove dead code in ciInstanceKlass
Reviewed-by: chagedorn, kvn
  • Loading branch information
TobiHartmann committed Oct 27, 2021
1 parent 4060602 commit a29273336bae75e8d185fa7f7c789acbec50a619
Showing 3 changed files with 0 additions and 15 deletions.
@@ -54,7 +54,6 @@ class ciArrayKlass : public ciKlass {

// What kind of vmObject is this?
bool is_array_klass() const { return true; }
bool is_java_klass() const { return true; }

static ciArrayKlass* make(ciType* element_type);
};
@@ -63,7 +63,6 @@ ciInstanceKlass::ciInstanceKlass(Klass* k) :
_has_finalizer = access_flags.has_finalizer();
_has_subklass = flags().is_final() ? subklass_false : subklass_unknown;
_init_state = ik->init_state();
_nonstatic_field_size = ik->nonstatic_field_size();
_has_nonstatic_fields = ik->has_nonstatic_fields();
_has_nonstatic_concrete_methods = ik->has_nonstatic_concrete_methods();
_is_hidden = ik->is_hidden();
@@ -123,7 +122,6 @@ ciInstanceKlass::ciInstanceKlass(ciSymbol* name,
{
assert(name->char_at(0) != JVM_SIGNATURE_ARRAY, "not an instance klass");
_init_state = (InstanceKlass::ClassState)0;
_nonstatic_field_size = -1;
_has_nonstatic_fields = false;
_nonstatic_fields = NULL;
_has_injected_fields = -1;
@@ -496,9 +494,6 @@ int ciInstanceKlass::compute_nonstatic_fields() {
}
assert(!is_java_lang_Object(), "bootstrap OK");

// Size in bytes of my fields, including inherited fields.
int fsize = nonstatic_field_size() * heapOopSize;

ciInstanceKlass* super = this->super();
GrowableArray<ciField*>* super_fields = NULL;
if (super != NULL && super->has_nonstatic_fields()) {
@@ -61,8 +61,6 @@ class ciInstanceKlass : public ciKlass {
bool _is_record;

ciFlags _flags;
jint _nonstatic_field_size;
jint _nonstatic_oop_map_size;

// Lazy fields get filled in only upon request.
ciInstanceKlass* _super;
@@ -173,15 +171,9 @@ class ciInstanceKlass : public ciKlass {
return (Klass::layout_helper_size_in_bytes(layout_helper())
>> LogHeapWordSize);
}
jint nonstatic_field_size() {
assert(is_loaded(), "must be loaded");
return _nonstatic_field_size; }
jint has_nonstatic_fields() {
assert(is_loaded(), "must be loaded");
return _has_nonstatic_fields; }
jint nonstatic_oop_map_size() {
assert(is_loaded(), "must be loaded");
return _nonstatic_oop_map_size; }
ciInstanceKlass* super();
jint nof_implementors() {
ciInstanceKlass* impl;
@@ -283,7 +275,6 @@ class ciInstanceKlass : public ciKlass {

// What kind of ciObject is this?
bool is_instance_klass() const { return true; }
bool is_java_klass() const { return true; }

virtual ciKlass* exact_klass() {
if (is_loaded() && is_final() && !is_interface()) {

1 comment on commit a292733

@openjdk-notifier
Copy link

@openjdk-notifier openjdk-notifier bot commented on a292733 Oct 27, 2021

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please sign in to comment.