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

feat(ci): patch ci cycle check #2046

Merged
merged 3 commits into from
Feb 7, 2025
Merged

feat(ci): patch ci cycle check #2046

merged 3 commits into from
Feb 7, 2025

Conversation

nhtyy
Copy link
Collaborator

@nhtyy nhtyy commented Feb 7, 2025

This PR adds a CI workflow to compare cycle counts if there are changes to patch-testing/*.

Note: Some tests are non deterministic in their cycle count.

@ratankaliani ratankaliani changed the title feat: patch ci cycle check feat(ci): patch ci cycle check Feb 7, 2025
Copy link
Contributor

github-actions bot commented Feb 7, 2025

Test Old New Diff
rust_crypto_rsa_test_pkcs_verify_100 174390482 174934218 -0.3118 %
curve25519_dalek_ng_test_decompressed_noncanonical 204793 204793 0.0000 %
p256_test_recover_pubkey_infinity 417746 417746 0.0000 %
curve25519_dalek_ng_test_zero_mul 197964 197964 0.0000 %
curve25519_dalek_ng_test_add_then_multiply 5250637 6791187 -29.3402 %
p256_test_verify_rand_lte_100 1202667473 1202571226 0.0080 %
bls12_381_tests_test_inverse_fp2_100 3085079 3085079 0.0000 %
bn_test_bn_test_g1_add_100 811492 811485 0.0009 %
k256_test_verify_rand_lte_100 461766339 461681433 0.0184 %
secp256k1_program_test_recover_rand_lte_100 16583422 16563990 0.1172 %
rustcrypto_bigint_test_bigint_mul_mod_special 2332628 2332628 0.0000 %
keccack_test_expected_digest_lte_100 2032597 2032926 -0.0162 %
curve25519_dalek_test_zero_msm 213931 213931 0.0000 %
curve25519_dalek_test_decompressed_expected_value 15999990 15999990 0.0000 %
p256_test_recover_rand_lte_100 65983441 65980867 0.0039 %
curve25519_dalek_test_ed25519_verify 32286281 32287184 -0.0028 %
bls12_381_tests_test_sqrt_fp_100 1106903 1106859 0.0040 %
bls12_381_tests_test_sqrt_fp2_100 1833214 1882757 -2.7025 %
bn_test_bn_test_fq_sqrt_100 989049 989049 0.0000 %
sha_test_sha2_expected_digest_lte_100_times 3771654 3763769 0.2091 %
bn_test_bn_test_g1_double_100 624788 624809 -0.0034 %
curve25519_dalek_test_decompressed_noncanonical 9263 9263 0.0000 %
bls12_381_tests_test_inverse_fp_100 1607625 1607625 0.0000 %
bls12_381_tests_test_bls_add_100 15580049 15580049 0.0000 %
bn_test_bn_test_fq_inverse_100 969449 969449 0.0000 %
curve25519_dalek_ng_test_zero_msm 217587 217587 0.0000 %
k256_test_recover_rand_lte_100 15566108 15565228 0.0057 %
curve25519_dalek_test_zero_mul 197994 197994 0.0000 %
k256_test_recover_pubkey_infinity 364703 364703 0.0000 %
p256_test_recover_high_hash_high_recid 22089439 23679699 -7.1992 %
rustcrypto_bigint_test_bigint_mul_add_residue 2250012 2250012 0.0000 %
bn_test_bn_test_fr_inverse_100 990649 990649 0.0000 %
secp256k1_program_test_verify_rand_lte_100 178901265 178813606 0.0490 %
sha_test_sha3_expected_digest_lte_100_times 1793419 1793346 0.0041 %
k256_test_recover_high_hash_high_recid 9712388 9945596 -2.4011 %
bls12_381_tests_test_bls_double_100 9670245 9670245 0.0000 %
curve25519_dalek_test_add_then_multiply 6639271 7854928 -18.3101 %

Copy link
Member

@ratankaliani ratankaliani left a comment

Choose a reason for hiding this comment

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

LGTM

@nhtyy nhtyy merged commit 29f993a into dev Feb 7, 2025
15 checks passed
@nhtyy nhtyy deleted the n/patch-ci branch February 7, 2025 21:56
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.

2 participants