NoQ accepted this revision.
NoQ added a comment.
This revision is now accepted and ready to land.

Looks good. To think: maybe we need a better name (and/or a place to live) for 
this API, now that it's not only related to clone detection, but is useful for 
more stuff?



================
Comment at: include/clang/Analysis/CloneDetection.h:20-24
 #include "clang/Basic/SourceLocation.h"
 #include "llvm/ADT/SmallVector.h"
 #include "llvm/ADT/StringRef.h"
 #include "llvm/Support/Regex.h"
 #include <vector>
----------------
Most of these #includes are now redundant.


https://reviews.llvm.org/D34880



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
  • [PATCH] D34880: [analyzer]... Artem Dergachev via Phabricator via cfe-commits

Reply via email to