[PATCH] Use __builtin_memmove for trivially copy assignable types

Glen Fernandes glen.fernandes@gmail.com
Tue Jun 19 01:29:00 GMT 2018


On Mon, Jun 18, 2018 at 6:12 PM Ville Voutilainen wrote:
> On 16 June 2018 at 18:05, Marc Glisse wrote:
> > On Sat, 16 Jun 2018, Glen Fernandes wrote:
> >
> >> Use __builtin_memmove for trivially copy assignable types
> >>
> >> 2018-06-14  Glen Joseph Fernandes  <glenjofe@gmail.com>
> >>
> >>        * include/bits/stl_algobase.h
> >>        (__is_simple_copy_move): Defined helper.
> >>        (__copy_move_a): Used helper.
> >>        (__copy_move_backward_a): Likewise.
> >>        * testsuite/25_algorithms/copy/58982.cc: Updated tests.
> >>        * testsuite/25_algorithms/copy_n/58982.cc: Likewise.
> >>
> >> Tested x86_64-pc-linux-gnu.
> >
> > https://gcc.gnu.org/bugzilla/show_bug.cgi?id=68350 looks related but for a
> > different function. Hopefully the issues discussed there don't apply here...
>
> I would certainly prefer this sort of patches to check via tests that
> uninitialized_{copy,move} is unaffected.

Will do.

Glen



More information about the Libstdc++ mailing list