This was a missed optimization in the extraction of gej+ge from gej+gej.
Remove redundant secp256k1_fe_normalize from secp256k1_gej_add_ge_var. #139
pull gmaxwell wants to merge 1 commits into bitcoin-core:master from gmaxwell:redundant_normalize changing 1 files +1 −1-
gmaxwell commented at 1:12 PM on December 6, 2014: contributor
-
1c29f2eb49
Remove redundant secp256k1_fe_normalize from secp256k1_gej_add_ge_var.
This was a missed optimization in the extraction of gej+ge from gej+gej.
- gmaxwell cross-referenced this on Dec 6, 2014 from issue Variable time normalize by sipa
-
sipa commented at 3:59 PM on December 6, 2014: contributor
Maybe 0.1-0.2% speedup, but very close to sampling noise.
ACK though.
- sipa merged this on Dec 6, 2014
- sipa closed this on Dec 6, 2014
- sipa referenced this in commit 69dcaab08d on Dec 6, 2014
