[PATCH] [libcxx] Add __atomic header that handles needed atomic operations.

Jonathan Roelofs jonathan at codesourcery.com
Thu Jun 11 18:27:16 PDT 2015


================
Comment at: include/__atomic:85
@@ +84,3 @@
+{
+    *__dest = __val;
+}
----------------
Should these fall back on the __sync_* ones instead of just dropping the atomicness? It seems a little precarious to silently default to non-atomic implementations for these (unless this is for a single-threaded build of the library).

http://reviews.llvm.org/D10406

EMAIL PREFERENCES
  http://reviews.llvm.org/settings/panel/emailpreferences/






More information about the cfe-commits mailing list