[BUILDROBOT] Still warns on integer overflow in expression

Nick Clifton nickc@redhat.com
Fri Feb 5 13:39:00 GMT 2016


Hi Jan-Benedict,

> Even with this cast in place, gcc111 still warns on this:
> 
> /home/jbglaw/repos/binutils_gdb/gas/config/tc-microblaze.c: In function 'parse_imm':
> /home/jbglaw/repos/binutils_gdb/gas/config/tc-microblaze.c:742:21: error: integer overflow in expression [-Werror=overflow]
>    e->X_add_number |= -((offsetT) 1 << 31);

I think that this was fixed by:

commit ac0d427f4b3a2b82dcf395da74fdbc71ee598095
Author: Nick Clifton <nickc@redhat.com>
Date:   Mon Feb 1 11:36:59 2016 +0000

    Fix error building Microblaze assembler on a 32-bit host.
    
        * config/tc-microblaze.c (parse_imm): Fix compile time warning
        message extending a negative 32-bit value into a larger signed
        value on a 32-bit host.

Can you confirm ?

Cheers
  Nick




More information about the Binutils mailing list