* [Bug fortran/95352] ICE on select rank with assumed-size selector and lbound intrinsic
2020-05-27 0:34 [Bug fortran/95352] New: ICE on extra parenthesis jrfsousa at gmail dot com
@ 2020-05-27 0:41 ` jrfsousa at gmail dot com
2020-05-27 1:26 ` kargl at gcc dot gnu.org
` (6 subsequent siblings)
7 siblings, 0 replies; 9+ messages in thread
From: jrfsousa at gmail dot com @ 2020-05-27 0:41 UTC (permalink / raw)
To: gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=95352
--- Comment #1 from José Rui Faustino de Sousa <jrfsousa at gmail dot com> ---
On further investigation the ICE is not generated by the extra parenthesis, but
by the use of the lbound intrinsic.
^ permalink raw reply [flat|nested] 9+ messages in thread
* [Bug fortran/95352] ICE on select rank with assumed-size selector and lbound intrinsic
2020-05-27 0:34 [Bug fortran/95352] New: ICE on extra parenthesis jrfsousa at gmail dot com
2020-05-27 0:41 ` [Bug fortran/95352] ICE on select rank with assumed-size selector and lbound intrinsic jrfsousa at gmail dot com
@ 2020-05-27 1:26 ` kargl at gcc dot gnu.org
2020-05-27 8:34 ` marxin at gcc dot gnu.org
` (5 subsequent siblings)
7 siblings, 0 replies; 9+ messages in thread
From: kargl at gcc dot gnu.org @ 2020-05-27 1:26 UTC (permalink / raw)
To: gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=95352
kargl at gcc dot gnu.org changed:
What |Removed |Added
----------------------------------------------------------------------------
CC| |kargl at gcc dot gnu.org
Priority|P3 |P4
--- Comment #2 from kargl at gcc dot gnu.org ---
(In reply to José Rui Faustino de Sousa from comment #1)
> On further investigation the ICE is not generated by the extra parenthesis,
> but by the use of the lbound intrinsic.
Index: gcc/fortran/simplify.c
===================================================================
--- gcc/fortran/simplify.c (revision 280157)
+++ gcc/fortran/simplify.c (working copy)
@@ -4019,7 +4023,7 @@ simplify_bound_dim (gfc_expr *array, gfc_expr *kind, i
|| (coarray && d == as->rank + as->corank
&& (!upper || flag_coarray == GFC_FCOARRAY_SINGLE)))
{
- if (as->lower[d-1]->expr_type == EXPR_CONSTANT)
+ if (as->lower[d-1] && as->lower[d-1]->expr_type == EXPR_CONSTANT)
{
gfc_free_expr (result);
return gfc_copy_expr (as->lower[d-1]);
^ permalink raw reply [flat|nested] 9+ messages in thread
* [Bug fortran/95352] ICE on select rank with assumed-size selector and lbound intrinsic
2020-05-27 0:34 [Bug fortran/95352] New: ICE on extra parenthesis jrfsousa at gmail dot com
2020-05-27 0:41 ` [Bug fortran/95352] ICE on select rank with assumed-size selector and lbound intrinsic jrfsousa at gmail dot com
2020-05-27 1:26 ` kargl at gcc dot gnu.org
@ 2020-05-27 8:34 ` marxin at gcc dot gnu.org
2020-08-20 23:15 ` jrfsousa at gmail dot com
` (4 subsequent siblings)
7 siblings, 0 replies; 9+ messages in thread
From: marxin at gcc dot gnu.org @ 2020-05-27 8:34 UTC (permalink / raw)
To: gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=95352
Martin Liška <marxin at gcc dot gnu.org> changed:
What |Removed |Added
----------------------------------------------------------------------------
CC| |marxin at gcc dot gnu.org
Status|UNCONFIRMED |NEW
Last reconfirmed| |2020-05-27
Ever confirmed|0 |1
--- Comment #3 from Martin Liška <marxin at gcc dot gnu.org> ---
Started to ICE with r10-2912-g70570ec192745095.
^ permalink raw reply [flat|nested] 9+ messages in thread
* [Bug fortran/95352] ICE on select rank with assumed-size selector and lbound intrinsic
2020-05-27 0:34 [Bug fortran/95352] New: ICE on extra parenthesis jrfsousa at gmail dot com
` (2 preceding siblings ...)
2020-05-27 8:34 ` marxin at gcc dot gnu.org
@ 2020-08-20 23:15 ` jrfsousa at gmail dot com
2020-08-21 0:42 ` sgk at troutmask dot apl.washington.edu
` (3 subsequent siblings)
7 siblings, 0 replies; 9+ messages in thread
From: jrfsousa at gmail dot com @ 2020-08-20 23:15 UTC (permalink / raw)
To: gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=95352
--- Comment #4 from José Rui Faustino de Sousa <jrfsousa at gmail dot com> ---
I have tested the patch posted by Steve Kargl and it seems to do the trick.
Can I do anything to get this going?
Best regards,
José Rui
^ permalink raw reply [flat|nested] 9+ messages in thread
* [Bug fortran/95352] ICE on select rank with assumed-size selector and lbound intrinsic
2020-05-27 0:34 [Bug fortran/95352] New: ICE on extra parenthesis jrfsousa at gmail dot com
` (3 preceding siblings ...)
2020-08-20 23:15 ` jrfsousa at gmail dot com
@ 2020-08-21 0:42 ` sgk at troutmask dot apl.washington.edu
2020-08-21 10:35 ` jrfsousa at gmail dot com
` (2 subsequent siblings)
7 siblings, 0 replies; 9+ messages in thread
From: sgk at troutmask dot apl.washington.edu @ 2020-08-21 0:42 UTC (permalink / raw)
To: gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=95352
--- Comment #5 from Steve Kargl <sgk at troutmask dot apl.washington.edu> ---
On Thu, Aug 20, 2020 at 11:15:27PM +0000, jrfsousa at gmail dot com wrote:
> https://gcc.gnu.org/bugzilla/show_bug.cgi?id=95352
>
> --- Comment #4 from José Rui Faustino de Sousa <jrfsousa at gmail dot com> ---
> I have tested the patch posted by Steve Kargl and it seems to do the trick.
>
> Can I do anything to get this going?
>
I don't use git, so have way to move forward. If
you want to package the patch and add the testcase,
I would appreciate it.
^ permalink raw reply [flat|nested] 9+ messages in thread
* [Bug fortran/95352] ICE on select rank with assumed-size selector and lbound intrinsic
2020-05-27 0:34 [Bug fortran/95352] New: ICE on extra parenthesis jrfsousa at gmail dot com
` (4 preceding siblings ...)
2020-08-21 0:42 ` sgk at troutmask dot apl.washington.edu
@ 2020-08-21 10:35 ` jrfsousa at gmail dot com
2020-08-21 15:59 ` sgk at troutmask dot apl.washington.edu
2020-09-01 13:17 ` jrfsousa at gmail dot com
7 siblings, 0 replies; 9+ messages in thread
From: jrfsousa at gmail dot com @ 2020-08-21 10:35 UTC (permalink / raw)
To: gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=95352
José Rui Faustino de Sousa <jrfsousa at gmail dot com> changed:
What |Removed |Added
----------------------------------------------------------------------------
Keywords| |patch
--- Comment #6 from José Rui Faustino de Sousa <jrfsousa at gmail dot com> ---
Done!
https://gcc.gnu.org/pipermail/fortran/2020-August/054908.html
Thank you very much.
Best regards,
José Rui
^ permalink raw reply [flat|nested] 9+ messages in thread
* [Bug fortran/95352] ICE on select rank with assumed-size selector and lbound intrinsic
2020-05-27 0:34 [Bug fortran/95352] New: ICE on extra parenthesis jrfsousa at gmail dot com
` (5 preceding siblings ...)
2020-08-21 10:35 ` jrfsousa at gmail dot com
@ 2020-08-21 15:59 ` sgk at troutmask dot apl.washington.edu
2020-09-01 13:17 ` jrfsousa at gmail dot com
7 siblings, 0 replies; 9+ messages in thread
From: sgk at troutmask dot apl.washington.edu @ 2020-08-21 15:59 UTC (permalink / raw)
To: gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=95352
--- Comment #7 from Steve Kargl <sgk at troutmask dot apl.washington.edu> ---
On Fri, Aug 21, 2020 at 10:35:27AM +0000, jrfsousa at gmail dot com wrote:
> Done!
>
> https://gcc.gnu.org/pipermail/fortran/2020-August/054908.html
>
> Thank you very much.
>
Thanks for submitting. If no one reviews the patch in the
next day or so, I think you can commit it.
BTW, for the git log and ChangeLog, I use kargl@gcc.gnu.org
not my sgk@troutmask address.
^ permalink raw reply [flat|nested] 9+ messages in thread
* [Bug fortran/95352] ICE on select rank with assumed-size selector and lbound intrinsic
2020-05-27 0:34 [Bug fortran/95352] New: ICE on extra parenthesis jrfsousa at gmail dot com
` (6 preceding siblings ...)
2020-08-21 15:59 ` sgk at troutmask dot apl.washington.edu
@ 2020-09-01 13:17 ` jrfsousa at gmail dot com
7 siblings, 0 replies; 9+ messages in thread
From: jrfsousa at gmail dot com @ 2020-09-01 13:17 UTC (permalink / raw)
To: gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=95352
José Rui Faustino de Sousa <jrfsousa at gmail dot com> changed:
What |Removed |Added
----------------------------------------------------------------------------
Status|NEW |RESOLVED
Resolution|--- |FIXED
--- Comment #8 from José Rui Faustino de Sousa <jrfsousa at gmail dot com> ---
Fixed by:
https://gcc.gnu.org/git/gitweb.cgi?p=gcc.git;h=cd49b7067893b548a10c99ea0cb6aba2977eef2e
^ permalink raw reply [flat|nested] 9+ messages in thread