Bigcheese accepted this revision. Bigcheese added a comment. This revision is now accepted and ready to land.
LGTM with style nit. I like that this decouples `DependencyScanningTool` from printing the results. ================ Comment at: clang/tools/clang-scan-deps/ClangScanDeps.cpp:105-107 + llvm::Expected<std::string> &MaybeFile, + SharedStream &OS, + SharedStream &Errs) { ---------------- This looks like it needs to be clang-formatted. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D69186/new/ https://reviews.llvm.org/D69186 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits