sarnex wrote:

> Do we observe the issue there because intel's fork used to add 
> OpenCLKernelAttr implicitly in SemaSYCL?
Yes, since the attribute is created implicitly, there's no spelling to check so 
there's no `AttrName`.

I understand we can only reproduce the problem out-of-tree, but since it's just 
a `nullptr` check and in general the assumption in this code that `AttrName` is 
always available is wrong, I think it's okay to have the fix here.

https://github.com/llvm/llvm-project/pull/143546
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to