Request for review(XL): 6985015: C1 needs to support compressed oops

Vladimir Kozlov vladimir.kozlov at
Thu Nov 25 11:38:54 PST 2010


I don't like adding new br instructions which are used only 4 times.
Could you simple totally separate code for COOP and use corresponding br (icc) instructions?

Also there are no explanation/comments around new move_wide code. Why you need it (append()?) ? I am not C1 expert so it 
is not clear to me.

Otherwise it is better then before.


On 11/25/10 7:03 AM, Igor Veresov wrote:
> I've updated the webrev with includeDB changes and also removed unnecessary ifdefs.
> Webrev:
> igor
> On 11/18/10 7:12 PM, Igor Veresov wrote:
>> This change implements compressed oops for C1 for x64 and sparc.
>> The changes are mostly on the codegen level, with a few exceptions when
>> we do access things outside of the heap that are uncompressed from the IR.
>> Webrev:
>> Thanks,
>> igor

More information about the hotspot-compiler-dev mailing list