[PATCH] D40968: [OpenMP] Diagnose function name on the link clause
Kelvin Li via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Thu Dec 7 13:06:32 PST 2017
kkwli0 marked an inline comment as done.
kkwli0 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);
----------------
ABataev wrote:
> I would like to see this some in `check` functions, like `checkDeclIsAllowedInOpenMPTarget` rather than here
Sure. I will move the check into checkDeclIsAllowedInOpenMPTarget.
https://reviews.llvm.org/D40968
More information about the cfe-commits
mailing list