commit c108e7abdf25c7371cc23c07ea4e584efd33a73e
author Sandhya Viswanathan <sviswanathan@openjdk.org> Fri Apr 16 21:28:30 2021 +0000
committer Sandhya Viswanathan <sviswanathan@openjdk.org> Fri Apr 16 21:28:30 2021 +0000
tree 3597b8af54334fff2a95056d4ec12f30aea46aec
parent 2c4075cbb71b9f9566b1d3878e6b5a7b6d2a1aa1

8265154: vinserti128 operand mix up for KNL platforms

Reviewed-by: thartmann, kvn

src/hotspot/cpu/x86/macroAssembler_x86.hpp

diff --git a/src/hotspot/cpu/x86/macroAssembler_x86.hpp b/src/hotspot/cpu/x86/macroAssembler_x86.hpp
index e12cd16..01fb91c 100644
--- a/src/hotspot/cpu/x86/macroAssembler_x86.hpp
+++ b/src/hotspot/cpu/x86/macroAssembler_x86.hpp
@@ -1414,7 +1414,7 @@
 
   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 @@
 
   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);