diff options
author | Jack Lloyd <[email protected]> | 2018-03-10 11:33:25 -0500 |
---|---|---|
committer | Jack Lloyd <[email protected]> | 2018-03-10 11:33:25 -0500 |
commit | c8dd2559c28e7499d809c90b1f71e8f691d72150 (patch) | |
tree | 549f16ab57987be5ee6be50e66f41f9e326d1149 | |
parent | 6ae002ebd3a5f648860e932604b6e63ca7201011 (diff) |
Revert "Use move to avoid needless some needless copies"
This reverts commit 5185c2aaa8bf9556556e4507869042a71eaba6c0.
Clang says
warning: moving a temporary object prevents copy elision [-Wpessimizing-move]
-rw-r--r-- | src/lib/pubkey/ec_group/point_gfp.cpp | 32 |
1 files changed, 16 insertions, 16 deletions
diff --git a/src/lib/pubkey/ec_group/point_gfp.cpp b/src/lib/pubkey/ec_group/point_gfp.cpp index 8f1093418..c71a6cffe 100644 --- a/src/lib/pubkey/ec_group/point_gfp.cpp +++ b/src/lib/pubkey/ec_group/point_gfp.cpp @@ -375,22 +375,22 @@ PointGFp multi_exponentiate(const PointGFp& x, const BigInt& z1, const PointGFp y3(y2.plus(y, ws)); const PointGFp M[16] = { - x.zero(), // 0000 - x, // 0001 - x2, // 0010 - x3, // 0011 - y, // 0100 - std::move(y.plus(x, ws)), // 0101 - std::move(y.plus(x2, ws)), // 0110 - std::move(y.plus(x3, ws)), // 0111 - y2, // 1000 - std::move(y2.plus(x, ws)), // 1001 - std::move(y2.plus(x2, ws)), // 1010 - std::move(y2.plus(x3, ws)), // 1011 - y3, // 1100 - std::move(y3.plus(x, ws)), // 1101 - std::move(y3.plus(x2, ws)), // 1110 - std::move(y3.plus(x3, ws)), // 1111 + x.zero(), // 0000 + x, // 0001 + x2, // 0010 + x3, // 0011 + y, // 0100 + y.plus(x, ws), // 0101 + y.plus(x2, ws), // 0110 + y.plus(x3, ws), // 0111 + y2, // 1000 + y2.plus(x, ws), // 1001 + y2.plus(x2, ws), // 1010 + y2.plus(x3, ws), // 1011 + y3, // 1100 + y3.plus(x, ws), // 1101 + y3.plus(x2, ws), // 1110 + y3.plus(x3, ws), // 1111 }; PointGFp H = x.zero(); |