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

[Brian Gaeke]

Changes in directory llvm/lib/Target/X86:

Printer.cpp updated: 1.35 -> 1.36

---
Log message:

Add Initialization and Finalization methods for the Printer pass,
to print various things on a module-by-module basis (currently, only the
former is used).

Don't print < > around names. The assembler can't take it.

Print pseudoinstructions only as comments. The poor little assembler can't
take that, either.


---
Diffs of the changes:

Index: llvm/lib/Target/X86/Printer.cpp
diff -u llvm/lib/Target/X86/Printer.cpp:1.35 llvm/lib/Target/X86/Printer.cpp:1.36
--- llvm/lib/Target/X86/Printer.cpp:1.35	Mon May 26 19:03:17 2003
+++ llvm/lib/Target/X86/Printer.cpp	Thu Jun 19 14:32:32 2003
@@ -27,6 +27,10 @@
 
     void printConstantPool(MachineConstantPool *MCP, const TargetData &TD);
     bool runOnMachineFunction(MachineFunction &F);
+    
+    bool doInitialization(Module &M);
+    bool doFinalization(Module &M);
+
   };
 }
 
@@ -123,13 +127,13 @@
     O << (int)MO.getImmedValue();
     return;
   case MachineOperand::MO_PCRelativeDisp:
-    O << "<" << MO.getVRegValue()->getName() << ">";
+    O << MO.getVRegValue()->getName();
     return;
   case MachineOperand::MO_GlobalAddress:
-    O << "<" << MO.getGlobal()->getName() << ">";
+    O << MO.getGlobal()->getName();
     return;
   case MachineOperand::MO_ExternalSymbol:
-    O << "<" << MO.getSymbolName() << ">";
+    O << MO.getSymbolName();
     return;
   default:
     O << "<unknown op ty>"; return;    
@@ -208,6 +212,10 @@
 
   switch (Desc.TSFlags & X86II::FormMask) {
   case X86II::Pseudo:
+    // Print pseudo-instructions as comments; either they should have been
+    // turned into real instructions by now, or they don't need to be
+    // seen by the assembler (e.g., IMPLICIT_USEs.)
+    O << "# ";
     if (Opcode == X86::PHI) {
       printOp(O, MI->getOperand(0), RI);
       O << " = phi ";
@@ -457,3 +465,18 @@
     O << "\tUNKNOWN FORM:\t\t-"; MI->print(O, TM); break;
   }
 }
+
+bool Printer::doInitialization(Module &M)
+{
+  // Tell gas we are outputting Intel syntax (not AT&T syntax) assembly,
+  // with no % decorations on register names.
+  O << "\t.intel_syntax noprefix\n";
+  return false; // success
+}
+
+bool Printer::doFinalization(Module &M)
+{
+  // FIXME: We may have to print out constants here.
+  return false; // success
+}
+