public inbox for gcc-bugs@sourceware.org help / color / mirror / Atom feed
* [Bug middle-end/54862] New: [4.8 Regression] error: comparison between signed and unsigned integer expressions in simplify-rtx.c @ 2012-10-09 0:18 danglin at gcc dot gnu.org 2012-10-09 7:40 ` [Bug middle-end/54862] " mikpe at it dot uu.se ` (6 more replies) 0 siblings, 7 replies; 8+ messages in thread From: danglin at gcc dot gnu.org @ 2012-10-09 0:18 UTC (permalink / raw) To: gcc-bugs http://gcc.gnu.org/bugzilla/show_bug.cgi?id=54862 Bug #: 54862 Summary: [4.8 Regression] error: comparison between signed and unsigned integer expressions in simplify-rtx.c Classification: Unclassified Product: gcc Version: 4.8.0 Status: UNCONFIRMED Severity: normal Priority: P3 Component: middle-end AssignedTo: unassigned@gcc.gnu.org ReportedBy: danglin@gcc.gnu.org CC: rsandifo@gcc.gnu.org /xxx/gnu/gcc/objdir/./prev-gcc/g++ -B/xxx/gnu/gcc/objdir/./prev-gcc/ -B/opt/gnu/ gcc/gcc-4.8/hppa2.0w-hp-hpux11.00/bin/ -nostdinc++ -B/xxx/gnu/gcc/objdir/prev-hp pa2.0w-hp-hpux11.00/libstdc++-v3/src/.libs -B/xxx/gnu/gcc/objdir/prev-hppa2.0w-h p-hpux11.00/libstdc++-v3/libsupc++/.libs -I/xxx/gnu/gcc/objdir/prev-hppa2.0w-hp- hpux11.00/libstdc++-v3/include/hppa2.0w-hp-hpux11.00 -I/xxx/gnu/gcc/objdir/prev- hppa2.0w-hp-hpux11.00/libstdc++-v3/include -I/xxx/gnu/gcc/gcc/libstdc++-v3/libsu pc++ -L/xxx/gnu/gcc/objdir/prev-hppa2.0w-hp-hpux11.00/libstdc++-v3/src/.libs -L/ xxx/gnu/gcc/objdir/prev-hppa2.0w-hp-hpux11.00/libstdc++-v3/libsupc++/.libs -c -g -O2 -DIN_GCC -fno-exceptions -fno-rtti -fasynchronous-unwind-tables -W -Wal l -Wno-narrowing -Wwrite-strings -Wcast-qual -Wmissing-format-attribute -pedanti c -Wno-long-long -Wno-variadic-macros -Wno-overlength-strings -Werror -DHAVE_C ONFIG_H -I. -I. -I../../gcc/gcc -I../../gcc/gcc/. -I../../gcc/gcc/../include -I. ./../gcc/gcc/../libcpp/include -I/opt/gnu/gcc/gmp/include -I../../gcc/gcc/../li bdecnumber -I../../gcc/gcc/../libdecnumber/dpd -I../libdecnumber -I../../gcc/gcc /../libbacktrace ../../gcc/gcc/simplify-rtx.c -o simplify-rtx.o ../../gcc/gcc/simplify-rtx.c: In function 'rtx_def* simplify_truncation(machine_ mode, rtx, machine_mode)': ../../gcc/gcc/simplify-rtx.c:671:34: error: comparison between signed and unsign ed integer expressions [-Werror=sign-compare] && INTVAL (XEXP (op, 1)) < precision) ^ ../../gcc/gcc/simplify-rtx.c:683:34: error: comparison between signed and unsign ed integer expressions [-Werror=sign-compare] && INTVAL (XEXP (op, 1)) < precision) ^ ../../gcc/gcc/simplify-rtx.c:695:34: error: comparison between signed and unsign ed integer expressions [-Werror=sign-compare] && INTVAL (XEXP (op, 1)) < precision) ^ ../../gcc/gcc/simplify-rtx.c:709:34: error: comparison between signed and unsign ed integer expressions [-Werror=sign-compare] && INTVAL (XEXP (op, 1)) < op_precision) ^ cc1plus: all warnings being treated as errors make[3]: *** [simplify-rtx.o] Error 1 ^ permalink raw reply [flat|nested] 8+ messages in thread
* [Bug middle-end/54862] [4.8 Regression] error: comparison between signed and unsigned integer expressions in simplify-rtx.c 2012-10-09 0:18 [Bug middle-end/54862] New: [4.8 Regression] error: comparison between signed and unsigned integer expressions in simplify-rtx.c danglin at gcc dot gnu.org @ 2012-10-09 7:40 ` mikpe at it dot uu.se 2012-10-09 7:41 ` mikpe at it dot uu.se ` (5 subsequent siblings) 6 siblings, 0 replies; 8+ messages in thread From: mikpe at it dot uu.se @ 2012-10-09 7:40 UTC (permalink / raw) To: gcc-bugs http://gcc.gnu.org/bugzilla/show_bug.cgi?id=54862 Mikael Pettersson <mikpe at it dot uu.se> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |mikpe at it dot uu.se --- Comment #1 from Mikael Pettersson <mikpe at it dot uu.se> 2012-10-09 07:40:08 UTC --- *** Bug 54863 has been marked as a duplicate of this bug. *** ^ permalink raw reply [flat|nested] 8+ messages in thread
* [Bug middle-end/54862] [4.8 Regression] error: comparison between signed and unsigned integer expressions in simplify-rtx.c 2012-10-09 0:18 [Bug middle-end/54862] New: [4.8 Regression] error: comparison between signed and unsigned integer expressions in simplify-rtx.c danglin at gcc dot gnu.org 2012-10-09 7:40 ` [Bug middle-end/54862] " mikpe at it dot uu.se @ 2012-10-09 7:41 ` mikpe at it dot uu.se 2012-10-09 9:58 ` rguenth at gcc dot gnu.org ` (4 subsequent siblings) 6 siblings, 0 replies; 8+ messages in thread From: mikpe at it dot uu.se @ 2012-10-09 7:41 UTC (permalink / raw) To: gcc-bugs http://gcc.gnu.org/bugzilla/show_bug.cgi?id=54862 --- Comment #2 from Mikael Pettersson <mikpe at it dot uu.se> 2012-10-09 07:41:22 UTC --- I got the exact same failure with gcc-4.8-20121007 on m68k-linux. ^ permalink raw reply [flat|nested] 8+ messages in thread
* [Bug middle-end/54862] [4.8 Regression] error: comparison between signed and unsigned integer expressions in simplify-rtx.c 2012-10-09 0:18 [Bug middle-end/54862] New: [4.8 Regression] error: comparison between signed and unsigned integer expressions in simplify-rtx.c danglin at gcc dot gnu.org 2012-10-09 7:40 ` [Bug middle-end/54862] " mikpe at it dot uu.se 2012-10-09 7:41 ` mikpe at it dot uu.se @ 2012-10-09 9:58 ` rguenth at gcc dot gnu.org 2012-10-09 11:54 ` jakub at gcc dot gnu.org ` (3 subsequent siblings) 6 siblings, 0 replies; 8+ messages in thread From: rguenth at gcc dot gnu.org @ 2012-10-09 9:58 UTC (permalink / raw) To: gcc-bugs http://gcc.gnu.org/bugzilla/show_bug.cgi?id=54862 Richard Guenther <rguenth at gcc dot gnu.org> changed: What |Removed |Added ---------------------------------------------------------------------------- Target Milestone|--- |4.8.0 ^ permalink raw reply [flat|nested] 8+ messages in thread
* [Bug middle-end/54862] [4.8 Regression] error: comparison between signed and unsigned integer expressions in simplify-rtx.c 2012-10-09 0:18 [Bug middle-end/54862] New: [4.8 Regression] error: comparison between signed and unsigned integer expressions in simplify-rtx.c danglin at gcc dot gnu.org ` (2 preceding siblings ...) 2012-10-09 9:58 ` rguenth at gcc dot gnu.org @ 2012-10-09 11:54 ` jakub at gcc dot gnu.org 2012-10-10 7:41 ` rdsandiford at googlemail dot com ` (2 subsequent siblings) 6 siblings, 0 replies; 8+ messages in thread From: jakub at gcc dot gnu.org @ 2012-10-09 11:54 UTC (permalink / raw) To: gcc-bugs http://gcc.gnu.org/bugzilla/show_bug.cgi?id=54862 Jakub Jelinek <jakub at gcc dot gnu.org> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|UNCONFIRMED |ASSIGNED Last reconfirmed| |2012-10-09 AssignedTo|unassigned at gcc dot |jakub at gcc dot gnu.org |gnu.org | Ever Confirmed|0 |1 --- Comment #3 from Jakub Jelinek <jakub at gcc dot gnu.org> 2012-10-09 11:53:57 UTC --- Created attachment 28397 --> http://gcc.gnu.org/bugzilla/attachment.cgi?id=28397 gcc48-pr54862.patch Untested fix. Guess you get the warning only for 32-bit HWI host/target combinations. We shouldn't try to optimize anything for negative shift counts, so IMHO using UINTVAL is the right thing to do. ^ permalink raw reply [flat|nested] 8+ messages in thread
* [Bug middle-end/54862] [4.8 Regression] error: comparison between signed and unsigned integer expressions in simplify-rtx.c 2012-10-09 0:18 [Bug middle-end/54862] New: [4.8 Regression] error: comparison between signed and unsigned integer expressions in simplify-rtx.c danglin at gcc dot gnu.org ` (3 preceding siblings ...) 2012-10-09 11:54 ` jakub at gcc dot gnu.org @ 2012-10-10 7:41 ` rdsandiford at googlemail dot com 2012-10-10 10:41 ` jakub at gcc dot gnu.org 2012-10-10 11:12 ` jakub at gcc dot gnu.org 6 siblings, 0 replies; 8+ messages in thread From: rdsandiford at googlemail dot com @ 2012-10-10 7:41 UTC (permalink / raw) To: gcc-bugs http://gcc.gnu.org/bugzilla/show_bug.cgi?id=54862 --- Comment #4 from rdsandiford at googlemail dot com <rdsandiford at googlemail dot com> 2012-10-10 07:41:10 UTC --- Sorry for the breakage and thanks to Jakub for the fix. I agree UINTVAL is the right way to go FWIW. ^ permalink raw reply [flat|nested] 8+ messages in thread
* [Bug middle-end/54862] [4.8 Regression] error: comparison between signed and unsigned integer expressions in simplify-rtx.c 2012-10-09 0:18 [Bug middle-end/54862] New: [4.8 Regression] error: comparison between signed and unsigned integer expressions in simplify-rtx.c danglin at gcc dot gnu.org ` (4 preceding siblings ...) 2012-10-10 7:41 ` rdsandiford at googlemail dot com @ 2012-10-10 10:41 ` jakub at gcc dot gnu.org 2012-10-10 11:12 ` jakub at gcc dot gnu.org 6 siblings, 0 replies; 8+ messages in thread From: jakub at gcc dot gnu.org @ 2012-10-10 10:41 UTC (permalink / raw) To: gcc-bugs http://gcc.gnu.org/bugzilla/show_bug.cgi?id=54862 --- Comment #5 from Jakub Jelinek <jakub at gcc dot gnu.org> 2012-10-10 10:40:42 UTC --- Author: jakub Date: Wed Oct 10 10:40:35 2012 New Revision: 192302 URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=192302 Log: PR middle-end/54862 * simplify-rtx.c (simplify_truncation): Compare UINTVAL instead of INTVAL of second argument with precision resp. op_precision. Modified: trunk/gcc/ChangeLog trunk/gcc/simplify-rtx.c ^ permalink raw reply [flat|nested] 8+ messages in thread
* [Bug middle-end/54862] [4.8 Regression] error: comparison between signed and unsigned integer expressions in simplify-rtx.c 2012-10-09 0:18 [Bug middle-end/54862] New: [4.8 Regression] error: comparison between signed and unsigned integer expressions in simplify-rtx.c danglin at gcc dot gnu.org ` (5 preceding siblings ...) 2012-10-10 10:41 ` jakub at gcc dot gnu.org @ 2012-10-10 11:12 ` jakub at gcc dot gnu.org 6 siblings, 0 replies; 8+ messages in thread From: jakub at gcc dot gnu.org @ 2012-10-10 11:12 UTC (permalink / raw) To: gcc-bugs http://gcc.gnu.org/bugzilla/show_bug.cgi?id=54862 Jakub Jelinek <jakub at gcc dot gnu.org> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|ASSIGNED |RESOLVED Resolution| |FIXED --- Comment #6 from Jakub Jelinek <jakub at gcc dot gnu.org> 2012-10-10 11:12:13 UTC --- Fixed. ^ permalink raw reply [flat|nested] 8+ messages in thread
end of thread, other threads:[~2012-10-10 11:12 UTC | newest] Thread overview: 8+ messages (download: mbox.gz / follow: Atom feed) -- links below jump to the message on this page -- 2012-10-09 0:18 [Bug middle-end/54862] New: [4.8 Regression] error: comparison between signed and unsigned integer expressions in simplify-rtx.c danglin at gcc dot gnu.org 2012-10-09 7:40 ` [Bug middle-end/54862] " mikpe at it dot uu.se 2012-10-09 7:41 ` mikpe at it dot uu.se 2012-10-09 9:58 ` rguenth at gcc dot gnu.org 2012-10-09 11:54 ` jakub at gcc dot gnu.org 2012-10-10 7:41 ` rdsandiford at googlemail dot com 2012-10-10 10:41 ` jakub at gcc dot gnu.org 2012-10-10 11:12 ` jakub at gcc dot gnu.org
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox; as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).