[SYCL] Propagate to kernel only the attributes of its lambda or function object

It is expected to propagate certain attributes to SYCL kernel from only lambda
functions and function objects that are called directly from a kernel
(i.e., those passed to the parallel_for function)

This patch upholds this expectation by avoiding attributes being propagated
from functions called from within kernel.

Author: schittir

clang/lib/Sema/SemaSYCL.cpp

@@ -420,39 +420,39 @@ class MarkDeviceFunction : public RecursiveASTVisitor<MarkDeviceFunction> {
       // passed to the parallel_for function). For all other cases,
       // emit a warning and ignore.
       if (auto *A = FD->getAttr<SYCLIntelKernelArgsRestrictAttr>()) {
-        if (ParentFD == SYCLKernel) {
+        if (ParentFD == SYCLKernel && A->isImplicit()) {
           Attrs.insert(A);
         } else {
           SemaRef.Diag(A->getLocation(), diag::warn_attribute_ignored) << A;
           FD->dropAttr<SYCLIntelKernelArgsRestrictAttr>();
         }
       }
       if (auto *A = FD->getAttr<SYCLIntelNumSimdWorkItemsAttr>()) {
-        if (ParentFD == SYCLKernel) {
+        if (ParentFD == SYCLKernel && A->isImplicit()) {
           Attrs.insert(A);
         } else {
           SemaRef.Diag(A->getLocation(), diag::warn_attribute_ignored) << A;
           FD->dropAttr<SYCLIntelNumSimdWorkItemsAttr>();
         }
       }
       if (auto *A = FD->getAttr<SYCLIntelMaxWorkGroupSizeAttr>()) {
-        if (ParentFD == SYCLKernel) {
+        if (ParentFD == SYCLKernel && A->isImplicit()) {
           Attrs.insert(A);
         } else {
           SemaRef.Diag(A->getLocation(), diag::warn_attribute_ignored) << A;
           FD->dropAttr<SYCLIntelMaxWorkGroupSizeAttr>();
         }
       }
       if (auto *A = FD->getAttr<SYCLIntelMaxGlobalWorkDimAttr>()) {
-        if (ParentFD == SYCLKernel) {
+        if (ParentFD == SYCLKernel && A->isImplicit()) {
           Attrs.insert(A);
         } else {
           SemaRef.Diag(A->getLocation(), diag::warn_attribute_ignored) << A;
           FD->dropAttr<SYCLIntelMaxGlobalWorkDimAttr>();
         }
       }
       if (auto *A = FD->getAttr<SYCLIntelNoGlobalWorkOffsetAttr>()) {
-        if (ParentFD == SYCLKernel) {
+        if (ParentFD == SYCLKernel && A->isImplicit()) {
           Attrs.insert(A);
         } else {
           SemaRef.Diag(A->getLocation(), diag::warn_attribute_ignored) << A;
@@ -1677,7 +1677,9 @@ void Sema::MarkDevice(void) {
         case attr::Kind::SYCLIntelMaxGlobalWorkDim:
         case attr::Kind::SYCLIntelMaxWorkGroupSize:
         case attr::Kind::SYCLIntelNoGlobalWorkOffset: {
-          SYCLKernel->addAttr(A);
+          Attr *NewA = A->clone(getASTContext());
+          NewA->setImplicit(true);
+          SYCLKernel->addAttr(NewA);
           break;
         }
         // TODO: vec_len_hint should be handled here