| commit | ce50e716c5895e01795691d5c10c3ac736456d30 | [log] [tgz] |
|---|---|---|
| author | Jerome Forissier <jerome@forissier.org> | Sat Dec 28 19:14:36 2019 +0100 |
| committer | Jérôme Forissier <jerome@forissier.org> | Mon Dec 30 15:01:11 2019 +0100 |
| tree | 37146523a10055a224ac54db57edeb28dbc550de | |
| parent | bf7298044beca7a4501ece95c6146b5987cecaa4 [diff] |
ci: shippable: remove CCACHE_UNIFY=true
The ccache unify mode does not increase cache hits in the Shippable
build, contrary to what is claimed in commit a2a52e198719 ("ci:
shippable: enable ccache's 'unify' option"). Disable this mode,
because it is slower than the default mode [1] and might cause
corrupted builds [2].
Unify mode was enabled in the above commit, but that commit also set
CFG_DEBUG_INFO=n. It was in fact this configuration flag that allowed
better caching, by preventing -g3 from being added to the C flags.
With -g3, the preprocessor output contains all the macro definitions
(such as the configuration variables from config.h). Without it (or
with -g), those defines are not visible after the preprocessing stage
and therefore adding defines that do not change the actual code has no
impact and cached objects can be re-used.
$ echo '#define FOO 1' | gcc -x c -E - | grep '#define FOO 1'
$ echo '#define FOO 1' | gcc -x c -g3 -E - | grep '#define FOO 1'
#define FOO 1
[1] ccache(1) man page
[2] .incbin directive bug: https://github.com/ccache/ccache/issues/497
Signed-off-by: Jerome Forissier <jerome@forissier.org>
Acked-by: Jens Wiklander <jens.wiklander@linaro.org>
This git contains source code for the secure side implementation of OP-TEE project.
All official OP-TEE documentation has moved to http://optee.readthedocs.io.
// OP-TEE core maintainers