[12] RFR(XXS): JDK-8196950: AARCH64 - Add VM flags presets for Cavium Thunder X2 CPU

[White, Derek]

Hi Dmitry,

Looks good to me.

Do you want to fix this under JDK-8198294 instead?

 - Derek

> -----Original Message-----
> From: hotspot-compiler-dev <hotspot-compiler-dev-
> bounces at openjdk.java.net> On Behalf Of Dmitry Chuyko
> Sent: Wednesday, October 31, 2018 1:41 PM
> To: hotspot-compiler-dev at openjdk.java.net
> Subject: [12] RFR(XXS): JDK-8196950: AARCH64 - Add VM flags presets for
> Cavium Thunder X2 CPU
> 
> External Email
> 
> Hello,
> 
> Please review small defaults correction.
> 
> rfe: https://bugs.openjdk.java.net/browse/JDK-8196950
> code:
> 
> diff -r 16950b2eaebf src/hotspot/cpu/aarch64/vm_version_aarch64.cpp
> --- a/src/hotspot/cpu/aarch64/vm_version_aarch64.cpp    Tue Oct 30
> 09:13:00 2018 -0400
> +++ b/src/hotspot/cpu/aarch64/vm_version_aarch64.cpp    Wed Oct 31
> 20:33:41 2018 +0300
> @@ -219,7 +219,7 @@
>       }
>   #ifdef COMPILER2
>       if (FLAG_IS_DEFAULT(UseFPUForSpilling)) {
> -      FLAG_SET_DEFAULT(UseFPUForSpilling, true);
> +      FLAG_SET_DEFAULT(UseFPUForSpilling, false);
>       }
>   #endif
>     }
> 
> Having "-XX:+AvoidUnalignedAccesses -XX:+UseSIMDForMemoryOps -XX:-
> UseFPUForSpilling" as a default works (in general) better for SPECjvm and
> SPECjbb.
> 
> -Dmitry