aaron.ballman accepted this revision. aaron.ballman added a comment. This revision is now accepted and ready to land.
LGTM! I'm happy to commit on your behalf once the minor nits are fixed up. ================ Comment at: clang-tools-extra/clang-tidy/modernize/UseUsingCheck.cpp:73 + // But also we need to check that ranges belongs the same file because + // different files may contain overlap ranges. std::string Using = "using "; ---------------- that ranges -> that the ranges belongs -> belong to overlap -> overlapping Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D73090/new/ https://reviews.llvm.org/D73090 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits