RFR: 8282312: Minor corrections to evbroadcasti32x4 intrinsic on x86
Dean Long
dlong at openjdk.java.net
Tue Mar 8 02:21:59 UTC 2022
On Mon, 7 Mar 2022 18:08:25 GMT, Jamil Nimeh <jnimeh at openjdk.org> wrote:
> Hello,
>
> This makes a small change to the evbroadcasti32x4 method to make it consistent with the Intel SDM for the vbroadcasti32x4 instruction. It sets the requirements to AVX512F rather than AVX512DQ, changes the Op/En field to be T4 instead of T2 and sets the width to 32-bits rather than 64.
>
> Hotspot tier1 through 4 tests passed. I also performed a 100,000 iteration run of the TestChecksum, TestCRC and TestCRC32 regression tests with intrinsics/diagnostics printing turned on so I could see that the methods that call the kernel_crc32_avx512 macro are invoked.
>
> JBS: https://bugs.openjdk.java.net/browse/JDK-8282312
Looks good.
-------------
Marked as reviewed by dlong (Reviewer).
PR: https://git.openjdk.java.net/jdk/pull/7732
More information about the hotspot-compiler-dev
mailing list