[libcxx-commits] [libcxx] [libcxx] Add span includes to some mdspan tests (PR #142693)
Nikolas Klauser via libcxx-commits
libcxx-commits at lists.llvm.org
Wed Jun 4 10:04:19 PDT 2025
philnik777 wrote:
> > I think the proper fix here is to #include <__fwd/span.h> in . AFAICT this also requires an LWG issue, since dynamic_extent is almost certainly supposed to be provided via , but I think it's not specified to be.
>
> Is there something we can do in the short term to get this working? [5e19fd1](https://github.com/llvm/llvm-project/commit/5e19fd172063c8957a35c7fa3596620f79ebba97) did essentially what this patch is doing, but I'm pretty unfamiliar with how libc++ does development around WG21 decisions.
We can implement this without requiring the LWG issue to be resolved. We're just providing more symbols than strictly required by the standard, which we're already allowed to do. I'm just asking for the LWG issue to make sure the standard gets fixed. See https://cplusplus.github.io/LWG/lwg-active.html in case you haven't filed one before. Or if you don't want to file one for whatever reason I can do that too.
https://github.com/llvm/llvm-project/pull/142693
More information about the libcxx-commits
mailing list