Request for review(XL): 6985015: C1 needs to support compressed oops
vladimir.kozlov at oracle.com
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: http://cr.openjdk.java.net/~iveresov/6985015/webrev.02/
> 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: http://cr.openjdk.java.net/~iveresov/6985015/webrev.00/
More information about the hotspot-compiler-dev