cryspen / libcrux

The formally verified crypto library for Rust
https://cryspen.com/libcrux
Apache License 2.0
48 stars 8 forks source link

Make libcrux_ml_kem lax-check in F* #301

Open mamonet opened 3 weeks ago

mamonet commented 3 weeks ago

Make sure the extracted F* files of libcrux_ml_kem pass lax-check. This work is being tracked in branches dev_ml_kem_hax on libcrux and on hax.

("*" indicates that some hax changes, not yet upstreamed, are required to make the module pass)

franziskuskiefer commented 5 days ago

Blocked on some hax fixes (@W95Psp please update here with which PRs need to get merged for this) and restructuring of vector (PR incoming, please update here @karthikbhargavan).

W95Psp commented 5 days ago

Needs https://github.com/hacspec/hax/pull/726, and a fix for https://github.com/hacspec/hax/issues/719. Note I added a workaround in https://github.com/hacspec/hax/issues/719.

franziskuskiefer commented 3 days ago

What's the state here? Can we get this on CI and close it?

karthikbhargavan commented 3 days ago

There is still a bug in hax that needs to be fixed. Hopefully can be closed today or tomorrow.

karthikbhargavan commented 3 days ago

https://github.com/hacspec/hax/issues/719 is reopened

W95Psp commented 2 days ago

...and soon re-closed I hope: https://github.com/hacspec/hax/pull/738 :crossed_fingers: