jfb added inline comments.

================
Comment at: include/atomic:840
@@ +839,3 @@
+#define ATOMIC_LLONG_LOCK_FREE     __GCC_ATOMIC_LLONG_LOCK_FREE
+#define ATOMIC_POINTER_LOCK_FREE   __GCC_ATOMIC_POINTER_LOCK_FREE
+
----------------
Moving these from the bottom of the file since I need them.

================
Comment at: include/atomic:887
@@ +886,3 @@
+    false;
+#   endif
+# endif
----------------
This is slightly ugly, but I don't see a better way to go about without 
breaking libc++ when it's not exactly in sync with clang, or when it's used 
with GCC (until GCC picks up these macros).

================
Comment at: test/std/atomics/atomics.lockfree/lockfree.pass.cpp:23
@@ -21,2 +22,3 @@
 // #define ATOMIC_LLONG_LOCK_FREE unspecified
+// #define ATOMIC_POINTER_LOCK_FREE unspecified
 
----------------
BOOL and POINTER weren't tested, I'm adding them while I'm here.


http://reviews.llvm.org/D17951



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to