[PATCH] D57104: [AST] Pack GenericSelectionExpr

Aaron Ballman via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Thu Jan 24 12:39:05 PST 2019


aaron.ballman accepted this revision.
aaron.ballman added a comment.
This revision is now accepted and ready to land.

In D57104#1369885 <https://reviews.llvm.org/D57104#1369885>, @riccibruno wrote:

> In D57104#1369864 <https://reviews.llvm.org/D57104#1369864>, @steveire wrote:
>
> > Thanks, but I don't see the factored out changes in the repo. Are you going to commit those first?
> >
> > Also, please factor out the introduction and use of `::Create`. It seems to be unrelated to the trailing objects change.
>
>
> I was planning to yes. I don't agree that the introduction of `Create` is unrelated. It is very much part of the changes
>  since the introduction of the trailing objects implies the need to do a placement new, which implies the need to
>  use a `Create` function. Also see pretty much every other patch which introduced trailing objects for the
>  statement/expression nodes.


+1, the changes to the `Create` methods are germane to this review.

This LGTM, thank you for working on it!


Repository:
  rC Clang

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D57104/new/

https://reviews.llvm.org/D57104





More information about the cfe-commits mailing list