Re: isnan: slightly simplify configuration

2023-10-06 Thread Paul Eggert
On 2023-10-05 13:06, Bruno Haible wrote: Hi Paul, + if test $gl_cv_func_isnand_no_libm = yes || test "$ISNAND_LIBM"; then This doesn't look right: * If $ISNAND_LIBM is empty, it executes test '' which is invalid in at least some version of 'test'. Which version would that be

Re: [PATCH] totalorder, totalorderf, totalorderl: new modules

2023-10-06 Thread Paul Eggert
On 2023-10-02 04:33, Bruno Haible wrote: But your portable code appears to produce fairly decent x86_64 assembly code as well. Thanks for your fixes. Yes, the goal was portability rather than saving a few instructions. A high-performance implementation of totalorder would need help from the c

Re: sort and -lm

2023-10-06 Thread Bruno Haible
Paul Eggert wrote: > The win for GNU sort, once I get around to fixing totalorder's -lm > dependency Why would that be a problem? The average run time of a 'sort' invocation is long enough that an additional link dependency to libm shouldn't matter. 'sort' already links against libcrypto: $ nm s

Re: sort and -lm

2023-10-06 Thread Paul Eggert
On 2023-10-06 19:33, Bruno Haible wrote: Why would that be a problem? The average run time of a 'sort' invocation is long enough that an additional link dependency to libm shouldn't matter. We've avoided -lm in the past, I think more to avoid the dependency. If you really want to minimize the