yodalee / rsa256

An RSA 256 bits hardware implementation with verification
Other
6 stars 1 forks source link

Simplify read_port with unpack #145

Closed yodalee closed 1 year ago

johnjohnlin commented 1 year ago

While this simplies the code, for vint read, this change requires a tmp variable and might harm the performance. I am slight against this modification. How do you think? @yodalee

yodalee commented 1 year ago

I think you are right. Keep the old version for now.