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

Paul Hohensee paul.hohensee at oracle.com
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.

Paul

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.
>
> http://cr.openjdk.java.net/~jiangli/7117052/webrev.00/
>
> Thanks,
>
> Jiangli
>


More information about the hotspot-compiler-dev mailing list