Skip to content

Commit 22409d7

Browse files
authored
Merge pull request #33223 from artemcm/ImplicitActions
Do not attempt to parse the PlaceholderSwiftModule map when not present
2 parents 53d3dda + f894131 commit 22409d7

File tree

2 files changed

+4
-2
lines changed

2 files changed

+4
-2
lines changed

lib/Serialization/ModuleDependencyScanner.cpp

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -131,7 +131,9 @@ class PlaceholderSwiftModuleScanner : public ModuleDependencyScanner {
131131

132132
// FIXME: Find a better place for this map to live, to avoid
133133
// doing the parsing on every module.
134-
parsePlaceholderModuleMap(PlaceholderDependencyModuleMap);
134+
if (!PlaceholderDependencyModuleMap.empty()) {
135+
parsePlaceholderModuleMap(PlaceholderDependencyModuleMap);
136+
}
135137
}
136138

137139
std::error_code findModuleFilesInDirectory(

test/ScanDependencies/module_deps.swift

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -67,7 +67,7 @@ import SubE
6767
// CHECK-NEXT: "-target",
6868
// CHECK-NEXT: "-Xcc",
6969
// CHECK: "-fapinotes-swift-version=4"
70-
70+
// CHECK-NOT: "error: cannot open Swift placeholder dependency module map from"
7171
// CHECK: "bridgingHeader":
7272
// CHECK-NEXT: "path":
7373
// CHECK-SAME: Bridging.h

0 commit comments

Comments
 (0)