Tobias Burnus wrote:
Hi Mat,
However, first, I want to point out a compilation issue:
This fails for the GCN compilation with: […]
Glancing at it, I don't quite see why the Nvptx compilation succeeds,
but it seems to work.
The wonders of incremental builds …
On the other hand, this issue is fixed by applying 4/5 on top of it.
Still, it would be useful to have a patch that doesn't intermittently
break building the (GCN) compiler.
... nor the Nvptx build.
For nvptx, the fail is:
In file included from /home/tob/repos/gcc/libgomp/config/nvptx/task.c:41:
/home/tob/repos/gcc/libgomp/config/nvptx/../../task.c: In function
‘gomp_barrier_handle_tasks’:
/home/tob/repos/gcc/libgomp/config/nvptx/../../task.c:1573:10: error: too many
arguments to function ‘gomp_barrier_has_completed’; expected 2, have 3
1573 | && gomp_barrier_has_completed (state, &team->barrier, use_cancel))
| ^~~~~~~~~~~~~~~~~~~~~~~~~~ ~~~~~~~~~~
In file included from /home/tob/repos/gcc/libgomp/libgomp.h:219,
from /home/tob/repos/gcc/libgomp/config/nvptx/task.c:31:
/home/tob/repos/gcc/libgomp/config/nvptx/bar.h:181:1: note: declared here
181 | gomp_barrier_has_completed (gomp_barrier_state_t state, gomp_barrier_t
*bar)
| ^~~~~~~~~~~~~~~~~~~~~~~~~~
/home/tob/repos/gcc/libgomp/config/nvptx/../../task.c:1583:11: error: too many
arguments to function ‘gomp_team_barrier_done’; expected 2, have 3
1583 | gomp_team_barrier_done (&team->barrier, state, use_cancel);
| ^~~~~~~~~~~~~~~~~~~~~~ ~~~~~~~~~~
Tobias