openmp: Fix up taskloop reduction ICE if taskloop has no iterations [PR100471]
authorJakub Jelinek <jakub@redhat.com>
Tue, 11 May 2021 07:07:47 +0000 (09:07 +0200)
committerJakub Jelinek <jakub@redhat.com>
Tue, 11 May 2021 07:07:47 +0000 (09:07 +0200)
commit98acbb3111fcb5e57d5e63d46c0d92f4e53e3c2a
treed1203880287bb5726e87729b40c09fe2946a4967
parent1866182f6cf338880c68225d9de571b787b6abcd
openmp: Fix up taskloop reduction ICE if taskloop has no iterations [PR100471]

When a taskloop doesn't have any iterations, GOMP_taskloop* takes an early
return, doesn't create any tasks and more importantly, doesn't create
a taskgroup and doesn't register task reductions.  But, the code emitted
in the callers assumes task reductions have been registered and performs
the reduction handling and task reduction unregistration.  The pointer
to the task reduction private variables is reused, on input it is the alignment
and only on output it is the pointer, so in the case taskloop with no iterations
the caller attempts to dereference the alignment value as if it was a pointer
and crashes.  We could in the early returns register the task reductions
only to have them looped over and unregistered in the caller, but I think
it is better to tell the caller there is nothing to task reduce and bypass
all that.

2021-05-11  Jakub Jelinek  <jakub@redhat.com>

PR middle-end/100471
* omp-low.c (lower_omp_task_reductions): For OMP_TASKLOOP, if data
is 0, bypass the reduction loop including
GOMP_taskgroup_reduction_unregister call.

* taskloop.c (GOMP_taskloop): If GOMP_TASK_FLAG_REDUCTION and not
GOMP_TASK_FLAG_NOGROUP, when doing early return clear the task
reduction pointer.
* testsuite/libgomp.c/task-reduction-4.c: New test.
gcc/omp-low.c
libgomp/taskloop.c
libgomp/testsuite/libgomp.c/task-reduction-4.c [new file with mode: 0644]