public inbox for gcc-bugs@sourceware.org
help / color / mirror / Atom feed
* [Bug c/58385] New: likely wrong code bug
@ 2013-09-10 17:11 regehr at cs dot utah.edu
2013-09-10 18:06 ` [Bug middle-end/58385] " glisse at gcc dot gnu.org
` (6 more replies)
0 siblings, 7 replies; 8+ messages in thread
From: regehr at cs dot utah.edu @ 2013-09-10 17:11 UTC (permalink / raw)
To: gcc-bugs
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=58385
Bug ID: 58385
Summary: likely wrong code bug
Product: gcc
Version: 4.9.0
Status: UNCONFIRMED
Severity: normal
Priority: P3
Component: c
Assignee: unassigned at gcc dot gnu.org
Reporter: regehr at cs dot utah.edu
CC: chenyang at cs dot utah.edu
regehr@john-home ~/z/reduce/r112 $ gcc -O0 small.c ; ./a.out
0
regehr@john-home ~/z/reduce/r112 $ gcc -O1 small.c ; ./a.out
1
regehr@john-home ~/z/reduce/r112 $ cat small.c
int printf(const char *, ...);
int x0, x1 = 1;
int x2() {
x1 = 0;
return 0;
}
int main() {
((0 || x0) & x2() >= 0) <= 1 && 1;
printf("%d\n", x1);
return 0;
}
regehr@john-home ~/z/reduce/r112 $ gcc -v
Using built-in specs.
COLLECT_GCC=gcc
COLLECT_LTO_WRAPPER=/home/regehr/z/compiler-install/gcc-r202470-install/libexec/gcc/x86_64-unknown-linux-gnu/4.9.0/lto-wrapper
Target: x86_64-unknown-linux-gnu
Configured with: /home/regehr/z/compiler-source/gcc/configure
--prefix=/home/regehr/z/compiler-install/gcc-r202470-install
--enable-languages=c,c++ --disable-multilib
Thread model: posix
gcc version 4.9.0 20130910 (experimental) (GCC)
^ permalink raw reply [flat|nested] 8+ messages in thread
* [Bug middle-end/58385] likely wrong code bug
2013-09-10 17:11 [Bug c/58385] New: likely wrong code bug regehr at cs dot utah.edu
@ 2013-09-10 18:06 ` glisse at gcc dot gnu.org
2013-09-10 18:33 ` [Bug middle-end/58385] [4.7/4.8/4.9 Regression] " jakub at gcc dot gnu.org
` (5 subsequent siblings)
6 siblings, 0 replies; 8+ messages in thread
From: glisse at gcc dot gnu.org @ 2013-09-10 18:06 UTC (permalink / raw)
To: gcc-bugs
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=58385
Marc Glisse <glisse at gcc dot gnu.org> changed:
What |Removed |Added
----------------------------------------------------------------------------
Status|UNCONFIRMED |NEW
Last reconfirmed| |2013-09-10
Ever confirmed|0 |1
--- Comment #1 from Marc Glisse <glisse at gcc dot gnu.org> ---
Confirmed, already in 4.6. Somewhere in fold_range_test we forget to check for
side effects.
^ permalink raw reply [flat|nested] 8+ messages in thread
* [Bug middle-end/58385] [4.7/4.8/4.9 Regression] likely wrong code bug
2013-09-10 17:11 [Bug c/58385] New: likely wrong code bug regehr at cs dot utah.edu
2013-09-10 18:06 ` [Bug middle-end/58385] " glisse at gcc dot gnu.org
@ 2013-09-10 18:33 ` jakub at gcc dot gnu.org
2013-09-10 18:55 ` jakub at gcc dot gnu.org
` (4 subsequent siblings)
6 siblings, 0 replies; 8+ messages in thread
From: jakub at gcc dot gnu.org @ 2013-09-10 18:33 UTC (permalink / raw)
To: gcc-bugs
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=58385
Jakub Jelinek <jakub at gcc dot gnu.org> changed:
What |Removed |Added
----------------------------------------------------------------------------
CC| |jakub at gcc dot gnu.org
Target Milestone|--- |4.7.4
Summary|likely wrong code bug |[4.7/4.8/4.9 Regression]
| |likely wrong code bug
--- Comment #2 from Jakub Jelinek <jakub at gcc dot gnu.org> ---
Started with r145254.
^ permalink raw reply [flat|nested] 8+ messages in thread
* [Bug middle-end/58385] [4.7/4.8/4.9 Regression] likely wrong code bug
2013-09-10 17:11 [Bug c/58385] New: likely wrong code bug regehr at cs dot utah.edu
2013-09-10 18:06 ` [Bug middle-end/58385] " glisse at gcc dot gnu.org
2013-09-10 18:33 ` [Bug middle-end/58385] [4.7/4.8/4.9 Regression] " jakub at gcc dot gnu.org
@ 2013-09-10 18:55 ` jakub at gcc dot gnu.org
2013-09-11 11:35 ` jakub at gcc dot gnu.org
` (3 subsequent siblings)
6 siblings, 0 replies; 8+ messages in thread
From: jakub at gcc dot gnu.org @ 2013-09-10 18:55 UTC (permalink / raw)
To: gcc-bugs
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=58385
Jakub Jelinek <jakub at gcc dot gnu.org> changed:
What |Removed |Added
----------------------------------------------------------------------------
Status|NEW |ASSIGNED
Assignee|unassigned at gcc dot gnu.org |jakub at gcc dot gnu.org
--- Comment #3 from Jakub Jelinek <jakub at gcc dot gnu.org> ---
Created attachment 30795
--> http://gcc.gnu.org/bugzilla/attachment.cgi?id=30795&action=edit
gcc49-pr58385.patch
Untested fix.
^ permalink raw reply [flat|nested] 8+ messages in thread
* [Bug middle-end/58385] [4.7/4.8/4.9 Regression] likely wrong code bug
2013-09-10 17:11 [Bug c/58385] New: likely wrong code bug regehr at cs dot utah.edu
` (2 preceding siblings ...)
2013-09-10 18:55 ` jakub at gcc dot gnu.org
@ 2013-09-11 11:35 ` jakub at gcc dot gnu.org
2013-09-11 12:17 ` jakub at gcc dot gnu.org
` (2 subsequent siblings)
6 siblings, 0 replies; 8+ messages in thread
From: jakub at gcc dot gnu.org @ 2013-09-11 11:35 UTC (permalink / raw)
To: gcc-bugs
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=58385
--- Comment #4 from Jakub Jelinek <jakub at gcc dot gnu.org> ---
Author: jakub
Date: Wed Sep 11 11:35:51 2013
New Revision: 202494
URL: http://gcc.gnu.org/viewcvs?rev=202494&root=gcc&view=rev
Log:
PR tree-optimization/58385
* fold-const.c (build_range_check): If both low and high are NULL,
use omit_one_operand_loc to preserve exp side-effects.
* gcc.c-torture/execute/pr58385.c: New test.
Added:
trunk/gcc/testsuite/gcc.c-torture/execute/pr58385.c
Modified:
trunk/gcc/ChangeLog
trunk/gcc/fold-const.c
trunk/gcc/testsuite/ChangeLog
^ permalink raw reply [flat|nested] 8+ messages in thread
* [Bug middle-end/58385] [4.7/4.8/4.9 Regression] likely wrong code bug
2013-09-10 17:11 [Bug c/58385] New: likely wrong code bug regehr at cs dot utah.edu
` (3 preceding siblings ...)
2013-09-11 11:35 ` jakub at gcc dot gnu.org
@ 2013-09-11 12:17 ` jakub at gcc dot gnu.org
2013-09-11 12:19 ` [Bug middle-end/58385] [4.7 " jakub at gcc dot gnu.org
2014-06-12 13:29 ` rguenth at gcc dot gnu.org
6 siblings, 0 replies; 8+ messages in thread
From: jakub at gcc dot gnu.org @ 2013-09-11 12:17 UTC (permalink / raw)
To: gcc-bugs
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=58385
--- Comment #5 from Jakub Jelinek <jakub at gcc dot gnu.org> ---
Author: jakub
Date: Wed Sep 11 12:17:47 2013
New Revision: 202495
URL: http://gcc.gnu.org/viewcvs?rev=202495&root=gcc&view=rev
Log:
PR tree-optimization/58385
* fold-const.c (build_range_check): If both low and high are NULL,
use omit_one_operand_loc to preserve exp side-effects.
* gcc.c-torture/execute/pr58385.c: New test.
Added:
branches/gcc-4_8-branch/gcc/testsuite/gcc.c-torture/execute/pr58385.c
Modified:
branches/gcc-4_8-branch/gcc/ChangeLog
branches/gcc-4_8-branch/gcc/fold-const.c
branches/gcc-4_8-branch/gcc/testsuite/ChangeLog
^ permalink raw reply [flat|nested] 8+ messages in thread
* [Bug middle-end/58385] [4.7 Regression] likely wrong code bug
2013-09-10 17:11 [Bug c/58385] New: likely wrong code bug regehr at cs dot utah.edu
` (4 preceding siblings ...)
2013-09-11 12:17 ` jakub at gcc dot gnu.org
@ 2013-09-11 12:19 ` jakub at gcc dot gnu.org
2014-06-12 13:29 ` rguenth at gcc dot gnu.org
6 siblings, 0 replies; 8+ messages in thread
From: jakub at gcc dot gnu.org @ 2013-09-11 12:19 UTC (permalink / raw)
To: gcc-bugs
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=58385
Jakub Jelinek <jakub at gcc dot gnu.org> changed:
What |Removed |Added
----------------------------------------------------------------------------
Summary|[4.7/4.8/4.9 Regression] |[4.7 Regression] likely
|likely wrong code bug |wrong code bug
--- Comment #6 from Jakub Jelinek <jakub at gcc dot gnu.org> ---
Fixed for 4.8.2+ so far.
^ permalink raw reply [flat|nested] 8+ messages in thread
* [Bug middle-end/58385] [4.7 Regression] likely wrong code bug
2013-09-10 17:11 [Bug c/58385] New: likely wrong code bug regehr at cs dot utah.edu
` (5 preceding siblings ...)
2013-09-11 12:19 ` [Bug middle-end/58385] [4.7 " jakub at gcc dot gnu.org
@ 2014-06-12 13:29 ` rguenth at gcc dot gnu.org
6 siblings, 0 replies; 8+ messages in thread
From: rguenth at gcc dot gnu.org @ 2014-06-12 13:29 UTC (permalink / raw)
To: gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=58385
Richard Biener <rguenth at gcc dot gnu.org> changed:
What |Removed |Added
----------------------------------------------------------------------------
Status|ASSIGNED |RESOLVED
Known to work| |4.8.2
Resolution|--- |FIXED
Target Milestone|4.7.4 |4.8.2
Known to fail| |4.7.4
--- Comment #7 from Richard Biener <rguenth at gcc dot gnu.org> ---
Fixed for 4.8.2.
^ permalink raw reply [flat|nested] 8+ messages in thread
end of thread, other threads:[~2014-06-12 13:29 UTC | newest]
Thread overview: 8+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2013-09-10 17:11 [Bug c/58385] New: likely wrong code bug regehr at cs dot utah.edu
2013-09-10 18:06 ` [Bug middle-end/58385] " glisse at gcc dot gnu.org
2013-09-10 18:33 ` [Bug middle-end/58385] [4.7/4.8/4.9 Regression] " jakub at gcc dot gnu.org
2013-09-10 18:55 ` jakub at gcc dot gnu.org
2013-09-11 11:35 ` jakub at gcc dot gnu.org
2013-09-11 12:17 ` jakub at gcc dot gnu.org
2013-09-11 12:19 ` [Bug middle-end/58385] [4.7 " jakub at gcc dot gnu.org
2014-06-12 13:29 ` rguenth 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).