[PATCH] Workaround attribute ordering issue with kernel only attributes
Aaron Ballman
aaron.ballman at gmail.com
Thu Dec 4 18:44:03 PST 2014
On Thu, Dec 4, 2014 at 6:30 PM, Matt Arsenault
<Matthew.Arsenault at amd.com> wrote:
> Hi aaron.ballman,
>
> Placing the attribute after the kernel keyword would incorrectly
> reject the attribute, so use the smae workaround that other
> kernel only attributes use.
>
> Also add a FIXME because there are two different phrasings now
> for the same error, althoug amdgpu_num_[sv]gpr uses a consistent one.
LGTM! Thanks!
~Aaron
>
> http://reviews.llvm.org/D6542
>
> Files:
> include/clang/Basic/Attr.td
> lib/Sema/SemaDeclAttr.cpp
> test/SemaCUDA/amdgpu-num-gpr-attr.cu
> test/SemaOpenCL/amdgpu-num-register-attrs.cl
More information about the cfe-commits
mailing list