[libcxx-commits] [libcxx] [libc++] Specialize unique_ptr for the default deleter (PR #96504)

[Nikolas Klauser via libcxx-commits]

philnik777 wrote:

@Michael137 Thanks for all the work on this! I'm pretty sure this whole effort will drastically improve the QoI and maintainability of libc++. BTW sorry for being responsible for a significant amount of the additional complexity due to changed layouts in the pretty printers. Do you have any plans to remove support for old versions of the layouts at any point?

@zmodem Thanks for the info. Would you be able to also provide numbers for https://github.com/llvm/llvm-project/pull/76756? Something in the release notes like "this change resulted in a 5% decrease in debug info size for Chromium" always sounds great.

I hope it's good enough if https://github.com/llvm/llvm-project/pull/76756 lands in ~2 months? It would also be great if you could run the patch through its paces within google.

P.S. I noticed that my wording on "not caring about the pretty printers" was a bit misleading - I meant that we don't care to wait for them to propagate, not that we don't care that they work within the same release.


https://github.com/llvm/llvm-project/pull/96504