[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: Bignum speedup patch causes crash at startup
From: |
Pip Cet |
Subject: |
Re: Bignum speedup patch causes crash at startup |
Date: |
Wed, 5 Sep 2018 15:50:27 +0000 |
On Wed, Sep 5, 2018 at 3:41 PM Paul Eggert <address@hidden> wrote:
> or the eassert buried inside XFIXNUM
There is no eassert inside XFIXNUM, which makes it different from
XSYMBOL etc. I think there should be; that would break the code we're
talking about here, but IMHO it's much cleaner not to call XFIXNUM on
a non-fixnum, knowing that the clever way the code is written will
prevent the meaningless integer thus achieved from ever being used. I
doubt there is a performance gain to be had from that way of writing
code, and it's very hard to read. And there might even be a
performance win to assuming XFIXNUM's argument has the right tag
value.
- Re: Bignum speedup patch causes crash at startup, (continued)
- 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, 2018/09/05
- Re: Bignum speedup patch causes crash at startup, Paul Eggert, 2018/09/05
- Re: Bignum speedup patch causes crash at startup,
Pip Cet <=
- Re: Bignum speedup patch causes crash at startup, Paul Eggert, 2018/09/05