summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorHÃ¥vard Pettersen <3535158+havardpe@users.noreply.github.com>2020-05-04 17:27:28 +0200
committerGitHub <noreply@github.com>2020-05-04 17:27:28 +0200
commit6e07be846490ce9579f29d4a5de47ca86cea430a (patch)
treeb9c6a278953933e9df648ac417d1735f12d2a199
parenta5a736466f91700bfe7bb72538463aa274ce26e3 (diff)
parent878dc5c7ae259a70c13fee9689a62e4c4ff30a87 (diff)
Merge pull request #13149 from vespa-engine/toregge/avoid-unneeded-temporaries-in-range-loop
Avoid making copies of container elements.
-rw-r--r--eval/src/vespa/eval/tensor/dense/dense_lambda_peek_optimizer.cpp4
1 files changed, 2 insertions, 2 deletions
diff --git a/eval/src/vespa/eval/tensor/dense/dense_lambda_peek_optimizer.cpp b/eval/src/vespa/eval/tensor/dense/dense_lambda_peek_optimizer.cpp
index cb42ff86fbe..54ea35e9efe 100644
--- a/eval/src/vespa/eval/tensor/dense/dense_lambda_peek_optimizer.cpp
+++ b/eval/src/vespa/eval/tensor/dense/dense_lambda_peek_optimizer.cpp
@@ -91,10 +91,10 @@ struct PeekAnalyzer {
PeekAnalyzer(const ValueType &dst_type, const ValueType &src_type,
const TensorPeek::DimList &dim_list)
{
- for (const auto dim: dst_type.dimensions()) {
+ for (const auto& dim: dst_type.dimensions()) {
dst_dim_sizes.push_back(dim.size);
}
- for (const auto dim: src_type.dimensions()) {
+ for (const auto& dim: src_type.dimensions()) {
src_dim_sizes.push_back(dim.size);
}
Node_UP idx_expr;