Skip to content

Commit ead9ab1

Browse files
committed
Remove move-ctor benchmarks accoding to ldionne's comments
1 parent f598153 commit ead9ab1

File tree

2 files changed

+0
-24
lines changed

2 files changed

+0
-24
lines changed

libcxx/test/benchmarks/containers/ContainerBenchmarks.h

Lines changed: 0 additions & 20 deletions
Original file line numberDiff line numberDiff line change
@@ -39,16 +39,6 @@ void BM_CopyConstruct(benchmark::State& st, Container) {
3939
}
4040
}
4141

42-
template <class Container>
43-
void BM_MoveConstruct(benchmark::State& st, Container) {
44-
auto size = st.range(0);
45-
Container c(size);
46-
for (auto _ : st) {
47-
auto v = std::move(c);
48-
DoNotOptimizeData(v);
49-
}
50-
}
51-
5242
template <class Container, class Allocator>
5343
void BM_CopyConstruct_Alloc(benchmark::State& st, Container, Allocator a) {
5444
auto size = st.range(0);
@@ -59,16 +49,6 @@ void BM_CopyConstruct_Alloc(benchmark::State& st, Container, Allocator a) {
5949
}
6050
}
6151

62-
template <class Container, class Allocator>
63-
void BM_MoveConstruct_Alloc(benchmark::State& st, Container, Allocator a) {
64-
auto size = st.range(0);
65-
Container c(size);
66-
for (auto _ : st) {
67-
Container v(std::move(c), a);
68-
DoNotOptimizeData(v);
69-
}
70-
}
71-
7252
template <class Container>
7353
void BM_Assignment(benchmark::State& st, Container) {
7454
auto size = st.range(0);

libcxx/test/benchmarks/containers/vector_bool_operations.bench.cpp

Lines changed: 0 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -25,12 +25,8 @@
2525
using namespace ContainerBenchmarks;
2626

2727
BENCHMARK_CAPTURE(BM_CopyConstruct, vector_bool, std::vector<bool>{})->Arg(5140480);
28-
BENCHMARK_CAPTURE(BM_MoveConstruct, vector_bool, std::vector<bool>{})->Arg(5140480);
2928
BENCHMARK_CAPTURE(
3029
BM_CopyConstruct_Alloc, vector_bool, std::vector<bool, test_allocator<bool>>(), test_allocator<bool>(3))
3130
->Arg(5140480);
32-
BENCHMARK_CAPTURE(
33-
BM_MoveConstruct_Alloc, vector_bool, std::vector<bool, test_allocator<bool>>(), test_allocator<bool>(3))
34-
->Arg(5140480);
3531

3632
BENCHMARK_MAIN();

0 commit comments

Comments
 (0)