small patch to suppress a warning

Stephan Pleines pleines.stephan at gmail.com
Wed Feb 10 22:55:15 UTC 2021


Can you please elaborate why it is a compiler bug?

On Wed, Feb 10, 2021 at 2:00 PM Hans Åberg <haberg-1 at telia.com> wrote:

> You do not say which compiler it is, but it looks like a clang bug.
>
>
> > On 10 Feb 2021, at 17:26, Stephan Pleines <pleines.stephan at gmail.com>
> wrote:
> >
> > Hi,
> >
> > This is a tiny patch to suppress a warning about operator precedence.
> >
> > Thank you,
> > Stephan
> >
> > diff -r 925753a1f950 mpz/pprime_p.c
> > --- a/mpz/pprime_p.c    Mon Dec 21 00:48:03 2020 +0100
> > +++ b/mpz/pprime_p.c    Wed Feb 10 08:21:36 2021 -0800
> > @@ -60,7 +60,7 @@
> >          int is_prime;
> >          unsigned long n0;
> >          n0 = mpz_get_ui (n);
> > -         is_prime = n0 & (n0 > 1) ? isprime (n0) : n0 == 2;
> > +         is_prime = (n0 & (n0 > 1)) ? isprime (n0) : n0 == 2;
> >          return is_prime ? 2 : 0;
> >        }
> >       /* Negative number.  Negate and fall out.  */
> > <patch.txt>_______________________________________________
> > gmp-bugs mailing list
> > gmp-bugs at gmplib.org
> > https://gmplib.org/mailman/listinfo/gmp-bugs
>
>


More information about the gmp-bugs mailing list