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

_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
  • [PATCH] D105876... Sri Hari Krishna Narayanan via Phabricator via cfe-commits
    • [PATCH] D1... Johannes Doerfert via Phabricator via cfe-commits
    • [PATCH] D1... Alexey Bataev via Phabricator via cfe-commits
    • [PATCH] D1... Sri Hari Krishna Narayanan via Phabricator via cfe-commits
    • [PATCH] D1... Alexey Bataev via Phabricator via cfe-commits
    • [PATCH] D1... Sri Hari Krishna Narayanan via Phabricator via cfe-commits
    • [PATCH] D1... Johannes Doerfert via Phabricator via cfe-commits
    • [PATCH] D1... Sri Hari Krishna Narayanan via Phabricator via cfe-commits
    • [PATCH] D1... Sri Hari Krishna Narayanan via Phabricator via cfe-commits
    • [PATCH] D1... Alexey Bataev via Phabricator via cfe-commits
    • [PATCH] D1... Sri Hari Krishna Narayanan via Phabricator via cfe-commits
    • [PATCH] D1... Sri Hari Krishna Narayanan via Phabricator via cfe-commits
    • [PATCH] D1... Sri Hari Krishna Narayanan via Phabricator via cfe-commits

Reply via email to