[libcxx-commits] [PATCH] D154997: [libc++][hardening] Deprecate `_LIBCPP_ENABLE_ASSERTIONS`.
Nico Weber via Phabricator via libcxx-commits
libcxx-commits at lists.llvm.org
Wed Aug 9 06:51:38 PDT 2023
thakis added a comment.
We are currently shipping the old _LIBCPP_ENABLE_ASSERTION mode. Is there a list of checks that we'll lose when we switch to this new hardened mode?
Also, it's a bit unfortunate that the Windows build of libc++ got broken before this landed (by D153709 <https://reviews.llvm.org/D153709>) and only fixed after this landed (in D155185 <https://reviews.llvm.org/D155185>). That means we can't update to the revision before this change here, then turn switch to hardened mode independent of updating libc++, and then keep updating. It'd be nice to have a temporary opt-out to make it possible to update libc++ in one change and then switch to this new mode in a separate change without updating libc++.
Currently, we'd have to pull in over 100 libc++ revisions at once, including this change (which as far as I understand causes us to lose a not-further-described list of checks).
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D154997/new/
https://reviews.llvm.org/D154997
More information about the libcxx-commits
mailing list