[PATCH] D37742: Add more tests for OpenCL atomic builtin functions

2017-09-13 Thread Yaxun Liu via Phabricator via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rL313172: Add more tests for OpenCL atomic builtin functions (authored by yaxunl). Changed prior to commit: https://reviews.llvm.org/D37742?vs=114824&id=115090#toc Repository: rL LLVM https://reviews.

[PATCH] D37742: Add more tests for OpenCL atomic builtin functions

2017-09-12 Thread Anastasia Stulova via Phabricator via cfe-commits
Anastasia accepted this revision. Anastasia added a comment. This revision is now accepted and ready to land. LGTM generally, apart from I am not sure why address spaces are only tested for atomic stores... https://reviews.llvm.org/D37742 ___ cfe-c

[PATCH] D37742: Add more tests for OpenCL atomic builtin functions

2017-09-12 Thread Yaxun Liu via Phabricator via cfe-commits
yaxunl created this revision. Add tests for different address spaces and insert some blank lines to make them more readable. https://reviews.llvm.org/D37742 Files: test/CodeGenOpenCL/atomic-ops-libcall.cl test/CodeGenOpenCL/atomic-ops.cl Index: test/CodeGenOpenCL/atomic-ops.cl ===