[PATCH] CMake: add LLVM_INSTALL_LIBRARIES variable

Chandler Carruth chandlerc at google.com
Fri Aug 16 16:22:23 PDT 2013


On Fri, Aug 16, 2013 at 3:48 PM, Hans Wennborg <hans at chromium.org> wrote:

>   Change option name to LLVM_INSTALL_DEV_FILES. Also move cmake files and
> llvm-lit under this flag.
>
>   (Not changing the code review title to hopefully keep this in the same
> email thread.)
>

Er, no?

So, first note that this changes users of 'make install' in addition to
users of cpack.

There are two layers of this:

1) install development tools / resources useful for developing other *LLVM*
projects (tblgen, etc)

2) install *anything* but executable binaries.

I think these should be different. I think #1 should be default-off and
turned on for users of the stand-alone Clang cmake build. It should be
called something about "development".

I think #2 should be default-on, because many many users of LLVM
specifically want the core libraries. I think the best way is to negate the
flag: make it INSTALL_TOOLCHAIN_BINARIES_ONLY or something, and when
enabled, you only get the binaries used as part of a toolchain
(llvm-{ar,nm,dwarfdump,...}, clang, clang-format, etc., but not opt or llc.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20130816/83c97892/attachment.html>


More information about the llvm-commits mailing list