From 3dc6ca12ad631a8826fed1558d877982ce3a0bb3 Mon Sep 17 00:00:00 2001 From: George Burgess IV Date: Mon, 24 Jan 2022 00:08:35 -0800 Subject: compiler_wrapper: disable ccache during src_configure This seems to speed things up in cmake by 10% or so, which is consistent with expectations (using `ccache` adds 15-30ms in experimental testing; cmake checks are often faster than that). BUG=b:215747936 TEST=CQ Change-Id: I1a12e50277b37af7bb0b6a58fea2de10006983c8 Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/third_party/toolchain-utils/+/3411542 Reviewed-by: Manoj Gupta Commit-Queue: George Burgess Tested-by: George Burgess --- compiler_wrapper/ccache_flag.go | 7 +++++++ 1 file changed, 7 insertions(+) (limited to 'compiler_wrapper/ccache_flag.go') diff --git a/compiler_wrapper/ccache_flag.go b/compiler_wrapper/ccache_flag.go index 265b8fc2..02fb43ac 100644 --- a/compiler_wrapper/ccache_flag.go +++ b/compiler_wrapper/ccache_flag.go @@ -19,6 +19,13 @@ func processCCacheFlag(builder *commandBuilder) { return arg.value }) + // Disable ccache during portage's src_configure phase. Using ccache here is generally a + // waste of time, since these files are very small. Experimentally, this speeds up + // configuring by ~13%. + if val, present := builder.env.getenv("EBUILD_PHASE"); present && val == "configure" { + useCCache = false + } + if builder.cfg.useCCache && useCCache { // Note: we used to also set CCACHE_BASEDIR but don't do it // anymore for reasons outlined in crrev.com/c/2103170. -- cgit v1.2.3