Skip to content

Commit d946df7

Browse files
committed
Include Incremental Dependencies in Module Trace
1 parent 67fadb2 commit d946df7

File tree

2 files changed

+12
-5
lines changed

2 files changed

+12
-5
lines changed

lib/Frontend/ModuleInterfaceBuilder.cpp

Lines changed: 7 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -79,10 +79,13 @@ bool ModuleInterfaceBuilder::collectDepsForSerialization(
7979
path::native(ResourcePath);
8080

8181
auto DTDeps = SubInstance.getDependencyTracker()->getDependencies();
82-
SmallVector<StringRef, 16> InitialDepNames(DTDeps.begin(), DTDeps.end());
83-
InitialDepNames.push_back(interfacePath);
84-
InitialDepNames.insert(InitialDepNames.end(),
85-
extraDependencies.begin(), extraDependencies.end());
82+
SmallVector<std::string, 16> InitialDepNames(DTDeps.begin(), DTDeps.end());
83+
auto IncDeps = SubInstance.getDependencyTracker()->getIncrementalDependencyPaths();
84+
InitialDepNames.append(IncDeps.begin(), IncDeps.end());
85+
InitialDepNames.push_back(interfacePath.str());
86+
for (const auto &extra : extraDependencies) {
87+
InitialDepNames.push_back(extra.str());
88+
}
8689
SmallString<128> Scratch;
8790

8891
for (const auto &InitialDepName : InitialDepNames) {

lib/FrontendTool/LoadedModuleTrace.cpp

Lines changed: 5 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -577,7 +577,11 @@ static void computeSwiftModuleTraceInfo(
577577
path::replace_extension(modPath, swiftInterfaceExt);
578578
};
579579

580-
for (auto &depPath : depTracker.getDependencies()) {
580+
auto deps = depTracker.getDependencies();
581+
SmallVector<std::string, 16> dependencies{deps.begin(), deps.end()};
582+
auto incrDeps = depTracker.getIncrementalDependencyPaths();
583+
dependencies.append(incrDeps.begin(), incrDeps.end());
584+
for (const auto &depPath : dependencies) {
581585

582586
// Decide if this is a swiftmodule based on the extension of the raw
583587
// dependency path, as the true file may have a different one.

0 commit comments

Comments
 (0)