[libcxx-commits] [PATCH] D150908: [libc++][ci] Use ccache in the jobs that build Clang

[Nikolas Klauser via Phabricator via libcxx-commits]

philnik accepted this revision.
philnik added a comment.

In D150908#4356524 <https://reviews.llvm.org/D150908#4356524>, @Mordante wrote:

> When the cache is preserved between runs we might want to do the same for libc++.

I don't think that makes a lot of sense. The libc++ build takes only a few seconds and (unfortunately) relies on large parts of our headers, which change with pretty much every patch, making it probably not very profitable. Also, while it's not a big problem, it does add another point of failure to the build. It's most likely worth the small risk for clang builds, but I't not convinced it is for libc++ builds.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D150908