[clang] [clang][deps] Use `ModuleDepCollector` for Make output (PR #182063)

[Jan Svoboda via cfe-commits]

================
@@ -40,7 +40,11 @@ class MakeDependencyPrinterConsumer : public DependencyConsumer {
   // set of deps, and handleFileDependency handles enough for implicitly
   // built modules to work.
   void handlePrebuiltModuleDependency(PrebuiltModuleDep PMD) override {}
-  void handleModuleDependency(ModuleDeps MD) override {}
+  void handleModuleDependency(ModuleDeps MD) override {
+    MD.forEachFileDep([this](StringRef File) {
+      ModuleDependencies.push_back(std::string(File));
+    });
+  }
----------------
jansvoboda11 wrote:

This was added to propagate keep the modules' file dependencies into the Make output. These would normally be picked up due to `compileModuleImpl()` passing the dependency collector along to module compiles, or by hooking into the `ASTReader`.

https://github.com/llvm/llvm-project/pull/182063