[Bug fortran/83149] [6- and 7-branches] Missing test for sym->ns->proc_name: crash_signal in toplev.c:325

2018-05-16 Thread pault at gcc dot gnu.org 
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=83149

Paul Thomas  changed:

   What|Removed |Added

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

--- Comment #16 from Paul Thomas  ---
Fixed on 6-branch through to trunk

Thanks for the report.

Paul

[Bug fortran/83149] [6- and 7-branches] Missing test for sym->ns->proc_name: crash_signal in toplev.c:325

2018-05-16 Thread pault at gcc dot gnu.org 
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=83149

--- Comment #15 from Paul Thomas  ---
Author: pault
Date: Wed May 16 11:42:47 2018
New Revision: 260286

URL: https://gcc.gnu.org/viewcvs?rev=260286&root=gcc&view=rev
Log:
2018-05-16  Paul Thomas  

PR fortran/83149
Backport from trunk
* trans-decl.c (gfc_finish_var_decl): Test sym->ns->proc_name
before accessing its components.
* trans-types.c (gfc_sym_type): If a character result has null
backend_decl, try the procedure symbol.

2018-05-16  Paul Thomas  

PR fortran/83149
Backport from trunk
* gfortran.dg/pr83149_1.f90: New test.
* gfortran.dg/pr83149.f90: Additional source for previous.
* gfortran.dg/pr83149_b.f90: New test.
* gfortran.dg/pr83149_a.f90: Additional source for previous.


Added:
branches/gcc-6-branch/gcc/testsuite/gfortran.dg/pr83149.f90
branches/gcc-6-branch/gcc/testsuite/gfortran.dg/pr83149_1.f90
branches/gcc-6-branch/gcc/testsuite/gfortran.dg/pr83149_a.f90
branches/gcc-6-branch/gcc/testsuite/gfortran.dg/pr83149_b.f90
Modified:
branches/gcc-6-branch/gcc/fortran/ChangeLog
branches/gcc-6-branch/gcc/fortran/trans-decl.c
branches/gcc-6-branch/gcc/fortran/trans-types.c
branches/gcc-6-branch/gcc/testsuite/ChangeLog

[Bug fortran/83149] [6- and 7-branches] Missing test for sym->ns->proc_name: crash_signal in toplev.c:325

2018-05-16 Thread pault at gcc dot gnu.org 
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=83149

--- Comment #14 from Paul Thomas  ---
Author: pault
Date: Wed May 16 11:17:10 2018
New Revision: 260285

URL: https://gcc.gnu.org/viewcvs?rev=260285&root=gcc&view=rev
Log:
2018-05-16  Paul Thomas  

PR fortran/83149
Backport from trunk
* trans-decl.c (gfc_finish_var_decl): Test sym->ns->proc_name
before accessing its components.
* trans-types.c (gfc_sym_type): If a character result has null
backend_decl, try the procedure symbol..

2018-05-16  Paul Thomas  

PR fortran/83149
Backport from trunk
* gfortran.dg/pr83149_1.f90: New test.
* gfortran.dg/pr83149.f90: Additional source for previous.
* gfortran.dg/pr83149_b.f90: New test.
* gfortran.dg/pr83149_a.f90: Additional source for previous.


Added:
branches/gcc-7-branch/gcc/testsuite/gfortran.dg/pr83149.f90
branches/gcc-7-branch/gcc/testsuite/gfortran.dg/pr83149_1.f90
branches/gcc-7-branch/gcc/testsuite/gfortran.dg/pr83149_a.f90
branches/gcc-7-branch/gcc/testsuite/gfortran.dg/pr83149_b.f90
Modified:
branches/gcc-7-branch/gcc/fortran/ChangeLog
branches/gcc-7-branch/gcc/fortran/trans-decl.c
branches/gcc-7-branch/gcc/fortran/trans-types.c
branches/gcc-7-branch/gcc/testsuite/ChangeLog

[Bug fortran/83149] [6- and 7-branches] Missing test for sym->ns->proc_name: crash_signal in toplev.c:325

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

Jakub Jelinek  changed:

   What|Removed |Added

   Target Milestone|8.0 |8.2

--- Comment #13 from Jakub Jelinek  ---
GCC 8.1 has been released.

[Bug fortran/83149] [6- and 7-branches] Missing test for sym->ns->proc_name: crash_signal in toplev.c:325

2018-03-04 Thread dominiq at lps dot ens.fr 
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=83149

Dominique d'Humieres  changed:

   What|Removed |Added

 Status|NEW |ASSIGNED

[Bug fortran/83149] [6- and 7-branches] Missing test for sym->ns->proc_name: crash_signal in toplev.c:325

2018-02-23 Thread pault at gcc dot gnu.org 
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=83149

Paul Thomas  changed:

   What|Removed |Added

   Assignee|unassigned at gcc dot gnu.org  |pault at gcc dot gnu.org
Summary|[8 Regression] ICE on   |[6- and 7-branches] Missing
   |SELECT CASE: crash_signal   |test for
   |in toplev.c:325 |sym->ns->proc_name:
   ||crash_signal in
   ||toplev.c:325

--- Comment #12 from Paul Thomas  ---
The original regression is now fixed. The testcases of comments 2 and 7 have
been fixed on trunk but remain on 6- and 7-branches. I will await advice from
the list as to whether to backport or not and so will keep the PR open.

Neil, you were right about the common origin. Thanks for the report.

Paul