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

[Dmitry Chuyko]

Derek, thanks for review and for the correction.

Yes, should be https://bugs.openjdk.java.net/browse/JDK-8198294

-Dmitry

On 10/31/18 9:16 PM, White, Derek wrote:
> 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