Rename ConstPoolVal -> Constant

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

diff --git a/lib/VMCore/Value.cpp b/lib/VMCore/Value.cpp
index 48f9347f962e6394ef8c0593973066cbd4e435ba..3429d7c3b38a4a0c30a5338e851eea3b32d1d125 100644 (file)
--- a/lib/VMCore/Value.cpp
+++ b/lib/VMCore/Value.cpp
@@ -1,6 +1,6 @@
 //===-- Value.cpp - Implement the Value class -----------------------------===//
 //
-// This file implements the Value class. 
+// This file implements the Value, User, and SymTabValue classes. 
 //
 //===----------------------------------------------------------------------===//
 
@@ -8,8 +8,6 @@
 #include "llvm/InstrTypes.h"
 #include "llvm/SymbolTable.h"
 #include "llvm/SymTabValue.h"
-#include "llvm/ConstantPool.h"
-#include "llvm/ConstPoolVals.h"
 #include "llvm/Type.h"
 #ifndef NDEBUG      // Only in -g mode...
 #include "llvm/Assembly/Writer.h"
@@ -20,8 +18,8 @@
 //                                Value Class
 //===----------------------------------------------------------------------===//
 
-Value::Value(const Type *ty, ValueTy vty, const string &name = "") : Name(name){
-  Ty = ty;
+Value::Value(const Type *ty, ValueTy vty, const string &name = "")
+  : Name(name), Ty(ty, this) {
   VTy = vty;
 }
 
@@ -34,6 +32,7 @@ Value::~Value() {
   // a <badref>
   //
   if (Uses.begin() != Uses.end()) {
+    cerr << "While deleting: " << this;
     for (use_const_iterator I = Uses.begin(); I != Uses.end(); ++I)
       cerr << "Use still stuck around after Def is destroyed:" << *I << endl;
   }
@@ -45,7 +44,7 @@ void Value::replaceAllUsesWith(Value *D) {
   assert(D && "Value::replaceAllUsesWith(<null>) is invalid!");
   assert(D != this && "V->replaceAllUsesWith(V) is NOT valid!");
   while (!Uses.empty()) {
-    User *Use = Uses.front();
+    User *Use = Uses.back();
 #ifndef NDEBUG
     unsigned NumUses = Uses.size();
 #endif
@@ -59,6 +58,16 @@ void Value::replaceAllUsesWith(Value *D) {
   }
 }
 
+// refineAbstractType - This function is implemented because we use
+// potentially abstract types, and these types may be resolved to more
+// concrete types after we are constructed.  For the value class, we simply
+// change Ty to point to the right type.  :)
+//
+void Value::refineAbstractType(const DerivedType *OldTy, const Type *NewTy) {
+  assert(Ty.get() == (const Type*)OldTy &&"Can't refine anything but my type!");
+  Ty = NewTy;
+}
+
 void Value::killUse(User *i) {
   if (i == 0) return;
   use_iterator I = find(Uses.begin(), Uses.end(), i);
@@ -74,6 +83,11 @@ User *Value::use_remove(use_iterator &I) {
   return i;
 }
 
+#ifndef NDEBUG      // Only in -g mode...
+void Value::dump() const {
+  cerr << this;
+}
+#endif
 
 //===----------------------------------------------------------------------===//
 //                                 User Class
@@ -103,22 +117,13 @@ void User::replaceUsesOfWith(Value *From, Value *To) {
 //                             SymTabValue Class
 //===----------------------------------------------------------------------===//
 
-// Instantiate Templates - This ugliness is the price we have to pay
-// for having a ValueHolderImpl.h file seperate from ValueHolder.h!  :(
-//
-template class ValueHolder<ConstPoolVal, SymTabValue, SymTabValue>;
-
-SymTabValue::SymTabValue(Value *p) : ConstPool(this), ValueParent(p) { 
+SymTabValue::SymTabValue(Value *p) : ValueParent(p) { 
   assert(ValueParent && "SymTavValue without parent!?!");
   ParentSymTab = SymTab = 0;
 }
 
 
 SymTabValue::~SymTabValue() {
-  ConstPool.dropAllReferences();
-  ConstPool.delete_all();
-  ConstPool.setParent(0);
-
   delete SymTab;
 }