[PATCH] D108979: [clang][deps] NFC: Stop going through ClangTool

[Duncan P. N. Exon Smith via Phabricator via cfe-commits]

dexonsmith added a comment.

Thanks for explaining why ToolInvocation fits well here. SGTM.

In D108979#2979484 <https://reviews.llvm.org/D108979#2979484>, @jansvoboda11 wrote:

> I think we can move that conversion into the current constructor that accepts `std::vector<std::string>`, add another constructor that takes `ArrayRef<const char *>` and use that from the dependency scanner.

Yeah, that makes sense. (No reason to block this though... can be done later.)


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D108979