[Lldb-commits] [PATCH] D58273: Fix TestDataFormatterLibcxxListLoop.py test

Pavel Labath via Phabricator via lldb-commits lldb-commits at lists.llvm.org
Fri Feb 15 01:39:23 PST 2019


labath accepted this revision.
labath added a comment.
This revision is now accepted and ready to land.

The slight problem here is that we don't know which libc++ version we are going to build with, so this will then fail for people still using the old one. However, choosing the latest libc++ version here seems like the right thing to do.

The background here is that we don't have a good way of testing that our pretty-printers behave sensibly when the structures they are interpreting get corrupted. This test was my attempt at improving that situation, but it's not without its shortfalls.

The other thing I tried a while back when writing the std::tuple formatter was to re-create a faithful mock of the class in the test. The advantage there would be that we are not tied to the libc++ version, and we would be even able to test formatting of different versions simultaneously. However, writing the mock proved to be so tedious, that I just gave up on the idea.


Repository:
  rLLDB LLDB

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

https://reviews.llvm.org/D58273





More information about the lldb-commits mailing list