[PATCH] D23045: [Include-fixer] Install executables and support scripts
Eugene Zelenko via cfe-commits
cfe-commits at lists.llvm.org
Tue Aug 2 10:11:45 PDT 2016
Eugene.Zelenko added inline comments.
================
Comment at: include-fixer/find-all-symbols/tool/CMakeLists.txt:22
@@ +21,2 @@
+ DESTINATION share/clang
+ COMPONENT find-all-symbols)
----------------
hokein wrote:
> I think we can put it in `clang-include-fixer` as find-all-symbols is a sub-tool of include-fixer.
In other make files executable name and COMPONENT for scripts are same. Should I add COMPONENT to find-all-symbols executable too?
Repository:
rL LLVM
https://reviews.llvm.org/D23045
More information about the cfe-commits
mailing list