[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