Skip to content

Commit 02b0178

Browse files
committed
Don't take references in predicates
1 parent 004e561 commit 02b0178

File tree

7 files changed

+12
-12
lines changed

7 files changed

+12
-12
lines changed

libcxx/test/benchmarks/algorithms/modifying/remove_copy_if.bench.cpp

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -44,7 +44,7 @@ int main(int argc, char** argv) {
4444
for ([[maybe_unused]] auto _ : st) {
4545
benchmark::DoNotOptimize(c);
4646
benchmark::DoNotOptimize(out);
47-
auto pred = [&x](auto& element) {
47+
auto pred = [&x](auto element) {
4848
benchmark::DoNotOptimize(element);
4949
return element == x;
5050
};
@@ -85,7 +85,7 @@ int main(int argc, char** argv) {
8585
for ([[maybe_unused]] auto _ : st) {
8686
benchmark::DoNotOptimize(c);
8787
benchmark::DoNotOptimize(out);
88-
auto pred = [&](auto& element) {
88+
auto pred = [&](auto element) {
8989
benchmark::DoNotOptimize(element);
9090
return element == x;
9191
};

libcxx/test/benchmarks/algorithms/modifying/remove_if.bench.cpp

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -44,7 +44,7 @@ int main(int argc, char** argv) {
4444
std::fill_n(std::fill_n(c[i].begin(), half, x), half, y);
4545
}
4646

47-
auto pred = [&](auto& element) {
47+
auto pred = [&](auto element) {
4848
benchmark::DoNotOptimize(element);
4949
return element == x;
5050
};
@@ -102,7 +102,7 @@ int main(int argc, char** argv) {
102102
alternate(c[i].begin(), size);
103103
}
104104

105-
auto pred = [&](auto& element) {
105+
auto pred = [&](auto element) {
106106
benchmark::DoNotOptimize(element);
107107
return element == x;
108108
};

libcxx/test/benchmarks/algorithms/modifying/replace_if.bench.cpp

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -46,7 +46,7 @@ int main(int argc, char** argv) {
4646
for ([[maybe_unused]] auto _ : st) {
4747
benchmark::DoNotOptimize(c);
4848
benchmark::DoNotOptimize(z);
49-
auto pred = [&x](auto& element) {
49+
auto pred = [&x](auto element) {
5050
benchmark::DoNotOptimize(element);
5151
return element == x;
5252
};
@@ -86,7 +86,7 @@ int main(int argc, char** argv) {
8686
for ([[maybe_unused]] auto _ : st) {
8787
benchmark::DoNotOptimize(c);
8888
benchmark::DoNotOptimize(z);
89-
auto pred = [&x](auto& element) {
89+
auto pred = [&x](auto element) {
9090
benchmark::DoNotOptimize(element);
9191
return element == x;
9292
};

libcxx/test/benchmarks/algorithms/modifying/transform.binary.bench.cpp

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -38,7 +38,7 @@ int main(int argc, char** argv) {
3838

3939
std::vector<ValueType> out(size);
4040

41-
auto f = [](auto& x, auto& y) {
41+
auto f = [](auto x, auto y) {
4242
benchmark::DoNotOptimize(x);
4343
benchmark::DoNotOptimize(y);
4444
return x + y;

libcxx/test/benchmarks/algorithms/modifying/transform.unary.bench.cpp

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -35,7 +35,7 @@ int main(int argc, char** argv) {
3535

3636
std::vector<ValueType> out(size);
3737

38-
auto f = [](auto& element) {
38+
auto f = [](auto element) {
3939
benchmark::DoNotOptimize(element);
4040
return element;
4141
};

libcxx/test/benchmarks/algorithms/modifying/unique_copy_pred.bench.cpp

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -41,7 +41,7 @@ int main(int argc, char** argv) {
4141

4242
std::vector<ValueType> out(size);
4343

44-
auto pred = [](auto& a, auto& b) {
44+
auto pred = [](auto a, auto b) {
4545
benchmark::DoNotOptimize(a);
4646
benchmark::DoNotOptimize(b);
4747
return a == b;
@@ -88,7 +88,7 @@ int main(int argc, char** argv) {
8888

8989
std::vector<ValueType> out(size);
9090

91-
auto pred = [](auto& a, auto& b) {
91+
auto pred = [](auto a, auto b) {
9292
benchmark::DoNotOptimize(a);
9393
benchmark::DoNotOptimize(b);
9494
return a == b;

libcxx/test/benchmarks/algorithms/modifying/unique_pred.bench.cpp

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -44,7 +44,7 @@ int main(int argc, char** argv) {
4444
std::fill_n(std::fill_n(c[i].begin(), half, x), half, y);
4545
}
4646

47-
auto pred = [](auto& a, auto& b) {
47+
auto pred = [](auto a, auto b) {
4848
benchmark::DoNotOptimize(a);
4949
benchmark::DoNotOptimize(b);
5050
return a == b;
@@ -104,7 +104,7 @@ int main(int argc, char** argv) {
104104
alternate(c[i].begin(), size);
105105
}
106106

107-
auto pred = [](auto& a, auto& b) {
107+
auto pred = [](auto a, auto b) {
108108
benchmark::DoNotOptimize(a);
109109
benchmark::DoNotOptimize(b);
110110
return a == b;

0 commit comments

Comments
 (0)