ehsan added a comment.

In https://reviews.llvm.org/D29806#673329, @alexfh wrote:

> What's your use case? Can it be addressed by just forwarding the -p flag to 
> clang-tidy?


I just need to pass the full path to the compilation DB to clang-tidy.  The 
problem is that invoking `clang-tidy-diff.py -- -p PATH` will run `clang-tidy 
-- -p PATH`, in order words adding -p to the compiler command line, not 
clang-tidy's.

> The script shouldn't know anything about implementation details of the 
> compilation database being used (since it can be something other than JSON 
> compilation database).

I copied the code to look for the DB verbatim from run-clang-tidy.py.  I 
personally don't need the search logic, and just tried to keep this consistent 
with run-clang-tidy.py.  I'd be happy to remove the search logic if you prefer 
that.


https://reviews.llvm.org/D29806



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

Reply via email to