[libcxx-commits] [libcxx] [libc++] Put _LIBCPP_NODEBUG on all internal aliases (PR #118710)

[Louis Dionne via libcxx-commits]

ldionne wrote:

> > Are we able to narrow down to which declarations are causing problems? Reverting this is going to create problems for people who monitor their debug info sizes.
> 
> This has been in the tree for all of a day. Is this really going to cause a problem so quickly?

Reverting this patch itself would not cause an issue for people monitoring their debug info size, but the plan was not to revert only this patch, it was to make `_LIBCPP_NODEBUG` a no-op. That has impact on existing typedefs that were marked with `_LIBCPP_NODEBUG` previously as well, and will cause debug info sizes to jump up for people monitoring them.

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