[Bug fortran/60341] [4.7/4.8/4.9 Regression] ICE compiling Nonmem 6.2.0

2014-03-02 Thread mikael at gcc dot gnu.org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=60341 --- Comment #12 from Mikael Morin --- Author: mikael Date: Sun Mar 2 18:49:18 2014 New Revision: 208259 URL: http://gcc.gnu.org/viewcvs?rev=208259&root=gcc&view=rev Log: fortran/ PR fortran/60341 * frontend-passes.c (optimize_com

[Bug fortran/60341] [4.7/4.8/4.9 Regression] ICE compiling Nonmem 6.2.0

2014-03-02 Thread mikael at gcc dot gnu.org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=60341 Mikael Morin changed: What|Removed |Added Status|ASSIGNED|RESOLVED Resolution|---

[Bug fortran/60370] TRANSPOSE on rhs of allocatable array assignment gives error with -fcheck=bounds

2014-03-03 Thread mikael at gcc dot gnu.org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=60370 --- Comment #6 from Mikael Morin --- (In reply to janus from comment #5) > However, it also seems to introduce another ordering issue: In particular > the bounds of 'b' are now set as > > b.dim[1].lbound = 1; > b.dim[1].ubound = (

[Bug fortran/60392] Problem with TRANSPOSE and CONTIGUOUS dummy arguments

2014-03-05 Thread mikael at gcc dot gnu.org
||2014-03-05 CC||mikael at gcc dot gnu.org Ever confirmed|0 |1 --- Comment #3 from Mikael Morin --- for both calls (my_mul and my_mul_cont) we create a descriptor for a (with transposed bounds as expected) and

[Bug fortran/60392] Problem with TRANSPOSE and CONTIGUOUS dummy arguments

2014-03-05 Thread mikael at gcc dot gnu.org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=60392 --- Comment #4 from Mikael Morin --- (In reply to Mikael Morin from comment #3) > So we pass the transposed array with > transposed bounds -> we pass the original array. Not exactly in fact: In my_mul_cont, a.stride[0] == 2 and a.stride[1] == 1.

[Bug fortran/60392] Problem with TRANSPOSE and CONTIGUOUS dummy arguments

2014-03-07 Thread mikael at gcc dot gnu.org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=60392 --- Comment #5 from Mikael Morin --- Created attachment 32307 --> http://gcc.gnu.org/bugzilla/attachment.cgi?id=32307&action=edit preliminary patch This removes the difference between my_mul/my_mul_cont. However this is not yet correct, with th

[Bug fortran/60392] Problem with TRANSPOSE and CONTIGUOUS dummy arguments

2014-03-07 Thread mikael at gcc dot gnu.org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=60392 --- Comment #6 from Mikael Morin --- (In reply to Mikael Morin from comment #5) > This removes the difference between my_mul/my_mul_cont. > However this is not yet correct, with the patch the program output is: > Maybe it's correct after all. It'

[Bug fortran/60522] [4.7/4.8/4.9 Regression] WHERE construct causes an ICE in gfc_trans_where_2

2014-03-14 Thread mikael at gcc dot gnu.org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=60522 Mikael Morin changed: What|Removed |Added CC||mikael at gcc dot gnu.org --- Comment #3

[Bug fortran/60392] Problem with TRANSPOSE and CONTIGUOUS dummy arguments

2014-03-14 Thread mikael at gcc dot gnu.org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=60392 --- Comment #7 from Mikael Morin --- Author: mikael Date: Fri Mar 14 21:28:59 2014 New Revision: 208581 URL: http://gcc.gnu.org/viewcvs?rev=208581&root=gcc&view=rev Log: fortran/ PR fortran/60392 * trans-array.c (gfc_conv_array_pa

[Bug fortran/60392] Problem with TRANSPOSE and CONTIGUOUS dummy arguments

2014-03-14 Thread mikael at gcc dot gnu.org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=60392 --- Comment #8 from Mikael Morin --- Fixed on trunk.

[Bug fortran/55603] [F03] Memory leak with scalar allocatable function result

2015-09-25 Thread mikael at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=55603 --- Comment #10 from Mikael Morin --- Author: mikael Date: Fri Sep 25 20:28:33 2015 New Revision: 228151 URL: https://gcc.gnu.org/viewcvs?rev=228151&root=gcc&view=rev Log: Add PR fortran/55603 working test gcc/testsuite/ PR fortran/5560

[Bug fortran/37336] [F03] Finish derived-type finalization

