Page MenuHomePhabricator

[OpenCL] Test on image access modifiers and image type can only be a type of a function argument.

Authored by echuraev on Jul 4 2017, 6:29 AM.

Diff Detail

Event Timeline

echuraev created this revision.Jul 4 2017, 6:29 AM
Anastasia added inline comments.Jul 4 2017, 11:37 AM

Would it be better to rename to make the purpose more clear:
myIndifferent -> myDefault


Like above I would try to give the function some descriptive names...


I don't get this comment.


I don't get this comment too?


For consistency we could add

//read_only = read_only

I think we test this in test/SemaOpenCL/

Feel free to combine in one file if you like.

echuraev updated this revision to Diff 105218.Jul 4 2017, 10:44 PM
echuraev marked 6 inline comments as done.
Anastasia accepted this revision.Jul 5 2017, 9:58 AM
This revision is now accepted and ready to land.Jul 5 2017, 9:58 AM

LGTM! Thanks!

echuraev closed this revision.Jul 6 2017, 12:06 AM