[PATCH] D28397: [lld][cmake] Fix LLVM_LINK_LLVM_DYLIB build

[Eric Fiselier via Phabricator via llvm-commits]

EricWF added a comment.

In https://reviews.llvm.org/D28397#651770, @davidlt wrote:

> TL;DR This broke lld.
>
> I was trying to build an RPM for LLVM/Clang 4.0.0 RC1 toolchain and noticed issues, which at first looks a caused by this change.
>
> With this change you are no more installing required shared libraries:
>  error: Failed dependencies:
>
>   liblldCOFF.so.4()(64bit) is needed by external+llvm+4.0.0-cms-1-1.x86_64
>   liblldDriver.so.4()(64bit) is needed by external+llvm+4.0.0-cms-1-1.x86_64
>   liblldELF.so.4()(64bit) is needed by external+llvm+4.0.0-cms-1-1.x86_64
>   
>
> I checked CMake install logs and they were not listed as installed.
>
> Looking at packages (thus installed) lld binary it depends on them.


+1

I'm running into the same problems trying to use lld.


Repository:
  rL LLVM

https://reviews.llvm.org/D28397