On 23.09.21 21:13, Tobias Burnus wrote:
On 20.09.21 09:58, Tobias Burnus wrote:
On 20.09.21 06:01, Sandra Loosemore wrote:
This patch fixes some bugs in handling of assumed-rank arguments
revealed by the TS29113 testsuite, ... giving a bogus error when
passing one as the first argument to the ASSOCIATED intrinsic. ...
... if I try the following testcase, which is now permitted, I get
two ICEs. Can you check?
* The first one seems to be a bug in gfc_conv_intrinsic_function, which
assumes also for assumed rank that if the first argument is an array,
the second argument must also be an array.
* For the second one, I see in the dump:
p->dim[p->dtype.rank + -1].stride
is seems as '-1' is gfc_array_index_type while 'dtype.rank' is
signed_char_type_node.
I fixed that issue + extended the testcase.
OK for mainline?
Hi Tobias,
looks good to me.
Thanks for the patch!
Best regards
Thomas
