Re: GMP 6.1.2 t-count_zeros failure on ARM with assertions

2017-12-27 Thread Niels Möller
t...@gmplib.org (Torbjörn Granlund) writes: > The idea with that was to allow some usages that want to pass 0 to avoid > a condition. Isn't that used? The idea makes sense, but I couldn't find any such use. I was thinking that any use like that would be accompanied by an #ifdef

Re: GMP 6.1.2 t-count_zeros failure on ARM with assertions

2017-12-27 Thread Marco Bodrato
Ciao, Il Mer, 27 Dicembre 2017 6:10 pm, Marc Glisse ha scritto: > On Wed, 27 Dec 2017, Niels Möller wrote: >> And below, a patch to delete all mention of COUNT_LEADING_ZEROS_0, >> except for tune/many.pl, which I'm not familiar with. What do you think? > It seems a bit sad to drop this

Re: GMP 6.1.2 t-count_zeros failure on ARM with assertions

2017-12-27 Thread Marc Glisse
On Wed, 27 Dec 2017, Niels Möller wrote: ni...@lysator.liu.se (Niels Möller) writes: Pushed now. And below, a patch to delete all mention of COUNT_LEADING_ZEROS_0, except for tune/many.pl, which I'm not familiar with. What do you think? It seems a bit sad to drop this information, in case

Re: GMP 6.1.2 t-count_zeros failure on ARM with assertions

2017-12-27 Thread Torbjörn Granlund
ni...@lysator.liu.se (Niels Möller) writes: And below, a patch to delete all mention of COUNT_LEADING_ZEROS_0, except for tune/many.pl, which I'm not familiar with. What do you think? The idea with that was to allow some usages that want to pass 0 to avoid a condition. Isn't that used?

Re: GMP 6.1.2 t-count_zeros failure on ARM with assertions

2017-12-27 Thread Niels Möller
ni...@lysator.liu.se (Niels Möller) writes: > Pushed now. And below, a patch to delete all mention of COUNT_LEADING_ZEROS_0, except for tune/many.pl, which I'm not familiar with. What do you think? Regards, /Niels diff -Nprc2 gmp.0e10f8f34eb4/longlong.h gmp/longlong.h ***

Re: GMP 6.1.2 t-count_zeros failure on ARM with assertions

2017-12-27 Thread Niels Möller
Vincent Lefevre writes: >> diff -r 20cf1131dc94 longlong.h >> --- a/longlong.h Thu Aug 31 01:00:02 2017 +0200 >> +++ b/longlong.h Tue Dec 26 10:59:24 2017 +0100 >> @@ -535,7 +535,6 @@ extern UWtype __MPN(udiv_qrnnd) (UWtype >> #endif /* defined(__ARM_ARCH_2__) ...