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

Paul Hohensee paul.hohensee at
Thu Dec 1 10:57:05 PST 2011

I can review everything except the c2 change, for which one of the compiler
folks might chime in.  Everything else looks good.


On 12/1/11 1:51 PM, 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