[PATCH] D99679: [OPENMP51]Initial support for novariants clause

Jennifer Yu via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Apr 2 11:35:37 PDT 2021


jyu2 added inline comments.


================
Comment at: clang/lib/Sema/SemaOpenMP.cpp:13567-13568
+    case OMPD_dispatch:
+      CaptureRegion = OMPD_task;
+      break;
+    default:
----------------
ABataev wrote:
> What about other directives?
Hi Alexey,

As of now "novariants" clause only allowed in disapatch directive.  And dispatch directive can have implicit task directive. So for other directives will be unreachable.  Am I missing something here?  
Thanks.
Jennifer


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D99679/new/

https://reviews.llvm.org/D99679



More information about the llvm-commits mailing list