Re: [PATCH] D13976: [AST] Store Decl* and Stmt* directly into the ParentMap.

2015-10-22 Thread Phabricator via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rL251008: [AST] Store Decl* and Stmt* directly into the ParentMap. (authored by d0k). Changed prior to commit: http://reviews.llvm.org/D13976?vs=38105&id=38110#toc Repository: rL LLVM http://reviews.l

Re: [PATCH] D13976: [AST] Store Decl* and Stmt* directly into the ParentMap.

2015-10-22 Thread Manuel Klimek via cfe-commits
klimek accepted this revision. klimek added a comment. This revision is now accepted and ready to land. lg Comment at: include/clang/AST/ASTContext.h:463 @@ +462,3 @@ + class DynTypedNodeList { +typedef ast_type_traits::DynTypedNode DynTypedNode; +typedef ArrayRef ARef;

[PATCH] D13976: [AST] Store Decl* and Stmt* directly into the ParentMap.

2015-10-22 Thread Benjamin Kramer via cfe-commits
bkramer created this revision. bkramer added reviewers: klimek, djasper. bkramer added a subscriber: cfe-commits. These are by far the most common types to be parents in the AST so it makes sense to optimize for them. Put them directly into the value of the map. This currently saves 32 bytes per p