[libcxx-commits] [PATCH] D58966: [libc++] Fix <atomic> failures on GCC

[Louis Dionne via Phabricator via libcxx-commits]

ldionne marked an inline comment as done.
ldionne added inline comments.


================
Comment at: libcxx/trunk/include/atomic:1458
       _LIBCPP_CHECK_STORE_MEMORY_ORDER(__m)
-        {__cxx_atomic_store(&__a_, __d, static_cast<__memory_order_underlying_t>(__m));}
+        {__cxx_atomic_store(&__a_, __d, __m);}
     _LIBCPP_INLINE_VISIBILITY
----------------
zoecarver wrote:
> ldionne wrote:
> > zoecarver wrote:
> > > Some reason these arent `static_cast`ed?
> > Yes -- all the `__cxx_atomic_foo` functions take a `memory_order` argument, not an `int`.
> Oh, my bad. How did this pass the tests then..?
The mismatch was only under the GCC implementation.


Repository:
  rL LLVM

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D58966/new/

https://reviews.llvm.org/D58966