Request for review: 7117052 instanceKlass::_init_state can be u1 type
jiangli.zhou at oracle.com
Thu Dec 1 10:59:45 PST 2011
Thanks for your review!
On 12/01/2011 10:57 AM, Paul Hohensee wrote:
> I can review everything except the c2 change, for which one of the
> 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.
More information about the hotspot-compiler-dev