[PATCH] D50261: [AST] Remove unnecessary indirections in DeclarationNameTable

[Bruno Ricci via Phabricator via cfe-commits]

bricci added a comment.

It seems to me that the increased size of DeclarationNameTable is irrelevant since it is
only used as a member in ASTContext and never copied nor moved from. Also,
at least for C++ it seems to me that this structure is never "rarely used" since it is
used for every overloaded op/ctor/dtor (but I admit that I am not familiar with Sema)

I will do a comparison of the build time this evening. If this is a problem we can try to
find a way to avoid the QualType in CXXSpecialName (which is the reason why we need
Type.h) by replacing it by uintptr_t and doing some casts.


Repository:
  rC Clang

https://reviews.llvm.org/D50261