[all-commits] [llvm/llvm-project] 8de6cb: [flang] Fix DYLIB builds

Andrzej WarzyƄski via All-commits all-commits at lists.llvm.org
Thu Apr 14 01:59:48 PDT 2022


  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: 8de6cb129736bdb47c8d6305a8bf5033fbac9f2f
      https://github.com/llvm/llvm-project/commit/8de6cb129736bdb47c8d6305a8bf5033fbac9f2f
  Author: Andrzej Warzynski <andrzej.warzynski at arm.com>
  Date:   2022-04-14 (Thu, 14 Apr 2022)

  Changed paths:
    M flang/lib/Frontend/CMakeLists.txt

  Log Message:
  -----------
  [flang] Fix DYLIB builds

https://reviews.llvm.org/D123211 broke builds that set both
`LLVM_BUILD_LLVM_DYLIB` and `LLVM_LINK_LLVM_DYLIB` (see [1]). This patch
fixes that.

The build failure was caused by the fact that the LLVMPasses library,
which is an LLVM "component", was listed directly as link-time
dependency. Instead, one should use `LINK_COMPONENTS` in CMake files. I
made an identical mistake recently and then subsequently fixed it in
https://reviews.llvm.org/D121461 - please visit that revision for more
detail.

I'm merging this without a review. The change is straightforward, we
recently discussed it and I was able to confirm locally that it fixes
the build issue.

[1] https://lab.llvm.org/buildbot/#/builders/177/builds/4619




More information about the All-commits mailing list