[libcxx-commits] [libcxx] [libc++] Remove some std::minmax_element benchmarks (PR #210272)

[via libcxx-commits]

llvmorg-github-actions[bot] wrote:


<!--LLVM PR SUMMARY COMMENT-->

@llvm/pr-subscribers-libcxx

Author: Nikolas Klauser (philnik777)

<details>
<summary>Changes</summary>

There isn't much value in benchmarking lots of similar sizes. We can remove them to improve the time it takes to run the benchmarks.


---
Full diff: https://github.com/llvm/llvm-project/pull/210272.diff


1 Files Affected:

- (modified) libcxx/test/benchmarks/algorithms/min_max_element.bench.cpp (+1-15) 


``````````diff
diff --git a/libcxx/test/benchmarks/algorithms/min_max_element.bench.cpp b/libcxx/test/benchmarks/algorithms/min_max_element.bench.cpp
index 67a52f5d7a80a..2ad03b0ed4472 100644
--- a/libcxx/test/benchmarks/algorithms/min_max_element.bench.cpp
+++ b/libcxx/test/benchmarks/algorithms/min_max_element.bench.cpp
@@ -13,21 +13,7 @@
 
 #include <benchmark/benchmark.h>
 
-void run_sizes(benchmark::Benchmark* benchmark) {
-  benchmark->Arg(1)
-      ->Arg(2)
-      ->Arg(3)
-      ->Arg(4)
-      ->Arg(64)
-      ->Arg(512)
-      ->Arg(1024)
-      ->Arg(4000)
-      ->Arg(4096)
-      ->Arg(5500)
-      ->Arg(64000)
-      ->Arg(65536)
-      ->Arg(70000);
-}
+void run_sizes(benchmark::Benchmark* benchmark) { benchmark->Arg(2)->Arg(5500)->Arg(70000); }
 
 template <class T>
 void BM_std_minmax_element(benchmark::State& state) {

``````````

</details>


https://github.com/llvm/llvm-project/pull/210272