================
@@ -5465,15 +5465,19 @@ static void handleGlobalAttr(Sema &S, Decl *D, const
ParsedAttr &AL) {
if (FD->isInlineSpecified() && !S.getLangOpts().CUDAIsDevice)
S.Diag(FD->getBeginLoc(), diag::warn_kern_is_inline) << FD;
- if (AL.getKind() == ParsedAttr::AT_DeviceKernel)
- D->addAttr(::new (S.Context) DeviceKernelAttr(S.Context, AL));
- else
+ // ***REVIEWER***: the existing code clearly expects either DeviceKernel or
----------------
steffenlarsen wrote:
I assume it's a question for reviewers? Under that assumption, I would say it
is indeed the expectation and I would be alright with adding an `assert`
checking that the attribute one or the other.
https://github.com/llvm/llvm-project/pull/218582
_______________________________________________
cfe-commits mailing list
[email protected]
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
