Integrated: 8296168: x86: Add reasonable constraints between AVX and SSE

Claes Redestad redestad at openjdk.org
Thu Nov 3 12:09:34 UTC 2022


On Wed, 2 Nov 2022 12:44:42 GMT, Claes Redestad <redestad at openjdk.org> wrote:

> We've not seen any x86 CPU, Intel or otherwise, where AVX features are available but SSE 4.1 is not supported. This patch suggests constraining setup so that any explicit value of UseSSE less than 4 (the default on any AVX-supporting CPU) implicitly disables AVX. This simplifies ergonomics and reduces the testing surface. Concretely this would allow #10847 to not have to guard the new intrinsic on UseSSE level to avoid some surprising test failures in tests verifying SSE-enabled intrinsics.
> 
> I've rearranged the initialization of UseAVX and UseSSE to allow AVX to look at the post-ergo values of UseSSE.
> 
> Testing: tier1-tier3, manual verification

This pull request has now been integrated.

Changeset: 6ee8ccfc
Author:    Claes Redestad <redestad at openjdk.org>
URL:       https://git.openjdk.org/jdk/commit/6ee8ccfcfea06b16383475f9bbef11753e7fcc22
Stats:     63 lines in 1 file changed: 31 ins; 30 del; 2 mod

8296168: x86: Add reasonable constraints between AVX and SSE

Reviewed-by: kvn, vlivanov

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

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


More information about the hotspot-compiler-dev mailing list