2015-09-25 Thread mikael at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=37336 Bug 37336 depends on bug 55603, which changed state. Bug 55603 Summary: [F03] Memory leak with scalar allocatable function result https://gcc.gnu.org/bugzilla/show_bug.cgi?id=55603 What|Removed |Added --

[Bug fortran/55603] [F03] Memory leak with scalar allocatable function result

2015-09-25 Thread mikael at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=55603 Mikael Morin changed: What|Removed |Added Status|NEW |RESOLVED Known to work|

[Bug fortran/67721] New: deep copy missing when assigning a derived type constructor to an array

2015-09-26 Thread mikael at gcc dot gnu.org
: normal Priority: P3 Component: fortran Assignee: unassigned at gcc dot gnu.org Reporter: mikael at gcc dot gnu.org Target Milestone: --- In the following test, the assignment to v does a shallow copy of the rhs, so that the c2 components are shared among array

[Bug fortran/67721] deep copy missing when assigning a derived type constructor to an array

2015-09-26 Thread mikael at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=67721 Mikael Morin changed: What|Removed |Added Keywords||patch, wrong-code Known to fail|

[Bug fortran/67721] deep copy missing when assigning a derived type constructor to an array

2015-09-26 Thread mikael at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=67721 --- Comment #2 from Mikael Morin --- Author: mikael Date: Sat Sep 26 20:34:39 2015 New Revision: 228170 URL: https://gcc.gnu.org/viewcvs?rev=228170&root=gcc&view=rev Log: Fix missing deep copy when assigning a DT constructor to an array This ad

[Bug fortran/67721] deep copy missing when assigning a derived type constructor to an array

2015-10-01 Thread mikael at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=67721 --- Comment #4 from Mikael Morin --- (In reply to Dominique d'Humieres from comment #3) > Any plan to back port the fix to 5.3? Yes, Paul suggested it in his approval message. I'm doing it now.

[Bug fortran/67721] deep copy missing when assigning a derived type constructor to an array

2015-10-01 Thread mikael at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=67721 --- Comment #5 from Mikael Morin --- Author: mikael Date: Thu Oct 1 14:01:37 2015 New Revision: 228339 URL: https://gcc.gnu.org/viewcvs?rev=228339&root=gcc&view=rev Log: Fix missing deep copy when assigning a DT constructor to an array This ad

[Bug fortran/67721] deep copy missing when assigning a derived type constructor to an array

2015-10-01 Thread mikael at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=67721 --- Comment #6 from Mikael Morin --- Author: mikael Revision: 228170 Modified property: svn:log Modified: svn:log at Thu Oct 1 14:03:32 2015 -- --- svn:log (original) +

[Bug fortran/67721] deep copy missing when assigning a derived type constructor to an array

2015-10-01 Thread mikael at gcc dot gnu.org
|--- |FIXED Assignee|unassigned at gcc dot gnu.org |mikael at gcc dot gnu.org --- Comment #7 from Mikael Morin --- Fixed for 5.3 and 6.0. Closing

[Bug fortran/67818] [5 Regression] FAIL: libgomp.fortran/alloc-comp-[23].f90

2015-10-03 Thread mikael at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=67818 --- Comment #3 from Mikael Morin --- memcpy was fixed by Andre at r222477 on trunk. I think it fixes the regression, but I'm a little uncomfortable with backporting it, because of its medium size. So the question is whether r222477 should be back

[Bug fortran/67818] [5 Regression] FAIL: libgomp.fortran/alloc-comp-[23].f90

2015-10-03 Thread mikael at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=67818 --- Comment #4 from Mikael Morin --- (In reply to Mikael Morin from comment #3) > So the question is whether r222477 should be backported, or r228361 reverted. I have looked at the r222477 patch more in details, and it seems reasonable after all

[Bug fortran/67758] [GCC 6 regression] ICE on invalid use of COMMON

2015-10-04 Thread mikael at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=67758 --- Comment #5 from Mikael Morin --- Author: mikael Date: Sun Oct 4 12:30:16 2015 New Revision: 228457 URL: https://gcc.gnu.org/viewcvs?rev=228457&root=gcc&view=rev Log: Fix fortran common-related error recovery ICE. Upon reverting a symbol in

[Bug fortran/67758] [GCC 6 regression] ICE on invalid use of COMMON

2015-10-05 Thread mikael at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=67758 Mikael Morin changed: What|Removed |Added Assignee|kargl at gcc dot gnu.org |mikael at gcc dot gnu.org

[Bug fortran/67894] bounds of assumed-rank dummy argument not equal to actual argument

2015-10-09 Thread mikael at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=67894 --- Comment #8 from Mikael Morin --- (In reply to Dominique d'Humieres from comment #7) > AFAICT the (l\u)bound have been set by r190098. Mikael CCed. I think that revision just made scalarization work. For the calls with the dim= argument, I do

[Bug fortran/47469] Check whether arrayfunc_assign_needs_temporary misses TBP/PPC attributes

2015-10-14 Thread mikael at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=47469 --- Comment #4 from Mikael Morin --- (In reply to Dominique d'Humieres from comment #3) > > Tobias' version is prettier! Does it apply without regressions? > > Obviously the patch in comment 0 no longer applies. > > Now finding 'gfc_expr_attr (

[Bug fortran/59678] [F03] Segfault on equalizing variables of a complex derived type

2015-10-18 Thread mikael at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=59678 --- Comment #21 from Mikael Morin --- Author: mikael Date: Sun Oct 18 15:01:03 2015 New Revision: 228945 URL: https://gcc.gnu.org/viewcvs?rev=228945&root=gcc&view=rev Log: PR fortran/67721 PR fortran/67818 Backport from m

[Bug fortran/67818] [5 Regression] FAIL: libgomp.fortran/alloc-comp-[23].f90

2015-10-18 Thread mikael at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=67818 --- Comment #9 from Mikael Morin --- Author: mikael Date: Sun Oct 18 15:01:03 2015 New Revision: 228945 URL: https://gcc.gnu.org/viewcvs?rev=228945&root=gcc&view=rev Log: PR fortran/67721 PR fortran/67818 Backport from ma

[Bug fortran/65841] Seg fault on intrinsic assignment to allocatable derived type with allocatable component

2015-10-18 Thread mikael at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=65841 --- Comment #7 from Mikael Morin --- Author: mikael Date: Sun Oct 18 15:01:03 2015 New Revision: 228945 URL: https://gcc.gnu.org/viewcvs?rev=228945&root=gcc&view=rev Log: PR fortran/67721 PR fortran/67818 Backport from ma

[Bug fortran/67721] deep copy missing when assigning a derived type constructor to an array

2015-10-18 Thread mikael at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=67721 --- Comment #9 from Mikael Morin --- Author: mikael Date: Sun Oct 18 15:01:03 2015 New Revision: 228945 URL: https://gcc.gnu.org/viewcvs?rev=228945&root=gcc&view=rev Log: PR fortran/67721 PR fortran/67818 Backport from ma

[Bug fortran/67758] [6 Regression] ICE on invalid use of COMMON

2015-10-18 Thread mikael at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=67758 --- Comment #8 from Mikael Morin --- Author: mikael Date: Sun Oct 18 17:17:21 2015 New Revision: 228947 URL: https://gcc.gnu.org/viewcvs?rev=228947&root=gcc&view=rev Log: Fix common-related error recovery ICE. Fix an inconsistent state, between

[Bug fortran/67758] [6 Regression] ICE on invalid use of COMMON

2015-10-18 Thread mikael at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=67758 Mikael Morin changed: What|Removed |Added Status|NEW |RESOLVED Resolution|---

[Bug fortran/67933] [4.9/5/6 Regression] ICE for array of a derived type with allocatable class in derived type object

2015-10-18 Thread mikael at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=67933 Mikael Morin changed: What|Removed |Added CC||mikael at gcc dot gnu.org

[Bug fortran/67900] [4.9/5/6 Regression] Interface bug: Binding parameters to C causes a compiler segmentation fault.

2015-10-18 Thread mikael at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=67900 Mikael Morin changed: What|Removed |Added CC||mikael at gcc dot gnu.org --- Comment #4

[Bug fortran/67818] [5 Regression] FAIL: libgomp.fortran/alloc-comp-[23].f90

2015-10-19 Thread mikael at gcc dot gnu.org
|--- |FIXED Assignee|unassigned at gcc dot gnu.org |mikael at gcc dot gnu.org --- Comment #10 from Mikael Morin --- (In reply to H.J. Lu from comment #8) > Yes, r222477 fixed the regression on gcc-5-branch without any new > testsuite failures. The patch ha

[Bug fortran/67721] deep copy missing when assigning a derived type constructor to an array

2015-10-19 Thread mikael at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=67721 Mikael Morin changed: What|Removed |Added Status|REOPENED|RESOLVED Resolution|---

