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

[Alexey Bataev via Phabricator via llvm-commits]

ABataev added inline comments.


================
Comment at: clang/lib/Sema/SemaOpenMP.cpp:13567-13568
+    case OMPD_dispatch:
+      CaptureRegion = OMPD_task;
+      break;
+    default:
----------------
jyu2 wrote:
> 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
No, I just was not aware of it.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D99679