Implement ConstantExprs in CWriter
[oota-llvm.git] / lib / Target / CBackend / CBackend.cpp
index dd5f0b1eb52e706b0bbbcaefe96ee71533366c97..c706d089ba244188520241defb3d6081d495c6f8 100644 (file)
 #include "llvm/Constants.h"
 #include "llvm/DerivedTypes.h"
 #include "llvm/Module.h"
-#include "llvm/GlobalVariable.h"
-#include "llvm/Function.h"
-#include "llvm/Argument.h"
-#include "llvm/BasicBlock.h"
 #include "llvm/iMemory.h"
 #include "llvm/iTerminators.h"
 #include "llvm/iPHINode.h"
@@ -31,179 +27,6 @@ using std::string;
 using std::map;
 using std::ostream;
 
-static std::string getConstStrValue(const Constant* CPV);
-
-
-static std::string getConstArrayStrValue(const Constant* CPV) {
-  std::string Result;
-  
-  // As a special case, print the array as a string if it is an array of
-  // ubytes or an array of sbytes with positive values.
-  // 
-  const Type *ETy = cast<ArrayType>(CPV->getType())->getElementType();
-  bool isString = (ETy == Type::SByteTy || ETy == Type::UByteTy);
-
-  // Make sure the last character is a null char, as automatically added by C
-  if (CPV->getNumOperands() == 0 ||
-      !cast<Constant>(*(CPV->op_end()-1))->isNullValue())
-    isString = false;
-  
-  if (isString) {
-    Result = "\"";
-    // Do not include the last character, which we know is null
-    for (unsigned i = 0, e = CPV->getNumOperands()-1; i != e; ++i) {
-      unsigned char C = (ETy == Type::SByteTy) ?
-        (unsigned char)cast<ConstantSInt>(CPV->getOperand(i))->getValue() :
-        (unsigned char)cast<ConstantUInt>(CPV->getOperand(i))->getValue();
-      
-      if (isprint(C)) {
-        Result += C;
-      } else {
-        switch (C) {
-        case '\n': Result += "\\n"; break;
-        case '\t': Result += "\\t"; break;
-        case '\r': Result += "\\r"; break;
-        case '\v': Result += "\\v"; break;
-        case '\a': Result += "\\a"; break;
-        default:
-          Result += "\\x";
-          Result += ( C/16  < 10) ? ( C/16 +'0') : ( C/16 -10+'A');
-          Result += ((C&15) < 10) ? ((C&15)+'0') : ((C&15)-10+'A');
-          break;
-        }
-      }
-    }
-    Result += "\"";
-  } else {
-    Result = "{";
-    if (CPV->getNumOperands()) {
-      Result += " " +  getConstStrValue(cast<Constant>(CPV->getOperand(0)));
-      for (unsigned i = 1; i < CPV->getNumOperands(); i++)
-        Result += ", " + getConstStrValue(cast<Constant>(CPV->getOperand(i)));
-    }
-    Result += " }";
-  }
-  
-  return Result;
-}
-
-static std::string getConstStrValue(const Constant* CPV) {
-  switch (CPV->getType()->getPrimitiveID()) {
-  case Type::BoolTyID:  return CPV == ConstantBool::False ? "0" : "1";
-  case Type::SByteTyID:
-  case Type::ShortTyID:
-  case Type::IntTyID:   return itostr(cast<ConstantSInt>(CPV)->getValue());
-  case Type::LongTyID:  return itostr(cast<ConstantSInt>(CPV)->getValue())+"ll";
-
-  case Type::UByteTyID:
-  case Type::UShortTyID:return utostr(cast<ConstantUInt>(CPV)->getValue());
-  case Type::UIntTyID:  return utostr(cast<ConstantUInt>(CPV)->getValue())+"u";
-  case Type::ULongTyID:return utostr(cast<ConstantUInt>(CPV)->getValue())+"ull";
-
-  case Type::FloatTyID:
-  case Type::DoubleTyID: return ftostr(cast<ConstantFP>(CPV)->getValue());
-
-  case Type::ArrayTyID:  return getConstArrayStrValue(CPV);
-
-  case Type::StructTyID: {
-    std::string Result = "{";
-    if (CPV->getNumOperands()) {
-      Result += " " + getConstStrValue(cast<Constant>(CPV->getOperand(0)));
-      for (unsigned i = 1; i < CPV->getNumOperands(); i++)
-        Result += ", " + getConstStrValue(cast<Constant>(CPV->getOperand(i)));
-    }
-    return Result + " }";
-  }
-
-  default:
-    cerr << "Unknown constant type: " << CPV << "\n";
-    abort();
-  }
-}
-
-// Pass the Type* variable and and the variable name and this prints out the 
-// variable declaration.
-//
-static string calcTypeNameVar(const Type *Ty,
-                              map<const Type *, string> &TypeNames, 
-                              const string &NameSoFar, bool ignoreName = false){
-  if (Ty->isPrimitiveType())
-    switch (Ty->getPrimitiveID()) {
-    case Type::VoidTyID:   return "void " + NameSoFar;
-    case Type::BoolTyID:   return "bool " + NameSoFar;
-    case Type::UByteTyID:  return "unsigned char " + NameSoFar;
-    case Type::SByteTyID:  return "signed char " + NameSoFar;
-    case Type::UShortTyID: return "unsigned short " + NameSoFar;
-    case Type::ShortTyID:  return "short " + NameSoFar;
-    case Type::UIntTyID:   return "unsigned " + NameSoFar;
-    case Type::IntTyID:    return "int " + NameSoFar;
-    case Type::ULongTyID:  return "unsigned long long " + NameSoFar;
-    case Type::LongTyID:   return "signed long long " + NameSoFar;
-    case Type::FloatTyID:  return "float " + NameSoFar;
-    case Type::DoubleTyID: return "double " + NameSoFar;
-    default :
-      cerr << "Unknown primitive type: " << Ty << "\n";
-      abort();
-    }
-  
-  // Check to see if the type is named.
-  if (!ignoreName) {
-    map<const Type *, string>::iterator I = TypeNames.find(Ty);
-    if (I != TypeNames.end())
-      return I->second + " " + NameSoFar;
-  }  
-
-  string Result;
-  switch (Ty->getPrimitiveID()) {
-  case Type::FunctionTyID: {
-    const FunctionType *MTy = cast<FunctionType>(Ty);
-    Result += calcTypeNameVar(MTy->getReturnType(), TypeNames, "");
-    Result += " " + NameSoFar + " (";
-    for (FunctionType::ParamTypes::const_iterator
-           I = MTy->getParamTypes().begin(),
-           E = MTy->getParamTypes().end(); I != E; ++I) {
-      if (I != MTy->getParamTypes().begin())
-        Result += ", ";
-      Result += calcTypeNameVar(*I, TypeNames, "");
-    }
-    if (MTy->isVarArg()) {
-      if (!MTy->getParamTypes().empty()) 
-       Result += ", ";
-      Result += "...";
-    }
-    return Result + ")";
-  }
-  case Type::StructTyID: {
-    const StructType *STy = cast<const StructType>(Ty);
-    Result = NameSoFar + " {\n";
-    unsigned indx = 0;
-    for (StructType::ElementTypes::const_iterator
-           I = STy->getElementTypes().begin(),
-           E = STy->getElementTypes().end(); I != E; ++I) {
-      Result += "  " +calcTypeNameVar(*I, TypeNames, "field" + utostr(indx++));
-      Result += ";\n";
-    }
-    return Result + "}";
-  }  
-
-  case Type::PointerTyID:
-    return calcTypeNameVar(cast<const PointerType>(Ty)->getElementType(), 
-                           TypeNames, "*" + NameSoFar);
-  
-  case Type::ArrayTyID: {
-    const ArrayType *ATy = cast<const ArrayType>(Ty);
-    int NumElements = ATy->getNumElements();
-    return calcTypeNameVar(ATy->getElementType(), TypeNames, 
-                           NameSoFar + "[" + itostr(NumElements) + "]");
-  }
-  default:
-    assert(0 && "Unhandled case in getTypeProps!");
-    abort();
-  }
-
-  return Result;
-}
-
 namespace {
   class CWriter : public InstVisitor<CWriter> {
     ostream& Out; 
@@ -218,12 +41,11 @@ namespace {
     
     inline void write(Module *M) { printModule(M); }
 
-    ostream& printType(const Type *Ty, const string &VariableName = "") {
-      return Out << calcTypeNameVar(Ty, TypeNames, VariableName);
-    }
+    ostream &printType(const Type *Ty, const string &VariableName = "",
+                       bool IgnoreName = false);
 
-    void writeOperand(const Value *Operand);
-    void writeOperandInternal(const Value *Operand);
+    void writeOperand(Value *Operand);
+    void writeOperandInternal(Value *Operand);
 
     string getValueName(const Value *V);
 
@@ -236,45 +58,47 @@ namespace {
     
     void printFunction(Function *);
 
+    void printConstant(Constant *CPV);
+    void printConstantArray(ConstantArray *CPA);
+
     // isInlinableInst - Attempt to inline instructions into their uses to build
     // trees as much as possible.  To do this, we have to consistently decide
     // what is acceptable to inline, so that variable declarations don't get
     // printed and an extra copy of the expr is not emitted.
     //
-    static bool isInlinableInst(Instruction *I) {
+    static bool isInlinableInst(const Instruction &I) {
       // Must be an expression, must be used exactly once.  If it is dead, we
       // emit it inline where it would go.
-      if (I->getType() == Type::VoidTy || I->use_size() != 1 ||
+      if (I.getType() == Type::VoidTy || I.use_size() != 1 ||
           isa<TerminatorInst>(I) || isa<CallInst>(I) || isa<PHINode>(I))
         return false;
 
       // Only inline instruction it it's use is in the same BB as the inst.
-      return I->getParent() == cast<Instruction>(I->use_back())->getParent();
+      return I.getParent() == cast<Instruction>(I.use_back())->getParent();
     }
 
     // Instruction visitation functions
     friend class InstVisitor<CWriter>;
 
-    void visitReturnInst(ReturnInst *I);
-    void visitBranchInst(BranchInst *I);
+    void visitReturnInst(ReturnInst &I);
+    void visitBranchInst(BranchInst &I);
 
-    void visitPHINode(PHINode *I) {}
-    void visitNot(GenericUnaryInst *I);
-    void visitBinaryOperator(Instruction *I);
+    void visitPHINode(PHINode &I) {}
+    void visitBinaryOperator(Instruction &I);
 
-    void visitCastInst(CastInst *I);
-    void visitCallInst(CallInst *I);
-    void visitShiftInst(ShiftInst *I) { visitBinaryOperator(I); }
+    void visitCastInst (CastInst &I);
+    void visitCallInst (CallInst &I);
+    void visitShiftInst(ShiftInst &I) { visitBinaryOperator(I); }
 
-    void visitMallocInst(MallocInst *I);
-    void visitAllocaInst(AllocaInst *I);
-    void visitFreeInst(FreeInst   *I);
-    void visitLoadInst(LoadInst   *I);
-    void visitStoreInst(StoreInst  *I);
-    void visitGetElementPtrInst(GetElementPtrInst *I);
+    void visitMallocInst(MallocInst &I);
+    void visitAllocaInst(AllocaInst &I);
+    void visitFreeInst  (FreeInst   &I);
+    void visitLoadInst  (LoadInst   &I);
+    void visitStoreInst (StoreInst  &I);
+    void visitGetElementPtrInst(GetElementPtrInst &I);
 
-    void visitInstruction(Instruction *I) {
-      cerr << "C Writer does not know about " << I;
+    void visitInstruction(Instruction &I) {
+      std::cerr << "C Writer does not know about " << I;
       abort();
     }
 
@@ -283,7 +107,8 @@ namespace {
     }
     void printBranchToBlock(BasicBlock *CurBlock, BasicBlock *SuccBlock,
                             unsigned Indent);
-    void printIndexingExpr(MemAccessInst *MAI);
+    void printIndexingExpression(Value *Ptr, User::op_iterator I,
+                                 User::op_iterator E);
   };
 }
 
@@ -303,9 +128,10 @@ static string makeNameProper(string x) {
 }
 
 string CWriter::getValueName(const Value *V) {
-  if (V->hasName()) {             // Print out the label if it exists...
-    if (isa<GlobalValue>(V) &&    // Do not mangle globals...
-        !MangledGlobals.count(V)) // Unless the name would collide unless we do.
+  if (V->hasName()) {              // Print out the label if it exists...
+    if (isa<GlobalValue>(V) &&     // Do not mangle globals...
+        cast<GlobalValue>(V)->hasExternalLinkage() && // Unless it's internal or
+        !MangledGlobals.count(V))  // Unless the name would collide if we don't
       return makeNameProper(V->getName());
 
     return "l" + utostr(V->getType()->getUniqueID()) + "_" +
@@ -317,33 +143,264 @@ string CWriter::getValueName(const Value *V) {
   return "ltmp_" + itostr(Slot) + "_" + utostr(V->getType()->getUniqueID());
 }
 
-void CWriter::writeOperandInternal(const Value *Operand) {
-  if (Operand->hasName()) {   
-    Out << getValueName(Operand);
-  } else if (const Constant *CPV = dyn_cast<const Constant>(Operand)) {
+// Pass the Type* and the variable name and this prints out the variable
+// declaration.
+//
+ostream &CWriter::printType(const Type *Ty, const string &NameSoFar,
+                            bool IgnoreName = false) {
+  if (Ty->isPrimitiveType())
+    switch (Ty->getPrimitiveID()) {
+    case Type::VoidTyID:   return Out << "void "               << NameSoFar;
+    case Type::BoolTyID:   return Out << "bool "               << NameSoFar;
+    case Type::UByteTyID:  return Out << "unsigned char "      << NameSoFar;
+    case Type::SByteTyID:  return Out << "signed char "        << NameSoFar;
+    case Type::UShortTyID: return Out << "unsigned short "     << NameSoFar;
+    case Type::ShortTyID:  return Out << "short "              << NameSoFar;
+    case Type::UIntTyID:   return Out << "unsigned "           << NameSoFar;
+    case Type::IntTyID:    return Out << "int "                << NameSoFar;
+    case Type::ULongTyID:  return Out << "unsigned long long " << NameSoFar;
+    case Type::LongTyID:   return Out << "signed long long "   << NameSoFar;
+    case Type::FloatTyID:  return Out << "float "              << NameSoFar;
+    case Type::DoubleTyID: return Out << "double "             << NameSoFar;
+    default :
+      std::cerr << "Unknown primitive type: " << Ty << "\n";
+      abort();
+    }
+  
+  // Check to see if the type is named.
+  if (!IgnoreName) {
+    map<const Type *, string>::iterator I = TypeNames.find(Ty);
+    if (I != TypeNames.end()) {
+      return Out << I->second << " " << NameSoFar;
+    }
+  }  
+
+  switch (Ty->getPrimitiveID()) {
+  case Type::FunctionTyID: {
+    const FunctionType *MTy = cast<FunctionType>(Ty);
+    printType(MTy->getReturnType(), "");
+    Out << " " << NameSoFar << " (";
+
+    for (FunctionType::ParamTypes::const_iterator
+           I = MTy->getParamTypes().begin(),
+           E = MTy->getParamTypes().end(); I != E; ++I) {
+      if (I != MTy->getParamTypes().begin())
+        Out << ", ";
+      printType(*I, "");
+    }
+    if (MTy->isVarArg()) {
+      if (!MTy->getParamTypes().empty()) 
+       Out << ", ";
+      Out << "...";
+    }
+    return Out << ")";
+  }
+  case Type::StructTyID: {
+    const StructType *STy = cast<StructType>(Ty);
+    Out << NameSoFar + " {\n";
+    unsigned Idx = 0;
+    for (StructType::ElementTypes::const_iterator
+           I = STy->getElementTypes().begin(),
+           E = STy->getElementTypes().end(); I != E; ++I) {
+      Out << "  ";
+      printType(*I, "field" + utostr(Idx++));
+      Out << ";\n";
+    }
+    return Out << "}";
+  }  
+
+  case Type::PointerTyID: {
+    const PointerType *PTy = cast<PointerType>(Ty);
+    return printType(PTy->getElementType(), "(*" + NameSoFar + ")");
+  }
+
+  case Type::ArrayTyID: {
+    const ArrayType *ATy = cast<ArrayType>(Ty);
+    unsigned NumElements = ATy->getNumElements();
+    return printType(ATy->getElementType(),
+                     NameSoFar + "[" + utostr(NumElements) + "]");
+  }
+  default:
+    assert(0 && "Unhandled case in getTypeProps!");
+    abort();
+  }
+
+  return Out;
+}
+
+void CWriter::printConstantArray(ConstantArray *CPA) {
+
+  // As a special case, print the array as a string if it is an array of
+  // ubytes or an array of sbytes with positive values.
+  // 
+  const Type *ETy = CPA->getType()->getElementType();
+  bool isString = (ETy == Type::SByteTy || ETy == Type::UByteTy);
+
+  // Make sure the last character is a null char, as automatically added by C
+  if (CPA->getNumOperands() == 0 ||
+      !cast<Constant>(*(CPA->op_end()-1))->isNullValue())
+    isString = false;
+  
+  if (isString) {
+    Out << "\"";
+    // Do not include the last character, which we know is null
+    for (unsigned i = 0, e = CPA->getNumOperands()-1; i != e; ++i) {
+      unsigned char C = (ETy == Type::SByteTy) ?
+        (unsigned char)cast<ConstantSInt>(CPA->getOperand(i))->getValue() :
+        (unsigned char)cast<ConstantUInt>(CPA->getOperand(i))->getValue();
+      
+      if (isprint(C)) {
+        Out << C;
+      } else {
+        switch (C) {
+        case '\n': Out << "\\n"; break;
+        case '\t': Out << "\\t"; break;
+        case '\r': Out << "\\r"; break;
+        case '\v': Out << "\\v"; break;
+        case '\a': Out << "\\a"; break;
+        default:
+          Out << "\\x";
+          Out << ( C/16  < 10) ? ( C/16 +'0') : ( C/16 -10+'A');
+          Out << ((C&15) < 10) ? ((C&15)+'0') : ((C&15)-10+'A');
+          break;
+        }
+      }
+    }
+    Out << "\"";
+  } else {
+    Out << "{";
+    if (CPA->getNumOperands()) {
+      Out << " ";
+      printConstant(cast<Constant>(CPA->getOperand(0)));
+      for (unsigned i = 1, e = CPA->getNumOperands(); i != e; ++i) {
+        Out << ", ";
+        printConstant(cast<Constant>(CPA->getOperand(i)));
+      }
+    }
+    Out << " }";
+  }
+}
+
+
+// printConstant - The LLVM Constant to C Constant converter.
+void CWriter::printConstant(Constant *CPV) {
+  if (const ConstantExpr *CE = dyn_cast<ConstantExpr>(CPV)) {
+    switch (CE->getOpcode()) {
+    case Instruction::Cast:
+      Out << "((";
+      printType(CPV->getType());
+      Out << ")";
+      printConstant(cast<Constant>(CPV->getOperand(0)));
+      Out << ")";
+      return;
+
+    case Instruction::GetElementPtr:
+      Out << "&(";
+      printIndexingExpression(CPV->getOperand(0),
+                              CPV->op_begin()+1, CPV->op_end());
+      Out << ")";
+      return;
+    case Instruction::Add:
+      Out << "(";
+      printConstant(cast<Constant>(CPV->getOperand(0)));
+      Out << " + ";
+      printConstant(cast<Constant>(CPV->getOperand(1)));
+      Out << ")";
+      return;
+    case Instruction::Sub:
+      Out << "(";
+      printConstant(cast<Constant>(CPV->getOperand(0)));
+      Out << " - ";
+      printConstant(cast<Constant>(CPV->getOperand(1)));
+      Out << ")";
+      return;
+
+    default:
+      std::cerr << "CWriter Error: Unhandled constant expression: "
+                << CE << "\n";
+      abort();
+    }
+  }
+
+  switch (CPV->getType()->getPrimitiveID()) {
+  case Type::BoolTyID:
+    Out << (CPV == ConstantBool::False ? "0" : "1"); break;
+  case Type::SByteTyID:
+  case Type::ShortTyID:
+  case Type::IntTyID:
+    Out << cast<ConstantSInt>(CPV)->getValue(); break;
+  case Type::LongTyID:
+    Out << cast<ConstantSInt>(CPV)->getValue() << "ll"; break;
+
+  case Type::UByteTyID:
+  case Type::UShortTyID:
+    Out << cast<ConstantUInt>(CPV)->getValue(); break;
+  case Type::UIntTyID:
+    Out << cast<ConstantUInt>(CPV)->getValue() << "u"; break;
+  case Type::ULongTyID:
+    Out << cast<ConstantUInt>(CPV)->getValue() << "ull"; break;
+
+  case Type::FloatTyID:
+  case Type::DoubleTyID:
+    Out << cast<ConstantFP>(CPV)->getValue(); break;
+
+  case Type::ArrayTyID:
+    printConstantArray(cast<ConstantArray>(CPV));
+    break;
+
+  case Type::StructTyID: {
+    Out << "{";
+    if (CPV->getNumOperands()) {
+      Out << " ";
+      printConstant(cast<Constant>(CPV->getOperand(0)));
+      for (unsigned i = 1, e = CPV->getNumOperands(); i != e; ++i) {
+        Out << ", ";
+        printConstant(cast<Constant>(CPV->getOperand(i)));
+      }
+    }
+    Out << " }";
+    break;
+  }
+
+  case Type::PointerTyID:
     if (isa<ConstantPointerNull>(CPV)) {
       Out << "((";
       printType(CPV->getType(), "");
       Out << ")NULL)";
-    } else
-      Out << getConstStrValue(CPV); 
-  } else {
-    int Slot = Table.getValSlot(Operand);
-    assert(Slot >= 0 && "Malformed LLVM!");
-    Out << "ltmp_" << Slot << "_" << Operand->getType()->getUniqueID();
+      break;
+    } else if (ConstantPointerRef *CPR = dyn_cast<ConstantPointerRef>(CPV)) {
+      writeOperand(CPR->getValue());
+      break;
+    }
+    // FALL THROUGH
+  default:
+    std::cerr << "Unknown constant type: " << CPV << "\n";
+    abort();
   }
 }
 
-void CWriter::writeOperand(const Value *Operand) {
+void CWriter::writeOperandInternal(Value *Operand) {
   if (Instruction *I = dyn_cast<Instruction>(Operand))
-    if (isInlinableInst(I)) {
+    if (isInlinableInst(*I)) {
       // Should we inline this instruction to build a tree?
       Out << "(";
-      visit(I);
+      visit(*I);
       Out << ")";    
       return;
     }
+  
+  if (Operand->hasName()) {   
+    Out << getValueName(Operand);
+  } else if (Constant *CPV = dyn_cast<Constant>(Operand)) {
+    printConstant(CPV); 
+  } else {
+    int Slot = Table.getValSlot(Operand);
+    assert(Slot >= 0 && "Malformed LLVM!");
+    Out << "ltmp_" << Slot << "_" << Operand->getType()->getUniqueID();
+  }
+}
 
+void CWriter::writeOperand(Value *Operand) {
   if (isa<GlobalVariable>(Operand))
     Out << "(&";  // Global variables are references as their addresses by llvm
 
@@ -356,21 +413,21 @@ void CWriter::writeOperand(const Value *Operand) {
 void CWriter::printModule(Module *M) {
   // Calculate which global values have names that will collide when we throw
   // away type information.
-  {  // Scope to declare the FoundNames set when we are done with it...
+  {  // Scope to delete the FoundNames set when we are done with it...
     std::set<string> FoundNames;
     for (Module::iterator I = M->begin(), E = M->end(); I != E; ++I)
-      if ((*I)->hasName())                      // If the global has a name...
-        if (FoundNames.count((*I)->getName()))  // And the name is already used
-          MangledGlobals.insert(*I);            // Mangle the name
+      if (I->hasName())                      // If the global has a name...
+        if (FoundNames.count(I->getName()))  // And the name is already used
+          MangledGlobals.insert(I);          // Mangle the name
         else
-          FoundNames.insert((*I)->getName());   // Otherwise, keep track of name
+          FoundNames.insert(I->getName());   // Otherwise, keep track of name
 
     for (Module::giterator I = M->gbegin(), E = M->gend(); I != E; ++I)
-      if ((*I)->hasName())                      // If the global has a name...
-        if (FoundNames.count((*I)->getName()))  // And the name is already used
-          MangledGlobals.insert(*I);            // Mangle the name
+      if (I->hasName())                      // If the global has a name...
+        if (FoundNames.count(I->getName()))  // And the name is already used
+          MangledGlobals.insert(I);          // Mangle the name
         else
-          FoundNames.insert((*I)->getName());   // Otherwise, keep track of name
+          FoundNames.insert(I->getName());   // Otherwise, keep track of name
   }
 
 
@@ -379,40 +436,61 @@ void CWriter::printModule(Module *M) {
 
   // get declaration for alloca
   Out << "/* Provide Declarations */\n"
+      << "#include <malloc.h>\n"
       << "#include <alloca.h>\n\n"
 
     // Provide a definition for null if one does not already exist.
       << "#ifndef NULL\n#define NULL 0\n#endif\n\n"
       << "typedef unsigned char bool;\n"
 
-      << "\n\n/* Global Symbols */\n";
+      << "\n\n/* Global Declarations */\n";
+
+  // First output all the declarations for the program, because C requires
+  // Functions & globals to be declared before they are used.
+  //
 
   // Loop over the symbol table, emitting all named constants...
   if (M->hasSymbolTable())
     printSymbolTable(*M->getSymbolTable());
 
-  Out << "\n\n/* Global Data */\n";
-  for (Module::const_giterator I = M->gbegin(), E = M->gend(); I != E; ++I) {
-    GlobalVariable *GV = *I;
-    if (GV->hasInternalLinkage()) Out << "static ";
-    printType(GV->getType()->getElementType(), getValueName(GV));
+  // Global variable declarations...
+  if (!M->gempty()) {
+    Out << "\n/* Global Variable Declarations */\n";
+    for (Module::giterator I = M->gbegin(), E = M->gend(); I != E; ++I) {
+      Out << (I->hasExternalLinkage() ? "extern " : "static ");
+      printType(I->getType()->getElementType(), getValueName(I));
+      Out << ";\n";
+    }
+  }
+
+  // Function declarations
+  if (!M->empty()) {
+    Out << "\n/* Function Declarations */\n";
+    for (Module::iterator I = M->begin(), E = M->end(); I != E; ++I)
+      printFunctionDecl(I);
+  }
 
-    if (GV->hasInitializer()) {
-      Out << " = " ;
-      writeOperand(GV->getInitializer());
+  // Output the global variable contents...
+  if (!M->gempty()) {
+    Out << "\n\n/* Global Data */\n";
+    for (Module::giterator I = M->gbegin(), E = M->gend(); I != E; ++I) {
+      if (I->hasInternalLinkage()) Out << "static ";
+      printType(I->getType()->getElementType(), getValueName(I));
+      
+      if (I->hasInitializer()) {
+        Out << " = " ;
+        writeOperand(I->getInitializer());
+      }
+      Out << ";\n";
     }
-    Out << ";\n";
   }
 
-  // First output all the declarations of the functions as C requires Functions 
-  // be declared before they are used.
-  //
-  Out << "\n\n/* Function Declarations */\n";
-  for_each(M->begin(), M->end(), bind_obj(this, &CWriter::printFunctionDecl));
-  
   // Output all of the functions...
-  Out << "\n\n/* Function Bodies */\n";
-  for_each(M->begin(), M->end(), bind_obj(this, &CWriter::printFunction));
+  if (!M->empty()) {
+    Out << "\n\n/* Function Bodies */\n";
+    for (Module::iterator I = M->begin(), E = M->end(); I != E; ++I)
+      printFunction(I);
+  }
 }
 
 
@@ -425,10 +503,9 @@ void CWriter::printSymbolTable(const SymbolTable &ST) {
     SymbolTable::type_const_iterator End = ST.type_end(TI->first);
     
     for (; I != End; ++I)
-      if (const Type *Ty = dyn_cast<const StructType>(I->second)) {
-       string Name = "struct l_" + I->first;
+      if (const Type *Ty = dyn_cast<StructType>(I->second)) {
+        string Name = "struct l_" + makeNameProper(I->first);
         Out << Name << ";\n";
-
         TypeNames.insert(std::make_pair(Ty, Name));
       }
   }
@@ -441,14 +518,15 @@ void CWriter::printSymbolTable(const SymbolTable &ST) {
     
     for (; I != End; ++I) {
       const Value *V = I->second;
-      if (const Type *Ty = dyn_cast<const Type>(V)) {
-       string Name = "l_" + I->first;
+      if (const Type *Ty = dyn_cast<Type>(V)) {
+       string Name = "l_" + makeNameProper(I->first);
         if (isa<StructType>(Ty))
-          Name = "struct " + Name;
+          Name = "struct " + makeNameProper(Name);
         else
           Out << "typedef ";
 
-       Out << calcTypeNameVar(Ty, TypeNames, Name, true) << ";\n";
+       printType(Ty, Name, true);
+        Out << ";\n";
       }
     }
   }
@@ -473,15 +551,13 @@ void CWriter::printFunctionSignature(const Function *F) {
   Out << getValueName(F) << "(";
     
   if (!F->isExternal()) {
-    if (!F->getArgumentList().empty()) {
-      printType(F->getArgumentList().front()->getType(),
-                getValueName(F->getArgumentList().front()));
+    if (!F->aempty()) {
+      printType(F->afront().getType(), getValueName(F->abegin()));
 
-      for (Function::ArgumentListType::const_iterator
-             I = F->getArgumentList().begin()+1,
-             E = F->getArgumentList().end(); I != E; ++I) {
+      for (Function::const_aiterator I = ++F->abegin(), E = F->aend();
+           I != E; ++I) {
         Out << ", ";
-        printType((*I)->getType(), getValueName(*I));
+        printType(I->getType(), getValueName(I));
       }
     }
   } else {
@@ -513,15 +589,15 @@ void CWriter::printFunction(Function *F) {
 
   // print local variable information for the function
   for (inst_iterator I = inst_begin(F), E = inst_end(F); I != E; ++I)
-    if ((*I)->getType() != Type::VoidTy && !isInlinableInst(*I)) {
+    if ((*I)->getType() != Type::VoidTy && !isInlinableInst(**I)) {
       Out << "  ";
       printType((*I)->getType(), getValueName(*I));
       Out << ";\n";
     }
  
   // print the basic blocks
-  for (Function::iterator I = F->begin(), E = F->end(); I != E; ++I) {
-    BasicBlock *BB = *I, *Prev = I != F->begin() ? *(I-1) : 0;
+  for (Function::iterator BB = F->begin(), E = F->end(); BB != E; ++BB) {
+    BasicBlock *Prev = BB->getPrev();
 
     // Don't print the label for the basic block if there are no uses, or if the
     // only terminator use is the precessor basic block's terminator.  We have
@@ -540,21 +616,19 @@ void CWriter::printFunction(Function *F) {
     if (NeedsLabel) Out << getValueName(BB) << ":\n";
 
     // Output all of the instructions in the basic block...
-    for (BasicBlock::iterator II = BB->begin(), E = BB->end()-1;
-         II != E; ++II) {
+    for (BasicBlock::iterator II = BB->begin(), E = --BB->end(); II != E; ++II){
       if (!isInlinableInst(*II) && !isa<PHINode>(*II)) {
-        Instruction *I = *II;
-        if (I->getType() != Type::VoidTy)
-          outputLValue(I);
+        if (II->getType() != Type::VoidTy)
+          outputLValue(II);
         else
           Out << "  ";
-        visit(I);
+        visit(*II);
         Out << ";\n";
       }
     }
 
     // Don't emit prefix or suffix for the terminator...
-    visit(BB->getTerminator());
+    visit(*BB->getTerminator());
   }
   
   Out << "}\n\n";
@@ -564,32 +638,26 @@ void CWriter::printFunction(Function *F) {
 // Specific Instruction type classes... note that all of the casts are
 // neccesary because we use the instruction classes as opaque types...
 //
-void CWriter::visitReturnInst(ReturnInst *I) {
+void CWriter::visitReturnInst(ReturnInst &I) {
   // Don't output a void return if this is the last basic block in the function
-  if (I->getNumOperands() == 0 && 
-      *(I->getParent()->getParent()->end()-1) == I->getParent())
+  if (I.getNumOperands() == 0 && 
+      &*--I.getParent()->getParent()->end() == I.getParent() &&
+      !I.getParent()->size() == 1) {
     return;
+  }
 
   Out << "  return";
-  if (I->getNumOperands()) {
+  if (I.getNumOperands()) {
     Out << " ";
-    writeOperand(I->getOperand(0));
+    writeOperand(I.getOperand(0));
   }
   Out << ";\n";
 }
 
-// Return true if BB1 immediately preceeds BB2.
-static bool BBFollowsBB(BasicBlock *BB1, BasicBlock *BB2) {
-  Function *F = BB1->getParent();
-  Function::iterator I = find(F->begin(), F->end(), BB1);
-  assert(I != F->end() && "BB not in function!");
-  return *(I+1) == BB2;  
-}
-
 static bool isGotoCodeNeccessary(BasicBlock *From, BasicBlock *To) {
   // If PHI nodes need copies, we need the copy code...
   if (isa<PHINode>(To->front()) ||
-      !BBFollowsBB(From, To))      // Not directly successor, need goto
+      From->getNext() != To)      // Not directly successor, need goto
     return true;
 
   // Otherwise we don't need the code.
@@ -599,7 +667,7 @@ static bool isGotoCodeNeccessary(BasicBlock *From, BasicBlock *To) {
 void CWriter::printBranchToBlock(BasicBlock *CurBB, BasicBlock *Succ,
                                            unsigned Indent) {
   for (BasicBlock::iterator I = Succ->begin();
-       PHINode *PN = dyn_cast<PHINode>(*I); ++I) {
+       PHINode *PN = dyn_cast<PHINode>(&*I); ++I) {
     //  now we have to do the printing
     Out << string(Indent, ' ');
     outputLValue(PN);
@@ -607,7 +675,7 @@ void CWriter::printBranchToBlock(BasicBlock *CurBB, BasicBlock *Succ,
     Out << ";   /* for PHI node */\n";
   }
 
-  if (!BBFollowsBB(CurBB, Succ)) {
+  if (CurBB->getNext() != Succ) {
     Out << string(Indent, ' ') << "  goto ";
     writeOperand(Succ);
     Out << ";\n";
@@ -617,53 +685,48 @@ void CWriter::printBranchToBlock(BasicBlock *CurBB, BasicBlock *Succ,
 // Brach instruction printing - Avoid printing out a brach to a basic block that
 // immediately succeeds the current one.
 //
-void CWriter::visitBranchInst(BranchInst *I) {
-  if (I->isConditional()) {
-    if (isGotoCodeNeccessary(I->getParent(), I->getSuccessor(0))) {
+void CWriter::visitBranchInst(BranchInst &I) {
+  if (I.isConditional()) {
+    if (isGotoCodeNeccessary(I.getParent(), I.getSuccessor(0))) {
       Out << "  if (";
-      writeOperand(I->getCondition());
+      writeOperand(I.getCondition());
       Out << ") {\n";
       
-      printBranchToBlock(I->getParent(), I->getSuccessor(0), 2);
+      printBranchToBlock(I.getParent(), I.getSuccessor(0), 2);
       
-      if (isGotoCodeNeccessary(I->getParent(), I->getSuccessor(1))) {
+      if (isGotoCodeNeccessary(I.getParent(), I.getSuccessor(1))) {
         Out << "  } else {\n";
-        printBranchToBlock(I->getParent(), I->getSuccessor(1), 2);
+        printBranchToBlock(I.getParent(), I.getSuccessor(1), 2);
       }
     } else {
       // First goto not neccesary, assume second one is...
       Out << "  if (!";
-      writeOperand(I->getCondition());
+      writeOperand(I.getCondition());
       Out << ") {\n";
 
-      printBranchToBlock(I->getParent(), I->getSuccessor(1), 2);
+      printBranchToBlock(I.getParent(), I.getSuccessor(1), 2);
     }
 
     Out << "  }\n";
   } else {
-    printBranchToBlock(I->getParent(), I->getSuccessor(0), 0);
+    printBranchToBlock(I.getParent(), I.getSuccessor(0), 0);
   }
   Out << "\n";
 }
 
 
-void CWriter::visitNot(GenericUnaryInst *I) {
-  Out << "~";
-  writeOperand(I->getOperand(0));
-}
-
-void CWriter::visitBinaryOperator(Instruction *I) {
+void CWriter::visitBinaryOperator(Instruction &I) {
   // binary instructions, shift instructions, setCond instructions.
-  if (isa<PointerType>(I->getType())) {
+  if (isa<PointerType>(I.getType())) {
     Out << "(";
-    printType(I->getType());
+    printType(I.getType());
     Out << ")";
   }
       
-  if (isa<PointerType>(I->getType())) Out << "(long long)";
-  writeOperand(I->getOperand(0));
+  if (isa<PointerType>(I.getType())) Out << "(long long)";
+  writeOperand(I.getOperand(0));
 
-  switch (I->getOpcode()) {
+  switch (I.getOpcode()) {
   case Instruction::Add: Out << " + "; break;
   case Instruction::Sub: Out << " - "; break;
   case Instruction::Mul: Out << "*"; break;
@@ -680,93 +743,110 @@ void CWriter::visitBinaryOperator(Instruction *I) {
   case Instruction::SetGT: Out << " > "; break;
   case Instruction::Shl : Out << " << "; break;
   case Instruction::Shr : Out << " >> "; break;
-  default: cerr << "Invalid operator type!" << I; abort();
+  default: std::cerr << "Invalid operator type!" << I; abort();
   }
 
-  if (isa<PointerType>(I->getType())) Out << "(long long)";
-  writeOperand(I->getOperand(1));
+  if (isa<PointerType>(I.getType())) Out << "(long long)";
+  writeOperand(I.getOperand(1));
 }
 
-void CWriter::visitCastInst(CastInst *I) {
+void CWriter::visitCastInst(CastInst &I) {
   Out << "(";
-  printType(I->getType());
+  printType(I.getType());
   Out << ")";
-  writeOperand(I->getOperand(0));
+  writeOperand(I.getOperand(0));
 }
 
-void CWriter::visitCallInst(CallInst *I) {
-  const PointerType  *PTy   = cast<PointerType>(I->getCalledValue()->getType());
+void CWriter::visitCallInst(CallInst &I) {
+  const PointerType  *PTy   = cast<PointerType>(I.getCalledValue()->getType());
   const FunctionType *FTy   = cast<FunctionType>(PTy->getElementType());
   const Type         *RetTy = FTy->getReturnType();
   
-  Out << getValueName(I->getOperand(0)) << "(";
+  Out << getValueName(I.getOperand(0)) << "(";
 
-  if (I->getNumOperands() > 1) {
-    writeOperand(I->getOperand(1));
+  if (I.getNumOperands() > 1) {
+    writeOperand(I.getOperand(1));
 
-    for (unsigned op = 2, Eop = I->getNumOperands(); op != Eop; ++op) {
+    for (unsigned op = 2, Eop = I.getNumOperands(); op != Eop; ++op) {
       Out << ", ";
-      writeOperand(I->getOperand(op));
+      writeOperand(I.getOperand(op));
     }
   }
   Out << ")";
 }  
 
-void CWriter::visitMallocInst(MallocInst *I) {
+void CWriter::visitMallocInst(MallocInst &I) {
   Out << "(";
-  printType(I->getType());
+  printType(I.getType());
   Out << ")malloc(sizeof(";
-  printType(I->getType()->getElementType());
+  printType(I.getType()->getElementType());
   Out << ")";
 
-  if (I->isArrayAllocation()) {
+  if (I.isArrayAllocation()) {
     Out << " * " ;
-    writeOperand(I->getOperand(0));
+    writeOperand(I.getOperand(0));
   }
   Out << ")";
 }
 
-void CWriter::visitAllocaInst(AllocaInst *I) {
+void CWriter::visitAllocaInst(AllocaInst &I) {
   Out << "(";
-  printType(I->getType());
+  printType(I.getType());
   Out << ") alloca(sizeof(";
-  printType(I->getType()->getElementType());
+  printType(I.getType()->getElementType());
   Out << ")";
-  if (I->isArrayAllocation()) {
+  if (I.isArrayAllocation()) {
     Out << " * " ;
-    writeOperand(I->getOperand(0));
+    writeOperand(I.getOperand(0));
   }
   Out << ")";
 }
 
-void CWriter::visitFreeInst(FreeInst *I) {
+void CWriter::visitFreeInst(FreeInst &I) {
   Out << "free(";
-  writeOperand(I->getOperand(0));
+  writeOperand(I.getOperand(0));
   Out << ")";
 }
 
-void CWriter::printIndexingExpr(MemAccessInst *MAI) {
-  MemAccessInst::op_iterator I = MAI->idx_begin(), E = MAI->idx_end();
+void CWriter::printIndexingExpression(Value *Ptr, User::op_iterator I,
+                                      User::op_iterator E) {
+  bool HasImplicitAddress = false;
+  // If accessing a global value with no indexing, avoid *(&GV) syndrome
+  if (GlobalValue *V = dyn_cast<GlobalValue>(Ptr)) {
+    HasImplicitAddress = true;
+  } else if (ConstantPointerRef *CPR = dyn_cast<ConstantPointerRef>(Ptr)) {
+    HasImplicitAddress = true;
+    Ptr = CPR->getValue();         // Get to the global...
+  }
+
   if (I == E) {
-    // If accessing a global value with no indexing, avoid *(&GV) syndrome
-    if (GlobalValue *V = dyn_cast<GlobalValue>(MAI->getPointerOperand())) {
-      writeOperandInternal(V);
-      return;
-    }
+    if (!HasImplicitAddress)
+      Out << "*";  // Implicit zero first argument: '*x' is equivalent to 'x[0]'
 
-    Out << "*";  // Implicit zero first argument: '*x' is equivalent to 'x[0]'
+    writeOperandInternal(Ptr);
+    return;
   }
 
-  writeOperand(MAI->getPointerOperand());
+  const Constant *CI = dyn_cast<Constant>(I->get());
+  if (HasImplicitAddress && (!CI || !CI->isNullValue()))
+    Out << "(&";
 
-  if (I == E) return;
+  writeOperandInternal(Ptr);
+
+  if (HasImplicitAddress && (!CI || !CI->isNullValue()))
+    Out << ")";
 
   // Print out the -> operator if possible...
-  Constant *CI = dyn_cast<Constant>(*I);
-  if (CI && CI->isNullValue() && I+1 != E &&
-      (*(I+1))->getType() == Type::UByteTy) {
-    Out << "->field" << cast<ConstantUInt>(*(I+1))->getValue();
-    I += 2;
+  if (CI && CI->isNullValue() && I+1 != E) {
+    if ((*(I+1))->getType() == Type::UByteTy) {
+      Out << (HasImplicitAddress ? "." : "->");
+      Out << "field" << cast<ConstantUInt>(*(I+1))->getValue();
+      I += 2;
+    } else {  // Performing array indexing. Just skip the 0
+      ++I;
+    }
+  } else if (HasImplicitAddress) {
+    
   }
     
   for (; I != E; ++I)
@@ -779,19 +859,19 @@ void CWriter::printIndexingExpr(MemAccessInst *MAI) {
     }
 }
 
-void CWriter::visitLoadInst(LoadInst *I) {
-  printIndexingExpr(I);
+void CWriter::visitLoadInst(LoadInst &I) {
+  printIndexingExpression(I.getPointerOperand(), I.idx_begin(), I.idx_end());
 }
 
-void CWriter::visitStoreInst(StoreInst *I) {
-  printIndexingExpr(I);
+void CWriter::visitStoreInst(StoreInst &I) {
+  printIndexingExpression(I.getPointerOperand(), I.idx_begin(), I.idx_end());
   Out << " = ";
-  writeOperand(I->getOperand(0));
+  writeOperand(I.getOperand(0));
 }
 
-void CWriter::visitGetElementPtrInst(GetElementPtrInst *I) {
+void CWriter::visitGetElementPtrInst(GetElementPtrInst &I) {
   Out << "&";
-  printIndexingExpr(I);
+  printIndexingExpression(I.getPointerOperand(), I.idx_begin(), I.idx_end());
 }
 
 //===----------------------------------------------------------------------===//