commit 5924d3bb79e0a7bd0c7f9b5ae6ef5ef155d7e3bc
author Anthony Steinhauser <asteinhauser@google.com> Thu Sep 24 22:03:52 2020 -0700
committer Copybara-Service <copybara-worker@google.com> Thu Oct 29 11:46:32 2020 -0700
tree 53d2368798f90e4402287344e4ed1384ec0a209b
parent 4f37129e0ab257516348b55309b59da888f9df9d

[libc] Using llvm_libc memcpy in mem* benchmarks.

Currently the mem* benchmarks use memcpy from Glibc and memset from LLVM libc.
That's misleading and produces inconsistent results and behaviors.

This change makes Memcpy.cpp consistent with Memset.cpp:
https://github.com/llvm/llvm-project/blob/master/libc/benchmarks/Memset.cpp#L49

Reviewers: sivachandra

Reviewed By: sivachandra

Differential Revision: https://reviews.llvm.org/D88271

GitOrigin-RevId: ef36e8380a9109c43ef4a564083428a14a9b818a

benchmarks/Memcpy.cpp

diff --git a/benchmarks/Memcpy.cpp b/benchmarks/Memcpy.cpp
index eb30853..4c7b86c 100644
--- a/benchmarks/Memcpy.cpp
+++ b/benchmarks/Memcpy.cpp
@@ -48,8 +48,8 @@
 
   BenchmarkResult benchmark(const BenchmarkOptions &Options,
                             StringRef FunctionName, size_t Size) override {
-    FunctionPrototype Function =
-        StringSwitch<FunctionPrototype>(FunctionName).Case("memcpy", &::memcpy);
+    FunctionPrototype Function = StringSwitch<FunctionPrototype>(FunctionName)
+                                     .Case("memcpy", &__llvm_libc::memcpy);
     return llvm::libc_benchmarks::benchmark(
         Options, PP, [this, Function, Size](ParameterType p) {
           Function(DstBuffer + p.DstOffset, SrcBuffer + p.SrcOffset, Size);