svenvh accepted this revision.
svenvh added a comment.
This revision is now accepted and ready to land.

LGTM, please address style issue before committing.



================
Comment at: test/Headers/opencl-c-header.cl:79
+{
+    atomic_add(a, 1);
+}
----------------
Probably better to follow the style of the other tests above (i.e. indent and 
brace placement).


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D62335/new/

https://reviews.llvm.org/D62335



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
  • [PATCH] D62335: [OpenCL... Sven van Haastregt via Phabricator via cfe-commits

Reply via email to