[PATCH] D53916: [ARM64] [Windows] Implement _InterlockedExchangeAdd*_* builtins.

2018-10-31 Thread Eli Friedman via Phabricator via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rC345779: [ARM64] [Windows] Implement _InterlockedExchangeAdd*_* builtins. (authored by efriedma, committed by ). Repository: rC Clang https://reviews.llvm.org/D53916 Files: include/clang/Basic/Builti

[PATCH] D53916: [ARM64] [Windows] Implement _InterlockedExchangeAdd*_* builtins.

2018-10-30 Thread Reid Kleckner via Phabricator via cfe-commits
rnk accepted this revision. rnk added a comment. This revision is now accepted and ready to land. lgtm Comment at: lib/CodeGen/CGBuiltin.cpp:100 +CodeGenFunction &CGF, llvm::AtomicRMWInst::BinOp Kind, const CallExpr *E, +AtomicOrdering Ordering = AtomicOrdering::Sequent

[PATCH] D53916: [ARM64] [Windows] Implement _InterlockedExchangeAdd*_* builtins.

2018-10-30 Thread Eli Friedman via Phabricator via cfe-commits
efriedma updated this revision to Diff 171853. efriedma added a comment. Upload correct diff. Repository: rC Clang https://reviews.llvm.org/D53916 Files: include/clang/Basic/BuiltinsAArch64.def include/clang/Basic/BuiltinsARM.def lib/CodeGen/CGBuiltin.cpp lib/Headers/intrin.h test/

[PATCH] D53916: [ARM64] [Windows] Implement _InterlockedExchangeAdd*_* builtins.

2018-10-30 Thread Eli Friedman via Phabricator via cfe-commits
efriedma updated this revision to Diff 171852. efriedma added a comment. Get rid of unnecessary constants. Repository: rC Clang https://reviews.llvm.org/D53916 Files: include/clang/Basic/BuiltinsAArch64.def include/clang/Basic/BuiltinsARM.def lib/CodeGen/CGBuiltin.cpp lib/Headers/int

[PATCH] D53916: [ARM64] [Windows] Implement _InterlockedExchangeAdd*_* builtins.

2018-10-30 Thread Eli Friedman via Phabricator via cfe-commits
efriedma created this revision. efriedma added reviewers: rnk, mstorsjo. Herald added subscribers: kristina, jfb, chrib, kristof.beyls, javed.absar. These apparently need to be proper builtins to handle the Windows SDK. Repository: rC Clang https://reviews.llvm.org/D53916 Files: include/cl