* [Bug tree-optimization/63841] Incorrect strlen optimization after complete unroll
2014-11-12 22:00 [Bug tree-optimization/63841] New: Incorrect strlen optimization after complete unroll tejohnson at google dot com
@ 2014-11-12 22:01 ` tejohnson at google dot com
2014-11-13 9:20 ` rguenth at gcc dot gnu.org
` (10 subsequent siblings)
11 siblings, 0 replies; 13+ messages in thread
From: tejohnson at google dot com @ 2014-11-12 22:01 UTC (permalink / raw)
To: gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=63841
--- Comment #1 from Teresa Johnson <tejohnson at google dot com> ---
Google ref b/18344370
^ permalink raw reply [flat|nested] 13+ messages in thread
* [Bug tree-optimization/63841] Incorrect strlen optimization after complete unroll
2014-11-12 22:00 [Bug tree-optimization/63841] New: Incorrect strlen optimization after complete unroll tejohnson at google dot com
2014-11-12 22:01 ` [Bug tree-optimization/63841] " tejohnson at google dot com
@ 2014-11-13 9:20 ` rguenth at gcc dot gnu.org
2014-11-13 14:27 ` [Bug tree-optimization/63841] [4.8/4.9/5 Regression] " tejohnson at google dot com
` (9 subsequent siblings)
11 siblings, 0 replies; 13+ messages in thread
From: rguenth at gcc dot gnu.org @ 2014-11-13 9:20 UTC (permalink / raw)
To: gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=63841
Richard Biener <rguenth at gcc dot gnu.org> changed:
What |Removed |Added
----------------------------------------------------------------------------
Keywords| |wrong-code
Status|UNCONFIRMED |NEW
Last reconfirmed| |2014-11-13
CC| |jakub at gcc dot gnu.org
Ever confirmed|0 |1
--- Comment #2 from Richard Biener <rguenth at gcc dot gnu.org> ---
Confirmed. Seems to be an opportunity to remove the clobber to be able to
remove a dead store... ;)
^ permalink raw reply [flat|nested] 13+ messages in thread
* [Bug tree-optimization/63841] [4.8/4.9/5 Regression] Incorrect strlen optimization after complete unroll
2014-11-12 22:00 [Bug tree-optimization/63841] New: Incorrect strlen optimization after complete unroll tejohnson at google dot com
2014-11-12 22:01 ` [Bug tree-optimization/63841] " tejohnson at google dot com
2014-11-13 9:20 ` rguenth at gcc dot gnu.org
@ 2014-11-13 14:27 ` tejohnson at google dot com
2014-11-13 15:37 ` tejohnson at gcc dot gnu.org
` (8 subsequent siblings)
11 siblings, 0 replies; 13+ messages in thread
From: tejohnson at google dot com @ 2014-11-13 14:27 UTC (permalink / raw)
To: gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=63841
--- Comment #4 from Teresa Johnson <tejohnson at google dot com> ---
On Thu, Nov 13, 2014 at 1:27 AM, jakub at gcc dot gnu.org
<gcc-bugzilla@gcc.gnu.org> wrote:
> https://gcc.gnu.org/bugzilla/show_bug.cgi?id=63841
>
> 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
> Target Milestone|--- |4.8.4
> Summary|Incorrect strlen |[4.8/4.9/5 Regression]
> |optimization after complete |Incorrect strlen
> |unroll |optimization after complete
> | |unroll
>
> --- Comment #3 from Jakub Jelinek <jakub at gcc dot gnu.org> ---
> Started with r181172. I'll take a look during stage3.
I have a fix under review:
https://gcc.gnu.org/ml/gcc-patches/2014-11/msg01481.html
Teresa
>
> --
> You are receiving this mail because:
> You are on the CC list for the bug.
> You reported the bug.
^ permalink raw reply [flat|nested] 13+ messages in thread
* [Bug tree-optimization/63841] [4.8/4.9/5 Regression] Incorrect strlen optimization after complete unroll
2014-11-12 22:00 [Bug tree-optimization/63841] New: Incorrect strlen optimization after complete unroll tejohnson at google dot com
` (2 preceding siblings ...)
2014-11-13 14:27 ` [Bug tree-optimization/63841] [4.8/4.9/5 Regression] " tejohnson at google dot com
@ 2014-11-13 15:37 ` tejohnson at gcc dot gnu.org
2014-11-13 21:51 ` tejohnson at gcc dot gnu.org
` (7 subsequent siblings)
11 siblings, 0 replies; 13+ messages in thread
From: tejohnson at gcc dot gnu.org @ 2014-11-13 15:37 UTC (permalink / raw)
To: gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=63841
--- Comment #5 from tejohnson at gcc dot gnu.org ---
Author: tejohnson
Date: Thu Nov 13 15:36:48 2014
New Revision: 217505
URL: https://gcc.gnu.org/viewcvs?rev=217505&root=gcc&view=rev
Log:
2014-11-13 Teresa Johnson <tejohnson@google.com>
gcc:
PR tree-optimization/63841
* tree.c (initializer_zerop): A clobber does not zero initialize.
gcc/testsuite:
PR tree-optimization/63841
* g++.dg/tree-ssa/pr63841.C: New test.
Added:
trunk/gcc/testsuite/g++.dg/tree-ssa/pr63841.C
Modified:
trunk/gcc/ChangeLog
trunk/gcc/testsuite/ChangeLog
trunk/gcc/tree.c
^ permalink raw reply [flat|nested] 13+ messages in thread
* [Bug tree-optimization/63841] [4.8/4.9/5 Regression] Incorrect strlen optimization after complete unroll
2014-11-12 22:00 [Bug tree-optimization/63841] New: Incorrect strlen optimization after complete unroll tejohnson at google dot com
` (3 preceding siblings ...)
2014-11-13 15:37 ` tejohnson at gcc dot gnu.org
@ 2014-11-13 21:51 ` tejohnson at gcc dot gnu.org
2014-11-14 6:36 ` tejohnson at gcc dot gnu.org
` (6 subsequent siblings)
11 siblings, 0 replies; 13+ messages in thread
From: tejohnson at gcc dot gnu.org @ 2014-11-13 21:51 UTC (permalink / raw)
To: gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=63841
--- Comment #6 from tejohnson at gcc dot gnu.org ---
Author: tejohnson
Date: Thu Nov 13 21:51:11 2014
New Revision: 217522
URL: https://gcc.gnu.org/viewcvs?rev=217522&root=gcc&view=rev
Log:
2014-11-13 Teresa Johnson <tejohnson@google.com>
gcc:
PR tree-optimization/63841
* tree-ssa-strlen.c (strlen_optimize_stmt): Ignore clobbers.
2014-11-13 Teresa Johnson <tejohnson@google.com>
gcc/testsuite:
PR tree-optimization/63841
* testsuite/g++.dg/tree-ssa/pr63841.C: New test.
Added:
branches/gcc-4_9-branch/gcc/testsuite/g++.dg/tree-ssa/pr63841.C
Modified:
branches/gcc-4_9-branch/gcc/ChangeLog
branches/gcc-4_9-branch/gcc/testsuite/ChangeLog
branches/gcc-4_9-branch/gcc/tree-ssa-strlen.c
^ permalink raw reply [flat|nested] 13+ messages in thread
* [Bug tree-optimization/63841] [4.8/4.9/5 Regression] Incorrect strlen optimization after complete unroll
2014-11-12 22:00 [Bug tree-optimization/63841] New: Incorrect strlen optimization after complete unroll tejohnson at google dot com
` (4 preceding siblings ...)
2014-11-13 21:51 ` tejohnson at gcc dot gnu.org
@ 2014-11-14 6:36 ` tejohnson at gcc dot gnu.org
2014-11-17 21:16 ` jakub at gcc dot gnu.org
` (5 subsequent siblings)
11 siblings, 0 replies; 13+ messages in thread
From: tejohnson at gcc dot gnu.org @ 2014-11-14 6:36 UTC (permalink / raw)
To: gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=63841
--- Comment #7 from tejohnson at gcc dot gnu.org ---
Author: tejohnson
Date: Fri Nov 14 06:35:35 2014
New Revision: 217537
URL: https://gcc.gnu.org/viewcvs?rev=217537&root=gcc&view=rev
Log:
2014-11-13 Teresa Johnson <tejohnson@google.com>
gcc:
PR tree-optimization/63841
* tree-ssa-strlen.c (strlen_optimize_stmt): Ignore clobbers.
2014-11-13 Teresa Johnson <tejohnson@google.com>
gcc/testsuite:
PR tree-optimization/63841
* g++.dg/tree-ssa/pr63841.C: Remove prints, use abort.
Modified:
trunk/gcc/ChangeLog
trunk/gcc/testsuite/ChangeLog
trunk/gcc/testsuite/g++.dg/tree-ssa/pr63841.C
trunk/gcc/tree-ssa-strlen.c
^ permalink raw reply [flat|nested] 13+ messages in thread
* [Bug tree-optimization/63841] [4.8/4.9/5 Regression] Incorrect strlen optimization after complete unroll
2014-11-12 22:00 [Bug tree-optimization/63841] New: Incorrect strlen optimization after complete unroll tejohnson at google dot com
` (5 preceding siblings ...)
2014-11-14 6:36 ` tejohnson at gcc dot gnu.org
@ 2014-11-17 21:16 ` jakub at gcc dot gnu.org
2014-11-17 21:17 ` [Bug tree-optimization/63841] [4.8 " jakub at gcc dot gnu.org
` (4 subsequent siblings)
11 siblings, 0 replies; 13+ messages in thread
From: jakub at gcc dot gnu.org @ 2014-11-17 21:16 UTC (permalink / raw)
To: gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=63841
Jakub Jelinek <jakub at gcc dot gnu.org> changed:
What |Removed |Added
----------------------------------------------------------------------------
Status|ASSIGNED |RESOLVED
Resolution|--- |FIXED
Assignee|jakub at gcc dot gnu.org |unassigned at gcc dot gnu.org
--- Comment #8 from Jakub Jelinek <jakub at gcc dot gnu.org> ---
Fixed by Teresa.
^ permalink raw reply [flat|nested] 13+ messages in thread
* [Bug tree-optimization/63841] [4.8 Regression] Incorrect strlen optimization after complete unroll
2014-11-12 22:00 [Bug tree-optimization/63841] New: Incorrect strlen optimization after complete unroll tejohnson at google dot com
` (6 preceding siblings ...)
2014-11-17 21:16 ` jakub at gcc dot gnu.org
@ 2014-11-17 21:17 ` jakub at gcc dot gnu.org
2014-11-17 21:27 ` tejohnson at google dot com
` (3 subsequent siblings)
11 siblings, 0 replies; 13+ messages in thread
From: jakub at gcc dot gnu.org @ 2014-11-17 21:17 UTC (permalink / raw)
To: gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=63841
Jakub Jelinek <jakub at gcc dot gnu.org> changed:
What |Removed |Added
----------------------------------------------------------------------------
Status|RESOLVED |REOPENED
Known to work| |4.9.3, 5.0
Resolution|FIXED |---
Summary|[4.8/4.9/5 Regression] |[4.8 Regression] Incorrect
|Incorrect strlen |strlen optimization after
|optimization after complete |complete unroll
|unroll |
--- Comment #9 from Jakub Jelinek <jakub at gcc dot gnu.org> ---
Ah, though not on the 4.8 branch.
^ permalink raw reply [flat|nested] 13+ messages in thread
* [Bug tree-optimization/63841] [4.8 Regression] Incorrect strlen optimization after complete unroll
2014-11-12 22:00 [Bug tree-optimization/63841] New: Incorrect strlen optimization after complete unroll tejohnson at google dot com
` (7 preceding siblings ...)
2014-11-17 21:17 ` [Bug tree-optimization/63841] [4.8 " jakub at gcc dot gnu.org
@ 2014-11-17 21:27 ` tejohnson at google dot com
2014-11-18 14:21 ` tejohnson at gcc dot gnu.org
` (2 subsequent siblings)
11 siblings, 0 replies; 13+ messages in thread
From: tejohnson at google dot com @ 2014-11-17 21:27 UTC (permalink / raw)
To: gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=63841
--- Comment #10 from Teresa Johnson <tejohnson at google dot com> ---
Missed that one, I will backport to 4.8.
Teresa
On Mon, Nov 17, 2014 at 1:17 PM, jakub at gcc dot gnu.org
<gcc-bugzilla@gcc.gnu.org> wrote:
> https://gcc.gnu.org/bugzilla/show_bug.cgi?id=63841
>
> Jakub Jelinek <jakub at gcc dot gnu.org> changed:
>
> What |Removed |Added
> ----------------------------------------------------------------------------
> Status|RESOLVED |REOPENED
> Known to work| |4.9.3, 5.0
> Resolution|FIXED |---
> Summary|[4.8/4.9/5 Regression] |[4.8 Regression] Incorrect
> |Incorrect strlen |strlen optimization after
> |optimization after complete |complete unroll
> |unroll |
>
> --- Comment #9 from Jakub Jelinek <jakub at gcc dot gnu.org> ---
> Ah, though not on the 4.8 branch.
>
> --
> You are receiving this mail because:
> You are on the CC list for the bug.
> You reported the bug.
^ permalink raw reply [flat|nested] 13+ messages in thread
* [Bug tree-optimization/63841] [4.8 Regression] Incorrect strlen optimization after complete unroll
2014-11-12 22:00 [Bug tree-optimization/63841] New: Incorrect strlen optimization after complete unroll tejohnson at google dot com
` (8 preceding siblings ...)
2014-11-17 21:27 ` tejohnson at google dot com
@ 2014-11-18 14:21 ` tejohnson at gcc dot gnu.org
2014-11-20 14:30 ` tejohnson at gcc dot gnu.org
2014-12-10 13:16 ` rguenth at gcc dot gnu.org
11 siblings, 0 replies; 13+ messages in thread
From: tejohnson at gcc dot gnu.org @ 2014-11-18 14:21 UTC (permalink / raw)
To: gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=63841
--- Comment #11 from tejohnson at gcc dot gnu.org ---
Author: tejohnson
Date: Tue Nov 18 14:20:58 2014
New Revision: 217715
URL: https://gcc.gnu.org/viewcvs?rev=217715&root=gcc&view=rev
Log:
2014-11-18 Teresa Johnson <tejohnson@google.com>
Backport from mainline and gcc-4_9 branch.
2014-11-13 Teresa Johnson <tejohnson@google.com>
PR tree-optimization/63841
* tree-ssa-strlen.c (strlen_optimize_stmt): Ignore clobbers.
2014-11-13 Teresa Johnson <tejohnson@google.com>
PR tree-optimization/63841
* testsuite/g++.dg/tree-ssa/pr63841.C: New test.
Added:
branches/gcc-4_8-branch/gcc/testsuite/g++.dg/tree-ssa/pr63841.C
Modified:
branches/gcc-4_8-branch/gcc/ChangeLog
branches/gcc-4_8-branch/gcc/testsuite/ChangeLog
branches/gcc-4_8-branch/gcc/tree-ssa-strlen.c
^ permalink raw reply [flat|nested] 13+ messages in thread
* [Bug tree-optimization/63841] [4.8 Regression] Incorrect strlen optimization after complete unroll
2014-11-12 22:00 [Bug tree-optimization/63841] New: Incorrect strlen optimization after complete unroll tejohnson at google dot com
` (9 preceding siblings ...)
2014-11-18 14:21 ` tejohnson at gcc dot gnu.org
@ 2014-11-20 14:30 ` tejohnson at gcc dot gnu.org
2014-12-10 13:16 ` rguenth at gcc dot gnu.org
11 siblings, 0 replies; 13+ messages in thread
From: tejohnson at gcc dot gnu.org @ 2014-11-20 14:30 UTC (permalink / raw)
To: gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=63841
--- Comment #12 from tejohnson at gcc dot gnu.org ---
Author: tejohnson
Date: Thu Nov 20 14:29:41 2014
New Revision: 217858
URL: https://gcc.gnu.org/viewcvs?rev=217858&root=gcc&view=rev
Log:
2014-11-20 Teresa Johnson <tejohnson@google.com>
Backport r217522 from gcc-4_9 for Google ref b/18344370 and b/18455179.
2014-11-13 Teresa Johnson <tejohnson@google.com>
PR tree-optimization/63841
* tree-ssa-strlen.c (strlen_optimize_stmt): Ignore clobbers.
2014-11-13 Teresa Johnson <tejohnson@google.com>
PR tree-optimization/63841
* testsuite/g++.dg/tree-ssa/pr63841.C: New test.
Added:
branches/google/gcc-4_9/gcc/testsuite/g++.dg/tree-ssa/pr63841.C
Modified:
branches/google/gcc-4_9/gcc/tree-ssa-strlen.c
^ permalink raw reply [flat|nested] 13+ messages in thread
* [Bug tree-optimization/63841] [4.8 Regression] Incorrect strlen optimization after complete unroll
2014-11-12 22:00 [Bug tree-optimization/63841] New: Incorrect strlen optimization after complete unroll tejohnson at google dot com
` (10 preceding siblings ...)
2014-11-20 14:30 ` tejohnson at gcc dot gnu.org
@ 2014-12-10 13:16 ` rguenth at gcc dot gnu.org
11 siblings, 0 replies; 13+ messages in thread
From: rguenth at gcc dot gnu.org @ 2014-12-10 13:16 UTC (permalink / raw)
To: gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=63841
Richard Biener <rguenth at gcc dot gnu.org> changed:
What |Removed |Added
----------------------------------------------------------------------------
Status|REOPENED |RESOLVED
Resolution|--- |FIXED
--- Comment #13 from Richard Biener <rguenth at gcc dot gnu.org> ---
Fixed.
^ permalink raw reply [flat|nested] 13+ messages in thread