[Bug testsuite/115140] [15 regression] libgomp.oacc-c++/../libgomp.oacc-c-c++-common/acc_prof-kernels-1.c excess errors after r15-579-ga9251ab3c91c8c
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=115140 Tobias Burnus changed: What|Removed |Added CC||burnus at gcc dot gnu.org --- Comment #5 from Tobias Burnus --- 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.)
[Bug testsuite/115140] [15 regression] libgomp.oacc-c++/../libgomp.oacc-c-c++-common/acc_prof-kernels-1.c excess errors after r15-579-ga9251ab3c91c8c
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=115140 Thiago Jung Bauermann changed: What|Removed |Added CC||thiago.bauermann at linaro dot org --- Comment #4 from Thiago Jung Bauermann --- Also seen on aarch64-linux-gnu and armv8l-linux-gnueabihf.
[Bug testsuite/115140] [15 regression] libgomp.oacc-c++/../libgomp.oacc-c-c++-common/acc_prof-kernels-1.c excess errors after r15-579-ga9251ab3c91c8c
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=115140
Thomas Schwinge changed:
What|Removed |Added
Host|powerpc64-linux-gnu,|
|powerpc64le-linux-gnu, |
|*-*-solaris2.11 |
Build|powerpc64-linux-gnu,|
|powerpc64le-linux-gnu, |
|*-*-solaris2.11 |
CC||tschwinge at gcc dot gnu.org
Status|UNCONFIRMED |NEW
Target|powerpc64-linux-gnu,|
|powerpc64le-linux-gnu, |
|*-*-solaris2.11 |
Last reconfirmed||2024-05-24
Ever confirmed|0 |1
--- Comment #3 from Thomas Schwinge ---
So the PASS -> FAIL regressions are due to parloops (for '-O2') no longer
parallelizing the simple OpenACC 'kernels' construct at line 185 (and two
more):
int x[N];
#pragma acc kernels
{
for (int i = 0; i < N; ++i)
x[i] = i * i;
}
(In reply to Richard Biener from comment #1)
> Looks like a testsuite artifact?
>
> volatile // TODO PR90488
> static int state = -1;
>
> I've not looked as to why/how we are getting that to influence points-to
> solutions (note as we track also integers volatile on non-pointers can
> matter).
Yeah, it's not obvious to me how that 'state' variable would have such an
effect -- but I've not yet 'diff'ed the dumps.
On the other hand, it's highly likely that there is some relation, as no other
OpenACC 'kernels' test cases did regress.
[Bug testsuite/115140] [15 regression] libgomp.oacc-c++/../libgomp.oacc-c-c++-common/acc_prof-kernels-1.c excess errors after r15-579-ga9251ab3c91c8c
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=115140 Rainer Orth changed: What|Removed |Added Host|powerpc64-linux-gnu,|powerpc64-linux-gnu, |powerpc64le-linux-gnu |powerpc64le-linux-gnu, ||*-*-solaris2.11 CC||ro at gcc dot gnu.org Build|powerpc64-linux-gnu,|powerpc64-linux-gnu, |powerpc64le-linux-gnu |powerpc64le-linux-gnu, ||*-*-solaris2.11 Target|powerpc64-linux-gnu,|powerpc64-linux-gnu, |powerpc64le-linux-gnu |powerpc64le-linux-gnu, ||*-*-solaris2.11 --- Comment #2 from Rainer Orth --- Also seen on Solaris/SPARC and x86.
[Bug testsuite/115140] [15 regression] libgomp.oacc-c++/../libgomp.oacc-c-c++-common/acc_prof-kernels-1.c excess errors after r15-579-ga9251ab3c91c8c
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=115140 Richard Biener changed: What|Removed |Added Component|other |testsuite Keywords||openacc --- Comment #1 from Richard Biener --- Looks like a testsuite artifact? volatile // TODO PR90488 static int state = -1; I've not looked as to why/how we are getting that to influence points-to solutions (note as we track also integers volatile on non-pointers can matter).
