diff options
author | Craig Topper <craig.topper@gmail.com> | 2012-08-23 04:57:36 +0000 |
---|---|---|
committer | Craig Topper <craig.topper@gmail.com> | 2012-08-23 04:57:36 +0000 |
commit | 8a5bc5ad905bd4e5c0705fdcb0736316af6fdbe1 (patch) | |
tree | 94d523855a7222f8dcdeacb906f864a286a0c892 /lib/Target/X86/X86InstrInfo.cpp | |
parent | 71eab96bfd4d57a14105324cc0e0cac8eb3f7c8e (diff) | |
download | llvm-8a5bc5ad905bd4e5c0705fdcb0736316af6fdbe1.tar.gz llvm-8a5bc5ad905bd4e5c0705fdcb0736316af6fdbe1.tar.bz2 llvm-8a5bc5ad905bd4e5c0705fdcb0736316af6fdbe1.tar.xz |
Use a switch statement instead of a bunch of if-else checks and pull out the common function call.
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@162428 91177308-0d34-0410-b5e6-96231b3b80d8
Diffstat (limited to 'lib/Target/X86/X86InstrInfo.cpp')
-rw-r--r-- | lib/Target/X86/X86InstrInfo.cpp | 18 |
1 files changed, 10 insertions, 8 deletions
diff --git a/lib/Target/X86/X86InstrInfo.cpp b/lib/Target/X86/X86InstrInfo.cpp index 459f01a458..79eafd0233 100644 --- a/lib/Target/X86/X86InstrInfo.cpp +++ b/lib/Target/X86/X86InstrInfo.cpp @@ -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; |