[Bug fortran/67933] [4.9/5/6 Regression] ICE for array of a derived type with allocatable class in derived type object

2015-10-20 Thread mikael at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=67933 --- Comment #4 from Mikael Morin --- (In reply to Paul Thomas from comment #3) > (In reply to Mikael Morin from comment #2) > > The generated finalization wrapper has code doing > > deallocate(ptr2%classes(:)%class_var) > > where ptr2 is a poin

[Bug fortran/67171] [6 regression] sourced allocation

2015-10-21 Thread mikael at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=67171 Mikael Morin changed: What|Removed |Added CC||mikael at gcc dot gnu.org --- Comment #7

[Bug fortran/68111] [6 Regression] 465.tonto in SPEC CPU 2006 failed to build

2015-10-27 Thread mikael at gcc dot gnu.org
||mikael at gcc dot gnu.org Resolution|--- |DUPLICATE --- Comment #1 from Mikael Morin --- Dup. *** This bug has been marked as a duplicate of bug 68108 ***

[Bug fortran/68108] [6 regression] erroneous error message 'scalar integer expression expected'

2015-10-27 Thread mikael at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=68108 Mikael Morin changed: What|Removed |Added CC||hjl.tools at gmail dot com --- Comment #2

[Bug fortran/68108] [6 regression] erroneous error message 'scalar integer expression expected'

2015-10-27 Thread mikael at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=68108 Mikael Morin changed: What|Removed |Added CC||mikael at gcc dot gnu.org --- Comment #3

[Bug fortran/47266] Optimization: Declare PRIVATE module procedures as "TREE_PUBLIC = 0" ("static function")

2015-11-13 Thread mikael at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=47266 --- Comment #9 from Mikael Morin --- (In reply to Dominique d'Humieres from comment #8) > In comment 6 > > * gfortran.dg/warn_unused_function_2.f90: New test. > > should be > > * gfortran.dg/module_private_2.f90: New test. > > The

[Bug libfortran/68347] Multiple build errors in libgfortran

2015-11-14 Thread mikael at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=68347 Mikael Morin changed: What|Removed |Added CC||mikael at gcc dot gnu.org --- Comment #5

[Bug fortran/68401] improve 'Allocation would exceed memory limit'

2015-11-19 Thread mikael at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=68401 Mikael Morin changed: What|Removed |Added CC||mikael at gcc dot gnu.org --- Comment #5

[Bug fortran/68401] improve 'Allocation would exceed memory limit'

2015-11-19 Thread mikael at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=68401 --- Comment #7 from Mikael Morin --- (In reply to Joost VandeVondele from comment #6) > (In reply to Mikael Morin from comment #5) > > (In reply to Joost VandeVondele from comment #4) > > > In the original post I suggested that I already looked a

[Bug fortran/61831] [4.9/ 5 Regression] runtime error: pointer being freed was not allocated

2015-01-25 Thread mikael at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=61831 --- Comment #36 from Mikael Morin --- >[...] > (*(struct string_t[1] * restrict) atmp.7.data)[0] = prt_in; prt_in.chars.data is copied to atmp.7.data[0]. >[...] >while (1) > { >[...] >D.3430 = (*(struct stri

[Bug fortran/62044] [4.8/4.9/5 Regression] ICE in USE statement with RENAME for extended derived type

2015-01-25 Thread mikael at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=62044 --- Comment #4 from Mikael Morin --- Author: mikael Date: Sun Jan 25 20:30:53 2015 New Revision: 220097 URL: https://gcc.gnu.org/viewcvs?rev=220097&root=gcc&view=rev Log: fortran/ PR fortran/62044 * decl.c (gfc_match_derived_decl): Don't

[Bug fortran/62044] [4.8/4.9 Regression] ICE in USE statement with RENAME for extended derived type

2015-01-25 Thread mikael at gcc dot gnu.org
||mikael at gcc dot gnu.org Resolution|--- |FIXED Summary|[4.8/4.9/5 Regression] ICE |[4.8/4.9 Regression] ICE in |in USE statement with |USE statement with RENAME |RENAME for extended derived

[Bug fortran/62044] [4.8/4.9 Regression] ICE in USE statement with RENAME for extended derived type

2015-01-26 Thread mikael at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=62044 Mikael Morin changed: What|Removed |Added Status|RESOLVED|REOPENED Resolution|FIXED

[Bug fortran/62044] [4.8/4.9 Regression] ICE in USE statement with RENAME for extended derived type

