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

[David Abdurachmanov via Phabricator via llvm-commits]

davidlt added a comment.

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.


Repository:
  rL LLVM

https://reviews.llvm.org/D28397