From: Daniel Dunbar <daniel@zuster.org>
Date: Tue, 2 Feb 2010 21:44:10 +0000 (+0000)
Subject: MCCodeEmitter/X86: Handle tied registers better when converting MCInst ->
X-Git-Url: http://demsky.eecs.uci.edu/git/?a=commitdiff_plain;h=1945e179e1ece0a521d394f5f9ae9706f0ec4f90;p=oota-llvm.git

MCCodeEmitter/X86: Handle tied registers better when converting MCInst ->
MCMachineInstr. This also fixes handling of tied registers for MRMSrcMem
instructions.

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

diff --git a/lib/Target/X86/X86CodeEmitter.cpp b/lib/Target/X86/X86CodeEmitter.cpp
index e2c31395b20..561558ba3e7 100644
--- a/lib/Target/X86/X86CodeEmitter.cpp
+++ b/lib/Target/X86/X86CodeEmitter.cpp
@@ -1002,10 +1002,10 @@ public:
     unsigned Opcode = MI.getOpcode();
     unsigned NumOps = MI.getNumOperands();
     unsigned CurOp = 0;
-    if (NumOps > 1 && Desc.getOperandConstraint(1, TOI::TIED_TO) != -1) {
-      Instr->addOperand(MachineOperand::CreateReg(0, false));
-      ++CurOp;
-    } else if (NumOps > 2 && 
+    bool AddTied = false;
+    if (NumOps > 1 && Desc.getOperandConstraint(1, TOI::TIED_TO) != -1)
+      AddTied = true;
+    else if (NumOps > 2 && 
              Desc.getOperandConstraint(NumOps-1, TOI::TIED_TO)== 0)
       // Skip the last source operand that is tied_to the dest reg. e.g. LXADD32
       --NumOps;
@@ -1016,7 +1016,9 @@ public:
     case X86II::MRMSrcReg:
       // Matching doesn't fill this in completely, we have to choose operand 0
       // for a tied register.
-      OK &= AddRegToInstr(MI, Instr, 0); CurOp++;
+      OK &= AddRegToInstr(MI, Instr, CurOp++);
+      if (AddTied)
+        OK &= AddRegToInstr(MI, Instr, CurOp++ - 1);
       OK &= AddRegToInstr(MI, Instr, CurOp++);
       if (CurOp < NumOps)
         OK &= AddImmToInstr(MI, Instr, CurOp);
@@ -1035,7 +1037,11 @@ public:
       break;
 
     case X86II::AddRegFrm:
+      // Matching doesn't fill this in completely, we have to choose operand 0
+      // for a tied register.
       OK &= AddRegToInstr(MI, Instr, CurOp++);
+      if (AddTied)
+        OK &= AddRegToInstr(MI, Instr, CurOp++ - 1);
       if (CurOp < NumOps)
         OK &= AddImmToInstr(MI, Instr, CurOp);
       break;
@@ -1046,7 +1052,9 @@ public:
     case X86II::MRM6r: case X86II::MRM7r:
       // Matching doesn't fill this in completely, we have to choose operand 0
       // for a tied register.
-      OK &= AddRegToInstr(MI, Instr, 0); CurOp++;
+      OK &= AddRegToInstr(MI, Instr, CurOp++);
+      if (AddTied)
+        OK &= AddRegToInstr(MI, Instr, CurOp++ - 1);
       if (CurOp < NumOps)
         OK &= AddImmToInstr(MI, Instr, CurOp);
       break;
@@ -1061,7 +1069,11 @@ public:
       break;
 
     case X86II::MRMSrcMem:
+      // Matching doesn't fill this in completely, we have to choose operand 0
+      // for a tied register.
       OK &= AddRegToInstr(MI, Instr, CurOp++);
+      if (AddTied)
+        OK &= AddRegToInstr(MI, Instr, CurOp++ - 1);
       if (Opcode == X86::LEA64r || Opcode == X86::LEA64_32r ||
           Opcode == X86::LEA16r || Opcode == X86::LEA32r)
         OK &= AddLMemToInstr(MI, Instr, CurOp);