[PATCH] D35235: [libc++] Replace __sync_* functions with __libcpp_atomic_* functions

[Saleem Abdulrasool via Phabricator via llvm-commits]

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

Why the additional inclusion of the support in the files which are not modified to use functions?  (e.g. `src/stdexcept.cpp`).  LG otherwise.



================
Comment at: src/include/atomic_support.h:150
+_ValueType __libcpp_atomic_exchange(_ValueType* __target,
+                                    _ValueType __value, int __order = _AO_Seq)
+{
----------------
Really annoying that you can't immediately tell that this is in the `_LIBCPP_NO_THREADS` case, and thus the atomicity is not required.


https://reviews.llvm.org/D35235