Repository navigation
md5: add x86_64 and AArch64 asm backends - #917
sjthomason wants to merge 1 commit into
Conversation
For maintainers, the original work changed license to Apache2.0/MIT . |
Port the implementations to Rust inline assembly based on: https://github.com/animetosho/md5-optimisation
|
I think it'd be helpful if this were split up into separate PRs for AArch64 vs x86(_64). Even #447 went by the wayside because these kinds of ASM backends are hard-to-review, even when adapted from seemingly mature sources of ASM. Though while this ASM seems to have a decent rationale, I'm not finding a whole lot in the way of a test suite. Proptests which compare the Rust reference implementation with the ASM implementation would be nice. Also from what I can tell CI isn't even testing It would also be helpful if these changes were justified with benchmarks. Edit: I looked for formally verified ASM for MD5 and wasn't able to find any |
Author of https://github.com/animetosho/md5-optimisation said most of them are included to OpenSSL. It might be able to reference OpenSSL's md5. |
Does this help? |
|
That probably works, though you can look at |
Port the implementations to Rust inline assembly based on: https://github.com/animetosho/md5-optimisation
The original work was placed in the public domain: animetosho/md5-optimisation#4