[libc-commits] [PATCH] D138111: [libc][benchmark] Fix wrong BatchSize leading to data not fitting in L1.

Guillaume Chatelet via Phabricator via libc-commits libc-commits at lists.llvm.org
Wed Nov 16 03:06:22 PST 2022


gchatelet added inline comments.


================
Comment at: libc/benchmarks/LibcMemoryBenchmark.cpp:85
     : BufferSize(getAvailableBufferSize() / BufferCount),
-      BatchSize(BufferSize / sizeof(ParameterType)), Parameters(BatchSize) {
+      BatchSize(ParameterStorageBytes / sizeof(ParameterType)),
+      Parameters(BatchSize) {
----------------
The error is here. It always should have been `ParameterStorageBytes` but we were using `BufferSize`.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D138111



More information about the libc-commits mailing list