[Bug fortran/71902] [5/6/7 Regression] Unneeded temporary on reallocatable character assignment

[tkoenig at gcc dot gnu.org]

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=71902

--- Comment #9 from Thomas Koenig  ---
Author: tkoenig
Date: Sun Sep  4 16:17:55 2016
New Revision: 239977

URL: https://gcc.gnu.org/viewcvs?rev=239977=gcc=rev
Log:
2016-09-04  Thomas Koenig  

PR fortran/71902
* frontend-passes.c (realloc_string_callback): Also check for the
lhs being deferred.  Name temporary variable "realloc_string".

2016-09-04  Thomas Koenig  

PR fortran/71902
* gfortran.dg/dependency_47.f90:  New test.
* gfortran.dg/dependency_49.f90:  New test.


Added:
trunk/gcc/testsuite/gfortran.dg/dependency_47.f90
trunk/gcc/testsuite/gfortran.dg/dependency_49.f90
Modified:
trunk/gcc/fortran/frontend-passes.c

[Bug fortran/71902] [5/6/7 Regression] Unneeded temporary on reallocatable character assignment

[tkoenig at gcc dot gnu.org]

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=71902

Thomas Koenig  changed:

   What|Removed |Added

URL||https://gcc.gnu.org/ml/gcc-
   ||patches/2016-08/msg01076.ht
   ||ml

--- Comment #8 from Thomas Koenig  ---
Patch at https://gcc.gnu.org/ml/gcc-patches/2016-08/msg01076.html .

[Bug fortran/71902] [5/6/7 Regression] Unneeded temporary on reallocatable character assignment

[tkoenig at gcc dot gnu.org]

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=71902

Thomas Koenig  changed:

   What|Removed |Added

Summary|[5/6 Regression] Unneeded   |[5/6/7 Regression] Unneeded
   |temporary on reallocatable  |temporary on reallocatable
   |character assignment|character assignment

--- Comment #7 from Thomas Koenig  ---
After the revert, it is again a 7 regression.

[Bug fortran/71902] [5/6/7 Regression] Unneeded temporary on reallocatable character assignment

[tkoenig at gcc dot gnu.org]

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=71902

--- Comment #2 from Thomas Koenig  ---
Author: tkoenig
Date: Tue Jul 19 21:25:33 2016
New Revision: 238497

URL: https://gcc.gnu.org/viewcvs?rev=238497=gcc=rev
Log:
2016-07-19  Thomas Koenig  

PR fortran/71902
* dependency.c (gfc_check_dependency): Use dep_ref.  Handle case
if identical is true and two array element references differ.
(gfc_dep_resovler):  Move most of the code to dep_ref.
(dep_ref):  New function.
* frontend-passes.c (realloc_string_callback):  Name temporary
variable "realloc_string".

2016-07-19  Thomas Koenig  

PR fortran/71902
* gfortran.dg/dependency_47.f90:  New test.


Added:
trunk/gcc/testsuite/gfortran.dg/dependency_47.f90
Modified:
trunk/gcc/fortran/ChangeLog
trunk/gcc/fortran/dependency.c
trunk/gcc/fortran/frontend-passes.c
trunk/gcc/testsuite/ChangeLog

[Bug fortran/71902] [5/6/7 Regression] Unneeded temporary on reallocatable character assignment

[rguenth at gcc dot gnu.org]

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=71902

Richard Biener  changed:

   What|Removed |Added

   Priority|P3  |P4

[Bug fortran/71902] [5/6/7 Regression] Unneeded temporary on reallocatable character assignment

[tkoenig at gcc dot gnu.org]

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=71902

Thomas Koenig  changed:

   What|Removed |Added

   Keywords||missed-optimization
 Status|UNCONFIRMED |ASSIGNED
   Last reconfirmed||2016-07-16
   Assignee|unassigned at gcc dot gnu.org  |tkoenig at gcc dot 
gnu.org
   Target Milestone|--- |5.5
 Ever confirmed|0   |1

--- Comment #1 from Thomas Koenig  ---
I have a patch.