Eugene.Zelenko added a comment. In D108893#2999654 <https://reviews.llvm.org/D108893#2999654>, @aaron.ballman wrote:
> In D108893#2999645 <https://reviews.llvm.org/D108893#2999645>, > @Eugene.Zelenko wrote: > >> What about adding `modernize`/`bugprone` aliases? > > I'd be fine if we wanted to add aliases, but I'd sort of expect some extra > functionality out of a check in those modules. I think it's fine to land this > now, and we can add aliases in a follow-up. WDYT? Aliases code is trivial comparing with check itself, so it make sense to add it in this patch. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D108893/new/ https://reviews.llvm.org/D108893 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits