[all-commits] [llvm/llvm-project] 1056c5: [lldb] Adjust libc++ string formatter for changes ...

Pavel Labath via All-commits all-commits at lists.llvm.org
Thu Apr 21 05:18:18 PDT 2022

  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: 1056c56786c10866ffd7e878f8c75ad1f0914c07
  Author: Pavel Labath <pavel at labath.sk>
  Date:   2022-04-21 (Thu, 21 Apr 2022)

  Changed paths:
    M lldb/source/Plugins/Language/CPlusPlus/LibCxx.cpp

  Log Message:
  [lldb] Adjust libc++ string formatter for changes in D123580

The code needs more TLC, but for now I've tried making only the changes
that are necessary to get the tests passing -- postponing the more
invasive changes after I create a more comprehensive test.

In a couple of places I have changed the index-based element accesses to
name-based ones (as these are less sensitive to code perturbations). I'm
not sure why the code was using indexes in the first place, but I've
(manually) tested the change with various libc++ versions, and found no
issues with this approach.

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

More information about the All-commits mailing list