Re: [Patch] OpenMP: Add iterator support to Fortran's depend; add affinity clause

2021-05-27 Thread Tobias Burnus
On 27.05.21 21:58, Joseph Myers wrote: On Thu, 27 May 2021, Tobias Burnus wrote: @Joseph: I CC'ed you in case you have comments regarding c-parser.c's c_parser_check_balanced_raw_token_sequence Pilot error on my side – doing three things in parallel (fixing a patch, updating docs + attending

[PATCH] PR fortran/99839 - [9/10/11/12 Regression] ICE in inline_matmul_assign, at fortran/frontend-passes.c:4234

2021-05-27 Thread Harald Anlauf via Fortran
Dear Fortranners, frontend optimization tries to inline matmul, but then it also needs to take care of the assignment to the result array. If that one is not of canonical type, we currently get an ICE. The straightforward solution is to simply punt in those cases and avoid inlining. Regtested o

Re: [Patch] OpenMP: Add iterator support to Fortran's depend; add affinity clause

2021-05-27 Thread Jakub Jelinek via Fortran
On Thu, May 27, 2021 at 07:58:03PM +, Joseph Myers wrote: > On Thu, 27 May 2021, Tobias Burnus wrote: > > > @Joseph: I CC'ed you in case you have comments regarding > > c-parser.c's c_parser_check_balanced_raw_token_sequence (comment update) > > and c_parser_check_tight_balanced_raw_token_sequ

Re: [Patch] OpenMP: Add iterator support to Fortran's depend; add affinity clause

2021-05-27 Thread Joseph Myers
On Thu, 27 May 2021, Tobias Burnus wrote: > @Joseph: I CC'ed you in case you have comments regarding > c-parser.c's c_parser_check_balanced_raw_token_sequence (comment update) > and c_parser_check_tight_balanced_raw_token_sequence (new); the latter > is essentially cp_parser_skip_balanced_tokens w

Re: [Patch] OpenMP: Add iterator support to Fortran's depend; add affinity clause

2021-05-27 Thread Jakub Jelinek via Fortran
On Thu, May 27, 2021 at 08:30:33PM +0200, Tobias Burnus wrote: > + if (c_parser_next_token_is (parser, CPP_NAME)) > +{ > + const char *p = IDENTIFIER_POINTER (c_parser_peek_token > (parser)->value); > + bool parse_iter = (strcmp ("iterator", p) == 0); > + if (parse_iter) I'd a

Re: [Patch] OpenMP: Add iterator support to Fortran's depend; add affinity clause

2021-05-27 Thread Tobias Burnus
@Joseph: I CC'ed you in case you have comments regarding c-parser.c's c_parser_check_balanced_raw_token_sequence (comment update) and c_parser_check_tight_balanced_raw_token_sequence (new); the latter is essentially cp_parser_skip_balanced_tokens with slight adaptions. On 27.05.21 10:22, Jakub Je