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).