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

poly_compress: one counter #21

Merged
merged 1 commit into from
Feb 9, 2024
Merged

poly_compress: one counter #21

merged 1 commit into from
Feb 9, 2024

Conversation

vbgl
Copy link
Contributor

@vbgl vbgl commented Feb 9, 2024

@vbgl vbgl requested a review from tfaoliveira February 9, 2024 06:07
tfaoliveira added a commit to formosa-crypto/libjade that referenced this pull request Feb 9, 2024
@tfaoliveira
mlkem: poly.jinc update from formosa-crypto/formosa-mlkem#21
b33e80e
@tfaoliveira
Copy link
Member

from https://github.com/formosa-crypto/libjade/tree/a5aeef92124d4bd73fec3fc11fa87a74d7ce33f7 to https://github.com/formosa-crypto/libjade/tree/e9f1d694a97717987e8bb67d33fef800100082b2 , which includes PR20 and PR21, there is a small performance improvement, which is always nice:

keypair, 302260, 301876
enc    , 353350, 352560
dec    , 425106, 424598

@tfaoliveira tfaoliveira merged commit ce7753b into formosa-crypto:master Feb 9, 2024
2 checks passed
@vbgl vbgl deleted the ref-poly-compress-one-counter branch February 9, 2024 13:09
@vbgl
Copy link
Contributor Author

vbgl commented Feb 9, 2024

Apart from saving a few instructions, these change may free a few registers.

Rixxc pushed a commit to formosa-crypto/libjade that referenced this pull request Apr 17, 2024
@tfaoliveira @Rixxc
mlkem: poly.jinc update from formosa-crypto/formosa-mlkem#21
739fccf
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@tfaoliveira tfaoliveira tfaoliveira 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
@vbgl @tfaoliveira