[libcxx-commits] [libcxx] [libc++][modules] Adds module testing. (PR #76246)
Louis Dionne via libcxx-commits
libcxx-commits at lists.llvm.org
Mon Jan 22 07:17:33 PST 2024
ldionne wrote:
> Thanks for reapplying the patch. It sounds very scary when libc++ finds headers of the Windows SDK. This means our test suite does not test libc++ but libc++ and the different implementation. @ldionne is this something libc++ officially supports or should support? I don't expect we test this in our pre-commit CI.
No, I don't think that's a configuration we intend to support. I'm surprised that this is how we run the tests on Windows. @mstorsjo would have more context though, this might be necessary for us to find some of the Windows SDK headers.
https://github.com/llvm/llvm-project/pull/76246
More information about the libcxx-commits
mailing list