lonico added a comment.


> The second way to handle this use case is possible on a different level: one 
> can run clang-tidy over the whole project with `-header-filter=.*` and then 
> filter the results and use a specialized tool to display them, e.g. 
> https://github.com/Ericsson/codechecker. I'm not sure though whether this 
> suits your needs or is an overkill.

This approach does not work for us, as our external headers generate a lot of 
warnings.  It's not so much the warnings that are an issue, but generating the 
notes is taking a lot of time.  We implemented this patch in our local 
installation, and it dramatically reduces the time it takes to run clang-tidy 
on a "component", where we can focus on the source and header files in a 
subdirectory tree, and exclude headers from other subdirectories.  In the worst 
case, we're really comparing 1 hour to a few minutes.

Even the first approach is not so interesting, as we would like warnings for 
another subdirectory tree to be visible in this subdirectory.   What we really 
want is to be able to segregate warnings per component (a subdirectory tree) 
and don't bother people who import headers from other subdirectory trees with 
local warnings.  So the owner of a subdirectory tree should see all warnings 
for their tree, but consumers of headers should not be bothered.


Repository:
  rL LLVM

https://reviews.llvm.org/D34654



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to