alexfh accepted this revision. alexfh added a comment. This revision is now accepted and ready to land.
LG ================ Comment at: clang-tidy/misc/UnusedUsingDeclsCheck.cpp:109 @@ -100,1 +108,3 @@ + // giving an incorrect message, we mark all of them as used. for (auto &Context : Contexts) { + if (Context.UsingTargetDecls.count(D->getCanonicalDecl()) > 0) ---------------- I suspect that linear search here may be the reason for the poor performance of the check (on a large file it's in top 3 by time spent). This should be addressed separately. Maybe leave a FIXME? http://reviews.llvm.org/D20909 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits