On Fri, Sep 14, 2012 at 7:48 PM, Richard Smith <[email protected]> wrote: > Hi, > > The attached patch adds an implementation of <stdatomic.h> to the set of > headers provided by Clang. Since this header is so compiler-dependent, it > seems that we are the most rational component to be providing this header > (even though, for instance, some flavors of BSD already provide their own). > Please review!
+// Clang allows memory_order_consume ordering for __c11_atomic_store, +// even though C11 doesn't allow it for atomic_store. That looks like a bug... Please put the new warning in a separate commit. It looks like standard requires that we expose functions named atomic_thread_fence, atomic_signal_fence, atomic_flag_test_and_set, atomic_flag_test_and_set_explicit, and atomic_flag_clear; your version of stdatomic.h doesn't include declarations for these functions (which is required by C11 7.1.4p1). -Eli _______________________________________________ cfe-commits mailing list [email protected] http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits
