[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: Bignum speedup patch causes crash at startup
From: |
Eli Zaretskii |
Subject: |
Re: Bignum speedup patch causes crash at startup |
Date: |
Wed, 05 Sep 2018 18:15:50 +0300 |
> From: Paul Eggert <address@hidden>
> Date: Tue, 4 Sep 2018 13:48:10 -0700
>
> > CC data.o
> > In file included from C:/emacs/git/emacs/master/src/data.c:31:
> > C:/emacs/git/emacs/master/src/data.c: In function 'arith_driver':
> > C:/emacs/git/emacs/master/src/lisp.h:2479:34: warning: 'accum' may be used
> > uninitialized in this function [-Wmaybe-uninitialized]
> > return FIXNUM_OVERFLOW_P (n) ? make_bigint (n) : make_fixnum (n);
> > ^~~~~~~~~~~~~~~
> > C:/emacs/git/emacs/master/src/data.c:2962:12: note: 'accum' was declared
> > here
> > intmax_t accum = XFIXNUM (val);
>
> GCC complains that 'accum' might be used uninitialized even though 'accum's
> declaration has an initializer? Looks like a clear GCC bug, and might be
> worth
> reporting there.
Maybe it's because eassert can abort?
- Re: Bignum speedup patch causes crash at startup, (continued)
- Re: Bignum speedup patch causes crash at startup, Eli Zaretskii, 2018/09/04
- Re: Bignum speedup patch causes crash at startup, Andy Moreton, 2018/09/04
- Re: Bignum speedup patch causes crash at startup, Paul Eggert, 2018/09/04
- Re: Bignum speedup patch causes crash at startup, Tom Tromey, 2018/09/04
- Re: Bignum speedup patch causes crash at startup, Andy Moreton, 2018/09/04
- Re: Bignum speedup patch causes crash at startup, Paul Eggert, 2018/09/04
- Re: Bignum speedup patch causes crash at startup, Eli Zaretskii, 2018/09/05
- Re: Bignum speedup patch causes crash at startup, Eli Zaretskii, 2018/09/05
- Re: Bignum speedup patch causes crash at startup, Andy Moreton, 2018/09/04
- Re: Bignum speedup patch causes crash at startup, Paul Eggert, 2018/09/04
- Re: Bignum speedup patch causes crash at startup,
Eli Zaretskii <=
- Re: Bignum speedup patch causes crash at startup, Paul Eggert, 2018/09/05
- Re: Bignum speedup patch causes crash at startup, Pip Cet, 2018/09/05
- Re: Bignum speedup patch causes crash at startup, Paul Eggert, 2018/09/05