Skip to content

Commit

Permalink
8265154: vinserti128 operand mix up for KNL platforms
Browse files Browse the repository at this point in the history
Reviewed-by: thartmann, kvn
  • Loading branch information
Sandhya Viswanathan committed Apr 16, 2021
1 parent 2c4075c commit c108e7a
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions src/hotspot/cpu/x86/macroAssembler_x86.hpp
Original file line number Diff line number Diff line change
Expand Up @@ -1414,7 +1414,7 @@ class MacroAssembler: public Assembler {

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);
Expand All @@ -1425,7 +1425,7 @@ class MacroAssembler: public Assembler {

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);
Expand Down

1 comment on commit c108e7a

@openjdk-notifier
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please sign in to comment.