[libcxx-commits] [PATCH] D151267: mdspan: implement layout_right

[Mark de Wever via Phabricator via libcxx-commits]

Mordante added a comment.

I didn't look closely at the patch, but I noticed an omission.



================
Comment at: libcxx/include/__fwd/mdspan.h:43
+// Layout policy with a mapping which corresponds to C-style array layouts
+struct layout_right {
+  template <class Extents>
----------------
FYI I expect the CI to fail in C++ Module std. This should be fixed by uncommenting `using std::layout_right` in `libcxx/modules/std/mdspan.cppm`. Module support has been added recently.


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

https://reviews.llvm.org/D151267