[NFC][SYCL] Rename poorly named diagnostic #3056
Conversation
Fznamznon
requested review from
elizabethandrews and
premanandrao
as code owners
clang/lib/Sema/SemaDeclAttr.cpp
Outdated
| if (!VD->hasGlobalStorage()) { | ||
| S.Diag(AL.getLoc(), diag::err_sycl_attibute_cannot_be_applied_here) | ||
| << AL << 0; | ||
| S.Diag(AL.getLoc(), diag::err_sycl_attribute_internal_function) << AL << 0; |
There was a problem hiding this comment.
Actually, I just took a closer look. Why is a diagnostic about static function/function in anonymous namespace being generated for a variable. This is out of scope of your patch but this looks like incorrect diagnostic?
There was a problem hiding this comment.
Do I have to fix it?
There was a problem hiding this comment.
If we'd like to upstream at some point, I'd recommend fixing it. (I think it's in scope for this patch given that the diagnostic quality is being regressed, FWIW).
There was a problem hiding this comment.
Ok. register_num has only 'GlobalVar' in subjects list, so it seems the part of the code where this diagnostic was incorrectly used is dead code. Removed.
There was a problem hiding this comment.
If we'd like to upstream at some point
Go for it! :-)
5 participants
No description provided.