[PATCH] D49729: [AST][1/4] Move the bit-fields from TagDecl, EnumDecl and RecordDecl into DeclContext

[Bruno Ricci via Phabricator via cfe-commits]

bricci added a comment.

added some inline comments



================
Comment at: include/clang/AST/DeclBase.h:1662
+  enum { NumBlockDeclBits = 5 };
 
   /// Pointer to the data structure used to lookup declarations
----------------
uint64_t is used here because we will always store more than 32 bits
but less than 64 bits in these SomethingDeclBitfields.


================
Comment at: include/clang/AST/DeclBase.h:1959
-  void makeDeclVisibleInContextInternal(NamedDecl *D);
-
   StoredDeclsMap *CreateStoredDeclsMap(ASTContext &C) const;
----------------
This declaration has no corresponding definition and is unused.
Therefore I removed it but if this is more appropriate for another
patch I will do the necessary changes.


Repository:
  rC Clang

https://reviews.llvm.org/D49729