[PATCH] D22557: [libcxx] Diagnose invalid memory order arguments in <atomic>. Fixes PR21179.

JF Bastien via cfe-commits cfe-commits at lists.llvm.org
Wed Jul 20 15:46:24 PDT 2016


jfb added inline comments.

================
Comment at: include/atomic:569
@@ +568,3 @@
+    __attribute__ ((__enable_if__(__m == memory_order_release                 \
+                               || __m == memory_order_acq_rel, "")))          \
+    __attribute__ ((__unavailable__("memory order argument to atomic operation is invalid")))
----------------
bcraig wrote:
> what about relaxed?
Load relaxed is valid.


https://reviews.llvm.org/D22557





More information about the cfe-commits mailing list