[llvm-commits] CVS: llvm/include/llvm/Constants.h

Chris Lattner sabre at nondot.org
Mon Feb 19 22:40:14 PST 2007



Changes in directory llvm/include/llvm:

Constants.h updated: 1.130 -> 1.131
---
Log message:

cleanup ConstantInt to use a single DenseMap for uniquing instead of the
heavy-weight ValueMap class.  This reduces mem usage bc reading kc++ by 29K, 
even though it only creates 2955 constant ints!


---
Diffs of the changes:  (+1 -4)

 Constants.h |    5 +----
 1 files changed, 1 insertion(+), 4 deletions(-)


Index: llvm/include/llvm/Constants.h
diff -u llvm/include/llvm/Constants.h:1.130 llvm/include/llvm/Constants.h:1.131
--- llvm/include/llvm/Constants.h:1.130	Tue Feb 20 00:11:36 2007
+++ llvm/include/llvm/Constants.h	Tue Feb 20 00:39:57 2007
@@ -41,12 +41,9 @@
 /// @brief Class for constant integers.
 class ConstantInt : public Constant {
   static ConstantInt *TheTrueVal, *TheFalseVal;
-protected:
-  uint64_t Val;
-protected:
   ConstantInt(const ConstantInt &);      // DO NOT IMPLEMENT
   ConstantInt(const IntegerType *Ty, uint64_t V);
-  friend struct ConstantCreator<ConstantInt, IntegerType, uint64_t>;
+  uint64_t Val;
 public:
   /// Return the constant as a 64-bit unsigned integer value after it
   /// has been zero extended as appropriate for the type of this constant.






More information about the llvm-commits mailing list