Skip to content

feat: sntrup with kem traits implemented and optimizations - #371

Open
mikelodder7 wants to merge 4 commits into
RustCrypto:masterfrom
mikelodder7:sntrup-kem
Open

feat: sntrup with kem traits implemented and optimizations#371
mikelodder7 wants to merge 4 commits into
RustCrypto:masterfrom
mikelodder7:sntrup-kem

Conversation

@mikelodder7

Copy link
Copy Markdown
Contributor

Implements kem crate traits and speed ups for sntrup.

Signed-off-by: Mike Lodder <redmike7@gmail.com>
Signed-off-by: Mike Lodder <redmike7@gmail.com>
Signed-off-by: Mike Lodder <redmike7@gmail.com>
Comment thread sntrup-kem/src/r3/vector.rs Outdated
@tarcieri

Copy link
Copy Markdown
Member

As a general comment: there's a lot going on in this PR (with over 8 klocs of additions) including quite a bit of unsafe code, although it's largely for SIMD it appears.

Signed-off-by: Mike Lodder <redmike7@gmail.com>
@mikelodder7

Copy link
Copy Markdown
Contributor Author

Yes I agree. If you want, I can break it down into smaller PRs if you think that would be helpful.

@tarcieri

Copy link
Copy Markdown
Member

That'd be great. It would be nice to have a PR with the kem traits separate from everything else at least. That would hopefully eliminate worries about unsafe.

IMO ideally each SIMD backend you add for a particular architecture would get its own PR too so they can be reviewed independent of each other.

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