Skip to content

perf: lazy initialize wrap prover #2544

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
yuwen01 merged 8 commits into from
Feb 4, 2026
Merged

perf: lazy initialize wrap prover #2544

yuwen01 merged 8 commits into from
Feb 4, 2026

Conversation

@yuwen01
Copy link
Contributor

@yuwen01 yuwen01 commented Feb 3, 2026 •
edited
Loading

No description provided.

@github-actions
Copy link
Contributor

github-actions bot commented Feb 3, 2026 •
edited
Loading

Test Old New Diff
bls12_381_tests_test_sqrt_fp2_100 1639195 1702508 3.8624 %
bn_test_bn_test_g1_add_100 986102 986137 0.0035 %
rust_crypto_rsa_test_pkcs_verify_100 29070346 28836407 -0.8047 %
sha_test_sha2_v0_9_9_expected_digest_lte_100_times 1258917 1260929 0.1598 %
curve25519_dalek_test_decompressed_noncanonical 7659 7659 0.0000 %
k256_test_recover_rand_lte_100 4418867 4417905 -0.0218 %
bls12_381_tests_test_inverse_fp_100 1434158 1434158 0.0000 %
curve25519_dalek_test_zero_msm 83451 83451 0.0000 %
curve25519_dalek_ng_test_decompressed_noncanonical 192589 192589 0.0000 %
bn_test_bn_test_g1_double_100 728595 728595 0.0000 %
curve25519_dalek_test_ed25519_verify 13271460 13270871 -0.0044 %
rustcrypto_bigint_test_bigint_mul_add_residue 1736229 1736249 0.0012 %
bls12_381_tests_test_sqrt_fp_100 955264 930975 -2.5426 %
secp256k1_program_test_verify_v0_30_0_rand_lte_100 17107458 17048282 -0.3459 %
curve25519_dalek_test_add_then_multiply 2790840 2989338 7.1125 %
bls12_381_tests_test_inverse_fp2_100 2763953 2763953 0.0000 %
p256_test_verify_rand_lte_100 11865618 11838841 -0.2257 %
sha_test_sha2_v0_10_9_expected_digest_lte_100_times 1765074 1770002 0.2792 %
bn_test_bn_test_fr_inverse_100 851612 851612 0.0000 %
sha_test_sha3_expected_digest_lte_100_times 1609170 1609086 -0.0052 %
curve25519_dalek_ng_test_add_then_multiply 4850012 4092520 -15.6184 %
curve25519_dalek_ng_test_zero_msm 125373 125373 0.0000 %
p256_test_recover_high_hash_high_recid 4904932 5219308 6.4094 %
k256_test_recover_high_hash_high_recid 2055258 2147628 4.4943 %
rustcrypto_bigint_test_bigint_mul_mod_special 1753658 1753658 0.0000 %
secp256k1_program_test_recover_rand_lte_100 5462197 5458256 -0.0722 %
secp256k1_program_test_recover_v0_30_0_rand_lte_100 5454396 5447363 -0.1289 %
sha_test_sha2_v0_10_6_expected_digest_lte_100_times 1764133 1770160 0.3416 %
bls12_381_tests_test_bls_double_100 6402021 6402021 0.0000 %
keccack_test_expected_digest_lte_100 1718377 1716190 -0.1273 %
k256_test_schnorr_verify 5668714 5675592 0.1213 %
curve25519_dalek_test_decompressed_expected_value 4262630 4503209 5.6439 %
bn_test_bn_test_fq_sqrt_100 833012 833012 0.0000 %
bls12_381_tests_test_bls_add_100 10573008 10573008 0.0000 %
k256_test_recover_pubkey_infinity 97611 97611 0.0000 %
curve25519_dalek_ng_test_zero_mul 107777 107777 0.0000 %
k256_test_verify_rand_lte_100 11842374 11847319 0.0418 %
secp256k1_program_test_verify_rand_lte_100 17098518 17103770 0.0307 %
curve25519_dalek_test_zero_mul 71794 71794 0.0000 %
sha_test_sha2_v0_10_8_expected_digest_lte_100_times 1766536 1768618 0.1179 %
p256_test_recover_rand_lte_100 15886364 15911042 0.1553 %
bn_test_bn_test_fq_inverse_100 834612 834612 0.0000 %
p256_test_recover_pubkey_infinity 101631 101631 0.0000 %

yuwen01 and others added 3 commits February 4, 2026 13:07
@yuwen01 @claude
Add CPU setup time benchmark binary to sp1-sdk
9f9a9f3 
Measures prover init time (CpuProver::new) and VK setup time separately.
Run with: cargo run -p sp1-sdk --release --features bench --bin bench_setup

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
@yuwen01
merge
7c3779f
@yuwen01
move setup perf binary
7976aa9
@yuwen01 yuwen01 merged commit 9dcc88e into multilinear_v6 Feb 4, 2026
11 checks passed
@yuwen01 yuwen01 deleted the yuwen/lazywrap branch February 4, 2026 23:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@tamirhemo tamirhemo tamirhemo approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@yuwen01 @tamirhemo