[libc-commits] [PATCH] D136865: [libc] Improve testing of mem functions

Clement Courbet via Phabricator via libc-commits libc-commits at lists.llvm.org
Fri Oct 28 01:00:34 PDT 2022


courbet added inline comments.


================
Comment at: libc/test/src/string/bcmp_test.cpp:45
+
+TEST(LlvmLibcBcmpTest, Thorough) {
+  static constexpr size_t kSize = 1024;
----------------
`SizeSweep` or `AllSizes` would be clearer IMO (here and below).


================
Comment at: libc/test/src/string/bcmp_test.cpp:46
+TEST(LlvmLibcBcmpTest, Thorough) {
+  static constexpr size_t kSize = 1024;
+  static constexpr auto Impl = CmpAdaptor<__llvm_libc::bcmp>;
----------------
`kMaxSize` (here and below)


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D136865



More information about the libc-commits mailing list