[OMPT] Add synchronization to threads_nested.c testcase

jprotze · jprotze · commit 00505b85a3fb · 2018-07-05T09:14:01.000Z
The testcase potentially fails when a thread is reused. The added synchronization makes sure this does not happen. Patch provided by Simon Convent Differential Revision: https://reviews.llvm.org/D48932 llvm-svn: 336326
diff --git a/openmp/runtime/test/ompt/misc/threads_nested.c b/openmp/runtime/test/ompt/misc/threads_nested.c
@@ -4,14 +4,16 @@
 #include <omp.h>
 
 int main() {
+
+  int condition = 0;
   int x = 0;
   omp_set_nested(1);
 #pragma omp parallel num_threads(2)
   {
 #pragma omp parallel num_threads(2)
     {
-#pragma omp atomic
-      x++;
+      OMPT_SIGNAL(condition);
+      OMPT_WAIT(condition, 4);
     }
   }
 

Original file line number	Diff line number	Diff line change
`@@ -4,14 +4,16 @@`
`4`	`4`	`#include <omp.h>`
`5`	`5`
`6`	`6`	`int main() {`
	`7`	`+`
	`8`	`+ int condition = 0;`
`7`	`9`	`int x = 0;`
`8`	`10`	`omp_set_nested(1);`
`9`	`11`	`#pragma omp parallel num_threads(2)`
`10`	`12`	`{`
`11`	`13`	`#pragma omp parallel num_threads(2)`
`12`	`14`	`{`
`13`		`-#pragma omp atomic`
`14`		`- x++;`
	`15`	`+ OMPT_SIGNAL(condition);`
	`16`	`+ OMPT_WAIT(condition, 4);`
`15`	`17`	`}`
`16`	`18`	`}`
`17`	`19`