[Bug fortran/102620] [12 Regression] ICE in gfc_get_array_span, at fortran/trans-array.c:865 since r12-1233-gd514626ee2566c68
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
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
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
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
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
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
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.