[PATCH] D28691: Support synchronisation scope in Clang atomic builtin functions

2017-01-17 Thread Yaxun Liu via Phabricator via cfe-commits
yaxunl added a comment. The link for the RFC: http://lists.llvm.org/pipermail/cfe-dev/2017-January/052304.html https://reviews.llvm.org/D28691 ___ cfe-commits mailing list cfe-commits@lists.llvm.org

[PATCH] D28691: Support synchronisation scope in Clang atomic builtin functions

2017-01-17 Thread Yaxun Liu via Phabricator via cfe-commits
yaxunl added a comment. In https://reviews.llvm.org/D28691#648267, @rjmccall wrote: > This patch changes the language design of the atomic builtins, which is > outside the normal scope of patch review. You need to post an RFC to > cfe-dev. I've gone ahead and made some material comments, but

[PATCH] D28691: Support synchronisation scope in Clang atomic builtin functions

2017-01-17 Thread John McCall via Phabricator via cfe-commits
rjmccall requested changes to this revision. rjmccall added a comment. This revision now requires changes to proceed. This patch changes the language design of the atomic builtins, which is outside the normal scope of patch review. You need to post an RFC to cfe-dev. I've gone ahead and made

[PATCH] D28691: Support synchronisation scope in Clang atomic builtin functions

2017-01-13 Thread Yaxun Liu via Phabricator via cfe-commits
yaxunl retitled this revision from "Support synchronisation scope in atomic builtin functions" to "Support synchronisation scope in Clang atomic builtin functions". yaxunl updated the summary for this revision. yaxunl updated this revision to Diff 84373. yaxunl added a comment. Fix typos and