RFR(L): 8145117: PPC64: Remove cpp interpreter implementation

Lindenmaier, Goetz goetz.lindenmaier at sap.com
Tue Dec 15 13:14:32 UTC 2015


Hi Martin,

I added the comment.  Thanks for the review.

Best regards,
  Goetz.

> -----Original Message-----
> From: Doerr, Martin
> Sent: Dienstag, 15. Dezember 2015 12:49
> To: Coleen Phillimore <coleen.phillimore at oracle.com>; hotspot-runtime-
> dev at openjdk.java.net; Lindenmaier, Goetz <goetz.lindenmaier at sap.com>
> Subject: RE: RFR(L): 8145117: PPC64: Remove cpp interpreter implementation
> 
> Hi Götz,
> 
> thank you for doing this change. It looks good.
> The only thing I'm missing is the description about the frame layout which
> was in the CC_INTERP part, but also describes frames which we still have. It
> would be nice if you could change the comments in frame_ppc.hpp like this:
> 
>   //  Frame layout for the Java template interpreter on PPC64.
>   //
>   //  In these figures the stack grows upwards, while memory grows
>   //  downwards. Square brackets denote regions possibly larger than
>   //  single 64 bit slots.
>   //
>   //  STACK (interpreter is active):
>   //    0       [TOP_IJAVA_FRAME]
>   //            [PARENT_IJAVA_FRAME]
>   //            ...
>   //            [PARENT_IJAVA_FRAME]
>   //            [ENTRY_FRAME]
>   //            [C_FRAME]
>   //            ...
>   //            [C_FRAME]
>   //
>   //  With the following frame layouts:
>   //  TOP_IJAVA_FRAME:
>   //    0       [TOP_IJAVA_FRAME_ABI]
>   //            alignment (optional)
>   //            [operand stack]
>   //            [monitors] (optional)
>   //            [IJAVA_STATE]
>   //            note: own locals are located in the caller frame.
>   //
>   //  PARENT_IJAVA_FRAME:
>   //    0       [PARENT_IJAVA_FRAME_ABI]
>   //            alignment (optional)
>   //            [callee's Java result]
>   //            [callee's locals w/o arguments]
>   //            [outgoing arguments]
>   //            [used part of operand stack w/o arguments]
>   //            [monitors]      (optional)
>   //            [IJAVA_STATE]
>   //
>   //  ENTRY_FRAME:
>   //    0       [PARENT_IJAVA_FRAME_ABI]
>   //            alignment (optional)
>   //            [callee's Java result]
>   //            [callee's locals w/o arguments]
>   //            [outgoing arguments]
>   //            [ENTRY_FRAME_LOCALS]
> 
> 
> Thanks and best regards,
>   Martin
> 
> -----Original Message-----
> From: hotspot-runtime-dev [mailto:hotspot-runtime-dev-
> bounces at openjdk.java.net] On Behalf Of Coleen Phillimore
> Sent: Donnerstag, 10. Dezember 2015 21:51
> To: hotspot-runtime-dev at openjdk.java.net
> Subject: Re: RFR(L): 8145117: PPC64: Remove cpp interpreter
> implementation
> 
> 
> Hi Goetz,
> 
> This looks great!  I have all but the formatting changes in the change
> that I'm working on also, and have the equivalent changes to the other
> platforms.
> 
> There are shared code changes that #include cppInterpreter_ppc.cpp that
> may prevent Zero from being built on ppc.  I will remove these in my
> upcoming patch.
> 
> The ppc implementation of the cppInterpreter was the only one that
> seemed to be close to complete (or was the most recently working
> implementation).  The others will are more clear that they are bit
> rotten and should be removed.
> 
> Thanks!
> Coleen
> 
> 
> On 12/10/15 10:35 AM, Lindenmaier, Goetz wrote:
> > Hi,
> >
> > This change removed the cpp interpreter form the ppc sources.
> > It's broken anyways.  The change is ppc only.
> > http://cr.openjdk.java.net/~goetz/webrevs/8145117-ccInterp/webrev.00/
> >
> > Best regards,
> >    Goetz.
> >
> >



More information about the hotspot-runtime-dev mailing list