[Lldb-commits] [PATCH] D102208: Remove Windows editline from LLDB
Neal via Phabricator via lldb-commits
lldb-commits at lists.llvm.org
Tue May 11 11:28:16 PDT 2021
nealsid added a comment.
I ran LLDB tests from a clean tree and with my change and the failures seemed to be the same, but there is a large amount of them. I also did manual testing with the changes. Build is still green.
Editline is only used when Cmake can find the editline libraries, which it never does on Windows. So the code that is actually used for console I/O is the code in the IOHandlerEditline class in IOHandler.cpp that is outside of the #ifdef LLDB_USE_EDITLINE preprocessor check. That uses POSIX functions for console I/O. I think it may be a good follow up change to make IOHandlerEditline be used in editline-specific cases and move the fallback/non-editline code in that class to another class.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D102208/new/
https://reviews.llvm.org/D102208
More information about the lldb-commits
mailing list