CBMC: Remove namespacing#895
Conversation
290ff7a to
bf34219
Compare
hanno-becker
left a comment
There was a problem hiding this comment.
Thanks @f15hr, can you also incorporate the latest changes pq-code-package/mlkem-native#1471 ?
bf34219 to
47f1c3d
Compare
91bdef8 to
d211dd8
Compare
Okay, I believe everything is sorted now. The test for SLOTHY failed because it was unable to secure an instance on aws. Everything else seems to work. |
d211dd8 to
2636ba2
Compare
hanno-becker
left a comment
There was a problem hiding this comment.
Thanks @f15hr! There's at least one function remaining where the pattern is not yet used. Can you check please?
CBMC Results (ML-DSA-44)Full Results (173 proofs)
|
CBMC Results (ML-DSA-65)
Full Results (173 proofs)
|
CBMC Results (ML-DSA-87)Full Results (173 proofs)
|
2636ba2 to
9120bf5
Compare
Ah, fixed now. |
9120bf5 to
c5093cd
Compare
mkannwischer
left a comment
There was a problem hiding this comment.
A couple functions are not yet following the format.
|
Can you try if you get the |
c5093cd to
bee3fa8
Compare
Signed-off-by: Liam Fisher <liam.louis.fisher@gmail.com>
bee3fa8 to
ed4fa4a
Compare
|
Sorry for the delay on this PR, I double checked and I believe there are no more issues, and it should be up to date with main. |
mkannwischer
left a comment
There was a problem hiding this comment.
Thanks @f15hr! LGTM.
Resolves #824
Based on the changes made by pq-code-package/mlkem-native@cdc2269. Additionally, I added the configuration file like in pq-code-package/mlkem-native@cdc2269 to more closely resemble the current state of
mlkem-native. I did not include themallocchanges.