[libcxx-commits] [PATCH] D157171: [libc++] mdspan - implement layout_stride

[Christian Trott via Phabricator via libcxx-commits]

crtrott marked 3 inline comments as done.
crtrott added inline comments.


================
Comment at: libcxx/test/libcxx/containers/views/mdspan/layout_stride/assert.index_operator.pass.cpp:11
+// UNSUPPORTED: c++03, c++11, c++14, c++17, c++20
+// UNSUPPORTED: !libcpp-hardening-mode=debug
+// XFAIL: availability-verbose_abort-missing
----------------
ldionne wrote:
> Shouldn't these assertions be enabled in the hardened mode too?
I think we agreed to do it only in debug mode, because for the vast majority of use cases the assertion will be caught inside mdspan already, since most people would use layouts only through mdspan (and later mdarray).


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

https://reviews.llvm.org/D157171