• Clemens Hammacher's avatar
    Remove all custom CopyCharsUnsigned implementations · 5d8c4890
    Clemens Hammacher authored
    It's unclear whether the custom implementation have any advantage over
    the standard library one's.
    Since we update our toolchain and standard library regularly, it might
    well be the case that the custom implementations are slower by now.
    
    Thus this CL removes all {CopyCharsUnsigned} implementations and
    implements {CopyChars} generically using {std::copy_n}.
    
    Note that this does not touch the {MemMove} and {MemCopy} functions
    yet, as we have seen regressions when trying to remove them before
    (https://crbug.com/v8/8675#c5).
    
    R=leszeks@chromium.org
    
    Bug: v8:9396
    Change-Id: I97a183afebcccd2fbb567bdba02e827331475608
    Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/1800577
    Commit-Queue: Clemens Hammacher <clemensh@chromium.org>
    Reviewed-by: 's avatarLeszek Swirski <leszeks@chromium.org>
    Cr-Commit-Position: refs/heads/master@{#63808}
    5d8c4890
memcopy.h 7.76 KB