commit cd11541cdfcb6d51d1d78f27d596cf7bdff346f7
author Andreas Auernhammer <aead@mail.de> Thu Apr 06 15:01:18 2017 +0200
committer Adam Langley <agl@golang.org> Thu Apr 13 00:59:54 2017 +0000
tree 3ab7cc89a085d7c215de7aa11a18e36d6753d21a
parent 9b9c1afcb078c2080cd1fac84be67797a33a32e9

curve25519: improve cswap

Simplify the constant swap function.

On amd64: Replace the CMOVQEQ scheme with SSE2 code similar to the non-amd64 code.
On non-amd64: Avoid unnecessary loop iterations.

The result is less and slightly faster code.

name 			old time/op 	new time/op 	delta
ScalarBaseMult-4   	653µs ± 0%   	636µs ± 0%   	~     (p=0.100 n=3+3)

name 			old time/op 	new time/op 	delta
ConstantSwap-4  	10.4ns ± 1%   	6.2ns ± 0%  	-39.86%  (p=0.029 n=4+4)

On an i7-65000U

Change-Id: Ia5eea92e0b3eabb6c291d25229aa582b51278552
Reviewed-on: https://go-review.googlesource.com/39693
Reviewed-by: Adam Langley <agl@golang.org>
Run-TryBot: Adam Langley <agl@golang.org>
TryBot-Result: Gobot Gobot <gobot@golang.org>