[Bug fortran/102620] [12 Regression] ICE in gfc_get_array_span, at fortran/trans-array.c:865 since r12-1233-gd514626ee2566c68

2024-04-26 Thread anlauf at gcc dot gnu.org via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=102620

anlauf at gcc dot gnu.org changed:

   What|Removed |Added

 Resolution|--- |FIXED
 Status|WAITING |RESOLVED

--- Comment #13 from anlauf at gcc dot gnu.org ---
Also fixed on 12-branch:

commit r12-10398-gb482968801158116dd8ba3b15a4c29143b2a423a
Author: Paul Thomas 
Date:   Tue May 23 06:46:37 2023 +0100

Fortran: Fix assumed length chars and len inquiry [PR103716]

2023-05-23  Paul Thomas  

gcc/fortran
PR fortran/103716
* resolve.cc (gfc_resolve_ref): Conversion of array_ref into an
element should be done for all characters without a len expr,
not just deferred lens, and for integer expressions.
* trans-expr.cc (conv_inquiry): For len and kind inquiry refs,
set the se string_length to NULL_TREE.

gcc/testsuite/
PR fortran/103716
* gfortran.dg/pr103716.f90 : New test.

(cherry picked from commit 842a432b02238361ecc601d301ac400a7f30f4fa)


The ICE is gone.  A remaining accepts-invalid on 11-branch could be fixed
by backporting the above if needed.

Thanks for the report to Gerhard, and to Paul for the above patch.  Closing.

[Bug fortran/102620] [12 Regression] ICE in gfc_get_array_span, at fortran/trans-array.c:865 since r12-1233-gd514626ee2566c68

2024-04-26 Thread anlauf at gcc dot gnu.org via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=102620

--- Comment #12 from anlauf at gcc dot gnu.org ---
(In reply to Paul Thomas from comment #11)
> It would be splendid if you would backport the patch. In the last week or
> so, I have built up quite a list of backports to do, which I will attend to
> over the weekend.

Hi Paul,

I've checked and found that your r14-1082-g842a432b022383 also needs to
be backported to get all testcases here fixed on 12-branch.

When regtesting completes successfully, I will push it.

[Bug fortran/102620] [12 Regression] ICE in gfc_get_array_span, at fortran/trans-array.c:865 since r12-1233-gd514626ee2566c68

2024-04-25 Thread pault at gcc dot gnu.org via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=102620

--- Comment #11 from Paul Thomas  ---
(In reply to anlauf from comment #10)
> (In reply to Paul Thomas from comment #9)
> > (In reply to anlauf from comment #8)
> > > I get the same behavior at r13-8559 as 14-mainline.  There seems to be
> > > another commit that fixed it independently.
> > > 
> > > Removing 13-branch from the regression list.
> > 
> > Mark as fixed or backport fixes?
> 
> Either I did something wrong, or the bug reappeared on 13-branch...
> 
> Anyway, I tried backporting Andre's patch to 13- and 12-branch.
> Works fine and regtests fine.
> 
> How to proceed?
> 
> I can push those changes, so that we are finally done with this PR.

Hi Harald,

It would be splendid if you would backport the patch. In the last week or so, I
have built up quite a list of backports to do, which I will attend to over the
weekend.

We are down from 105 regressions on 26th March to 94 now, of which 13 are now
fixed on mainline. Since there are still some P1 regressions, I have been
prowling around looking for more low hanging edibles while there is still time.

Regards

Paul

[Bug fortran/102620] [12 Regression] ICE in gfc_get_array_span, at fortran/trans-array.c:865 since r12-1233-gd514626ee2566c68

2024-04-25 Thread anlauf at gcc dot gnu.org via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=102620

--- Comment #10 from anlauf at gcc dot gnu.org ---
(In reply to Paul Thomas from comment #9)
> (In reply to anlauf from comment #8)
> > I get the same behavior at r13-8559 as 14-mainline.  There seems to be
> > another commit that fixed it independently.
> > 
> > Removing 13-branch from the regression list.
> 
> Mark as fixed or backport fixes?

Either I did something wrong, or the bug reappeared on 13-branch...

Anyway, I tried backporting Andre's patch to 13- and 12-branch.
Works fine and regtests fine.

How to proceed?

I can push those changes, so that we are finally done with this PR.

[Bug fortran/102620] [12 Regression] ICE in gfc_get_array_span, at fortran/trans-array.c:865 since r12-1233-gd514626ee2566c68

2024-04-25 Thread pault at gcc dot gnu.org via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=102620

Paul Thomas  changed:

   What|Removed |Added

 CC||pault at gcc dot gnu.org

--- Comment #9 from Paul Thomas  ---
(In reply to anlauf from comment #8)
> I get the same behavior at r13-8559 as 14-mainline.  There seems to be
> another commit that fixed it independently.
> 
> Removing 13-branch from the regression list.

Mark as fixed or backport fixes?

Cheers

Paul

[Bug fortran/102620] [12 Regression] ICE in gfc_get_array_span, at fortran/trans-array.c:865 since r12-1233-gd514626ee2566c68

2024-04-02 Thread anlauf at gcc dot gnu.org via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=102620

anlauf at gcc dot gnu.org changed:

   What|Removed |Added

Summary|[12/13 Regression] ICE in   |[12 Regression] ICE in
   |gfc_get_array_span, at  |gfc_get_array_span, at
   |fortran/trans-array.c:865   |fortran/trans-array.c:865
   |since   |since
   |r12-1233-gd514626ee2566c68  |r12-1233-gd514626ee2566c68

--- Comment #8 from anlauf at gcc dot gnu.org ---
I get the same behavior at r13-8559 as 14-mainline.  There seems to be
another commit that fixed it independently.

Removing 13-branch from the regression list.

[Bug fortran/102620] [12 Regression] ICE in gfc_get_array_span, at fortran/trans-array.c:865 since r12-1233-gd514626ee2566c68

2021-10-11 Thread marxin at gcc dot gnu.org via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=102620

Martin Liška  changed:

   What|Removed |Added

Summary|[12 Regression] ICE in  |[12 Regression] ICE in
   |gfc_get_array_span, at  |gfc_get_array_span, at
   |fortran/trans-array.c:865   |fortran/trans-array.c:865
   ||since
   ||r12-1233-gd514626ee2566c68
 CC||jrfsousa at gcc dot gnu.org,
   ||marxin at gcc dot gnu.org

--- Comment #3 from Martin Liška  ---
Confirmed, started with r12-1233-gd514626ee2566c68.