diff options
author | HÃ¥vard Pettersen <3535158+havardpe@users.noreply.github.com> | 2020-05-04 17:27:28 +0200 |
---|---|---|
committer | GitHub <noreply@github.com> | 2020-05-04 17:27:28 +0200 |
commit | 6e07be846490ce9579f29d4a5de47ca86cea430a (patch) | |
tree | b9c6a278953933e9df648ac417d1735f12d2a199 | |
parent | a5a736466f91700bfe7bb72538463aa274ce26e3 (diff) | |
parent | 878dc5c7ae259a70c13fee9689a62e4c4ff30a87 (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.cpp | 4 |
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; |