[PATCH] D102473: [clang][deps] NFC: Stop assuming the TU's context hash
Jan Svoboda via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Fri May 14 05:40:47 PDT 2021
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<std::string, ModuleDeps> ModularDeps;
+ std::unordered_map<const Module *, ModuleDeps> ModularDeps;
/// Options that control the dependency output generation.
----------------
Computing the context hash might be expensive now, so use `const Module *` instead.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D102473/new/
https://reviews.llvm.org/D102473
More information about the cfe-commits
mailing list