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

Alexey Bataev via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Mon Jan 28 08:08:03 PST 2019


ABataev 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"
----------------
lebedev.ri wrote:
> 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.
Then just remove this new line completely.


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