Anastasia added inline comments.
Comment at: lib/Sema/SemaExpr.cpp:13645
// Blocks are not allowed to capture arrays.
+ if (!S.getLangOpts().OpenCL && CaptureType->isArrayType()) {
I think the comment should be updated.
Comment at:
echuraev updated this revision to Diff 94503.
echuraev marked an inline comment as done.
https://reviews.llvm.org/D26794
Files:
lib/Sema/SemaExpr.cpp
test/SemaOpenCL/blocks_with_array.cl
Index: test/SemaOpenCL/blocks_with_array.cl
echuraev added a comment.
Yes, I have an access to the new revision and I have read it.
https://reviews.llvm.org/D26794
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
Anastasia added a comment.
In https://reviews.llvm.org/D26794#598784, @Anastasia wrote:
> I have created a bug to Khronos regarding this, but unfortunately I don't see
> it being progressed yet.
> https://cvs.khronos.org/bugzilla/show_bug.cgi?id=15659
>
> The problem here is that I am not sure
Anastasia added a comment.
I have created a bug to Khronos regarding this, but unfortunately I don't see
it being progressed yet.
https://cvs.khronos.org/bugzilla/show_bug.cgi?id=15659
The problem here is that I am not sure we should deviate from the ObjC
implementation because OpenCL blocks ar
echuraev created this revision.
echuraev added a reviewer: Anastasia.
echuraev added subscribers: cfe-commits, yaxunl, bader.
https://reviews.llvm.org/D26794
Files:
lib/Sema/SemaExpr.cpp
test/SemaOpenCL/blocks_with_arrays.cl
Index: test/SemaOpenCL/blocks_with_arrays.cl
=