Author casevh
Recipients casevh, h.venev, josh.r, mark.dickinson, pitrou, rhettinger, skrah, tim.peters, vstinner
Date 2014-07-12.16:11:35
SpamBayes Score -1.0
Marked as misclassified Yes
Message-id <1405181496.25.0.425167054493.issue21922@psf.upfronthosting.co.za>
In-reply-to
Content
Disclaimer: as Mark alluded to, I maintain gmpy2.

Some comments on MPIR/GMP:

For all practical purposes, building GMP on Windows requires some version of the mingw compiler toolchain. None of the performance gains of custom assembly code is available if GMP is build with the VS compiler. When compiled with mingw, GMP supports CPU detection to automatically use code optimized for the specific instruction set. This level of optimization may not be needed for Python, though.

The MPIR fork of GMP can be built with VS. Assembly code is supported via the YASM assembler plugin. Only a single instruction set is supported by the lib/dll.

gmpy2 currently uses MPIR. I've had no issues with its stability.

The mpz type has a maximum precision. IIRC, the maximum length is 2^31 bits on a 32-bit platform and 2^37 on a 64-bit platform. The mpn interface may or may not have the same restrictions. This might impact code that runs correctly, but slowly, with Python's normal PyLong implementation.

GMP does not handle out-of-memory situations gracefully. When GMP encounters a memory allocation failure (exceeding the limits above or when running our of scratch space), it will just abort. It is easy in gmpy2 to trigger an abort that will crash the Python interpreter.

My main concern is tightly linking the Python interpreter to a specific version of GMP (i.e. whatever version is used for the Windows builds or the version provided by the distribution). As long as gmpy2 can continue to use another version of GMP, it shouldn't matter to me.

GMP and MPIR are both licensed under LGPL v3+ (not v2+). I'll reserve any further licensing discussions for python-dev. 

I'll try to test the patch this weekend and that should answer some of my questions.
History
Date User Action Args
2014-07-12 16:11:36casevhsetrecipients: + casevh, tim.peters, rhettinger, mark.dickinson, pitrou, vstinner, skrah, josh.r, h.venev
2014-07-12 16:11:36casevhsetmessageid: <1405181496.25.0.425167054493.issue21922@psf.upfronthosting.co.za>
2014-07-12 16:11:36casevhlinkissue21922 messages
2014-07-12 16:11:35casevhcreate