[Bug gas/19880] New: gas/config/tc-arm.c:12519 error: comparison between signed and unsigned integer expressions

2016-03-29 Thread kieranbingham at gmail dot com
https://sourceware.org/bugzilla/show_bug.cgi?id=19880

Bug ID: 19880
   Summary: gas/config/tc-arm.c:12519 error: comparison between
signed and unsigned integer expressions
   Product: binutils
   Version: unspecified
Status: NEW
  Severity: normal
  Priority: P2
 Component: gas
  Assignee: unassigned at sourceware dot org
  Reporter: kieranbingham at gmail dot com
  Target Milestone: ---

This error fires on origin/master only if the optimiser is set to -O0


gcc -DHAVE_CONFIG_H -I.
-I/home/linuxembedded/linaro/lkd/openst-lkd/sources/binutils-gdb/gas  -I.
-I/home/linuxembedded/linaro/lkd/openst-lkd/sources/binutils-gdb/gas -I../bfd
-I/home/linuxembedded/linaro/lkd/openst-lkd/sources/binutils-gdb/gas/config
-I/home/linuxembedded/linaro/lkd/openst-lkd/sources/binutils-gdb/gas/../include
-I/home/linuxembedded/linaro/lkd/openst-lkd/sources/binutils-gdb/gas/..
-I/home/linuxembedded/linaro/lkd/openst-lkd/sources/binutils-gdb/gas/../bfd
-DLOCALEDIR="\"/usr/share/locale\""  -W -Wall -Wstrict-prototypes
-Wmissing-prototypes -Wshadow -Wstack-usage=262144 -Werror
-I/home/linuxembedded/linaro/lkd/openst-lkd/sources/binutils-gdb/gas/../zlib
-g3 -O0 -fsanitize=undefined -MT tc-arm.o -MD -MP -MF .deps/tc-arm.Tpo -c -o
tc-arm.o `test -f 'config/tc-arm.c' || echo
'/home/linuxembedded/linaro/lkd/openst-lkd/sources/binutils-gdb/gas/'`config/tc-arm.c
/home/linuxembedded/linaro/lkd/openst-lkd/sources/binutils-gdb/gas/config/tc-arm.c:
In function ‘do_t_push_pop’:
/home/linuxembedded/linaro/lkd/openst-lkd/sources/binutils-gdb/gas/config/tc-arm.c:12519:23:
error: comparison between signed and unsigned integer expressions
[-Werror=sign-compare]
 && (mask & ~0xff) == (1 << (inst.instruction == T_MNEM_push
   ^
cc1: all warnings being treated as errors
Makefile:972: recipe for target 'tc-arm.o' failed
make[5]: *** [tc-arm.o] Error 1

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
bug-binutils mailing list
bug-binutils@gnu.org
https://lists.gnu.org/mailman/listinfo/bug-binutils


[Bug gas/19880] gas/config/tc-arm.c:12519 error: comparison between signed and unsigned integer expressions

2016-03-30 Thread kieranbingham at gmail dot com
https://sourceware.org/bugzilla/show_bug.cgi?id=19880

Kieran Bingham  changed:

   What|Removed |Added

 CC||kieranbingham at gmail dot com

--- Comment #2 from Kieran Bingham  ---
Comment on attachment 9143
  --> https://sourceware.org/bugzilla/attachment.cgi?id=9143
Possible fix

Yes, that fixes it. I fixed it locally with an (unsigned) cast but your fix is
shorter, simpler and better :)

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
bug-binutils mailing list
bug-binutils@gnu.org
https://lists.gnu.org/mailman/listinfo/bug-binutils