RFR(M): 8068503: ppc64: Encode/Decode nodes for disjoint cOops mode

[Lindenmaier, Goetz]

Hi Coleen,

ok, I understand.  
Thanks for the input on this one so far, it really was helpful!

Best regards,
  Goetz.

-----Original Message-----
From: Coleen Phillimore [mailto:coleen.phillimore at oracle.com] 
Sent: Dienstag, 13. Januar 2015 22:37
To: Lindenmaier, Goetz; 'hotspot-dev at openjdk.java.net'
Subject: Re: RFR(M): 8068503: ppc64: Encode/Decode nodes for disjoint cOops mode


Since this is mostly PPC and compiler code, I can't properly review it, 
so please do not wait for me.
Coleen

On 1/12/15, 2:57 AM, Lindenmaier, Goetz wrote:
> Hi Coleen,
>
> Now I implemented the optimization in macroAssembler, also for the
> decode case, anyways.
> Unfortunately it's only used in the methodHandles code.
> Testing took a bit because the results were spoiled by unrelated problems.
>
> I uploaded a new webrev:
> http://cr.openjdk.java.net/~goetz/webrevs/8068503-ppcDisjoint/webrev.02/
>
> Best regards,
>    Goetz.
>
> -----Original Message-----
> From: hotspot-dev [mailto:hotspot-dev-bounces at openjdk.java.net] On Behalf Of Coleen Phillimore
> Sent: Wednesday, January 07, 2015 3:54 PM
> To: hotspot-dev at openjdk.java.net
> Subject: Re: RFR(M): 8068503: ppc64: Encode/Decode nodes for disjoint cOops mode
>
>
> Don't you have to make (or want to make) changes to
> MacroAssembler::decode_heap_oop_not_null in the macro assembler files?
>
> Coleen
>
> On 1/7/15, 3:26 AM, Lindenmaier, Goetz wrote:
>> Hi,
>>
>> this change contains the implementation (activation) of the
>> encode/decode nodes exploiting the disjoint heap base
>> available since "8064457: Introduce compressed oops mode disjoint base..."
>>
>> Please review this change.
>> http://cr.openjdk.java.net/~goetz/webrevs/8068503-ppcDisjoint/
>>
>> This is ppc64-only, so I don't need a sponsor ;)
>>
>> Best regards,
>>     Goetz.
>>