[Bug tree-optimization/84339] [8 Regression] Wrong-code with optimizing strlen

2018-02-13 Thread jakub at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=84339

Jakub Jelinek  changed:

   What|Removed |Added

 Status|ASSIGNED|RESOLVED
 Resolution|--- |FIXED

--- Comment #4 from Jakub Jelinek  ---
Fixed.

[Bug tree-optimization/84339] [8 Regression] Wrong-code with optimizing strlen

2018-02-13 Thread jakub at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=84339

--- Comment #3 from Jakub Jelinek  ---
Author: jakub
Date: Tue Feb 13 08:35:53 2018
New Revision: 257618

URL: https://gcc.gnu.org/viewcvs?rev=257618&root=gcc&view=rev
Log:
PR tree-optimization/84339
* gimple-fold.c (get_range_strlen): Set *FLEXP to true when handling
ARRAY_REF where first operand is array_at_struct_end_p COMPONENT_REF.
Formatting fixes.

* gcc.c-torture/execute/pr84339.c: New test.

Added:
trunk/gcc/testsuite/gcc.c-torture/execute/pr84339.c
Modified:
trunk/gcc/ChangeLog
trunk/gcc/gimple-fold.c
trunk/gcc/testsuite/ChangeLog

[Bug tree-optimization/84339] [8 Regression] Wrong-code with optimizing strlen

2018-02-12 Thread rguenth at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=84339

Richard Biener  changed:

   What|Removed |Added

   Keywords||wrong-code
   Priority|P3  |P1

[Bug tree-optimization/84339] [8 Regression] Wrong-code with optimizing strlen

2018-02-12 Thread jakub at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=84339

--- Comment #2 from Jakub Jelinek  ---
Created attachment 43396
  --> https://gcc.gnu.org/bugzilla/attachment.cgi?id=43396&action=edit
gcc8-pr84339.patch

Untested fix.

[Bug tree-optimization/84339] [8 Regression] Wrong-code with optimizing strlen

2018-02-12 Thread mpolacek at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=84339

Marek Polacek  changed:

   What|Removed |Added

 Status|UNCONFIRMED |ASSIGNED
   Last reconfirmed||2018-02-12
   Assignee|unassigned at gcc dot gnu.org  |jakub at gcc dot gnu.org
   Target Milestone|--- |8.0
 Ever confirmed|0   |1

--- Comment #1 from Marek Polacek  ---
Jakub has a fix.