[PATCH] D108647: [clang][deps] Use top-level modules as precompiled dependencies

2021-08-25 Thread Jan Svoboda via Phabricator via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rG3b8f536fec2f: [clang][deps] Use top-level modules as precompiled dependencies (authored by jansvoboda11). Changed prior to commit: https://reviews.llvm.org/D108647?vs=368394&id=368580#toc Repository:

[PATCH] D108647: [clang][deps] Use top-level modules as precompiled dependencies

2021-08-24 Thread Duncan P. N. Exon Smith via Phabricator via cfe-commits
dexonsmith accepted this revision. dexonsmith added a comment. This revision is now accepted and ready to land. This LGTM. > The documentation of Module::PresumedModuleMapFile says this field is > non-empty only when building from preprocessed source. This means there can > still be cases where

[PATCH] D108647: [clang][deps] Use top-level modules as precompiled dependencies

2021-08-24 Thread Jan Svoboda via Phabricator via cfe-commits
jansvoboda11 created this revision. jansvoboda11 added reviewers: Bigcheese, dexonsmith, arphaman. jansvoboda11 requested review of this revision. Herald added a project: clang. Herald added a subscriber: cfe-commits. The `ASTReader` populates `Module::PresumedModuleMapFile` only for top-level mo