================ @@ -12053,6 +12053,10 @@ bool Sema::CheckFunctionDeclaration(Scope *S, FunctionDecl *NewFD, if (LangOpts.OpenMP) OpenMP().ActOnFinishedFunctionDefinitionInOpenMPAssumeScope(NewFD); + if (LangOpts.isSYCL() && NewFD->hasAttr<SYCLKernelEntryPointAttr>() && + !NewFD->isInvalidDecl() && !NewFD->isDependentContext()) ---------------- erichkeane wrote:
I don't doubt that others got through review, and a few of those I clicked on are different enough situations as to not be the same as this. This is a novel use of that function, and one caught during review. https://github.com/llvm/llvm-project/pull/111389 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits