whisperity added a comment. In D124447#3645197 <https://reviews.llvm.org/D124447#3645197>, @sammccall wrote:
> In D124447#3608747 <https://reviews.llvm.org/D124447#3608747>, @aaron.ballman > wrote: > >> In general, I think this is looking pretty close to good. Whether clang-tidy >> should get this functionality in this form or not is a bit less clear to me. >> *I* think it's helpful functionality and the current approach is reasonable, >> but I think it might be worthwhile to have a community RFC to see if others >> agree. CC @alexfh in case he wants to make a code owner decision instead. > > +1 to a need for the RFC here, there are ecosystem questions that I think > should be addressed first but aren't in scope of this patch. I will attempt to write a concise response to this today (and tomorrow)! Sorry, I was away travelling to and doing post-action bureaucracy of conferences the past few weeks. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D124447/new/ https://reviews.llvm.org/D124447 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
