[PATCH] D33109: Enhance synchscope representation (clang)

2017-07-11 Thread Konstantin Zhuravlyov via Phabricator via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rL307723: Enhance synchscope representation (clang) (authored by kzhuravl). Changed prior to commit: https://reviews.llvm.org/D33109?vs=105499=106106#toc Repository: rL LLVM

[PATCH] D33109: Enhance synchscope representation (clang)

2017-07-06 Thread Konstantin Zhuravlyov via Phabricator via cfe-commits
kzhuravl added a comment. In https://reviews.llvm.org/D33109#73, @mehdi_amini wrote: > Make sure your commit message correctly reference the LLVM change. Will do, thanks. https://reviews.llvm.org/D33109 ___ cfe-commits mailing list

[PATCH] D33109: Enhance synchscope representation (clang)

2017-07-06 Thread Konstantin Zhuravlyov via Phabricator via cfe-commits
kzhuravl updated this revision to Diff 105499. kzhuravl marked an inline comment as done. kzhuravl added a comment. Address review feedback. https://reviews.llvm.org/D33109 Files: lib/CodeGen/CGBuiltin.cpp test/CodeGen/ms-barriers-intrinsics.c Index: test/CodeGen/ms-barriers-intrinsics.c

[PATCH] D33109: Enhance synchscope representation (clang)

2017-07-05 Thread Mehdi AMINI via Phabricator via cfe-commits
mehdi_amini accepted this revision. mehdi_amini added a comment. Make sure your commit message correctly reference the LLVM change. Comment at: test/CodeGen/ms-barriers-intrinsics.c:39 #endif - Don't remove: file should end with a new line I believe.

[PATCH] D33109: Enhance synchscope representation (clang)

2017-07-05 Thread Yaxun Liu via Phabricator via cfe-commits
yaxunl accepted this revision. yaxunl added a comment. LGTM. Thanks! https://reviews.llvm.org/D33109 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[PATCH] D33109: Enhance synchscope representation (clang)

2017-05-11 Thread Tony Tye via Phabricator via cfe-commits
t-tye accepted this revision. t-tye added a comment. This revision is now accepted and ready to land. LGTM https://reviews.llvm.org/D33109 ___ cfe-commits mailing list cfe-commits@lists.llvm.org

[PATCH] D33109: Enhance synchscope representation (clang)

2017-05-11 Thread Konstantin Zhuravlyov via Phabricator via cfe-commits
kzhuravl created this revision. Herald added a subscriber: wdng. https://reviews.llvm.org/D33109 Files: lib/CodeGen/CGBuiltin.cpp test/CodeGen/ms-barriers-intrinsics.c Index: test/CodeGen/ms-barriers-intrinsics.c === ---