[libcxx-commits] [PATCH] D61170: Use std::move in numeric algorithms
Zoe Carver via Phabricator via libcxx-commits
libcxx-commits at lists.llvm.org
Thu Apr 25 20:27:05 PDT 2019
zoecarver marked an inline comment as done.
zoecarver added inline comments.
================
Comment at: include/numeric:277
+ __t = _VSTD::move(__t) + *__first;
+ *__result = _VSTD::move(__t);
}
----------------
Using `std::move` in places like this, while not specified in the standard (or paper), also improves performance a lot (about 7x in my tests). I think it would be good to try to find other places in numeric/algorithms where values could be moved instead of copied (because there is no downside and a lot of performance gains).
Repository:
rCXX libc++
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D61170/new/
https://reviews.llvm.org/D61170
More information about the libcxx-commits
mailing list