[PATCH] D23125: Modules: add command line option to support loading prebuilt modules on demand, without parsing any module map

Manman Ren via cfe-commits cfe-commits at lists.llvm.org
Tue Aug 9 16:18:26 PDT 2016


manmanren added a comment.



In https://reviews.llvm.org/D23125#504942, @benlangmuir wrote:

> How about -fmodules-use-prebuilt-**module-cache** for the flag name?  Saying "prebuilt-modules" is confusing to me, since -fmodule-file can also be used to load a prebuilt module, but doesn't use a cache.


Oops, forgot to address your other comments.
Yes, I will change the flag name as you suggested and use if (IsPrebuilt).

Cheers,
Manman


https://reviews.llvm.org/D23125





More information about the cfe-commits mailing list