Skip to content

use new aggregateverify #21

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 6 commits into from
Oct 31, 2025
Merged

use new aggregateverify #21

merged 6 commits into from
Oct 31, 2025

Conversation

@spiral-ladder
Copy link
Contributor

@spiral-ladder spiral-ladder commented Oct 17, 2025

No description provided.

@spiral-ladder spiral-ladder self-assigned this Oct 17, 2025
@spiral-ladder spiral-ladder mentioned this pull request Oct 17, 2025
Open
@spiral-ladder
Copy link
Contributor Author

spiral-ladder commented Oct 20, 2025 •
edited
Loading

With this PR, using zig build -Doptimize=ReleaseFast on a arm64 mac (M2 Pro):

bench/blst.bench.ts
  blst
    ✔ blst - keygen                                                        1724138 ops/s    580.0000 ns/op        -    1555879 runs   1.21 s
    ✔ other - keygen                                                      727802.0 ops/s    1.374000 us/op        -    1134618 runs   1.82 s
    ✔ blst - sign                                                         3447.004 ops/s    290.1070 us/op        -       1042 runs  0.652 s
    ✔ other - sign                                                        3470.848 ops/s    288.1140 us/op        -       1740 runs  0.799 s
    ✔ blst - verify                                                       1038.304 ops/s    963.1090 us/op        -        303 runs   1.10 s
    ✔ other - verify                                                      1509.477 ops/s    662.4810 us/op        -        294 runs   1.08 s
    ✔ blst - aggregateVerify 16                                           655.4519 ops/s    1.525665 ms/op        -         53 runs   3.02 s
    ✔ other - aggregateVerify 16                                          825.8020 ops/s    1.210944 ms/op        -        118 runs   4.19 s

verify is still slower by a lot and aggregateVerify is slower by just a bit - probably down to implementation details now.

ReleaseSafe is about the same.

@spiral-ladder spiral-ladder marked this pull request as ready for review October 31, 2025 08:35
@spiral-ladder spiral-ladder requested a review from a team as a code owner October 31, 2025 08:35
@spiral-ladder
Copy link
Contributor Author

spiral-ladder commented Oct 31, 2025

merging into #18 for easier management

@spiral-ladder spiral-ladder merged commit 69bfae9 into bing/blst-bun Oct 31, 2025
0 of 2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

@spiral-ladder spiral-ladder

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@spiral-ladder