[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