[Lldb-commits] [PATCH] D51520: Add libc++ data formatter for std::variant

Shafik Yaghmour via Phabricator via lldb-commits lldb-commits at lists.llvm.org
Wed Sep 19 10:26:08 PDT 2018


shafik updated this revision to Diff 166144.
shafik added a comment.

Updating LibcxxVariantGetIndexValidity() to no longer do type check of __index. It was left over from the old method of checking for an empty variant and was also breaking clang 5.


https://reviews.llvm.org/D51520

Files:
  lldb.xcodeproj/project.pbxproj
  packages/Python/lldbsuite/test/functionalities/data-formatter/data-formatter-stl/libcxx/variant/Makefile
  packages/Python/lldbsuite/test/functionalities/data-formatter/data-formatter-stl/libcxx/variant/TestDataFormatterLibcxxVariant.py
  packages/Python/lldbsuite/test/functionalities/data-formatter/data-formatter-stl/libcxx/variant/main.cpp
  source/Plugins/Language/CPlusPlus/CMakeLists.txt
  source/Plugins/Language/CPlusPlus/CPlusPlusLanguage.cpp
  source/Plugins/Language/CPlusPlus/LibCxx.h
  source/Plugins/Language/CPlusPlus/LibCxxVariant.cpp
  source/Plugins/Language/CPlusPlus/LibCxxVariant.h

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D51520.166144.patch
Type: text/x-patch
Size: 21157 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/lldb-commits/attachments/20180919/8147597f/attachment-0001.bin>


More information about the lldb-commits mailing list