2015-01-26 Thread mikael at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=62044 --- Comment #9 from Mikael Morin --- Created attachment 34571 --> https://gcc.gnu.org/bugzilla/attachment.cgi?id=34571&action=edit segregate derived type namespaces from regular namespaces To convince yourself that sym_root fields are never us

[Bug fortran/63744] [4.8/4.9/5 Regression] Duplicate use-statement causes error

2015-02-03 Thread mikael at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=63744 Mikael Morin changed: What|Removed |Added CC||mikael at gcc dot gnu.org --- Comment #7

[Bug fortran/59765] [4.9/5 Regression] [OOP] ICE on valid with finalizable array components

2015-02-03 Thread mikael at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=59765 Mikael Morin changed: What|Removed |Added CC||mikael at gcc dot gnu.org --- Comment

[Bug fortran/64952] Missing temporary in assignment from elemental function

2015-02-06 Thread mikael at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=64952 Mikael Morin changed: What|Removed |Added CC||mikael at gcc dot gnu.org --- Comment #2

[Bug fortran/63744] [4.8/4.9/5 Regression] Duplicate use-statement causes error

2015-02-06 Thread mikael at gcc dot gnu.org
|ASSIGNED Assignee|unassigned at gcc dot gnu.org |mikael at gcc dot gnu.org --- Comment #8 from Mikael Morin --- testing the patch

[Bug fortran/61138] [4.8/4.9/5 Regression] Wrong code with pointer-bounds remapping

2015-02-06 Thread mikael at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=61138 --- Comment #6 from Mikael Morin --- (In reply to Dominique d'Humieres from comment #1) > Likely r190641. Yes (In reply to Mikael Morin from comment #2) > gfc_trans_pointer_assignment sets lse.descriptor_only before calling > gfc_conv_expr_des

[Bug fortran/64952] Missing temporary in assignment from elemental function

2015-02-08 Thread mikael at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=64952 --- Comment #4 from Mikael Morin --- Hello Paul, setting potentially_aliased should be done inside gfc_walk_elemental_function_args, as the ss argument may be returned unmodified. In fact, I think it's better to do all the trans-array.c code ins

[Bug fortran/63744] [4.8/4.9/5 Regression] Duplicate use-statement causes error

2015-02-08 Thread mikael at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=63744 --- Comment #9 from Mikael Morin --- Author: mikael Date: Sun Feb 8 14:18:16 2015 New Revision: 220515 URL: https://gcc.gnu.org/viewcvs?rev=220515&root=gcc&view=rev Log: Use the local name instead of the original name in the check for name con

[Bug fortran/64973] New: Duplicate use-statements could be diagnosed

2015-02-08 Thread mikael at gcc dot gnu.org
: fortran Assignee: unassigned at gcc dot gnu.org Reporter: mikael at gcc dot gnu.org This is a followup to PR63744 and was suggested in: https://gcc.gnu.org/ml/fortran/2015-02/msg00036.htm We accept duplicate use-rename statements like: use m, only: A => X use m, only: A =

[Bug fortran/64973] Duplicate use-statements could be diagnosed

2015-02-08 Thread mikael at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=64973 Mikael Morin changed: What|Removed |Added Keywords||diagnostic Priority|P3

[Bug fortran/61138] [4.8/4.9/5 Regression] Wrong code with pointer-bounds remapping

2015-02-09 Thread mikael at gcc dot gnu.org
||https://gcc.gnu.org/ml/gcc- ||patches/2015-02/msg00531.ht ||ml Assignee|unassigned at gcc dot gnu.org |mikael at gcc dot gnu.org --- Comment #7 from Mikael Morin

[Bug fortran/64980] [5.0 Regression] ICE in trans-expr.c

2015-02-09 Thread mikael at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=64980 Mikael Morin changed: What|Removed |Added CC||mikael at gcc dot gnu.org --- Comment #4

[Bug fortran/64980] [5 Regression] ICE in trans-expr.c

2015-02-09 Thread mikael at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=64980 --- Comment #5 from Mikael Morin --- (In reply to Mikael Morin from comment #4) > This cures it: Another way to cure it: Index: trans-expr.c === --- trans-expr.c(révision 22051

[Bug fortran/60898] [4.8/4.9/5 Regression] model compile error with gfortran 4.7 and gcc 4.9

2015-02-10 Thread mikael at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=60898 Mikael Morin changed: What|Removed |Added CC||mikael at gcc dot gnu.org --- Comment

[Bug fortran/63744] [4.8/4.9/5 Regression] Duplicate use-statement causes error

2015-02-13 Thread mikael at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=63744 --- Comment #10 from Mikael Morin --- Author: mikael Date: Fri Feb 13 18:48:35 2015 New Revision: 220690 URL: https://gcc.gnu.org/viewcvs?rev=220690&root=gcc&view=rev Log: Use the local name instead of the original name in the check for name co

[Bug fortran/63744] [4.8/4.9/5 Regression] Duplicate use-statement causes error

2015-02-13 Thread mikael at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=63744 --- Comment #11 from Mikael Morin --- Author: mikael Date: Fri Feb 13 19:33:27 2015 New Revision: 220692 URL: https://gcc.gnu.org/viewcvs?rev=220692&root=gcc&view=rev Log: Use the local name instead of the original name in the check for name co

[Bug fortran/64980] [5 Regression] ICE in trans-expr.c

2015-02-13 Thread mikael at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=64980 --- Comment #9 from Mikael Morin --- (In reply to Bernd Edlinger from comment #8) > I will post this new patch, which combines Mikael's patch and > fixes class_41.f03 and these test cases, Actually, my patch was not supposed to be a real fix, ra

[Bug fortran/63744] [4.8/4.9/5 Regression] Duplicate use-statement causes error

2015-02-13 Thread mikael at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=63744 Mikael Morin changed: What|Removed |Added Status|ASSIGNED|RESOLVED Resolution|---

[Bug fortran/61831] [4.9/ 5 Regression] runtime error: pointer being freed was not allocated

2015-02-14 Thread mikael at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=61831 --- Comment #38 from Mikael Morin --- (In reply to Dominique d'Humieres from comment #37) > > >[...] > > >if (D.3430.chars.data != 0B) > > > { > > >__builtin_free ((void *) D.3430.chars.data); > > > }

[Bug fortran/64980] [5 Regression] ICE in trans-expr.c

2015-02-14 Thread mikael at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=64980 --- Comment #11 from Mikael Morin --- Created attachment 34755 --> https://gcc.gnu.org/bugzilla/attachment.cgi?id=34755&action=edit Another proposed fix (In reply to Bernd Edlinger from comment #10) > Yes, but I have no idea how to know how t

[Bug fortran/60898] [4.8/4.9/5 Regression] model compile error with gfortran 4.7 and gcc 4.9

2015-02-14 Thread mikael at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=60898 --- Comment #18 from Mikael Morin --- Created attachment 34756 --> https://gcc.gnu.org/bugzilla/attachment.cgi?id=34756&action=edit patch fixing comment #7 (In reply to Mikael Morin from comment #17) > Using Tobias' analysis in comment #14, th

[Bug fortran/64980] [5 Regression] ICE in trans-expr.c

2015-02-14 Thread mikael at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=64980 --- Comment #13 from Mikael Morin --- Created attachment 34757 --> https://gcc.gnu.org/bugzilla/attachment.cgi?id=34757&action=edit Yet another attempt using TYPE_CANONICAL this tries to remove any use of VIEW_CONVERT_EXPR by setting (well, tr

[Bug fortran/64980] [5 Regression] ICE in trans-expr.c

2015-02-14 Thread mikael at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=64980 --- Comment #14 from Mikael Morin --- Bernd, do you have a reliable way to test a patch, checking for aliasing violations?

[Bug fortran/60898] [4.8/4.9/5 Regression] model compile error with gfortran 4.7 and gcc 4.9

2015-02-14 Thread mikael at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=60898 --- Comment #20 from Mikael Morin --- (In reply to Dominique d'Humieres from comment #19) > > patch fixing comment #7 > > Confirmed, but it doesn't fix the test in comment 2 (with the addition in > comment 10). Confirmed; for some reason it wor

[Bug fortran/60898] [4.8/4.9/5 Regression] model compile error with gfortran 4.7 and gcc 4.9

2015-02-16 Thread mikael at gcc dot gnu.org
|ASSIGNED Assignee|unassigned at gcc dot gnu.org |mikael at gcc dot gnu.org --- Comment #22 from Mikael Morin --- patch posted at: https://gcc.gnu.org/ml/gcc-patches/2015-02/msg00917.html

[Bug fortran/61831] [4.9/ 5 Regression] runtime error: pointer being freed was not allocated

2015-02-22 Thread mikael at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=61831 --- Comment #39 from Mikael Morin --- (In reply to Mikael Morin from comment #38) > By the way, I'm not sure that it's at all correct to avoid deep copies. This seems to be safe: as long as the procedure is pure, there is no way it can modify its

[Bug fortran/61831] [4.9/ 5 Regression] runtime error: pointer being freed was not allocated

2015-02-23 Thread mikael at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=61831 --- Comment #40 from Mikael Morin --- (In reply to Mikael Morin from comment #39) > A function will do a deep copy, so I think the only problematic cases are > EXPR_ARRAY, and maybe EXPR_STRUCTURE. EXPR_STRUCTURE deep-copy variable components, a

[Bug fortran/61831] [4.9/ 5 Regression] runtime error: pointer being freed was not allocated

2015-02-23 Thread mikael at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=61831 --- Comment #41 from Mikael Morin --- Created attachment 34846 --> https://gcc.gnu.org/bugzilla/attachment.cgi?id=34846&action=edit extended testcase My work patch is: Index: trans-expr.c ==

[Bug fortran/61831] [4.9/ 5 Regression] runtime error: pointer being freed was not allocated

2015-02-23 Thread mikael at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=61831 --- Comment #42 from Mikael Morin --- (In reply to Mikael Morin from comment #41) > My work patch is: updated work patch below. The testcase passes with it, at the price of leaking memory Index: trans-expr.c

[Bug fortran/61831] [4.9/ 5 Regression] runtime error: pointer being freed was not allocated

2015-03-03 Thread mikael at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=61831 --- Comment #44 from Mikael Morin --- Created attachment 34942 --> https://gcc.gnu.org/bugzilla/attachment.cgi?id=34942&action=edit Better patch I'm not working on this, so I'm attaching the current patch in my work tree, before it's lost. If

[Bug fortran/60898] [4.8/4.9/5 Regression] model compile error with gfortran 4.7 and gcc 4.9

2015-03-08 Thread mikael at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=60898 --- Comment #23 from Mikael Morin --- Author: mikael Date: Sun Mar 8 11:52:51 2015 New Revision: 221262 URL: https://gcc.gnu.org/viewcvs?rev=221262&root=gcc&view=rev Log: PR fortran/60898 fortran/ * resolve.c (resolve_symbol): C

[Bug fortran/63640] move_alloc memory leak

2014-12-23 Thread mikael at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=63640 Mikael Morin changed: What|Removed |Added CC||mikael at gcc dot gnu.org --- Comment #6

[Bug fortran/61775] Allocatable array initialized by implied-do loop array constructor gives invalid memory reference

2014-07-11 Thread mikael at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=61775 Mikael Morin changed: What|Removed |Added CC||mikael at gcc dot gnu.org --- Comment #4

[Bug fortran/61780] [4.8/4.9/4.10 Regression] Wrong code when shifting elements of a multidimensional array

2014-07-12 Thread mikael at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=61780 Mikael Morin changed: What|Removed |Added CC||mikael at gcc dot gnu.org --- Comment #1

[Bug fortran/61780] [4.8/4.9/4.10 Regression] Wrong code when shifting elements of a multidimensional array

2014-07-13 Thread mikael at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=61780 --- Comment #5 from Mikael Morin --- (In reply to paul.richard.tho...@gmail.com from comment #3) > Dear Mikael, > > I didn't see your posting, which was about an hour before mine. At > least we came to the same conclusion! > Yes, that's good.

[Bug fortran/37131] inline matmul for small matrix sizes

2014-10-03 Thread mikael at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=37131 --- Comment #16 from Mikael Morin --- (In reply to Thomas Koenig from comment #15) > Hi Mikael, > > do you think you can do this using the scalarizer? > Not in its current state. And as I don't see the scalarizer being changed soon to allow it,

[Bug fortran/68560] [6 Regression] The test gfortran.dg/shape_8.f90 now fails when compiled with -flto

2015-12-03 Thread mikael at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=68560 Mikael Morin changed: What|Removed |Added CC||mikael at gcc dot gnu.org --- Comment #4

[Bug fortran/68560] [6 Regression] The test gfortran.dg/shape_8.f90 now fails when compiled with -flto

2015-12-03 Thread mikael at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=68560 --- Comment #6 from Mikael Morin --- (In reply to Dominique d'Humieres from comment #5) > > I think the problem is that it's actually working. The optional arguments > > are > > removed... When they are not present, the procedure declaration al

[Bug fortran/68560] [6 Regression] The test gfortran.dg/shape_8.f90 now fails when compiled with -flto

2015-12-03 Thread mikael at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=68560 --- Comment #8 from Mikael Morin --- Created attachment 36902 --> https://gcc.gnu.org/bugzilla/attachment.cgi?id=36902&action=edit Make the intrinsic procedure declarations less context-dependant I can't test this right now (bootstrap broken).

[Bug fortran/68560] [6 Regression] The test gfortran.dg/shape_8.f90 now fails when compiled with -flto

2015-12-04 Thread mikael at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=68560 --- Comment #11 from Mikael Morin --- (In reply to Dominique d'Humieres from comment #10) > >I can't test this right now (bootstrap broken). > > Sorry about that. Anything serious? > Looks like it's PR68540.

[Bug fortran/68560] [6 Regression] The test gfortran.dg/shape_8.f90 now fails when compiled with -flto

2015-12-05 Thread mikael at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=68560 --- Comment #13 from Mikael Morin --- (In reply to Dominique d'Humieres from comment #12) > The statement in fortran/trans-intrinsic.c > > gfc_copy_formal_args_intr (sym, expr->value.function.isym, > ignore_optiona

[Bug fortran/68560] [6 Regression] The test gfortran.dg/shape_8.f90 now fails when compiled with -flto

2015-12-05 Thread mikael at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=68560 --- Comment #15 from Mikael Morin --- (In reply to Dominique d'Humieres from comment #14) > > > The statement in fortran/trans-intrinsic.c > > > > > > gfc_copy_formal_args_intr (sym, expr->value.function.isym, > > >

[Bug fortran/68560] [6 Regression] The test gfortran.dg/shape_8.f90 now fails when compiled with -flto

2015-12-05 Thread mikael at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=68560 --- Comment #16 from Mikael Morin --- (In reply to Mikael Morin from comment #15) > (In reply to Dominique d'Humieres from comment #12) > > > Looks like it's PR68540. > > > > So it should work now? > > > I have yet to find the best way to insta

[Bug fortran/68560] [6 Regression] The test gfortran.dg/shape_8.f90 now fails when compiled with -flto

2015-12-06 Thread mikael at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=68560 Mikael Morin changed: What|Removed |Added Attachment #36902|0 |1 is obsolete|

[Bug fortran/68560] [6 Regression] The test gfortran.dg/shape_8.f90 now fails when compiled with -flto

2015-12-13 Thread mikael at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=68560 Mikael Morin changed: What|Removed |Added Attachment #36937|0 |1 is obsolete|

[Bug fortran/68560] [6 Regression] The test gfortran.dg/shape_8.f90 now fails when compiled with -flto

2015-12-13 Thread mikael at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=68560 --- Comment #19 from Mikael Morin --- (In reply to Mikael Morin from comment #18) > Lightly tested so far. It actually regress heavily.

[Bug fortran/68560] [6 Regression] The test gfortran.dg/shape_8.f90 now fails when compiled with -flto

2015-12-13 Thread mikael at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=68560 Mikael Morin changed: What|Removed |Added Attachment #37024|0 |1 is obsolete|

[Bug fortran/69080] No automatic deallocation of allocatable function results

2016-01-06 Thread mikael at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=69080 Mikael Morin changed: What|Removed |Added CC||mikael at gcc dot gnu.org --- Comment

[Bug fortran/66459] bogus warning 'w.offset' may be used uninitialized in this function [-Wmaybe-uninitialized]

2015-06-08 Thread mikael at gcc dot gnu.org
||2015-06-08 CC||mikael at gcc dot gnu.org Ever confirmed|0 |1 --- Comment #1 from Mikael Morin --- Confirmed. Possibly related bug: PR60500 (In reply to Mario Baumann from comment #0) > - -O0 works fine T

[Bug fortran/66549] [5/6 Regression] ICE on valid in OMP parallel region

2015-06-17 Thread mikael at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=66549 --- Comment #3 from Mikael Morin --- (In reply to Jakub Jelinek from comment #2) > r221586 in particular. There is no openmp change in that revision. :-/ --- comment_0.f90.003t.original.good2015-06-17 21:29:00.667012376 +0200 +++ comment_0.f

[Bug fortran/66549] [5/6 Regression] ICE on valid in OMP parallel region

2015-06-17 Thread mikael at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=66549 --- Comment #4 from Mikael Morin --- Created attachment 35796 --> https://gcc.gnu.org/bugzilla/attachment.cgi?id=35796&action=edit Draft patch

<    1   2   3   4   5   6   7   8   9   10   >