[Openmp-commits] [PATCH] Add support for nowait and depend clauses in target construct in 4.1

hfinkel at anl.gov hfinkel at anl.gov
Wed May 6 09:46:23 PDT 2015


This LGTM. I'd prefer this be broken up into three commits:

1. Makefile changes to add 4.1 support
2. Header changes to add 4.1 support
3. Adding task proxy / task full


REPOSITORY
  rL LLVM

================
Comment at: runtime/src/kmp_barrier.cpp:1491
@@ -1467,3 +1490,3 @@
         if (__kmp_tasking_mode != tskm_immediate_exec) {
-            __kmp_task_team_setup(this_thr, team, 1);  // 1 indicates setup both task teams
+            __kmp_task_team_setup(this_thr, team, 1, 0);  // 1 indicates setup both task teams
         }
----------------
The comment might also explain what 0 means too.

http://reviews.llvm.org/D9306

EMAIL PREFERENCES
  http://reviews.llvm.org/settings/panel/emailpreferences/






More information about the Openmp-commits mailing list