RFR: 8296168: x86: Add reasonable constraints between AVX and SSE [v2]

Claes Redestad redestad at openjdk.org
Wed Nov 2 22:09:30 UTC 2022


On Wed, 2 Nov 2022 22:03:48 GMT, Vladimir Ivanov <vlivanov at openjdk.org> wrote:

>>> Should such ranges be narrowed down to the actual values supported?
>> 
>> Yes. 
>> 
>> At some point, `UseAVX` was set to `99` by default and then downgraded to the actual version. (BTW the same still happens for `UseSSE`.) I think the highest supported version should be used as the default instead.
>> 
>> Feel free to file a followup RFE or fix it right away.
>
> Also, as another cleanup idea, it makes sense to convert `UseSEE`/`UseAVX` from `intx` to `int` and get rid of `(int)` casts.

Might need a CSR since we'd upgrade from a warning to an error. I'll defer that to a follow-up.

-------------

PR: https://git.openjdk.org/jdk/pull/10946


More information about the hotspot-compiler-dev mailing list