Skip to content

Commit 736edda

Browse files
author
Yuanke Luo
committed
Address Phoebe's comments
1 parent fd3e352 commit 736edda

File tree

1 file changed

+4
-4
lines changed

1 file changed

+4
-4
lines changed

llvm/lib/Target/X86/X86InstrInfo.cpp

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -6127,15 +6127,15 @@ static bool expandSHXDROT(MachineInstrBuilder &MIB, const MCInstrDesc &Desc) {
61276127
}
61286128

61296129
static bool expandMOVSHP(MachineInstrBuilder &MIB, MachineInstr &MI,
6130-
const TargetInstrInfo &TII, const X86Subtarget &STI) {
6130+
const TargetInstrInfo &TII, bool HasAVX) {
61316131
unsigned NewOpc;
61326132
if (MI.getOpcode() == X86::MOVSHPrm) {
6133-
NewOpc = STI.hasAVX() ? X86::VMOVSSrm : X86::MOVSSrm;
6133+
NewOpc = HasAVX ? X86::VMOVSSrm : X86::MOVSSrm;
61346134
Register Reg = MI.getOperand(0).getReg();
61356135
if (Reg > X86::XMM15)
61366136
NewOpc = X86::VMOVSSZrm;
61376137
} else {
6138-
NewOpc = STI.hasAVX() ? X86::VMOVSSmr : X86::MOVSSmr;
6138+
NewOpc = HasAVX ? X86::VMOVSSmr : X86::MOVSSmr;
61396139
Register Reg = MI.getOperand(5).getReg();
61406140
if (Reg > X86::XMM15)
61416141
NewOpc = X86::VMOVSSZmr;
@@ -6219,7 +6219,7 @@ bool X86InstrInfo::expandPostRAPseudo(MachineInstr &MI) const {
62196219
}
62206220
case X86::MOVSHPmr:
62216221
case X86::MOVSHPrm:
6222-
return expandMOVSHP(MIB, MI, *this, Subtarget);
6222+
return expandMOVSHP(MIB, MI, *this, Subtarget.hasAVX());
62236223
case X86::V_SETALLONES:
62246224
return Expand2AddrUndef(MIB,
62256225
get(HasAVX ? X86::VPCMPEQDrr : X86::PCMPEQDrr));

0 commit comments

Comments
 (0)