On Wed, 3 Apr 2019 at 11:04, Ville Voutilainen <ville.voutilai...@gmail.com> wrote: > > On Wed, 3 Apr 2019 at 12:48, Jonathan Wakely <jwak...@redhat.com> wrote: > > > > The __glibcxx_assert macro should be used to check preconditions that > > users must meet, not to check postconditions that the implementation > > must meet. We have tests to verify std::variant meets its > > postconditions, users shouldn't pay for those checks at runtime. > > > > PR libstdc++/85184 > > * include/std/variant (_Copy_assign_base, _Move_assign_base, > > variant): > > Remove assertions. > > (variant::emplace<_Tp>): Remove result of emplace<N> directly. > > > > Tested powerpc64le-linux, committed to trunk. > > The latter changelog entry should say "Return resulf of emplace<N> > directly", not "Remove result...".
Oops! I'll fix that, thanks.
