Skip to content

Commit

Permalink
Browse files Browse the repository at this point in the history
8263908: Build fails due to initialize_static_field_for_dump defined …
…but not used after JDK-8263771

Reviewed-by: iklam, dholmes
  • Loading branch information
DamonFool committed Mar 21, 2021
1 parent cd45538 commit 35cd945
Showing 1 changed file with 4 additions and 0 deletions.
4 changes: 4 additions & 0 deletions src/hotspot/share/classfile/javaClasses.cpp
Expand Up @@ -841,6 +841,7 @@ static void initialize_static_string_field(fieldDescriptor* fd, Handle mirror, T
mirror()->obj_field_put(fd->offset(), string);
}

#if INCLUDE_CDS_JAVA_HEAP
static void initialize_static_string_field_for_dump(fieldDescriptor* fd, Handle mirror) {
DEBUG_ONLY(assert_valid_static_string_field(fd);)
assert(DumpSharedSpaces, "must be");
Expand All @@ -853,6 +854,7 @@ static void initialize_static_string_field_for_dump(fieldDescriptor* fd, Handle
guarantee(false, "Unexpected");
}
}
#endif

static void initialize_static_primitive_field(fieldDescriptor* fd, Handle mirror) {
assert(fd->has_initial_value(), "caller should have checked this");
Expand Down Expand Up @@ -900,6 +902,7 @@ static void initialize_static_field(fieldDescriptor* fd, Handle mirror, TRAPS) {
}
}

#if INCLUDE_CDS_JAVA_HEAP
static void initialize_static_field_for_dump(fieldDescriptor* fd, Handle mirror) {
assert(mirror.not_null() && fd->is_static(), "just checking");
if (fd->has_initial_value()) {
Expand All @@ -910,6 +913,7 @@ static void initialize_static_field_for_dump(fieldDescriptor* fd, Handle mirror)
}
}
}
#endif

void java_lang_Class::fixup_mirror(Klass* k, TRAPS) {
assert(InstanceMirrorKlass::offset_of_static_fields() != 0, "must have been computed already");
Expand Down

1 comment on commit 35cd945

@openjdk-notifier
Copy link

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.