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

Reply via email to