Re: [PATCH] Change module interface to no longer use GMP objects directly.

[Eli Zaretskii <eliz@gnu.org>]

> From: Philipp Stephani <p.stephani2@gmail.com>
> Date: Wed, 4 Dec 2019 21:31:42 +0100
> Cc: Philipp Stephani <phst@google.com>, Andreas Schwab <schwab@linux-m68k.org>,
>  Stefan Monnier <monnier@iro.umontreal.ca>,
>  Emacs developers <emacs-devel@gnu.org>
> 
> Since there were no more comments or objections, and all other points
> of discussion have been resolved, I've pushed a slightly modified
> version of the patch as 096be9c454. Except for extremely exotic
> architectures or custom GMP builds, the chosen limb should be the same
> as the default GMP limb.

Thanks.

However, this:

  #if ULONG_MAX == 0xFFFFFFFF
  typedef unsigned long long emacs_limb_t;
  # define EMACS_LIMB_MAX ULLONG_MAX
  #else
  typedef unsigned long emacs_limb_t;
  # define EMACS_LIMB_MAX ULONG_MAX
  #endif

seems to punish every 32-bit build of Emacs (and on MS-Windows even
the 64-bit builds, AFAIU).  Is there a reason for doing this?  Testing
for the native size of an 'unsigned long' data type is not
significantly more complicated than the above, and GMP goes with that
when it determines the type of mp_limb_t, AFAIK.