[Bug fortran/58007] [4.7/4.9 Regression] [OOP] ICE in free_pi_tree(): Unresolved fixup - resolve_fixups does not fixup component of __class_bsr_Bsr_matrix

2014-01-27 Thread mikael at gcc dot gnu.org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=58007

--- Comment #20 from Mikael Morin mikael at gcc dot gnu.org ---
Author: mikael
Date: Mon Jan 27 21:23:16 2014
New Revision: 207152

URL: http://gcc.gnu.org/viewcvs?rev=207152root=gccview=rev
Log:
fortran/
PR fortran/58007
* module.c (skip_list): Don't use default argument value.
(load_derived_extensions, read_module): Update callers.


Modified:
branches/gcc-4_7-branch/gcc/fortran/ChangeLog
branches/gcc-4_7-branch/gcc/fortran/module.c


[Bug fortran/58007] [4.7/4.9 Regression] [OOP] ICE in free_pi_tree(): Unresolved fixup - resolve_fixups does not fixup component of __class_bsr_Bsr_matrix

2014-01-26 Thread mikael at gcc dot gnu.org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=58007

--- Comment #16 from Mikael Morin mikael at gcc dot gnu.org ---
Author: mikael
Date: Sun Jan 26 13:04:54 2014
New Revision: 207117

URL: http://gcc.gnu.org/viewcvs?rev=207117root=gccview=rev
Log:
fortran/
PR fortran/58007
* module.c
(fp2, find_pointer2): Remove.
(mio_component_ref): Don't forcedfully set the containing derived type
symbol for loading.  Remove unused argument.
(mio_ref): Update caller
(skip_list): New argument nest_level.  Initialize level with the new
argument.
(read_module): Add forced pointer components association for derived
type symbols.

testsuite/
PR fortran/58007
* gfortran.dg/unresolved_fixup_1.f90: New test.
* gfortran.dg/unresolved_fixup_2.f90: New test.


Added:
branches/gcc-4_8-branch/gcc/testsuite/gfortran.dg/unresolved_fixup_1.f90
branches/gcc-4_8-branch/gcc/testsuite/gfortran.dg/unresolved_fixup_2.f90
Modified:
branches/gcc-4_8-branch/gcc/fortran/ChangeLog
branches/gcc-4_8-branch/gcc/fortran/module.c
branches/gcc-4_8-branch/gcc/testsuite/ChangeLog


[Bug fortran/58007] [4.7/4.9 Regression] [OOP] ICE in free_pi_tree(): Unresolved fixup - resolve_fixups does not fixup component of __class_bsr_Bsr_matrix

2014-01-26 Thread mikael at gcc dot gnu.org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=58007

--- Comment #17 from Mikael Morin mikael at gcc dot gnu.org ---
Author: mikael
Date: Sun Jan 26 14:12:50 2014
New Revision: 207118

URL: http://gcc.gnu.org/viewcvs?rev=207118root=gccview=rev
Log:
fortran/
PR fortran/58007
* module.c (read_module): Assert for component name correctness.


Modified:
trunk/gcc/fortran/ChangeLog
trunk/gcc/fortran/module.c


[Bug fortran/58007] [4.7/4.9 Regression] [OOP] ICE in free_pi_tree(): Unresolved fixup - resolve_fixups does not fixup component of __class_bsr_Bsr_matrix

2014-01-26 Thread mikael at gcc dot gnu.org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=58007

--- Comment #18 from Mikael Morin mikael at gcc dot gnu.org ---
Author: mikael
Date: Sun Jan 26 14:49:47 2014
New Revision: 207119

URL: http://gcc.gnu.org/viewcvs?rev=207119root=gccview=rev
Log:
fortran/
PR fortran/58007
* module.c (fp2, find_pointer2): Remove.
(mio_component_ref): Don't forcedfully set the containing derived type
symbol for loading.  Remove unused argument.
(mio_ref): Update caller
(skip_list): New argument nest_level.  Initialize level with the new
argument.
(read_module): Add forced pointer components association for derived
type symbols.

testsuite/
PR fortran/58007
* gfortran.dg/unresolved_fixup_1.f90: New test.
* gfortran.dg/unresolved_fixup_2.f90: New test.


Added:
branches/gcc-4_7-branch/gcc/testsuite/gfortran.dg/unresolved_fixup_1.f90
branches/gcc-4_7-branch/gcc/testsuite/gfortran.dg/unresolved_fixup_2.f90
Modified:
branches/gcc-4_7-branch/gcc/fortran/ChangeLog
branches/gcc-4_7-branch/gcc/fortran/module.c
branches/gcc-4_7-branch/gcc/testsuite/ChangeLog


[Bug fortran/58007] [4.7/4.9 Regression] [OOP] ICE in free_pi_tree(): Unresolved fixup - resolve_fixups does not fixup component of __class_bsr_Bsr_matrix

2014-01-26 Thread mikael at gcc dot gnu.org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=58007

Mikael Morin mikael at gcc dot gnu.org changed:

   What|Removed |Added

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

--- Comment #19 from Mikael Morin mikael at gcc dot gnu.org ---
Fixed for 4.7.4, 4.8.3 and 4.9.0.


[Bug fortran/58007] [4.7/4.9 Regression] [OOP] ICE in free_pi_tree(): Unresolved fixup - resolve_fixups does not fixup component of __class_bsr_Bsr_matrix

2014-01-18 Thread mikael at gcc dot gnu.org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=58007

--- Comment #13 from Mikael Morin mikael at gcc dot gnu.org ---
Author: mikael
Date: Sat Jan 18 20:05:25 2014
New Revision: 206759

