[libcxx-commits] [libcxx] [libc++][test] Avoid some transitive includes in standard headers (PR #145846)
Nikolas Klauser via libcxx-commits
libcxx-commits at lists.llvm.org
Wed Jun 25 23:59:22 PDT 2025
philnik777 wrote:
> > I'm pretty sure `errc` should be provided via `<charconv>`.
>
> Yes. But it seems that relying on transitive inclusion is sometimes problematic, as least in CI.
>
> See
>
> * https://github.com/llvm/llvm-project/actions/runs/15888063616/job/44819278583?pr=144249
That is a problem with the implementation, not the tests. We shouldn't "fix" that by changing our tests. To me this looks like our modulemap is missing an export.
https://github.com/llvm/llvm-project/pull/145846
More information about the libcxx-commits
mailing list