[PATCH] D65609: Add gdb pretty printers for a wide variety of libc++ data structures.

Mon Aug 5 10:21:42 PDT 2019

ldionne requested changes to this revision.
ldionne added a comment.
This revision now requires changes to proceed.

In D65609#1615153 <https://reviews.llvm.org/D65609#1615153>, @Eugene.Zelenko wrote:

> In D65609#1615142 <https://reviews.llvm.org/D65609#1615142>, @ldionne wrote:
>
> > In D65609#1611310 <https://reviews.llvm.org/D65609#1611310>, @Eugene.Zelenko wrote:
> >
> > > Please also add installation, since pretty-printers should be shipped with libc++, same as GCC does with libstdc++ and its pretty-printers.
> >
> >
> > Did these pretty-printers exist for libc++ before this patch? If so, how were they shipped?
>
>
> There is separate project on GitHub <https://github.com/koutheir/libcxx-pretty-printers>. It was mentioned in libcxx/docs/UsingLibcxx.rst.

I had missed that, thanks!

In that case, this patch should update the documentation.

> GCC ships libstdc++ pretty-printers in <install directory>/share/gcc-<version>/python/libstdcxx/.

I guess we could start with doing the same for libc++'s pretty-printers.

Repository:
  rL LLVM

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

https://reviews.llvm.org/D65609