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 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits