jdk-sandbox: comparison hotspot/src/share/vm/classfile/classFileParser.cpp

equal deleted inserted replaced

-:0a8fbbe180db
+:847fddcb639b
 NONSTATIC_ALIGNED_DOUBLE
 };
 struct FieldAllocationCount {
-int static_oop_count;
+unsigned int static_oop_count;
-int static_byte_count;
+unsigned int static_byte_count;
-int static_short_count;
+unsigned int static_short_count;
-int static_word_count;
+unsigned int static_word_count;
-int static_double_count;
+unsigned int static_double_count;
-int nonstatic_oop_count;
+unsigned int nonstatic_oop_count;
-int nonstatic_byte_count;
+unsigned int nonstatic_byte_count;
-int nonstatic_short_count;
+unsigned int nonstatic_short_count;
-int nonstatic_word_count;
+unsigned int nonstatic_word_count;
-int nonstatic_double_count;
+unsigned int nonstatic_double_count;
 };
 typeArrayHandle ClassFileParser::parse_fields(constantPoolHandle cp, bool is_interface,
 struct FieldAllocationCount *fac,
 objArrayHandle* fields_annotations, TRAPS) {
 && class_loader.is_null()) {
 java_lang_ref_Reference_fix_pre(&fields, cp, &fac, CHECK_(nullHandle));
 }
 // end of "discovered" field compactibility fix
-int nonstatic_double_count = fac.nonstatic_double_count;
+unsigned int nonstatic_double_count = fac.nonstatic_double_count;
-int nonstatic_word_count   = fac.nonstatic_word_count;
+unsigned int nonstatic_word_count   = fac.nonstatic_word_count;
-int nonstatic_short_count  = fac.nonstatic_short_count;
+unsigned int nonstatic_short_count  = fac.nonstatic_short_count;
-int nonstatic_byte_count   = fac.nonstatic_byte_count;
+unsigned int nonstatic_byte_count   = fac.nonstatic_byte_count;
-int nonstatic_oop_count    = fac.nonstatic_oop_count;
+unsigned int nonstatic_oop_count    = fac.nonstatic_oop_count;
 bool super_has_nonstatic_fields =
 (super_klass() != NULL && super_klass->has_nonstatic_fields());
 bool has_nonstatic_fields  =  super_has_nonstatic_fields ||
 ((nonstatic_double_count + nonstatic_word_count +
 nonstatic_short_count + nonstatic_byte_count +
 nonstatic_oop_count) != 0);
-// Prepare list of oops for oop maps generation.
+// Prepare list of oops for oop map generation.
-u2* nonstatic_oop_offsets;
+int* nonstatic_oop_offsets;
-u2* nonstatic_oop_length;
+unsigned int* nonstatic_oop_counts;
-int nonstatic_oop_map_count = 0;
+unsigned int nonstatic_oop_map_count = 0;
 nonstatic_oop_offsets = NEW_RESOURCE_ARRAY_IN_THREAD(
-THREAD, u2,  nonstatic_oop_count+1);
+THREAD, int, nonstatic_oop_count + 1);
-nonstatic_oop_length  = NEW_RESOURCE_ARRAY_IN_THREAD(
+nonstatic_oop_counts  = NEW_RESOURCE_ARRAY_IN_THREAD(
-THREAD, u2,  nonstatic_oop_count+1);
+THREAD, unsigned int, nonstatic_oop_count + 1);
 // Add fake fields for java.lang.Class instances (also see above).
 // FieldsAllocationStyle and CompactFields values will be reset to default.
 if(class_name() == vmSymbols::java_lang_Class() && class_loader.is_null()) {
 java_lang_Class_fix_post(&next_nonstatic_field_offset);
-nonstatic_oop_offsets[0] = (u2)first_nonstatic_field_offset;
+nonstatic_oop_offsets[0] = first_nonstatic_field_offset;
-int fake_oop_count       = (( next_nonstatic_field_offset -
+const uint fake_oop_count = (next_nonstatic_field_offset -
-first_nonstatic_field_offset ) / heapOopSize);
+first_nonstatic_field_offset) / heapOopSize;
-nonstatic_oop_length [0] = (u2)fake_oop_count;
+nonstatic_oop_counts[0] = fake_oop_count;
-nonstatic_oop_map_count  = 1;
+nonstatic_oop_map_count = 1;
-nonstatic_oop_count     -= fake_oop_count;
+nonstatic_oop_count -= fake_oop_count;
 first_nonstatic_oop_offset = first_nonstatic_field_offset;
 } else {
 first_nonstatic_oop_offset = 0; // will be set for first oop field
 }
 next_nonstatic_oop_offset += heapOopSize;
 }
 // Update oop maps
 if( nonstatic_oop_map_count > 0 &&
 nonstatic_oop_offsets[nonstatic_oop_map_count - 1] ==
-(u2)(real_offset - nonstatic_oop_length[nonstatic_oop_map_count - 1] * heapOopSize) ) {
+real_offset -
+int(nonstatic_oop_counts[nonstatic_oop_map_count - 1]) *
+heapOopSize ) {
 // Extend current oop map
-nonstatic_oop_length[nonstatic_oop_map_count - 1] += 1;
+nonstatic_oop_counts[nonstatic_oop_map_count - 1] += 1;
 } else {
 // Create new oop map
-nonstatic_oop_offsets[nonstatic_oop_map_count] = (u2)real_offset;
+nonstatic_oop_offsets[nonstatic_oop_map_count] = real_offset;
-nonstatic_oop_length [nonstatic_oop_map_count] = 1;
+nonstatic_oop_counts [nonstatic_oop_map_count] = 1;
 nonstatic_oop_map_count += 1;
 if( first_nonstatic_oop_offset == 0 ) { // Undefined
 first_nonstatic_oop_offset = real_offset;
 }
 }
 next_nonstatic_type_offset = align_size_up(notaligned_offset, wordSize );
 instance_size = align_object_size(next_nonstatic_type_offset / wordSize);
 assert(instance_size == align_object_size(align_size_up((instanceOopDesc::base_offset_in_bytes() + nonstatic_field_size*heapOopSize), wordSize) / wordSize), "consistent layout helper value");
