[PATCH] D132405: [clang][deps] Split translation units into individual -cc1 or other commands

Ben Langmuir via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Wed Aug 24 15:47:32 PDT 2022


benlangmuir added a comment.

In D132405#3747232 <https://reviews.llvm.org/D132405#3747232>, @jansvoboda11 wrote:

> I'd like to see this split into multiple patches. I can see some formatting changes, removal of `CompilerInvocation` from `ModuleDeps`, isolated changes to `Tooling`, etc. That'd make it much easier to review.

`ToolInvocation` change: https://reviews.llvm.org/D132615
Remove `CompilerInvocation` from `ModuleDeps`: https://reviews.llvm.org/D132616
Factoring out the `addModule*Files` functions, sink `DirectPrebuiltModularDeps`, etc: https://reviews.llvm.org/D132617


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D132405/new/

https://reviews.llvm.org/D132405



More information about the cfe-commits mailing list