Skip to content
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.

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

8275975: Remove dead code in ciInstanceKlass #6118

Closed
wants to merge 1 commit into from
Closed
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Jump to
Jump to file
Failed to load files.
Diff view
Diff view
@@ -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()) {