HOL-Light/AArch64: Port proofs from mlkem-native#965
Merged
mkannwischer merged 2 commits intomainfrom Apr 6, 2026
Merged
Conversation
Contributor
CBMC Results (ML-DSA-65)Full Results (184 proofs)
|
Contributor
CBMC Results (ML-DSA-44)Full Results (184 proofs)
|
Contributor
CBMC Results (ML-DSA-87)Full Results (184 proofs)
|
mkannwischer
requested changes
Feb 17, 2026
Contributor
mkannwischer
left a comment
There was a problem hiding this comment.
Thanks @willieyz. Happy Near Year!
1fcede9 to
2740fdf
Compare
598a7de to
d0b8f77
Compare
d0b8f77 to
66a79da
Compare
mkannwischer
approved these changes
Apr 3, 2026
Contributor
mkannwischer
left a comment
There was a problem hiding this comment.
Thanks @willieyz! Sorry for the very long wait. I made a few minor changes to comments, but the rest looks very good.
We still need to port the safety proofs and align it with the x86_64 Keccak - but this can be one in a follow-up.
257397c to
5cbde12
Compare
This commit ports the HOL-Light correctness proof of the shared AArch64 Keccak routines from mlkem-native. Signed-off-by: willieyz <willie.zhao@chelpis.com>
This commit adds the CBMC proofs for the native AArch64 Keccak routines from mlkem-native. This shows that the native contracts fulfills our API contracts. Signed-off-by: willieyz <willie.zhao@chelpis.com>
5cbde12 to
24d5679
Compare
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
After PR #948 merged, mldsa-native and mlkem-native use identical AArch64 assembly for Keccak-F1600. Therefore, the HOL-light proofs on mlkem-native for Keccak-F1600 can also apply to mldsa-native as well.
This PR port the hol-light proof for AArch64 FIPS202 backend from mlkem-native to mldsa-native, also update to CI workflow.
The PR changes according to follwoing step:
proofs/hol_light/aarch64/, copied frommldsa/src/fips202/native/aarch64/src/and modified with reference to mlkem.keccak_f1600_*.mlandkeccak_spec.mlfrom mlkem-native to mldsa-native.keccak_f1600_*.oobjects.tests hol_light..github/workflows/hol_light.ymlto add the series of Keccak F1600 HOL-Light proofs to thehol_light_proofsfield.