- User Since
- Oct 20 2017, 4:34 PM (95 w, 1 d)
Tue, Jul 23
Mon, Jul 22
Sun, Jul 21
Fri, Jul 19
Yup! Sorry about that, this was fixed in git commit 474ca495a914fca0e.
Jul 18 2019
So add_llvm_utility exports it's target if LLVM_DISTRIBUTION_COMPONENTS doesn't exist. So I guess the better solution would just be setting that.
@JDevlieghere is this what you had in mind?
Jul 17 2019
Am I guessing correctly that this happens when you installed combined llvm+lldb build, and then try to build lldb standalone against that?
If you want to add the Swift-specific variable too, you could put up a PR for that on swift-lldb.
adjust for beanz comments
You can use CROSS_TOOLCHAIN_FLAGS_<PROJECT_NAME> (in this case CROSS_TOOLCHAIN_FLAGS_LLDB) to pass additional variables to the cross-compiling configure. That seems more appropriate for setting these options instead of hardcoding them in LLDB's CMake.
Jul 16 2019
LGTM! So the other functions (e.g. GetOrCreateDeclContextForUid, GetParentDeclContext etc.) will be also replaced in the future in a corresponding manner?
Was there a proposal or discussion (e.g., a thread on lldb-dev) about making PdbAstBuilder abstract in order to handle other languages? It sounds like a good idea, but I'd like to catch up on any context I may have missed.
I'm pretty sure that this doesn't work for Visual Studio (where you do not have a cross-compiler).
@aleksandr.urakov Nope, there is no eintr on Windows. There's supposed to be a ERROR_OPERATION_ABORTED according to the MSDN docs, but it evidently is a known bug that it's not happening. So you can't catch that it failed. This change does nothing other than let the user hit ctrl+c twice before exiting since we can't test if ctrl+c was hit in any way (without sleeping on the real thread, that is). The normal signal handling is still done as always, but Windows handles it asynchronously independent of this.
Still testing the builds, just figured I'd put it up early.