[libcxx-commits] [PATCH] D149092: [libcxxabi] disable lib{std}c++ assertions
Nikolas Klauser via Phabricator via libcxx-commits
libcxx-commits at lists.llvm.org
Wed May 3 10:55:24 PDT 2023
philnik added a comment.
In D149092#4316184 <https://reviews.llvm.org/D149092#4316184>, @nickdesaulniers wrote:
> Hot take; this patch is NFC for libc++. IF any code from libc++ being used in libcxxabi could assert, libcxxabi would have failed to build with a linkage failure against `__libcpp_verbose_abort` since libc++ asserts call `__libcpp_verbose_abort` upon assertion failure.
>
> Also, this patch isn't something small/for fun, it's blocking keeping libcxxabi's demangler in sync with LLVM's, so it's not something that we can ignore.
Even if it is NFC, it's still surprising. Switching to `string_view` is in this case a functional change, so we should consider what we want to do here.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D149092/new/
https://reviews.llvm.org/D149092
More information about the libcxx-commits
mailing list