[PATCH] D39522: [clang-rename] Use add_clang_tool

2017-11-01 Thread Shoaib Meenai via Phabricator via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rL317150: [clang-rename] Use add_clang_tool (authored by smeenai). Repository: rL LLVM https://reviews.llvm.org/D39522 Files: cfe/trunk/tools/clang-rename/CMakeLists.txt Index: cfe/trunk/tools/clang

[PATCH] D39522: [clang-rename] Use add_clang_tool

2017-11-01 Thread Alex Lorenz via Phabricator via cfe-commits
arphaman accepted this revision. arphaman added a comment. This revision is now accepted and ready to land. LGTM. https://reviews.llvm.org/D39522 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/

[PATCH] D39522: [clang-rename] Use add_clang_tool

2017-11-01 Thread Shoaib Meenai via Phabricator via cfe-commits
smeenai created this revision. Herald added a subscriber: mgorny. `add_clang_tool` includes a call to `add_clang_executable`, but it also sets up the install rule, and adds an `install-*` target. The latter is required for using `LLVM_DISTRIBUTION_COMPONENTS`. https://reviews.llvm.org/D39522 Fi