[PATCH] D105876: OMPIRBuilder for Interop directive

Johannes Doerfert via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Thu Jul 22 13:37:38 PDT 2021


jdoerfert added a comment.

you need to squash your commits locally and update this revision with a single commit/diff.



================
Comment at: clang/lib/CodeGen/CGStmtOpenMP.cpp:6323-6327
+  if (S.hasClausesOfKind<OMPNowaitClause>() &&
+      !(S.getSingleClause<OMPInitClause>() ||
+        S.getSingleClause<OMPDestroyClause>() ||
+        S.getSingleClause<OMPUseClause>()))
+    assert("OMPNowaitClause clause is used separately in OMPInteropDirective.");
----------------
This is not how assertions work.

assert(EXPR && "MESSAGE");


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D105876



More information about the cfe-commits mailing list