diff -r 1ecc914fb707 -r 3221f5e14866 src/hotspot/share/runtime/reflection.cpp --- a/src/hotspot/share/runtime/reflection.cpp Wed Sep 19 21:31:33 2018 +0200 +++ b/src/hotspot/share/runtime/reflection.cpp Wed Sep 19 15:25:05 2018 -0400 @@ -898,28 +898,17 @@ java_lang_reflect_Method::set_exception_types(mh(), exception_types()); java_lang_reflect_Method::set_modifiers(mh(), modifiers); java_lang_reflect_Method::set_override(mh(), false); - if (java_lang_reflect_Method::has_signature_field() && - method->generic_signature() != NULL) { + if (method->generic_signature() != NULL) { Symbol* gs = method->generic_signature(); Handle sig = java_lang_String::create_from_symbol(gs, CHECK_NULL); java_lang_reflect_Method::set_signature(mh(), sig()); } - if (java_lang_reflect_Method::has_annotations_field()) { - typeArrayOop an_oop = Annotations::make_java_array(method->annotations(), CHECK_NULL); - java_lang_reflect_Method::set_annotations(mh(), an_oop); - } - if (java_lang_reflect_Method::has_parameter_annotations_field()) { - typeArrayOop an_oop = Annotations::make_java_array(method->parameter_annotations(), CHECK_NULL); - java_lang_reflect_Method::set_parameter_annotations(mh(), an_oop); - } - if (java_lang_reflect_Method::has_annotation_default_field()) { - typeArrayOop an_oop = Annotations::make_java_array(method->annotation_default(), CHECK_NULL); - java_lang_reflect_Method::set_annotation_default(mh(), an_oop); - } - if (java_lang_reflect_Method::has_type_annotations_field()) { - typeArrayOop an_oop = Annotations::make_java_array(method->type_annotations(), CHECK_NULL); - java_lang_reflect_Method::set_type_annotations(mh(), an_oop); - } + typeArrayOop an_oop = Annotations::make_java_array(method->annotations(), CHECK_NULL); + java_lang_reflect_Method::set_annotations(mh(), an_oop); + an_oop = Annotations::make_java_array(method->parameter_annotations(), CHECK_NULL); + java_lang_reflect_Method::set_parameter_annotations(mh(), an_oop); + an_oop = Annotations::make_java_array(method->annotation_default(), CHECK_NULL); + java_lang_reflect_Method::set_annotation_default(mh(), an_oop); return mh(); } @@ -948,24 +937,15 @@ java_lang_reflect_Constructor::set_exception_types(ch(), exception_types()); java_lang_reflect_Constructor::set_modifiers(ch(), modifiers); java_lang_reflect_Constructor::set_override(ch(), false); - if (java_lang_reflect_Constructor::has_signature_field() && - method->generic_signature() != NULL) { + if (method->generic_signature() != NULL) { Symbol* gs = method->generic_signature(); Handle sig = java_lang_String::create_from_symbol(gs, CHECK_NULL); java_lang_reflect_Constructor::set_signature(ch(), sig()); } - if (java_lang_reflect_Constructor::has_annotations_field()) { - typeArrayOop an_oop = Annotations::make_java_array(method->annotations(), CHECK_NULL); - java_lang_reflect_Constructor::set_annotations(ch(), an_oop); - } - if (java_lang_reflect_Constructor::has_parameter_annotations_field()) { - typeArrayOop an_oop = Annotations::make_java_array(method->parameter_annotations(), CHECK_NULL); - java_lang_reflect_Constructor::set_parameter_annotations(ch(), an_oop); - } - if (java_lang_reflect_Constructor::has_type_annotations_field()) { - typeArrayOop an_oop = Annotations::make_java_array(method->type_annotations(), CHECK_NULL); - java_lang_reflect_Constructor::set_type_annotations(ch(), an_oop); - } + typeArrayOop an_oop = Annotations::make_java_array(method->annotations(), CHECK_NULL); + java_lang_reflect_Constructor::set_annotations(ch(), an_oop); + an_oop = Annotations::make_java_array(method->parameter_annotations(), CHECK_NULL); + java_lang_reflect_Constructor::set_parameter_annotations(ch(), an_oop); return ch(); } @@ -986,20 +966,13 @@ // Note the ACC_ANNOTATION bit, which is a per-class access flag, is never set here. java_lang_reflect_Field::set_modifiers(rh(), fd->access_flags().as_int() & JVM_RECOGNIZED_FIELD_MODIFIERS); java_lang_reflect_Field::set_override(rh(), false); - if (java_lang_reflect_Field::has_signature_field() && - fd->has_generic_signature()) { + if (fd->has_generic_signature()) { Symbol* gs = fd->generic_signature(); Handle sig = java_lang_String::create_from_symbol(gs, CHECK_NULL); java_lang_reflect_Field::set_signature(rh(), sig()); } - if (java_lang_reflect_Field::has_annotations_field()) { - typeArrayOop an_oop = Annotations::make_java_array(fd->annotations(), CHECK_NULL); - java_lang_reflect_Field::set_annotations(rh(), an_oop); - } - if (java_lang_reflect_Field::has_type_annotations_field()) { - typeArrayOop an_oop = Annotations::make_java_array(fd->type_annotations(), CHECK_NULL); - java_lang_reflect_Field::set_type_annotations(rh(), an_oop); - } + typeArrayOop an_oop = Annotations::make_java_array(fd->annotations(), CHECK_NULL); + java_lang_reflect_Field::set_annotations(rh(), an_oop); return rh(); }