lebedev.ri marked an inline comment as done.
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:
> 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.
Yep, great idea, did that.


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

Reply via email to