maxjhandsome
/
llvm-project
forked from OSchip/llvm-project
1
0
Fork 0
Code Issues Pull Requests Packages Releases Wiki Activity

Use a switch statement instead of a bunch of if-else checks and pull out the common function call. 

llvm-svn: 162428
This commit is contained in:
Craig Topper 2012-08-23 04:57:36 +00:00
parent 9db4b0020c
commit f911597494
1 changed files with 10 additions and 8 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

18
llvm/lib/Target/X86/X86InstrInfo.cpp
View File

@ -3557,14 +3557,16 @@ X86InstrInfo::foldMemoryOperandImpl(MachineFunction &MF,
OpcodeTablePtr = &RegOp2MemOpTable2Addr;
isTwoAddrFold = true;
} else if (i == 0) { // If operand 0
if (MI->getOpcode() == X86::MOV64r0)
NewMI = MakeM0Inst(*this, X86::MOV64mi32, MOs, MI);
else if (MI->getOpcode() == X86::MOV32r0)
NewMI = MakeM0Inst(*this, X86::MOV32mi, MOs, MI);
else if (MI->getOpcode() == X86::MOV16r0)
NewMI = MakeM0Inst(*this, X86::MOV16mi, MOs, MI);
else if (MI->getOpcode() == X86::MOV8r0)
NewMI = MakeM0Inst(*this, X86::MOV8mi, MOs, MI);
unsigned Opc = 0;
switch (MI->getOpcode()) {
default: break;
case X86::MOV64r0: Opc = X86::MOV64mi32; break;
case X86::MOV32r0: Opc = X86::MOV32mi; break;
case X86::MOV16r0: Opc = X86::MOV16mi; break;
case X86::MOV8r0: Opc = X86::MOV8mi; break;
}
if (Opc)
NewMI = MakeM0Inst(*this, Opc, MOs, MI);
if (NewMI)
return NewMI;