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

Eric Fiselier via Phabricator via libcxx-commits libcxx-commits at lists.llvm.org
Wed May 24 08:36:52 PDT 2023


EricWF added a comment.

Great tests.



================
Comment at: libcxx/include/__mdspan/layouts.h:34
+// Will be implemented with follow on revision
+#  if 0
+// Layout policy with a mapping which corresponds to FORTRAN-style array layouts
----------------
I don't like adding commented or ifdefed out code like this.

Maybe just leave a comment declaration.


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

https://reviews.llvm.org/D151267



More information about the libcxx-commits mailing list