[PATCH] D133229: [driver] Prune module-map related flags, if they are not going to be needed

Argyrios Kyrtzidis via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Fri Sep 2 17:27:44 PDT 2022


akyrtzi added a comment.

In D133229#3768101 <https://reviews.llvm.org/D133229#3768101>, @rsmith wrote:

> I think the approach you're taking here is probably doomed -- too many things in Clang depend on whether we've read module map files, and it seems unlikely to me that you'll be able to catch all of them from the driver.

I see, the driver approach does seem like a non-starter, thank you for the feedback! This needs reevaluation.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D133229



More information about the cfe-commits mailing list