https://gcc.gnu.org/bugzilla/show_bug.cgi?id=69891
Jakub Jelinek changed:
What|Removed |Added
CC||sukhovvl at gmail dot com
--- Comment
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=69891
Jakub Jelinek changed:
What|Removed |Added
Target Milestone|--- |4.9.4
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=69891
--- Comment #19 from Zdenek Sojka ---
Target milestone 4.9.4 is not set.
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=69891
Jakub Jelinek changed:
What|Removed |Added
Status|ASSIGNED|RESOLVED
Resolution|---
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=69891
--- Comment #17 from Jakub Jelinek ---
Author: jakub
Date: Thu Jul 7 21:47:40 2016
New Revision: 238137
URL: https://gcc.gnu.org/viewcvs?rev=238137=gcc=rev
Log:
Backported from mainline
2016-02-26 Jakub Jelinek
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=69891
--- Comment #16 from Jakub Jelinek ---
Fixed for 5.4+ too.
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=69891
--- Comment #15 from Jakub Jelinek ---
Author: jakub
Date: Wed Mar 30 12:34:48 2016
New Revision: 234555
URL: https://gcc.gnu.org/viewcvs?rev=234555=gcc=rev
Log:
Backported from mainline
2016-02-26 Jakub Jelinek
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=69891
--- Comment #14 from Jakub Jelinek ---
Fixed on the trunk so far.
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=69891
--- Comment #13 from Jakub Jelinek ---
Author: jakub
Date: Fri Feb 26 15:53:43 2016
New Revision: 233743
URL: https://gcc.gnu.org/viewcvs?rev=233743=gcc=rev
Log:
PR rtl-optimization/69891
* dse.c (scan_insn): If we can't figure
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=69891
--- Comment #12 from Uroš Bizjak ---
(In reply to Uroš Bizjak from comment #11)
> (In reply to Jakub Jelinek from comment #9)
> > Created attachment 37801 [details]
> > gcc6-pr69891.patch
> >
> > Full patch I'm going to bootstrap/regtest.
>
>
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=69891
--- Comment #11 from Uroš Bizjak ---
(In reply to Jakub Jelinek from comment #9)
> Created attachment 37801 [details]
> gcc6-pr69891.patch
>
> Full patch I'm going to bootstrap/regtest.
+/* PR rtl-optimization/69891 */
+/* { dg-do run } */
+/*
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=69891
Eric Botcazou changed:
What|Removed |Added
Assignee|ebotcazou at gcc dot gnu.org |jakub at gcc dot gnu.org
---
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=69891
--- Comment #9 from Jakub Jelinek ---
Created attachment 37801
--> https://gcc.gnu.org/bugzilla/attachment.cgi?id=37801=edit
gcc6-pr69891.patch
Full patch I'm going to bootstrap/regtest.
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=69891
--- Comment #8 from Jakub Jelinek ---
I think the right fix is just:
--- gcc/dse.c.jj2016-01-19 13:32:12.0 +0100
+++ gcc/dse.c 2016-02-26 11:03:36.694206088 +0100
@@ -2556,6 +2556,8 @@ scan_insn (bb_info_t bb_info, rtx_insn *
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=69891
Eric Botcazou changed:
What|Removed |Added
Status|NEW |ASSIGNED
Assignee|unassigned
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=69891
--- Comment #6 from Eric Botcazou ---
> But the memset could be also SIBLING_CALL_P.
> Wouldn't it be better to change the else if to if, and add
> if (const_call) return;
> plus return to the end of mems_found == 1 then block? Then it would
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=69891
--- Comment #5 from Jakub Jelinek ---
(In reply to Eric Botcazou from comment #3)
> The problematic store is based on argp so it isn't killed by the memset,
> since only those based on sp or fp are:
>
> **scanning insn=20
> mem:
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=69891
--- Comment #4 from Eric Botcazou ---
> It's apparently a small loophole in the PR middle-end/31150 enhancement.
In fact this is sufficient:
Index: dse.c
===
--- dse.c
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=69891
Eric Botcazou changed:
What|Removed |Added
CC||jakub at gcc dot gnu.org
--- Comment #3
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=69891
Uroš Bizjak changed:
What|Removed |Added
CC||ebotcazou at gcc dot gnu.org,
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=69891
Uroš Bizjak changed:
What|Removed |Added
Status|UNCONFIRMED |NEW
Last reconfirmed|
21 matches
Mail list logo
