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



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to