[Openmp-commits] [openmp] e32e1da - [OpenMP][Tests] Fix test compatibility

[Joachim Protze via Openmp-commits]

Author: Joachim Protze
Date: 2021-07-28T00:08:32+02:00
New Revision: e32e1dae61f20569fc16951b15ab9f63b9acfe08

URL: https://github.com/llvm/llvm-project/commit/e32e1dae61f20569fc16951b15ab9f63b9acfe08
DIFF: https://github.com/llvm/llvm-project/commit/e32e1dae61f20569fc16951b15ab9f63b9acfe08.diff

LOG: [OpenMP][Tests] Fix test compatibility

gcc and clang disagree in how the event handle needs to be handled.
According to OpenMP LC, gcc is right. Will open clang bug report

Added: 
    

Modified: 
    openmp/runtime/test/tasking/detach_nested_task.c

Removed: 
    


################################################################################
diff  --git a/openmp/runtime/test/tasking/detach_nested_task.c b/openmp/runtime/test/tasking/detach_nested_task.c
index 3427fbda93a9..bb0847409c58 100644
--- a/openmp/runtime/test/tasking/detach_nested_task.c
+++ b/openmp/runtime/test/tasking/detach_nested_task.c
@@ -23,7 +23,7 @@ int foo(int n)
   int ret = 0;
   for (int i = 0; i < n; ++i) {
     omp_event_handle_t event;
-    #pragma omp task detach(event) firstprivate(i,n) shared(ret) default(none)
+    #pragma omp task detach(event) firstprivate(i,n) shared(ret)
     {
       for (int j = 0; j < n; ++j) {
         #pragma omp task firstprivate(event,i,j,n) shared(ret) default(none) depend(out:ret)