llvmbot wrote:
<!--LLVM PR SUMMARY COMMENT--> @llvm/pr-subscribers-clang Author: Naveen Seth Hanig (naveen-seth) <details> <summary>Changes</summary> The OutputPaths field of FullDependencyConsumer is not used, and the resulting TranslationUnitDeps has no corresponding field. This change removes the unused member. It was added in commit f978ea4, and this comment in the Differential Revision suggests it was intended to be removed before landing: https://reviews.llvm.org/D70268#<!-- -->1772032 --- Full diff: https://github.com/llvm/llvm-project/pull/155523.diff 1 Files Affected: - (modified) clang/include/clang/Tooling/DependencyScanning/DependencyScanningTool.h (-1) ``````````diff diff --git a/clang/include/clang/Tooling/DependencyScanning/DependencyScanningTool.h b/clang/include/clang/Tooling/DependencyScanning/DependencyScanningTool.h index c3601a4e73e1f..f222ded8a966a 100644 --- a/clang/include/clang/Tooling/DependencyScanning/DependencyScanningTool.h +++ b/clang/include/clang/Tooling/DependencyScanning/DependencyScanningTool.h @@ -220,7 +220,6 @@ class FullDependencyConsumer : public DependencyConsumer { std::vector<std::string> VisibleModules; std::vector<Command> Commands; std::string ContextHash; - std::vector<std::string> OutputPaths; const llvm::DenseSet<ModuleID> &AlreadySeen; }; `````````` </details> https://github.com/llvm/llvm-project/pull/155523 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
