Skip to content

chore: try arkworks-bls12-381 benchmarks#2329

Closed
kevaundray wants to merge 12 commits intobluealloy:mainfrom
kevaundray:kw/run-arkworks-on-bls-benchmarks
Closed

chore: try arkworks-bls12-381 benchmarks#2329
kevaundray wants to merge 12 commits intobluealloy:mainfrom
kevaundray:kw/run-arkworks-on-bls-benchmarks

Conversation

@kevaundray
Copy link
Contributor

@kevaundray kevaundray commented Mar 27, 2025

This is just a PR to compare arkworks to blst.

It is based off of #2327

Comment on lines -3 to +11
cfg_if::cfg_if! {
if #[cfg(feature = "blst")]{
mod blst;
use blst as crypto_backend;
} else {
mod arkworks;
use arkworks as crypto_backend;
}
}
// cfg_if::cfg_if! {
// if #[cfg(feature = "blst")]{
// mod blst;
// use blst as crypto_backend;
// } else {
// }
// }
mod arkworks;
use arkworks as crypto_backend;
Copy link
Contributor Author

Choose a reason for hiding this comment

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

main hack to essentially always enable arkworks

@codspeed-hq
Copy link

codspeed-hq bot commented Mar 27, 2025

CodSpeed Performance Report

Merging #2329 will not alter performance

Comparing kevaundray:kw/run-arkworks-on-bls-benchmarks (9b8ebcf) with main (d79ff44)

Summary

✅ 9 untouched benchmarks
🆕 16 new benchmarks

Benchmarks breakdown

Benchmark BASE HEAD Change
🆕 g1_add N/A 47.5 µs N/A
🆕 g1_msm (size 1) N/A 990.9 µs N/A
🆕 g1_msm (size 128) N/A 50.3 ms N/A
🆕 g1_msm (size 2) N/A 2.2 ms N/A
🆕 g1_msm (size 256) N/A 94.9 ms N/A
🆕 g2_add N/A 63 µs N/A
🆕 g2_msm (size 1) N/A 2.6 ms N/A
🆕 g2_msm (size 128) N/A 106.6 ms N/A
🆕 g2_msm (size 2) N/A 5.5 ms N/A
🆕 g2_msm (size 256) N/A 195.3 ms N/A
🆕 map_fp2_to_g2 N/A 1.4 ms N/A
🆕 map_fp_to_g1 N/A 553.5 µs N/A
🆕 pairing (1 pairs) N/A 5.4 ms N/A
🆕 pairing (16 pairs) N/A 36.8 ms N/A
🆕 pairing (2 pairs) N/A 7.3 ms N/A
🆕 pairing (8 pairs) N/A 19.8 ms N/A

@kevaundray
Copy link
Contributor Author

Closing since we have checked against arkworks

@kevaundray kevaundray closed this Mar 27, 2025
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.

1 participant