[PATCH] D56022: [AST] Store the arguments of CXXConstructExpr in a trailing array

2018-12-22 Thread Phabricator via Phabricator via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rL350003: [AST] Store the arguments of CXXConstructExpr in a trailing array (authored by brunoricci, committed by ). Herald added a subscriber: llvm-commits. Changed prior to commit: https://reviews.llvm.

[PATCH] D56022: [AST] Store the arguments of CXXConstructExpr in a trailing array

2018-12-21 Thread John McCall via Phabricator via cfe-commits
rjmccall accepted this revision. rjmccall added a comment. This revision is now accepted and ready to land. LGTM. Comment at: include/clang/AST/ExprCXX.h:1613 + + TypeSourceInfo *TSI = nullptr; + riccibruno wrote: > Changed the name from `Type` since `Type` wa

[PATCH] D56022: [AST] Store the arguments of CXXConstructExpr in a trailing array

2018-12-21 Thread Bruno Ricci via Phabricator via cfe-commits
riccibruno marked 2 inline comments as done. riccibruno added inline comments. Comment at: include/clang/AST/ExprCXX.h:1295 + /// A pointer to the constructor which will be ultimately called. + CXXConstructorDecl *Constructor; + Removed the default member initi

[PATCH] D56022: [AST] Store the arguments of CXXConstructExpr in a trailing array

2018-12-21 Thread Bruno Ricci via Phabricator via cfe-commits
riccibruno created this revision. riccibruno added a reviewer: rjmccall. riccibruno added a project: clang. Herald added a reviewer: shafik. Herald added a subscriber: cfe-commits. Store the arguments of `CXXConstructExpr` in a trailing array. This is very similar to the `CallExpr` case in D55771