[libcxx-commits] [PATCH] D131082: [libcxx] Add _LIBCPP_NODEBUG to std::conditional related typedfs

[Louis Dionne via Phabricator via libcxx-commits]

ldionne added a comment.

FWIW @EricWF and I met offline just now and we agreed that it would probably improve the debugging experience to apply `_LIBCPP_NODEBUG_TYPE` more liberally to typedefs that represent computational things. We'd want to be careful about things that are alternate names for types, for example `std::size_t` and `Container::value_type`. So we'll have to consider each application individually for its own merits, but we both agree that it probably makes sense to apply it more liberally.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D131082