[llvm] [test-suite] Document the LLVM test-suite benchmark apps (PR #105843)
Sjoerd Meijer via llvm-commits
llvm-commits at lists.llvm.org
Fri Aug 30 07:19:03 PDT 2024
sjoerdmeijer wrote:
> Seems like a good idea to me. Is it worth adding a column to state whether it's suitable for performance measurements? I guess my broader question would be whether that information is still encoded correctly in CMake too...
Thanks, good point.
I have checked that all apps are suitable for perf measurements. I.e., non of the top-level apps/directories are excluded, and only a few apps (benchmark suites) have sub-directories that are excluded when TEST_SUITE_BENCHMARKING_ONLY is not set, for example this is the pattern:
...
add_subdirectory(17-bintr)
add_subdirectory(18-imp)
if(NOT TEST_SUITE_BENCHMARKING_ONLY)
add_subdirectory(15-trie)
endif()
I have removed these benchmark from the "remark" column if it was included and spelled out to be part of the benchmark suite, and added a sentence that should clarify that all apps are suitable for perf measurements.
https://github.com/llvm/llvm-project/pull/105843
More information about the llvm-commits
mailing list