I think it's fine to update others too, this is a trivially-safe change and a nice readability improvement. No need to put everything in the same patch though.
On Fri, Sep 28, 2018, 16:23 Kirill Bobyrev via Phabricator < revi...@reviews.llvm.org> wrote: > kbobyrev added a comment. > > Also, I'm not sure whether I should update Clang-Tidy and other tools in > the scope of this patch. It makes sense to me, but I don't know whether the > maintainers of these projects are happy with the change. WDYT? > > > https://reviews.llvm.org/D52650 > > > >
_______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits