[PATCH] CMake: add LLVM_INSTALL_DEV_FILES variable

[Chandler Carruth]

On Wed, Aug 21, 2013 at 3:50 PM, Hans Wennborg <hans at chromium.org> wrote:

> $ diff --unified /tmp/library.txt /tmp/all.txt | grep -e "^+"
> +++ /tmp/all.txt        2013-08-21 15:22:31.163831100 -0700
> +./bin/clang-tblgen.exe
> +./bin/llvm-lit
> +./bin/llvm-tblgen.exe
> +./share
> +./share/llvm
> +./share/llvm/cmake
> +./share/llvm/cmake/AddLLVM.cmake
> +./share/llvm/cmake/AddLLVMDefinitions.cmake
> +./share/llvm/cmake/ChooseMSVCCRT.cmake
> +./share/llvm/cmake/GetSVN.cmake
> +./share/llvm/cmake/HandleLLVMOptions.cmake
> +./share/llvm/cmake/LLVM-Config.cmake
> +./share/llvm/cmake/LLVMConfig.cmake
> +./share/llvm/cmake/LLVMConfigVersion.cmake
> +./share/llvm/cmake/LLVMParseArguments.cmake
> +./share/llvm/cmake/LLVMProcessSources.cmake
> +./share/llvm/cmake/TableGen.cmake
>

> Note that FileCheck, count, not, et al. are not installed, and they
> never were (my "all" variant is equivalent to our current install
> target). I should have noticed that earlier :/
>

Interesting!


>
> The cmake files are needed if someone builds e.g. Clang out-of-tree
> against the LLVM install dir.
>
> I don't think it makes sense to install llvm-tblgen and clang-tblgen
> at all. (Maybe it made sense before, when clang used llvm's tablegen).
>
> Given the above, would it make sense to merge #1 and #2?
>

Quite possibly... Can we avoid installing llvm-lit as well? (probably ask
Jordan?) If so, then SGTM.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20130821/1af42a63/attachment.html>