https://gcc.gnu.org/bugzilla/show_bug.cgi?id=115140
Tobias Burnus <burnus at gcc dot gnu.org> changed:
What |Removed |Added
----------------------------------------------------------------------------
CC| |burnus at gcc dot gnu.org
--- Comment #5 from Tobias Burnus <burnus at gcc dot gnu.org> ---
The following makes the testcase pass again,
both without and with offloading configured (nvptx + gcn,
executed with nvptx hardware):
--- a/libgomp/testsuite/libgomp.oacc-c-c++-common/acc_prof-kernels-1.c
+++ b/libgomp/testsuite/libgomp.oacc-c-c++-common/acc_prof-kernels-1.c
@@ -45 +44 @@
-volatile // TODO PR90488
Digging shows that this TODO line was added already in the first commit,
i.e. r10-472-g5fae049dc27214
Trying the variant with or without 'volatile' seems to work with GCC 12, GCC 13
and GCC 14 (using GCC 13's libgomp) - while mainline w/ volatile fails and only
w/o works.
@Thomas: Is there any reason for keeping that line? It seems as if just
removing that line will yield a working testcase. (And, possibly, adding a note
to PR90488.)
