[all-commits] [llvm/llvm-project] 2ea3c8: [formatters] Add a deque formatter for libstdcpp a...

Danil Stefaniuc via All-commits all-commits at lists.llvm.org
Mon Dec 6 13:42:22 PST 2021


  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: 2ea3c8a50add5436cf939d59c3235408ca0255c1
      https://github.com/llvm/llvm-project/commit/2ea3c8a50add5436cf939d59c3235408ca0255c1
  Author: Walter Erquinigo <wallace at fb.com>
  Date:   2021-12-06 (Mon, 06 Dec 2021)

  Changed paths:
    M lldb/examples/synthetic/gnu_libstdcpp.py
    M lldb/examples/synthetic/libcxx.py
    M lldb/source/Plugins/Language/CPlusPlus/CPlusPlusLanguage.cpp
    A lldb/test/API/functionalities/data-formatter/data-formatter-stl/generic/deque/Makefile
    A lldb/test/API/functionalities/data-formatter/data-formatter-stl/generic/deque/TestDataFormatterGenericDeque.py
    A lldb/test/API/functionalities/data-formatter/data-formatter-stl/generic/deque/main.cpp
    R lldb/test/API/functionalities/data-formatter/data-formatter-stl/libcxx/deque/Makefile
    R lldb/test/API/functionalities/data-formatter/data-formatter-stl/libcxx/deque/TestDataFormatterLibcxxDeque.py
    R lldb/test/API/functionalities/data-formatter/data-formatter-stl/libcxx/deque/main.cpp

  Log Message:
  -----------
  [formatters] Add a deque formatter for libstdcpp and fix the libcxx one

This adds the formatters for libstdcpp's deque as a python
implementation. It adds comprehensive tests for the two different
storage strategies deque uses. Besides that, this fixes a couple of bugs
in the libcxx implementation. Finally, both implementation run against
the same tests.

This is a minor improvement on top of Danil Stefaniuc's formatter.


  Commit: 6622c1411339488aa086704ac3fad3b040604dff
      https://github.com/llvm/llvm-project/commit/6622c1411339488aa086704ac3fad3b040604dff
  Author: Danil Stefaniuc <danilashtefan at gmail.com>
  Date:   2021-12-06 (Mon, 06 Dec 2021)

  Changed paths:
    M lldb/source/Plugins/Language/CPlusPlus/CPlusPlusLanguage.cpp
    M lldb/test/API/functionalities/data-formatter/data-formatter-stl/generic/forward_list/TestDataFormatterGenericForwardList.py
    M lldb/test/API/functionalities/data-formatter/data-formatter-stl/generic/forward_list/main.cpp
    M lldb/test/API/functionalities/data-formatter/data-formatter-stl/generic/list/TestDataFormatterGenericList.py
    M lldb/test/API/functionalities/data-formatter/data-formatter-stl/generic/list/main.cpp

  Log Message:
  -----------
  [formatters] Add a pointer and reference tests for a list and forward_list formatters for libstdcpp and libcxx

This adds extra tests for libstdcpp and libcxx list and forward_list formatters to check whether formatter behaves correctly when applied on pointer and reference values.

Reviewed By: wallace

Differential Revision: https://reviews.llvm.org/D115137


Compare: https://github.com/llvm/llvm-project/compare/68a7c001ad9a...6622c1411339


More information about the All-commits mailing list