On Fri, Jan 31, 2025 at 10:42:09PM +0100, Marc Glisse wrote:
> Hi,
>
> would it work for you if I replaced __int128__ with __int128, i.e. the name
> that gcc documents at
> https://gcc.gnu.org/onlinedocs/gcc/_005f_005fint128.html ? I think clang
> handles that fine.
Do you mean this[1]? If so it does work.
It has been a long time since I sent the patch, and I've forgotten about
why I chose unsigned __int128...
> On Sat, 23 Nov 2024, Yao Zi wrote:
>
> > # HG changeset patch
> > # User Yao Zi <zi...@disroot.org>
> > # Date 1732371310 -28800
> > # Sat Nov 23 22:15:10 2024 +0800
> > # Node ID e2b6d6999ac46ca8e1296343a9a7c7e2293664ff
> > # Parent 6df5dd697f5a547d8ac172efbace161914e23f78
> > longlong.h: loongarch64: replace unsigned __int128__ with __uint128_t
> >
> > Both as compiler extensions, __uint128_t is supported more widely than
> > unsigned __int128__. This change enables gmp to build with Clang 19.
> >
> > diff -r 6df5dd697f5a -r e2b6d6999ac4 longlong.h
> > --- a/longlong.h Fri Oct 18 19:16:58 2024 +0200
> > +++ b/longlong.h Sat Nov 23 22:15:10 2024 +0800
> > @@ -1162,7 +1162,7 @@
> > do {
> > \
> > UDItype __u = (u), __v = (v); \
> > (w0) = __u * __v;
> > \
> > - (w1) = (unsigned __int128__) __u * __v >> 64; \
> > + (w1) = (__uint128_t) __u * __v >> 64; \
> > } while (0)
> > #endif
> >
> >
> >
> > _______________________________________________
> > gmp-devel mailing list
> > gmp-devel@gmplib.org
> > https://gmplib.org/mailman/listinfo/gmp-devel
>
> --
> Marc Glisse
[1]:
https://github.com/eweOS/packages/blob/d421e2b9d0078786952eafbc7f2cea162883005b/fix-loongarch64-int128.patch#L10
_______________________________________________
gmp-devel mailing list
gmp-devel@gmplib.org
https://gmplib.org/mailman/listinfo/gmp-devel
