[PATCH] D82518: [openmp] Use Directive_enumSize instead of OMPD_unknown position

Johannes Doerfert via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Wed Jun 24 22:14:10 PDT 2020


jdoerfert accepted this revision.
jdoerfert added a comment.
This revision is now accepted and ready to land.

LGTM, maybe a minor adjustment necessary.



================
Comment at: clang/lib/Basic/OpenMPKinds.cpp:583
     OpenMPDirectiveKind DKind) {
-  assert(DKind <= OMPD_unknown);
+  assert(unsigned(DKind) <= llvm::omp::Directive_enumSize);
   switch (DKind) {
----------------
`<`, right?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D82518





More information about the cfe-commits mailing list