aaron.ballman added a comment. In D108893#2999681 <https://reviews.llvm.org/D108893#2999681>, @Eugene.Zelenko wrote:
> 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. So you'd like to see that extra functionality added now? (I don't think it makes sense to have the check as-is in both `readability` and `modernize`.) 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