diff options
author | Volodymyr Sapsai <vsapsai@apple.com> | 2019-08-28 23:31:32 +0000 |
---|---|---|
committer | Volodymyr Sapsai <vsapsai@apple.com> | 2019-08-28 23:31:32 +0000 |
commit | 39c70ef6ac1d2d280abedca1b64c57dcff22b796 (patch) | |
tree | 9b0c3ca1b3085b1567834f9634e0239b86518881 /test/Modules | |
parent | b5785156315284fcbfecf996f6303b4263309618 (diff) |
[Modules] Fix rebuilding an updated module for each of its consumers.
Marking a module for a rebuild when its signature differs from the
expected one causes redundant module rebuilds for incremental builds.
When a module is updated, its signature changes. But its consumers still
have the old signature and loading them will result in signature
mismatches. It will correctly cause the rebuilds for the consumers but
we don't need to rebuild the common module for each of them as it is
already up to date.
In practice this bug causes longer build times. We are doing more work
than required and only a single process can build a module, so parallel
builds degrade to a single-process mode where extra processes are just
waiting on a file lock.
Fix by not marking a module dependency for a rebuild on signature
mismatch. We'll check if it is up to date when we load it.
rdar://problem/50212358
Reviewers: dexonsmith, bruno, rsmith
Reviewed By: dexonsmith, bruno
Subscribers: jkorous, ributzka, cfe-commits, aprantl
Differential Revision: https://reviews.llvm.org/D66907
git-svn-id: https://llvm.org/svn/llvm-project/cfe/trunk@370274 91177308-0d34-0410-b5e6-96231b3b80d8
Diffstat (limited to 'test/Modules')
5 files changed, 44 insertions, 0 deletions
diff --git a/test/Modules/Inputs/implicit-invalidate-common/A.h b/test/Modules/Inputs/implicit-invalidate-common/A.h new file mode 100644 index 0000000000..7ad7e6bf65 --- /dev/null +++ b/test/Modules/Inputs/implicit-invalidate-common/A.h @@ -0,0 +1,2 @@ +// A +#include "Common.h" diff --git a/test/Modules/Inputs/implicit-invalidate-common/B.h b/test/Modules/Inputs/implicit-invalidate-common/B.h new file mode 100644 index 0000000000..2952dcbe82 --- /dev/null +++ b/test/Modules/Inputs/implicit-invalidate-common/B.h @@ -0,0 +1,2 @@ +// B +#include "Common.h" diff --git a/test/Modules/Inputs/implicit-invalidate-common/Common.h b/test/Modules/Inputs/implicit-invalidate-common/Common.h new file mode 100644 index 0000000000..01419c7ebc --- /dev/null +++ b/test/Modules/Inputs/implicit-invalidate-common/Common.h @@ -0,0 +1 @@ +// Common diff --git a/test/Modules/Inputs/implicit-invalidate-common/module.modulemap b/test/Modules/Inputs/implicit-invalidate-common/module.modulemap new file mode 100644 index 0000000000..97c674a461 --- /dev/null +++ b/test/Modules/Inputs/implicit-invalidate-common/module.modulemap @@ -0,0 +1,3 @@ +module A { header "A.h" } +module B { header "B.h" } +module Common { header "Common.h" } diff --git a/test/Modules/implicit-invalidate-common.c b/test/Modules/implicit-invalidate-common.c new file mode 100644 index 0000000000..d6cf90e12d --- /dev/null +++ b/test/Modules/implicit-invalidate-common.c @@ -0,0 +1,36 @@ +// REQUIRES: shell +// RUN: rm -rf %t +// RUN: mkdir -p %t/implicit-invalidate-common +// RUN: cp -r %S/Inputs/implicit-invalidate-common %t/ +// RUN: echo '#include "A.h"' > %t/A.c +// RUN: echo '#include "B.h"' > %t/B.c + +// Build with an empty module cache. Module 'Common' should be built only once. +// +// RUN: %clang_cc1 -fmodules -fimplicit-module-maps -fmodules-cache-path=%t/Cache \ +// RUN: -fsyntax-only -I %t/implicit-invalidate-common -Rmodule-build \ +// RUN: %t/A.c 2> %t/initial_build.txt +// RUN: %clang_cc1 -fmodules -fimplicit-module-maps -fmodules-cache-path=%t/Cache \ +// RUN: -fsyntax-only -I %t/implicit-invalidate-common -Rmodule-build \ +// RUN: %t/B.c 2>> %t/initial_build.txt +// RUN: FileCheck %s --implicit-check-not "remark:" --input-file %t/initial_build.txt + +// Update module 'Common' and build with the populated module cache. Module +// 'Common' still should be built only once. Note that we are using the same +// flags for A.c and B.c to avoid building Common.pcm at different paths. +// +// RUN: echo ' // ' >> %t/implicit-invalidate-common/Common.h +// RUN: %clang_cc1 -fmodules -fimplicit-module-maps -fmodules-cache-path=%t/Cache \ +// RUN: -fsyntax-only -I %t/implicit-invalidate-common -Rmodule-build \ +// RUN: %t/A.c 2> %t/incremental_build.txt +// RUN: %clang_cc1 -fmodules -fimplicit-module-maps -fmodules-cache-path=%t/Cache \ +// RUN: -fsyntax-only -I %t/implicit-invalidate-common -Rmodule-build \ +// RUN: %t/B.c 2>> %t/incremental_build.txt +// RUN: FileCheck %s --implicit-check-not "remark:" --input-file %t/incremental_build.txt + +// CHECK: remark: building module 'A' +// CHECK: remark: building module 'Common' +// CHECK: remark: finished building module 'Common' +// CHECK: remark: finished building module 'A' +// CHECK: remark: building module 'B' +// CHECK: remark: finished building module 'B' |