klimek added a comment. In D80961#2068865 <https://reviews.llvm.org/D80961#2068865>, @ymandel wrote:
> Thank you for bringing up this issue. I think it highlights an underlying > problem -- editing templates is quite difficult -- that neither setting will > address, as Dmitri expanded on above. Given the parallel to macros, I'd say > your change is better than the status quo. Most clang tidies and other > rewriting tools that I've encountered simply skip code in macro expansions, > rather than reason about how to update the macro definition or whatnot. So, > by that reasoning, we should skip template instantations. I've personally written quite a few tools that changed macro definitions. Do you have numbers on how many of the library team's tools generally want to match template instantiations? Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D80961/new/ https://reviews.llvm.org/D80961 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
