[PATCH] D68429: [clang] [cmake] Use add_clang_tool() to install all tools

[Chris Bieneman via Phabricator via cfe-commits]

beanz added a comment.

Are these tools intended to be installed? I thought they were developer focused tools. I don't think we should make install targets for things that the project doesn't want to support publicly.

This also changes the behavior of the `install` target to include these extra tools, which will increase the size of installs that many people are using today with these tools. At the very least if we go down this direction we need to add some option like `LLVM_INSTALL_TOOLCHAIN_ONLY` so that we don't radically alter the current `install` behavior.


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D68429/new/

https://reviews.llvm.org/D68429