[clang] [clang][deps] Propagate the entire service (PR #128959)

[Ben Langmuir via cfe-commits]

================
@@ -130,11 +130,11 @@ class DependencyScanningWorker {
                                   DependencyActionController &Controller,
                                   StringRef ModuleName);
 
-  bool shouldEagerLoadModules() const { return EagerLoadModules; }
-
   llvm::vfs::FileSystem &getVFS() const { return *BaseFS; }
 
 private:
+  /// The parent dependency scanning service.
+  DependencyScanningService &Service;
----------------
benlangmuir wrote:

I'm guessing no one is likely to break this rule in practice, but I would add a 
note to the `DependencyScanningWorker` and `DependencyScanningTool` 
constructors that the service must outlive the worker.

https://github.com/llvm/llvm-project/pull/128959
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits