RFR (XL): 8019972: PPC64 (part 9): platform files for interpreter only VM.

[David Holmes]

Hi Goetz,

Only a brief glance through ...

I think orderAccess_linux_ppc.inline.hpp should have:

   34 #ifndef _LP64
   35 #error "Atomic currently only impleneted for PPC64"
   36 #endif

the same as in atomic_linux_ppc.inline.hpp (the jlong variants will only 
be atomic on ppc64).

BTW typo: 35 #error "Atomic currently only impleneted for PPC64"

I also find the ppc_ prefix used in the assembly code somewhat redundant.

David
-----

On 18/07/2013 1:34 AM, Lindenmaier, Goetz wrote:
> Hi,
>
> This time with webrev. Sorry for the double mails.
>
> This change contains all the files in cpu/ppc and os_cpu/linux_ppc needed for
> the PPC64 interpreter port on linux.
>
> With this change you can do a core build on ppc64 and run the VM interpreter only.
> It executes simple programs as jvm98.
> The change requires
>
> *         8016697: Use stubs to implement safefetch
>
> *         8020059: The flag introduced by 8014972 is not defined ...
> which will arrive soon in the staging repository.
>
> I marked the change as XL as it contains a lot of code.  Nevertheless the
> code has no impact on the existing Oracle platforms.
>
> The change touches a single shared file, globals.hpp, removing a
> special case introduced for the port.  This is because we
> integrated some changes earlier than originally intended.
>
> Please review the change.  Does it need testing on Oracle side?
> http://cr.openjdk.java.net/~goetz/webrevs/8019972-ppc_files/
>
> Best regards,
>    Goetz.
>