This is the mail archive of the
binutils@sourceware.org
mailing list for the binutils project.
Re: [BUILDROBOT] Still warns on integer overflow in expression
- From: Nick Clifton <nickc at redhat dot com>
- To: Jan-Benedict Glaw <jbglaw at lug-owl dot de>, Mike Frysinger <vapier at gentoo dot org>
- Cc: binutils at sourceware dot org
- Date: Fri, 5 Feb 2016 13:38:57 +0000
- Subject: Re: [BUILDROBOT] Still warns on integer overflow in expression
- Authentication-results: sourceware.org; auth=none
- References: <1447573743-30866-1-git-send-email-vapier at gentoo dot org> <20160124020852 dot GF15292 at lug-owl dot de>
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