☠ Buildbot (GNU Toolchain): binutils-gdb - failed update (failure) (master)

[Mark Wielaard]

Hi,

On Thu, 2022-05-19 at 10:58 +0000, builder--- via Binutils wrote:
> A new failure has been detected on builder binutils-fedora-ppc64le
> while building binutils-gdb.
> 
> Full details are available at:
>     https://builder.sourceware.org/buildbot/#builders/86/builds/36
> 
> Build state: failed update (failure)
> Revision: (unknown)
> Worker: fedora-ppc64le
> Build Reason: (unknown)
> Blamelist: Jan Beulich <jbeulich@suse.com>
> 
> Steps:
> 
> - 0: worker_preparation ( success )
> 
> - 1: git checkout ( failure )
>     Logs:
>         - stdio: 
> https://builder.sourceware.org/buildbot/#builders/86/builds/36/steps/1/logs/stdio

Sorry about that. That was obviously just a network/git transport
issue. I have restarted that build and it seems fine now.
https://builder.sourceware.org/buildbot/#/builders/86

Note that the builder did find a real issue with:

https://sourceware.org/git/?p=binutils-gdb.git;a=commitdiff;h=3363d8560ff5229b724827a77b533ec8b2a23729

commit 3363d8560ff5229b724827a77b533ec8b2a23729
Author: Victor Do Nascimento <victor.donascimento@arm.com>
Date:   Wed May 18 16:25:12 2022 +0100

    arm: Add unwind support for mixed register lists
    
            * config/tc-arm.c (parse_reg_list): Add handling of mixed register
            types.
            (reg_names): Enumerate pseudoregister according to mapped physical
            register number.
            (s_arm_unwind_save_pseudo): Modify function signature.
            (s_arm_unwind_save_core): Likewise.
            (s_arm_unwind_save_mixed): New function.
            (s_arm_unwind_save): Generate register list mask to pass to nested
            functions.
            * testsuite/gas/arm/unwind-pacbti-m.s: Expand test for mixed
            register type lists.
            * testsuite/gas/arm/unwind-pacbti-m.d: Likewise.
            * testsuite/gas/arm/unwind-pacbti-m-readelf.d: Likewise.

https://builder.sourceware.org/buildbot/#/changes/1165

https://builder.sourceware.org/buildbot/#/builders/80/builds/68

../../binutils-gdb/gas/config/tc-arm.c: In function ‘s_arm_unwind_save_mixed’:
../../binutils-gdb/gas/config/tc-arm.c:4730:17: error: comparison of integer expressions of different signedness: ‘unsigned int’ and ‘long int’ [-Werror=sign-compare]
   for (; lim_hi <= roof; lim_hi++)
                 ^~
cc1: all warnings being treated as errors
make[3]: *** [Makefile:1423: config/tc-arm.o] Error 1

But didn't report it through email because armhf was already failing
the testsuite, so the buildbot didn't believe this was a new failure.

Cheers,

Mark