Eugene.Zelenko added a comment. In D137740#3918335 <https://reviews.llvm.org/D137740#3918335>, @stephanemoore wrote:
> In D137740#3918271 <https://reviews.llvm.org/D137740#3918271>, > @Eugene.Zelenko wrote: > >> Will be good idea to combine this patch with D137738 >> <https://reviews.llvm.org/D137738>. > > My intent was to land the simpler linter fixes first while iterating on the > fixes in D137738 <https://reviews.llvm.org/D137738> as necessary but I can > combine the two patches if that is preferred. I don't think that such patches deserve dedicated commit unless you fix them on entire Clang-tidy code. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D137740/new/ https://reviews.llvm.org/D137740 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
