[PATCH] D57280: [clang][OpenMP] OMPFlushClause is synthetic, no such clause exists

Roman Lebedev via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Mon Jan 28 08:03:42 PST 2019


lebedev.ri added inline comments.


================
Comment at: lib/Basic/OpenMPKinds.cpp:56
 #define OPENMP_CLAUSE(Name, Class) .Case(#Name, OMPC_##Name)
+      OPENMP_CLAUSE(flush, OMPFlushClause)
 #include "clang/Basic/OpenMPKinds.def"
----------------
ABataev wrote:
> Just `.Case("flush", OMPC_flush)`, similar to `uniform`
Hm, we won't ever trigger that case, since we have `if (Str == "flush") return OMPC_unknown;` before this.


Repository:
  rC Clang

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

https://reviews.llvm.org/D57280





More information about the cfe-commits mailing list