[libcxx-commits] [PATCH] D71994: SFINAE span default constructor on Extent == 0

Casey Carter via Phabricator via libcxx-commits libcxx-commits at lists.llvm.org
Tue Jan 7 09:18:10 PST 2020


CaseyCarter requested changes to this revision.
CaseyCarter added inline comments.
This revision now requires changes to proceed.


================
Comment at: libcxx/test/std/containers/views/span.cons/default.pass.cpp:17
+
+#ifdef __cpp_lib_concepts
+#include <concepts>
----------------
Late comment: `<span>` isn't guaranteed to export `__cpp_lib_concepts`; this should include `<version>` before checking `__cpp_lib_concepts`.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D71994





More information about the libcxx-commits mailing list