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

[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

Files:
  tools/clang-rename/CMakeLists.txt


Index: tools/clang-rename/CMakeLists.txt
===================================================================
--- tools/clang-rename/CMakeLists.txt
+++ tools/clang-rename/CMakeLists.txt
@@ -3,7 +3,7 @@
   Support
   )
 
-add_clang_executable(clang-rename ClangRename.cpp)
+add_clang_tool(clang-rename ClangRename.cpp)
 
 target_link_libraries(clang-rename
   clangBasic
@@ -14,8 +14,6 @@
   clangToolingRefactor
   )
 
-install(TARGETS clang-rename RUNTIME DESTINATION bin)
-
 install(PROGRAMS clang-rename.py
   DESTINATION share/clang
   COMPONENT clang-rename)


-------------- next part --------------
A non-text attachment was scrubbed...
Name: D39522.121216.patch
Type: text/x-patch
Size: 563 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20171102/6068d1c5/attachment.bin>