On Thu, 15 Jul 2021, 18:21 François Dumont via Libstdc++, < libstd...@gcc.gnu.org> wrote:
> On 15/07/21 5:26 pm, Jonathan Wakely via Libstdc++ wrote: > > This results in slightly smaller code when assertions are enabled when > > either using Clang (because it adds code to call std::terminate when > > potentially-throwing functions are called in a noexcept function) or a > > freestanding or non-verbose build (because it doesn't use printf). > > > > Signed-off-by: Jonathan Wakely <jwak...@redhat.com> > > > > libstdc++-v3/ChangeLog: > > > > PR libstdc++/101429 > > * include/bits/c++config (__replacement_assert): Add noexcept. > > [!_GLIBCXX_VERBOSE] (__glibcxx_assert_impl): Use __builtin_trap > > instead of __replacement_assert. > > > > Tested powerpc64le-linux. Committed to trunk. > > > ChangeLog is talking about __builtin_trap but there is none in the > attached patch. > Yes I already noticed that and mentioned it in the bugzilla PR. It uses __builtin_abort not __builtin_trap. I'll fix the ChangeLog file tomorrow after it gets generated. The Git commit message will stay wrong though.
