Skip to content

Commit

Permalink
Merge pull request #69749 from benlangmuir/update
Browse files Browse the repository at this point in the history
Update for clang maybeAddDependency API change
  • Loading branch information
benlangmuir authored Nov 9, 2023
2 parents 2d75cf5 + 383d52f commit 1e350f5
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 4 deletions.
1 change: 0 additions & 1 deletion lib/AST/ModuleLoader.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -49,7 +49,6 @@ DependencyTracker::addDependency(StringRef File, bool IsSystem) {
// dimension, which we accept and pass along to the clang DependencyCollector.
clangCollector->maybeAddDependency(File, /*FromModule=*/false,
IsSystem, /*IsModuleFile=*/false,
/* FileManager=*/ nullptr,
/*IsMissing=*/false);
}

Expand Down
5 changes: 2 additions & 3 deletions lib/ClangImporter/ClangImporter.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -377,12 +377,11 @@ class ClangImporterDependencyCollector : public clang::DependencyCollector
}

void maybeAddDependency(StringRef Filename, bool FromModule, bool IsSystem,
bool IsModuleFile, clang::FileManager *fileManager,
bool IsMissing) override {
bool IsModuleFile, bool IsMissing) override {
if (FileCollector)
FileCollector->addFile(Filename);
clang::DependencyCollector::maybeAddDependency(
Filename, FromModule, IsSystem, IsModuleFile, fileManager, IsMissing);
Filename, FromModule, IsSystem, IsModuleFile, IsMissing);
}
};
} // end anonymous namespace
Expand Down

0 comments on commit 1e350f5

Please sign in to comment.