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.

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

Faster scalar inversions for P-384 and P-521 #4213

Merged
merged 1 commit into from
Jul 15, 2024
Merged

Faster scalar inversions for P-384 and P-521 #4213

merged 1 commit into from
Jul 15, 2024

Conversation

randombit
Copy link
Owner

@randombit randombit commented Jul 15, 2024 •
edited
Loading

@coveralls
Copy link

coveralls commented Jul 15, 2024 •
edited
Loading

Coverage Status

coverage: 91.722% (+0.02%) from 91.702%
when pulling 033612a on jack/faster-pcurves-inversion-scalar
into 25356e1 on master.

reneme
reneme approved these changes Jul 15, 2024
View reviewed changes
src/lib/math/pcurves/pcurves_impl/pcurves_wrap.h Outdated Show resolved Hide resolved
@randombit randombit force-pushed the jack/faster-pcurves-inversion-scalar branch 2 times, most recently from 2594fc0 to e47d451 Compare July 15, 2024 12:42
@randombit
Add precomputed addition chains for P-384 and P-521 scalar inversion
033612a 
Saves approximately 11K cycles for P-384 and 36K cycles for P-521,
improving ECDSA signing for both curves by ~4%.

GH #4027 #1479
@randombit randombit force-pushed the jack/faster-pcurves-inversion-scalar branch from e47d451 to 033612a Compare July 15, 2024 21:19
@randombit randombit merged commit faec13a into master Jul 15, 2024
39 checks passed
@randombit randombit deleted the jack/faster-pcurves-inversion-scalar branch July 15, 2024 22:13
@randombit randombit mentioned this pull request Jul 15, 2024
Open
20 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@reneme reneme reneme 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.
3 participants
@randombit @coveralls @reneme