lebedev.ri added a comment. In D57896#1389046 <https://reviews.llvm.org/D57896#1389046>, @michaelplatings wrote:
> In D57896#1389042 <https://reviews.llvm.org/D57896#1389042>, @lebedev.ri > wrote: > > > Pretty sure this patch should have gone to llvm-commits, not cfe-commits. > > > I just set the repository, Phabricator did the rest - apparently the magic > isn't working so well. 1. Does clang-tidy warn on every single existing variable now? 2. It might be best to give this more visibility, by submitting a mail to llvm-dev, with a **noticeable** subject, like "RFC: changing variable naming rules in LLVM codebase" Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D57896/new/ https://reviews.llvm.org/D57896 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
