[PATCH] D57098: [WIP][AST] NFC: Introduce new class GenericSelectionExpr::Association

Stephen Kelly via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Wed Jan 23 08:14:57 PST 2019


steveire added a comment.

In D57098#1367852 <https://reviews.llvm.org/D57098#1367852>, @riccibruno wrote:

> In D57098#1367816 <https://reviews.llvm.org/D57098#1367816>, @steveire wrote:
>
> > Thanks for doing this!
> >
> > It would be easier to review (now and in the future!) if you split it into at least 3 commits
> >
> > - Refactor use trailing objects without introducing the Association class
> > - The change to use bitfields and Stmt.h
> > - Introduce new class GenericSelectionExpr::Association
> >
> >   Thanks,
> >
> >   Stephen
>
>
> Is it okay if I split it into:
>
> 1. `llvm::TrailingObjects` + bit-fields of `Stmt`
> 2. Introduce `GenericSelectionExpr::Association`
>
>   since using `llvm::TrailingObjects` and using the bit-fields of `Stmt` are closely related and come under the title "[AST] Pack GenericSelectionExpr".


I didn't realize that, so your proposal sounds good to me!


Repository:
  rC Clang

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

https://reviews.llvm.org/D57098





More information about the cfe-commits mailing list