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

Jan Svoboda via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Fri May 14 02:00:58 PDT 2021


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 original translation unit if we modify its CompilerInvocation.

Stop assuming the TU's context hash everywhere.


Repository:
  rG LLVM Github Monorepo

https://reviews.llvm.org/D102473

Files:
  clang/include/clang/Tooling/DependencyScanning/ModuleDepCollector.h
  clang/lib/Tooling/DependencyScanning/DependencyScanningTool.cpp
  clang/lib/Tooling/DependencyScanning/ModuleDepCollector.cpp
  clang/tools/clang-scan-deps/ClangScanDeps.cpp

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D102473.345376.patch
Type: text/x-patch
Size: 8279 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20210514/53ea660b/attachment.bin>


More information about the cfe-commits mailing list