http://gcc.gnu.org/bugzilla/show_bug.cgi?id=50078
Jakub Jelinek changed:
What|Removed |Added
Target Milestone|4.6.2 |4.6.3
--- Comment #9 from Jakub Jelinek
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=50078
--- Comment #8 from Steven Bosscher 2011-09-11
15:47:21 UTC ---
(In reply to comment #7)
> Comes from SSA expand => Matz
Comes from SSA expand because it is already wrong in the .expand dump:
;; MEM[(volatile unsigned int *)&var][arg_1(D)] ={v}
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=50078
Steven Bosscher changed:
What|Removed |Added
AssignedTo|steven at gcc dot gnu.org |matz at gcc dot gnu.org
--- Comment #7
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=50078
--- Comment #6 from Steven Bosscher 2011-09-11
15:22:36 UTC ---
int
set_noop_p (const_rtx set)
{
rtx src = SET_SRC (set);
rtx dst = SET_DEST (set);
if (dst == pc_rtx && src == pc_rtx)
return 1;
if (MEM_P (dst) && MEM_P (src))
re
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=50078
--- Comment #5 from Steven Bosscher 2011-09-11
15:14:19 UTC ---
Actually it is not noop_move_p that's at fault here, but the disgusting hack
for NOOP_MOVE_INSN_CODE. The insn is marked as a NOOP_MOVE somewhere else in
combine.
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=50078
Steven Bosscher changed:
What|Removed |Added
Status|NEW |ASSIGNED
CC|
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=50078
Joseph S. Myers changed:
What|Removed |Added
Priority|P3 |P2
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=50078
--- Comment #4 from Richard Guenther 2011-08-15
09:13:40 UTC ---
noop_move_p returns true for this - ignoding the side-effects.
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=50078
Richard Guenther changed:
What|Removed |Added
Keywords||wrong-code
Status|ASSIGNED