steveire added a comment.

There's definitely a better possible ordering in two commits:

1. Introduce `::Create` and port to it
2. Use trailing objects, taking advantage of the fact that `::Create` exists.

That would make it clear in the future to other people because both commits 
would be cleaner, both commit messages would say what the commit does, and 
neither commit would have the noise of the other change.

Not splitting this commit makes it less reviewable to people who are not around 
today.


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

Reply via email to