[PATCH] D40968: [OpenMP] Diagnose function name on the link clause

[Alexey Bataev via Phabricator via cfe-commits]

ABataev added inline comments.


================
Comment at: lib/Sema/SemaOpenMP.cpp:12576
 
+    if (MT == OMPDeclareTargetDeclAttr::MT_Link && isa<FunctionDecl>(ND)) {
+      Diag(Id.getLoc(), diag::err_omp_function_in_link_clause);
----------------
I would like to see this some in `check` functions, like 
`checkDeclIsAllowedInOpenMPTarget` rather than here 


https://reviews.llvm.org/D40968



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits