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

_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to