[libcxx-commits] [PATCH] D125331: [libc++] Make sure that all headers can be included with modules enabled

Louis Dionne via Phabricator via libcxx-commits libcxx-commits at lists.llvm.org
Tue Oct 25 10:40:32 PDT 2022


ldionne added a comment.

In D125331#3883141 <https://reviews.llvm.org/D125331#3883141>, @tra wrote:

> Would it be possible to split `modules_include.sh.cpp` and possibly other tests into multiple files, so lit could run them in parallel?
>
> Running all of these test cases sequentially often results in this test being the lonely remaining test that continues to run long after all other tests in `ninja check` are done. Splitting it would noticeably reduce `ninja check` wall time for many users.

This is currently impossible because we generate the test and want to keep it in a single file. However, it would be possible to implement a test format where Lit will actually create one Lit test per run line and then those could run in parallel. So -- not impossible, but requires some work in Lit.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D125331/new/

https://reviews.llvm.org/D125331



More information about the libcxx-commits mailing list