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

[Brian Gaeke]

Changes in directory llvm/lib/Target:

TargetInstrInfo.cpp updated: 1.7 -> 1.8

---
Log message:

Nice tasty llc fixes. These should fix LLC for x86 for everything in
SingleSource except oopack and Oscar.  (Sorry, Oscar.)

include/llvm/Target/TargetInstrInfo.h: Remove virtual print method. Add
 accessors for ImplicitUses/Defs.
lib/Target/TargetInstrInfo.cpp: Remove virtual print method. If you
 really wanted this, just use MI->print(O, TM); instead...
lib/Target/X86:
FloatingPoint.cpp: ...like this.
X86InstrInfo.h: Remove virtual print method. Define the PrintImplUses
 target-specific flag bit.
X86InstrInfo.def: Add the PrintImplUses flag to all the instructions
 which implicitly use CL, because the assembler needs to see the CL in
 order to generate the right instruction.
Printer.cpp: Ditch fnIndex at Chris's request. Now we use CurrentFnName
  to name constants in the constant pool for each function instead. This
  avoids keeping state between runOnMachineFunction() invocations, which
  is a no-no. Having MangledGlobals be global is a bogon I'd like to get
  rid of too, but making it a static member of Printer causes link errors
  (why???).
 Make NumberForBB into a member of Printer instead of a global, too.
 Make printOp and printMemReference into methods of Printer.
 X86InstrInfo::print is now Printer::printMachineInstruction, because
  TargetInstrInfo::print is history. (Because of this, we have to qualify
  the names of some TargetInstrInfo methods we call.)
 Print out the ImplicitUses field of any instruction we print that has
  the PrintImplUses bit set. 


---
Diffs of the changes:

Index: llvm/lib/Target/TargetInstrInfo.cpp
diff -u llvm/lib/Target/TargetInstrInfo.cpp:1.7 llvm/lib/Target/TargetInstrInfo.cpp:1.8
--- llvm/lib/Target/TargetInstrInfo.cpp:1.7	Tue Jan 14 15:59:13 2003
+++ llvm/lib/Target/TargetInstrInfo.cpp	Thu Jun 26 19:00:41 2003
@@ -28,11 +28,6 @@
   TargetInstrDescriptors = NULL;	// reset global variable
 }
 
-void TargetInstrInfo::print(const MachineInstr *MI, std::ostream &O,
-			    const TargetMachine &TM) const {
-  MI->print(O, TM);
-}
-
 bool TargetInstrInfo::constantFitsInImmedField(MachineOpCode opCode,
 					       int64_t intValue) const {
   // First, check if opCode has an immed field.