[llvm-commits] CVS: llvm/lib/Target/X86/X86RegisterInfo.cpp

[Misha Brukman]

Changes in directory llvm/lib/Target/X86:

X86RegisterInfo.cpp updated: 1.9 -> 1.10

---
Log message:

Moves now select correct opcode based on the data size.


---
Diffs of the changes:

Index: llvm/lib/Target/X86/X86RegisterInfo.cpp
diff -u llvm/lib/Target/X86/X86RegisterInfo.cpp:1.9 llvm/lib/Target/X86/X86RegisterInfo.cpp:1.10
--- llvm/lib/Target/X86/X86RegisterInfo.cpp:1.9	Wed Dec  4 17:57:03 2002
+++ llvm/lib/Target/X86/X86RegisterInfo.cpp	Thu Dec 12 22:24:53 2002
@@ -23,7 +23,6 @@
   : MRegisterInfo(X86Regs, sizeof(X86Regs)/sizeof(X86Regs[0])) {
 }
 
-
 MachineBasicBlock::iterator
 X86RegisterInfo::storeReg2RegOffset(MachineBasicBlock *MBB,
                                     MachineBasicBlock::iterator MBBI,
@@ -31,7 +30,15 @@
                                     unsigned ImmOffset, unsigned dataSize)
   const
 {
-  MachineInstr *MI = addRegOffset(BuildMI(X86::MOVrm32, 5),
+  unsigned opcode;
+  switch (dataSize) {
+  case 1: opcode = X86::MOVrm8; break;
+  case 2: opcode = X86::MOVrm16; break;
+  case 4: opcode = X86::MOVrm32; break;
+  default: assert(0 && "Invalid data size!");
+  }
+
+  MachineInstr *MI = addRegOffset(BuildMI(opcode, 5),
                                   DestReg, ImmOffset).addReg(SrcReg);
   return ++(MBB->insert(MBBI, MI));
 }
@@ -43,7 +50,15 @@
                                    unsigned ImmOffset, unsigned dataSize)
   const
 {
-  MachineInstr *MI = addRegOffset(BuildMI(X86::MOVmr32, 5).addReg(DestReg),
+  unsigned opcode;
+  switch (dataSize) {
+  case 1: opcode = X86::MOVmr8; break;
+  case 2: opcode = X86::MOVmr16; break;
+  case 4: opcode = X86::MOVmr32; break;
+  default: assert(0 && "Invalid data size!");
+  }
+
+  MachineInstr *MI = addRegOffset(BuildMI(opcode, 5).addReg(DestReg),
                                   SrcReg, ImmOffset);
   return ++(MBB->insert(MBBI, MI));
 }