Remove support for Not ConstantExpr. This simplifies the unary case to only

[oota-llvm.git] / lib / VMCore / Instruction.cpp

diff --git a/lib/VMCore/Instruction.cpp b/lib/VMCore/Instruction.cpp
index d5fe1c68e3e1b1dd06d7ea560068f985fba26a7a..e6ff3d28d2d1321bb42bf575493364fb528ed705 100644 (file)
--- a/lib/VMCore/Instruction.cpp
+++ b/lib/VMCore/Instruction.cpp
@@ -4,55 +4,74 @@
 //
 //===----------------------------------------------------------------------===//
 
-#include "llvm/Instruction.h"
-#include "llvm/BasicBlock.h"
-#include "llvm/Method.h"
+#include "llvm/Function.h"
 #include "llvm/SymbolTable.h"
-#include "llvm/Codegen/MachineInstr.h"
+#include "llvm/Type.h"
 
-Instruction::Instruction(const Type *ty, unsigned it, const string &Name) 
-  : User(ty, Value::InstructionVal, Name),
-    machineInstrVec(new MachineCodeForVMInstr)
-{
+Instruction::Instruction(const Type *ty, unsigned it, const std::string &Name) 
+  : User(ty, Value::InstructionVal, Name) {
   Parent = 0;
   iType = it;
 }
 
-Instruction::~Instruction() {
-  assert(getParent() == 0 && "Instruction still embedded in basic block!");
-  delete machineInstrVec;
-}
-
 // Specialize setName to take care of symbol table majik
-void Instruction::setName(const string &name) {
-  BasicBlock *P = 0; Method *PP = 0;
+void Instruction::setName(const std::string &name, SymbolTable *ST) {
+  BasicBlock *P = 0; Function *PP = 0;
+  assert((ST == 0 || !getParent() || !getParent()->getParent() || 
+         ST == getParent()->getParent()->getSymbolTable()) &&
+        "Invalid symtab argument!");
   if ((P = getParent()) && (PP = P->getParent()) && hasName())
     PP->getSymbolTable()->remove(this);
   Value::setName(name);
   if (PP && hasName()) PP->getSymbolTableSure()->insert(this);
 }
 
-void
-Instruction::addMachineInstruction(MachineInstr* minstr)
-{
-  machineInstrVec->push_back(minstr);
-}
 
-#if 0
-// Dont make this inline because you would need to include
-// MachineInstr.h in Instruction.h, which creates a circular
-// sequence of forward declarations.  Trying to fix that will
-// cause a serious circularity in link order.
-// 
-const vector<Value*>&
-Instruction::getTempValuesForMachineCode() const
-{
-  return machineInstrVec->getTempValues();
-}
-#endif
+const char *Instruction::getOpcodeName(unsigned OpCode) {
+  switch (OpCode) {
+  // Terminators
+  case Ret:    return "ret";
+  case Br:     return "br";
+  case Switch: return "switch";
+  case Invoke: return "invoke";
+    
+  // Standard binary operators...
+  case Add: return "add";
+  case Sub: return "sub";
+  case Mul: return "mul";
+  case Div: return "div";
+  case Rem: return "rem";
+
+  // Logical operators...
+  case And: return "and";
+  case Or : return "or";
+  case Xor: return "xor";
 
-void
-Instruction::dropAllReferences() {
-  machineInstrVec->dropAllReferences();
-  User::dropAllReferences();
+  // SetCC operators...
+  case SetLE:  return "setle";
+  case SetGE:  return "setge";
+  case SetLT:  return "setlt";
+  case SetGT:  return "setgt";
+  case SetEQ:  return "seteq";
+  case SetNE:  return "setne";
+    
+  // Memory instructions...
+  case Malloc:        return "malloc";
+  case Free:          return "free";
+  case Alloca:        return "alloca";
+  case Load:          return "load";
+  case Store:         return "store";
+  case GetElementPtr: return "getelementptr";
+    
+  // Other instructions...
+  case PHINode: return "phi";
+  case Cast:    return "cast";
+  case Call:    return "call";
+  case Shl:     return "shl";
+  case Shr:     return "shr";
+    
+  default: return "<Invalid operator> ";
+  }
+  
+  return 0;
 }