This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [og7] vector_length extension part 4: target hooks and automatic parallelism
On 03/26/2018 06:33 PM, Tom de Vries wrote:
+ loop->mask = targetm.goacc.adjust_parallelism (loop->mask, outer_mask);
loop->mask |= this_mask;
I committed the above, but the original:
...
@@ -1397,6 +1407,8 @@ oacc_loop_auto_partitions (oacc_loop *loop, unsigned outer_mask,
}
loop->mask |= this_mask;
+ loop->mask = targetm.goacc.adjust_parallelism (loop->mask, outer_mask);
+
if (!loop->mask && noisy)
warning_at (loop->loc, 0,
tiling
...
has the two loop->mask lines in the reverse order.
Fixed in attached patch.
Committed.
Thanks,
- Tom
[openacc] Fix adjust_parallism usage in oacc_loop_auto_partitions
2018-03-27 Tom de Vries <tom@codesourcery.com>
* omp-offload.c (oacc_loop_auto_partitions): Fix adjust_parallism usage.
---
gcc/omp-offload.c | 2 +-
2 files changed, 5 insertions(+), 1 deletion(-)
diff --git a/gcc/omp-offload.c b/gcc/omp-offload.c
index aa4de24..ed17160 100644
--- a/gcc/omp-offload.c
+++ b/gcc/omp-offload.c
@@ -1404,8 +1404,8 @@ oacc_loop_auto_partitions (oacc_loop *loop, unsigned outer_mask,
" to parallelize element loop");
}
- loop->mask = targetm.goacc.adjust_parallelism (loop->mask, outer_mask);
loop->mask |= this_mask;
+ loop->mask = targetm.goacc.adjust_parallelism (loop->mask, outer_mask);
if (!loop->mask && noisy)
warning_at (loop->loc, 0,