[libcxx-commits] [PATCH] D150300: [libc++][WIP] Try disabling transitive includes in all configurations to see impact on CI times

Louis Dionne via Phabricator via libcxx-commits libcxx-commits at lists.llvm.org
Mon May 15 10:03:38 PDT 2023


ldionne added a comment.

Across our test suite, this does seem to provide a pretty significant compilation speed up.

Without transitive includes (this change)
F27456745: Screenshot 2023-05-15 at 10.01.37 AM.png <https://reviews.llvm.org/F27456745>

With transitive includes (status quo)
F27456756: Screenshot 2023-05-15 at 10.02.11 AM.png <https://reviews.llvm.org/F27456756>

Overall, this seems to provide roughly a 20% improvement in the speed of the CI. It is possible that this would also provide a similar speed up in the compilation of code bases using C++ heavily assuming they include what they use fairly consistently.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D150300



More information about the libcxx-commits mailing list