Why is this library not expected to work on 32 bit?

[konsumlamm]

The readme states this under Current limitations and possible enhancements, but I don't see any obvious reason why this is the case. Could anyone elaborate on what exactly breaks on 32 bit platforms?

[pietroppeter]

I am assuming that is because in operations like realUnsignedAddition and multiplication you use a tmp variable that is a uint64 and I would not expect that to be available on a 32-bit architecture

[konsumlamm]

I've never heard of uint64 not being available on 32 bit platforms, it'll just be slower (since it's stored as two 32 bit values).

[pietroppeter]

You are right. That was my assumption, so maybe it works on a 32 bit system or it has another issue. What I can tell you is that I created that line in the readme starting from a todo file (now removed), saying "fix the library in 32 bit". See https://github.com/nim-lang/bigints/pull/32/files

Maybe @def- knows more about this?

[def-]

I think I added that since I found some bugs when running it on 32bit, I assume wrong results. But haven't tried in a while obviously, so not sure what the status is.

[konsumlamm]

I think I added that since I found some bugs when running it on 32bit, I assume wrong results. But haven't tried in a while obviously, so not sure what the status is.

Do you remember any specific examples that didn't work? If not, I don't think it makes much sense to keep the note about 32 bit.

[def-]

Just the internal tests I guess