[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