* [Bug rtl-optimization/64756] wrong code at -O3 on x86_64-linux-gnu (in 32-bit mode)
2015-01-23 22:16 [Bug rtl-optimization/64756] New: wrong code at -O3 on x86_64-linux-gnu (in 32-bit mode) su at cs dot ucdavis.edu
@ 2015-01-31 16:45 ` mikpelinux at gmail dot com
2015-01-31 17:41 ` mikpelinux at gmail dot com
` (6 subsequent siblings)
7 siblings, 0 replies; 9+ messages in thread
From: mikpelinux at gmail dot com @ 2015-01-31 16:45 UTC (permalink / raw)
To: gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=64756
Mikael Pettersson <mikpelinux at gmail dot com> changed:
What |Removed |Added
----------------------------------------------------------------------------
CC| |mikpelinux at gmail dot com
--- Comment #1 from Mikael Pettersson <mikpelinux at gmail dot com> ---
Started with r210113, aka wide-int merge. Still occurs with today's trunk.
^ permalink raw reply [flat|nested] 9+ messages in thread
* [Bug rtl-optimization/64756] wrong code at -O3 on x86_64-linux-gnu (in 32-bit mode)
2015-01-23 22:16 [Bug rtl-optimization/64756] New: wrong code at -O3 on x86_64-linux-gnu (in 32-bit mode) su at cs dot ucdavis.edu
2015-01-31 16:45 ` [Bug rtl-optimization/64756] " mikpelinux at gmail dot com
@ 2015-01-31 17:41 ` mikpelinux at gmail dot com
2015-02-01 13:16 ` [Bug rtl-optimization/64756] [5 Regression] " hjl.tools at gmail dot com
` (5 subsequent siblings)
7 siblings, 0 replies; 9+ messages in thread
From: mikpelinux at gmail dot com @ 2015-01-31 17:41 UTC (permalink / raw)
To: gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=64756
--- Comment #2 from Mikael Pettersson <mikpelinux at gmail dot com> ---
Also fails on m68k, but not on powerpc64, sparc64, or ARMv5.
^ permalink raw reply [flat|nested] 9+ messages in thread
* [Bug rtl-optimization/64756] [5 Regression] wrong code at -O3 on x86_64-linux-gnu (in 32-bit mode)
2015-01-23 22:16 [Bug rtl-optimization/64756] New: wrong code at -O3 on x86_64-linux-gnu (in 32-bit mode) su at cs dot ucdavis.edu
2015-01-31 16:45 ` [Bug rtl-optimization/64756] " mikpelinux at gmail dot com
2015-01-31 17:41 ` mikpelinux at gmail dot com
@ 2015-02-01 13:16 ` hjl.tools at gmail dot com
2015-02-02 8:47 ` rguenth at gcc dot gnu.org
` (4 subsequent siblings)
7 siblings, 0 replies; 9+ messages in thread
From: hjl.tools at gmail dot com @ 2015-02-01 13:16 UTC (permalink / raw)
To: gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=64756
H.J. Lu <hjl.tools at gmail dot com> changed:
What |Removed |Added
----------------------------------------------------------------------------
Priority|P3 |P1
Target Milestone|--- |5.0
Summary|wrong code at -O3 on |[5 Regression] wrong code
|x86_64-linux-gnu (in 32-bit |at -O3 on x86_64-linux-gnu
|mode) |(in 32-bit mode)
^ permalink raw reply [flat|nested] 9+ messages in thread
* [Bug rtl-optimization/64756] [5 Regression] wrong code at -O3 on x86_64-linux-gnu (in 32-bit mode)
2015-01-23 22:16 [Bug rtl-optimization/64756] New: wrong code at -O3 on x86_64-linux-gnu (in 32-bit mode) su at cs dot ucdavis.edu
` (2 preceding siblings ...)
2015-02-01 13:16 ` [Bug rtl-optimization/64756] [5 Regression] " hjl.tools at gmail dot com
@ 2015-02-02 8:47 ` rguenth at gcc dot gnu.org
2015-02-02 10:44 ` jakub at gcc dot gnu.org
` (3 subsequent siblings)
7 siblings, 0 replies; 9+ messages in thread
From: rguenth at gcc dot gnu.org @ 2015-02-02 8:47 UTC (permalink / raw)
To: gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=64756
Richard Biener <rguenth at gcc dot gnu.org> changed:
What |Removed |Added
----------------------------------------------------------------------------
Status|UNCONFIRMED |NEW
Last reconfirmed| |2015-02-02
Ever confirmed|0 |1
^ permalink raw reply [flat|nested] 9+ messages in thread
* [Bug rtl-optimization/64756] [5 Regression] wrong code at -O3 on x86_64-linux-gnu (in 32-bit mode)
2015-01-23 22:16 [Bug rtl-optimization/64756] New: wrong code at -O3 on x86_64-linux-gnu (in 32-bit mode) su at cs dot ucdavis.edu
` (3 preceding siblings ...)
2015-02-02 8:47 ` rguenth at gcc dot gnu.org
@ 2015-02-02 10:44 ` jakub at gcc dot gnu.org
2015-02-02 10:57 ` jakub at gcc dot gnu.org
` (2 subsequent siblings)
7 siblings, 0 replies; 9+ messages in thread
From: jakub at gcc dot gnu.org @ 2015-02-02 10:44 UTC (permalink / raw)
To: gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=64756
Jakub Jelinek <jakub at gcc dot gnu.org> changed:
What |Removed |Added
----------------------------------------------------------------------------
CC| |ebotcazou at gcc dot gnu.org,
| |jakub at gcc dot gnu.org,
| |law at gcc dot gnu.org
--- Comment #3 from Jakub Jelinek <jakub at gcc dot gnu.org> ---
No idea what this has to do with wide_int, I don't see any relation
(unfortunately my r210112 and r210113 copies are without debug info at this
point).
But, it really seems to be a CSE bug to me (during cse2).
The thing is, in insn 29 tmp is stored to using volatile store:
(insn 29 27 30 7 (set (mem/v/f/c:SI (symbol_ref:SI ("tmp") <var_decl
0x7f0ea7350cf0 tmp>) [1 MEM[(int * volatile *)&tmp]+0 S4 A32])
(const_int 0 [0])) pr64756.c:19 90 {*movsi_internal}
(nil))
but when the hash of dest is computed, we actually use:
if (MEM_P (dest))
{
#ifdef PUSH_ROUNDING
/* Stack pushes invalidate the stack pointer. */
rtx addr = XEXP (dest, 0);
if (GET_RTX_CLASS (GET_CODE (addr)) == RTX_AUTOINC
&& XEXP (addr, 0) == stack_pointer_rtx)
invalidate (stack_pointer_rtx, VOIDmode);
#endif
dest = fold_rtx (dest, insn);
}
/* Compute the hash code of the destination now,
before the effects of this instruction are recorded,
since the register values used in the address computation
are those before this instruction. */
sets[i].dest_hash = HASH (dest, mode);
and so dest is folded first into symbol_ref ("a"). That means when computing
hash we do not get do_not_record flag set, which we would otherwise get because
the memory is volatile, nor we do get the hash_arg_in_memory flag set.
Next we recompute the hash, but do not look at do_not_record anymore:
if (sets[i].rtl != 0 && dest != SET_DEST (sets[i].rtl))
sets[i].dest_hash = HASH (SET_DEST (sets[i].rtl), mode);
Later on we insert this mem/v/f/c into the hash table and set the in_memory
flag (correctly):
elt->in_memory = (MEM_P (sets[i].inner_dest)
&& !MEM_READONLY_P (sets[i].inner_dest));
Later on merge_equiv_classes is called, and that one obviously doesn't count
with the option that do_not_record might be true, and when we set
do_not_record, we don't set hash_arg_in_memory, so we end up with a new elt
that for the MEM
at this time doesn't even have in_memory flag set, so when we later in insn 36
store into memory that aliases with this, we do not even invalidate it.
^ permalink raw reply [flat|nested] 9+ messages in thread
* [Bug rtl-optimization/64756] [5 Regression] wrong code at -O3 on x86_64-linux-gnu (in 32-bit mode)
2015-01-23 22:16 [Bug rtl-optimization/64756] New: wrong code at -O3 on x86_64-linux-gnu (in 32-bit mode) su at cs dot ucdavis.edu
` (4 preceding siblings ...)
2015-02-02 10:44 ` jakub at gcc dot gnu.org
@ 2015-02-02 10:57 ` jakub at gcc dot gnu.org
2015-02-03 20:42 ` jakub at gcc dot gnu.org
2015-02-04 8:42 ` jakub at gcc dot gnu.org
7 siblings, 0 replies; 9+ messages in thread
From: jakub at gcc dot gnu.org @ 2015-02-02 10:57 UTC (permalink / raw)
To: gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=64756
--- Comment #4 from Jakub Jelinek <jakub at gcc dot gnu.org> ---
Created attachment 34643
--> https://gcc.gnu.org/bugzilla/attachment.cgi?id=34643&action=edit
gcc5-pr64756.patch
I'd say we just should never record volatile MEMs into the hash table, this
patch attempts to ensure that.
^ permalink raw reply [flat|nested] 9+ messages in thread
* [Bug rtl-optimization/64756] [5 Regression] wrong code at -O3 on x86_64-linux-gnu (in 32-bit mode)
2015-01-23 22:16 [Bug rtl-optimization/64756] New: wrong code at -O3 on x86_64-linux-gnu (in 32-bit mode) su at cs dot ucdavis.edu
` (5 preceding siblings ...)
2015-02-02 10:57 ` jakub at gcc dot gnu.org
@ 2015-02-03 20:42 ` jakub at gcc dot gnu.org
2015-02-04 8:42 ` jakub at gcc dot gnu.org
7 siblings, 0 replies; 9+ messages in thread
From: jakub at gcc dot gnu.org @ 2015-02-03 20:42 UTC (permalink / raw)
To: gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=64756
--- Comment #5 from Jakub Jelinek <jakub at gcc dot gnu.org> ---
Author: jakub
Date: Tue Feb 3 20:41:38 2015
New Revision: 220377
URL: https://gcc.gnu.org/viewcvs?rev=220377&root=gcc&view=rev
Log:
PR rtl-optimization/64756
* cse.c (invalidate_dest): New function.
(cse_insn): Use it. If dest != SET_DEST (sets[i].rtl) and
HASH (SET_DEST (sets[i].rtl), mode) computation sets do_not_record,
invalidate and do not record it.
* gcc.c-torture/execute/pr64756.c: New test.
Added:
trunk/gcc/testsuite/gcc.c-torture/execute/pr64756.c
Modified:
trunk/gcc/ChangeLog
trunk/gcc/cse.c
trunk/gcc/testsuite/ChangeLog
^ permalink raw reply [flat|nested] 9+ messages in thread
* [Bug rtl-optimization/64756] [5 Regression] wrong code at -O3 on x86_64-linux-gnu (in 32-bit mode)
2015-01-23 22:16 [Bug rtl-optimization/64756] New: wrong code at -O3 on x86_64-linux-gnu (in 32-bit mode) su at cs dot ucdavis.edu
` (6 preceding siblings ...)
2015-02-03 20:42 ` jakub at gcc dot gnu.org
@ 2015-02-04 8:42 ` jakub at gcc dot gnu.org
7 siblings, 0 replies; 9+ messages in thread
From: jakub at gcc dot gnu.org @ 2015-02-04 8:42 UTC (permalink / raw)
To: gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=64756
Jakub Jelinek <jakub at gcc dot gnu.org> changed:
What |Removed |Added
----------------------------------------------------------------------------
Status|NEW |RESOLVED
CC|jakub at gcc dot gnu.org |
Resolution|--- |FIXED
Assignee|unassigned at gcc dot gnu.org |jakub at gcc dot gnu.org
--- Comment #6 from Jakub Jelinek <jakub at gcc dot gnu.org> ---
Fixed.
^ permalink raw reply [flat|nested] 9+ messages in thread