bcrypto 5.0.0 fails to install on Windows about bcrypto HOT 5 CLOSED

Very strange. The 64 bit field element backends should not be built on windows: https://github.com/bcoin-org/bcrypto/blob/master/binding.gyp#L86

I wonder if the node-gyp OS variable is being set to something other than win?

from bcrypto.

Oh, it's not building the 64 bit backends. Strange that the 32 bit fiat backends require __int128. Hmm.

from bcrypto.

Okay, it seems like only unsaturated solinas primes require an __int128 type for 32 bit. I think we could maybe compile those backends using montgomery reduction instead to make them work on windows.

from bcrypto.

Strangely enough, p25519 32 bit doesn't require __int128. Investigating more.

from bcrypto.

It seems as though the shape of the prime and the number of limbs determines whether __int128 is used for 32 bit. I've tweaked the fiat build script and increased the number of limbs for those backends to avoid using __int128.

from bcrypto.