-
Notifications
You must be signed in to change notification settings - Fork 14.3k
[flang] Use clang_target_link_libraries() for clang dependency #126037
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
This dependency is part of libclang-cpp, so it should use clang_target_link_libraries.
@llvm/pr-subscribers-flang-driver Author: Nikita Popov (nikic) ChangesThis dependency is part of libclang-cpp, so it should use clang_target_link_libraries. Full diff: https://github.com/llvm/llvm-project/pull/126037.diff 1 Files Affected:
diff --git a/flang/unittests/Frontend/CMakeLists.txt b/flang/unittests/Frontend/CMakeLists.txt
index 9177997f41f53ac..2e3e7cb596c5802 100644
--- a/flang/unittests/Frontend/CMakeLists.txt
+++ b/flang/unittests/Frontend/CMakeLists.txt
@@ -12,7 +12,6 @@ add_flang_unittest(FlangFrontendTests
target_link_libraries(FlangFrontendTests
PRIVATE
- clangBasic
flangFrontend
flangFrontendTool
FortranLower
@@ -22,6 +21,11 @@ target_link_libraries(FlangFrontendTests
FortranEvaluate
)
+clang_target_link_libraries(FlangFrontendTests
+ PRIVATE
+ clangBasic
+)
+
mlir_target_link_libraries(FlangFrontendTests
PRIVATE
MLIRIR
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks!
/cherry-pick 7c695e4 |
/pull-request #126062 |
LLVM Buildbot has detected a new failure on builder Full details are available at: https://lab.llvm.org/buildbot/#/builders/153/builds/22117 Here is the relevant piece of the build log for the reference
|
…126037) This dependency is part of libclang-cpp, so it should use clang_target_link_libraries.
This dependency is part of libclang-cpp, so it should use clang_target_link_libraries.