URL: http://gcc.gnu.org/viewcvs?rev=206759root=gccview=rev
Log:
fortran/
PR fortran/58007
* module.c (MOD_VERSION): Bump.
(fp2, find_pointer2): Remove.
(mio_component_ref): Don't forcedfully set the containing derived type
symbol for loading.  Remove unused argument.
(mio_ref): Update caller
(mio_symbol): Dump component list earlier.
(skip_list): New argument nest_level.  Initialize level with the new
argument.
(read_module): Add forced pointer components association for derived
type symbols.

testsuite/
PR fortran/58007
* gfortran.dg/unresolved_fixup_1.f90: New test.
* gfortran.dg/unresolved_fixup_2.f90: New test.


Added:
trunk/gcc/testsuite/gfortran.dg/unresolved_fixup_1.f90
trunk/gcc/testsuite/gfortran.dg/unresolved_fixup_2.f90
Modified:
trunk/gcc/fortran/ChangeLog
trunk/gcc/fortran/module.c
trunk/gcc/testsuite/ChangeLog


[Bug fortran/58007] [4.7/4.9 Regression] [OOP] ICE in free_pi_tree(): Unresolved fixup - resolve_fixups does not fixup component of __class_bsr_Bsr_matrix

2014-01-18 Thread mikael at gcc dot gnu.org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=58007

Mikael Morin mikael at gcc dot gnu.org changed:

   What|Removed |Added

URL||http://gcc.gnu.org/ml/gcc-p
   ||atches/2014-01/msg00038.htm
   ||l
   Assignee|unassigned at gcc dot gnu.org  |mikael at gcc dot 
gnu.org

--- Comment #14 from Mikael Morin mikael at gcc dot gnu.org ---
I have just committed a fix on trunk, so I may as well take the bug.
For reference, the patch submission was at:
http://gcc.gnu.org/ml/gcc-patches/2014-01/msg00038.html


[Bug fortran/58007] [4.7/4.9 Regression] [OOP] ICE in free_pi_tree(): Unresolved fixup - resolve_fixups does not fixup component of __class_bsr_Bsr_matrix

2014-01-18 Thread mikael at gcc dot gnu.org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=58007

Mikael Morin mikael at gcc dot gnu.org changed:

   What|Removed |Added

 Status|NEW |ASSIGNED

--- Comment #15 from Mikael Morin mikael at gcc dot gnu.org ---
(In reply to Mikael Morin from comment #14)
 so I may as well take the bug.

Grrr! ASSIGNED, I said.


[Bug fortran/58007] [4.7/4.9 Regression] [OOP] ICE in free_pi_tree(): Unresolved fixup - resolve_fixups does not fixup component of __class_bsr_Bsr_matrix

2013-12-25 Thread mikael at gcc dot gnu.org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=58007

Mikael Morin mikael at gcc dot gnu.org changed:

   What|Removed |Added

  Attachment #30656|0   |1
is obsolete||

--- Comment #12 from Mikael Morin mikael at gcc dot gnu.org ---
Created attachment 31511
  -- http://gcc.gnu.org/bugzilla/attachment.cgi?id=31511action=edit
Patch fixing both comment #4 and comment #6

Let's hope it's not too late to put this under the christmas tree.

This patch does forces pointer association of components of a derived type
which has already been loaded.
I don't like it much, as it is dependent on the order of the symbol contents as
specified by the mio_symbol code.


[Bug fortran/58007] [4.7/4.9 Regression] [OOP] ICE in free_pi_tree(): Unresolved fixup - resolve_fixups does not fixup component of __class_bsr_Bsr_matrix

2013-12-23 Thread townsend at astro dot wisc.edu
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=58007

--- Comment #11 from Rich Townsend townsend at astro dot wisc.edu ---
#6 fails with 4.9.0 (svn rev. 206179), on both OS X and Linux.


[Bug fortran/58007] [4.7/4.9 Regression] [OOP] ICE in free_pi_tree(): Unresolved fixup - resolve_fixups does not fixup component of __class_bsr_Bsr_matrix

2013-12-19 Thread rguenth at gcc dot gnu.org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=58007

Richard Biener rguenth at gcc dot gnu.org changed:

   What|Removed |Added

   Target Milestone|--- |4.7.4


[Bug fortran/58007] [4.7/4.9 Regression] [OOP] ICE in free_pi_tree(): Unresolved fixup - resolve_fixups does not fixup component of __class_bsr_Bsr_matrix

2013-12-19 Thread rguenth at gcc dot gnu.org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=58007

Richard Biener rguenth at gcc dot gnu.org changed:

   What|Removed |Added

   Priority|P3  |P4


[Bug fortran/58007] [4.7/4.9 Regression] [OOP] ICE in free_pi_tree(): Unresolved fixup - resolve_fixups does not fixup component of __class_bsr_Bsr_matrix

2013-12-06 Thread janus at gcc dot gnu.org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=58007

janus at gcc dot gnu.org changed:

   What|Removed |Added

  Known to work||4.6.4, 4.8.1
Summary|[OOP] ICE in|[4.7/4.9 Regression] [OOP]
   |free_pi_tree(): Unresolved  |ICE in free_pi_tree():
   |fixup - resolve_fixups does |Unresolved fixup -
   |not fixup component of  |resolve_fixups does not
   |__class_bsr_Bsr_matrix  |fixup component of
   ||__class_bsr_Bsr_matrix
  Known to fail||4.7.3, 4.9.0

--- Comment #10 from janus at gcc dot gnu.org ---
Marking this one as a regression, since comment 6 seems to fail only with 4.7
and trunk.