Integrated: 8265154: vinserti128 operand mix up for KNL platforms

[Sandhya Viswanathan]

On Wed, 14 Apr 2021 00:25:40 GMT, Sandhya Viswanathan <sviswanathan at openjdk.org> wrote:

> There is a bug in macro assembler in vinserti128 special handling for platforms like KNL that do not support AVX512VL.
>  
> The following:
>    void vinserti128(XMMRegister dst, XMMRegister nds, XMMRegister src, uint8_t imm8) {
>      if (UseAVX > 2 && VM_Version::supports_avx512novl()) {
>        Assembler::vinserti32x4(dst, dst, src, imm8);
>      }
>      ...
>   }
>  
> Should have been:
>    void vinserti128(XMMRegister dst, XMMRegister nds, XMMRegister src, uint8_t imm8) {
>      if (UseAVX > 2 && VM_Version::supports_avx512novl()) {
>        Assembler::vinserti32x4(dst, nds, src, imm8);
>      }
>     ...
>   }
> 
> Best Regards,
> Sandhya

This pull request has now been integrated.

Changeset: c108e7ab
Author:    Sandhya Viswanathan <sviswanathan at openjdk.org>
URL:       https://git.openjdk.java.net/jdk/commit/c108e7ab
Stats:     2 lines in 1 file changed: 0 ins; 0 del; 2 mod

8265154: vinserti128 operand mix up for KNL platforms

Reviewed-by: thartmann, kvn

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

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