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

Chris Lattner sabre at nondot.org
Fri Sep 22 23:09:59 PDT 2006



Changes in directory llvm/include/llvm:

Type.h updated: 1.88 -> 1.89
---
Log message:

remove misleading comment


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

 Type.h |    8 +-------
 1 files changed, 1 insertion(+), 7 deletions(-)


Index: llvm/include/llvm/Type.h
diff -u llvm/include/llvm/Type.h:1.88 llvm/include/llvm/Type.h:1.89
--- llvm/include/llvm/Type.h:1.88	Tue Jul 25 18:22:00 2006
+++ llvm/include/llvm/Type.h	Sat Sep 23 01:09:45 2006
@@ -371,13 +371,7 @@
 //===----------------------------------------------------------------------===//
 // Define some inline methods for the AbstractTypeUser.h:PATypeHandle class.
 // These are defined here because they MUST be inlined, yet are dependent on
-// the definition of the Type class.  Of course Type derives from Value, which
-// contains an AbstractTypeUser instance, so there is no good way to factor out
-// the code.  Hence this bit of uglyness.
-//
-// In the long term, Type should not derive from Value, allowing
-// AbstractTypeUser.h to #include Type.h, allowing us to eliminate this
-// nastyness entirely.
+// the definition of the Type class.
 //
 inline void PATypeHandle::addUser() {
   assert(Ty && "Type Handle has a null type!");






More information about the llvm-commits mailing list