Re: [Linaro-TCWG-CI] gcc-15-8035-g7ee31bc9276: 2 regressions on master-thumb_m23_soft_eabi
On Sun, 16 Mar 2025 at 21:54, Jonathan Wakely wrote: > > On Sun, 16 Mar 2025 at 13:16, wrote: > > > > Dear contributor, > > > > Our automatic CI has detected problems related to your patch(es). Please > > find some details below. > > > > In arm-eabi cortex-m23 soft, after: > > | commit gcc-15-8035-g7ee31bc9276 > > | Author: Jonathan Wakely > > | Date: Thu Mar 13 13:34:55 2025 + > > | > > | libstdc++: Implement for C++26 (P3370R1) > > | > > | This is the first part of the P3370R1 proposal just approved by the > > | committee in Wrocław. This adds C++ equivalents of the functions > > added > > | to C23 by WG14 N3022. > > | ... 16 lines of the commit log omitted. > > > > Produces 2 regressions: > > | > > | regressions.sum: > > | Running libstdc++:libstdc++-dg/conformance.exp ... > > | FAIL: 20_util/stdbit/1.cc -std=gnu++26 (test for excess errors) > > | UNRESOLVED: 20_util/stdbit/1.cc -std=gnu++26 compilation failed to > > produce executable > > > > Used configuration : > > *CI config* tcwg_gnu_embed_check_gcc arm-eabi -mthumb -march=armv8-m.base > > -mtune=cortex-m23 -mfloat-abi=soft -mfpu=auto > > *configure and test flags:* --target arm-eabi --disable-multilib > > --with-mode=thumb --with-cpu=cortex-m23 --with-float=soft > > --target_board=-mthumb/-march=armv8-m.base/-mtune=cortex-m23/-mfloat-abi=soft/-mfpu=auto > > qemu_cpu=cortex-m33 > > > > We track this bug report under > > https://linaro.atlassian.net/browse/GNU-1543. Please let us know if you > > have a fix. > > All the errors are of the form: > error: 'ULLONG_MAX' was not declared in this scope > but the test includes . > > So this target doesn't support long long? Or just doesn't define ULLONG_MAX? > It does... I've manually reproduced it, and ISTM the problem is __STDC_VERSION__ is not defined, as gcc/glimits.h expects: https://gcc.gnu.org/git/?p=gcc.git;a=blob;f=gcc/glimits.h;h=d5877602bf741383cfddb13236fbba1cf0b5b520;hb=HEAD#l102 Compiling == #include unsigned long long var = ULLONG_MAX; works with the same compiler, in C mode. But why would that work on arm-linux-gnueabihf and not on arm-none-eabi? Christophe > ___ > linaro-toolchain mailing list -- linaro-toolch...@lists.linaro.org > To unsubscribe send an email to linaro-toolchain-le...@lists.linaro.org
Re: [Linaro-TCWG-CI] gcc-15-8035-g7ee31bc9276: 2 regressions on master-thumb_m23_soft_eabi
On Monday, 17 March 2025, Christophe Lyon wrote: > On Sun, 16 Mar 2025 at 21:54, Jonathan Wakely wrote: >> >> On Sun, 16 Mar 2025 at 13:16, wrote: >> > >> > Dear contributor, >> > >> > Our automatic CI has detected problems related to your patch(es). Please find some details below. >> > >> > In arm-eabi cortex-m23 soft, after: >> > | commit gcc-15-8035-g7ee31bc9276 >> > | Author: Jonathan Wakely >> > | Date: Thu Mar 13 13:34:55 2025 + >> > | >> > | libstdc++: Implement for C++26 (P3370R1) >> > | >> > | This is the first part of the P3370R1 proposal just approved by the >> > | committee in Wrocław. This adds C++ equivalents of the functions added >> > | to C23 by WG14 N3022. >> > | ... 16 lines of the commit log omitted. >> > >> > Produces 2 regressions: >> > | >> > | regressions.sum: >> > | Running libstdc++:libstdc++-dg/conformance.exp ... >> > | FAIL: 20_util/stdbit/1.cc -std=gnu++26 (test for excess errors) >> > | UNRESOLVED: 20_util/stdbit/1.cc -std=gnu++26 compilation failed to produce executable >> > >> > Used configuration : >> > *CI config* tcwg_gnu_embed_check_gcc arm-eabi -mthumb -march=armv8-m.base -mtune=cortex-m23 -mfloat-abi=soft -mfpu=auto >> > *configure and test flags:* --target arm-eabi --disable-multilib --with-mode=thumb --with-cpu=cortex-m23 --with-float=soft --target_board=-mthumb/-march=armv8-m.base/-mtune=cortex-m23/-mfloat-abi=soft/-mfpu=auto qemu_cpu=cortex-m33 >> > >> > We track this bug report under https://linaro.atlassian.net/browse/GNU-1543. Please let us know if you have a fix. >> >> All the errors are of the form: >> error: 'ULLONG_MAX' was not declared in this scope >> but the test includes . >> >> So this target doesn't support long long? Or just doesn't define ULLONG_MAX? >> > > It does... > > I've manually reproduced it, and ISTM the problem is __STDC_VERSION__ > is not defined, > as gcc/glimits.h expects: > https://gcc.gnu.org/git/?p=gcc.git;a=blob;f=gcc/glimits.h;h=d5877602bf741383cfddb13236fbba1cf0b5b520;hb=HEAD#l102 Aha! Thanks. > Compiling > == > #include > unsigned long long var = ULLONG_MAX; > > works with the same compiler, in C mode. > > But why would that work on arm-linux-gnueabihf and not on arm-none-eabi? I think glimits.h use only used if libc doesn't provide one, and I guess glibc's limits.h is used for gnueabihf The C++ standard says it's implementation-defined whether __STDC_VERSION__ is defined by a C++ compiler, and if defined, it's implementation-defined what is value is GCC/glimits.h should check || (defined(__cplusplus) && __cplusplus >= 201103L)) i.e. long long should be supported for C++11 and later. Libstdc++ actually assumes long long is always supported even for C++98 so I'm surprised we've never noticed this before! I think we probably use the type without using the LLONG_MAX macro, so it just happens to work. I can adjust the test to be agnostic to that macro, but I'll also propose a patch to check __cplusplus in glimits.h Thanks again for finding the cause here. > > Christophe > >> ___ >> linaro-toolchain mailing list -- linaro-toolch...@lists.linaro.org >> To unsubscribe send an email to linaro-toolchain-le...@lists.linaro.org > >
Re: [Linaro-TCWG-CI] gcc-15-8035-g7ee31bc9276: 2 regressions on master-thumb_m23_soft_eabi
On Mon, 17 Mar 2025 at 09:53, Jonathan Wakely wrote: > > > > On Monday, 17 March 2025, Christophe Lyon wrote: > > On Sun, 16 Mar 2025 at 21:54, Jonathan Wakely wrote: > >> > >> On Sun, 16 Mar 2025 at 13:16, wrote: > >> > > >> > Dear contributor, > >> > > >> > Our automatic CI has detected problems related to your patch(es). Please > >> > find some details below. > >> > > >> > In arm-eabi cortex-m23 soft, after: > >> > | commit gcc-15-8035-g7ee31bc9276 > >> > | Author: Jonathan Wakely > >> > | Date: Thu Mar 13 13:34:55 2025 + > >> > | > >> > | libstdc++: Implement for C++26 (P3370R1) > >> > | > >> > | This is the first part of the P3370R1 proposal just approved by > >> > the > >> > | committee in Wrocław. This adds C++ equivalents of the functions > >> > added > >> > | to C23 by WG14 N3022. > >> > | ... 16 lines of the commit log omitted. > >> > > >> > Produces 2 regressions: > >> > | > >> > | regressions.sum: > >> > | Running libstdc++:libstdc++-dg/conformance.exp ... > >> > | FAIL: 20_util/stdbit/1.cc -std=gnu++26 (test for excess errors) > >> > | UNRESOLVED: 20_util/stdbit/1.cc -std=gnu++26 compilation failed to > >> > produce executable > >> > > >> > Used configuration : > >> > *CI config* tcwg_gnu_embed_check_gcc arm-eabi -mthumb > >> > -march=armv8-m.base -mtune=cortex-m23 -mfloat-abi=soft -mfpu=auto > >> > *configure and test flags:* --target arm-eabi --disable-multilib > >> > --with-mode=thumb --with-cpu=cortex-m23 --with-float=soft > >> > --target_board=-mthumb/-march=armv8-m.base/-mtune=cortex-m23/-mfloat-abi=soft/-mfpu=auto > >> > qemu_cpu=cortex-m33 > >> > > >> > We track this bug report under > >> > https://linaro.atlassian.net/browse/GNU-1543. Please let us know if you > >> > have a fix. > >> > >> All the errors are of the form: > >> error: 'ULLONG_MAX' was not declared in this scope > >> but the test includes . > >> > >> So this target doesn't support long long? Or just doesn't define > >> ULLONG_MAX? > >> > > > > It does... > > > > I've manually reproduced it, and ISTM the problem is __STDC_VERSION__ > > is not defined, > > as gcc/glimits.h expects: > > https://gcc.gnu.org/git/?p=gcc.git;a=blob;f=gcc/glimits.h;h=d5877602bf741383cfddb13236fbba1cf0b5b520;hb=HEAD#l102 > > > Aha! Thanks. > > > Compiling > > == > > #include > > unsigned long long var = ULLONG_MAX; > > > > works with the same compiler, in C mode. > > > > But why would that work on arm-linux-gnueabihf and not on arm-none-eabi? > > I think glimits.h use only used if libc doesn't provide one, and I guess > glibc's limits.h is used for gnueabihf > > The C++ standard says it's implementation-defined whether __STDC_VERSION__ is > defined by a C++ compiler, and if defined, it's implementation-defined what > is value is > > GCC/glimits.h should check || (defined(__cplusplus) && __cplusplus >= > 201103L)) > > i.e. long long should be supported for C++11 and later. > > Libstdc++ actually assumes long long is always supported even for C++98 so > I'm surprised we've never noticed this before! I think we probably use the > type without using the LLONG_MAX macro, so it just happens to work. > > I can adjust the test to be agnostic to that macro, but I'll also propose a > patch to check __cplusplus in glimits.h > > Thanks again for finding the cause here. Hmm, except that libstdc++ provides which should add ULLONG_MAX if it's not defined by libc: https://gcc.gnu.org/cgit/gcc/tree/libstdc++-v3/include/c_global/climits And should find the libstdc++ version which includes ., but we're not installing the libstdc++ version of . That's a libstdc++ bug.
Regressions on master at commit r15-8120 vs commit r15-8082 on Linux/x86_64
Regressions on master at commit r15-8120 vs commit r15-8082 on Linux/x86_64 New failures: New passes: FAIL: gfortran.dg/gomp/declare-variant-mod-1.f90 -O (test for excess errors) FAIL: gfortran.dg/gomp/declare-variant-mod-1.f90 -O (test for excess errors)
Regressions on native/releases/gcc-13 at commit r13-9433 vs commit r13-9427 on Linux/x86_64
Regressions on releases/gcc-13 at commit r13-9433 vs commit r13-9427 on Linux/x86_64 New failures: FAIL: c-c++-common/tsan/simple_race.c -O0 output pattern test FAIL: c-c++-common/tsan/simple_stack.c -O0 output pattern test FAIL: c-c++-common/tsan/simple_stack.c -O2 output pattern test FAIL: c-c++-common/tsan/thread_leak1.c -O0 output pattern test FAIL: c-c++-common/tsan/thread_leak1.c -O2 output pattern test FAIL: c-c++-common/tsan/thread_leak2.c -O0 output pattern test FAIL: c-c++-common/tsan/thread_leak2.c -O2 output pattern test FAIL: c-c++-common/tsan/thread_leak.c -O0 execution test FAIL: c-c++-common/tsan/thread_leak.c -O2 execution test FAIL: c-c++-common/tsan/tiny_race.c -O0 output pattern test FAIL: c-c++-common/tsan/tiny_race.c -O2 output pattern test FAIL: c-c++-common/tsan/tls_race.c -O0 output pattern test FAIL: c-c++-common/tsan/tls_race.c -O2 output pattern test FAIL: c-c++-common/tsan/volatile.c -O0 execution test FAIL: c-c++-common/tsan/volatile.c -O2 execution test New passes: FAIL: c-c++-common/tsan/atomic_stack.c -O0 output pattern test FAIL: c-c++-common/tsan/atomic_stack.c -O2 output pattern test FAIL: c-c++-common/tsan/bitfield_race.c -O0 output pattern test FAIL: c-c++-common/tsan/bitfield_race.c -O2 output pattern test FAIL: c-c++-common/tsan/fd_pipe_race.c -O0 output pattern test FAIL: c-c++-common/tsan/fd_pipe_race.c -O2 output pattern test FAIL: c-c++-common/tsan/free_race2.c -O0 output pattern test FAIL: c-c++-common/tsan/free_race.c -O0 output pattern test FAIL: c-c++-common/tsan/free_race.c -O2 output pattern test FAIL: c-c++-common/tsan/mutexset1.c -O2 output pattern test
Regressions on native/master at commit r15-8100 vs commit r15-8082 on Linux/x86_64
Regressions on master at commit r15-8100 vs commit r15-8082 on Linux/x86_64 New failures: New passes: FAIL: gfortran.dg/gomp/declare-variant-mod-1.f90 -O (test for excess errors) FAIL: gfortran.dg/gomp/declare-variant-mod-1.f90 -O (test for excess errors)
