Request for review: 7117052 instanceKlass::_init_state can be u1 type

David Holmes david.holmes at
Thu Dec 1 17:18:23 PST 2011

As already mentioned in internal email, there seem to be changes missing 
for debug builds. In instanceKlass set_init_state has two 
implementations - one for product and one for debug. The debug version 
has not been modified to accommodate the change in type.


On 2/12/2011 4:51 AM, Jiangli Zhou wrote:
> The instanceKlass::_init_state is defined as instanceKlass::ClassState
> type, which is an enum. Currently there are 7 class states defined. The
> instanceKlass::_init_state can be changed to u1 type, which could hold
> up 256 states. There are unused bytes after
> instanceKlass::_idnum_allocated_count field. Changing _init_state to u1
> and move it to after the _idnum_allocate_count field would save 4-byte
> for each loaded class.
> Thanks,
> Jiangli

More information about the hotspot-compiler-dev mailing list