[libcxx-commits] [PATCH] D58201: Make std::memory_order an enum class (P0439R0)
Louis Dionne via Phabricator via libcxx-commits
libcxx-commits at lists.llvm.org
Tue Mar 5 06:50:54 PST 2019
ldionne added inline comments.
Comment at: include/atomic:613
+ unsigned>::value, "Underlying type differs from unsigned unexpectedly");
> `typename` is not needed here, this is not a dependent context.
Also, you need to wrap the `is_same<underlying_type<...>::type, unsigned>::value` into parenthesis because this will otherwise trip up our `static_assert` emulation. This could have been noticed by running the test suite in c++03 mode with `./build/bin/llvm-lit -sv libcxx/test/std/atomics --param=std=c++03` (assuming a standard CMake setup).
CHANGES SINCE LAST ACTION
More information about the libcxx-commits