-
Notifications
You must be signed in to change notification settings - Fork 13.5k
[windows] fix flaky linker error when building LLDB #138249
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
[windows] fix flaky linker error when building LLDB #138249
Conversation
(cherry picked from commit e186f01)
…b-Convert-inline-test-to-a-python-test-NFC [Cherry-pick into next] [lldb] Convert inline test to a python test (NFC)
@llvm/pr-subscribers-lldb Author: Charles Zablit (charles-zablit) ChangesWhen building LLDB on Windows with
This PR fixes this build error. Full diff: https://github.com/llvm/llvm-project/pull/138249.diff 1 Files Affected:
diff --git a/lldb/source/API/CMakeLists.txt b/lldb/source/API/CMakeLists.txt
index 3bc569608e458..715aa4344f37c 100644
--- a/lldb/source/API/CMakeLists.txt
+++ b/lldb/source/API/CMakeLists.txt
@@ -142,6 +142,7 @@ add_lldb_library(liblldb SHARED ${option_framework}
lldbValueObject
lldbVersion
${LLDB_ALL_PLUGINS}
+ swiftCore
LINK_COMPONENTS
Support
|
@@ -142,6 +142,7 @@ add_lldb_library(liblldb SHARED ${option_framework} | |||
lldbValueObject | |||
lldbVersion | |||
${LLDB_ALL_PLUGINS} | |||
swiftCore |
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.
I don't see a library named this in llvm/llvm-project
, did you mean to submit this to Apple's llvm fork?
Which is https://github.com/swiftlang/llvm-project (I think).
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.
You are correct, sorry about that. I opened the PR here: swiftlang#10612
4884f91
to
54eac41
Compare
When building LLDB on Windows with
build.ps1
, the following linker error happens, especially after rebuilding incrementally. The error sometimes goes away after deleting CMakeCache.txt, but that's not reliable.This PR fixes this build error.