Integrated: 8279668: x86: AVX2 versions of vpxor should be asserted

Aleksey Shipilev shade at openjdk.java.net
Tue Jan 11 10:31:33 UTC 2022


On Mon, 10 Jan 2022 10:31:30 GMT, Aleksey Shipilev <shade at openjdk.org> wrote:

> Got the SIGILLs on some machines when testing JDK-8279621. That patch started using shorter `vpxor` versions on `UseAVX = 1` path, which tried to use `VEX.256`-encoded `vpxor` instruction that is only available on AVX2. This should be at very least asserted in assembler code. 
> 
> Also fixed the implicit `bool` -> `int` conversion to `vector_len`.
> 
> Additional testing:
>  - [x] Linux x86_64 fastdebug `tier1` with `-XX:UseAVX=1`

This pull request has now been integrated.

Changeset: 2bbeae3f
Author:    Aleksey Shipilev <shade at openjdk.org>
URL:       https://git.openjdk.java.net/jdk/commit/2bbeae3f056243a224b0bda021f16cdcbee3b3d6
Stats:     15 lines in 2 files changed: 12 ins; 0 del; 3 mod

8279668: x86: AVX2 versions of vpxor should be asserted

Reviewed-by: kvn, jiefu

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

PR: https://git.openjdk.java.net/jdk/pull/7005


More information about the hotspot-dev mailing list