[PATCH] D102473: [clang][deps] 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/lib/Tooling/DependencyScanning/ModuleDepCollector.cpp:147 const Module *TopLevelModule = Imported->getTopLevelModule(); - MDC.ModularDeps[MDC.ContextHash + TopLevelModule->getFullModuleName()] - .ImportedByMainFile =

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

2021-05-14 Thread Jan Svoboda via Phabricator via cfe-commits
jansvoboda11 updated this revision to Diff 345399. jansvoboda11 added a comment. Shuffle lines around to minimize diff in the next patch Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D102473/new/ https://reviews.llvm.org/D102473 Files: clang/incl

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

2021-05-14 Thread Jan Svoboda via Phabricator via cfe-commits
jansvoboda11 updated this revision to Diff 345383. jansvoboda11 added a comment. Use `std::unordered_map` again Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D102473/new/ https://reviews.llvm.org/D102473 Files: clang/include/clang/Tooling/Depende

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

2021-05-14 Thread Jan Svoboda via Phabricator via cfe-commits
jansvoboda11 created this revision. jansvoboda11 added reviewers: Bigcheese, dexonsmith. jansvoboda11 requested review of this revision. Herald added a project: clang. Herald added a subscriber: cfe-commits. The context hash of modular dependencies can be different from the context hash of the or