[Lldb-commits] [PATCH] D66447: Add char8_t support (C++20)

Jonas Devlieghere via Phabricator via lldb-commits lldb-commits at lists.llvm.org
Wed Aug 21 09:30:43 PDT 2019


JDevlieghere added a comment.

I changed the test to use `frame variable` again. With `target variable` the UTF-8 formatting doesn't work. Given that this patch just copies the Char16 and Char32 implementation, I think that's something for a different patch. I'll file a PR when this gets in.

  (lldb) target variable ab
  (const char8_t *) ab = 0x0000000100000fa6
  (lldb) target variable abc
  (char8_t [9]) abc = {
    [0] = 0xe4 u8'
                    [1] = 0xbd u8''                                                                                                                                                                                                                                                                                                                     [2] = 0xa0 u8''                                                                                                                                                                                                                                                                                                                                     [3] = 0xe5 u8'
                    [4] = 0xa5 u8''
    [5] = 0xbd u8''
    [6] = 0x00 u8'\0'
    [7] = 0x00 u8'\0'
    [8] = 0x00 u8'\0'
  }


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D66447/new/

https://reviews.llvm.org/D66447





More information about the lldb-commits mailing list