Skip to content

Conversation

@spiral-ladder
Copy link
Contributor

Packaged blst for use by zig projects in this
repo, inspired by projects in allyourcodebase.

This also decouples managing blst options into that that repo's build.zig.

@spiral-ladder spiral-ladder self-assigned this Sep 26, 2025
@spiral-ladder spiral-ladder mentioned this pull request Sep 26, 2025
Packaged `blst` for use by zig projects in this
[repo](https://github.com/chainsafe/blst.zig), inspired by projects in
[allyourcodebase](https://github.com/allyourcodebase).

This also decouples managing `blst` options into that that repo's
`build.zig`.
with:
name: ${{ matrix.settings.bun-target }}
path: 'zig-out/lib/*blst_min_pk.*'
path: 'zig-out/lib/*eth_blst.*'
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Had to update this name to match the output in build.zig.

@spiral-ladder
Copy link
Contributor Author

note that I also removed building min_sig in this PR (which is inconsequential anyway because #49 does that as well)

@github-actions
Copy link

Performance Report

✔️ no performance regression detected

Full benchmark results
Benchmark suite Current: 94ade28 Previous: null Ratio
SecretKey.fromKeygen 2.4330 us/op
SecretKey serialization 659.00 ns/op
SecretKey deserialization 605.00 ns/op
SecretKey.toPublicKey 151.63 us/op
SecretKey.sign 484.84 us/op
PublicKey serialization 593.00 ns/op
PublicKey deserialize 18.242 us/op
PublicKey deserialize and validate - 1 keys 80.954 us/op
PublicKey deserialize and validate - 100 keys 8.0138 ms/op
PublicKey deserialize and validate - 10000 keys 800.70 ms/op
Signature serialization 938.00 ns/op
Signature deserialize 36.589 us/op
Signatures deserialize and validate - 1 sets 102.59 us/op
Signatures deserialize and validate - 100 sets 10.148 ms/op
Signatures deserialize and validate - 10000 sets 1.0168 s/op
aggregatePublicKeys - 1 sets 651.00 ns/op
aggregatePublicKeys - 8 sets 9.0510 us/op
aggregatePublicKeys - 32 sets 26.347 us/op
aggregatePublicKeys - 128 sets 96.079 us/op
aggregatePublicKeys - 256 sets 198.33 us/op
aggregateSignatures - 1 sets 743.00 ns/op
aggregateSignatures - 8 sets 15.781 us/op
aggregateSignatures - 32 sets 54.931 us/op
aggregateSignatures - 128 sets 211.06 us/op
aggregateSignatures - 256 sets 427.64 us/op
aggregateWithRandomness - 1 sets 264.76 us/op
aggregateWithRandomness - 16 sets 2.7015 ms/op
aggregateWithRandomness - 128 sets 16.970 ms/op
aggregateVerify - 1 sets 1.6513 ms/op
aggregateVerify - 8 sets 3.0902 ms/op
aggregateVerify - 32 sets 8.1653 ms/op
aggregateVerify - 128 sets 29.206 ms/op
verifyMultipleAggregateSignatures - 1 sets 1.8667 ms/op
verifyMultipleAggregateSignatures - 8 sets 3.6308 ms/op
verifyMultipleAggregateSignatures - 32 sets 10.375 ms/op
verifyMultipleAggregateSignatures - 128 sets 38.289 ms/op
Same message - 1 sets 1.8841 ms/op
Same message - 8 sets 2.6193 ms/op
Same message - 32 sets 5.1120 ms/op
Same message - 128 sets 15.117 ms/op
Same message - 256 sets 28.440 ms/op

by benchmarkbot/action

@spiral-ladder spiral-ladder marked this pull request as ready for review September 26, 2025 16:11
@wemeetagain wemeetagain merged commit a36c213 into main Sep 26, 2025
14 checks passed
@wemeetagain wemeetagain deleted the bing/use-packaged-blst branch September 26, 2025 21:30
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

Successfully merging this pull request may close these issues.

3 participants