Re: crc-x86_64: undefined behaviour

2025-01-16 Thread Paul Eggert
On 2025-01-16 21:25, Jeffrey Walton wrote: On Fri, Jan 17, 2025 at 12:07 AM Bruno Haible via Gnulib discussion list wrote: Yes, the undefined behaviour really starts here, in line 35: const __m128i *data = buf; 'buf' was not aligned, 'const __m128i *' is 16-byte aligned. Disassemble the

Re: crc-x86_64: undefined behaviour

2025-01-16 Thread Jeffrey Walton
On Fri, Jan 17, 2025 at 12:07 AM Bruno Haible via Gnulib discussion list wrote: > > Paul Eggert wrote: > > > ../lib/crc-x86_64-pclmul.c:120:26: runtime error: load of misaligned > > > address 0x5572a8d5f161 for type 'const __m128i *', which requires 16 byte > > > alignment > > > 0x5572a8d5f161:

Re: crc-x86_64: undefined behaviour

2025-01-16 Thread Bruno Haible via Gnulib discussion list
Paul Eggert wrote: > > ../lib/crc-x86_64-pclmul.c:120:26: runtime error: load of misaligned > > address 0x5572a8d5f161 for type 'const __m128i *', which requires 16 byte > > alignment > > 0x5572a8d5f161: note: pointer points here > > 2a 27 00 fc 75 47 2e 58 58 bf 5a d1 0f bb b4 48 98 72 83 a

Re: crc-x86_64: undefined behaviour

2025-01-16 Thread Paul Eggert
On 2025-01-16 11:29, Bruno Haible via Gnulib discussion list wrote: ../lib/crc-x86_64-pclmul.c:120:26: runtime error: load of misaligned address 0x5572a8d5f161 for type 'const __m128i *', which requires 16 byte alignment 0x5572a8d5f161: note: pointer points here 2a 27 00 fc 75 47 2e 58 58 bf

Re: C++ problem building groff with gnulib 2025-01 on Solaris 11

2025-01-16 Thread Bruno Haible via Gnulib discussion list
G. Branden Robinson wrote: > > 1) Details about the OS: There are several variants of Solaris 11. Which > >one do you mean? > > I'm using what's installed at gcc211.fsffrance.org. > ... > g++ (GCC) 5.5.0 > ... > > 4) And, last not least, the tarball which you are attempting to compile? > > 8.

Re: C++ problem building groff with gnulib 2025-01 on Solaris 11

2025-01-16 Thread G. Branden Robinson
Hi Bruno, At 2025-01-16T21:45:44+0100, Bruno Haible wrote: > G. Branden Robinson wrote: > > [please keep groff@ in CC list, as I'm not subscribed] > > > > I'm having C++-related declaration clash trouble on Solaris 11. > > Since both header files involved in the clash are generated by > > gnulib,

Re: C++ problem building groff with gnulib 2025-01 on Solaris 11

2025-01-16 Thread Bruno Haible via Gnulib discussion list
Hi, G. Branden Robinson wrote: > [please keep groff@ in CC list, as I'm not subscribed] > > I'm having C++-related declaration clash trouble on Solaris 11. Since > both header files involved in the clash are generated by gnulib, it's > tempting not to blame the system headers. > > CXX sr

wchar_h: Fix for glibc 2.5

2025-01-16 Thread Bruno Haible via Gnulib discussion list
Building a current coreutils with a current gnulib on CentOS 5 (a glibc 2.5 system), I see these compilation warning: ../lib/vasnprintf.c:3400: warning: implicit declaration of function 'mbszero' ../src/df.c:323: warning: implicit declaration of function 'mbszero' ../src/ls.c:4586: warning: implic

crc-x86_64: undefined behaviour

2025-01-16 Thread Bruno Haible via Gnulib discussion list
Hi Sam, Testing the current coreutils with the current gnulib, there is an undefined behaviour in crc-x86_64-pclmul.c. Found by building on Ubuntu 24.04, with clang 19, CC="clang -fsanitize=address,undefined,signed-integer-overflow,shift,integer-divide-by-zero -fno-sanitize-recover=undefined"

Re: [PATCH] readutmp: work around glibc utmpx bug

2025-01-16 Thread Bruno Haible via Gnulib discussion list
Hi Pádraig, > A small issue with the readutmp test is it fails on systems with an uptime >= > 5 years. > cfarm135 is such a system currently, and the test fails like: > > FAIL: test-readutmp > === > Here are the read_utmp results. > Flags: B = Boot, U = User Process > Time (

Re: [PATCH] readutmp: work around glibc utmpx bug

2025-01-16 Thread Pádraig Brady
On 30/07/2023 14:21, Bruno Haible wrote: Paul Eggert wrote: +static void +copy_utmp_entry (STRUCT_UTMP *dst, STRUCT_UTMP *src) +{ +#if __GLIBC__ && _TIME_BITS == 64 + /* Convert from external form in SRC to internal form in DST. + It is OK to convert now, rather than earlier, before + d