From c108e7abdf25c7371cc23c07ea4e584efd33a73e Mon Sep 17 00:00:00 2001 From: Sandhya Viswanathan Date: Fri, 16 Apr 2021 21:28:30 +0000 Subject: [PATCH] 8265154: vinserti128 operand mix up for KNL platforms Reviewed-by: thartmann, kvn --- src/hotspot/cpu/x86/macroAssembler_x86.hpp | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/src/hotspot/cpu/x86/macroAssembler_x86.hpp b/src/hotspot/cpu/x86/macroAssembler_x86.hpp index e12cd168851..01fb91cace8 100644 --- a/src/hotspot/cpu/x86/macroAssembler_x86.hpp +++ b/src/hotspot/cpu/x86/macroAssembler_x86.hpp @@ -1414,7 +1414,7 @@ public: void vinserti128(XMMRegister dst, XMMRegister nds, XMMRegister src, uint8_t imm8) { if (UseAVX > 2 && VM_Version::supports_avx512novl()) { - Assembler::vinserti32x4(dst, dst, src, imm8); + Assembler::vinserti32x4(dst, nds, src, imm8); } else if (UseAVX > 1) { // vinserti128 is available only in AVX2 Assembler::vinserti128(dst, nds, src, imm8); @@ -1425,7 +1425,7 @@ public: void vinserti128(XMMRegister dst, XMMRegister nds, Address src, uint8_t imm8) { if (UseAVX > 2 && VM_Version::supports_avx512novl()) { - Assembler::vinserti32x4(dst, dst, src, imm8); + Assembler::vinserti32x4(dst, nds, src, imm8); } else if (UseAVX > 1) { // vinserti128 is available only in AVX2 Assembler::vinserti128(dst, nds, src, imm8);