diff options
| author | Andrzej Warzynski <andrzej.warzynski@arm.com> | 2022-04-14 08:40:02 +0000 |
|---|---|---|
| committer | Andrzej Warzynski <andrzej.warzynski@arm.com> | 2022-04-14 08:58:34 +0000 |
| commit | 8de6cb129736bdb47c8d6305a8bf5033fbac9f2f (patch) | |
| tree | 8b0d3c48d4fb0d3fe9c66023ff054a86e713e4b1 /flang/lib/Frontend/CompilerInvocation.cpp | |
| parent | d79ad2f1dbc2db63121620f55d6cfa915f2733ac (diff) | |
| download | llvm-8de6cb129736bdb47c8d6305a8bf5033fbac9f2f.zip llvm-8de6cb129736bdb47c8d6305a8bf5033fbac9f2f.tar.gz llvm-8de6cb129736bdb47c8d6305a8bf5033fbac9f2f.tar.bz2 | |
[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
Diffstat (limited to 'flang/lib/Frontend/CompilerInvocation.cpp')
0 files changed, 0 insertions, 0 deletions