Re: [PATCH] PR fortran/101536 - ICE in gfc_conv_expr_descriptor, at fortran/trans-array.c:7324

2021-07-23 Thread Tobias Burnus
Hi Harald, On 22.07.21 21:03, Harald Anlauf wrote: you are right in that I was barking up the wrong tree. I was focussed too much on the testcase in the PR. [...] Well, I tried and this does not work. Which makes sense if one thinks about it: When using 'a(5,:)', the parser already sets e->ra

Re: [PATCH] PR fortrsn/101564 - ICE in resolve_allocate_deallocate, at fortran/resolve.c:8169

2021-07-23 Thread Tobias Burnus
Hi Harald, On 22.07.21 21:50, Harald Anlauf wrote: I am afraid we're really opening a can of worms here which is not too bad if there are only two earthworms in there ;-) Additionally, I wonder whether that will work with: I think a "working" testcase for this could be: program p impl

Re: [PATCH 3/3] [PR libfortran/101305] Fix ISO_Fortran_binding.h paths in gfortran testsuite

2021-07-23 Thread Tobias Burnus
Hi Sandra, On 21.07.21 12:17, Tobias Burnus wrote: On 13.07.21 23:28, Sandra Loosemore wrote: ISO_Fortran_binding.h is now generated in the libgfortran build directory where it is on the default include path. Adjust includes in the gfortran testsuite not to include an explicit path pointing at

Re: [PATCH] PR fortran/101536 - ICE in gfc_conv_expr_descriptor, at fortran/trans-array.c:7324

2021-07-23 Thread Harald Anlauf via Fortran
Hi Tobias, > > However, an additional plain check on e->rank != 0 also in the > > CLASS cases fixes the original issue as well as your example: > [...] > > And regtests ok. :-) > > See attached updated patch. > > I think you still need to remove the 'return true;' from > the 'if (e->rank != 0 &&

Re: [PATCH 3/3] [PR libfortran/101305] Fix ISO_Fortran_binding.h paths in gfortran testsuite

2021-07-23 Thread Sandra Loosemore
On 7/23/21 8:15 AM, Tobias Burnus wrote: Hi Sandra, On 21.07.21 12:17, Tobias Burnus wrote: On 13.07.21 23:28, Sandra Loosemore wrote: ISO_Fortran_binding.h is now generated in the libgfortran build directory where it is on the default include path.  Adjust includes in the gfortran testsuite n