[Lldb-commits] [PATCH] D112564: [lldb] Add support for UTF-8 unicode formatting

2021-12-25 Thread Luís Ferreira via Phabricator via lldb-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rG46cdcf087300: [lldb] Add support for UTF-8 unicode formatting (authored by ljmf00). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D112564/new/ https://review

[Lldb-commits] [PATCH] D112564: [lldb] Add support for UTF-8 unicode formatting

2021-12-22 Thread Pavel Labath via Phabricator via lldb-commits
labath accepted this revision. labath added a comment. This revision is now accepted and ready to land. This looks fine. Thanks for tracking this down. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D112564/new/ https://reviews.llvm.org/D112564 ___

[Lldb-commits] [PATCH] D112564: [lldb] Add support for UTF-8 unicode formatting

2021-12-21 Thread Luís Ferreira via Phabricator via lldb-commits
ljmf00 added a comment. @labath I found the missing part, DumpTypeValue depends on lldb::Format and eFormatUnicode8 case should be added there too. I added tests and also changed the patch message. Can you re-review? CHANGES SINCE LAST ACTION https://reviews.llvm.org/D112564/new/ https://re

[Lldb-commits] [PATCH] D112564: [lldb] Add support for UTF-8 unicode formatting

2021-12-21 Thread Luís Ferreira via Phabricator via lldb-commits
ljmf00 updated this revision to Diff 395777. ljmf00 retitled this revision from "[lldb] Add support for custom char8_t types with different name" to "[lldb] Add support for UTF-8 unicode formatting". ljmf00 edited the summary of this revision. CHANGES SINCE LAST ACTION https://reviews.llvm.org/