Repository navigation
#2476 Salsa20Engine.processBytes now XORs whole 64-byte keystream blo… - #2478
winne42 wants to merge 1 commit into
Conversation
Sorry, something went wrong.
…m blocks in a plain loop instead of stepping a keystream index per byte, so Salsa20, XSalsa20, ChaCha and XChaCha20 run 1.3 to 1.75 times as fast Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
|
The patch looks good, but there doesn't appear to be an AI declaration on this - it's clearly done with AI assistance though. Please see CONTRIBUTING.md |
Sorry, something went wrong.
|
@dghgit sorry about that, AI disclosure note added to both ticket and PR (should it be only on one and not the other?). I found this wording of disclosure in another PR, hope this format is OK. |
Sorry, something went wrong.
|
Thanks for the PR! Merged with minor revisions and now up on https://www.bouncycastle.org/betas |
Sorry, something went wrong.
…cks in a plain loop instead of stepping a keystream index per byte, so Salsa20, XSalsa20, ChaCha and XChaCha20 run 1.3 to 1.75 times as fast
See #2476 for detailed explanation and benchmarks.
AI Disclosure
Parts of this contribution were produced with the assistance of a generative AI tool (Claude Code),
under the direction and review of the submitter, in line with the contributing guidelines. The submitter
has reviewed and understands the code. To the best of his knowledge, the contribution does not reproduce third-party material.