Eugene.Zelenko added a comment. In D131780#3720896 <https://reviews.llvm.org/D131780#3720896>, @carlosgalvezp wrote:
> In D131780#3719881 <https://reviews.llvm.org/D131780#3719881>, > @Eugene.Zelenko wrote: > >> Please mention changes in Release Notes. > > Thanks for the quick review, @Eugene.Zelenko ! > > This check was newly added a couple of days ago (as per the "New checks" > section in the Release Notes). I thought the "Changes to exiting checks" > section was only meant for checks already existing prior to the new release, > to avoid adding excessive clutter to the documentation. New checks typically > take a few iterations to polish when testing on larger projects. > > What do you think? Sorry, I didn't know history. Sure, it doesn't make sense for checks added in `main`. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D131780/new/ https://reviews.llvm.org/D131780 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
