From 956d71a63a26902f7aa3f9740c59849055f3c6bc Mon Sep 17 00:00:00 2001
From: Chris Lattner <sabre@nondot.org>
Date: Fri, 13 Nov 2009 00:57:01 +0000
Subject: [PATCH] add a fixme, inheriting from PointerIntPair is gross :)

llvm-svn: 87048
---
 clang/include/clang/AST/Redeclarable.h | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/clang/include/clang/AST/Redeclarable.h b/clang/include/clang/AST/Redeclarable.h
index 1e6871ff3b9c..c04f3cd4e3a3 100644
--- a/clang/include/clang/AST/Redeclarable.h
+++ b/clang/include/clang/AST/Redeclarable.h
@@ -23,6 +23,8 @@ template<typename decl_type>
 class Redeclarable {
 
 protected:
+  // FIXME: PointerIntPair is a value class that should not be inherited from.
+  // This should change to using containment.
   struct DeclLink : public llvm::PointerIntPair<decl_type *, 1, bool> {
     DeclLink(decl_type *D, bool isLatest)
       : llvm::PointerIntPair<decl_type *, 1, bool>(D, isLatest) { }