Re: [PATCH] D13206: Add clang-query tool to installation targets

2015-09-28 Thread Evangelos Foutras via cfe-commits
foutrelis added a comment. Thanks for the review. Can you please commit the patch to `clang-tools-extra/trunk` (since I don't have commit rights)? http://reviews.llvm.org/D13206 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.

Re: [PATCH] D13206: Add clang-query tool to installation targets

2015-09-28 Thread Manuel Klimek via cfe-commits
klimek closed this revision. klimek added a comment. Submitted in r248710 http://reviews.llvm.org/D13206 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Re: [PATCH] D13206: Add clang-query tool to installation targets

2015-09-28 Thread Manuel Klimek via cfe-commits
klimek added a subscriber: klimek. klimek accepted this revision. klimek added a reviewer: klimek. klimek added a comment. This revision is now accepted and ready to land. LG As we already a) install it from auto-tools and b) this tool is useful for non-llvm/clang devs http://reviews.llvm.org/

[PATCH] D13206: Add clang-query tool to installation targets

2015-09-28 Thread Evangelos Foutras via cfe-commits
foutrelis created this revision. foutrelis added a subscriber: cfe-commits. It has been requested in a bug report downstream in Arch Linux to include the clang-query tool as part of our clang-tools-extra package. [1] This point has also been brought up on the cfe-dev list and the LLVM bug track

Re: [PATCH] D13206: Add clang-query tool to installation targets

2015-09-28 Thread Evangelos Foutras via cfe-commits
foutrelis added a comment. Just a note that this change is specific to CMake since autotools already installs clang-query (along with some other tools). http://reviews.llvm.org/D13206 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://l