[PATCH] Use __builtin_memmove for trivially copy assignable types

Jonathan Wakely jwakely@redhat.com
Thu Jul 19 18:59:00 GMT 2018


On 19/07/18 11:04 -0400, Glen Fernandes wrote:
>On Thu, Jul 19, 2018 at 10:40 AM Jonathan Wakely <jwakely@redhat.com> wrote:
>> On 19/07/18 10:32 -0400, Glen Fernandes wrote:
>> >Attached: patch.txt
>> >Use __builtin_memmove for trivially copyable types
>> >2018-07-19  Glen Joseph Fernandes  <glenjofe@gmail.com>
>> >    * include/bits/stl_algobase.h
>> >    (__copy_move_a): Used __is_trivially_copyable.
>> >    (__copy_move_backward_a): Likewise.
>> >Tested x86_64-pc-linux-gnu.
>>
>> Ah, that was quick :-)
>>
>> Can we keep the new test you added in the previous patch? It seems
>> useful to add anyway.
>
>Affirmative. Attached: patch.txt
>
>Use __builtin_memmove for trivially copyable types
>
>2018-07-19  Glen Joseph Fernandes  <glenjofe@gmail.com>
>
>    * include/bits/stl_algobase.h
>    (__copy_move_a): Used __is_trivially_copyable.
>    (__copy_move_backward_a): Likewise.
>    * testsuite/20_util/specialized_algorithms/uninitialized_copy/1.cc:
>    New test.

Tested on x86_64-linux and committed to trunk - thanks!




More information about the Libstdc++ mailing list