Skip to content

Initial MulVartime impls#1715

Merged
tarcieri merged 1 commit intomasterfrom
elliptic-curves/mul-vartime
Apr 10, 2026
Merged

Initial MulVartime impls#1715
tarcieri merged 1 commit intomasterfrom
elliptic-curves/mul-vartime

Conversation

@tarcieri
Copy link
Copy Markdown
Member

Companion PR to RustCrypto/traits#2379

This adds initial impls of the MulVartime trait which are required by the bounds added in the PR above.

These don't yet use variable-time implementations as noted in the TODOs, however the idea is we can opportunistically plug in wNAF when the alloc feature is enabled. However, actually implementing that has been saved for a follow-up.

This also adds an impl of the new PointWithBasepointTable to k256, which makes the table accessible in a generic context.

@tarcieri
Copy link
Copy Markdown
Member Author

Note: still need to do ed448-goldilocks

Companion PR to RustCrypto/traits#2379

This adds initial impls of the `MulVartime` trait which are required by
the bounds added in the PR above.

These don't yet use variable-time implementations as noted in the TODOs,
however the idea is we can opportunistically plug in wNAF when the
`alloc` feature is enabled. However, actually implementing that has been
saved for a follow-up.

This also adds an impl of the new `PointWithBasepointTable` to `k256`,
which makes the table accessible in a generic context.
@tarcieri tarcieri force-pushed the elliptic-curves/mul-vartime branch from d8b2dce to 5bfa226 Compare April 10, 2026 17:51
@tarcieri tarcieri changed the title [WIP] Initial MulVartime impls Initial MulVartime impls Apr 10, 2026
@tarcieri tarcieri marked this pull request as ready for review April 10, 2026 17:51
@tarcieri tarcieri merged commit a0acdeb into master Apr 10, 2026
159 checks passed
@tarcieri tarcieri deleted the elliptic-curves/mul-vartime branch April 10, 2026 19: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.

1 participant