[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