Re: Automatic openacc loop partitioning

2015-11-13 Thread Nathan Sidwell
On 11/13/15 16:40, Bernd Schmidt wrote: + this_mask = (this_mask & -this_mask); Unnecessary parens. + if (!this_mask && noisy) +warning_at (loop->loc, 0, +"insufficient partitioning available to parallelize loop"); Should this really be an unconditional warning? Is

Re: Automatic openacc loop partitioning

2015-11-13 Thread Bernd Schmidt
+ this_mask = (this_mask & -this_mask); Unnecessary parens. + if (!this_mask && noisy) + warning_at (loop->loc, 0, + "insufficient partitioning available to parallelize loop"); Should this really be an unconditional warning? Isn't sequential execution a val

Automatic openacc loop partitioning

2015-11-13 Thread Nathan Sidwell
Jakub, this patch applies automatic loop partitioning to loops that are marked 'auto' and 'independent'. 'independent' is implicit inside a parallel region. We were unnecessarily still emitting a sorry for the auto, seq and independent clauses in omp lowering. The main event is in the target