From: Chris Lattner <sabre@nondot.org>
Date: Thu, 3 Sep 2009 04:44:53 +0000 (+0000)
Subject: implement lowering support for constant pool index operands, this gets a bunch more
X-Git-Url: http://plrg.eecs.uci.edu/git/?a=commitdiff_plain;h=118c27c6d4211401a97183d642f823460e86a854;p=oota-llvm.git

implement lowering support for constant pool index operands, this gets a bunch more
olden programs working.


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

diff --git a/lib/Target/X86/AsmPrinter/X86ATTAsmPrinter.h b/lib/Target/X86/AsmPrinter/X86ATTAsmPrinter.h
index ada3ef5ca24..0dd422f52e4 100644
--- a/lib/Target/X86/AsmPrinter/X86ATTAsmPrinter.h
+++ b/lib/Target/X86/AsmPrinter/X86ATTAsmPrinter.h
@@ -71,9 +71,11 @@ class VISIBILITY_HIDDEN X86ATTAsmPrinter : public AsmPrinter {
   // New MCInst printing stuff.
   void printInstruction(const MCInst *MI);
   MCSymbol *GetPICBaseSymbol();
-  MCOperand LowerJumpTableOperand(const MachineOperand &MO);
   MCOperand LowerGlobalAddressOperand(const MachineOperand &MO);
   MCOperand LowerExternalSymbolOperand(const MachineOperand &MO);
+  MCOperand LowerJumpTableOperand(const MachineOperand &MO);
+  MCOperand LowerConstantPoolIndexOperand(const MachineOperand &MO);
+
 
   virtual void printMCInst(const MCInst *MI) { printInstruction(MI); }
 
diff --git a/lib/Target/X86/AsmPrinter/X86MCInstLower.cpp b/lib/Target/X86/AsmPrinter/X86MCInstLower.cpp
index 77b0d359331..b55026eaefb 100644
--- a/lib/Target/X86/AsmPrinter/X86MCInstLower.cpp
+++ b/lib/Target/X86/AsmPrinter/X86MCInstLower.cpp
@@ -163,18 +163,20 @@ LowerExternalSymbolOperand(const MachineOperand &MO) {
 MCOperand X86ATTAsmPrinter::LowerJumpTableOperand(const MachineOperand &MO) {
   SmallString<256> Name;
   raw_svector_ostream(Name) << MAI->getPrivateGlobalPrefix() << "JTI"
-  << getFunctionNumber() << '_' << MO.getIndex();
+    << getFunctionNumber() << '_' << MO.getIndex();
   
   MCSymbol *NegatedSymbol = 0;
   switch (MO.getTargetFlags()) {
-    default:
-      llvm_unreachable("Unknown target flag on GV operand");
-    case X86II::MO_PIC_BASE_OFFSET:
-    case X86II::MO_DARWIN_NONLAZY_PIC_BASE:
-    case X86II::MO_DARWIN_HIDDEN_NONLAZY_PIC_BASE:
-      // Subtract the pic base.
-      NegatedSymbol = GetPICBaseSymbol();
-      break;
+  default:
+    llvm_unreachable("Unknown target flag on GV operand");
+  case X86II::MO_NO_FLAG:    // No flag.
+    break;
+  case X86II::MO_PIC_BASE_OFFSET:
+  case X86II::MO_DARWIN_NONLAZY_PIC_BASE:
+  case X86II::MO_DARWIN_HIDDEN_NONLAZY_PIC_BASE:
+    // Subtract the pic base.
+    NegatedSymbol = GetPICBaseSymbol();
+    break;
   }
   
   // Create a symbol for the name.
@@ -190,6 +192,38 @@ MCOperand X86ATTAsmPrinter::LowerJumpTableOperand(const MachineOperand &MO) {
 }
 
 
+MCOperand X86ATTAsmPrinter::
+LowerConstantPoolIndexOperand(const MachineOperand &MO) {
+  SmallString<256> Name;
+  raw_svector_ostream(Name) << MAI->getPrivateGlobalPrefix() << "CPI"
+  << getFunctionNumber() << '_' << MO.getIndex();
+  
+  MCSymbol *NegatedSymbol = 0;
+  switch (MO.getTargetFlags()) {
+  default:
+    llvm_unreachable("Unknown target flag on GV operand");
+  case X86II::MO_NO_FLAG:    // No flag.
+    break;
+  case X86II::MO_PIC_BASE_OFFSET:
+  case X86II::MO_DARWIN_NONLAZY_PIC_BASE:
+  case X86II::MO_DARWIN_HIDDEN_NONLAZY_PIC_BASE:
+    // Subtract the pic base.
+    NegatedSymbol = GetPICBaseSymbol();
+    break;
+  }
+  
+  // Create a symbol for the name.
+  MCSymbol *Sym = OutContext.GetOrCreateSymbol(Name.str());
+  // FIXME: We would like an efficient form for this, so we don't have to do a
+  // lot of extra uniquing.
+  const MCExpr *Expr = MCSymbolRefExpr::Create(Sym, OutContext);
+  if (NegatedSymbol)
+    Expr = MCBinaryExpr::CreateSub(Expr, MCSymbolRefExpr::Create(NegatedSymbol,
+                                                                 OutContext),
+                                   OutContext);
+  return MCOperand::CreateExpr(Expr);
+}
+
 void X86ATTAsmPrinter::
 printInstructionThroughMCStreamer(const MachineInstr *MI) {
   
@@ -246,9 +280,6 @@ printInstructionThroughMCStreamer(const MachineInstr *MI) {
       O.flush();
       errs() << "Cannot lower operand #" << i << " of :" << *MI;
       llvm_unreachable("Unimp");
-    case MachineOperand::MO_JumpTableIndex:
-      MCOp = LowerJumpTableOperand(MO);
-      break;
     case MachineOperand::MO_Register:
       MCOp = MCOperand::CreateReg(MO.getReg());
       break;
@@ -265,6 +296,12 @@ printInstructionThroughMCStreamer(const MachineInstr *MI) {
     case MachineOperand::MO_ExternalSymbol:
       MCOp = LowerExternalSymbolOperand(MO);
       break;
+    case MachineOperand::MO_JumpTableIndex:
+      MCOp = LowerJumpTableOperand(MO);
+      break;
+    case MachineOperand::MO_ConstantPoolIndex:
+      MCOp = LowerConstantPoolIndexOperand(MO);
+      break;
     }
     
     TmpInst.addOperand(MCOp);