[PATCH] D102473: [clang][deps] NFC: Stop assuming the TU's context hash

2021-05-17 Thread Jan Svoboda via Phabricator via cfe-commits
This revision was landed with ongoing or failed builds. This revision was automatically updated to reflect the committed changes. Closed by commit rGb9d5b0c20180: [clang][deps] NFC: Stop assuming the TU's context hash (authored by jansvoboda11). Repository: rG LLVM Github Monorepo CHANGES SINC

[PATCH] D102473: [clang][deps] NFC: Stop assuming the TU's context hash

2021-05-14 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. LGTM. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D102473/new/ https://reviews.llvm.org/D102473 __

[PATCH] D102473: [clang][deps] NFC: Stop assuming the TU's context hash

2021-05-14 Thread Jan Svoboda via Phabricator via cfe-commits
jansvoboda11 added inline comments. Comment at: clang/include/clang/Tooling/DependencyScanning/ModuleDepCollector.h:182 /// Direct and transitive modular dependencies of the main source file. - std::unordered_map ModularDeps; + std::unordered_map ModularDeps; /// Options