Adjust to the changed StructType interface. In particular, getElementTypes() is...
[oota-llvm.git] / lib / AsmParser / llvmAsmParser.y
index e02f3065c07ee6c43e1e5642c8b0596585cf1152..ad431d03887d7357fdc1d45736bd077a683dae8b 100644 (file)
@@ -1,76 +1,85 @@
-//===-- llvmAsmParser.y - Parser for llvm assembly files ---------*- C++ -*--=//
+//===-- llvmAsmParser.y - Parser for llvm assembly files --------*- C++ -*-===//
+// 
+//                     The LLVM Compiler Infrastructure
+//
+// This file was developed by the LLVM research group and is distributed under
+// the University of Illinois Open Source License. See LICENSE.TXT for details.
+// 
+//===----------------------------------------------------------------------===//
 //
 //  This file implements the bison parser for LLVM assembly languages files.
 //
-//===------------------------------------------------------------------------=//
+//===----------------------------------------------------------------------===//
 
 %{
 #include "ParserInternals.h"
 #include "llvm/SymbolTable.h"
 #include "llvm/Module.h"
-#include "llvm/GlobalVariable.h"
 #include "llvm/iTerminators.h"
 #include "llvm/iMemory.h"
+#include "llvm/iOperators.h"
 #include "llvm/iPHINode.h"
-#include "llvm/Argument.h"
 #include "Support/STLExtras.h"
-#include "Support/DepthFirstIterator.h"
 #include <list>
-#include <utility>            // Get definition of pair class
+#include <utility>
 #include <algorithm>
-#include <iostream>
-using std::list;
-using std::vector;
-using std::pair;
-using std::map;
-using std::pair;
-using std::make_pair;
-using std::cerr;
-using std::string;
 
 int yyerror(const char *ErrorMsg); // Forward declarations to prevent "implicit
 int yylex();                       // declaration" of xxx warnings.
 int yyparse();
 
+namespace llvm {
+
 static Module *ParserResult;
-string CurFilename;
+std::string CurFilename;
 
 // DEBUG_UPREFS - Define this symbol if you want to enable debugging output
 // relating to upreferences in the input stream.
 //
 //#define DEBUG_UPREFS 1
 #ifdef DEBUG_UPREFS
-#define UR_OUT(X) cerr << X
+#define UR_OUT(X) std::cerr << X
 #else
 #define UR_OUT(X)
 #endif
 
-// This contains info used when building the body of a method.  It is destroyed
-// when the method is completed.
+#define YYERROR_VERBOSE 1
+
+// HACK ALERT: This variable is used to implement the automatic conversion of
+// variable argument instructions from their old to new forms.  When this
+// compatiblity "Feature" is removed, this should be too.
 //
-typedef vector<Value *> ValueList;           // Numbered defs
-static void ResolveDefinitions(vector<ValueList> &LateResolvers,
-                               vector<ValueList> *FutureLateResolvers = 0);
+static BasicBlock *CurBB;
+static bool ObsoleteVarArgs;
+
+
+// This contains info used when building the body of a function.  It is
+// destroyed when the function is completed.
+//
+typedef std::vector<Value *> ValueList;           // Numbered defs
+static void ResolveDefinitions(std::vector<ValueList> &LateResolvers,
+                               std::vector<ValueList> *FutureLateResolvers = 0);
 
 static struct PerModuleInfo {
   Module *CurrentModule;
-  vector<ValueList>    Values;     // Module level numbered definitions
-  vector<ValueList>    LateResolveValues;
-  vector<PATypeHolder> Types;
-  map<ValID, PATypeHolder> LateResolveTypes;
+  std::vector<ValueList>    Values;     // Module level numbered definitions
+  std::vector<ValueList>    LateResolveValues;
+  std::vector<PATypeHolder> Types;
+  std::map<ValID, PATypeHolder> LateResolveTypes;
 
   // GlobalRefs - This maintains a mapping between <Type, ValID>'s and forward
   // references to global values.  Global values may be referenced before they
   // are defined, and if so, the temporary object that they represent is held
   // here.  This is used for forward references of ConstantPointerRefs.
   //
-  typedef map<pair<const PointerType *, ValID>, GlobalVariable*> GlobalRefsType;
+  typedef std::map<std::pair<const PointerType *,
+                             ValID>, GlobalVariable*> GlobalRefsType;
   GlobalRefsType GlobalRefs;
 
   void ModuleDone() {
-    // If we could not resolve some methods at method compilation time (calls to
-    // methods before they are defined), resolve them now...  Types are resolved
-    // when the constant pool has been completely parsed.
+    // If we could not resolve some functions at function compilation time
+    // (calls to functions before they are defined), resolve them now...  Types
+    // are resolved when the constant pool has been completely parsed.
     //
     ResolveDefinitions(LateResolveValues);
 
@@ -78,7 +87,7 @@ static struct PerModuleInfo {
     // resolved!
     //
     if (!GlobalRefs.empty()) {
-      string UndefinedReferences = "Unresolved global references exist:\n";
+      std::string UndefinedReferences = "Unresolved global references exist:\n";
       
       for (GlobalRefsType::iterator I = GlobalRefs.begin(), E =GlobalRefs.end();
            I != E; ++I) {
@@ -88,42 +97,41 @@ static struct PerModuleInfo {
       ThrowException(UndefinedReferences);
     }
 
-    Values.clear();         // Clear out method local definitions
+    Values.clear();         // Clear out function local definitions
     Types.clear();
     CurrentModule = 0;
   }
 
 
-  // DeclareNewGlobalValue - Called every type a new GV has been defined.  This
+  // DeclareNewGlobalValue - Called every time a new GV has been defined.  This
   // is used to remove things from the forward declaration map, resolving them
   // to the correct thing as needed.
   //
   void DeclareNewGlobalValue(GlobalValue *GV, ValID D) {
     // Check to see if there is a forward reference to this global variable...
     // if there is, eliminate it and patch the reference to use the new def'n.
-    GlobalRefsType::iterator I = GlobalRefs.find(make_pair(GV->getType(), D));
+    GlobalRefsType::iterator I =
+      GlobalRefs.find(std::make_pair(GV->getType(), D));
 
     if (I != GlobalRefs.end()) {
       GlobalVariable *OldGV = I->second;   // Get the placeholder...
-      I->first.second.destroy();  // Free string memory if neccesary
+      I->first.second.destroy();  // Free string memory if necessary
       
       // Loop over all of the uses of the GlobalValue.  The only thing they are
-      // allowed to be at this point is ConstantPointerRef's.
-      assert(OldGV->use_size() == 1 && "Only one reference should exist!");
-      while (!OldGV->use_empty()) {
-       User *U = OldGV->use_back();  // Must be a ConstantPointerRef...
-       ConstantPointerRef *CPPR = cast<ConstantPointerRef>(U);
-       assert(CPPR->getValue() == OldGV && "Something isn't happy");
-       
-       // Change the const pool reference to point to the real global variable
-       // now.  This should drop a use from the OldGV.
-       CPPR->mutateReference(GV);
-      }
-    
-      // Remove GV from the module...
+      // allowed to be is ConstantPointerRef's.
+      assert(OldGV->hasOneUse() && "Only one reference should exist!");
+      User *U = OldGV->use_back();  // Must be a ConstantPointerRef...
+      ConstantPointerRef *CPR = cast<ConstantPointerRef>(U);
+        
+      // Change the const pool reference to point to the real global variable
+      // now.  This should drop a use from the OldGV.
+      CPR->mutateReferences(OldGV, GV);
+      assert(OldGV->use_empty() && "All uses should be gone now!");
+      
+      // Remove OldGV from the module...
       CurrentModule->getGlobalList().remove(OldGV);
       delete OldGV;                        // Delete the old placeholder
-
+      
       // Remove the map entry for the global now that it has been created...
       GlobalRefs.erase(I);
     }
@@ -132,21 +140,20 @@ static struct PerModuleInfo {
 } CurModule;
 
 static struct PerFunctionInfo {
-  Function *CurrentFunction;     // Pointer to current method being created
+  Function *CurrentFunction;     // Pointer to current function being created
 
-  vector<ValueList> Values;      // Keep track of numbered definitions
-  vector<ValueList> LateResolveValues;
-  vector<PATypeHolder> Types;
-  map<ValID, PATypeHolder> LateResolveTypes;
-  bool isDeclare;                // Is this method a forward declararation?
+  std::vector<ValueList> Values;      // Keep track of numbered definitions
+  std::vector<ValueList> LateResolveValues;
+  std::vector<PATypeHolder> Types;
+  std::map<ValID, PATypeHolder> LateResolveTypes;
+  SymbolTable LocalSymtab;
+  bool isDeclare;                // Is this function a forward declararation?
 
   inline PerFunctionInfo() {
     CurrentFunction = 0;
     isDeclare = false;
   }
 
-  inline ~PerFunctionInfo() {}
-
   inline void FunctionStart(Function *M) {
     CurrentFunction = M;
   }
@@ -156,21 +163,42 @@ static struct PerFunctionInfo {
     // resolve the branches now...
     ResolveDefinitions(LateResolveValues, &CurModule.LateResolveValues);
 
-    Values.clear();         // Clear out method local definitions
-    Types.clear();
+    // Make sure to resolve any constant expr references that might exist within
+    // the function we just declared itself.
+    ValID FID;
+    if (CurrentFunction->hasName()) {
+      FID = ValID::create((char*)CurrentFunction->getName().c_str());
+    } else {
+      unsigned Slot = CurrentFunction->getType()->getUniqueID();
+      assert(CurModule.Values.size() > Slot && "Function not inserted?");
+      // Figure out which slot number if is...
+      for (unsigned i = 0; ; ++i) {
+        assert(i < CurModule.Values[Slot].size() && "Function not found!");
+        if (CurModule.Values[Slot][i] == CurrentFunction) {
+          FID = ValID::create((int)i);
+          break;
+        }
+      }
+    }
+    CurModule.DeclareNewGlobalValue(CurrentFunction, FID);
+
+    Values.clear();         // Clear out function local definitions
+    Types.clear();          // Clear out function local types
+    LocalSymtab.clear();    // Clear out function local symbol table
     CurrentFunction = 0;
     isDeclare = false;
   }
-} CurMeth;  // Info for the current method...
+} CurFun;  // Info for the current function...
 
-static bool inFunctionScope() { return CurMeth.CurrentFunction != 0; }
+static bool inFunctionScope() { return CurFun.CurrentFunction != 0; }
 
 
 //===----------------------------------------------------------------------===//
 //               Code to handle definitions of all the types
 //===----------------------------------------------------------------------===//
 
-static int InsertValue(Value *D, vector<ValueList> &ValueTab = CurMeth.Values) {
+static int InsertValue(Value *D,
+                       std::vector<ValueList> &ValueTab = CurFun.Values) {
   if (D->hasName()) return -1;           // Is this a numbered definition?
 
   // Yes, insert the value into the value table...
@@ -183,13 +211,13 @@ static int InsertValue(Value *D, vector<ValueList> &ValueTab = CurMeth.Values) {
 }
 
 // TODO: FIXME when Type are not const
-static void InsertType(const Type *Ty, vector<PATypeHolder> &Types) {
+static void InsertType(const Type *Ty, std::vector<PATypeHolder> &Types) {
   Types.push_back(Ty);
 }
 
 static const Type *getTypeVal(const ValID &D, bool DoNotImprovise = false) {
   switch (D.Type) {
-  case 0: {                 // Is it a numbered definition?
+  case ValID::NumberVal: {                 // Is it a numbered definition?
     unsigned Num = (unsigned)D.Num;
 
     // Module constants occupy the lowest numbered slots...
@@ -199,31 +227,33 @@ static const Type *getTypeVal(const ValID &D, bool DoNotImprovise = false) {
     Num -= CurModule.Types.size();
 
     // Check that the number is within bounds...
-    if (Num <= CurMeth.Types.size())
-      return CurMeth.Types[Num];
+    if (Num <= CurFun.Types.size())
+      return CurFun.Types[Num];
     break;
   }
-  case 1: {                // Is it a named definition?
-    string Name(D.Name);
+  case ValID::NameVal: {                // Is it a named definition?
+    std::string Name(D.Name);
     SymbolTable *SymTab = 0;
-    if (inFunctionScope()) SymTab = CurMeth.CurrentFunction->getSymbolTable();
-    Value *N = SymTab ? SymTab->lookup(Type::TypeTy, Name) : 0;
+    Value *N = 0;
+    if (inFunctionScope()) {
+      SymTab = &CurFun.CurrentFunction->getSymbolTable();
+      N = SymTab->lookup(Type::TypeTy, Name);
+    }
 
     if (N == 0) {
-      // Symbol table doesn't automatically chain yet... because the method
+      // Symbol table doesn't automatically chain yet... because the function
       // hasn't been added to the module...
       //
-      SymTab = CurModule.CurrentModule->getSymbolTable();
-      if (SymTab)
-        N = SymTab->lookup(Type::TypeTy, Name);
+      SymTab = &CurModule.CurrentModule->getSymbolTable();
+      N = SymTab->lookup(Type::TypeTy, Name);
       if (N == 0) break;
     }
 
     D.destroy();  // Free old strdup'd memory...
-    return cast<const Type>(N);
+    return cast<Type>(N);
   }
   default:
-    ThrowException("Invalid symbol type reference!");
+    ThrowException("Internal parser error: Invalid symbol type reference!");
   }
 
   // If we reached here, we referenced either a symbol that we don't know about
@@ -232,24 +262,24 @@ static const Type *getTypeVal(const ValID &D, bool DoNotImprovise = false) {
   //
   if (DoNotImprovise) return 0;  // Do we just want a null to be returned?
 
-  map<ValID, PATypeHolder> &LateResolver = inFunctionScope() ? 
-    CurMeth.LateResolveTypes : CurModule.LateResolveTypes;
+  std::map<ValID, PATypeHolder> &LateResolver = inFunctionScope() ? 
+    CurFun.LateResolveTypes : CurModule.LateResolveTypes;
   
-  map<ValID, PATypeHolder>::iterator I = LateResolver.find(D);
+  std::map<ValID, PATypeHolder>::iterator I = LateResolver.find(D);
   if (I != LateResolver.end()) {
     return I->second;
   }
 
   Type *Typ = OpaqueType::get();
-  LateResolver.insert(make_pair(D, Typ));
+  LateResolver.insert(std::make_pair(D, Typ));
   return Typ;
 }
 
-static Value *lookupInSymbolTable(const Type *Ty, const string &Name) {
-  SymbolTable *SymTab = 
-    inFunctionScope() ? CurMeth.CurrentFunction->getSymbolTable() :
+static Value *lookupInSymbolTable(const Type *Ty, const std::string &Name) {
+  SymbolTable &SymTab = 
+    inFunctionScope() ? CurFun.CurrentFunction->getSymbolTable() :
                         CurModule.CurrentModule->getSymbolTable();
-  return SymTab ? SymTab->lookup(Ty, Name) : 0;
+  return SymTab.lookup(Ty, Name);
 }
 
 // getValNonImprovising - Look up the value specified by the provided type and
@@ -275,16 +305,16 @@ static Value *getValNonImprovising(const Type *Ty, const ValID &D) {
     }
 
     // Make sure that our type is within bounds
-    if (CurMeth.Values.size() <= type) return 0;
+    if (CurFun.Values.size() <= type) return 0;
 
     // Check that the number is within bounds...
-    if (CurMeth.Values[type].size() <= Num) return 0;
+    if (CurFun.Values[type].size() <= Num) return 0;
   
-    return CurMeth.Values[type][Num];
+    return CurFun.Values[type][Num];
   }
 
   case ValID::NameVal: {                // Is it a named definition?
-    Value *N = lookupInSymbolTable(Ty, string(D.Name));
+    Value *N = lookupInSymbolTable(Ty, std::string(D.Name));
     if (N == 0) return 0;
 
     D.destroy();  // Free old strdup'd memory...
@@ -294,20 +324,17 @@ static Value *getValNonImprovising(const Type *Ty, const ValID &D) {
   // Check to make sure that "Ty" is an integral type, and that our 
   // value will fit into the specified type...
   case ValID::ConstSIntVal:    // Is it a constant pool reference??
-    if (Ty == Type::BoolTy) {  // Special handling for boolean data
-      return ConstantBool::get(D.ConstPool64 != 0);
-    } else {
-      if (!ConstantSInt::isValueValidForType(Ty, D.ConstPool64))
-       ThrowException("Symbolic constant pool value '" +
-                      itostr(D.ConstPool64) + "' is invalid for type '" + 
-                      Ty->getDescription() + "'!");
-      return ConstantSInt::get(Ty, D.ConstPool64);
-    }
+    if (!ConstantSInt::isValueValidForType(Ty, D.ConstPool64))
+      ThrowException("Signed integral constant '" +
+                     itostr(D.ConstPool64) + "' is invalid for type '" + 
+                     Ty->getDescription() + "'!");
+    return ConstantSInt::get(Ty, D.ConstPool64);
 
   case ValID::ConstUIntVal:     // Is it an unsigned const pool reference?
     if (!ConstantUInt::isValueValidForType(Ty, D.UConstPool64)) {
       if (!ConstantSInt::isValueValidForType(Ty, D.ConstPool64)) {
-       ThrowException("Integral constant pool reference is invalid!");
+       ThrowException("Integral constant '" + utostr(D.UConstPool64) +
+                       "' is invalid or out of range!");
       } else {     // This is really a signed reference.  Transmogrify.
        return ConstantSInt::get(Ty, D.ConstPool64);
       }
@@ -325,6 +352,11 @@ static Value *getValNonImprovising(const Type *Ty, const ValID &D) {
       ThrowException("Cannot create a a non pointer null!");
     return ConstantPointerNull::get(cast<PointerType>(Ty));
     
+  case ValID::ConstantVal:       // Fully resolved constant?
+    if (D.ConstantValue->getType() != Ty)
+      ThrowException("Constant expression type different from required type!");
+    return D.ConstantValue;
+
   default:
     assert(0 && "Unhandled case!");
     return 0;
@@ -360,7 +392,7 @@ static Value *getVal(const Type *Ty, const ValID &D) {
 
   assert(d != 0 && "How did we not make something?");
   if (inFunctionScope())
-    InsertValue(d, CurMeth.LateResolveValues);
+    InsertValue(d, CurFun.LateResolveValues);
   else 
     InsertValue(d, CurModule.LateResolveValues);
   return d;
@@ -375,7 +407,7 @@ static Value *getVal(const Type *Ty, const ValID &D) {
 // values not defined yet... for example, a forward branch, or the PHI node for
 // a loop body.
 //
-// This keeps a table (CurMeth.LateResolveValues) of all such forward references
+// This keeps a table (CurFun.LateResolveValues) of all such forward references
 // and back patchs after we are done.
 //
 
@@ -383,8 +415,8 @@ static Value *getVal(const Type *Ty, const ValID &D) {
 // time (forward branches, phi functions for loops, etc...) resolve the 
 // defs now...
 //
-static void ResolveDefinitions(vector<ValueList> &LateResolvers,
-                               vector<ValueList> *FutureLateResolvers = 0) {
+static void ResolveDefinitions(std::vector<ValueList> &LateResolvers,
+                               std::vector<ValueList> *FutureLateResolvers) {
   // Loop over LateResolveDefs fixing up stuff that couldn't be resolved
   for (unsigned ty = 0; ty < LateResolvers.size(); ty++) {
     while (!LateResolvers[ty].empty()) {
@@ -424,19 +456,19 @@ static void ResolveDefinitions(vector<ValueList> &LateResolvers,
 // refering to the number can be resolved.  Do this now.
 //
 static void ResolveTypeTo(char *Name, const Type *ToTy) {
-  vector<PATypeHolder> &Types = inFunctionScope() ? 
-     CurMeth.Types : CurModule.Types;
+  std::vector<PATypeHolder> &Types = inFunctionScope() ? 
+     CurFun.Types : CurModule.Types;
 
    ValID D;
    if (Name) D = ValID::create(Name);
    else      D = ValID::create((int)Types.size());
 
-   map<ValID, PATypeHolder> &LateResolver = inFunctionScope() ? 
-     CurMeth.LateResolveTypes : CurModule.LateResolveTypes;
+   std::map<ValID, PATypeHolder> &LateResolver = inFunctionScope() ? 
+     CurFun.LateResolveTypes : CurModule.LateResolveTypes;
   
-   map<ValID, PATypeHolder>::iterator I = LateResolver.find(D);
+   std::map<ValID, PATypeHolder>::iterator I = LateResolver.find(D);
    if (I != LateResolver.end()) {
-     cast<DerivedType>(I->second.get())->refineAbstractTypeTo(ToTy);
+     ((DerivedType*)I->second.get())->refineAbstractTypeTo(ToTy);
      LateResolver.erase(I);
    }
 }
@@ -444,7 +476,7 @@ static void ResolveTypeTo(char *Name, const Type *ToTy) {
 // ResolveTypes - At this point, all types should be resolved.  Any that aren't
 // are errors.
 //
-static void ResolveTypes(map<ValID, PATypeHolder> &LateResolveTypes) {
+static void ResolveTypes(std::map<ValID, PATypeHolder> &LateResolveTypes) {
   if (!LateResolveTypes.empty()) {
     const ValID &DID = LateResolveTypes.begin()->first;
 
@@ -467,61 +499,84 @@ static void ResolveTypes(map<ValID, PATypeHolder> &LateResolveTypes) {
 static bool setValueName(Value *V, char *NameStr) {
   if (NameStr == 0) return false;
   
-  string Name(NameStr);           // Copy string
+  std::string Name(NameStr);      // Copy string
   free(NameStr);                  // Free old string
 
   if (V->getType() == Type::VoidTy) 
     ThrowException("Can't assign name '" + Name + 
                   "' to a null valued instruction!");
 
-  SymbolTable *ST = inFunctionScope() ? 
-    CurMeth.CurrentFunction->getSymbolTableSure() : 
-    CurModule.CurrentModule->getSymbolTableSure();
+  SymbolTable &ST = inFunctionScope() ? 
+    CurFun.CurrentFunction->getSymbolTable() : 
+    CurModule.CurrentModule->getSymbolTable();
 
-  Value *Existing = ST->lookup(V->getType(), Name);
+  Value *Existing = ST.lookup(V->getType(), Name);
   if (Existing) {    // Inserting a name that is already defined???
     // There is only one case where this is allowed: when we are refining an
     // opaque type.  In this case, Existing will be an opaque type.
-    if (const Type *Ty = dyn_cast<const Type>(Existing)) {
-      if (OpaqueType *OpTy = dyn_cast<OpaqueType>(Ty)) {
+    if (const Type *Ty = dyn_cast<Type>(Existing)) {
+      if (const OpaqueType *OpTy = dyn_cast<OpaqueType>(Ty)) {
        // We ARE replacing an opaque type!
-       OpTy->refineAbstractTypeTo(cast<Type>(V));
+       ((OpaqueType*)OpTy)->refineAbstractTypeTo(cast<Type>(V));
        return true;
       }
     }
 
     // Otherwise, we are a simple redefinition of a value, check to see if it
     // is defined the same as the old one...
-    if (const Type *Ty = dyn_cast<const Type>(Existing)) {
-      if (Ty == cast<const Type>(V)) return true;  // Yes, it's equal.
-      // cerr << "Type: " << Ty->getDescription() << " != "
-      //      << cast<const Type>(V)->getDescription() << "!\n";
+    if (const Type *Ty = dyn_cast<Type>(Existing)) {
+      if (Ty == cast<Type>(V)) return true;  // Yes, it's equal.
+      // std::cerr << "Type: " << Ty->getDescription() << " != "
+      //      << cast<Type>(V)->getDescription() << "!\n";
+    } else if (const Constant *C = dyn_cast<Constant>(Existing)) {
+      if (C == V) return true;      // Constants are equal to themselves
     } else if (GlobalVariable *EGV = dyn_cast<GlobalVariable>(Existing)) {
       // We are allowed to redefine a global variable in two circumstances:
       // 1. If at least one of the globals is uninitialized or 
       // 2. If both initializers have the same value.
       //
-      // This can only be done if the const'ness of the vars is the same.
-      //
       if (GlobalVariable *GV = dyn_cast<GlobalVariable>(V)) {
-        if (EGV->isConstant() == GV->isConstant() &&
-            (!EGV->hasInitializer() || !GV->hasInitializer() ||
-             EGV->getInitializer() == GV->getInitializer())) {
+        if (!EGV->hasInitializer() || !GV->hasInitializer() ||
+             EGV->getInitializer() == GV->getInitializer()) {
 
-          // Make sure the existing global version gets the initializer!
+          // Make sure the existing global version gets the initializer!  Make
+          // sure that it also gets marked const if the new version is.
           if (GV->hasInitializer() && !EGV->hasInitializer())
             EGV->setInitializer(GV->getInitializer());
+          if (GV->isConstant())
+            EGV->setConstant(true);
+          EGV->setLinkage(GV->getLinkage());
           
          delete GV;     // Destroy the duplicate!
           return true;   // They are equivalent!
         }
       }
     }
+
     ThrowException("Redefinition of value named '" + Name + "' in the '" +
                   V->getType()->getDescription() + "' type plane!");
   }
 
-  V->setName(Name, ST);
+  // Set the name
+  V->setName(Name, &ST);
+
+  // If we're in function scope
+  if (inFunctionScope()) {
+    // Look up the symbol in the function's local symboltable
+    Existing = CurFun.LocalSymtab.lookup(V->getType(),Name);
+
+    // If it already exists
+    if (Existing) {
+      // Bail
+      ThrowException("Redefinition of value named '" + Name + "' in the '" +
+                  V->getType()->getDescription() + "' type plane!");
+
+    // otherwise, since it doesn't exist
+    } else {
+      // Insert it.
+      CurFun.LocalSymtab.insert(V);
+    }
+  }
   return false;
 }
 
@@ -530,42 +585,69 @@ static bool setValueName(Value *V, char *NameStr) {
 // Code for handling upreferences in type names...
 //
 
-// TypeContains - Returns true if Ty contains E in it.
+// TypeContains - Returns true if Ty directly contains E in it.
 //
 static bool TypeContains(const Type *Ty, const Type *E) {
-  return find(df_begin(Ty), df_end(Ty), E) != df_end(Ty);
+  return find(Ty->subtype_begin(), Ty->subtype_end(), E) != Ty->subtype_end();
 }
 
+namespace {
+  struct UpRefRecord {
+    // NestingLevel - The number of nesting levels that need to be popped before
+    // this type is resolved.
+    unsigned NestingLevel;
+    
+    // LastContainedTy - This is the type at the current binding level for the
+    // type.  Every time we reduce the nesting level, this gets updated.
+    const Type *LastContainedTy;
+
+    // UpRefTy - This is the actual opaque type that the upreference is
+    // represented with.
+    OpaqueType *UpRefTy;
+
+    UpRefRecord(unsigned NL, OpaqueType *URTy)
+      : NestingLevel(NL), LastContainedTy(URTy), UpRefTy(URTy) {}
+  };
+}
 
-static vector<pair<unsigned, OpaqueType *> > UpRefs;
+// UpRefs - A list of the outstanding upreferences that need to be resolved.
+static std::vector<UpRefRecord> UpRefs;
 
+/// HandleUpRefs - Every time we finish a new layer of types, this function is
+/// called.  It loops through the UpRefs vector, which is a list of the
+/// currently active types.  For each type, if the up reference is contained in
+/// the newly completed type, we decrement the level count.  When the level
+/// count reaches zero, the upreferenced type is the type that is passed in:
+/// thus we can complete the cycle.
+///
 static PATypeHolder HandleUpRefs(const Type *ty) {
+  if (!ty->isAbstract()) return ty;
   PATypeHolder Ty(ty);
-  UR_OUT("Type '" << ty->getDescription() << 
+  UR_OUT("Type '" << Ty->getDescription() << 
          "' newly formed.  Resolving upreferences.\n" <<
          UpRefs.size() << " upreferences active!\n");
-  for (unsigned i = 0; i < UpRefs.size(); ) {
+  for (unsigned i = 0; i != UpRefs.size(); ++i) {
     UR_OUT("  UR#" << i << " - TypeContains(" << Ty->getDescription() << ", " 
           << UpRefs[i].second->getDescription() << ") = " 
-          << (TypeContains(Ty, UpRefs[i].second) ? "true" : "false") << endl);
-    if (TypeContains(Ty, UpRefs[i].second)) {
-      unsigned Level = --UpRefs[i].first;   // Decrement level of upreference
-      UR_OUT("  Uplevel Ref Level = " << Level << endl);
+          << (TypeContains(Ty, UpRefs[i].second) ? "true" : "false") << "\n");
+    if (TypeContains(Ty, UpRefs[i].LastContainedTy)) {
+      // Decrement level of upreference
+      unsigned Level = --UpRefs[i].NestingLevel;
+      UpRefs[i].LastContainedTy = Ty;
+      UR_OUT("  Uplevel Ref Level = " << Level << "\n");
       if (Level == 0) {                     // Upreference should be resolved! 
        UR_OUT("  * Resolving upreference for "
-               << UpRefs[i].second->getDescription() << endl;
-              string OldName = UpRefs[i].second->getDescription());
-       UpRefs[i].second->refineAbstractTypeTo(Ty);
-       UpRefs.erase(UpRefs.begin()+i);     // Remove from upreference list...
+               << UpRefs[i].second->getDescription() << "\n";
+              std::string OldName = UpRefs[i].UpRefTy->getDescription());
+       UpRefs[i].UpRefTy->refineAbstractTypeTo(Ty);
        UR_OUT("  * Type '" << OldName << "' refined upreference to: "
-              << (const void*)Ty << ", " << Ty->getDescription() << endl);
-       continue;
+              << (const void*)Ty << ", " << Ty->getDescription() << "\n");
+       UpRefs.erase(UpRefs.begin()+i);     // Remove from upreference list...
+        --i;                                // Do not skip the next element...
       }
     }
-
-    ++i;                                  // Otherwise, no resolve, move on...
   }
-  // FIXME: TODO: this should return the updated type
+
   return Ty;
 }
 
@@ -574,43 +656,122 @@ static PATypeHolder HandleUpRefs(const Type *ty) {
 //            RunVMAsmParser - Define an interface to this parser
 //===----------------------------------------------------------------------===//
 //
-Module *RunVMAsmParser(const string &Filename, FILE *F) {
+Module *RunVMAsmParser(const std::string &Filename, FILE *F) {
   llvmAsmin = F;
   CurFilename = Filename;
   llvmAsmlineno = 1;      // Reset the current line number...
+  ObsoleteVarArgs = false;
+
+  // Allocate a new module to read
+  CurModule.CurrentModule = new Module(Filename);
+
+  try {
+    yyparse();       // Parse the file.
+  } catch (...) {
+    // Clear the symbol table so it doesn't complain when it
+    // gets destructed
+    CurFun.LocalSymtab.clear();
+    throw;
+  }
 
-  CurModule.CurrentModule = new Module();  // Allocate a new module to read
-  yyparse();       // Parse the file.
   Module *Result = ParserResult;
+
+  // Check to see if they called va_start but not va_arg..
+  if (!ObsoleteVarArgs)
+    if (Function *F = Result->getNamedFunction("llvm.va_start"))
+      if (F->asize() == 1) {
+        std::cerr << "WARNING: this file uses obsolete features.  "
+                  << "Assemble and disassemble to update it.\n";
+        ObsoleteVarArgs = true;
+      }
+
+
+  if (ObsoleteVarArgs) {
+    // If the user is making use of obsolete varargs intrinsics, adjust them for
+    // the user.
+    if (Function *F = Result->getNamedFunction("llvm.va_start")) {
+      assert(F->asize() == 1 && "Obsolete va_start takes 1 argument!");
+
+      const Type *RetTy = F->getFunctionType()->getParamType(0);
+      RetTy = cast<PointerType>(RetTy)->getElementType();
+      Function *NF = Result->getOrInsertFunction("llvm.va_start", RetTy, 0);
+      
+      while (!F->use_empty()) {
+        CallInst *CI = cast<CallInst>(F->use_back());
+        Value *V = new CallInst(NF, "", CI);
+        new StoreInst(V, CI->getOperand(1), CI);
+        CI->getParent()->getInstList().erase(CI);
+      }
+      Result->getFunctionList().erase(F);
+    }
+    
+    if (Function *F = Result->getNamedFunction("llvm.va_end")) {
+      assert(F->asize() == 1 && "Obsolete va_end takes 1 argument!");
+      const Type *ArgTy = F->getFunctionType()->getParamType(0);
+      ArgTy = cast<PointerType>(ArgTy)->getElementType();
+      Function *NF = Result->getOrInsertFunction("llvm.va_end", Type::VoidTy,
+                                                 ArgTy, 0);
+
+      while (!F->use_empty()) {
+        CallInst *CI = cast<CallInst>(F->use_back());
+        Value *V = new LoadInst(CI->getOperand(1), "", CI);
+        new CallInst(NF, V, "", CI);
+        CI->getParent()->getInstList().erase(CI);
+      }
+      Result->getFunctionList().erase(F);
+    }
+
+    if (Function *F = Result->getNamedFunction("llvm.va_copy")) {
+      assert(F->asize() == 2 && "Obsolete va_copy takes 2 argument!");
+      const Type *ArgTy = F->getFunctionType()->getParamType(0);
+      ArgTy = cast<PointerType>(ArgTy)->getElementType();
+      Function *NF = Result->getOrInsertFunction("llvm.va_copy", ArgTy,
+                                                 ArgTy, 0);
+
+      while (!F->use_empty()) {
+        CallInst *CI = cast<CallInst>(F->use_back());
+        Value *V = new CallInst(NF, CI->getOperand(2), "", CI);
+        new StoreInst(V, CI->getOperand(1), CI);
+        CI->getParent()->getInstList().erase(CI);
+      }
+      Result->getFunctionList().erase(F);
+    }
+  }
+
   llvmAsmin = stdin;    // F is about to go away, don't use it anymore...
   ParserResult = 0;
 
   return Result;
 }
 
+} // End llvm namespace
+
+using namespace llvm;
+
 %}
 
 %union {
-  Module                           *ModuleVal;
-  Function                         *FunctionVal;
-  std::pair<Argument*, char*>      *ArgVal;
-  BasicBlock                       *BasicBlockVal;
-  TerminatorInst                   *TermInstVal;
-  Instruction                      *InstVal;
-  Constant                         *ConstVal;
-
-  const Type                       *PrimType;
-  PATypeHolder                     *TypeVal;
-  Value                            *ValueVal;
-
-  std::list<std::pair<Argument*,char*> > *ArgList;
-  std::vector<Value*>              *ValueList;
-  std::list<PATypeHolder>          *TypeList;
-  std::list<std::pair<Value*,
-                      BasicBlock*> > *PHIList; // Represent the RHS of PHI node
-  std::vector<std::pair<Constant*, BasicBlock*> > *JumpTable;
-  std::vector<Constant*>           *ConstVector;
-
+  llvm::Module                           *ModuleVal;
+  llvm::Function                         *FunctionVal;
+  std::pair<llvm::PATypeHolder*, char*>  *ArgVal;
+  llvm::BasicBlock                       *BasicBlockVal;
+  llvm::TerminatorInst                   *TermInstVal;
+  llvm::Instruction                      *InstVal;
+  llvm::Constant                         *ConstVal;
+
+  const llvm::Type                       *PrimType;
+  llvm::PATypeHolder                     *TypeVal;
+  llvm::Value                            *ValueVal;
+
+  std::vector<std::pair<llvm::PATypeHolder*,char*> > *ArgList;
+  std::vector<llvm::Value*>              *ValueList;
+  std::list<llvm::PATypeHolder>          *TypeList;
+  std::list<std::pair<llvm::Value*,
+                      llvm::BasicBlock*> > *PHIList; // Represent the RHS of PHI node
+  std::vector<std::pair<llvm::Constant*, llvm::BasicBlock*> > *JumpTable;
+  std::vector<llvm::Constant*>           *ConstVector;
+
+  llvm::GlobalValue::LinkageTypes         Linkage;
   int64_t                           SInt64Val;
   uint64_t                          UInt64Val;
   int                               SIntVal;
@@ -619,13 +780,13 @@ Module *RunVMAsmParser(const string &Filename, FILE *F) {
   bool                              BoolVal;
 
   char                             *StrVal;   // This memory is strdup'd!
-  ValID                             ValIDVal; // strdup'd memory maybe!
+  llvm::ValID                             ValIDVal; // strdup'd memory maybe!
 
-  Instruction::UnaryOps             UnaryOpVal;
-  Instruction::BinaryOps            BinaryOpVal;
-  Instruction::TermOps              TermOpVal;
-  Instruction::MemoryOps            MemOpVal;
-  Instruction::OtherOps             OtherOpVal;
+  llvm::Instruction::BinaryOps            BinaryOpVal;
+  llvm::Instruction::TermOps              TermOpVal;
+  llvm::Instruction::MemoryOps            MemOpVal;
+  llvm::Instruction::OtherOps             OtherOpVal;
+  llvm::Module::Endianness                Endianness;
 }
 
 %type <ModuleVal>     Module FunctionList
@@ -633,7 +794,7 @@ Module *RunVMAsmParser(const string &Filename, FILE *F) {
 %type <BasicBlockVal> BasicBlock InstructionList
 %type <TermInstVal>   BBTerminatorInst
 %type <InstVal>       Inst InstVal MemoryInst
-%type <ConstVal>      ConstVal
+%type <ConstVal>      ConstVal ConstExpr
 %type <ConstVector>   ConstVector
 %type <ArgList>       ArgList ArgListH
 %type <ArgVal>        ArgVal
@@ -642,7 +803,10 @@ Module *RunVMAsmParser(const string &Filename, FILE *F) {
 %type <ValueList>     IndexList                   // For GEP derived indices
 %type <TypeList>      TypeListI ArgTypeListI
 %type <JumpTable>     JumpTable
-%type <BoolVal>       GlobalType OptInternal      // GLOBAL or CONSTANT? Intern?
+%type <BoolVal>       GlobalType                  // GLOBAL or CONSTANT?
+%type <BoolVal>       OptVolatile                 // 'volatile' or not
+%type <Linkage>       OptLinkage
+%type <Endianness>    BigOrLittle
 
 // ValueRef - Unresolved reference to a definition or BB
 %type <ValIDVal>      ValueRef ConstValueRef SymbolicValueRef
@@ -667,178 +831,185 @@ Module *RunVMAsmParser(const string &Filename, FILE *F) {
 %token <PrimType> VOID BOOL SBYTE UBYTE SHORT USHORT INT UINT LONG ULONG
 %token <PrimType> FLOAT DOUBLE TYPE LABEL
 
-%token <StrVal>     VAR_ID LABELSTR STRINGCONSTANT
-%type  <StrVal>  OptVAR_ID OptAssign FuncName
+%token <StrVal> VAR_ID LABELSTR STRINGCONSTANT
+%type  <StrVal> Name OptName OptAssign
 
 
-%token IMPLEMENTATION TRUE FALSE BEGINTOK ENDTOK DECLARE GLOBAL CONSTANT UNINIT
-%token TO EXCEPT DOTDOTDOT STRING NULL_TOK CONST INTERNAL OPAQUE
+%token IMPLEMENTATION ZEROINITIALIZER TRUE FALSE BEGINTOK ENDTOK
+%token DECLARE GLOBAL CONSTANT VOLATILE
+%token TO DOTDOTDOT NULL_TOK CONST INTERNAL LINKONCE WEAK  APPENDING
+%token OPAQUE NOT EXTERNAL TARGET ENDIAN POINTERSIZE LITTLE BIG
 
 // Basic Block Terminating Operators 
-%token <TermOpVal> RET BR SWITCH
-
-// Unary Operators 
-%type  <UnaryOpVal> UnaryOps  // all the unary operators
-%token <UnaryOpVal> NOT
+%token <TermOpVal> RET BR SWITCH INVOKE UNWIND
 
 // Binary Operators 
 %type  <BinaryOpVal> BinaryOps  // all the binary operators
+%type  <BinaryOpVal> ArithmeticOps LogicalOps SetCondOps // Binops Subcatagories
 %token <BinaryOpVal> ADD SUB MUL DIV REM AND OR XOR
 %token <BinaryOpVal> SETLE SETGE SETLT SETGT SETEQ SETNE  // Binary Comarators
 
 // Memory Instructions
-%token <MemoryOpVal> MALLOC ALLOCA FREE LOAD STORE GETELEMENTPTR
+%token <MemOpVal> MALLOC ALLOCA FREE LOAD STORE GETELEMENTPTR
 
 // Other Operators
 %type  <OtherOpVal> ShiftOps
-%token <OtherOpVal> PHI CALL INVOKE CAST SHL SHR
+%token <OtherOpVal> PHI_TOK CALL CAST SHL SHR VAARG VANEXT
+%token VA_ARG // FIXME: OBSOLETE
 
 %start Module
 %%
 
 // Handle constant integer size restriction and conversion...
 //
-
-INTVAL : SINTVAL
+INTVAL : SINTVAL;
 INTVAL : UINTVAL {
   if ($1 > (uint32_t)INT32_MAX)     // Outside of my range!
     ThrowException("Value too large for type!");
   $$ = (int32_t)$1;
-}
+};
 
 
-EINT64VAL : ESINT64VAL       // These have same type and can't cause problems...
+EINT64VAL : ESINT64VAL;      // These have same type and can't cause problems...
 EINT64VAL : EUINT64VAL {
   if ($1 > (uint64_t)INT64_MAX)     // Outside of my range!
     ThrowException("Value too large for type!");
   $$ = (int64_t)$1;
-}
+};
 
 // Operations that are notably excluded from this list include: 
 // RET, BR, & SWITCH because they end basic blocks and are treated specially.
 //
-UnaryOps  : NOT
-BinaryOps : ADD | SUB | MUL | DIV | REM | AND | OR | XOR
-BinaryOps : SETLE | SETGE | SETLT | SETGT | SETEQ | SETNE
-ShiftOps  : SHL | SHR
+ArithmeticOps: ADD | SUB | MUL | DIV | REM;
+LogicalOps   : AND | OR | XOR;
+SetCondOps   : SETLE | SETGE | SETLT | SETGT | SETEQ | SETNE;
+BinaryOps : ArithmeticOps | LogicalOps | SetCondOps;
+
+ShiftOps  : SHL | SHR;
 
 // These are some types that allow classification if we only want a particular 
 // thing... for example, only a signed, unsigned, or integral type.
-SIntType :  LONG |  INT |  SHORT | SBYTE
-UIntType : ULONG | UINT | USHORT | UBYTE
-IntType  : SIntType | UIntType
-FPType   : FLOAT | DOUBLE
+SIntType :  LONG |  INT |  SHORT | SBYTE;
+UIntType : ULONG | UINT | USHORT | UBYTE;
+IntType  : SIntType | UIntType;
+FPType   : FLOAT | DOUBLE;
 
 // OptAssign - Value producing statements have an optional assignment component
-OptAssign : VAR_ID '=' {
+OptAssign : Name '=' {
     $$ = $1;
   }
   | /*empty*/ { 
     $$ = 0; 
-  }
+  };
 
-OptInternal : INTERNAL { $$ = true; } | /*empty*/ { $$ = false; }
+OptLinkage : INTERNAL  { $$ = GlobalValue::InternalLinkage; } |
+             LINKONCE  { $$ = GlobalValue::LinkOnceLinkage; } |
+             WEAK      { $$ = GlobalValue::WeakLinkage; } |
+             APPENDING { $$ = GlobalValue::AppendingLinkage; } |
+             /*empty*/ { $$ = GlobalValue::ExternalLinkage; };
 
 //===----------------------------------------------------------------------===//
 // Types includes all predefined types... except void, because it can only be
-// used in specific contexts (method returning void for example).  To have
+// used in specific contexts (function returning void for example).  To have
 // access to it, a user must explicitly use TypesV.
 //
 
 // TypesV includes all of 'Types', but it also includes the void type.
-TypesV    : Types    | VOID { $$ = new PATypeHolder($1); }
-UpRTypesV : UpRTypes | VOID { $$ = new PATypeHolder($1); }
+TypesV    : Types    | VOID { $$ = new PATypeHolder($1); };
+UpRTypesV : UpRTypes | VOID { $$ = new PATypeHolder($1); };
 
 Types     : UpRTypes {
-    if (UpRefs.size())
+    if (!UpRefs.empty())
       ThrowException("Invalid upreference in type: " + (*$1)->getDescription());
     $$ = $1;
-  }
+  };
 
 
 // Derived types are added later...
 //
-PrimType : BOOL | SBYTE | UBYTE | SHORT  | USHORT | INT   | UINT 
-PrimType : LONG | ULONG | FLOAT | DOUBLE | TYPE   | LABEL
+PrimType : BOOL | SBYTE | UBYTE | SHORT  | USHORT | INT   | UINT ;
+PrimType : LONG | ULONG | FLOAT | DOUBLE | TYPE   | LABEL;
 UpRTypes : OPAQUE {
     $$ = new PATypeHolder(OpaqueType::get());
   }
   | PrimType {
     $$ = new PATypeHolder($1);
-  }
-UpRTypes : ValueRef {                    // Named types are also simple types...
+  };
+UpRTypes : SymbolicValueRef {            // Named types are also simple types...
   $$ = new PATypeHolder(getTypeVal($1));
-}
+};
 
 // Include derived types in the Types production.
 //
 UpRTypes : '\\' EUINT64VAL {                   // Type UpReference
-    if ($2 > (uint64_t)INT64_MAX) ThrowException("Value out of range!");
+    if ($2 > (uint64_t)~0U) ThrowException("Value out of range!");
     OpaqueType *OT = OpaqueType::get();        // Use temporary placeholder
-    UpRefs.push_back(make_pair((unsigned)$2, OT));  // Add to vector...
+    UpRefs.push_back(UpRefRecord((unsigned)$2, OT));  // Add to vector...
     $$ = new PATypeHolder(OT);
     UR_OUT("New Upreference!\n");
   }
   | UpRTypesV '(' ArgTypeListI ')' {           // Function derived type?
-    vector<const Type*> Params;
+    std::vector<const Type*> Params;
     mapto($3->begin(), $3->end(), std::back_inserter(Params), 
-         std::mem_fun_ref(&PATypeHandle<Type>::get));
+         std::mem_fun_ref(&PATypeHolder::get));
     bool isVarArg = Params.size() && Params.back() == Type::VoidTy;
     if (isVarArg) Params.pop_back();
 
     $$ = new PATypeHolder(HandleUpRefs(FunctionType::get(*$1,Params,isVarArg)));
     delete $3;      // Delete the argument list
-    delete $1;      // Delete the old type handle
+    delete $1;      // Delete the return type handle
   }
   | '[' EUINT64VAL 'x' UpRTypes ']' {          // Sized array type?
     $$ = new PATypeHolder(HandleUpRefs(ArrayType::get(*$4, (unsigned)$2)));
     delete $4;
   }
   | '{' TypeListI '}' {                        // Structure type?
-    vector<const Type*> Elements;
+    std::vector<const Type*> Elements;
     mapto($2->begin(), $2->end(), std::back_inserter(Elements), 
-       std::mem_fun_ref(&PATypeHandle<Type>::get));
+       std::mem_fun_ref(&PATypeHolder::get));
 
     $$ = new PATypeHolder(HandleUpRefs(StructType::get(Elements)));
     delete $2;
   }
   | '{' '}' {                                  // Empty structure type?
-    $$ = new PATypeHolder(StructType::get(vector<const Type*>()));
+    $$ = new PATypeHolder(StructType::get(std::vector<const Type*>()));
   }
   | UpRTypes '*' {                             // Pointer type?
     $$ = new PATypeHolder(HandleUpRefs(PointerType::get(*$1)));
     delete $1;
-  }
+  };
 
-// TypeList - Used for struct declarations and as a basis for method type 
+// TypeList - Used for struct declarations and as a basis for function type 
 // declaration type lists
 //
 TypeListI : UpRTypes {
-    $$ = new list<PATypeHolder>();
+    $$ = new std::list<PATypeHolder>();
     $$->push_back(*$1); delete $1;
   }
   | TypeListI ',' UpRTypes {
     ($$=$1)->push_back(*$3); delete $3;
-  }
+  };
 
-// ArgTypeList - List of types for a method type declaration...
+// ArgTypeList - List of types for a function type declaration...
 ArgTypeListI : TypeListI
   | TypeListI ',' DOTDOTDOT {
     ($$=$1)->push_back(Type::VoidTy);
   }
   | DOTDOTDOT {
-    ($$ = new list<PATypeHolder>())->push_back(Type::VoidTy);
+    ($$ = new std::list<PATypeHolder>())->push_back(Type::VoidTy);
   }
   | /*empty*/ {
-    $$ = new list<PATypeHolder>();
-  }
-
+    $$ = new std::list<PATypeHolder>();
+  };
 
 // ConstVal - The various declarations that go into the constant pool.  This
-// includes all forward declarations of types, constants, and functions.
+// production is used ONLY to represent constants that show up AFTER a 'const',
+// 'constant' or 'global' token at global scope.  Constants that can be inlined
+// into other expressions (such as integers and constexprs) are handled by the
+// ResolvedVal, ValueRef and ConstValueRef productions.
 //
 ConstVal: Types '[' ConstVector ']' { // Nonempty unsized arr
-    const ArrayType *ATy = dyn_cast<const ArrayType>($1->get());
+    const ArrayType *ATy = dyn_cast<ArrayType>($1->get());
     if (ATy == 0)
       ThrowException("Cannot make array constant with type: '" + 
                      (*$1)->getDescription() + "'!");
@@ -863,7 +1034,7 @@ ConstVal: Types '[' ConstVector ']' { // Nonempty unsized arr
     delete $1; delete $3;
   }
   | Types '[' ']' {
-    const ArrayType *ATy = dyn_cast<const ArrayType>($1->get());
+    const ArrayType *ATy = dyn_cast<ArrayType>($1->get());
     if (ATy == 0)
       ThrowException("Cannot make array constant with type: '" + 
                      (*$1)->getDescription() + "'!");
@@ -872,11 +1043,11 @@ ConstVal: Types '[' ConstVector ']' { // Nonempty unsized arr
     if (NumElements != -1 && NumElements != 0) 
       ThrowException("Type mismatch: constant sized array initialized with 0"
                     " arguments, but has size of " + itostr(NumElements) +"!");
-    $$ = ConstantArray::get(ATy, vector<Constant*>());
+    $$ = ConstantArray::get(ATy, std::vector<Constant*>());
     delete $1;
   }
   | Types 'c' STRINGCONSTANT {
-    const ArrayType *ATy = dyn_cast<const ArrayType>($1->get());
+    const ArrayType *ATy = dyn_cast<ArrayType>($1->get());
     if (ATy == 0)
       ThrowException("Cannot make array constant with type: '" + 
                      (*$1)->getDescription() + "'!");
@@ -888,13 +1059,13 @@ ConstVal: Types '[' ConstVector ']' { // Nonempty unsized arr
       ThrowException("Can't build string constant of size " + 
                     itostr((int)(EndStr-$3)) +
                     " when array has size " + itostr(NumElements) + "!");
-    vector<Constant*> Vals;
+    std::vector<Constant*> Vals;
     if (ETy == Type::SByteTy) {
       for (char *C = $3; C != EndStr; ++C)
        Vals.push_back(ConstantSInt::get(ETy, *C));
     } else if (ETy == Type::UByteTy) {
       for (char *C = $3; C != EndStr; ++C)
-       Vals.push_back(ConstantUInt::get(ETy, *C));
+       Vals.push_back(ConstantUInt::get(ETy, (unsigned char)*C));
     } else {
       free($3);
       ThrowException("Cannot build string arrays of non byte sized elements!");
@@ -904,17 +1075,39 @@ ConstVal: Types '[' ConstVector ']' { // Nonempty unsized arr
     delete $1;
   }
   | Types '{' ConstVector '}' {
-    const StructType *STy = dyn_cast<const StructType>($1->get());
+    const StructType *STy = dyn_cast<StructType>($1->get());
     if (STy == 0)
       ThrowException("Cannot make struct constant with type: '" + 
                      (*$1)->getDescription() + "'!");
-    // FIXME: TODO: Check to see that the constants are compatible with the type
-    // initializer!
+
+    if ($3->size() != STy->getNumContainedTypes())
+      ThrowException("Illegal number of initializers for structure type!");
+
+    // Check to ensure that constants are compatible with the type initializer!
+    for (unsigned i = 0, e = $3->size(); i != e; ++i)
+      if ((*$3)[i]->getType() != STy->getElementType(i))
+        ThrowException("Expected type '" +
+                       STy->getElementType(i)->getDescription() +
+                       "' for element #" + utostr(i) +
+                       " of structure initializer!");
+
     $$ = ConstantStruct::get(STy, *$3);
     delete $1; delete $3;
   }
+  | Types '{' '}' {
+    const StructType *STy = dyn_cast<StructType>($1->get());
+    if (STy == 0)
+      ThrowException("Cannot make struct constant with type: '" + 
+                     (*$1)->getDescription() + "'!");
+
+    if (STy->getNumContainedTypes() != 0)
+      ThrowException("Illegal number of initializers for structure type!");
+
+    $$ = ConstantStruct::get(STy, std::vector<Constant*>());
+    delete $1;
+  }
   | Types NULL_TOK {
-    const PointerType *PTy = dyn_cast<const PointerType>($1->get());
+    const PointerType *PTy = dyn_cast<PointerType>($1->get());
     if (PTy == 0)
       ThrowException("Cannot make null pointer constant with type: '" + 
                      (*$1)->getDescription() + "'!");
@@ -923,12 +1116,24 @@ ConstVal: Types '[' ConstVector ']' { // Nonempty unsized arr
     delete $1;
   }
   | Types SymbolicValueRef {
-    const PointerType *Ty = dyn_cast<const PointerType>($1->get());
+    const PointerType *Ty = dyn_cast<PointerType>($1->get());
     if (Ty == 0)
       ThrowException("Global const reference must be a pointer type!");
 
+    // ConstExprs can exist in the body of a function, thus creating
+    // ConstantPointerRefs whenever they refer to a variable.  Because we are in
+    // the context of a function, getValNonImprovising will search the functions
+    // symbol table instead of the module symbol table for the global symbol,
+    // which throws things all off.  To get around this, we just tell
+    // getValNonImprovising that we are at global scope here.
+    //
+    Function *SavedCurFn = CurFun.CurrentFunction;
+    CurFun.CurrentFunction = 0;
+
     Value *V = getValNonImprovising(Ty, $2);
 
+    CurFun.CurrentFunction = SavedCurFn;
+
     // If this is an initializer for a constant pointer, which is referencing a
     // (currently) undefined variable, create a stub now that shall be replaced
     // in the future with the right type of variable.
@@ -939,19 +1144,18 @@ ConstVal: Types '[' ConstVector ']' { // Nonempty unsized arr
 
       // First check to see if the forward references value is already created!
       PerModuleInfo::GlobalRefsType::iterator I =
-       CurModule.GlobalRefs.find(make_pair(PT, $2));
+       CurModule.GlobalRefs.find(std::make_pair(PT, $2));
     
       if (I != CurModule.GlobalRefs.end()) {
        V = I->second;             // Placeholder already exists, use it...
+        $2.destroy();
       } else {
-       // TODO: Include line number info by creating a subclass of
-       // TODO: GlobalVariable here that includes the said information!
-       
        // Create a placeholder for the global variable reference...
        GlobalVariable *GV = new GlobalVariable(PT->getElementType(),
-                                                false, true);
+                                                false,
+                                                GlobalValue::ExternalLinkage);
        // Keep track of the fact that we have a forward ref to recycle it
-       CurModule.GlobalRefs.insert(make_pair(make_pair(PT, $2), GV));
+       CurModule.GlobalRefs.insert(std::make_pair(std::make_pair(PT, $2), GV));
 
        // Must temporarily push this value into the module table...
        CurModule.CurrentModule->getGlobalList().push_back(GV);
@@ -963,40 +1167,94 @@ ConstVal: Types '[' ConstVector ']' { // Nonempty unsized arr
     $$ = ConstantPointerRef::get(GV);
     delete $1;            // Free the type handle
   }
+  | Types ConstExpr {
+    if ($1->get() != $2->getType())
+      ThrowException("Mismatched types for constant expression!");
+    $$ = $2;
+    delete $1;
+  }
+  | Types ZEROINITIALIZER {
+    $$ = Constant::getNullValue($1->get());
+    delete $1;
+  };
 
-
-ConstVal : SIntType EINT64VAL {     // integral constants
+ConstVal : SIntType EINT64VAL {      // integral constants
     if (!ConstantSInt::isValueValidForType($1, $2))
       ThrowException("Constant value doesn't fit in type!");
     $$ = ConstantSInt::get($1, $2);
-  } 
-  | UIntType EUINT64VAL {           // integral constants
+  }
+  | UIntType EUINT64VAL {            // integral constants
     if (!ConstantUInt::isValueValidForType($1, $2))
       ThrowException("Constant value doesn't fit in type!");
     $$ = ConstantUInt::get($1, $2);
-  } 
-  | BOOL TRUE {                     // Boolean constants
+  }
+  | BOOL TRUE {                      // Boolean constants
     $$ = ConstantBool::True;
   }
-  | BOOL FALSE {                    // Boolean constants
+  | BOOL FALSE {                     // Boolean constants
     $$ = ConstantBool::False;
   }
   | FPType FPVAL {                   // Float & Double constants
     $$ = ConstantFP::get($1, $2);
+  };
+
+
+ConstExpr: CAST '(' ConstVal TO Types ')' {
+    if (!$3->getType()->isFirstClassType())
+      ThrowException("cast constant expression from a non-primitive type: '" +
+                     $3->getType()->getDescription() + "'!");
+    if (!$5->get()->isFirstClassType())
+      ThrowException("cast constant expression to a non-primitive type: '" +
+                     $5->get()->getDescription() + "'!");
+    $$ = ConstantExpr::getCast($3, $5->get());
+    delete $5;
   }
+  | GETELEMENTPTR '(' ConstVal IndexList ')' {
+    if (!isa<PointerType>($3->getType()))
+      ThrowException("GetElementPtr requires a pointer operand!");
+
+    const Type *IdxTy =
+      GetElementPtrInst::getIndexedType($3->getType(), *$4, true);
+    if (!IdxTy)
+      ThrowException("Index list invalid for constant getelementptr!");
 
-// ConstVector - A list of comma seperated constants.
+    std::vector<Constant*> IdxVec;
+    for (unsigned i = 0, e = $4->size(); i != e; ++i)
+      if (Constant *C = dyn_cast<Constant>((*$4)[i]))
+        IdxVec.push_back(C);
+      else
+        ThrowException("Indices to constant getelementptr must be constants!");
+
+    delete $4;
+
+    $$ = ConstantExpr::getGetElementPtr($3, IdxVec);
+  }
+  | BinaryOps '(' ConstVal ',' ConstVal ')' {
+    if ($3->getType() != $5->getType())
+      ThrowException("Binary operator types must match!");
+    $$ = ConstantExpr::get($1, $3, $5);
+  }
+  | ShiftOps '(' ConstVal ',' ConstVal ')' {
+    if ($5->getType() != Type::UByteTy)
+      ThrowException("Shift count for shift constant must be unsigned byte!");
+    if (!$3->getType()->isInteger())
+      ThrowException("Shift constant expression requires integer operand!");
+    $$ = ConstantExpr::get($1, $3, $5);
+  };
+
+
+// ConstVector - A list of comma separated constants.
 ConstVector : ConstVector ',' ConstVal {
     ($$ = $1)->push_back($3);
   }
   | ConstVal {
-    $$ = new vector<Constant*>();
+    $$ = new std::vector<Constant*>();
     $$->push_back($1);
-  }
+  };
 
 
 // GlobalType - Match either GLOBAL or CONSTANT for global declarations...
-GlobalType : GLOBAL { $$ = false; } | CONSTANT { $$ = true; }
+GlobalType : GLOBAL { $$ = false; } | CONSTANT { $$ = true; };
 
 
 //===----------------------------------------------------------------------===//
@@ -1009,15 +1267,15 @@ GlobalType : GLOBAL { $$ = false; } | CONSTANT { $$ = true; }
 Module : FunctionList {
   $$ = ParserResult = $1;
   CurModule.ModuleDone();
-}
+};
 
-// FunctionList - A list of methods, preceeded by a constant pool.
+// FunctionList - A list of functions, preceeded by a constant pool.
 //
 FunctionList : FunctionList Function {
     $$ = $1;
     assert($2->getParent() == 0 && "Function already in module!");
     $1->getFunctionList().push_back($2);
-    CurMeth.FunctionDone();
+    CurFun.FunctionDone();
   } 
   | FunctionList FunctionProto {
     $$ = $1;
@@ -1029,12 +1287,12 @@ FunctionList : FunctionList Function {
     $$ = CurModule.CurrentModule;
     // Resolve circular types before we parse the body of the module
     ResolveTypes(CurModule.LateResolveTypes);
-  }
+  };
 
 // ConstPool - Constants with optional names assigned to them.
 ConstPool : ConstPool OptAssign CONST ConstVal { 
-    if (setValueName($4, $2)) { assert(0 && "No redefinitions allowed!"); }
-    InsertValue($4);
+    if (!setValueName($4, $2))
+      InsertValue($4);
   }
   | ConstPool OptAssign TYPE TypesV {  // Types can be defined in the const pool
     // Eagerly resolve types.  This is not an optimization, this is a
@@ -1053,7 +1311,7 @@ ConstPool : ConstPool OptAssign CONST ConstVal {
       // If this is not a redefinition of a type...
       if (!$2) {
         InsertType($4->get(),
-                   inFunctionScope() ? CurMeth.Types : CurModule.Types);
+                   inFunctionScope() ? CurFun.Types : CurModule.Types);
       }
     }
 
@@ -1061,13 +1319,13 @@ ConstPool : ConstPool OptAssign CONST ConstVal {
   }
   | ConstPool FunctionProto {       // Function prototypes can be in const pool
   }
-  | ConstPool OptAssign OptInternal GlobalType ConstVal {
+  | ConstPool OptAssign OptLinkage GlobalType ConstVal {
     const Type *Ty = $5->getType();
     // Global declarations appear in Constant Pool
     Constant *Initializer = $5;
     if (Initializer == 0)
       ThrowException("Global value initializer is not a constant!");
-        
+    
     GlobalVariable *GV = new GlobalVariable(Ty, $4, $3, Initializer);
     if (!setValueName(GV, $2)) {   // If not redefining...
       CurModule.CurrentModule->getGlobalList().push_back(GV);
@@ -1081,10 +1339,10 @@ ConstPool : ConstPool OptAssign CONST ConstVal {
       }
     }
   }
-  | ConstPool OptAssign OptInternal UNINIT GlobalType Types {
-    const Type *Ty = *$6;
+  | ConstPool OptAssign EXTERNAL GlobalType Types {
+    const Type *Ty = *$5;
     // Global declarations appear in Constant Pool
-    GlobalVariable *GV = new GlobalVariable(Ty, $5, $3);
+    GlobalVariable *GV = new GlobalVariable(Ty,$4,GlobalValue::ExternalLinkage);
     if (!setValueName(GV, $2)) {   // If not redefining...
       CurModule.CurrentModule->getGlobalList().push_back(GV);
       int Slot = InsertValue(GV, CurModule.Values);
@@ -1097,137 +1355,167 @@ ConstPool : ConstPool OptAssign CONST ConstVal {
                                                (char*)GV->getName().c_str()));
       }
     }
-    delete $6;
+    delete $5;
+  }
+  | ConstPool TARGET TargetDefinition { 
   }
   | /* empty: end of list */ { 
+  };
+
+
+
+BigOrLittle : BIG    { $$ = Module::BigEndian; };
+BigOrLittle : LITTLE { $$ = Module::LittleEndian; };
+
+TargetDefinition : ENDIAN '=' BigOrLittle {
+    CurModule.CurrentModule->setEndianness($3);
   }
+  | POINTERSIZE '=' EUINT64VAL {
+    if ($3 == 32)
+      CurModule.CurrentModule->setPointerSize(Module::Pointer32);
+    else if ($3 == 64)
+      CurModule.CurrentModule->setPointerSize(Module::Pointer64);
+    else
+      ThrowException("Invalid pointer size: '" + utostr($3) + "'!");
+  };
 
 
 //===----------------------------------------------------------------------===//
 //                       Rules to match Function Headers
 //===----------------------------------------------------------------------===//
 
-OptVAR_ID : VAR_ID | /*empty*/ { $$ = 0; }
+Name : VAR_ID | STRINGCONSTANT;
+OptName : Name | /*empty*/ { $$ = 0; };
 
-ArgVal : Types OptVAR_ID {
-  $$ = new pair<Argument*, char*>(new Argument(*$1), $2);
-  delete $1;  // Delete the type handle..
-}
+ArgVal : Types OptName {
+  if (*$1 == Type::VoidTy)
+    ThrowException("void typed arguments are invalid!");
+  $$ = new std::pair<PATypeHolder*, char*>($1, $2);
+};
 
-ArgListH : ArgVal ',' ArgListH {
-    $$ = $3;
-    $3->push_front(*$1);
-    delete $1;
+ArgListH : ArgListH ',' ArgVal {
+    $$ = $1;
+    $1->push_back(*$3);
+    delete $3;
   }
   | ArgVal {
-    $$ = new list<pair<Argument*,char*> >();
-    $$->push_front(*$1);
+    $$ = new std::vector<std::pair<PATypeHolder*,char*> >();
+    $$->push_back(*$1);
     delete $1;
-  }
-  | DOTDOTDOT {
-    $$ = new list<pair<Argument*, char*> >();
-    $$->push_front(pair<Argument*,char*>(new Argument(Type::VoidTy), 0));
-  }
+  };
 
 ArgList : ArgListH {
     $$ = $1;
   }
+  | ArgListH ',' DOTDOTDOT {
+    $$ = $1;
+    $$->push_back(std::pair<PATypeHolder*,
+                            char*>(new PATypeHolder(Type::VoidTy), 0));
+  }
+  | DOTDOTDOT {
+    $$ = new std::vector<std::pair<PATypeHolder*,char*> >();
+    $$->push_back(std::make_pair(new PATypeHolder(Type::VoidTy), (char*)0));
+  }
   | /* empty */ {
     $$ = 0;
-  }
+  };
 
-FuncName : VAR_ID | STRINGCONSTANT;
-
-FunctionHeaderH : OptInternal TypesV FuncName '(' ArgList ')' {
-  UnEscapeLexed($3);
-  string FunctionName($3);
+FunctionHeaderH : TypesV Name '(' ArgList ')' {
+  UnEscapeLexed($2);
+  std::string FunctionName($2);
   
-  vector<const Type*> ParamTypeList;
-  if ($5)
-    for (list<pair<Argument*,char*> >::iterator I = $5->begin();
-         I != $5->end(); ++I)
-      ParamTypeList.push_back(I->first->getType());
+  if (!(*$1)->isFirstClassType() && *$1 != Type::VoidTy)
+    ThrowException("LLVM functions cannot return aggregate types!");
+
+  std::vector<const Type*> ParamTypeList;
+  if ($4) {   // If there are arguments...
+    for (std::vector<std::pair<PATypeHolder*,char*> >::iterator I = $4->begin();
+         I != $4->end(); ++I)
+      ParamTypeList.push_back(I->first->get());
+  }
 
   bool isVarArg = ParamTypeList.size() && ParamTypeList.back() == Type::VoidTy;
   if (isVarArg) ParamTypeList.pop_back();
 
-  const FunctionType *MT = FunctionType::get(*$2, ParamTypeList, isVarArg);
-  const PointerType *PMT = PointerType::get(MT);
-  delete $2;
-
-  Function *M = 0;
-  if (SymbolTable *ST = CurModule.CurrentModule->getSymbolTable()) {
-    // Is the function already in symtab?
-    if (Value *V = ST->lookup(PMT, FunctionName)) {
-      M = cast<Function>(V);
-
-      // Yes it is.  If this is the case, either we need to be a forward decl,
-      // or it needs to be.
-      if (!CurMeth.isDeclare && !M->isExternal())
-       ThrowException("Redefinition of method '" + FunctionName + "'!");      
+  const FunctionType *FT = FunctionType::get(*$1, ParamTypeList, isVarArg);
+  const PointerType *PFT = PointerType::get(FT);
+  delete $1;
+
+  Function *Fn = 0;
+  // Is the function already in symtab?
+  if ((Fn = CurModule.CurrentModule->getFunction(FunctionName, FT))) {
+    // Yes it is.  If this is the case, either we need to be a forward decl,
+    // or it needs to be.
+    if (!CurFun.isDeclare && !Fn->isExternal())
+      ThrowException("Redefinition of function '" + FunctionName + "'!");
+    
+    // If we found a preexisting function prototype, remove it from the
+    // module, so that we don't get spurious conflicts with global & local
+    // variables.
+    //
+    CurModule.CurrentModule->getFunctionList().remove(Fn);
 
-      // If we found a preexisting method prototype, remove it from the module,
-      // so that we don't get spurious conflicts with global & local variables.
-      //
-      CurModule.CurrentModule->getFunctionList().remove(M);
-    }
-  }
+    // Make sure to strip off any argument names so we can't get conflicts...
+    for (Function::aiterator AI = Fn->abegin(), AE = Fn->aend(); AI != AE; ++AI)
+      AI->setName("");
 
-  if (M == 0) {  // Not already defined?
-    M = new Function(MT, $1, FunctionName);
-    InsertValue(M, CurModule.Values);
-    CurModule.DeclareNewGlobalValue(M, ValID::create($3));
+  } else  {  // Not already defined?
+    Fn = new Function(FT, GlobalValue::ExternalLinkage, FunctionName);
+    InsertValue(Fn, CurModule.Values);
+    CurModule.DeclareNewGlobalValue(Fn, ValID::create($2));
   }
-  free($3);  // Free strdup'd memory!
+  free($2);  // Free strdup'd memory!
 
-  CurMeth.FunctionStart(M);
+  CurFun.FunctionStart(Fn);
 
-  // Add all of the arguments we parsed to the method...
-  if ($5 && !CurMeth.isDeclare) {        // Is null if empty...
-    Function::ArgumentListType &ArgList = M->getArgumentList();
+  // Add all of the arguments we parsed to the function...
+  if ($4) {                     // Is null if empty...
+    if (isVarArg) {  // Nuke the last entry
+      assert($4->back().first->get() == Type::VoidTy && $4->back().second == 0&&
+             "Not a varargs marker!");
+      delete $4->back().first;
+      $4->pop_back();  // Delete the last entry
+    }
+    Function::aiterator ArgIt = Fn->abegin();
+    for (std::vector<std::pair<PATypeHolder*, char*> >::iterator I =$4->begin();
+         I != $4->end(); ++I, ++ArgIt) {
+      delete I->first;                          // Delete the typeholder...
 
-    for (list<pair<Argument*, char*> >::iterator I = $5->begin();
-         I != $5->end(); ++I) {
-      if (setValueName(I->first, I->second)) {  // Insert into symtab...
+      if (setValueName(ArgIt, I->second))       // Insert arg into symtab...
         assert(0 && "No arg redef allowed!");
-      }
       
-      InsertValue(I->first);
-      ArgList.push_back(I->first);
-    }
-    delete $5;                     // We're now done with the argument list
-  } else if ($5) {
-    // If we are a declaration, we should free the memory for the argument list!
-    for (list<pair<Argument*, char*> >::iterator I = $5->begin(), E = $5->end();
-         I != E; ++I) {
-      if (I->second) free(I->second);   // Free the memory for the name...
-      delete I->first;                  // Free the unused function argument
+      InsertValue(ArgIt);
     }
-    delete $5;                          // Free the memory for the list itself
+
+    delete $4;                     // We're now done with the argument list
   }
-}
+};
 
 BEGIN : BEGINTOK | '{';                // Allow BEGIN or '{' to start a function
 
-FunctionHeader : FunctionHeaderH BEGIN {
-  $$ = CurMeth.CurrentFunction;
+FunctionHeader : OptLinkage FunctionHeaderH BEGIN {
+  $$ = CurFun.CurrentFunction;
 
-  // Resolve circular types before we parse the body of the method.
-  ResolveTypes(CurMeth.LateResolveTypes);
-}
+  // Make sure that we keep track of the linkage type even if there was a
+  // previous "declare".
+  $$->setLinkage($1);
+
+  // Resolve circular types before we parse the body of the function.
+  ResolveTypes(CurFun.LateResolveTypes);
+};
 
 END : ENDTOK | '}';                    // Allow end of '}' to end a function
 
 Function : BasicBlockList END {
   $$ = $1;
-}
+};
 
-FunctionProto : DECLARE { CurMeth.isDeclare = true; } FunctionHeaderH {
-  $$ = CurMeth.CurrentFunction;
+FunctionProto : DECLARE { CurFun.isDeclare = true; } FunctionHeaderH {
+  $$ = CurFun.CurrentFunction;
   assert($$->getParent() == 0 && "Function already in module!");
   CurModule.CurrentModule->getFunctionList().push_back($$);
-  CurMeth.FunctionDone();
-}
+  CurFun.FunctionDone();
+};
 
 //===----------------------------------------------------------------------===//
 //                        Rules to match Basic Blocks
@@ -1243,14 +1531,17 @@ ConstValueRef : ESINT64VAL {    // A reference to a direct constant
     $$ = ValID::create($1);
   }
   | TRUE {
-    $$ = ValID::create((int64_t)1);
+    $$ = ValID::create(ConstantBool::True);
   } 
   | FALSE {
-    $$ = ValID::create((int64_t)0);
+    $$ = ValID::create(ConstantBool::False);
   }
   | NULL_TOK {
     $$ = ValID::createNull();
   }
+  | ConstExpr {
+    $$ = ValID::create($1);
+  };
 
 // SymbolicValueRef - Reference to one of two ways of symbolically refering to
 // another value.
@@ -1258,12 +1549,12 @@ ConstValueRef : ESINT64VAL {    // A reference to a direct constant
 SymbolicValueRef : INTVAL {  // Is it an integer reference...?
     $$ = ValID::create($1);
   }
-  | VAR_ID {                 // Is it a named reference...?
+  | Name {                   // Is it a named reference...?
     $$ = ValID::create($1);
-  }
+  };
 
 // ValueRef - A reference to a definition... either constant or symbolic
-ValueRef : SymbolicValueRef | ConstValueRef
+ValueRef : SymbolicValueRef | ConstValueRef;
 
 
 // ResolvedVal - a <type> <value> pair.  This is used only in cases where the
@@ -1271,15 +1562,14 @@ ValueRef : SymbolicValueRef | ConstValueRef
 // pool references (for things like: 'ret [2 x int] [ int 12, int 42]')
 ResolvedVal : Types ValueRef {
     $$ = getVal(*$1, $2); delete $1;
-  }
-
+  };
 
 BasicBlockList : BasicBlockList BasicBlock {
-    ($$ = $1)->getBasicBlocks().push_back($2);
-  }
-  | FunctionHeader BasicBlock { // Do not allow methods with 0 basic blocks   
-    ($$ = $1)->getBasicBlocks().push_back($2);
+    ($$ = $1)->getBasicBlockList().push_back($2);
   }
+  | FunctionHeader BasicBlock { // Do not allow functions with 0 basic blocks   
+    ($$ = $1)->getBasicBlockList().push_back($2);
+  };
 
 
 // Basic blocks are terminated by branching instructions: 
@@ -1302,15 +1592,15 @@ BasicBlock : InstructionList OptAssign BBTerminatorInst  {
 
     InsertValue($2);
     $$ = $2;
-  }
+  };
 
 InstructionList : InstructionList Inst {
     $1->getInstList().push_back($2);
     $$ = $1;
   }
   | /* empty */ {
-    $$ = new BasicBlock();
-  }
+    $$ = CurBB = new BasicBlock();
+  };
 
 BBTerminatorInst : RET ResolvedVal {              // Return with a result...
     $$ = new ReturnInst($2);
@@ -1331,22 +1621,28 @@ BBTerminatorInst : RET ResolvedVal {              // Return with a result...
                                    cast<BasicBlock>(getVal(Type::LabelTy, $6)));
     $$ = S;
 
-    vector<pair<Constant*,BasicBlock*> >::iterator I = $8->begin(),
+    std::vector<std::pair<Constant*,BasicBlock*> >::iterator I = $8->begin(),
       E = $8->end();
     for (; I != E; ++I)
-      S->dest_push_back(I->first, I->second);
+      S->addCase(I->first, I->second);
+  }
+  | SWITCH IntType ValueRef ',' LABEL ValueRef '[' ']' {
+    SwitchInst *S = new SwitchInst(getVal($2, $3), 
+                                   cast<BasicBlock>(getVal(Type::LabelTy, $6)));
+    $$ = S;
   }
   | INVOKE TypesV ValueRef '(' ValueRefListE ')' TO ResolvedVal 
-    EXCEPT ResolvedVal {
-    const PointerType *PMTy;
+    UNWIND ResolvedVal {
+    const PointerType *PFTy;
     const FunctionType *Ty;
 
-    if (!(PMTy = dyn_cast<PointerType>($2->get())) ||
-        !(Ty = dyn_cast<FunctionType>(PMTy->getElementType()))) {
+    if (!(PFTy = dyn_cast<PointerType>($2->get())) ||
+        !(Ty = dyn_cast<FunctionType>(PFTy->getElementType()))) {
       // Pull out the types of all of the arguments...
-      vector<const Type*> ParamTypes;
+      std::vector<const Type*> ParamTypes;
       if ($5) {
-        for (vector<Value*>::iterator I = $5->begin(), E = $5->end(); I!=E; ++I)
+        for (std::vector<Value*>::iterator I = $5->begin(), E = $5->end();
+             I != E; ++I)
           ParamTypes.push_back((*I)->getType());
       }
 
@@ -1354,11 +1650,10 @@ BBTerminatorInst : RET ResolvedVal {              // Return with a result...
       if (isVarArg) ParamTypes.pop_back();
 
       Ty = FunctionType::get($2->get(), ParamTypes, isVarArg);
-      PMTy = PointerType::get(Ty);
+      PFTy = PointerType::get(Ty);
     }
-    delete $2;
 
-    Value *V = getVal(PMTy, $3);   // Get the method we're calling...
+    Value *V = getVal(PFTy, $3);   // Get the function we're calling...
 
     BasicBlock *Normal = dyn_cast<BasicBlock>($8);
     BasicBlock *Except = dyn_cast<BasicBlock>($10);
@@ -1368,14 +1663,14 @@ BBTerminatorInst : RET ResolvedVal {              // Return with a result...
 
     // Create the call node...
     if (!$5) {                                   // Has no arguments?
-      $$ = new InvokeInst(V, Normal, Except, vector<Value*>());
+      $$ = new InvokeInst(V, Normal, Except, std::vector<Value*>());
     } else {                                     // Has arguments?
       // Loop through FunctionType's arguments and ensure they are specified
       // correctly!
       //
-      FunctionType::ParamTypes::const_iterator I = Ty->getParamTypes().begin();
-      FunctionType::ParamTypes::const_iterator E = Ty->getParamTypes().end();
-      vector<Value*>::iterator ArgI = $5->begin(), ArgE = $5->end();
+      FunctionType::param_iterator I = Ty->param_begin();
+      FunctionType::param_iterator E = Ty->param_end();
+      std::vector<Value*>::iterator ArgI = $5->begin(), ArgE = $5->end();
 
       for (; ArgI != ArgE && I != E; ++ArgI, ++I)
        if ((*ArgI)->getType() != *I)
@@ -1387,8 +1682,12 @@ BBTerminatorInst : RET ResolvedVal {              // Return with a result...
 
       $$ = new InvokeInst(V, Normal, Except, *$5);
     }
+    delete $2;
     delete $5;
   }
+  | UNWIND {
+    $$ = new UnwindInst();
+  };
 
 
 
@@ -1398,73 +1697,135 @@ JumpTable : JumpTable IntType ConstValueRef ',' LABEL ValueRef {
     if (V == 0)
       ThrowException("May only switch on a constant pool value!");
 
-    $$->push_back(make_pair(V, cast<BasicBlock>(getVal($5, $6))));
+    $$->push_back(std::make_pair(V, cast<BasicBlock>(getVal($5, $6))));
   }
   | IntType ConstValueRef ',' LABEL ValueRef {
-    $$ = new vector<pair<Constant*, BasicBlock*> >();
+    $$ = new std::vector<std::pair<Constant*, BasicBlock*> >();
     Constant *V = cast<Constant>(getValNonImprovising($1, $2));
 
     if (V == 0)
       ThrowException("May only switch on a constant pool value!");
 
-    $$->push_back(make_pair(V, cast<BasicBlock>(getVal($4, $5))));
-  }
+    $$->push_back(std::make_pair(V, cast<BasicBlock>(getVal($4, $5))));
+  };
 
 Inst : OptAssign InstVal {
   // Is this definition named?? if so, assign the name...
   if (setValueName($2, $1)) { assert(0 && "No redefin allowed!"); }
   InsertValue($2);
   $$ = $2;
-}
+};
 
 PHIList : Types '[' ValueRef ',' ValueRef ']' {    // Used for PHI nodes
-    $$ = new list<pair<Value*, BasicBlock*> >();
-    $$->push_back(make_pair(getVal(*$1, $3), 
-                            cast<BasicBlock>(getVal(Type::LabelTy, $5))));
+    $$ = new std::list<std::pair<Value*, BasicBlock*> >();
+    $$->push_back(std::make_pair(getVal(*$1, $3), 
+                                 cast<BasicBlock>(getVal(Type::LabelTy, $5))));
     delete $1;
   }
   | PHIList ',' '[' ValueRef ',' ValueRef ']' {
     $$ = $1;
-    $1->push_back(make_pair(getVal($1->front().first->getType(), $4),
-                            cast<BasicBlock>(getVal(Type::LabelTy, $6))));
-  }
+    $1->push_back(std::make_pair(getVal($1->front().first->getType(), $4),
+                                 cast<BasicBlock>(getVal(Type::LabelTy, $6))));
+  };
 
 
 ValueRefList : ResolvedVal {    // Used for call statements, and memory insts...
-    $$ = new vector<Value*>();
+    $$ = new std::vector<Value*>();
     $$->push_back($1);
   }
   | ValueRefList ',' ResolvedVal {
     $$ = $1;
     $1->push_back($3);
-  }
+  };
 
 // ValueRefListE - Just like ValueRefList, except that it may also be empty!
-ValueRefListE : ValueRefList | /*empty*/ { $$ = 0; }
+ValueRefListE : ValueRefList | /*empty*/ { $$ = 0; };
 
-InstVal : BinaryOps Types ValueRef ',' ValueRef {
+InstVal : ArithmeticOps Types ValueRef ',' ValueRef {
+    if (!(*$2)->isInteger() && !(*$2)->isFloatingPoint())
+      ThrowException("Arithmetic operator requires integer or FP operands!");
     $$ = BinaryOperator::create($1, getVal(*$2, $3), getVal(*$2, $5));
     if ($$ == 0)
       ThrowException("binary operator returned null!");
     delete $2;
   }
-  | UnaryOps ResolvedVal {
-    $$ = UnaryOperator::create($1, $2);
+  | LogicalOps Types ValueRef ',' ValueRef {
+    if (!(*$2)->isIntegral())
+      ThrowException("Logical operator requires integral operands!");
+    $$ = BinaryOperator::create($1, getVal(*$2, $3), getVal(*$2, $5));
     if ($$ == 0)
-      ThrowException("unary operator returned null!");
+      ThrowException("binary operator returned null!");
+    delete $2;
+  }
+  | SetCondOps Types ValueRef ',' ValueRef {
+    $$ = new SetCondInst($1, getVal(*$2, $3), getVal(*$2, $5));
+    if ($$ == 0)
+      ThrowException("binary operator returned null!");
+    delete $2;
+  }
+  | NOT ResolvedVal {
+    std::cerr << "WARNING: Use of eliminated 'not' instruction:"
+              << " Replacing with 'xor'.\n";
+
+    Value *Ones = ConstantIntegral::getAllOnesValue($2->getType());
+    if (Ones == 0)
+      ThrowException("Expected integral type for not instruction!");
+
+    $$ = BinaryOperator::create(Instruction::Xor, $2, Ones);
+    if ($$ == 0)
+      ThrowException("Could not create a xor instruction!");
   }
   | ShiftOps ResolvedVal ',' ResolvedVal {
     if ($4->getType() != Type::UByteTy)
       ThrowException("Shift amount must be ubyte!");
+    if (!$2->getType()->isInteger())
+      ThrowException("Shift constant expression requires integer operand!");
     $$ = new ShiftInst($1, $2, $4);
   }
   | CAST ResolvedVal TO Types {
+    if (!$4->get()->isFirstClassType())
+      ThrowException("cast instruction to a non-primitive type: '" +
+                     $4->get()->getDescription() + "'!");
     $$ = new CastInst($2, *$4);
     delete $4;
   }
-  | PHI PHIList {
+  | VA_ARG ResolvedVal ',' Types {
+    // FIXME: This is emulation code for an obsolete syntax.  This should be
+    // removed at some point.
+    if (!ObsoleteVarArgs) {
+      std::cerr << "WARNING: this file uses obsolete features.  "
+                << "Assemble and disassemble to update it.\n";
+      ObsoleteVarArgs = true;
+    }
+
+    // First, load the valist...
+    Instruction *CurVAList = new LoadInst($2, "");
+    CurBB->getInstList().push_back(CurVAList);
+
+    // Emit the vaarg instruction.
+    $$ = new VAArgInst(CurVAList, *$4);
+    
+    // Now we must advance the pointer and update it in memory.
+    Instruction *TheVANext = new VANextInst(CurVAList, *$4);
+    CurBB->getInstList().push_back(TheVANext);
+
+    CurBB->getInstList().push_back(new StoreInst(TheVANext, $2));
+    delete $4;
+  }
+  | VAARG ResolvedVal ',' Types {
+    $$ = new VAArgInst($2, *$4);
+    delete $4;
+  }
+  | VANEXT ResolvedVal ',' Types {
+    $$ = new VANextInst($2, *$4);
+    delete $4;
+  }
+  | PHI_TOK PHIList {
     const Type *Ty = $2->front().first->getType();
+    if (!Ty->isFirstClassType())
+      ThrowException("PHI node operands must be of first class type!");
     $$ = new PHINode(Ty);
+    $$->op_reserve($2->size()*2);
     while ($2->begin() != $2->end()) {
       if ($2->front().first->getType() != Ty) 
        ThrowException("All elements of a PHI node must be of the same type!");
@@ -1474,15 +1835,16 @@ InstVal : BinaryOps Types ValueRef ',' ValueRef {
     delete $2;  // Free the list...
   } 
   | CALL TypesV ValueRef '(' ValueRefListE ')' {
-    const PointerType *PMTy;
+    const PointerType *PFTy;
     const FunctionType *Ty;
 
-    if (!(PMTy = dyn_cast<PointerType>($2->get())) ||
-        !(Ty = dyn_cast<FunctionType>(PMTy->getElementType()))) {
+    if (!(PFTy = dyn_cast<PointerType>($2->get())) ||
+        !(Ty = dyn_cast<FunctionType>(PFTy->getElementType()))) {
       // Pull out the types of all of the arguments...
-      vector<const Type*> ParamTypes;
+      std::vector<const Type*> ParamTypes;
       if ($5) {
-        for (vector<Value*>::iterator I = $5->begin(), E = $5->end(); I!=E; ++I)
+        for (std::vector<Value*>::iterator I = $5->begin(), E = $5->end();
+             I != E; ++I)
           ParamTypes.push_back((*I)->getType());
       }
 
@@ -1490,22 +1852,26 @@ InstVal : BinaryOps Types ValueRef ',' ValueRef {
       if (isVarArg) ParamTypes.pop_back();
 
       Ty = FunctionType::get($2->get(), ParamTypes, isVarArg);
-      PMTy = PointerType::get(Ty);
+      PFTy = PointerType::get(Ty);
     }
-    delete $2;
 
-    Value *V = getVal(PMTy, $3);   // Get the method we're calling...
+    Value *V = getVal(PFTy, $3);   // Get the function we're calling...
 
     // Create the call node...
     if (!$5) {                                   // Has no arguments?
-      $$ = new CallInst(V, vector<Value*>());
+      // Make sure no arguments is a good thing!
+      if (Ty->getNumParams() != 0)
+        ThrowException("No arguments passed to a function that "
+                       "expects arguments!");
+
+      $$ = new CallInst(V, std::vector<Value*>());
     } else {                                     // Has arguments?
       // Loop through FunctionType's arguments and ensure they are specified
       // correctly!
       //
-      FunctionType::ParamTypes::const_iterator I = Ty->getParamTypes().begin();
-      FunctionType::ParamTypes::const_iterator E = Ty->getParamTypes().end();
-      vector<Value*>::iterator ArgI = $5->begin(), ArgE = $5->end();
+      FunctionType::param_iterator I = Ty->param_begin();
+      FunctionType::param_iterator E = Ty->param_end();
+      std::vector<Value*>::iterator ArgI = $5->begin(), ArgE = $5->end();
 
       for (; ArgI != ArgE && I != E; ++ArgI, ++I)
        if ((*ArgI)->getType() != *I)
@@ -1517,37 +1883,43 @@ InstVal : BinaryOps Types ValueRef ',' ValueRef {
 
       $$ = new CallInst(V, *$5);
     }
+    delete $2;
     delete $5;
   }
   | MemoryInst {
     $$ = $1;
-  }
+  };
 
 
 // IndexList - List of indices for GEP based instructions...
 IndexList : ',' ValueRefList { 
-  $$ = $2; 
-} | /* empty */ { 
-  $$ = new vector<Value*>(); 
-}
+    $$ = $2; 
+  } | /* empty */ { 
+    $$ = new std::vector<Value*>(); 
+  };
+
+OptVolatile : VOLATILE {
+    $$ = true;
+  }
+  | /* empty */ {
+    $$ = false;
+  };
+
 
 MemoryInst : MALLOC Types {
-    $$ = new MallocInst(PointerType::get(*$2));
+    $$ = new MallocInst(*$2);
     delete $2;
   }
   | MALLOC Types ',' UINT ValueRef {
-    const Type *Ty = PointerType::get(*$2);
-    $$ = new MallocInst(Ty, getVal($4, $5));
+    $$ = new MallocInst(*$2, getVal($4, $5));
     delete $2;
   }
   | ALLOCA Types {
-    $$ = new AllocaInst(PointerType::get(*$2));
+    $$ = new AllocaInst(*$2);
     delete $2;
   }
   | ALLOCA Types ',' UINT ValueRef {
-    const Type *Ty = PointerType::get(*$2);
-    Value *ArrSize = getVal($4, $5);
-    $$ = new AllocaInst(Ty, ArrSize);
+    $$ = new AllocaInst(*$2, getVal($4, $5));
     delete $2;
   }
   | FREE ResolvedVal {
@@ -1557,41 +1929,46 @@ MemoryInst : MALLOC Types {
     $$ = new FreeInst($2);
   }
 
-  | LOAD Types ValueRef IndexList {
-    if (!isa<PointerType>(*$2))
+  | OptVolatile LOAD Types ValueRef {
+    if (!isa<PointerType>($3->get()))
       ThrowException("Can't load from nonpointer type: " +
-                    (*$2)->getDescription());
-    if (LoadInst::getIndexedType(*$2, *$4) == 0)
-      ThrowException("Invalid indices for load instruction!");
-
-    $$ = new LoadInst(getVal(*$2, $3), *$4);
-    delete $4;   // Free the vector...
-    delete $2;
+                    (*$3)->getDescription());
+    $$ = new LoadInst(getVal(*$3, $4), "", $1);
+    delete $3;
   }
-  | STORE ResolvedVal ',' Types ValueRef IndexList {
-    if (!isa<PointerType>(*$4))
+  | OptVolatile STORE ResolvedVal ',' Types ValueRef {
+    const PointerType *PT = dyn_cast<PointerType>($5->get());
+    if (!PT)
       ThrowException("Can't store to a nonpointer type: " +
-                     (*$4)->getDescription());
-    const Type *ElTy = StoreInst::getIndexedType(*$4, *$6);
-    if (ElTy == 0)
-      ThrowException("Can't store into that field list!");
-    if (ElTy != $2->getType())
-      ThrowException("Can't store '" + $2->getType()->getDescription() +
+                     (*$5)->getDescription());
+    const Type *ElTy = PT->getElementType();
+    if (ElTy != $3->getType())
+      ThrowException("Can't store '" + $3->getType()->getDescription() +
                      "' into space of type '" + ElTy->getDescription() + "'!");
-    $$ = new StoreInst($2, getVal(*$4, $5), *$6);
-    delete $4; delete $6;
+
+    $$ = new StoreInst($3, getVal(*$5, $6), $1);
+    delete $5;
   }
   | GETELEMENTPTR Types ValueRef IndexList {
-    if (!isa<PointerType>(*$2))
+    if (!isa<PointerType>($2->get()))
       ThrowException("getelementptr insn requires pointer operand!");
     if (!GetElementPtrInst::getIndexedType(*$2, *$4, true))
       ThrowException("Can't get element ptr '" + (*$2)->getDescription()+ "'!");
     $$ = new GetElementPtrInst(getVal(*$2, $3), *$4);
     delete $2; delete $4;
-  }
+  };
+
 
 %%
 int yyerror(const char *ErrorMsg) {
-  ThrowException(string("Parse error: ") + ErrorMsg);
+  std::string where 
+    = std::string((CurFilename == "-") ? std::string("<stdin>") : CurFilename)
+                  + ":" + utostr((unsigned) llvmAsmlineno) + ": ";
+  std::string errMsg = std::string(ErrorMsg) + "\n" + where + " while reading ";
+  if (yychar == YYEMPTY)
+    errMsg += "end-of-file.";
+  else
+    errMsg += "token: '" + std::string(llvmAsmtext, llvmAsmleng) + "'";
+  ThrowException(errMsg);
   return 0;
 }