Skip to content

Commit 8090fcf

Browse files
authored
Merge pull request #72499 from artemcm/NoStatOnSDKDepsSerialization
[Explicit Module Build] Do not serialize binary .swiftmodule dependencies when building explicit interface dependencies
2 parents d0825ad + a38071e commit 8090fcf

File tree

1 file changed

+4
-4
lines changed

1 file changed

+4
-4
lines changed

lib/FrontendTool/FrontendTool.cpp

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -410,12 +410,12 @@ static bool buildModuleFromInterface(CompilerInstance &Instance) {
410410
bool IgnoreAdjacentModules = Instance.hasASTContext() &&
411411
Instance.getASTContext().IgnoreAdjacentModules;
412412

413-
// When caching is enabled, the explicit module build dependencies are
413+
// When building explicit module dependencies, they are
414414
// discovered by dependency scanner and the swiftmodule is already rebuilt
415415
// ignoring candidate module. There is no need to serialized dependencies for
416-
// validation purpose.
417-
bool ShouldSerializeDeps =
418-
!Instance.getInvocation().getCASOptions().EnableCaching;
416+
// validation purpose because the build system (swift-driver) is then
417+
// responsible for checking whether inputs are up-to-date.
418+
bool ShouldSerializeDeps = !FEOpts.ExplicitInterfaceBuild;
419419

420420
// If an explicit interface build was requested, bypass the creation of a new
421421
// sub-instance from the interface which will build it in a separate thread,

0 commit comments

Comments
 (0)