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