Re: [PATCH] Fix PR83017 (fortran part)

2017-11-17 Thread Richard Biener
On Fri, 17 Nov 2017, Janne Blomqvist wrote: > On Fri, Nov 17, 2017 at 3:03 PM, Richard Biener wrote: > > On Fri, 17 Nov 2017, Janne Blomqvist wrote: > > > >> On Fri, Nov 17, 2017 at 11:13 AM, Richard Biener wrote: > >> > This patch changes the Fortran frontend to annotate DO CONCURRENT > >> > wi

Re: [PATCH] Fix PR83017 (fortran part)

2017-11-17 Thread Janne Blomqvist
On Fri, Nov 17, 2017 at 3:03 PM, Richard Biener wrote: > On Fri, 17 Nov 2017, Janne Blomqvist wrote: > >> On Fri, Nov 17, 2017 at 11:13 AM, Richard Biener wrote: >> > This patch changes the Fortran frontend to annotate DO CONCURRENT >> > with parallel instead of ivdep. >> > >> > The patch is not

Re: [PATCH] Fix PR83017 (fortran part)

2017-11-17 Thread Richard Biener
On Fri, 17 Nov 2017, Janne Blomqvist wrote: > On Fri, Nov 17, 2017 at 11:13 AM, Richard Biener wrote: > > This patch changes the Fortran frontend to annotate DO CONCURRENT > > with parallel instead of ivdep. > > > > The patch is not enough to enable a runtime benefit because of > > some autopar c

Re: [PATCH] Fix PR83017 (fortran part)

2017-11-17 Thread Janne Blomqvist
On Fri, Nov 17, 2017 at 11:13 AM, Richard Biener wrote: > This patch changes the Fortran frontend to annotate DO CONCURRENT > with parallel instead of ivdep. > > The patch is not enough to enable a runtime benefit because of > some autopar costing issues but for other cases it should enable > auto

[PATCH] Fix PR83017 (fortran part)

2017-11-17 Thread Richard Biener
This adds a new ANNOTATE_EXPR kind, annot_expr_parallel_kind, which is stronger than ivdep which maps semantically to safelen=INT_MAX which alone doesn't tell us enough to auto-parallelize anything. annot_expr_parallel_kind can map to the already existing loop flag can_be_parallel which can be us