[libcxx-commits] [PATCH] D150476: [libc++] Run headers_declare_verbose_abort.sh.cpp in parallel

Nikolas Klauser via Phabricator via libcxx-commits libcxx-commits at lists.llvm.org
Mon May 15 11:43:27 PDT 2023


philnik added a comment.

In D150476#4343308 <https://reviews.llvm.org/D150476#4343308>, @Mordante wrote:

> Please let's not do this. I mentioned that this was troublesome in https://reviews.llvm.org/D144640#inline-1408386. I really would appreciate to have that test fixed too.
>
> I use lit to do determine how many jobs to start and now these tests start to spam extra jobs. I'm not against fixing this issue, but basically deciding for users that they can spawn 16 extra parallel jobs really sounds wrong to me. This has lead me a few times to an unresponsive system since I had other heavy tasks running in parallel.

I don't plan to land this as-is. This was only to test how much the modules build can be improved, and I published it to remember to talk with Louis about this.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D150476



More information about the libcxx-commits mailing list