Hi All, Committed as revision 257550. In the course of testing this patch, Dominique found some problems that turned out to be due to revision r257065. These latter were encapsulated in PR84155, which was fixed 257356. However, Richi pointed me in the direction of a much cleaner fix to this PR and it seems that the failing tests that Dominique found survive this patch :-)
Paul 2018-02-10 Paul Thomas <pa...@gcc.gnu.org> PR fortran/84141 PR fortran/84155 * trans-array.c (gfc_array_init_size): Revert the change made in revision 257356 setting the dtype. * trans-types.c (gfc_get_dtype): Do not use the cached dtype. Call gfc_get_dtype_rank_type every time. PR fortran/56691 * trans-array.c (gfc_conv_expr_descriptor): If the source array is a descriptor type, use its offset, removing the condition that is be a class expression. 2018-02-10 Paul Thomas <pa...@gcc.gnu.org> PR fortran/56691 * gfortran.dg/type_to_class_4.f03: New test. On 27 January 2018 at 12:41, Paul Richard Thomas <paul.richard.tho...@gmail.com> wrote: > I am worried that this fix seems to easy by half and so I am posting > it for approval, rather than committing it as obvious. I would be > obliged if somebody would test it thoroughly. > > Bootstraps and regtests on FC23/x86_64 - OK for trunk and 7 branch? > > Paul > > 2018-27-01 Paul Thomas <pa...@gcc.gnu.org> > > PR fortran/56691 > * trans-array.c (gfc_conv_expr_descriptor): If the source array > is a descriptor type, use its offset, removing the condition > that is be a class expression. > > 2018-27-01 Paul Thomas <pa...@gcc.gnu.org> > > PR fortran/56691 > * gfortran.dg/type_to_class_4.f03: New test. -- "If you can't explain it simply, you don't understand it well enough" - Albert Einstein
