[Lldb-commits] [PATCH] D47625: [cmake] Detect presence of wide-char libedit at build time

Pavel Labath via lldb-commits lldb-commits at lists.llvm.org
Thu Feb 28 07:18:24 PST 2019


On 28/02/2019 02:37, Davide Italiano via Phabricator wrote:
> davide added a comment.
> Herald added a subscriber: jdoerfert.
> Herald added a project: LLVM.
> 
> Pavel, this broke unicode handling for lldb on MacOS. If you type something in the lldb cmdline, it won't print the right character but a series of unicode sequences.
> The main concern is that this also breaks the Swift REPL (powered by lldb).
> 
> Looks like something like this:
> 
>    $ git diff
>    diff --git a/lldb/include/lldb/Host/Editline.h b/lldb/include/lldb/Host/Editline.h
>    index cb40f1e6917..e53991c450f 100644
>    --- a/lldb/include/lldb/Host/Editline.h
>    +++ b/lldb/include/lldb/Host/Editline.h
>    @@ -51,6 +51,7 @@
>     #include <string>
>     #include <vector>
>     
>    +#include "lldb/Host/Config.h"
>     #include "lldb/Host/ConnectionFileDescriptor.h"
>     #include "lldb/Host/Predicate.h"
>     #include "lldb/Utility/FileSpec.h"
> 
> "fixes" the CMake build. I'm seeing failures if I apply the same patch to the Xcode project.

Have you tried putting the `#include "lldb/Host/Config.h"` at the top of 
Editline.h, so that LLDB_EDITLINE_USE_WCHAR is defined before the
```
#if LLDB_EDITLINE_USE_WCHAR
#include <codecvt>
#endif
```
block ?


More information about the lldb-commits mailing list