Argyle-Software / kyber

A rust implementation of the Kyber post-quantum KEM
https://docs.rs/pqc_kyber/
Apache License 2.0
163 stars 37 forks source link

use store64() for the same purpose (to store a 64bit integer as a byte array in little-endian order) #88

Closed francescomedina closed 1 year ago

francescomedina commented 1 year ago

I notice that the target line computes a simple store of a 64bit integer as a byte array in little-endian order. The store64() was created with this purpose and is already called in other functions. Since both ways produce the same output with same elaboration time (on release mode) in order to have a more readable code this contribution was introduced.

@mberry

mberry commented 1 year ago

Nice catch.