Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Need some performance banchmark. #2

Open
jerry-sl opened this issue Dec 2, 2017 · 1 comment
Open

Need some performance banchmark. #2

jerry-sl opened this issue Dec 2, 2017 · 1 comment

Comments

@jerry-sl
Copy link

jerry-sl commented Dec 2, 2017

Hi, sorpaas.
Thx for this pure rust secp256k1 lib. I wonder there has some performance test or has some work in process?

I wrote a demo to benchmark sign, verify and recover. But the result is not good than c-binding lib. May be i am test the wrong way. If you have any time to review my code and give some advice. I was glad to fix and improve my test. Thx.

@sorpaas
Copy link
Collaborator

sorpaas commented Dec 2, 2017

Hi. Thanks! I'll take a look next week on this.

What's the C version of secp256k1 benchmark you're comparing against? Depending on how you compiled the C version, it might give out different results. For example, in Rust we don't yet have assembly optimizations to keep it pure no_std and capable to compile to WebAssembly.

Right now it might be fair right now to compare with the C version compiled with the following config flags:

  • USE_NUM_NONE
  • USE_FIELD_INV_BUILTIN
  • USE_SCALAR_INV_BUILTIN
  • USE_FIELD_10X26
  • USE_SCALAR_8X32
  • #undef USE_ASM_X86_64
  • #undef USE_ENDOMORPHISM

tomtau pushed a commit to crypto-com/libsecp256k1-rs that referenced this issue Mar 8, 2019
bench PublicKey and Signature parsing and serialization
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants