RFR: 8296168: x86: Add reasonable constraints between AVX and SSE
Vladimir Ivanov
vlivanov at openjdk.org
Wed Nov 2 17:29:23 UTC 2022
On Wed, 2 Nov 2022 17:12:35 GMT, Vladimir Ivanov <vlivanov at openjdk.org> wrote:
>> src/hotspot/cpu/x86/vm_version_x86.cpp line 1041:
>>
>>> 1039: }
>>> 1040: } else {
>>> 1041: if (UseSSE > 3) {
>>
>> We need an other RFE to clean this up. We should disable CPU features according UseSSE value as we already do for UseAVX.
>> For these changes this is fine.
>
> It's already the case for `UseSSE`:
> https://github.com/openjdk/jdk/blob/master/src/hotspot/cpu/x86/vm_version_x86.cpp#L858
I find the original check clearer: it says that `UseAESCTRIntrinsics`-related code depends specifically on sse4.1. Please, leave it as is.
-------------
PR: https://git.openjdk.org/jdk/pull/10946
More information about the hotspot-compiler-dev
mailing list