[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