-// Size of non-static oop map blocks (in words) allocated at end of klass
+// Number of non-static oop map blocks allocated at end of klass.
-int nonstatic_oop_map_size = compute_oop_map_size(super_klass, nonstatic_oop_map_count, first_nonstatic_oop_offset);
+const unsigned int total_oop_map_count =
+compute_oop_map_count(super_klass, nonstatic_oop_map_count,
+first_nonstatic_oop_offset);
 // Compute reference type
 ReferenceType rt;
 if (super_klass() == NULL) {
 rt = REF_NONE;
 } else {
 rt = super_klass->reference_type();
 }
 // We can now create the basic klassOop for this klass
-klassOop ik = oopFactory::new_instanceKlass(
+klassOop ik = oopFactory::new_instanceKlass(vtable_size, itable_size,
-vtable_size, itable_size,
+static_field_size,
-static_field_size, nonstatic_oop_map_size,
+total_oop_map_count,
 rt, CHECK_(nullHandle));
 instanceKlassHandle this_klass (THREAD, ik);
-assert(this_klass->static_field_size() == static_field_size &&
+assert(this_klass->static_field_size() == static_field_size, "sanity");
-this_klass->nonstatic_oop_map_size() == nonstatic_oop_map_size, "sanity check");
+assert(this_klass->nonstatic_oop_map_count() == total_oop_map_count,
+"sanity");
 // Fill in information already parsed
 this_klass->set_access_flags(access_flags);
 if (verify) {
 this_klass->set_should_verify_class();
 // Initialize itable offset tables
 klassItable::setup_itable_offset_table(this_klass);
 // Do final class setup
-fill_oop_maps(this_klass, nonstatic_oop_map_count, nonstatic_oop_offsets, nonstatic_oop_length);
+fill_oop_maps(this_klass, nonstatic_oop_map_count, nonstatic_oop_offsets, nonstatic_oop_counts);
 set_precomputed_flags(this_klass);
 // reinitialize modifiers, using the InnerClasses attribute
 int computed_modifiers = this_klass->compute_modifier_flags(CHECK_(nullHandle));
 return this_klass;
 }
-int ClassFileParser::compute_oop_map_size(instanceKlassHandle super, int nonstatic_oop_map_count, int first_nonstatic_oop_offset) {
+unsigned int
-int map_size = super.is_null() ? 0 : super->nonstatic_oop_map_size();
+ClassFileParser::compute_oop_map_count(instanceKlassHandle super,
+unsigned int nonstatic_oop_map_count,
+int first_nonstatic_oop_offset) {
+unsigned int map_count =
+super.is_null() ? 0 : super->nonstatic_oop_map_count();
 if (nonstatic_oop_map_count > 0) {
 // We have oops to add to map
-if (map_size == 0) {
+if (map_count == 0) {
-map_size = nonstatic_oop_map_count;
+map_count = nonstatic_oop_map_count;
 } else {
-// Check whether we should add a new map block or whether the last one can be extended
+// Check whether we should add a new map block or whether the last one can
-OopMapBlock* first_map = super->start_of_nonstatic_oop_maps();
+// be extended
-OopMapBlock* last_map = first_map + map_size - 1;
+OopMapBlock* const first_map = super->start_of_nonstatic_oop_maps();
+OopMapBlock* const last_map = first_map + map_count - 1;
-int next_offset = last_map->offset() + (last_map->length() * heapOopSize);
+int next_offset = last_map->offset() + last_map->count() * heapOopSize;
 if (next_offset == first_nonstatic_oop_offset) {
 // There is no gap bettwen superklass's last oop field and first
 // local oop field, merge maps.
 nonstatic_oop_map_count -= 1;
 } else {
 // Superklass didn't end with a oop field, add extra maps
-assert(next_offset<first_nonstatic_oop_offset, "just checking");
+assert(next_offset < first_nonstatic_oop_offset, "just checking");
 }
-map_size += nonstatic_oop_map_count;
+map_count += nonstatic_oop_map_count;
 }
 }
-return map_size;
+return map_count;
 }
 void ClassFileParser::fill_oop_maps(instanceKlassHandle k,
-int nonstatic_oop_map_count,
+unsigned int nonstatic_oop_map_count,
-u2* nonstatic_oop_offsets, u2* nonstatic_oop_length) {
+int* nonstatic_oop_offsets,
+unsigned int* nonstatic_oop_counts) {
 OopMapBlock* this_oop_map = k->start_of_nonstatic_oop_maps();
-OopMapBlock* last_oop_map = this_oop_map + k->nonstatic_oop_map_size();
+const instanceKlass* const super = k->superklass();
-instanceKlass* super = k->superklass();
+const unsigned int super_count = super ? super->nonstatic_oop_map_count() : 0;
-if (super != NULL) {
+if (super_count > 0) {
-int super_oop_map_size     = super->nonstatic_oop_map_size();
+// Copy maps from superklass
 OopMapBlock* super_oop_map = super->start_of_nonstatic_oop_maps();
-// Copy maps from superklass
+for (unsigned int i = 0; i < super_count; ++i) {
-while (super_oop_map_size-- > 0) {
 *this_oop_map++ = *super_oop_map++;
 }
 }
 if (nonstatic_oop_map_count > 0) {
-if (this_oop_map + nonstatic_oop_map_count > last_oop_map) {
+if (super_count + nonstatic_oop_map_count > k->nonstatic_oop_map_count()) {
-// Calculated in compute_oop_map_size() number of oop maps is less then
+// The counts differ because there is no gap between superklass's last oop
-// collected oop maps since there is no gap between superklass's last oop
+// field and the first local oop field.  Extend the last oop map copied
-// field and first local oop field. Extend the last oop map copied
 // from the superklass instead of creating new one.
 nonstatic_oop_map_count--;
 nonstatic_oop_offsets++;
 this_oop_map--;
-this_oop_map->set_length(this_oop_map->length() + *nonstatic_oop_length++);
+this_oop_map->set_count(this_oop_map->count() + *nonstatic_oop_counts++);
 this_oop_map++;
 }
-assert((this_oop_map + nonstatic_oop_map_count) == last_oop_map, "just checking");
 // Add new map blocks, fill them
 while (nonstatic_oop_map_count-- > 0) {
 this_oop_map->set_offset(*nonstatic_oop_offsets++);
-this_oop_map->set_length(*nonstatic_oop_length++);
+this_oop_map->set_count(*nonstatic_oop_counts++);
 this_oop_map++;
 }
+assert(k->start_of_nonstatic_oop_maps() + k->nonstatic_oop_map_count() ==
+this_oop_map, "sanity");
 }
 }
 void ClassFileParser::set_precomputed_flags(instanceKlassHandle k) {

changeset 3821	847fddcb639b
parent 3820	0a8fbbe180db
parent 3694	942b7bc7f28c
child 3822	388b0393a042