RFR: 8372942: AArch64: Set JVM flags for Neoverse V3AE core
Ruben
duke at openjdk.org
Tue Jan 13 14:24:53 UTC 2026
On Thu, 1 Jan 2026 10:02:02 GMT, Andrew Haley <aph at openjdk.org> wrote:
>> For Neoverse N1, N2, N3, V1, V2 and V3, the following JVM flags are set:
>> - UseSIMDForMemoryOps=true
>> - OnSpinWaitInst=isb
>> - OnSpinWaitInstCount=1
>> - AlwaysMergeDMB=false
>>
>> Additionally, for Neoverse V1, V2 and V3 only, these flags are set:
>> - UseCryptoPmullForCRC32=true
>> - CodeEntryAlignment=32
>>
>> Enable the same flags for Neoverse V3AE.
>
> Thanks, this is fine.
> I wonder if we should be thinking about replacing some of this open-coded logic with something more expressive and concise. This bunch of model_is() expressions could be a switch, for example.
Thank you for review, @theRealAph,
> I wonder if we should be thinking about replacing some of this open-coded logic with something more expressive and concise. This bunch of model_is() expressions could be a switch, for example.
While switch-case might not be easily applicable because we have two variables, both of which have to be compared with the values, perhaps an interface like `bool is_model_any_of(std::initializer_list list)` can simplify the code. Would this approach be suitable?
Would you like this to be changed within this PR?
-------------
PR Comment: https://git.openjdk.org/jdk/pull/28607#issuecomment-3744603433
More information about the hotspot-dev
mailing list