[libcxx-commits] [PATCH] D93461: [libc++] Implements concept default_initializable.

[Mark de Wever via Phabricator via libcxx-commits]

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


================
Comment at: libcxx/test/std/concepts/concept.default.init/default_initializable.verify.cpp:33
+    static_assert( brace_initializable<S0>);
+    static_assert( std::__default_initializable<S0>);
+    static_assert( std::default_initializable<S0>);
----------------
ldionne wrote:
> These implementation details of libc++ shouldn't be tested here, or if they are, then we should use `LIBCPP_STATIC_ASSERT` (which expands to nothing when testing a non-libc++ library).
I'll use the `LIBCPP_STATIC_ASSERT` since during debugging the code it was good to see whether the expected test failed.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D93461