[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