Convert to SymbolTable's new iteration interface.
[oota-llvm.git] / lib / VMCore / SymbolTable.cpp
index 80f8e100e839d5e84cbfaef729830bf15faba849..dc2900ef9ab04a1e1f3857e9b4c9917dfcceb379 100644 (file)
@@ -1,50 +1,46 @@
-//===-- SymbolTable.cpp - Implement the SymbolTable class -------------------=//
+//===-- SymbolTable.cpp - Implement the SymbolTable class -----------------===//
+// 
+//                     The LLVM Compiler Infrastructure
+//
+// This file was developed by the LLVM research group and revised by Reid
+// Spencer. It is distributed under the University of Illinois Open Source 
+// License. See LICENSE.TXT for details.
+// 
+//===----------------------------------------------------------------------===//
 //
 // This file implements the SymbolTable class for the VMCore library.
 //
 //===----------------------------------------------------------------------===//
 
 #include "llvm/SymbolTable.h"
-#include "llvm/InstrTypes.h"
 #include "llvm/DerivedTypes.h"
 #include "llvm/Module.h"
-#include "llvm/Method.h"
 #include "Support/StringExtras.h"
-#include <iostream>
+#include <algorithm>
 
-using std::string;
-using std::pair;
-using std::make_pair;
-using std::map;
-using std::cerr;
-using std::cout;
+using namespace llvm;
 
 #define DEBUG_SYMBOL_TABLE 0
 #define DEBUG_ABSTYPE 0
 
 SymbolTable::~SymbolTable() {
   // Drop all abstract type references in the type plane...
-  iterator TyPlane = find(Type::TypeTy);
-  if (TyPlane != end()) {
-    VarMap &TyP = TyPlane->second;
-    for (VarMap::iterator I = TyP.begin(), E = TyP.end(); I != E; ++I) {
-      const Type *Ty = cast<const Type>(I->second);
-      if (Ty->isAbstract())   // If abstract, drop the reference...
-       cast<DerivedType>(Ty)->removeAbstractTypeUser(this);
-    }
+  for (type_iterator TI = tmap.begin(), TE = tmap.end(); TI != TE; ++TI) {
+    if (TI->second->isAbstract())   // If abstract, drop the reference...
+      cast<DerivedType>(TI->second)->removeAbstractTypeUser(this);
   }
 
- // TODO: FIXME: BIG ONE: This doesn't unreference abstract types for the planes
- // that could still have entries!
+ // TODO: FIXME: BIG ONE: This doesn't unreference abstract types for the 
+ // planes that could still have entries!
 
 #ifndef NDEBUG   // Only do this in -g mode...
   bool LeftoverValues = true;
-  for (iterator i = begin(); i != end(); ++i) {
-    for (type_iterator I = i->second.begin(); I != i->second.end(); ++I)
-      if (!isa<Constant>(I->second) && !isa<Type>(I->second)) {
-       cerr << "Value still in symbol table! Type = '"
-             << i->first->getDescription() << "' Name = '"
-             << I->first << "'\n";
+  for (plane_iterator PI = pmap.begin(); PI != pmap.end(); ++PI) {
+    for (value_iterator VI = PI->second.begin(); VI != PI->second.end(); ++VI)
+      if (!isa<Constant>(VI->second) ) {
+       std::cerr << "Value still in symbol table! Type = '"
+                  << PI->first->getDescription() << "' Name = '"
+                  << VI->first << "'\n";
        LeftoverValues = false;
       }
   }
@@ -57,61 +53,66 @@ SymbolTable::~SymbolTable() {
 // it (or derived from it) that does not already occur in the symbol table for
 // the specified type.
 //
-string SymbolTable::getUniqueName(const Type *Ty, const string &BaseName) {
-  iterator I = find(Ty);
-  if (I == end()) return BaseName;
-
-  string TryName = BaseName;
-  unsigned Counter = 0;
-  type_iterator End = I->second.end();
-
-  while (I->second.find(TryName) != End)     // Loop until we find unoccupied
-    TryName = BaseName + utostr(++Counter);  // Name in the symbol table
+std::string SymbolTable::getUniqueName(const Type *Ty,
+                                       const std::string &BaseName) const {
+  // Find the plane
+  plane_const_iterator PI = pmap.find(Ty);
+  if (PI == pmap.end()) return BaseName;
+
+  std::string TryName = BaseName;
+  const ValueMap& vmap = PI->second;
+  value_const_iterator End = vmap.end();
+
+  // See if the name exists
+  while (vmap.find(TryName) != End)            // Loop until we find a free
+    TryName = BaseName + utostr(++LastUnique); // name in the symbol table
   return TryName;
 }
 
 
-
-// lookup - Returns null on failure...
-Value *SymbolTable::localLookup(const Type *Ty, const string &Name) {
-  iterator I = find(Ty);
-  if (I != end()) {                      // We have symbols in that plane...
-    type_iterator J = I->second.find(Name);
-    if (J != I->second.end())            // and the name is in our hash table...
-      return J->second;
+// lookup a value - Returns null on failure...
+Value *SymbolTable::lookup(const Type *Ty, const std::string &Name) const {
+  plane_const_iterator PI = pmap.find(Ty);
+  if (PI != pmap.end()) {                  // We have symbols in that plane...
+    value_const_iterator VI = PI->second.find(Name);
+    if (VI != PI->second.end())            // and the name is in our hash table...
+      return VI->second;
   }
-
   return 0;
 }
 
-// lookup - Returns null on failure...
-Value *SymbolTable::lookup(const Type *Ty, const string &Name) {
-  Value *LV = localLookup(Ty, Name);
-  if (LV) return LV;
-  return ParentSymTab ? ParentSymTab->lookup(Ty, Name) : 0;
+
+// lookup a type by name - returns null on failure
+Type* SymbolTable::lookupType( const std::string& Name ) const {
+  type_const_iterator TI = tmap.find( Name );
+  if ( TI != tmap.end() )
+    return TI->second;
+  return 0;
 }
 
+// Remove a value
 void SymbolTable::remove(Value *N) {
   assert(N->hasName() && "Value doesn't have name!");
   if (InternallyInconsistent) return;
 
-  iterator I = find(N->getType());
-  assert(I != end() &&
-         "Trying to remove a type that doesn't have a plane yet!");
-  removeEntry(I, I->second.find(N->getName()));
+  plane_iterator PI = pmap.find(N->getType());
+  assert(PI != pmap.end() &&
+         "Trying to remove a value that doesn't have a type plane yet!");
+  removeEntry(PI, PI->second.find(N->getName()));
 }
 
+
 // removeEntry - Remove a value from the symbol table...
-//
-Value *SymbolTable::removeEntry(iterator Plane, type_iterator Entry) {
+Value *SymbolTable::removeEntry(plane_iterator Plane, value_iterator Entry) {
   if (InternallyInconsistent) return 0;
-  assert(Plane != super::end() &&
+  assert(Plane != pmap.end() &&
          Entry != Plane->second.end() && "Invalid entry to remove!");
 
   Value *Result = Entry->second;
   const Type *Ty = Result->getType();
 #if DEBUG_SYMBOL_TABLE
-  cerr << this << " Removing Value: " << Result->getName() << endl;
+  dump();
+  std::cerr << " Removing Value: " << Result->getName() << "\n";
 #endif
 
   // Remove the value from the plane...
@@ -124,39 +125,66 @@ Value *SymbolTable::removeEntry(iterator Plane, type_iterator Entry) {
     //
     if (Plane->first->isAbstract()) {
 #if DEBUG_ABSTYPE
-      cerr << "Plane Empty: Removing type: " << Plane->first->getDescription()
-           << endl;
+      std::cerr << "Plane Empty: Removing type: "
+                << Plane->first->getDescription() << "\n";
 #endif
       cast<DerivedType>(Plane->first)->removeAbstractTypeUser(this);
     }
 
-    erase(Plane);
+    pmap.erase(Plane);
   }
+  return Result;
+}
+
+
+// remove - Remove a type
+void SymbolTable::remove(Type* Ty ) {
+  type_iterator TI = this->type_begin();
+  type_iterator TE = this->type_end();
+
+  // Search for the entry
+  while ( TI != TE && TI->second != Ty )
+    ++TI;
+
+  if ( TI != TE )
+    this->removeEntry( TI );
+}
+
+
+// removeEntry - Remove a type from the symbol table...
+Type* SymbolTable::removeEntry(type_iterator Entry) {
+  if (InternallyInconsistent) return 0;
+  assert( Entry != tmap.end() && "Invalid entry to remove!");
+
+  Type* Result = Entry->second;
+
+#if DEBUG_SYMBOL_TABLE
+  dump();
+  std::cerr << " Removing Value: " << Result->getName() << "\n";
+#endif
+
+  tmap.erase(Entry);
 
   // If we are removing an abstract type, remove the symbol table from it's use
   // list...
-  if (Ty == Type::TypeTy) {
-    const Type *T = cast<const Type>(Result);
-    if (T->isAbstract()) {
+  if (Result->isAbstract()) {
 #if DEBUG_ABSTYPE
-      cerr << "Removing abs type from symtab" << T->getDescription() << endl;
+    std::cerr << "Removing abstract type from symtab" << Result->getDescription()<<"\n";
 #endif
-      cast<DerivedType>(T)->removeAbstractTypeUser(this);
-    }
+    cast<DerivedType>(Result)->removeAbstractTypeUser(this);
   }
 
   return Result;
 }
 
-// insertEntry - Insert a value into the symbol table with the specified
-// name...
-//
-void SymbolTable::insertEntry(const string &Name, const Type *VTy, Value *V) {
 
+// insertEntry - Insert a value into the symbol table with the specified name.
+void SymbolTable::insertEntry(const std::string &Name, const Type *VTy,
+                              Value *V) {
   // Check to see if there is a naming conflict.  If so, rename this value!
-  if (localLookup(VTy, Name)) {
-    string UniqueName = getUniqueName(VTy, Name);
-    assert(InternallyInconsistent == false && "Infinite loop inserting entry!");
+  if (lookup(VTy, Name)) {
+    std::string UniqueName = getUniqueName(VTy, Name);
+    assert(InternallyInconsistent == false && "Infinite loop inserting value!");
     InternallyInconsistent = true;
     V->setName(UniqueName, this);
     InternallyInconsistent = false;
@@ -164,15 +192,16 @@ void SymbolTable::insertEntry(const string &Name, const Type *VTy, Value *V) {
   }
 
 #if DEBUG_SYMBOL_TABLE
-  cerr << this << " Inserting definition: " << Name << ": " 
-       << VTy->getDescription() << endl;
+  dump();
+  std::cerr << " Inserting definition: " << Name << ": " 
+            << VTy->getDescription() << "\n";
 #endif
 
-  iterator I = find(VTy);
-  if (I == end()) {      // Not in collection yet... insert dummy entry
+  plane_iterator PI = pmap.find(VTy);
+  if (PI == pmap.end()) {      // Not in collection yet... insert dummy entry
     // Insert a new empty element.  I points to the new elements.
-    I = super::insert(make_pair(VTy, VarMap())).first;
-    assert(I != end() && "How did insert fail?");
+    PI = pmap.insert(make_pair(VTy, ValueMap())).first;
+    assert(PI != pmap.end() && "How did insert fail?");
 
     // Check to see if the type is abstract.  If so, it might be refined in the
     // future, which would cause the plane of the old type to get merged into
@@ -181,71 +210,171 @@ void SymbolTable::insertEntry(const string &Name, const Type *VTy, Value *V) {
     if (VTy->isAbstract()) {
       cast<DerivedType>(VTy)->addAbstractTypeUser(this);
 #if DEBUG_ABSTYPE
-      cerr << "Added abstract type value: " << VTy->getDescription() << endl;
+      std::cerr << "Added abstract type value: " << VTy->getDescription()
+                << "\n";
 #endif
     }
   }
 
-  I->second.insert(make_pair(Name, V));
+  PI->second.insert(make_pair(Name, V));
+}
+
+
+// insertEntry - Insert a value into the symbol table with the specified
+// name...
+//
+void SymbolTable::insertEntry(const std::string& Name, Type* T) {
+
+  // Check to see if there is a naming conflict.  If so, rename this type!
+  std::string UniqueName = Name;
+  if (lookupType(Name))
+    UniqueName = getUniqueName(T, Name);
+
+#if DEBUG_SYMBOL_TABLE
+  dump();
+  std::cerr << " Inserting type: " << UniqueName << ": " 
+            << T->getDescription() << "\n";
+#endif
+
+  // Insert the tmap entry
+  tmap.insert(make_pair(UniqueName, T));
 
   // If we are adding an abstract type, add the symbol table to it's use list.
-  if (VTy == Type::TypeTy) {
-    const Type *T = cast<const Type>(V);
-    if (T->isAbstract()) {
-      cast<DerivedType>(T)->addAbstractTypeUser(this);
+  if (T->isAbstract()) {
+    cast<DerivedType>(T)->addAbstractTypeUser(this);
 #if DEBUG_ABSTYPE
-      cerr << "Added abstract type to ST: " << T->getDescription() << endl;
+    std::cerr << "Added abstract type to ST: " << T->getDescription() << "\n";
 #endif
+  }
+}
+
+
+// Determine how many entries for a given type.
+unsigned SymbolTable::type_size(const Type *Ty) const {
+  plane_const_iterator PI = pmap.find(Ty);
+  if ( PI == pmap.end() ) return 0;
+  return PI->second.size();
+}
+
+
+// Get the name of a value
+std::string SymbolTable::get_name( const Value* V ) const {
+  value_const_iterator VI = this->value_begin( V->getType() );
+  value_const_iterator VE = this->value_end( V->getType() );
+
+  // Search for the entry
+  while ( VI != VE && VI->second != V )
+    ++VI;
+
+  if ( VI != VE )
+    return VI->first;
+
+  return "";
+}
+
+
+// Get the name of a type
+std::string SymbolTable::get_name( const Type* T ) const {
+  if (tmap.empty()) return ""; // No types at all.
+
+  type_const_iterator TI = tmap.begin();
+  type_const_iterator TE = tmap.end();
+
+  // Search for the entry
+  while (TI != TE && TI->second != T )
+    ++TI;
+
+  if (TI != TE)  // Must have found an entry!
+    return TI->first;
+  return "";     // Must not have found anything...
+}
+
+
+// Strip the symbol table of its names.
+bool SymbolTable::strip( void ) {
+  bool RemovedSymbol = false;
+  for (plane_iterator I = pmap.begin(); I != pmap.end();) {
+    // Removing items from the plane can cause the plane itself to get deleted.
+    // If this happens, make sure we incremented our plane iterator already!
+    ValueMap &Plane = (I++)->second;
+    value_iterator B = Plane.begin(), Bend = Plane.end();
+    while (B != Bend) {   // Found nonempty type plane!
+      Value *V = B->second;
+      if (isa<Constant>(V)) {
+       remove(V);
+        RemovedSymbol = true;
+      } else {
+        if (!isa<GlobalValue>(V) || cast<GlobalValue>(V)->hasInternalLinkage()){
+          // Set name to "", removing from symbol table!
+          V->setName("", this);
+          RemovedSymbol = true;
+        }
+      }
+      ++B;
     }
   }
+
+  for (type_iterator TI = tmap.begin(); TI != tmap.end(); ) {
+    Type* T = (TI++)->second;
+    remove(T);
+    RemovedSymbol = true;
+  }
+  return RemovedSymbol;
 }
 
+
 // This function is called when one of the types in the type plane are refined
 void SymbolTable::refineAbstractType(const DerivedType *OldType,
                                     const Type *NewType) {
-  if (OldType == NewType && OldType->isAbstract())
-    return;  // Noop, don't waste time dinking around
 
-  // Search to see if we have any values of the type oldtype.  If so, we need to
+  // Search to see if we have any values of the type Oldtype.  If so, we need to
   // move them into the newtype plane...
-  iterator TPI = find(OldType);
-  if (OldType != NewType && TPI != end()) {
+  plane_iterator PI = pmap.find(OldType);
+  if (PI != pmap.end()) {
     // Get a handle to the new type plane...
-    iterator NewTypeIt = find(NewType);
-    if (NewTypeIt == super::end()) {      // If no plane exists, add one
-      NewTypeIt = super::insert(make_pair(NewType, VarMap())).first;
+    plane_iterator NewTypeIt = pmap.find(NewType);
+    if (NewTypeIt == pmap.end()) {      // If no plane exists, add one
+      NewTypeIt = pmap.insert(make_pair(NewType, ValueMap())).first;
       
       if (NewType->isAbstract()) {
         cast<DerivedType>(NewType)->addAbstractTypeUser(this);
 #if DEBUG_ABSTYPE
-        cerr << "[Added] refined to abstype: "<<NewType->getDescription()<<endl;
+        std::cerr << "[Added] refined to abstype: " << NewType->getDescription()
+                  << "\n";
 #endif
       }
     }
 
-    VarMap &NewPlane = NewTypeIt->second;
-    VarMap &OldPlane = TPI->second;
+    ValueMap &NewPlane = NewTypeIt->second;
+    ValueMap &OldPlane = PI->second;
     while (!OldPlane.empty()) {
-      pair<const string, Value*> V = *OldPlane.begin();
+      std::pair<const std::string, Value*> V = *OldPlane.begin();
 
       // Check to see if there is already a value in the symbol table that this
       // would collide with.
-      type_iterator TI = NewPlane.find(V.first);
-      if (TI != NewPlane.end() && TI->second == V.second) {
+      value_iterator VI = NewPlane.find(V.first);
+      if (VI != NewPlane.end() && VI->second == V.second) {
         // No action
 
-      } else if (TI != NewPlane.end()) {
-        // The only thing we are allowing for now is two method prototypes being
+      } else if (VI != NewPlane.end()) {
+        // The only thing we are allowing for now is two external global values
         // folded into one.
         //
-        Method *ExistM = dyn_cast<Method>(TI->second);
-        Method *NewM = dyn_cast<Method>(V.second);
+        GlobalValue *ExistGV = dyn_cast<GlobalValue>(VI->second);
+        GlobalValue *NewGV = dyn_cast<GlobalValue>(V.second);
 
-        if (ExistM && NewM && ExistM->isExternal() && NewM->isExternal()) {
-          // Ok we have two external methods.  Make all uses of the new one
-          // use the old one...
-          //
-          NewM->replaceAllUsesWith(ExistM);
+        if (ExistGV && NewGV) {
+          assert((ExistGV->isExternal() || NewGV->isExternal()) &&
+                 "Two planes folded together with overlapping value names!");
+
+          // Make sure that ExistGV is the one we want to keep!
+          if (!NewGV->isExternal())
+            std::swap(NewGV, ExistGV);
+
+          // Ok we have two external global values.  Make all uses of the new
+          // one use the old one...
+          NewGV->uncheckedReplaceAllUsesWith(ExistGV);
           
           // Now we just convert it to an unnamed method... which won't get
           // added to our symbol table.  The problem is that if we call
@@ -260,16 +389,23 @@ void SymbolTable::refineAbstractType(const DerivedType *OldType,
           InternallyInconsistent = true;
 
           // Remove newM from the symtab
-          NewM->setName("");
+          NewGV->setName("");
           InternallyInconsistent = false;
 
-          // Now we can remove this method from the module entirely...
-          NewM->getParent()->getMethodList().remove(NewM);
-          delete NewM;
-
+          // Now we can remove this global from the module entirely...
+          Module *M = NewGV->getParent();
+          if (Function *F = dyn_cast<Function>(NewGV))
+            M->getFunctionList().remove(F);
+          else
+            M->getGlobalList().remove(cast<GlobalVariable>(NewGV));
+          delete NewGV;
         } else {
-          assert(0 && "Two ploanes folded together with overlapping "
-                 "value names!");
+          // If they are not global values, they must be just random values who
+          // happen to conflict now that types have been resolved.  If this is
+          // the case, reinsert the value into the new plane, allowing it to get
+          // renamed.
+          assert(V.second->getType() == NewType &&"Type resolution is broken!");
+          insert(V.second);
         }
       } else {
         insertEntry(V.first, NewType, V.second);
@@ -282,68 +418,79 @@ void SymbolTable::refineAbstractType(const DerivedType *OldType,
     // Ok, now we are not referencing the type anymore... take me off your user
     // list please!
 #if DEBUG_ABSTYPE
-    cerr << "Removing type " << OldType->getDescription() << endl;
+    std::cerr << "Removing type " << OldType->getDescription() << "\n";
 #endif
     OldType->removeAbstractTypeUser(this);
 
     // Remove the plane that is no longer used
-    erase(TPI);
-  } else if (TPI != end()) {
-    assert(OldType == NewType);
-#if DEBUG_ABSTYPE
-    cerr << "Removing SELF type " << OldType->getDescription() << endl;
-#endif
-    OldType->removeAbstractTypeUser(this);
+    pmap.erase(PI);
   }
 
-  TPI = find(Type::TypeTy);
-  if (TPI != end()) {  
-    // Loop over all of the types in the symbol table, replacing any references to
-    // OldType with references to NewType.  Note that there may be multiple
-    // occurances, and although we only need to remove one at a time, it's faster
-    // to remove them all in one pass.
-    //
-    VarMap &TyPlane = TPI->second;
-    for (VarMap::iterator I = TyPlane.begin(), E = TyPlane.end(); I != E; ++I)
-      if (I->second == (Value*)OldType) {  // FIXME when Types aren't const.
+  // Loop over all of the types in the symbol table, replacing any references
+  // to OldType with references to NewType.  Note that there may be multiple
+  // occurrences, and although we only need to remove one at a time, it's
+  // faster to remove them all in one pass.
+  //
+  for (type_iterator I = type_begin(), E = type_end(); I != E; ++I) {
+    if (I->second == (Type*)OldType) {  // FIXME when Types aren't const.
 #if DEBUG_ABSTYPE
-        cerr << "Removing type " << OldType->getDescription() << endl;
+      std::cerr << "Removing type " << OldType->getDescription() << "\n";
 #endif
-        OldType->removeAbstractTypeUser(this);
+      OldType->removeAbstractTypeUser(this);
         
-        I->second = (Value*)NewType;  // TODO FIXME when types aren't const
-        if (NewType->isAbstract()) {
+      I->second = (Type*)NewType;  // TODO FIXME when types aren't const
+      if (NewType->isAbstract()) {
 #if DEBUG_ABSTYPE
-          cerr << "Added type " << NewType->getDescription() << endl;
+       std::cerr << "Added type " << NewType->getDescription() << "\n";
 #endif
-          cast<const DerivedType>(NewType)->addAbstractTypeUser(this);
-        }
+       cast<DerivedType>(NewType)->addAbstractTypeUser(this);
       }
+    }
   }
 }
 
 
-#ifndef NDEBUG
-#include "llvm/Assembly/Writer.h"
-#include <algorithm>
+// Handle situation where type becomes Concreate from Abstract
+void SymbolTable::typeBecameConcrete(const DerivedType *AbsTy) {
+  plane_iterator PI = pmap.find(AbsTy);
+
+  // If there are any values in the symbol table of this type, then the type
+  // plane is a use of the abstract type which must be dropped.
+  if (PI != pmap.end())
+    AbsTy->removeAbstractTypeUser(this);
+
+  // Loop over all of the types in the symbol table, dropping any abstract
+  // type user entries for AbsTy which occur because there are names for the
+  // type.
+  for (type_iterator TI = type_begin(), TE = type_end(); TI != TE; ++TI)
+    if (TI->second == (Type*)AbsTy)   // FIXME when Types aren't const.
+      AbsTy->removeAbstractTypeUser(this);
+}
 
-static void DumpVal(const pair<const string, Value *> &V) {
-  cout << "  '" << V.first << "' = " << V.second << "\n";
+static void DumpVal(const std::pair<const std::string, Value *> &V) {
+  std::cerr << "  '" << V.first << "' = ";
+  V.second->dump();
+  std::cerr << "\n";
 }
 
-static void DumpPlane(const pair<const Type *, map<const string, Value *> >&P) {
-  cout << "  Plane: " << P.first << "\n";
+static void DumpPlane(const std::pair<const Type *,
+                                      std::map<const std::string, Value *> >&P){
+  P.first->dump();
+  std::cerr << "\n";
   for_each(P.second.begin(), P.second.end(), DumpVal);
 }
 
-void SymbolTable::dump() const {
-  cout << "Symbol table dump:\n";
-  for_each(begin(), end(), DumpPlane);
+static void DumpTypes(const std::pair<const std::string, Type*>& T ) {
+  std::cerr << "  '" << T.first << "' = ";
+  T.second->dump();
+  std::cerr << "\n";
+}
 
-  if (ParentSymTab) {
-    cout << "Parent ";
-    ParentSymTab->dump();
-  }
+void SymbolTable::dump() const {
+  std::cerr << "Symbol table dump:\n  Plane:";
+  for_each(pmap.begin(), pmap.end(), DumpPlane);
+  std::cerr << "  Types: ";
+  for_each(tmap.begin(), tmap.end(), DumpTypes);
 }
 
-#endif
+// vim: sw=2 ai