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

[Jiangli Zhou]

Hi Paul,

Thanks for your review!

Jiangli

On 12/01/2011 10:57 AM, Paul Hohensee wrote:
> 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
>>