[PATCH] CMake: add LLVM_INSTALL_DEV_FILES variable

[Hans Wennborg]

On Wed, Aug 21, 2013 at 6:38 PM, Hans Wennborg <hans at chromium.org> wrote:
> On Wed, Aug 21, 2013 at 6:32 PM, Chandler Carruth <chandlerc at google.com> wrote:
>>> >> Quite possibly... Can we avoid installing llvm-lit as well? (probably
>>> >> ask
>>> >> Jordan?) If so, then SGTM.
>>> >
>>> > Yes, I believe so. After chatting with Michael, I don't think there
>>> > are builds that rely on llvm-lit being installed.
>>> >
>>> > I'd like to go ahead and start this off by removing llvm-lit and the
>>> > tablegens from the install (patch attached) to make sure nothing
>>> > breaks and no one gets upset. OK to commit?
>>>
>>> Actually, let's start with removing only llvm-lit.
>>
>>
>> LGTM, please commit.
>
> r188976
>
>>> Turns out clang (and lld too) depends on llvm-tblgen for generating
>>> the option parsing tables. It even uses the existence of llvm-tblgen
>>> to detect a valid "build or install" directory for llvm :/
>>
>>
>> It would seem better to use llvm-config.
>>
>>>
>>> I guess one could argue that the option parsing stuff, including
>>> generating the tables, is part of llvm's functionality as a library.
>>> The rests of what llvm-tblgen does seems pretty internal to llvm
>>> though.
>>
>>
>> I think it is all internal. The option parsing stuff is definitely internal.
>> I think that clang and lld should use llvm-tblgen (if they need it) out of
>> the llvm build tree just like they use lit out of that tree. I think they
>> can find it with llvm-config?
>
> Sounds like a plan. I'll look into it.

OK, so llvm-lit and {clang,llvm}-tblgen are now no longer part of the install.

I've uploaded a new patch on http://llvm-reviews.chandlerc.com/D1428
which adds the LLVM_INSTALL_TOOLCHAIN_ONLY option. The Clang side is
at http://llvm-reviews.chandlerc.com/D1498.

Please take a look!

 - Hans