commit 161476ec343161c6c6ab16df775d853fcecd77c3
author Bruno Cardoso Lopes <bruno.cardoso@gmail.com> Fri Jun 25 23:33:42 2010 +0000
committer Bruno Cardoso Lopes <bruno.cardoso@gmail.com> Fri Jun 25 23:33:42 2010 +0000
tree 7b97983bc05a46fc13e5986a16d3275431d8aa15
parent e39e06af38281bb8923d28b554a11a74e4eb67b7

Reapply r106896:
Add several AVX MOV flavors
Support VEX encoding for MRMDestReg



git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@106912 91177308-0d34-0410-b5e6-96231b3b80d8

lib/Target/X86/X86MCCodeEmitter.cpp

diff --git a/lib/Target/X86/X86MCCodeEmitter.cpp b/lib/Target/X86/X86MCCodeEmitter.cpp
index 2223d54..3b9b20a 100644
--- a/lib/Target/X86/X86MCCodeEmitter.cpp
+++ b/lib/Target/X86/X86MCCodeEmitter.cpp
@@ -431,7 +431,6 @@
 
   switch (TSFlags & X86II::Op0Mask) {
   default: assert(0 && "Invalid prefix!");
-  case 0: break;  // No prefix!
   case X86II::T8:  // 0F 38
     VEX_5M = 0x2;
     break;
@@ -448,21 +447,29 @@
   case X86II::XD:  // F2 0F
     VEX_PP = 0x3;
     break;
+  case X86II::TB:  // Bypass: Not used by VEX
+  case 0:
+    break;  // No prefix!
   }
 
   unsigned NumOps = MI.getNumOperands();
-  unsigned i = 0, CurOp = 0;
-  bool IsSrcMem = false;
+  unsigned CurOp = 0;
+
+  if ((TSFlags & X86II::FormMask) == X86II::MRMDestMem)
+    NumOps = CurOp = X86AddrNumOperands;
 
   switch (TSFlags & X86II::FormMask) {
   case X86II::MRMInitReg: assert(0 && "FIXME: Remove this!");
   case X86II::MRMSrcMem:
-    IsSrcMem = true;
+  case X86II::MRMDestMem:
   case X86II::MRMSrcReg:
     if (MI.getOperand(CurOp).isReg() &&
         X86InstrInfo::isX86_64ExtendedReg(MI.getOperand(CurOp).getReg()))
       VEX_R = 0x0;
-    CurOp++;
+
+    // If the memory destination has been checked first,
+    // go back to the first operand
+    CurOp = (CurOp == NumOps) ? 0 : CurOp+1;
 
     // On regular x86, both XMM0-XMM7 and XMM8-XMM15 are encoded in the
     // range 0-7 and the difference between the 2 groups is given by the
@@ -486,12 +493,12 @@
       CurOp++;
     }
 
-    i = CurOp;
-    for (; i != NumOps; ++i) {
-      const MCOperand &MO = MI.getOperand(i);
+    for (; CurOp != NumOps; ++CurOp) {
+      const MCOperand &MO = MI.getOperand(CurOp);
       if (MO.isReg() && X86InstrInfo::isX86_64ExtendedReg(MO.getReg()))
         VEX_B = 0x0;
-      if (!VEX_B && MO.isReg() && IsSrcMem &&
+      if (!VEX_B && MO.isReg() &&
+          ((TSFlags & X86II::FormMask) == X86II::MRMSrcMem) &&
           X86InstrInfo::isX86_64ExtendedReg(MO.getReg()))
         VEX_X = 0x0;
     }