[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