On 11/05/15 21:10, Cesar Philippidis wrote:
I've applied this patch to trunk. It also includes the fortran and
template changes. Note that there is a new regression in
gfortran.dg/goacc/combined_loop.f90. Basically, the gimplifier is
complaining about reduction variables appearing in multiple clauses.
E.g. 'acc parallel reduction(+:var) copy(var)'. Nathan's upcoming
gimplifier changes should address that.
Also, because of these reduction problems, I decided not to merge
combined_loops.f90 with combined-directives.f90 yet because the latter
relies on scanning which would fail with the errors detected during
gimplfication. I'm planning on adding a couple of more test cases once
acc reductions are working on trunk.
Reductions are already on trunk. do you mean one or both of:
1) firstprivate
2) default handlinng (depends on #1)
I expect to post #1 today. #2 (a smaller patch) may not make it today, as I
have to rebase it on the reworking of #1 I did to remove the two enums I
disucssed with Jakub.
nathan
--
Nathan Sidwell