OpenCL specification forbids use of several types as kernel
arguments. This patch improves existing diagnostic to look through
arrays.
Details
Details
Diff Detail
Diff Detail
- Repository
- rC Clang
Event Timeline
Comment Actions
It seems however the restriction on pointer to pointer was removed (see s6.9.a last item) in CL2.0.
lib/Sema/SemaDecl.cpp | ||
---|---|---|
8187 | Do we need to assert PT here too? It doesn't seem to be modified in this loop... |
Comment Actions
Right, and it seems that pointers in struct arguments should also be legal in CL2.0.
I'll submit another patch to remove this check for CL2.0.
lib/Sema/SemaDecl.cpp | ||
---|---|---|
8187 | We should check for a field instead. Thanks! |