[libc-commits] [PATCH] D72516: [llvm-libc] Add memory function benchmarks

[Guillaume Chatelet via Phabricator via libc-commits]

gchatelet updated this revision to Diff 238462.
gchatelet marked 5 inline comments as done.
gchatelet added a comment.

- Add missing headers
- Split benchmarking targets into render and display
- Fix missing return value warning
- Make small configuration run up to 1KiB
- Change window position in graph
- Add README with quickstart
- Add missing newline
- Remove trailing _ in header guard
- Use quotes for includes
- Rename cc files to cpp
- Remove `*- C++ -*` in cpp files
- Change filename case
- Reorder include
- Fix test error
- Fix JSON comments and tests
- Fix missing comments trailing dot
- Fix enum case
- Moved NDEBUG warning to main
- Removed unneeded inline keyword
- Remove function pointer dereferencing
- Used regular EXPECT wherever possible
- Remove heading whitespace
- Format README.md to respect 80 columns
- Use SUITE instead of custom test target
- anchor `memcpy`


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D72516/new/

https://reviews.llvm.org/D72516

Files:
  libc/cmake/modules/LLVMLibCRules.cmake
  libc/utils/CMakeLists.txt
  libc/utils/HdrGen/CMakeLists.txt
  libc/utils/benchmarks/CMakeLists.txt
  libc/utils/benchmarks/JSON.cpp
  libc/utils/benchmarks/JSON.h
  libc/utils/benchmarks/JSONTest.cpp
  libc/utils/benchmarks/LibcBenchmark.cpp
  libc/utils/benchmarks/LibcBenchmark.h
  libc/utils/benchmarks/LibcBenchmarkTest.cpp
  libc/utils/benchmarks/LibcMemoryBenchmark.cpp
  libc/utils/benchmarks/LibcMemoryBenchmark.h
  libc/utils/benchmarks/LibcMemoryBenchmarkMain.cpp
  libc/utils/benchmarks/LibcMemoryBenchmarkMain.h
  libc/utils/benchmarks/LibcMemoryBenchmarkTest.cpp
  libc/utils/benchmarks/Memcmp.cpp
  libc/utils/benchmarks/Memcpy.cpp
  libc/utils/benchmarks/Memset.cpp
  libc/utils/benchmarks/README.md
  libc/utils/benchmarks/configuration_big.json
  libc/utils/benchmarks/configuration_small.json
  libc/utils/benchmarks/render.py

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D72516.238462.patch
Type: text/x-patch
Size: 82829 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/libc-commits/attachments/20200116/428f764f/attachment-0001.bin>