nridge accepted this revision. nridge added a comment. This revision is now accepted and ready to land.
I'm not a clang-tidy maintainer so I don't know if I have the authority to green-light this patch, but I'm fairly confident the fix is correct (it's restoring the previous usage of the SourceManager API in this loop prior to this refactor <https://github.com/llvm/llvm-project/commit/5da7c040030c4af72dcc21220f579098469c554e>), and the tests capture the changed behaviour. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D126138/new/ https://reviews.llvm.org/D126138 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits