Move signature cache from CKey::Verify to CheckSig in script.cpp #1380

pull gavinandresen wants to merge 1 commits into bitcoin:master from gavinandresen:optimize2 changing 2 files +75 −69

gavinandresen commented at 6:03 PM on May 22, 2012: contributor

More than doubles the speed of verifying already-cached signatures that use compressed pubkeys: Before: ~200 microseconds After: ~80 microseconds (no caching at all: ~3,300 microseconds per signature)

Thanks to @sipa for suggesting

Move signature cache from CKey::Verify to CheckSig in script.cpp

More than doubles the speed of verifying already-cached signatures
that use compressed pubkeys:
Before: ~200 microseconds
After:  ~80 microseconds
(no caching at all: ~3,300 microseconds per signature)

Also encapsulates the signature cache code in a class
and fixes a signed/unsigned comparison warning.

acf513cfe7

jgarzik commented at 7:07 PM on May 22, 2012: contributor

ACK
sipa commented at 8:51 PM on May 22, 2012: member

ACK
gmaxwell commented at 9:35 PM on May 22, 2012: contributor

ACK.
sipa referenced this in commit f29f242758 on May 22, 2012
sipa merged this on May 22, 2012
sipa closed this on May 22, 2012
coblee referenced this in commit be92b92189 on Jul 17, 2012
lateminer referenced this in commit 05981d8984 on Jan 22, 2019
lateminer referenced this in commit a767875424 on Jan 22, 2019
DrahtBot locked this on Sep 8, 2021

Contributors