jdoerfert accepted this revision. jdoerfert added a comment. This revision is now accepted and ready to land.
LGTM, one nit below ================ Comment at: clang/lib/Sema/SemaOpenMP.cpp:10179 Diag(StartLoc, diag::err_omp_no_clause_for_directive) - << "'map' or 'use_device_ptr'" - << getOpenMPDirectiveName(OMPD_target_data); + << Expected << getOpenMPDirectiveName(OMPD_target_data); return StmtError(); ---------------- The restriction text quoted above doesn't seem to match the code anymore (also before). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D80404/new/ https://reviews.llvm.org/D80404 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits