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

[Lindenmaier, Goetz]

Hi Coleen, 

thanks for this hint!   We never added the optimization there
because it was hardly used when we had the cppInterpreter.  C2
doesn't call it, and there is no C1.  But now it should be there with
us having ported the template interpreter, and the C1 port is 
planned for a long time now ;)

Unfortunately it's not easy to add in the decodes, where it's most
useful, as it requires that src and dst registers are different.
But I'll implement it for the encode and make a new webrev
after testing it tonight. 

Best regards,
  Goetz.


-----Original Message-----
From: hotspot-dev [mailto:hotspot-dev-bounces at openjdk.java.net] On Behalf Of Coleen Phillimore
Sent: Mittwoch, 7. Januar 2015 15:54
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.
>