riccibruno added a comment. 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. Repository: rC Clang CHANGES SINCE LAST ACTION https://reviews.llvm.org/D57104/new/ https://reviews.llvm.org/D57104 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits