Re: [patch, fortran] Fix PR 57023, packing for some array temporaries

2015-01-21 Thread Thomas Koenig
Hi Tobias, > Thomas Koenig wrote: >> This patch takes the approach that any array bound which >> contains a dummy variable which is not INTENT(IN) may be >> changed by the user, and that we cannot be assured that >> it will not be changed. > > How about sym->attr.flavor == FL_PARAMETER? I am not

Re: [patch, fortran] Fix PR 57023, packing for some array temporaries

2015-01-21 Thread Tobias Burnus
Hi Thomas, Thomas Koenig wrote: > This patch takes the approach that any array bound which > contains a dummy variable which is not INTENT(IN) may be > changed by the user, and that we cannot be assured that > it will not be changed. How about sym->attr.flavor == FL_PARAMETER? I am not sure whet

[patch, fortran] Fix PR 57023, packing for some array temporaries

2015-01-20 Thread Thomas Koenig
Hello world, this patch fixes a longstanding regression where an upper array bound and the upper bound of an array section compared equal (using gfc_dep_compare_expr), but they weren't because the value of the upper bound had been changed in the meantime. This led to gfc_full_array_ref_p to erron