-
Notifications
You must be signed in to change notification settings - Fork 158
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Support 32 bits #465
Comments
@hannesm why do you think these 2 lines might break something? It seems to me that we always try to serialise to 64b, whatever the platform is, which is the right way to have a portable binary serialization. WDYT? |
I said On 64bit, the Int64.to_int is also "unsafe": |
See #469 |
See mirage/mirage-www#562 (comment)
It seems that the recent commits to speed-up hashing broke 32 bits support. We should test this (and fix the issue, probably by removing support for
int
inIrmin.Type
)The text was updated successfully, but these errors were encountered: